[dpdk-dev] [PATCH] kernel: remove igb_uio

[Thomas Monjalon]

08/09/2020 14:06, Ferruh Yigit:
> On 9/7/2020 11:50 PM, Thomas Monjalon wrote:
> > As decided in the Technical Board in November 2019,
> > the kernel module igb_uio is moved to the dpdk-kmods repository
> > in the /linux/igb_uio/ directory.
> > 
> > Minutes of Technical Board meeting:
> > https://mails.dpdk.org/archives/dev/2019-November/151763.html
> > 
> > Signed-off-by: Thomas Monjalon <thomas at monjalon.net>
> > ---
> >  MAINTAINERS                          |   1 -
> >  doc/guides/rel_notes/deprecation.rst |   7 -
> >  kernel/linux/igb_uio/Kbuild          |   2 -
> >  kernel/linux/igb_uio/compat.h        | 154 -------
> >  kernel/linux/igb_uio/igb_uio.c       | 660 ---------------------------
> >  kernel/linux/igb_uio/meson.build     |  20 -
> >  kernel/linux/meson.build             |   2 +-
> >  7 files changed, 1 insertion(+), 845 deletions(-)
> >  delete mode 100644 kernel/linux/igb_uio/Kbuild
> >  delete mode 100644 kernel/linux/igb_uio/compat.h
> >  delete mode 100644 kernel/linux/igb_uio/igb_uio.c
> >  delete mode 100644 kernel/linux/igb_uio/meson.build
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 3b16d7a4b8..d74bec58e1 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -292,7 +292,6 @@ F: doc/guides/linux_gsg/
> >  
> >  Linux UIO
> >  M: Ferruh Yigit <ferruh.yigit at intel.com>
> > -F: kernel/linux/igb_uio/
> >  F: drivers/bus/pci/linux/*uio*
> >  
> 
> What do you think to have the new git repo information here as
> "T: ..."
> To help people looking for the module.

I don't think we should reference the new home in the file MAINTAINERS
of dpdk.git. But yes, it should be documented. I'll add something in v2.

> And I think at least 'doc/guides/linux_gsg/linux_drivers.rst' should be updated
> to document the new home of the kernel module, but there are multiple references
> in the documentation to the 'igb_uio', perhaps they should be replaced with
> 'vfio' too?

As part of this patch, I will document the new home of igb_uio,
but won't change the usage references.
I think moving to VFIO in docs is another patch, and probably multiple.