All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: "Pali Rohár" <pali.rohar@gmail.com>,
	"Henrique de Moraes Holschuh" <ibm-acpi@hmh.eng.br>,
	ibm-acpi-devel@lists.sourceforge.net,
	platform-driver-x86@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: ThinkPad T480s & LED_MUTE, LED_MICMUTE
Date: Sat, 16 Jun 2018 09:33:34 +0200	[thread overview]
Message-ID: <20180616073334.GA13880@amd> (raw)
In-Reply-To: <20180615233628.gy2ffgupctheyqof@khazad-dum.debian.net>

[-- Attachment #1: Type: text/plain, Size: 1280 bytes --]

Hi!

On Fri 2018-06-15 20:36:28, Henrique de Moraes Holschuh wrote:
> On Fri, 15 Jun 2018, Pali Rohár wrote:
> > This means that kernel should not export any led class device. Or when
> > exported, then "set" operation should always fail.
> 
> "not export" is right.
> 
> > > 2. Otherwise implement it in-kernel, so that userspace cannot unmute
> > >    when the human has activated the "mute" switch, and the LED cannot be
> > >    controlled by userspace to lie (report mute when it is not mute).
> > 
> > This looks like a good candidate to use led "trigger" interface. Create
> > a mute trigger and attach it to that led device.
> 
> Maybe, as long as done in-kernel and not possible to mess with from
> userspace.

Question is if we want flexibility or security.

If we want security, going through LED subsystem makes no sense, just
control the LED as hardware would, or let hardware do it.

For full flexibility, just export the LED and use normal mechanisms we
have (such as triggers). root should be allowed to configure the LEDs,
and he can change the kernel, too, so...

Best regards,
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

  reply	other threads:[~2018-06-16  7:33 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-08 11:10 ThinkPad T480s & LED_MUTE, LED_MICMUTE Pali Rohár
2018-06-15 11:26 ` Pavel Machek
2018-06-15 11:37   ` Pali Rohár
2018-06-15 12:30     ` Henrique de Moraes Holschuh
2018-06-15 19:09       ` Pali Rohár
2018-06-15 23:36         ` Henrique de Moraes Holschuh
2018-06-16  7:33           ` Pavel Machek [this message]
2018-06-16 14:12             ` Henrique de Moraes Holschuh
2018-06-16 15:58               ` Pavel Machek
     [not found]           ` <20180615233628.gy2ffgupctheyqof-ZGHd14iZgfaRjzvQDGKj+xxZW9W5cXbT@public.gmane.org>
2018-06-18 10:11             ` Pali Rohár
2018-06-18 10:11               ` Pali Rohár
2018-06-21 11:37               ` Pali Rohár
2018-06-15 12:51 ` Takashi Iwai
2018-06-15 19:09   ` Pali Rohár
2018-06-16  7:05     ` Takashi Iwai
2018-06-16 15:43       ` Pali Rohár
2018-06-16 16:02         ` Takashi Iwai
2018-06-18 10:28           ` Pali Rohár
2018-06-18 10:36             ` Takashi Iwai
2018-06-18 11:21               ` Pali Rohár
2018-06-18 11:26                 ` Pali Rohár
2018-06-18 15:35                   ` Takashi Iwai
2018-06-21 11:30                     ` Pali Rohár
2018-06-21 11:35                       ` [ibm-acpi-devel] " Damjan Georgievski
2018-06-21 11:39                         ` Takashi Iwai
2018-06-23 12:46                           ` Damjan Georgievski
2018-06-21 11:40                         ` Pali Rohár
2018-06-19  8:37                 ` Pali Rohár
2018-06-19  8:37                   ` Pali Rohár
2018-06-19  8:42                   ` Takashi Iwai
2018-06-21 11:24                     ` Pali Rohár
2018-06-21 11:29                       ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180616073334.GA13880@amd \
    --to=pavel@ucw.cz \
    --cc=hmh@hmh.eng.br \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=ibm-acpi@hmh.eng.br \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pali.rohar@gmail.com \
    --cc=platform-driver-x86@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.