netdev - Re: pull request: wireless 2011-12-15

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACna6rwjrg6m3SEBxfYWgRSTN5_4-TuA0tdG6P2pFuUvwvvjZA@mail.gmail.com>
Date:	Tue, 3 Jan 2012 23:06:59 +0100
From:	Rafał Miłecki <zajec5@...il.com>
To:	Greg KH <greg@...ah.com>
Cc:	"John W. Linville" <linville@...driver.com>, davem@...emloft.net,
	linux-wireless@...r.kernel.org, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: pull request: wireless 2011-12-15

W dniu 3 stycznia 2012 21:52 użytkownik Greg KH <greg@...ah.com> 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@...il.com> napisał:
>> > W dniu 15 grudnia 2011 22:38 użytkownik John W. Linville
>> > <linville@...driver.com> napisał:
>> >> On Thu, Dec 15, 2011 at 09:04:29PM +0100, Rafał Miłecki wrote:
>> >>> 2011/12/15 John W. Linville <linville@...driver.com>:
>> >>> > 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@...il.com>
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@...r.kernel.org
    Signed-off-by: Rafał Miłecki <zajec5@...il.com>
    Signed-off-by: John W. Linville <linville@...driver.com>

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 netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html