On Sat, Jun 10, 2023 at 04:06:08AM +0000, Alan Huang wrote:
> We should exit the RCU read-side critical section before re-entering.
>
> Signed-off-by: Alan Huang <mmpgouride@xxxxxxxxx>
> ---
> Documentation/RCU/rculist_nulls.rst | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/RCU/rculist_nulls.rst b/Documentation/RCU/rculist_nulls.rst
> index 9a734bf54..4370ac864 100644
> --- a/Documentation/RCU/rculist_nulls.rst
> +++ b/Documentation/RCU/rculist_nulls.rst
> @@ -30,6 +30,7 @@ algorithms:
> obj = lockless_lookup(key);
> if (obj) {
> if (!try_get_ref(obj)) // might fail for free objects
> + rcu_read_unlock();
Good eyes! But don't we also need a pair of braces?
Thanx, Paul
> goto begin;
> /*
> * Because a writer could delete object, and a writer could
> --
> 2.34.1
>
