* [PATCH v2] docs/RCU: Add the missing rcu_read_unlock()
@ 2023-06-13 0:57 Alan Huang
2023-06-13 3:16 ` Paul E. McKenney
0 siblings, 1 reply; 2+ messages in thread
From: Alan Huang @ 2023-06-13 0:57 UTC (permalink / raw)
To: paulmck; +Cc: corbet, rcu, linux-doc, Alan Huang
We should exit the RCU read-side critical section before re-entering.
Signed-off-by: Alan Huang <mmpgouride@gmail.com>
---
Documentation/RCU/rculist_nulls.rst | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/RCU/rculist_nulls.rst b/Documentation/RCU/rculist_nulls.rst
index 9a734bf54..0612a6387 100644
--- a/Documentation/RCU/rculist_nulls.rst
+++ b/Documentation/RCU/rculist_nulls.rst
@@ -29,8 +29,10 @@ algorithms:
rcu_read_lock()
obj = lockless_lookup(key);
if (obj) {
- if (!try_get_ref(obj)) // might fail for free objects
+ if (!try_get_ref(obj)) { // might fail for free objects
+ rcu_read_unlock();
goto begin;
+ }
/*
* Because a writer could delete object, and a writer could
* reuse these object before the RCU grace period, we
--
2.34.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] docs/RCU: Add the missing rcu_read_unlock()
2023-06-13 0:57 [PATCH v2] docs/RCU: Add the missing rcu_read_unlock() Alan Huang
@ 2023-06-13 3:16 ` Paul E. McKenney
0 siblings, 0 replies; 2+ messages in thread
From: Paul E. McKenney @ 2023-06-13 3:16 UTC (permalink / raw)
To: Alan Huang; +Cc: corbet, rcu, linux-doc
On Tue, Jun 13, 2023 at 12:57:01AM +0000, Alan Huang wrote:
> We should exit the RCU read-side critical section before re-entering.
>
> Signed-off-by: Alan Huang <mmpgouride@gmail.com>
Queued and pushed, thank you!
Thanx, Paul
> ---
> Documentation/RCU/rculist_nulls.rst | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/RCU/rculist_nulls.rst b/Documentation/RCU/rculist_nulls.rst
> index 9a734bf54..0612a6387 100644
> --- a/Documentation/RCU/rculist_nulls.rst
> +++ b/Documentation/RCU/rculist_nulls.rst
> @@ -29,8 +29,10 @@ algorithms:
> rcu_read_lock()
> obj = lockless_lookup(key);
> if (obj) {
> - if (!try_get_ref(obj)) // might fail for free objects
> + if (!try_get_ref(obj)) { // might fail for free objects
> + rcu_read_unlock();
> goto begin;
> + }
> /*
> * Because a writer could delete object, and a writer could
> * reuse these object before the RCU grace period, we
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-06-13 3:18 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-13 0:57 [PATCH v2] docs/RCU: Add the missing rcu_read_unlock() Alan Huang
2023-06-13 3:16 ` Paul E. McKenney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).