W dniu 3 stycznia 2012 21:52 użytkownik Greg KH <greg@xxxxxxxxx> napisał: > On Mon, Dec 26, 2011 at 12:36:59AM +0100, Rafał Miłecki wrote: >> W dniu 16 grudnia 2011 07:40 użytkownik Rafał Miłecki >> <zajec5@xxxxxxxxx> napisał: >> > W dniu 15 grudnia 2011 22:38 użytkownik John W. Linville >> > <linville@xxxxxxxxxxxxx> napisał: >> >> On Thu, Dec 15, 2011 at 09:04:29PM +0100, Rafał Miłecki wrote: >> >>> 2011/12/15 John W. Linville <linville@xxxxxxxxxxxxx>: >> >>> > commit 42a3b63bb2ca4996a3d1210a004eae2333f1119e >> >>> > >> >>> > Dave, >> >>> > >> >>> > Here are a few more fixes intended for the 3.2 release. They are >> >>> > all small and narrowly focused. >> >>> >> >>> John, I've made a mistake and didn't use [PATCH 3.2] header to make it >> >>> clean my patch is fix. Could you take a look at >> >>> [PATCH] bcma: support for suspend and resume >> >>> please? >> >>> >> >>> It's not one-liner, but fixes lock ups, which I believe - we really >> >>> want to avoid. >> >> >> >> It's late in the release cycle, and Dave specifically asked me to slow down. >> >> >> >> Are these suspend/resume lockups a regression? Or have they always >> >> been there? Do they happen to everyone? >> > >> > The bug is present since first days of bcma. That's why I even decided >> > to add stable to CC. >> > >> > Personally I've tested that only on 1 machine (I don't have more >> > suspendable machines with mini PCIe slot). However all Macbook 8.1/8.2 >> > users have to remove b43 & bcma before suspending [0], they complain >> > about that since ever. >> > >> > Arend: I know you're also complaining for suspend in bcma. Can you >> > comment on this? >> > >> > [0] http://ubuntuforums.org/showthread.php?t=1695746 >> >> User nephyrin on #bcm-users has confirmed this patch fixes >> suspend&resume for him. Without this patch he got deadlock without >> seeing any kernel panic logs - quite an ugly case. >> >> I still think this patch may be worth taking as fix and backporting too. > > What patch specifically? What is the git commit id of it in Linus's > tree? Without that information, telling stable@ about it is > pointless... Sorry Greg, it's about commit 775ab52142b02237a54184238e922251c59a2b5c Author: Rafał Miłecki <zajec5@xxxxxxxxx> Date: Fri Dec 9 22:16:07 2011 +0100 bcma: support for suspend and resume bcma used to lock up machine without enabling PCI or initializing CC. Cc: stable@xxxxxxxxxxxxxxx Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx> Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx> it's located in wireless-testing: git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-testing.git John didn't pass this to Linus yet. -- Rafał -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html