Skip to content

Cleanup unload_nls() calls

Thomas Gleixner lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest11 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/11fs: Make unload_nls() NULL pointer safefs/nls/nls_base.c+2 -10 replies
2/11fat: cleanup fat_put_super()fs/fat/inode.c+4 -121 reply
3/11befs: cleanup befs_put_super()fs/befs/linuxvfs.c+1 -60 replies
4/11cifs: cleanup unload_nls() callfs/cifs/cifsfs.c+1 -20 replies
5/11hfs: clenaup unload_nls() callsfs/hfs/mdb.c+2 -40 replies
6/11hfsplus: cleanup unload_nls() callsfs/hfsplus/super.c+2 -40 replies
7/11isofs: cleanup unload_nls() callsfs/isofs/inode.c+2 -60 replies
8/11jfs: cleanup unload_nls() callsfs/jfs/super.c+3 -61 reply
9/11ncpfs: cleanup unload_nls() callsfs/ncpfs/inode.c fs/ncpfs/ioctl.c+4 -140 replies
10/11ntfs: cleanup unload_nls() callsfs/ntfs/super.c+4 -60 replies
11/11smbfs: cleanup unload_nls() callsfs/smbfs/inode.c+2 -80 replies

Reviews trailers per patch

1 of 11 patches has a Reviewed-by or Acked-by.

PatchDave Kleikamp
1/11
2/11
3/11
4/11
5/11
6/11
7/11
8/11A-b
9/11
10/11
11/11

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [patch 00/11] Cleanup unload_nls() calls

While looking through the BKL call sites in fs/* I noticed that most callsites of unload_nls() do

	  if (nls) 
	     	   unload_nls(nls);

The patch series moves the NULL pointer check into unload_nls() and cleans up the various users.

Thanks,

tglx
---

befs/linuxvfs.c |    7 +------
cifs/cifsfs.c   |    3 +--
fat/inode.c     |   16 ++++------------
hfs/mdb.c       |    6 ++----
hfsplus/super.c |    6 ++----
isofs/inode.c   |    8 ++------
jfs/super.c     |    9 +++------
ncpfs/inode.c   |   12 ++----------
ncpfs/ioctl.c   |    6 ++----
nls/nls_base.c  |    3 ++-
ntfs/super.c    |   10 ++++------
smbfs/inode.c   |   10 ++--------
12 files changed, 27 insertions(+), 69 deletions(-)

Read the cover letter in its thread