linux-kernel - Re: mmotm 2008-10-16-00-52 uploaded (HID)

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-next>] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.64.0810161558341.14604@shark.he.net>
Date:	Thu, 16 Oct 2008 16:05:25 -0700 (PDT)
From:	"Randy.Dunlap" <rdunlap@...otime.net>
To:	linux-kernel@...r.kernel.org
cc:	linux-input@...r.kernel.org, jkosina@...e.cz
Subject: Re: mmotm 2008-10-16-00-52 uploaded (HID)

On Thu, 16 Oct 2008, akpm@...ux-foundation.org wrote:

> The mm-of-the-moment snapshot 2008-10-16-00-52 has been uploaded to
> 
>    http://userweb.kernel.org/~akpm/mmotm/
> 
> It contains the following patches against 2.6.27:

mmotm has several various build failures when
HID=y and USB=n
or
HID=y and USB_HID=m.

These are (mostly) caused by drivers/hid/Kconfig doing lots of "select"s
when EMBEDDED=n, such as

config HID
	tristate "Generic HID support"
	depends on INPUT
	default y
        select HID_A4TECH if !EMBEDDED
        select HID_APPLE if !EMBEDDED
        select HID_BELKIN if !EMBEDDED
        select HID_BRIGHT if !EMBEDDED

and HID_BRIGHT (just to pick one of them) says:

config HID_BRIGHT
	tristate "Bright"
	default m
	depends on USB_HID

but that "depends on USB_HID" isn't being "honored" because "select"
doesn't honor (or care) about dependencies.

I think that you already know about this, but drivers/hid/Kconfig
does need fixing.


Build failure examples:

9168:
HID=y, USB=n
hid-bright.c:(.text+0x1013f0): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0x1019bc): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0x102310): undefined reference to `usbhid_set_leds'
hid-lgff.c:(.text+0x102509): undefined reference to `usbhid_submit_report'
hid-lgff.c:(.text+0x102566): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0x102bbb): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0x102ddf): undefined reference to `usbhid_submit_report'
hid-sony.c:(.text+0x103339): undefined reference to `usb_control_msg'

9170:
HID=y, USB_HID=m
hid-bright.c:(.text+0xa9d2e): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0xaa332): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0xaa7e8): undefined reference to `usbhid_set_leds'
hid-pl.c:(.text+0xab669): undefined reference to `usbhid_submit_report'
hid-pl.c:(.text+0xab88d): undefined reference to `usbhid_submit_report'

9174:
HID=y, USB=n
hid-bright.c:(.text+0x1b4cf2): undefined reference to `usbhid_set_leds'
hid-dell.c:(.text+0x1b52e2): undefined reference to `usbhid_set_leds'
hid-lg.c:(.text+0x1b5790): undefined reference to `usbhid_set_leds'
hid-lg2ff.c:(.text+0x1b6077): undefined reference to `usbhid_submit_report'
(.text+0x1b61d2): undefined reference to `usbhid_submit_report'
hid-sony.c:(.text+0x1b6db7): undefined reference to `usb_control_msg'

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