linux-kernel - Re: [PATCH v6] driver core: Add sysfs support for physical location of a device

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAOvb9yi_YYVjT1ZVKE0oOH8GeqgqbDkcsNC2WMwUJFw6J37-OQ@mail.gmail.com>
Date:   Wed, 23 Mar 2022 10:54:44 -0700
From:   Won Chung <wonchung@...gle.com>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Heikki Krogerus <heikki.krogerus@...ux.intel.com>,
        "Rafael J . Wysocki" <rafael@...nel.org>,
        Benson Leung <bleung@...omium.org>,
        Prashant Malani <pmalani@...omium.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6] driver core: Add sysfs support for physical location
 of a device

On Wed, Mar 23, 2022 at 12:21 AM Greg Kroah-Hartman
<gregkh@...uxfoundation.org> wrote:
>
> On Tue, Mar 22, 2022 at 09:36:02AM -0700, Won Chung wrote:
> > On Mon, Mar 14, 2022 at 12:55 PM Won Chung <wonchung@...gle.com> wrote:
> > I am sorry to keep bothering you with this, but can you take a look at
> > this patch for a review when you have time? Thank you very much!
>
> It is the middle of the merge window now, and I will look at this after
> 5.18-rc1 is out.  Due to all of the constant churn in this (and you only
> fixing the memory leak in the last submission), I had put this on the
> bottom of my review queue.
>
> Also, I don't see an actual user of these new attributes anywhere, so
> even if this was accepted, who is going to use it?
>
> thanks,
>
> greg k-h

Hi Greg,

Thank you for the information.

As a start, we are planning to use this in Chrome OS to specify the
physical location of each USB device since Coreboot adds _PLD fields
to usb port and typec connector. Currently, if there is one USB device
connected on the left panel and another USB device connected on the
right panel, the userspace cannot distinguish which device is on the
left or the right panel. Once each USB device exposes its location
info, we can gather more data on users' preference on port usage.
Also, it can apply to not just USB devices, but also other types like
HDMI ports in the future.
Would this answer your question?

Hey Benson, can you take a look at this if I am not mistaken or
missing some details?

Thanks,
Won