Re: [resend PATCH] MAINTAINERS: Clarify UIO vs UACCESS maintainer
On Mon, Oct 08, 2018 at 11:50:09AM -0700, Dan Williams wrote: > The UIO file mask in MAINTAINERS was incorrectly directing UACCESS > (include/linux/uio.h) patches to Greg. > > Tag Al as the UACCESS maintainer as Ingo and others have explicitly > required his ack before taking architecture patches that touch > lib/iov_iter.c. > > Cc: Al Viro > Reported-by: Greg Kroah-Hartman > Acked-by: Greg Kroah-Hartman > Signed-off-by: Dan Williams > --- > I got a bounce last time I tried to send this, hopefully the situation > has improved now. > > Al, let me know if you want this entry. Alternatively we can just do the > UIO file mask fixup by itself. Sure, no problem, except that UACCESS refers to different things. I'd probably call it UIOVEC, if we want a name that does refer to the same somewhere (*BSD). If anything, it's generalization of copyin/copyout for non-userland destinations/sources; uaccess answer to (subset of) the same problem would be set_fs()/get_fs() and the fewer we have left of that, the better...
[resend PATCH] MAINTAINERS: Clarify UIO vs UACCESS maintainer
The UIO file mask in MAINTAINERS was incorrectly directing UACCESS (include/linux/uio.h) patches to Greg. Tag Al as the UACCESS maintainer as Ingo and others have explicitly required his ack before taking architecture patches that touch lib/iov_iter.c. Cc: Al Viro Reported-by: Greg Kroah-Hartman Acked-by: Greg Kroah-Hartman Signed-off-by: Dan Williams --- I got a bounce last time I tried to send this, hopefully the situation has improved now. Al, let me know if you want this entry. Alternatively we can just do the UIO file mask fixup by itself. MAINTAINERS |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index b22e7fdfd2ea..7abd97b96725 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15366,13 +15366,19 @@ F:arch/x86/um/ F: fs/hostfs/ F: fs/hppfs/ +USERSPACE ACCESS (UACCESS) +M: Alexander Viro +S: Maintained +F: lib/iov_iter.c +F: include/linux/uio.h + USERSPACE I/O (UIO) M: Greg Kroah-Hartman S: Maintained T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git F: Documentation/driver-api/uio-howto.rst F: drivers/uio/ -F: include/linux/uio*.h +F: include/linux/uio_driver.h UTIL-LINUX PACKAGE M: Karel Zak
Re: [PATCH] MAINTAINERS: Clarify UIO vs UACCESS maintainer
On Wed, Sep 12, 2018 at 10:23 PM Dan Williams wrote: > > The UIO file mask in MAINTAINERS was incorrectly directing UACCESS > (include/linux/uio.h) patches to Greg. > > Tag Al as the UACCESS maintainer as Ingo and others have explicitly > required his ack before taking architecture patches that touch > lib/iov_iter.c. > > Cc: Al Viro > Reported-by: Greg Kroah-Hartman > Signed-off-by: Dan Williams > --- > Al, let me know if this is ok. Alternatively we could just fix the file > mask for UIO and not add a new UACCESS section. Al, ack / nak? > > MAINTAINERS |8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index d870cb57c887..be6c29cfeb55 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -15344,13 +15344,19 @@ F:arch/x86/um/ > F: fs/hostfs/ > F: fs/hppfs/ > > +USERSPACE ACCESS (UACCESS) > +M: Alexander Viro > +S: Maintained > +F: lib/iov_iter.c > +F: include/linux/uio.h > + > USERSPACE I/O (UIO) > M: Greg Kroah-Hartman > S: Maintained > T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git > F: Documentation/driver-api/uio-howto.rst > F: drivers/uio/ > -F: include/linux/uio*.h > +F: include/linux/uio_driver.h > > UTIL-LINUX PACKAGE > M: Karel Zak >
Re: [PATCH] MAINTAINERS: Clarify UIO vs UACCESS maintainer
On Wed, Sep 12, 2018 at 10:09:28PM -0700, Dan Williams wrote: > The UIO file mask in MAINTAINERS was incorrectly directing UACCESS > (include/linux/uio.h) patches to Greg. > > Tag Al as the UACCESS maintainer as Ingo and others have explicitly > required his ack before taking architecture patches that touch > lib/iov_iter.c. > > Cc: Al Viro > Reported-by: Greg Kroah-Hartman > Signed-off-by: Dan Williams > --- > Al, let me know if this is ok. Alternatively we could just fix the file > mask for UIO and not add a new UACCESS section. Thanks for doing this! Acked-by: Greg Kroah-Hartman
[PATCH] MAINTAINERS: Clarify UIO vs UACCESS maintainer
The UIO file mask in MAINTAINERS was incorrectly directing UACCESS (include/linux/uio.h) patches to Greg. Tag Al as the UACCESS maintainer as Ingo and others have explicitly required his ack before taking architecture patches that touch lib/iov_iter.c. Cc: Al Viro Reported-by: Greg Kroah-Hartman Signed-off-by: Dan Williams --- Al, let me know if this is ok. Alternatively we could just fix the file mask for UIO and not add a new UACCESS section. MAINTAINERS |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index d870cb57c887..be6c29cfeb55 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -15344,13 +15344,19 @@ F:arch/x86/um/ F: fs/hostfs/ F: fs/hppfs/ +USERSPACE ACCESS (UACCESS) +M: Alexander Viro +S: Maintained +F: lib/iov_iter.c +F: include/linux/uio.h + USERSPACE I/O (UIO) M: Greg Kroah-Hartman S: Maintained T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git F: Documentation/driver-api/uio-howto.rst F: drivers/uio/ -F: include/linux/uio*.h +F: include/linux/uio_driver.h UTIL-LINUX PACKAGE M: Karel Zak