diff options
author | David Howells <dhowells@redhat.com> | 2023-10-25 19:53:33 +0300 |
---|---|---|
committer | David Howells <dhowells@redhat.com> | 2023-12-24 18:22:53 +0300 |
commit | aa453becce5d1ae1b94b7fc22f47d7b05d22b14e (patch) | |
tree | 933b826c996b2688608d7108d04ff03d390b9585 /fs/afs/dir_silly.c | |
parent | 6f2ff7e89bd05677f4c08fccafcf625ca3e09c1c (diff) | |
download | linux-aa453becce5d1ae1b94b7fc22f47d7b05d22b14e.tar.xz |
afs: Simplify error handling
Simplify error handling a bit by moving it from the afs_addr_cursor struct
to the afs_operation and afs_vl_cursor structs and using the error
prioritisation function for accumulating errors from multiple sources (AFS
tries to rotate between multiple fileservers, some of which may be
inaccessible or in some state of offlinedness).
Signed-off-by: David Howells <dhowells@redhat.com>
cc: Marc Dionne <marc.dionne@auristor.com>
cc: linux-afs@lists.infradead.org
Diffstat (limited to 'fs/afs/dir_silly.c')
-rw-r--r-- | fs/afs/dir_silly.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/afs/dir_silly.c b/fs/afs/dir_silly.c index bb5807e87fa4..a1e581946b93 100644 --- a/fs/afs/dir_silly.c +++ b/fs/afs/dir_silly.c @@ -218,7 +218,7 @@ static int afs_do_silly_unlink(struct afs_vnode *dvnode, struct afs_vnode *vnode /* If there was a conflict with a third party, check the status of the * unlinked vnode. */ - if (op->error == 0 && (op->flags & AFS_OPERATION_DIR_CONFLICT)) { + if (op->cumul_error.error == 0 && (op->flags & AFS_OPERATION_DIR_CONFLICT)) { op->file[1].update_ctime = false; op->fetch_status.which = 1; op->ops = &afs_fetch_status_operation; |