linux-kernel - Re: sony-laptop driver: Volume keys on Sony Vaio TX3 don't work

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:	Tue, 18 Oct 2011 06:45:44 +0900
From:	Mattia Dongili <malattia@...ux.it>
To:	Valdis.Kletnieks@...edu
Cc:	John Hughes <john@...vaedi.com>, linux-kernel@...r.kernel.org,
	Stelian Pop <stelian@...ies.net>,
	platform-driver-x86 <platform-driver-x86@...r.kernel.org>
Subject: Re: sony-laptop driver: Volume keys on Sony Vaio TX3 don't work

On Sun, Oct 16, 2011 at 05:50:13PM -0400, Valdis.Kletnieks@...edu wrote:
> On Mon, 17 Oct 2011 06:38:01 +0900, Mattia Dongili said:
> > On Sun, Oct 16, 2011 at 02:20:18PM -0400, Valdis.Kletnieks@...edu wrote:
> > > Do we know if this is applicable to all Sony laptops, or only a few busticated
> > > models?
> >
> > I believe the latter is the case, the old A series also had this
> > problem. Of course applying the patch will break other models (SZ is one
> > I'm sure about) so there is no actual winner here unless we figure out
> > how to properly decode keys on sonypi models (this problem is
> > specifically related to two phase hotkey decoding, see the irq handler).
> 
> And let me guess - two phase hotkey decoding is sufficiently weird that
> nobody wants to actually fix *that* code, right? (Rhetorical question, I
> already know the answer to that one).
> 
> If that's not an option, perhaps we need a per-model quirk that patches up the
> tables at boot time?  If we do that, we may want to re-arrange the table so the
> affected entries are at the front or something to help insure we don't patch
> the wrong entries if somebody else inserts new lines.

Wouldn't udev do that already?
We may actually want to remove those events that never get looked up in
the tables instead.

-- 
mattia
:wq!

Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)