linux-kernel - Re: bcm43xx regression in 2.6.24 (with patch)

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: <f19298770802261338r40e3f252j2249c1325b725614@mail.gmail.com>
Date:	Wed, 27 Feb 2008 00:38:56 +0300
From:	"Alexey Zaytsev" <alexey.zaytsev@...il.com>
To:	"John W. Linville" <linville@...driver.com>
Cc:	"Michael Buesch" <mb@...sch.de>, "Ingo Molnar" <mingo@...e.hu>,
	"Alexey Zaytsev" <zaytsev.a@...tei.ru>, "Greg KH" <greg@...ah.com>,
	linux-kernel@...r.kernel.org
Subject: Re: bcm43xx regression in 2.6.24 (with patch)

On Tue, Feb 26, 2008 at 11:20 PM, John W. Linville
<linville@...driver.com> wrote:
> On Sat, Feb 23, 2008 at 11:20:58PM +0100, Michael Buesch wrote:
>  > On Saturday 23 February 2008 22:32:46 Alexey Zaytsev wrote:
>
>
> > > And you knew that the new driver did no work with the bcm4311
>  > > chips, which is the sad thing.
>  >
>  > That is not true. It doesn't work with exactly _one_ revision
>  > of the bcm4311 card. And that is already fixed in 2.6.25.
>  > I'd like to have that in .24-stable, too, but I guess it's too big.
>  > It changes some parts of the DMA engine code.
>
>  Which patch is it?  Is it this one?  Or is there another one that touches 4311?
>
>  commit 013978b688d2a27af3ab55ca739e8c8ac7254870
>  Author: Larry Finger <Larry.Finger@...inger.net>
>  Date:   Mon Nov 26 10:29:47 2007 -0600
>
>     b43: Changes to enable BCM4311 rev 02 with wireless core revision 13
>
>     The BCM94311MCG rev 02 chip has an 802.11 core with revision 13 and
>     has not been supported until now. The changes include the following:
>
>  <snip>
>
>  If so, perhaps we should just consider applying this to -stable?
>

I see a few problems here:

This commit seems to depend on some commits prior to it, as it touches
drivers/net/wireless/b43/wa.c which is absent in 2.6.24.3. When I get home,
I'll test if it still works if you ignore wa.c

The patch may be too complex for -stable. I think there was a hard line
count limit, like no more than 100 lines. I'm not sure if this still stands.

This  does not fix the bcm43xx driver.

>  I really do wish everyone would tone down their rhetoric on this
>  issue...
>
>  John
>  --
>  John W. Linville
>  linville@...driver.com
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/