Re: [locks] 6d390e4b5d: will-it-scale.per_process_ops -96.6% regression

[Jeff Layton]

On Tue, 2020-03-17 at 09:45 +1100, NeilBrown wrote:
> > +
> > +	/*
> > +	 * Tell the world we're done with it - see comment at top
> > +	 * of this function
> 
> This comment might be misleading.  The world doesn't care.
> Only this thread cares where ->fl_blocker is NULL.  We need the release
> semantics when some *other* thread sets fl_blocker to NULL, not when
> this thread does.
> I don't think we need to spell that out and I'm not against using
> store_release here, but locks_delete_block cannot race with itself, so
> referring to the comment at the top of this function is misleading.
> 
> So:
>   Reviewed-by: NeilBrown <neilb@xxxxxxx>
> 
> but I'm not totally happy with the comments.
> 
> 

Thanks Neil. We can clean up the comments before merge. How about this
revision to the earlier patch? I took the liberty of poaching your your
proposed verbiage:

------------------8<---------------------