Searched hist:77041 (Results 1 - 5 of 5) sorted by relevance

/linux-master/drivers/net/ethernet/cavium/liquidio/
H A Docteon_mailbox.cdiff 77041e89 Fri Apr 28 06:57:15 MDT 2017 Dan Carpenter <dan.carpenter@oracle.com> liquidio: silence a locking static checker warning

Presumably we never hit this return, but static checkers complain that
we need to unlock so we may as well fix that.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Felix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
/linux-master/fs/nfs/
H A Dnfs4_fs.hdiff 8003d3c4 Thu May 01 04:28:45 MDT 2014 Jeff Layton <jlayton@kernel.org> nfs4: treat lock owners as opaque values

Do the following set of ops with a file on a NFSv4 mount:

exec 3>>/file/on/nfsv4
flock -x 3
exec 3>&-

You'll see the LOCK request go across the wire, but no LOCKU when the
file is closed.

What happens is that the fd is passed across a fork, and the final close
is done in a different process than the opener. That makes
__nfs4_find_lock_state miss finding the correct lock state because it
uses the fl_pid as a search key. A new one is created, and the locking
code treats it as a delegation stateid (because NFS_LOCK_INITIALIZED
isn't set).

The root cause of this breakage seems to be commit 77041ed9b49a9e
(NFSv4: Ensure the lockowners are labelled using the fl_owner and/or
fl_pid).

That changed it so that flock lockowners are allocated based on the
fl_pid. I think this is incorrect. flock locks should be "owned" by the
struct file, and that is already accounted for in the fl_owner field of
the lock request when it comes through nfs_flock.

This patch basically reverts the above commit and with it, a LOCKU is
sent in the above reproducer.

Signed-off-by: Jeff Layton <jlayton@poochiereds.net>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
diff 77041ed9 Wed Jun 30 22:49:11 MDT 2010 Trond Myklebust <Trond.Myklebust@netapp.com> NFSv4: Ensure the lockowners are labelled using the fl_owner and/or fl_pid

flock locks want to be labelled using the process pid, while posix locks
want to be labelled using the fl_owner.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
H A Dnfs4state.cdiff 8003d3c4 Thu May 01 04:28:45 MDT 2014 Jeff Layton <jlayton@kernel.org> nfs4: treat lock owners as opaque values

Do the following set of ops with a file on a NFSv4 mount:

exec 3>>/file/on/nfsv4
flock -x 3
exec 3>&-

You'll see the LOCK request go across the wire, but no LOCKU when the
file is closed.

What happens is that the fd is passed across a fork, and the final close
is done in a different process than the opener. That makes
__nfs4_find_lock_state miss finding the correct lock state because it
uses the fl_pid as a search key. A new one is created, and the locking
code treats it as a delegation stateid (because NFS_LOCK_INITIALIZED
isn't set).

The root cause of this breakage seems to be commit 77041ed9b49a9e
(NFSv4: Ensure the lockowners are labelled using the fl_owner and/or
fl_pid).

That changed it so that flock lockowners are allocated based on the
fl_pid. I think this is incorrect. flock locks should be "owned" by the
struct file, and that is already accounted for in the fl_owner field of
the lock request when it comes through nfs_flock.

This patch basically reverts the above commit and with it, a LOCKU is
sent in the above reproducer.

Signed-off-by: Jeff Layton <jlayton@poochiereds.net>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
diff 77041ed9 Wed Jun 30 22:49:11 MDT 2010 Trond Myklebust <Trond.Myklebust@netapp.com> NFSv4: Ensure the lockowners are labelled using the fl_owner and/or fl_pid

flock locks want to be labelled using the process pid, while posix locks
want to be labelled using the fl_owner.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
H A Dnfs4xdr.cdiff 77041ed9 Wed Jun 30 22:49:11 MDT 2010 Trond Myklebust <Trond.Myklebust@netapp.com> NFSv4: Ensure the lockowners are labelled using the fl_owner and/or fl_pid

flock locks want to be labelled using the process pid, while posix locks
want to be labelled using the fl_owner.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
H A Dnfs4proc.cdiff 77041ed9 Wed Jun 30 22:49:11 MDT 2010 Trond Myklebust <Trond.Myklebust@netapp.com> NFSv4: Ensure the lockowners are labelled using the fl_owner and/or fl_pid

flock locks want to be labelled using the process pid, while posix locks
want to be labelled using the fl_owner.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>

Completed in 926 milliseconds