linux-kernel - Re: [PATCH] Re: linux-2.6.19-rc5-g088406bc build #120 failed

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: <45588BF0.3000100@garzik.org>
Date:	Mon, 13 Nov 2006 10:14:56 -0500
From:	Jeff Garzik <jeff@...zik.org>
To:	Paul Fulghum <paulkf@...rogate.com>
CC:	Toralf Förster <toralf.foerster@....de>,
	khc@...waw.pl, linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...l.org>
Subject: Re: [PATCH] Re: linux-2.6.19-rc5-g088406bc build #120 failed

Paul Fulghum wrote:
> Jeff Garzik wrote:
>> Toralf Förster wrote:
>>
>>> Hello,
>>>
>>> the build with the attached .config failed, make ends with:
>>> ... UPD     include/linux/compile.h
>>>   CC      init/version.o
>>>   LD      init/built-in.o
>>>   LD      .tmp_vmlinux1
>>> drivers/built-in.o: In function `hdlcdev_open':
>>> synclink.c:(.text+0x650d5): undefined reference to `hdlc_open'
>>> synclink.c:(.text+0x6510d): undefined reference to `hdlc_open'
>>> ...
>>> synclink_cs.c:(.text+0x7aece): undefined reference to `hdlc_ioctl'
>>> drivers/built-in.o: In function `hdlcdev_init':
>>> synclink_cs.c:(.text+0x7b336): undefined reference to `alloc_hdlcdev'
>>> drivers/built-in.o: In function `hdlcdev_exit':
>>> synclink_cs.c:(.text+0x7b434): undefined reference to 
>>> `unregister_hdlc_device'
>>> make: *** [.tmp_vmlinux1] Error 1
>>
>>
>> Does this patch work for you?
>>
>>     Jeff
> 
> No, this patch is not acceptable.
> 
> This has been beaten to death in previous threads.
> The problem is a mismatch in your kernel config between
> generic hdlc (M) and synclink (Y).
> 
> synclink drivers can *optionally* support generic hdlc.
> You *must* be able to build synclink driver without generic hdlc.
> Because of this you *can't* just put in the generic hdlc dependency.
> 
> Several alternative patches were posted (3 or 4 months) ago.
> No particular patch won the approval of all kernel developers,
> so nothing was done.


The existing build breakage is unacceptable too.  It's this bogosity

	#ifdef CONFIG_HDLC_MODULE
	#define CONFIG_HDLC 1
	#endif

which directly causes the build breakage.

Unless someone fixes it The Right Way, my patch seems like the best we 
can do.  People who complain "you must be able to build without hdlc" 
should step up and make it so.  Currently, it is not so.

	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/