mbox series

[00/10] netfs, afs, cifs, cachefiles, erofs: Miscellaneous fixes

Message ID 20240122123845.3822570-1-dhowells@redhat.com (mailing list archive)
Headers show
Series netfs, afs, cifs, cachefiles, erofs: Miscellaneous fixes | expand

Message

David Howells Jan. 22, 2024, 12:38 p.m. UTC
Hi Christian,

Here are some miscellaneous fixes for netfslib and a number of filesystems:

 (1) Replace folio_index() with folio->index in netfs, afs and cifs.

 (2) Fix an oops in fscache_put_cache().

 (3) Fix error handling in netfs_perform_write().

 (4) Fix an oops in cachefiles when not using erofs ondemand mode.

 (5) In afs, hide silly-rename files from getdents() to avoid problems with
     tar and suchlike.

 (6) In afs, fix error handling in lookup with a bulk status fetch.

 (7) In afs, afs_dynroot_d_revalidate() is redundant, so remove it.

 (8) In afs, fix the RCU unlocking in afs_proc_addr_prefs_show().

The patches can also be found here:

	https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=netfs-fixes

Thanks,
David

Dan Carpenter (2):
  netfs, fscache: Prevent Oops in fscache_put_cache()
  netfs: Fix a NULL vs IS_ERR() check in netfs_perform_write()

David Howells (8):
  netfs: Don't use certain internal folio_*() functions
  afs: Don't use certain internal folio_*() functions
  cifs: Don't use certain internal folio_*() functions
  cachefiles, erofs: Fix NULL deref in when cachefiles is not doing
    ondemand-mode
  afs: Hide silly-rename files from userspace
  afs: Fix error handling with lookup via FS.InlineBulkStatus
  afs: Remove afs_dynroot_d_revalidate() as it is redundant
  afs: Fix missing/incorrect unlocking of RCU read lock

 fs/afs/dir.c               | 30 ++++++++++++++++++++++--------
 fs/afs/dynroot.c           |  9 ---------
 fs/afs/proc.c              |  5 +++--
 fs/cachefiles/namei.c      | 16 ++++++++++------
 fs/netfs/buffered_read.c   | 12 ++++++------
 fs/netfs/buffered_write.c  | 15 ++++++++-------
 fs/netfs/fscache_cache.c   |  3 ++-
 fs/netfs/io.c              |  2 +-
 fs/netfs/misc.c            |  2 +-
 fs/smb/client/file.c       | 10 +++++-----
 include/trace/events/afs.h | 25 +++++++++++++++++++++++++
 11 files changed, 83 insertions(+), 46 deletions(-)

Comments

Christian Brauner Jan. 22, 2024, 3:18 p.m. UTC | #1
On Mon, Jan 22, 2024 at 12:38:33PM +0000, David Howells wrote:
> Hi Christian,
> 
> Here are some miscellaneous fixes for netfslib and a number of filesystems:
> 
>  (1) Replace folio_index() with folio->index in netfs, afs and cifs.
> 
>  (2) Fix an oops in fscache_put_cache().
> 
>  (3) Fix error handling in netfs_perform_write().
> 
>  (4) Fix an oops in cachefiles when not using erofs ondemand mode.
> 
>  (5) In afs, hide silly-rename files from getdents() to avoid problems with
>      tar and suchlike.
> 
>  (6) In afs, fix error handling in lookup with a bulk status fetch.
> 
>  (7) In afs, afs_dynroot_d_revalidate() is redundant, so remove it.
> 
>  (8) In afs, fix the RCU unlocking in afs_proc_addr_prefs_show().
> 
> The patches can also be found here:
> 
> 	https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=netfs-fixes

