lkml.org 
[lkml]   [2023]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/5] srcu: Fix error handling in init_srcu_struct_fields()
On Wed, Jul 26, 2023 at 11:04:04PM -0400, Joel Fernandes wrote:
>
>
> 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!

Please include it with your next series, which has at least one other
change anyway. ;-)

Thanx, Paul

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