2022-09-06 21:30:20

by David Howells

Subject: [PATCH] afs: Return -EAGAIN, not -EREMOTEIO, when a file already locked

Hi Linus,

Can you apply this please?

Thanks,
David
---
afs: Return -EAGAIN, not -EREMOTEIO, when a file already locked

When trying to get a file lock on an AFS file, the server may return
UAEAGAIN to indicate that the lock is already held. This is currently
translated by the default path to -EREMOTEIO. Translate it instead to
-EAGAIN so that we know we can retry it.

Signed-off-by: David Howells <[email protected]>
Reviewed-by: Jeffrey E Altman <[email protected]>
cc: Marc Dionne <[email protected]>
cc: [email protected]
Link: https://lore.kernel.org/r/166075761334.3533338.2591992675160918098.stgit@warthog.procyon.org.uk/
---
fs/afs/misc.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/fs/afs/misc.c b/fs/afs/misc.c
index 933e67fcdab1..805328ca5428 100644
--- a/fs/afs/misc.c
+++ b/fs/afs/misc.c
@@ -69,6 +69,7 @@ int afs_abort_to_error(u32 abort_code)
/* Unified AFS error table */
case UAEPERM: return -EPERM;
case UAENOENT: return -ENOENT;
+ case UAEAGAIN: return -EAGAIN;
case UAEACCES: return -EACCES;
case UAEBUSY: return -EBUSY;
case UAEEXIST: return -EEXIST;

2022-09-07 01:46:38

by Linus Torvalds

Subject: Re: [PATCH] afs: Return -EAGAIN, not -EREMOTEIO, when a file already locked

On Tue, Sep 6, 2022 at 5:09 PM David Howells <[email protected]> wrote:
>
> Can you apply this please?

Done.

Linus