linux-kernel - Re: [PATCH v2] staging: Remove the drivers for the Unisys s-Par

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] [day] [month] [year] [list]

Message-ID: <2100119.irdbgypaU6@leap>
Date:   Thu, 14 Apr 2022 10:40:26 +0200
From:   "Fabio M. De Francesco" <fmdefrancesco@...il.com>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org,
        outreachy@...ts.linux.dev,
        David Kershner <david.kershner@...sys.com>,
        sparmaintainer@...sys.com
Subject: Re: [PATCH v2] staging: Remove the drivers for the Unisys s-Par

On gioved? 14 aprile 2022 09:14:29 CEST Greg Kroah-Hartman wrote:
> On Wed, Apr 13, 2022 at 10:01:11AM +0200, Fabio M. De Francesco wrote:
> > The Unisys sub-tree contains three drivers for the "Unisys Secure 
Partition"
> > (s-Par(R)): visorhba, visorinput, visornic.
> > 
> > They have no maintainers, in fact the only one that is listed in 
MAINTAINERS
> > has an unreacheable email address. During 2021 and 2022 several patches 
have
> > been submitted to these drivers but nobody at Unisys cared of reviewing 
the
> > changes. Probably, also the "sparmaintainer" internal list of 
unisys.com is
> > not anymore read by interested Unisys' engineers.
> > 
> > Therefore, remove the ./unisys subdirectory and delete the relevant 
entries
> > in the MAINTAINERS, Kconfig, Makefile files.
> > 
> > Cc: David Kershner <david.kershner@...sys.com>
> > Cc: sparmaintainer@...sys.com
> > Suggested-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > Signed-off-by: Fabio M. De Francesco <fmdefrancesco@...il.com>
> > ---
> > 
> > v1->v2: Add two "Cc:" lines, one to David Kershner and the second to 
the
> > "sparmaintainer" list at unisys.com, as requested by Greg Kroah-
Hartman.
> > 
> >  MAINTAINERS                                   |    8 -
> >  drivers/staging/Kconfig                       |    2 -
> >  drivers/staging/Makefile                      |    1 -
> >  .../ABI/sysfs-platform-visorchipset           |   89 -
> >  .../staging/unisys/Documentation/overview.txt |  337 ---
> >  drivers/staging/unisys/Kconfig                |   16 -
> >  drivers/staging/unisys/MAINTAINERS            |    5 -
> >  drivers/staging/unisys/Makefile               |    7 -
> >  drivers/staging/unisys/TODO                   |   16 -
> >  drivers/staging/unisys/include/iochannel.h    |  571 -----
> >  drivers/staging/unisys/visorhba/Kconfig       |   15 -
> >  drivers/staging/unisys/visorhba/Makefile      |   10 -
> >  .../staging/unisys/visorhba/visorhba_main.c   | 1142 ---------
> >  drivers/staging/unisys/visorinput/Kconfig     |   16 -
> >  drivers/staging/unisys/visorinput/Makefile    |    7 -
> >  .../staging/unisys/visorinput/visorinput.c    |  788 ------
> >  drivers/staging/unisys/visornic/Kconfig       |   16 -
> >  drivers/staging/unisys/visornic/Makefile      |   10 -
> >  .../staging/unisys/visornic/visornic_main.c   | 2131 -----------------
> 
> 
> 
> >  19 files changed, 5187 deletions(-)
> >  delete mode 100644 drivers/staging/unisys/Documentation/ABI/sysfs-
platform-visorchipset
> >  delete mode 100644 drivers/staging/unisys/Documentation/overview.txt
> >  delete mode 100644 drivers/staging/unisys/Kconfig
> >  delete mode 100644 drivers/staging/unisys/MAINTAINERS
> >  delete mode 100644 drivers/staging/unisys/Makefile
> >  delete mode 100644 drivers/staging/unisys/TODO
> >  delete mode 100644 drivers/staging/unisys/include/iochannel.h
> >  delete mode 100644 drivers/staging/unisys/visorhba/Kconfig
> >  delete mode 100644 drivers/staging/unisys/visorhba/Makefile
> >  delete mode 100644 drivers/staging/unisys/visorhba/visorhba_main.c
> >  delete mode 100644 drivers/staging/unisys/visorinput/Kconfig
> >  delete mode 100644 drivers/staging/unisys/visorinput/Makefile
> >  delete mode 100644 drivers/staging/unisys/visorinput/visorinput.c
> >  delete mode 100644 drivers/staging/unisys/visornic/Kconfig
> >  delete mode 100644 drivers/staging/unisys/visornic/Makefile
> >  delete mode 100644 drivers/staging/unisys/visornic/visornic_main.c
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 3ed62dcd144e..a62da6d0f943 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -20183,14 +20183,6 @@ F:	drivers/cdrom/cdrom.c
> >  F:	include/linux/cdrom.h
> >  F:	include/uapi/linux/cdrom.h
> >  
> > -UNISYS S-PAR DRIVERS
> > -M:	David Kershner <david.kershner@...sys.com>
> > -L:	sparmaintainer@...sys.com (Unisys internal)
> > -S:	Supported
> > -F:	drivers/staging/unisys/
> > -F:	drivers/visorbus/
> 
> You forgot to delete the files in this directory as well :(
> 
Sorry, that's worse than just being forgetful. I did not carefully read the
lines I deleted in MAINTAINERS :(

I will post a v3 soon. In the meantime, I'll also add between the lines
"Cc:" that other person that Alison found in drivers/staging/unisys/TODO.

Thanks,

Fabio M. De Francesco