linux-kernel - RE: [External] Re: [PATCH] thinkpad_acpi: Add sysfs entry for lcdshadow feature

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: <HK2PR0302MB25937E2946BF38583B3A905DBD130@HK2PR0302MB2593.apcprd03.prod.outlook.com>
Date:   Thu, 20 Feb 2020 15:14:25 +0000
From:   Mark Pearson <mpearson@...ovo.com>
To:     Andy Shevchenko <andy.shevchenko@...il.com>,
        Nitin Joshi <nitjoshi@...il.com>,
        Mat King <mathewk@...gle.com>,
        Jani Nikula <jani.nikula@...ux.intel.com>,
        Daniel Thompson <daniel.thompson@...aro.org>,
        Jingoo Han <jingoohan1@...il.com>,
        Rajat Jain <rajatja@...gle.com>
CC:     Henrique de Moraes Holschuh <ibm-acpi@....eng.br>,
        Darren Hart <dvhart@...radead.org>,
        Andy Shevchenko <andy@...radead.org>,
        "Thinkpad-acpi devel ML" <ibm-acpi-devel@...ts.sourceforge.net>,
        Platform Driver <platform-driver-x86@...r.kernel.org>,
        Nitin Joshi1 <njoshi1@...ovo.com>,
        Benjamin Berg <bberg@...hat.com>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        dri-devel <dri-devel@...ts.freedesktop.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: RE: [External]  Re: [PATCH] thinkpad_acpi: Add sysfs entry for
 lcdshadow feature

Hi Andy

> -----Original Message-----
> From: Andy Shevchenko <andy.shevchenko@...il.com>
> Sent: Thursday, February 20, 2020 5:43 AM
> 
> On Thu, Feb 20, 2020 at 9:48 AM Nitin Joshi <nitjoshi@...il.com> wrote:
> >
> >   This feature is supported on some Thinkpad products like T490s, Thinkpad
> >   X1 yoga 4th Gen etc . The lcdshadow feature can be enabled and disabled
> >   when user press "Fn" + "D" key. Currently, no user feedback is given for
> >   this action. Adding as sysfs entry allows userspace to show an On Screen
> >   Display whenever the setting changes.
> >
> >   Summary of changes is mentioned below :
> >
> >  - Added TP_HKEY_EV_LCDSHADOW_CHANGED for consistency inside the
> driver
> >  - Added unmapped LCDSHADOW to keymap
> >  - Added lcdshadow_get function to read value using ACPI
> >  - Added lcdshadow_refresh function to re-read value and send notification
> >  - Added sysfs group creation to tpaci_lcdshadow_init
> >  - Added lcdshadow_exit to remove sysfs group again
> >  - Implemented lcdshadow_enable_show/lcdshadow_enable_store
> >  - Added handler to tpacpi_driver_event to update refresh lcdshadow
> >  - Explicitly call tpacpi_driver_event for extended keyset
> 
> Adding custom PrivacyGuard support to this driver was my mistake,
> There is a discussion [1] how to do this in generic way to cover other
> possible users.
> I Cc this to people from that discussion.
> 
> [1]: https://lore.kernel.org/dri-
> devel/CAL_quvRknSSVvXN3q_Se0hrziw2oTNS3ENNoeHYhvciCRq9Yww@...l
> .gmail.com/
> 
Thanks for the pointer to that thread - really useful and interesting, we weren't aware there was an ongoing exercise to do this.

I work with Nitin as part of the Linux team at Lenovo. We're trying to get more directly and actively involved in the open source community to improve the Linux experience on Lenovo devices and of course want to make sure we contribute the right way. We're all still pretty new so pointers and help are very much appreciated (we've been getting some great support from the distros to get us started).

For this particular issue what is the best way to contribute and get involved? We'd like to make it so ePrivacy can be used more easily from Linux. I agree a more generic way of controlling it would be good.
I looked at the proposed patch from Rajat (https://lkml.org/lkml/2019/10/22/967) - it seems like a good solution to me. We can help with testing that on our platforms if that would be useful.

I need to understand how we connect that implementation with the ACPI controls we have (as I believe what we have are thinkpad specific and not to a drm spec; we need to confirm that). We also have the ACPI events that notify if ePrivacy was changed by the hotkeys and that seems like something that should be done in thinkpad_acpi.c and not the drm code. Not sure if the two need to be connected somehow (or if handling the event is actually not important and polling is acceptable)?

As a note Nitin has been working with the Red Hat folk and is looking at the user space aspect of this (in particularl gnome settings) as well.

Thanks
Mark Pearson