linux-kernel - Re: Blackfin STAMP537 compiling error in 2.6.25-rc8

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: <47FA813D.6090505@garzik.org>
Date:	Mon, 07 Apr 2008 16:17:01 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	Bryan Wu <cooloney@...nel.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	David Miller <davem@...emloft.net>
CC:	LKML <linux-kernel@...r.kernel.org>, magnus.damm@...il.com,
	damm@...l.co.jp, lethal@...ux-sh.org, nico@....org,
	Mike Frysinger <vapier.adi@...il.com>
Subject: Re: Blackfin STAMP537 compiling error in 2.6.25-rc8

Bryan Wu wrote:
> On Mon, Apr 7, 2008 at 1:13 AM, Jeff Garzik <jeff@...zik.org> wrote:
>> Bryan Wu wrote:
>>
>>> We found the latest kernel compile failed with Blackfin STAMP537
>>> configuration on our daily build machine.
>>> ---
>>>  CC      drivers/net/smc91x.o
>>> drivers/net/smc91x.c: In function 'smc_rcv':
>>> drivers/net/smc91x.c:513: warning: passing argument 1 of 'insw' makes
>>> integer from pointer without a cast
>>> drivers/net/smc91x.c: In function 'smc_hardware_send_pkt':
>>> drivers/net/smc91x.c:620: warning: passing argument 1 of 'outsw' makes
>>> integer from pointer without a cast
>>> drivers/net/smc91x.c:1859:32: error: macro "SMC_GET_MAC_ADDR" requires
>>> 2 arguments, but only 1 given
>>> drivers/net/smc91x.c: In function 'smc_probe':
>>> drivers/net/smc91x.c:1859: error: 'SMC_GET_MAC_ADDR' undeclared (first
>>> use in this function)
>>> drivers/net/smc91x.c:1859: error: (Each undeclared identifier is
>>> reported only once
>>> drivers/net/smc91x.c:1859: error: for each function it appears in.)
>>> make[2]: *** [drivers/net/smc91x.o] Error 1
>>> make[1]: *** [drivers/net] Error 2
>>> make: *** [drivers] Error 2
>>> ---
>>>
>>> That is because my -mm tree fixing bug patch:
>>>
>> (http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=9e6db60825ef7e7999abc610ce256ba768e58162)
>>> was merged into mainline before the smc91x API change patch in -mm:
>>>
>> http://www.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.25-rc5/2.6.25-rc5-mm1/broken-out/smc91x-pass-along-private-data-v2.patch
>>> Did I miss something here? or this patch is on the way before 2.6.25
>>> final release?
>>>
>>  It looks like I missed something...  I thought akpm was going to send the
>> patch for -mm in?
>>
>>  If not, let's revert this one...
>>
> 
> ok, no problem, it is ok for me.

Cool.

David or Linus, please revert 9e6db60825ef7e7999abc610ce256ba768e58162

It was merged without the API it needed, causing build breakage.

Acked-by: Jeff Garzik <jgarzik@...hat.com>



Andrew,

Since it was an arch patch to a driver I cannot build, I made the 
[mistaken] assumption that your sending it to me was an indication that 
it should actually be merged :)  Especially with a subject like "fix 
build breakage", for a platform I cannot easily build...

Are there any process tweaks that could be made here?  Maybe -mm stuff 
sent to maintainers could be tagged, noting dependencies on other -mm 
patches that are not yet upstream?

Thanks,

	Jeff


--
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/