diff options
| author | Jeff Layton <[email protected]> | 2015-11-29 13:46:14 +0000 |
|---|---|---|
| committer | J. Bruce Fields <[email protected]> | 2015-12-16 16:49:58 +0000 |
| commit | be20aa00c67102aaa54599518c086a2338b19f4c (patch) | |
| tree | f44c188279fe553b8b47f6398b7782e26ec60bbb /drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |
| parent | Linux 4.4-rc1 (diff) | |
| download | kernel-be20aa00c67102aaa54599518c086a2338b19f4c.tar.gz kernel-be20aa00c67102aaa54599518c086a2338b19f4c.zip | |
nfsd: don't hold ls_mutex across a layout recall
We do need to serialize layout stateid morphing operations, but we
currently hold the ls_mutex across a layout recall which is pretty
ugly. It's also unnecessary -- once we've bumped the seqid and
copied it, we don't need to serialize the rest of the CB_LAYOUTRECALL
vs. anything else. Just drop the mutex once the copy is done.
This was causing a "workqueue leaked lock or atomic" warning and an
occasional deadlock.
There's more work to be done here but this fixes the immediate
regression.
Fixes: cc8a55320b5f "nfsd: serialize layout stateid morphing operations"
Cc: [email protected]
Reported-by: Kinglong Mee <[email protected]>
Signed-off-by: Jeff Layton <[email protected]>
Signed-off-by: J. Bruce Fields <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c')
0 files changed, 0 insertions, 0 deletions