linux-kernel - Re: [origin tree build failure] [PATCH] thinkpad-acpi: Fix build on !CONFIG_THINKPAD_ACPI_HOTKEY

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]

Date:	Mon, 28 Sep 2009 16:44:49 -0300
From:	Henrique de Moraes Holschuh <hmh@....eng.br>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Len Brown <lenb@...nel.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-acpi@...r.kernel.org,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [origin tree build failure] [PATCH] thinkpad-acpi: Fix build
 on !CONFIG_THINKPAD_ACPI_HOTKEY_POLL

On Mon, 28 Sep 2009, Ingo Molnar wrote:
> * Len Brown <lenb@...nel.org> wrote:
> > > > >  drivers/platform/x86/thinkpad_acpi.c    |  632 +++++++++++++++++++---------
> > > > 
> > > > -tip testing found that these changes caused a build failure 
> > > > in drivers/platform/x86/thinkpad_acpi.c when 
> > > > !CONFIG_THINKPAD_ACPI_HOTKEY_POLL - the fix is attached  below.
> > > 
> > > Fixed by a different patch, which will hit mainline soon....
> > 
> > Ingo,
> > 2.6.32-rc1 should build fine* -- let me know if it doesn't.
> > 
> > thanks,
> > Len Brown, Intel Open Source Technology Center
> > 
> > * module this message, which Henruque assures me will go away soon
> > drivers/platform/x86/thinkpad_acpi.c:2225: warning: 
> > ???tpacpi_hotkey_driver_mask_set??? defined but not used
> 
> well, the warning is fixed properly in my patch, as pointed out in the 
> changelog. Have you read that?

Your patch will break the driver when the functionality that uses
tpacpi_hotkey_driver_mask_set() lands in mainline.  That functionality is
already in a subsystem tree waiting a push to Linus.

Of course, you had no way to know that, and I failed to convey that detail
(mostly because _I_ screwed up and forgot I had that patch already applied)
when I replied to you.  Sorry about that.

The patch adding that functionality had already been submitted at the time
you sent in your patch, and it will be merged through the backlight tree
very soon.  That patch couldn't arrive earlier because it needed changes
both in the backlight tree AND in the ACPI tree, and nobody knew which tree
would merge first.

Now that all dependencies (both from backlight and ACPI) are already in
mainline, I asked Richard Purdie to send it to Linus together with his next
batch of backlight updates.  That will silence the warning about
tpacpi_hotkey_driver_mask_set().

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh
--
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/