acct: fix build warning when ACCT_VERSION != 3

PATCH In review Luis Henriques 2 participants 3 messages Oct 1, 2014 – Oct 2, 2014

lore

Share & tools

lore

Luis Henriques

author

2014-10-01 23:06 UTC #0 lore raw

PATCH 1/1 acct: fix build warning when ACCT_VERSION != 3

struct pid_namespace *ns is used only when ACCT_VERSION is 3.

kernel/acct.c:475:24: warning: unused variable 'ns' [-Wunused-variable]

Signed-off-by Luis Henriques <[email protected]>

---
 kernel/acct.c | 2 ++
 1 file changed, 2 insertions(+)

kernel/acct.c +2 -0

index b4c667d22e79..5f277f5c5e29 100644
@@ -472,7 +472,9 @@ static void do_acct_process(struct bsd_acct_struct *acct)
 	acct_t ac;
 	unsigned long flim;
 	const struct cred *orig_cred;
+#if ACCT_VERSION == 3
 	struct pid_namespace *ns = acct->ns;
+#endif
 	struct file *file = acct->file;
 
 	/*

Signature · 1 line

2.1.0

Andrew Morton

2014-10-02 21:09 UTC #1 lore raw

Quoted 14 lines · On Thu, 2 Oct 2014 00:06:53 +0100 Luis Henriques…

struct pid_namespace *ns is used only when ACCT_VERSION is 3.
kernel/acct.c:475:24: warning: unused variable 'ns' [-Wunused-variable]
Signed-off-by Luis Henriques <[email protected]>
---
 kernel/acct.c | 2 ++
 1 file changed, 2 insertions(+)
kernel/acct.c +0 -0
index b4c667d22e79..5f277f5c5e29 100644

kernel/acct.c static void do_acct_process(struct bsd_acct_struct *acct) · line 480

Quoted 9 lines

kernel/acct.c +2 -0

@@ -472,7 +472,9 @@ static void do_acct_process(struct bsd_acct_struct *acct)
 	acct_t ac;
 	unsigned long flim;
 	const struct cred *orig_cred;
+#if ACCT_VERSION == 3
 	struct pid_namespace *ns = acct->ns;
+#endif
 	struct file *file = acct->file;
 
 	/*

It's always a good idea to check linux-next. This has been addressed by
http://ozlabs.org/~akpm/mmots/broken-out/acct-eliminate-compile-warning.patch
http://ozlabs.org/~akpm/mmots/broken-out/acct-eliminate-compile-warning-fix.patch

Luis Henriques

author

2014-10-02 21:55 UTC #2 lore raw

Quoted 24 lines · Andrew Morton

Quoted 22 lines · On Thu, 2 Oct 2014 00:06:53 +0100 Luis Henriques…
struct pid_namespace *ns is used only when ACCT_VERSION is 3.
kernel/acct.c:475:24: warning: unused variable 'ns' [-Wunused-variable]
Signed-off-by Luis Henriques <[email protected]>
---
 kernel/acct.c | 2 ++
 1 file changed, 2 insertions(+)
kernel/acct.c +2 -0
index b4c667d22e79..5f277f5c5e29 100644
@@ -472,7 +472,9 @@ static void do_acct_process(struct bsd_acct_struct *acct)
 	acct_t ac;
 	unsigned long flim;
 	const struct cred *orig_cred;
+#if ACCT_VERSION == 3
 	struct pid_namespace *ns = acct->ns;
+#endif
 	struct file *file = acct->file;
 

Quoted 7 lines

kernel/acct.c static void do_acct_process(struct bsd_acct_struct *acct) · line 480
Quoted 1 line
kernel/acct.c +0 -0
 	/*
It's always a good idea to check linux-next. This has been addressed by
http://ozlabs.org/~akpm/mmots/broken-out/acct-eliminate-compile-warning.patch
http://ozlabs.org/~akpm/mmots/broken-out/acct-eliminate-compile-warning-fix.patch

Ah, awesome! I missed these patches. Thanks Andrew.

Cheers,
--
Luís