Patchwork [TRIVIAL,V2] Doc: percpu_counter doc code for better readability

login
register
mail settings
Submitter Jesper Dangaard Brouer
Date Feb. 4, 2013, 10:14 p.m.
Message ID <20130204221356.29548.35746.stgit@dragon>
Download mbox | patch
Permalink /patch/218097/
State Not Applicable
Delegated to: David Miller
Headers show

Comments

Tejun Heo - Feb. 4, 2013, 10:10 p.m.
On Mon, Feb 4, 2013 at 2:14 PM, Jesper Dangaard Brouer
<brouer@redhat.com> wrote:
> Help people reading the percpu_counter code, to notice the ifdef
> else statement that seperates CONFIG_SMP.
>
> Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
> Acked-by: Tejun Heo <tj@kernel.org>

Jiri, can you please take this?

Thank you.
Jesper Dangaard Brouer - Feb. 4, 2013, 10:14 p.m.
Help people reading the percpu_counter code, to notice the ifdef
else statement that seperates CONFIG_SMP.

Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Acked-by: Tejun Heo <tj@kernel.org>
---

 include/linux/percpu_counter.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Jiri Kosina - Feb. 5, 2013, 10:54 a.m.
On Mon, 4 Feb 2013, Tejun Heo wrote:

> > Help people reading the percpu_counter code, to notice the ifdef
> > else statement that seperates CONFIG_SMP.
> >
> > Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
> > Acked-by: Tejun Heo <tj@kernel.org>
> 
> Jiri, can you please take this?

Absolutely, now applied. Thanks,

Patch

diff --git a/include/linux/percpu_counter.h b/include/linux/percpu_counter.h
index b9df9ed..d5dd465 100644
--- a/include/linux/percpu_counter.h
+++ b/include/linux/percpu_counter.h
@@ -83,7 +83,7 @@  static inline int percpu_counter_initialized(struct percpu_counter *fbc)
 	return (fbc->counters != NULL);
 }
 
-#else
+#else /* !CONFIG_SMP */
 
 struct percpu_counter {
 	s64 count;