[GIT PULL] afs, rxrpc: Clean up refcounting on afs_cell and afs_server records
David Howells
dhowells at redhat.com
Fri Feb 28 06:45:18 PST 2025
Jakub Kicinski <kuba at kernel.org> wrote:
> fs/afs/cell.c:203:5-22: WARNING: Unsigned expression compared with zero: cell -> dynroot_ino < 0
I'll make this change:
--- a/fs/afs/cell.c
+++ b/fs/afs/cell.c
@@ -200,7 +200,7 @@ static struct afs_cell *afs_alloc_cell(struct afs_net *net,
atomic_inc(&net->cells_outstanding);
cell->dynroot_ino = idr_alloc_cyclic(&net->cells_dyn_ino, cell,
2, INT_MAX / 2, GFP_KERNEL);
- if (cell->dynroot_ino < 0)
+ if ((int)cell->dynroot_ino < 0)
goto error;
cell->debug_id = atomic_inc_return(&cell_debug_id);
to patch 2 ("afs: Change dynroot to create contents on demand").
I'm not sure why gcc didn't warn about this - I'm sure it used to.
David
More information about the linux-afs
mailing list