From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 87682] Horizontal lines in radeon driver on kernel 3.15 and upwards Date: Wed, 04 May 2016 22:13:39 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0851690314==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 1ED4E6E280 for ; Wed, 4 May 2016 22:13:39 +0000 (UTC) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0851690314== Content-Type: multipart/alternative; boundary="14624000191.e8D846e64.17233"; charset="UTF-8" --14624000191.e8D846e64.17233 Date: Wed, 4 May 2016 22:13:39 +0000 MIME-Version: 1.0 Content-Type: text/plain https://bugs.freedesktop.org/show_bug.cgi?id=87682 --- Comment #8 from Chris Bainbridge --- > As far as I understand; 3.15.0-rc2-00077-g76e7745 is a later revision (good) > than 3.15.0-rc2-00069-g1aae31c (bad) This is not correct. The 77/69 does not imply a linear ordering because of forks: $ git merge-base --is-ancestor 3.15.0-rc2-00069-g1aae31c 3.15.0-rc2-00077-g76e7745; echo $? 1 Trust git ;-) > c2fb3094669a3205f16a32f4119d0afe40b1a1fd is the first bad commit Not familiar with this code, but from the patch the PLL values are printed out: DRM_DEBUG_KMS("%d - %d, pll dividers - fb: %d.%d ref: %d, post %d\n", freq, *dot_clock_p * 10, *fb_div_p, *frac_fb_div_p, ref_div, post_div); So suggest enabling debug log and compare those two lines from a working and non-working kernel. It should also be trivial to checkout a recent tag and revert the bad commit (there is a conflict but just delete the avivo_get_fb_ref_div function to resolve it). -- You are receiving this mail because: You are the assignee for the bug. --14624000191.e8D846e64.17233 Date: Wed, 4 May 2016 22:13:39 +0000 MIME-Version: 1.0 Content-Type: text/html

Comment # 8 on bug 87682 from
> As far as I understand; 3.15.0-rc2-00077-g76e7745 is a later revision (good)
> than 3.15.0-rc2-00069-g1aae31c (bad)

This is not correct. The 77/69 does not imply a linear ordering because of
forks:

$ git merge-base --is-ancestor 3.15.0-rc2-00069-g1aae31c
3.15.0-rc2-00077-g76e7745; echo $?
1

Trust git ;-)

> c2fb3094669a3205f16a32f4119d0afe40b1a1fd is the first bad commit

Not familiar with this code, but from the patch the PLL values are printed out:

         DRM_DEBUG_KMS("%d - %d, pll dividers - fb: %d.%d ref: %d, post %d\n",
                      freq, *dot_clock_p * 10, *fb_div_p, *frac_fb_div_p,
                      ref_div, post_div);

So suggest enabling debug log and compare those two lines from a working and
non-working kernel.

It should also be trivial to checkout a recent tag and revert the bad commit
(there is a conflict but just delete the avivo_get_fb_ref_div function to
resolve it).


You are receiving this mail because:
  • You are the assignee for the bug.
--14624000191.e8D846e64.17233-- --===============0851690314== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============0851690314==--