Re: [lustre-devel] [PATCH 030/124] staging: lustre: llite: Replace write mutex with range lock

From: Dilger, Andreas
Date: Mon Sep 19 2016 - 05:25:57 EST

On Sep 19, 2016, at 09:28, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> On Sun, Sep 18, 2016 at 04:37:29PM -0400, James Simmons wrote:
>> + * Range lock is used to allow multiple threads writing a single shared
>> + * file given each thread is writing to a non-overlapping portion of the
>> + * file.
>> + *
>> + * Refer to the possible upstream kernel version of range lock by
>> + * Jan Kara <jack@xxxxxxx>:
>> + *
>> + * This file could later replaced by the upstream kernel version.
> It doesn't look like range_lock ever got accepted in the kernel tree,
> any idea what happened to it? Having a per-filesystem lock type seems
> odd to me...

I've added Jan and linux-fsdevel to the CC list to see what interest
there is in the range locking implementaion. At the time we added this
to Lustre it appeared that this was moving nicely torward landing, but
it seems to have stalled.

I think the range locking implementation is fairly generic, and if there
are other users in the kernel it could easily be pulled out of the staging
dir into vfs/. I'm not against it going into vfs/ directly either, but
not sure whether that is acceptable if the only user is in staging.

Cheers, Andreas