Re: [PATCH V4 10/13] fs/xfs: Clean up locking in dax invalidate

From: Ira Weiny
Date: Fri Feb 21 2020 - 13:06:10 EST


On Fri, Feb 21, 2020 at 06:45:22PM +0100, Christoph Hellwig wrote:
> On Thu, Feb 20, 2020 at 04:41:31PM -0800, ira.weiny@xxxxxxxxx wrote:
> > From: Ira Weiny <ira.weiny@xxxxxxxxx>
> >
> > Define a variable to hold the lock flags to ensure that the correct
> > locks are returned or released on error.
>
> I don't see how this cleans up anything..

It ensures the correct flags are released in the error path without having to
add the flag in the next patch in 2 places.

Ira