[PATCH tip/core/rcu 07/12] rcu: Rearrange locking in rcu_start_gp()

2013-01-26 Thread Paul E. McKenney
From: "Paul E. McKenney" 

If CPUs are to give prior notice of needed grace periods, it will be
necessary to invoke rcu_start_gp() without dropping the root rcu_node
structure's ->lock.  This commit takes a first step in this direction
by moving the release of this lock to the end of rcu_start_gp().

Signed-off-by: Paul E. McKenney 
Signed-off-by: Paul E. McKenney 
---
 kernel/rcutree.c |6 ++
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/kernel/rcutree.c b/kernel/rcutree.c
index 7b1d776..2c6a931 100644
--- a/kernel/rcutree.c
+++ b/kernel/rcutree.c
@@ -1521,16 +1521,14 @@ rcu_start_gp(struct rcu_state *rsp, unsigned long flags)
raw_spin_unlock_irqrestore(>lock, flags);
return;
}
-
rsp->gp_flags = RCU_GP_FLAG_INIT;
-   raw_spin_unlock(>lock); /* Interrupts remain disabled. */
 
/* Ensure that CPU is aware of completion of last grace period. */
-   rcu_process_gp_end(rsp, rdp);
-   local_irq_restore(flags);
+   __rcu_process_gp_end(rsp, rdp->mynode, rdp);
 
/* Wake up rcu_gp_kthread() to start the grace period. */
wake_up(>gp_wq);
+   raw_spin_unlock_irqrestore(>lock, flags);
 }
 
 /*
-- 
1.7.8

--
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/


[PATCH tip/core/rcu 07/12] rcu: Rearrange locking in rcu_start_gp()

2013-01-26 Thread Paul E. McKenney
From: Paul E. McKenney paul.mcken...@linaro.org

If CPUs are to give prior notice of needed grace periods, it will be
necessary to invoke rcu_start_gp() without dropping the root rcu_node
structure's -lock.  This commit takes a first step in this direction
by moving the release of this lock to the end of rcu_start_gp().

Signed-off-by: Paul E. McKenney paul.mcken...@linaro.org
Signed-off-by: Paul E. McKenney paul...@linux.vnet.ibm.com
---
 kernel/rcutree.c |6 ++
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/kernel/rcutree.c b/kernel/rcutree.c
index 7b1d776..2c6a931 100644
--- a/kernel/rcutree.c
+++ b/kernel/rcutree.c
@@ -1521,16 +1521,14 @@ rcu_start_gp(struct rcu_state *rsp, unsigned long flags)
raw_spin_unlock_irqrestore(rnp-lock, flags);
return;
}
-
rsp-gp_flags = RCU_GP_FLAG_INIT;
-   raw_spin_unlock(rnp-lock); /* Interrupts remain disabled. */
 
/* Ensure that CPU is aware of completion of last grace period. */
-   rcu_process_gp_end(rsp, rdp);
-   local_irq_restore(flags);
+   __rcu_process_gp_end(rsp, rdp-mynode, rdp);
 
/* Wake up rcu_gp_kthread() to start the grace period. */
wake_up(rsp-gp_wq);
+   raw_spin_unlock_irqrestore(rnp-lock, flags);
 }
 
 /*
-- 
1.7.8

--
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/