On 12/9/24 11:52, Zhu Jun wrote: > Change f to unsigned int to resolve the mismatch in format specifiers for > snprintf(), where %u should be used for unsigned integers instead of %d > > Signed-off-by: Zhu Jun <zhujun2@xxxxxxxxxxxxxxxxxxxx> The patch title is now wrong... And you forgot to add a Fixes tag. Will fix that when applying. Thanks. > --- > Changes: > v2: > A better fix is to make f "unsigned int" as that is consistent with > g_nr_zones and the fact that up to "unsigned int" total number of zones are > allowed for a zoned device. > > Can you send a v2 ? Also please add a Fixes tag. > > fs/zonefs/super.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/zonefs/super.c b/fs/zonefs/super.c > index faf1eb878..695da258a 100644 > --- a/fs/zonefs/super.c > +++ b/fs/zonefs/super.c > @@ -791,7 +791,7 @@ static int zonefs_readdir_zgroup(struct file *file, > int fname_len; > char *fname; > ino_t ino; > - int f; > + unsigned int f; > > /* > * The size of zone group directories is equal to the number -- Damien Le Moal Western Digital Research
