[Bug 16127] Boot freeze on HP Compaq nx6325 (RS482) with Radeon KMS
https://bugzilla.kernel.org/show_bug.cgi?id=16127 Rafael J. Wysocki r...@sisk.pl changed: What|Removed |Added Status|NEW |RESOLVED CC||r...@sisk.pl Resolution||PATCH_ALREADY_AVAILABLE --- Comment #7 from Rafael J. Wysocki r...@sisk.pl 2010-06-08 21:59:50 --- Handled-By : Dave Airlie airl...@linux.ie Patch : https://bugzilla.kernel.org/attachment.cgi?id=26677 -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are watching the assignee of the bug. -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
2.6.35-rc2-git2: Reported regressions from 2.6.34
[NOTES: * This by no means is a complete list, but we only put e-mail reports that are at least 1 week old into the Bugzilla. * Quite a few of the already reported regressions may be related to the bug fixed by 386f40c86d6c8d5b717ef20620af1a750d0dacb4 (Revert tty: fix a little bug in scrup, vt.c), so reporters please retest with this commit applied.] This message contains a list of some regressions from 2.6.34, for which there are no fixes in the mainline known to the tracking team. If any of them have been fixed already, please let us know. If you know of any other unresolved regressions from 2.6.34, please let us know either and we'll add them to the list. Also, please let us know if any of the entries below are invalid. Each entry from the list will be sent additionally in an automatic reply to this message with CCs to the people involved in reporting and handling the issue. Listed regressions statistics: Date Total Pending Unresolved 2010-06-09 15 13 10 Unresolved regressions -- Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16163 Subject : [2.6.35-rc1 Regression] i915: Commit cfecde causes VGA to stay off Submitter : David John david...@xenontk.org Date: 2010-06-02 12:52 (7 days old) Message-ID : 4c065423.3000...@xenontk.org References : http://marc.info/?l=linux-kernelm=127548313828613w=2 Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16161 Subject : [2.6.35-rc1 regression] sysfs: cannot create duplicate filename ... XVR-600 related? Submitter : Mikael Pettersson mi...@it.uu.se Date: 2010-06-01 19:57 (8 days old) Message-ID : 19461.26166.427857.612...@pilspetsen.it.uu.se References : http://marc.info/?l=linux-kernelm=127542227511925w=2 Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16160 Subject : 2.6.35 Radeon KMS power management regression? Submitter : Nigel Cunningham ncunning...@crca.org.au Date: 2010-06-01 6:23 (8 days old) Message-ID : 4c04a767.8000...@crca.org.au References : http://marc.info/?l=linux-kernelm=127537343722290w=2 Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16145 Subject : Unable to boot after ACPI: Don't let acpi_pad needlessly mark TSC unstable Submitter : Tom Gundersen t...@jklm.no Date: 2010-06-07 13:11 (2 days old) Handled-By : Venkatesh Pallipadi ve...@google.com Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16129 Subject : BUG: using smp_processor_id() in preemptible [] code: jbd2/sda2 Submitter : Jan Kreuzer kontrolla...@gmx.de Date: 2010-06-05 06:15 (4 days old) Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16122 Subject : 2.6.35-rc1: WARNING at fs/fs-writeback.c:1142 __mark_inode_dirty+0x103/0x170 Submitter : Larry Finger larry.fin...@lwfinger.net Date: 2010-06-04 13:18 (5 days old) Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16120 Subject : Oops: [#1] SMP, unable to handle kernel NULL pointer dereference at (null) Submitter : Alex Zhavnerchik alex.vi...@gmail.com Date: 2010-06-04 09:25 (5 days old) Handled-By : Eric Dumazet eric.duma...@gmail.com Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16104 Subject : Radeon KMS does not start after merge of the new PM-Code Submitter : Jan Kreuzer kontrolla...@gmx.de Date: 2010-06-02 07:47 (7 days old) Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16090 Subject : sysfs: cannot create duplicate filename Submitter : Tobias devn...@plzk.org Date: 2010-06-01 15:59 (8 days old) Handled-By : Jesse Barnes jbar...@virtuousgeek.org Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16037 Subject : NULL Pointer dereference in __ir_input_register/budget_ci_attach Submitter : Sean Finney sean...@debian.org Date: 2010-05-23 19:52 (17 days old) Regressions with patches Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16131 Subject : kernel BUG at fs/btrfs/extent-tree.c:4363 (btrfs_free_tree_block) Submitter : Chow Loong Jin hyper...@ubuntu.com Date: 2010-06-05 18:53 (4 days old) Handled-By : Yan Zheng zheng@oracle.com Patch : https://patchwork.kernel.org/patch/103235/ Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16127 Subject : Boot freeze on HP Compaq nx6325 (RS482) with Radeon KMS Submitter : Jure Repinc jlp.b...@gmail.com Date: 2010-06-04 21:14 (5 days old) Handled-By : Dave Airlie airl...@linux.ie Patch : https://bugzilla.kernel.org/attachment.cgi?id=26677 Bug-Entry :
Re: 2.6.35-rc2-git2: Reported regressions from 2.6.34
[ Added lots of cc's to direct specific people to look at the regressions that may or may not be theirs... ] On Wed, 9 Jun 2010, Rafael J. Wysocki wrote: * Quite a few of the already reported regressions may be related to the bug fixed by 386f40c86d6c8d5b717ef20620af1a750d0dacb4 (Revert tty: fix a little bug in scrup, vt.c), so reporters please retest with this commit applied.] From a quick look, most of them look unrelated to that unfortunate bug. It's hard to tell for sure, of course (memory corruption can do pretty much anything), but I'm not seeing the 07200720.. pattern at least. And some of them do seem to be bisected to likely culprits and/or have patches that are claimed to have fixed them. Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16163 Subject : [2.6.35-rc1 Regression] i915: Commit cfecde causes VGA to stay off Submitter : David John david...@xenontk.org Date : 2010-06-02 12:52 (7 days old) Message-ID: 4c065423.3000...@xenontk.org References: http://marc.info/?l=linux-kernelm=127548313828613w=2 That has a reverting the commit fixes it, and a confirmation from Nick Bowler. Eric, Carl: should I just revert that commit? Or do you have a fix? Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16145 Subject : Unable to boot after ACPI: Don't let acpi_pad needlessly mark TSC unstable Submitter : Tom Gundersen t...@jklm.no Date : 2010-06-07 13:11 (2 days old) Handled-By: Venkatesh Pallipadi ve...@google.com Hmm. This does seem to be a properly bisected commit, but at the same time it looks from the bugzilla like it's just pure luck on that machine that the acpi_pad driver happened to mark TSC unstable - so while the commit bisected is the real one, it's not the deeper reason for the problem. Venki, any updates? Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16129 Subject : BUG: using smp_processor_id() in preemptible [] code: jbd2/sda2 Submitter : Jan Kreuzer kontrolla...@gmx.de Date : 2010-06-05 06:15 (4 days old) This seems to have been introduced by commit 7cbaef9c83e58bbd4bdd534b09052b6c5ec457d5 Author: Ingo Molnar mi...@elte.hu Date: Sat Nov 8 17:05:38 2008 +0100 sched: optimize sched_clock() a bit sched_clock() uses cycles_2_ns() needlessly - which is an irq-disabling variant of __cycles_2_ns(). Most of the time sched_clock() is called with irqs disabled already. The few places that call it with irqs enabled need to be updated. Signed-off-by: Ingo Molnar mi...@elte.hu and this seems to be one of those calling cases that need to be updated. Ingo? The call trace is: BUG: using smp_processor_id() in preemptible [] code: jbd2/sda2-8/337 caller is native_sched_clock+0x3c/0x68 Pid: 337, comm: jbd2/sda2-8 Not tainted 2.6.35-rc1jan+ #4 Call Trace: [812362c5] debug_smp_processor_id+0xc9/0xe4 [8101059d] native_sched_clock+0x3c/0x68 [8101043d] sched_clock+0x9/0xd [81212d7a] blk_rq_init+0x97/0xa3 [81214d71] get_request+0x1c4/0x2d0 [81214ea6] get_request_wait+0x29/0x1a6 [81215537] __make_request+0x338/0x45b [812147c2] generic_make_request+0x2bb/0x330 [81214909] submit_bio+0xd2/0xef [811413cb] submit_bh+0xf4/0x116 [81144853] block_write_full_page_endio+0x89/0x96 [81144875] block_write_full_page+0x15/0x17 [8119b00a] ext4_writepage+0x356/0x36b [810e1f91] __writepage+0x1a/0x39 [810e32a6] write_cache_pages+0x20d/0x346 [810e3406] generic_writepages+0x27/0x29 [811ca279] journal_submit_data_buffers+0x110/0x17d [811ca986] jbd2_journal_commit_transaction+0x4cb/0x156d [811d0cba] kjournald2+0x147/0x37a (from the bugzilla thing) Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16122 Subject : 2.6.35-rc1: WARNING at fs/fs-writeback.c:1142 __mark_inode_dirty+0x103/0x170 Submitter : Larry Finger larry.fin...@lwfinger.net Date : 2010-06-04 13:18 (5 days old) Jens? Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16120 Subject : Oops: [#1] SMP, unable to handle kernel NULL pointer dereference at (null) Submitter : Alex Zhavnerchik alex.vi...@gmail.com Date : 2010-06-04 09:25 (5 days old) Handled-By: Eric Dumazet eric.duma...@gmail.com This one seems to have a patch in bugzilla. Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16104 Subject : Radeon KMS does not start after merge of the new PM-Code Submitter : Jan Kreuzer kontrolla...@gmx.de Date : 2010-06-02 07:47 (7 days old) This one
Re: 2.6.35-rc2-git2: Reported regressions from 2.6.34
Em 08-06-2010 22:53, Linus Torvalds escreveu: Bug-Entry: http://bugzilla.kernel.org/show_bug.cgi?id=16037 Subject : NULL Pointer dereference in __ir_input_register/budget_ci_attach Submitter: Sean Finney sean...@debian.org Date : 2010-05-23 19:52 (17 days old) Perhaps related to commit 13c24497086418010bf4f76378bcae241d7f59cd? David Härdeman, Mauro Carvalho Chehab added to cc. This patch probably solves the issue: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=84b14f181a36eea6591779156ef356f8d198bbfd The patch were already applied upstream. I've already asked the reporter to test it, via BZ. Cheers, Mauro -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
[PATCH] drm/radeon: fix dual-head on rv250
From: Dave Airlie airl...@redhat.com Plugged in FireMV with the rv250 on it, and the second crtc/dac didn't work, we were reading/writing different registers than we were modifying in the code. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 22 +- 1 files changed, 9 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c index 9b5f62b..23ea127 100644 --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c @@ -931,16 +931,14 @@ static void radeon_legacy_tv_dac_mode_set(struct drm_encoder *encoder, if (ASIC_IS_R300(rdev)) { gpiopad_a = RREG32(RADEON_GPIOPAD_A) | 1; disp_output_cntl = RREG32(RADEON_DISP_OUTPUT_CNTL); - } - - if (rdev-family == CHIP_R200 || ASIC_IS_R300(rdev)) - disp_tv_out_cntl = RREG32(RADEON_DISP_TV_OUT_CNTL); - else + } else if (rdev-family != CHIP_R200) disp_hw_debug = RREG32(RADEON_DISP_HW_DEBUG); - - if (rdev-family == CHIP_R200) + else if (rdev-family == CHIP_R200) fp2_gen_cntl = RREG32(RADEON_FP2_GEN_CNTL); + if (rdev-family = CHIP_R200) + disp_tv_out_cntl = RREG32(RADEON_DISP_TV_OUT_CNTL); + if (is_tv) { uint32_t dac_cntl; @@ -1005,15 +1003,13 @@ static void radeon_legacy_tv_dac_mode_set(struct drm_encoder *encoder, if (ASIC_IS_R300(rdev)) { WREG32_P(RADEON_GPIOPAD_A, gpiopad_a, ~1); WREG32(RADEON_DISP_OUTPUT_CNTL, disp_output_cntl); - } + } else if (rdev-family != CHIP_R200) + WREG32(RADEON_DISP_HW_DEBUG, disp_hw_debug); + else if (rdev-family == CHIP_R200) + WREG32(RADEON_FP2_GEN_CNTL, fp2_gen_cntl); if (rdev-family = CHIP_R200) WREG32(RADEON_DISP_TV_OUT_CNTL, disp_tv_out_cntl); - else - WREG32(RADEON_DISP_HW_DEBUG, disp_hw_debug); - - if (rdev-family == CHIP_R200) - WREG32(RADEON_FP2_GEN_CNTL, fp2_gen_cntl); if (is_tv) radeon_legacy_tv_mode_set(encoder, mode, adjusted_mode); -- 1.6.5.2 -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
Re: [PATCH] drm/radeon: fix dual-head on rv250
On Wed, Jun 9, 2010 at 12:02 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com Plugged in FireMV with the rv250 on it, and the second crtc/dac didn't work, we were reading/writing different registers than we were modifying in the code. Signed-off-by: Dave Airlie airl...@redhat.com Signed-off-by: Alex Deucher alexdeuc...@gmail.com --- drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 22 +- 1 files changed, 9 insertions(+), 13 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c index 9b5f62b..23ea127 100644 --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c @@ -931,16 +931,14 @@ static void radeon_legacy_tv_dac_mode_set(struct drm_encoder *encoder, if (ASIC_IS_R300(rdev)) { gpiopad_a = RREG32(RADEON_GPIOPAD_A) | 1; disp_output_cntl = RREG32(RADEON_DISP_OUTPUT_CNTL); - } - - if (rdev-family == CHIP_R200 || ASIC_IS_R300(rdev)) - disp_tv_out_cntl = RREG32(RADEON_DISP_TV_OUT_CNTL); - else + } else if (rdev-family != CHIP_R200) disp_hw_debug = RREG32(RADEON_DISP_HW_DEBUG); - - if (rdev-family == CHIP_R200) + else if (rdev-family == CHIP_R200) fp2_gen_cntl = RREG32(RADEON_FP2_GEN_CNTL); + if (rdev-family = CHIP_R200) + disp_tv_out_cntl = RREG32(RADEON_DISP_TV_OUT_CNTL); + if (is_tv) { uint32_t dac_cntl; @@ -1005,15 +1003,13 @@ static void radeon_legacy_tv_dac_mode_set(struct drm_encoder *encoder, if (ASIC_IS_R300(rdev)) { WREG32_P(RADEON_GPIOPAD_A, gpiopad_a, ~1); WREG32(RADEON_DISP_OUTPUT_CNTL, disp_output_cntl); - } + } else if (rdev-family != CHIP_R200) + WREG32(RADEON_DISP_HW_DEBUG, disp_hw_debug); + else if (rdev-family == CHIP_R200) + WREG32(RADEON_FP2_GEN_CNTL, fp2_gen_cntl); if (rdev-family = CHIP_R200) WREG32(RADEON_DISP_TV_OUT_CNTL, disp_tv_out_cntl); - else - WREG32(RADEON_DISP_HW_DEBUG, disp_hw_debug); - - if (rdev-family == CHIP_R200) - WREG32(RADEON_FP2_GEN_CNTL, fp2_gen_cntl); if (is_tv) radeon_legacy_tv_mode_set(encoder, mode, adjusted_mode); -- 1.6.5.2 -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
Re: 2.6.35-rc2-git2: Reported regressions from 2.6.34
* Linus Torvalds torva...@linux-foundation.org wrote: Bug-Entry : http://bugzilla.kernel.org/show_bug.cgi?id=16129 Subject : BUG: using smp_processor_id() in preemptible [] code: jbd2/sda2 Submitter : Jan Kreuzer kontrolla...@gmx.de Date: 2010-06-05 06:15 (4 days old) This seems to have been introduced by commit 7cbaef9c83e58bbd4bdd534b09052b6c5ec457d5 Author: Ingo Molnar mi...@elte.hu Date: Sat Nov 8 17:05:38 2008 +0100 sched: optimize sched_clock() a bit sched_clock() uses cycles_2_ns() needlessly - which is an irq-disabling variant of __cycles_2_ns(). Most of the time sched_clock() is called with irqs disabled already. The few places that call it with irqs enabled need to be updated. Signed-off-by: Ingo Molnar mi...@elte.hu and this seems to be one of those calling cases that need to be updated. That's a commit from 2008. Ingo? The call trace is: BUG: using smp_processor_id() in preemptible [] code: jbd2/sda2-8/337 caller is native_sched_clock+0x3c/0x68 Pid: 337, comm: jbd2/sda2-8 Not tainted 2.6.35-rc1jan+ #4 Call Trace: [812362c5] debug_smp_processor_id+0xc9/0xe4 [8101059d] native_sched_clock+0x3c/0x68 [8101043d] sched_clock+0x9/0xd [81212d7a] blk_rq_init+0x97/0xa3 [81214d71] get_request+0x1c4/0x2d0 [81214ea6] get_request_wait+0x29/0x1a6 [81215537] __make_request+0x338/0x45b [812147c2] generic_make_request+0x2bb/0x330 [81214909] submit_bio+0xd2/0xef [811413cb] submit_bh+0xf4/0x116 [81144853] block_write_full_page_endio+0x89/0x96 [81144875] block_write_full_page+0x15/0x17 [8119b00a] ext4_writepage+0x356/0x36b [810e1f91] __writepage+0x1a/0x39 [810e32a6] write_cache_pages+0x20d/0x346 [810e3406] generic_writepages+0x27/0x29 [811ca279] journal_submit_data_buffers+0x110/0x17d [811ca986] jbd2_journal_commit_transaction+0x4cb/0x156d [811d0cba] kjournald2+0x147/0x37a (from the bugzilla thing) The warning was introduced by this fresh commit (and a followup commit) merged in the .35 merge window: | commit 9195291e5f05e01d67f9a09c756b8aca8f009089 | Author: Divyesh Shah dps...@google.com | Date: Thu Apr 1 15:01:41 2010 -0700 | | blkio: Increment the blkio cgroup stats for real now IIRC Jens posted a fix for the regression. Jens, what's the status of that? As the code there started using a raw sched_clock() call for block statistics purposes, which was a poorly thought out (and buggy) approach: - it takes timestamps on different cpus and then compares then, but doesnt consider that sched_clock() is not comparable between CPUs without extra care - it doesnt consider the possibility for the sched_clock() result going backwards on certain platforms (such as x86) - it doesnt consider preemptability (There's work ongoing to add a clock variant that can be used for such purposes, but that's .36 fodder.) Thanks, Ingo -- ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo -- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel