percpu_ref: introduce PERCPU_REF_ALLOW_REINIT flag
authorRoman Gushchin <guro@fb.com>
Tue, 7 May 2019 17:01:47 +0000 (10:01 -0700)
committerDennis Zhou <dennis@kernel.org>
Thu, 9 May 2019 17:49:47 +0000 (10:49 -0700)
In most cases percpu reference counters are not switched to the
percpu mode after they reach the atomic mode. Some obvious exceptions
are reference counters which are initialized into the atomic
mode (using PERCPU_REF_INIT_ATOMIC and PERCPU_REF_INIT_DEAD flags),
and there are few other exceptions.

But in most cases there is no way back, and once the reference counter
is switched to the atomic mode, there is no reason to wait for
percpu_ref_exit() to release the percpu memory. Of course, the size
of a single counter is not so big, but because it can pin the whole
percpu block in memory, the memory footprint can be noticeable
(e.g. on my 32 CPUs machine a percpu block is 8Mb large).

To make releasing of the percpu memory as early as possible, let's
introduce the PERCPU_REF_ALLOW_REINIT flag with the following semantics:
it has to be set in order to switch a percpu reference counter to the
percpu mode after the initialization. PERCPU_REF_INIT_ATOMIC and
PERCPU_REF_INIT_DEAD flags will implicitly assume PERCPU_REF_ALLOW_REINIT.

This patch doesn't introduce any functional change to avoid any
regressions. It will be done later in the patchset after adjusting
all call sites, which are reviving percpu counters.

Signed-off-by: Roman Gushchin <guro@fb.com>
Acked-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Dennis Zhou <dennis@kernel.org>
include/linux/percpu-refcount.h

index b297cd1cd4f190ccb36a75d73ec2d56a3300c2c0..0f0240af8520d9d5ab3216b3208ad12f944ba79a 100644 (file)
@@ -75,14 +75,21 @@ enum {
         * operation using percpu_ref_switch_to_percpu().  If initialized
         * with this flag, the ref will stay in atomic mode until
         * percpu_ref_switch_to_percpu() is invoked on it.
+        * Implies ALLOW_REINIT.
         */
        PERCPU_REF_INIT_ATOMIC  = 1 << 0,
 
        /*
         * Start dead w/ ref == 0 in atomic mode.  Must be revived with
-        * percpu_ref_reinit() before used.  Implies INIT_ATOMIC.
+        * percpu_ref_reinit() before used.  Implies INIT_ATOMIC and
+        * ALLOW_REINIT.
         */
        PERCPU_REF_INIT_DEAD    = 1 << 1,
+
+       /*
+        * Allow switching from atomic mode to percpu mode.
+        */
+       PERCPU_REF_ALLOW_REINIT = 1 << 2,
 };
 
 struct percpu_ref {