Re: [PATCH] fs/afs/flock and fs/locks: Fix possible sleep-in-atomic bugs in posix_lock_file

From: J. Bruce Fields
Date: Wed Oct 11 2017 - 09:45:05 EST


On Wed, Oct 11, 2017 at 10:47:43AM +0100, David Howells wrote:
> J. Bruce Fields <bfields@xxxxxxxxxxxx> wrote:
>
> > Does that mean nobody's tested fcntl locking over afs since that change in
> > 2010?
>
> Quite feasibly not. I've been beating kAFS into shape and I'm aware of the
> lock thing. It's on the list after getting the core server rotation working
> properly since that will affect a whole bunch of code, including the locking
> code.
>
> I've more or less finished the server rotation bit. See:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git/log/?h=afs-2-experimental

Neat!

Well, pending a real fix I suppose it's probably clearest to leave the
locking code in an obviously broken state. Might be worth a comment
though.

--b.