Re: [PATCH v6 0/9] net: fix sysfs permssions when device changes network

[David Miller]

From: David Miller <davem@xxxxxxxxxxxxx>
Date: Wed, 26 Feb 2020 17:37:50 -0800 (PST)

> From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> Date: Wed, 26 Feb 2020 09:17:57 +0100
> 
>> On Tue, Feb 25, 2020 at 02:19:29PM +0100, Christian Brauner wrote:
>>> Hey everyone,
>>> 
>>> /* v6 */
>>> This is v6 with two small fixups. I missed adapting the commit message
>>> to reflect the renamed helper for changing the owner of sysfs files and
>>> I also forgot to make the new dpm helper static inline.
>> 
>> All of the sysfs and driver core bits look good to me now.  Thanks for
>> taking the time to update the documentation and other bits based on
>> reviews.
>> 
>> So now it's just up to the netdev developers to review the netdev parts :)
>> 
>> The sysfs and driver core patches can all go through the netdev tree to
>> make it easier for you.
> 
> I'm fine with these changes, and will apply this series to net-next.

Actually, you'll need to respin this with these warnings fixed, thanks:

fs/sysfs/group.c: In function ʽsysfs_group_change_ownerʼ:
fs/sysfs/group.c:517:9: warning: unused variable ʽgidʼ [-Wunused-variable]
  kgid_t gid;
         ^~~
fs/sysfs/group.c:516:9: warning: unused variable ʽuidʼ [-Wunused-variable]
  kuid_t uid;
         ^~~