Thank you! I can pull this in right and will send a pr together with the
other changes around Wednesday/Thursday for -rc2. So reviews before that
would be nice.
Jeff Layton Jan. 22, 2024, 3:57 p.m. UTC | #2
On Mon, 2024-01-22 at 12:38 +0000, David Howells wrote:
> Hi Christian,
> 
> Here are some miscellaneous fixes for netfslib and a number of filesystems:
> 
>  (1) Replace folio_index() with folio->index in netfs, afs and cifs.
> 
>  (2) Fix an oops in fscache_put_cache().
> 
>  (3) Fix error handling in netfs_perform_write().
> 
>  (4) Fix an oops in cachefiles when not using erofs ondemand mode.
> 
>  (5) In afs, hide silly-rename files from getdents() to avoid problems with
>      tar and suchlike.
> 
>  (6) In afs, fix error handling in lookup with a bulk status fetch.
> 
>  (7) In afs, afs_dynroot_d_revalidate() is redundant, so remove it.
> 
>  (8) In afs, fix the RCU unlocking in afs_proc_addr_prefs_show().
> 
> The patches can also be found here:
> 
> 	https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=netfs-fixes
> 
> Thanks,
> David
> 
> Dan Carpenter (2):
>   netfs, fscache: Prevent Oops in fscache_put_cache()
>   netfs: Fix a NULL vs IS_ERR() check in netfs_perform_write()
> 
> David Howells (8):
>   netfs: Don't use certain internal folio_*() functions
>   afs: Don't use certain internal folio_*() functions
>   cifs: Don't use certain internal folio_*() functions
>   cachefiles, erofs: Fix NULL deref in when cachefiles is not doing
>     ondemand-mode
>   afs: Hide silly-rename files from userspace
>   afs: Fix error handling with lookup via FS.InlineBulkStatus
>   afs: Remove afs_dynroot_d_revalidate() as it is redundant
>   afs: Fix missing/incorrect unlocking of RCU read lock
> 
>  fs/afs/dir.c               | 30 ++++++++++++++++++++++--------
>  fs/afs/dynroot.c           |  9 ---------
>  fs/afs/proc.c              |  5 +++--
>  fs/cachefiles/namei.c      | 16 ++++++++++------
>  fs/netfs/buffered_read.c   | 12 ++++++------
>  fs/netfs/buffered_write.c  | 15 ++++++++-------
>  fs/netfs/fscache_cache.c   |  3 ++-
>  fs/netfs/io.c              |  2 +-
>  fs/netfs/misc.c            |  2 +-
>  fs/smb/client/file.c       | 10 +++++-----
>  include/trace/events/afs.h | 25 +++++++++++++++++++++++++
>  11 files changed, 83 insertions(+), 46 deletions(-)
> 

These all look fine to me. You can add this to the set:

Reviewed-by: Jeff Layton <jlayton@kernel.org>
Christian Brauner Jan. 23, 2024, 3:03 p.m. UTC | #3
On Mon, Jan 22, 2024 at 04:18:08PM +0100, Christian Brauner wrote:
> On Mon, Jan 22, 2024 at 12:38:33PM +0000, David Howells wrote:
> > Hi Christian,
> > 
> > Here are some miscellaneous fixes for netfslib and a number of filesystems:
> > 
> >  (1) Replace folio_index() with folio->index in netfs, afs and cifs.
> > 
> >  (2) Fix an oops in fscache_put_cache().
> > 
> >  (3) Fix error handling in netfs_perform_write().
> > 
> >  (4) Fix an oops in cachefiles when not using erofs ondemand mode.
> > 
> >  (5) In afs, hide silly-rename files from getdents() to avoid problems with
> >      tar and suchlike.
> > 
> >  (6) In afs, fix error handling in lookup with a bulk status fetch.
> > 
> >  (7) In afs, afs_dynroot_d_revalidate() is redundant, so remove it.
> > 
> >  (8) In afs, fix the RCU unlocking in afs_proc_addr_prefs_show().
> > 
> > The patches can also be found here:
> > 
> > 	https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=netfs-fixes
> 
> Thank you! I can pull this in right and will send a pr together with the
> other changes around Wednesday/Thursday for -rc2. So reviews before that
> would be nice.

Pulled and pushed:

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.netfs

Timeline still the same.