lkml.org 
[lkml]   [2023]   [Jul]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/5] srcu: Fix error handling in init_srcu_struct_fields()
From


On 7/26/23 17:07, Paul E. McKenney wrote:
>> - free_percpu(ssp->sda);
>> - ssp->sda = NULL;
>> - kfree(ssp->srcu_sup);
>> - return -ENOMEM;
>> - }
>> - } else {
>> + if (!init_srcu_struct_nodes(ssp, GFP_ATOMIC))
>> + goto err_free_sda;
>> + else
>> WRITE_ONCE(ssp->srcu_sup->srcu_size_state, SRCU_SIZE_BIG);
> Given that the "then" clause is a goto, what is the "else" clause doing
> for us?
>

Not much. Agreed we can get rid of "else" and reduce indent of the
WRITE_ONCE that follows.

Would you mind making this fixup in the patch for your apply, or would
you like me to refresh the patch? Let me know either way, thank you!

- Joel

\
 
 \ /
  Last update: 2023-07-27 05:04    [W:0.175 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site