On Mon, 2024-07-01 at 12:50 +0200, Jan Kara wrote: > Atomic types should better be initialized with atomic_long_set() > instead > of relying on zeroing done by kzalloc(). Clean this up. > > Reviewed-by: Sagi Grimberg <sagi@xxxxxxxxxxx> > Signed-off-by: Jan Kara <jack@xxxxxxx> > --- > fs/nfs/client.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/nfs/client.c b/fs/nfs/client.c > index de77848ae654..3b252dceebf5 100644 > --- a/fs/nfs/client.c > +++ b/fs/nfs/client.c > @@ -994,6 +994,8 @@ struct nfs_server *nfs_alloc_server(void) > > server->change_attr_type = NFS4_CHANGE_TYPE_IS_UNDEFINED; > > + atomic_long_set(&server->writeback, 0); > + > ida_init(&server->openowner_id); > ida_init(&server->lockowner_id); > pnfs_init_server(server); It seems a little pointless, but ok. Acked-by: Jeff Layton <jlayton@xxxxxxxxxx>