* [PATCH] 9p: Remove some unneeded #include
@ 2022-11-27 12:59 Christophe JAILLET
2022-11-27 13:17 ` Dominique Martinet
2022-11-27 14:07 ` Christian Schoenebeck
0 siblings, 2 replies; 4+ messages in thread
From: Christophe JAILLET @ 2022-11-27 12:59 UTC (permalink / raw)
To: Eric Van Hensbergen, Latchesar Ionkov, Dominique Martinet,
Christian Schoenebeck
Cc: linux-kernel, kernel-janitors, Christophe JAILLET, v9fs-developer
The 9p fs does not use IDR or IDA functionalities. So there is no point in
including <linux/idr.h>.
Remove it.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
fs/9p/fid.c | 1 -
fs/9p/v9fs.c | 1 -
fs/9p/vfs_addr.c | 1 -
fs/9p/vfs_dentry.c | 1 -
fs/9p/vfs_dir.c | 1 -
fs/9p/vfs_file.c | 1 -
fs/9p/vfs_inode.c | 1 -
fs/9p/vfs_inode_dotl.c | 1 -
fs/9p/vfs_super.c | 1 -
9 files changed, 9 deletions(-)
diff --git a/fs/9p/fid.c b/fs/9p/fid.c
index 23cf9b2fbfe4..805151114e96 100644
--- a/fs/9p/fid.c
+++ b/fs/9p/fid.c
@@ -11,7 +11,6 @@
#include <linux/fs.h>
#include <linux/slab.h>
#include <linux/sched.h>
-#include <linux/idr.h>
#include <net/9p/9p.h>
#include <net/9p/client.h>
diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c
index 0129de2ea31a..3a9c4517265f 100644
--- a/fs/9p/v9fs.c
+++ b/fs/9p/v9fs.c
@@ -14,7 +14,6 @@
#include <linux/sched.h>
#include <linux/cred.h>
#include <linux/parser.h>
-#include <linux/idr.h>
#include <linux/slab.h>
#include <linux/seq_file.h>
#include <net/9p/9p.h>
diff --git a/fs/9p/vfs_addr.c b/fs/9p/vfs_addr.c
index 47b9a1122f34..93373486ab04 100644
--- a/fs/9p/vfs_addr.c
+++ b/fs/9p/vfs_addr.c
@@ -14,7 +14,6 @@
#include <linux/string.h>
#include <linux/inet.h>
#include <linux/pagemap.h>
-#include <linux/idr.h>
#include <linux/sched.h>
#include <linux/swap.h>
#include <linux/uio.h>
diff --git a/fs/9p/vfs_dentry.c b/fs/9p/vfs_dentry.c
index f89f01734587..65fa2df5e49b 100644
--- a/fs/9p/vfs_dentry.c
+++ b/fs/9p/vfs_dentry.c
@@ -15,7 +15,6 @@
#include <linux/string.h>
#include <linux/inet.h>
#include <linux/namei.h>
-#include <linux/idr.h>
#include <linux/sched.h>
#include <linux/slab.h>
#include <net/9p/9p.h>
diff --git a/fs/9p/vfs_dir.c b/fs/9p/vfs_dir.c
index 000fbaae9b18..1675a196c2ba 100644
--- a/fs/9p/vfs_dir.c
+++ b/fs/9p/vfs_dir.c
@@ -14,7 +14,6 @@
#include <linux/string.h>
#include <linux/sched.h>
#include <linux/inet.h>
-#include <linux/idr.h>
#include <linux/slab.h>
#include <linux/uio.h>
#include <linux/fscache.h>
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index aec43ba83799..b740017634ef 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -18,7 +18,6 @@
#include <linux/pagemap.h>
#include <linux/utsname.h>
#include <linux/uaccess.h>
-#include <linux/idr.h>
#include <linux/uio.h>
#include <linux/slab.h>
#include <net/9p/9p.h>
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
index 4d1a4a8d9277..27a04a226d97 100644
--- a/fs/9p/vfs_inode.c
+++ b/fs/9p/vfs_inode.c
@@ -17,7 +17,6 @@
#include <linux/string.h>
#include <linux/inet.h>
#include <linux/namei.h>
-#include <linux/idr.h>
#include <linux/sched.h>
#include <linux/slab.h>
#include <linux/xattr.h>
diff --git a/fs/9p/vfs_inode_dotl.c b/fs/9p/vfs_inode_dotl.c
index 03c1743c4aff..f806b3f11649 100644
--- a/fs/9p/vfs_inode_dotl.c
+++ b/fs/9p/vfs_inode_dotl.c
@@ -15,7 +15,6 @@
#include <linux/string.h>
#include <linux/inet.h>
#include <linux/namei.h>
-#include <linux/idr.h>
#include <linux/sched.h>
#include <linux/slab.h>
#include <linux/xattr.h>
diff --git a/fs/9p/vfs_super.c b/fs/9p/vfs_super.c
index 2d9ee073d12c..266c4693e20c 100644
--- a/fs/9p/vfs_super.c
+++ b/fs/9p/vfs_super.c
@@ -15,7 +15,6 @@
#include <linux/inet.h>
#include <linux/pagemap.h>
#include <linux/mount.h>
-#include <linux/idr.h>
#include <linux/sched.h>
#include <linux/slab.h>
#include <linux/statfs.h>
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] 9p: Remove some unneeded #include
2022-11-27 12:59 [PATCH] 9p: Remove some unneeded #include Christophe JAILLET
@ 2022-11-27 13:17 ` Dominique Martinet
2022-11-27 14:07 ` Christian Schoenebeck
1 sibling, 0 replies; 4+ messages in thread
From: Dominique Martinet @ 2022-11-27 13:17 UTC (permalink / raw)
To: Christophe JAILLET
Cc: Eric Van Hensbergen, Latchesar Ionkov, Christian Schoenebeck,
linux-kernel, kernel-janitors, v9fs-developer
Christophe JAILLET wrote on Sun, Nov 27, 2022 at 01:59:25PM +0100:
> Subject: Re: [PATCH] 9p: Remove some unneeded #include
I'd go out a limb and write it as 'Remove unneeded idr.h #include',
but this nitpick aside this looks good to me;
I'll pick it for next cycle.
> The 9p fs does not use IDR or IDA functionalities. So there is no point in
> including <linux/idr.h>.
> Remove it.
>
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
--
Dominique
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] 9p: Remove some unneeded #include
2022-11-27 12:59 [PATCH] 9p: Remove some unneeded #include Christophe JAILLET
2022-11-27 13:17 ` Dominique Martinet
@ 2022-11-27 14:07 ` Christian Schoenebeck
2022-11-27 14:45 ` Christophe JAILLET
1 sibling, 1 reply; 4+ messages in thread
From: Christian Schoenebeck @ 2022-11-27 14:07 UTC (permalink / raw)
To: Eric Van Hensbergen, Latchesar Ionkov, Dominique Martinet,
Christophe JAILLET
Cc: linux-kernel, kernel-janitors, Christophe JAILLET, v9fs-developer
On Sunday, November 27, 2022 1:59:25 PM CET Christophe JAILLET wrote:
> The 9p fs does not use IDR or IDA functionalities. So there is no point in
> including <linux/idr.h>.
> Remove it.
>
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> ---
Right, it's used by net/9p/client.c only. Probably some more files in net/9p
could therefore be deflated as well. Anyway:
Reviewed-by: Christian Schoenebeck <linux_oss@crudebyte.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] 9p: Remove some unneeded #include
2022-11-27 14:07 ` Christian Schoenebeck
@ 2022-11-27 14:45 ` Christophe JAILLET
0 siblings, 0 replies; 4+ messages in thread
From: Christophe JAILLET @ 2022-11-27 14:45 UTC (permalink / raw)
To: Christian Schoenebeck, Eric Van Hensbergen, Latchesar Ionkov,
Dominique Martinet
Cc: linux-kernel, kernel-janitors, v9fs-developer
Le 27/11/2022 à 15:07, Christian Schoenebeck a écrit :
> On Sunday, November 27, 2022 1:59:25 PM CET Christophe JAILLET wrote:
>> The 9p fs does not use IDR or IDA functionalities. So there is no point in
>> including <linux/idr.h>.
>> Remove it.
>>
>> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
>> ---
>
> Right, it's used by net/9p/client.c only. Probably some more files in net/9p
> could therefore be deflated as well. Anyway:
>
> Reviewed-by: Christian Schoenebeck <linux_oss@crudebyte.com>
>
>
Hi,
i was unsure if net/9p and fs/9p should be patched at the same time or not.
I'll send another patch for net/9p.
CJ
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-11-27 14:45 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-27 12:59 [PATCH] 9p: Remove some unneeded #include Christophe JAILLET
2022-11-27 13:17 ` Dominique Martinet
2022-11-27 14:07 ` Christian Schoenebeck
2022-11-27 14:45 ` Christophe JAILLET
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.