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  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]
Date:	Thu, 28 Feb 2008 13:30:08 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	gordonfarquharson@...il.com
Cc:	linville@...driver.com, netdev@...r.kernel.org,
	linux-wireless@...r.kernel.org
Subject: Re: pull request: wireless-2.6 2008-02-27

From: "Gordon Farquharson" <gordonfarquharson@...il.com>
Date: Thu, 28 Feb 2008 14:21:21 -0700

> Hi David
> 
> On Thu, Feb 28, 2008 at 12:51 AM, David Miller <davem@...emloft.net> wrote:
> > From: "John W. Linville" <linville@...driver.com>
> >  Date: Wed, 27 Feb 2008 20:41:25 -0500
> >
> >
> >  > Gordon Farquharson (1):
> >  >       Fix b43 driver build for arm
> >
> >  Please revert this change and resend your pull request.
> >
> >  If file2alias doesn't work properly in a cross build environment, we
> >  don't fix it by crapping up mod_devalias.h with these silly align
> >  attribute annotations.
> 
> This was extensively discussed on LKML [1, 2] and the patch has been
> merged into Andrew Morton's -mm tree. It is understood that this is
> not the Right Way to fix to problem, hence the big fat comment in the
> code. It would be great if you could reconsider your decision.

If you put that thing in there now, on ARM the structure size changes
as does the alignment of that final member.

Then we're stuck with it and you can't remove that hack from
mod_devalias.h.  It can't be removed even after the cross build issues
of file2alias are resolved properly without re-breaking ARM.

Feel free to push that via Andrew or someone else, but that turd isn't
going in by my tree because to me it's clearly the wrong thing to do.
--
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

Powered by blists - more mailing lists