linux-kernel - Re: [PATCH] Add driver to force WMI Thunderbolt controller power status

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: <1504727178.2677.158.camel@intel.com>
Date:   Wed, 6 Sep 2017 19:46:14 +0000
From:   "Bernat, Yehezkel" <yehezkel.bernat@...el.com>
To:     "mario.limonciello@...l.com" <mario.limonciello@...l.com>
CC:     "mika.westerberg@...ux.intel.com" <mika.westerberg@...ux.intel.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>,
        "dvhart@...radead.org" <dvhart@...radead.org>,
        "hughsient@...il.com" <hughsient@...il.com>
Subject: Re: [PATCH] Add driver to force WMI Thunderbolt controller power
 status

(Resending with a fixed CC list. Sorry.)

> Current implementations of Intel Thunderbolt controllers will go
> into a low power mode when not in use.
> 
> Many machines containing these controllers also have a GPIO wired up
> that can force the controller awake.  This is offered via a ACPI-WMI
> interface intended to be manipulated by a userspace utility.


> This mechanism is provided by Intel to OEMs to include in BIOS.
> It uses an industry wide GUID that is populated in a separate _WDG
> entry with no binary MOF.
> 
> This interface allow software such as fwupd to wake up thunderbolt
> controllers to query the firmware version or flash new firmware.

As this is a Thunderbolt specific function, maybe it's better to be
exposed from the Thunderbolt driver?


> +
> +static DEVICE_ATTR_WO(force_power);
> +

I'm not sure what is the convention for permissions for this type of
attributes but I feel like this worth being root-only writable, as it
can be used to power-off the controller in the middle of a FW update,
for example.