On Thu, Feb 13, 2014 at 04:32:07PM -0800, Paul E. McKenney wrote:
> On Tue, Feb 11, 2014 at 04:10:12PM -0500, Paul Gortmaker wrote:
> > To fix:
> >
> > kernel/ksysfs.c:143:5: warning: symbol 'rcu_expedited' was not declared.
> > Should it be static?
> >
> > Cc: "Paul E. McKenney" <paul...@linux.vnet.ibm.com>
> > Signed-off-by: Paul Gortmaker <paul.gortma...@windriver.com>
>
> Queued, thank you!
Upon further review, I moved the rcu_expedited declaration to
include/linux/rcupdate.h in order to avoid including RCU-specific
kernel/rcu/rcu.h into non-RCU files. This resulted in the patch below.
Please let me know if you have any objections to this change.
Thanx, Paul
------------------------------------------------------------------------
rcu: Fix sparse warning for rcu_expedited from kernel/ksysfs.c
This commit fixes the follwoing warning:
kernel/ksysfs.c:143:5: warning: symbol 'rcu_expedited' was not declared. Should
it be static?
Signed-off-by: Paul Gortmaker <paul.gortma...@windriver.com>
[ paulmck: Moved the declaration to include/linux/rcupdate.h to avoid
including the RCU-internal rcu.h file outside of RCU. ]
Signed-off-by: Paul E. McKenney <paul...@linux.vnet.ibm.com>
diff --git a/include/linux/rcupdate.h b/include/linux/rcupdate.h
index 55b605b49315..e4be6263807e 100644
--- a/include/linux/rcupdate.h
+++ b/include/linux/rcupdate.h
@@ -46,6 +46,7 @@
#include <linux/compiler.h>
#include <asm/barrier.h>
+extern int rcu_expedited; /* for sysctl */
#ifdef CONFIG_RCU_TORTURE_TEST
extern int rcutorture_runnable; /* for sysctl */
#endif /* #ifdef CONFIG_RCU_TORTURE_TEST */
diff --git a/kernel/ksysfs.c b/kernel/ksysfs.c
index d945a949760f..e660964086e2 100644
--- a/kernel/ksysfs.c
+++ b/kernel/ksysfs.c
@@ -19,6 +19,8 @@
#include <linux/sched.h>
#include <linux/capability.h>
+#include <linux/rcupdate.h> /* rcu_expedited */
+
#define KERNEL_ATTR_RO(_name) \
static struct kobj_attribute _name##_attr = __ATTR_RO(_name)
diff --git a/kernel/rcu/rcu.h b/kernel/rcu/rcu.h
index 1bd787fddcb2..af2e60a8425d 100644
--- a/kernel/rcu/rcu.h
+++ b/kernel/rcu/rcu.h
@@ -116,8 +116,6 @@ static inline bool __rcu_reclaim(const char *rn, struct
rcu_head *head)
}
}
-extern int rcu_expedited;
-
#ifdef CONFIG_RCU_STALL_COMMON
extern int rcu_cpu_stall_suppress;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
