On Mon, 6 Nov 2023 16:27:23 +0800 zhaimingbing <zhaimingbing@xxxxxxxxxxxxxxxxxxxx> wrote: > The long type should use "%ld" instead of "%lu". > > ... > > --- a/mm/hugetlb_cgroup.c > +++ b/mm/hugetlb_cgroup.c > @@ -696,7 +696,7 @@ static int __hugetlb_events_show(struct seq_file *seq, bool local) > else > max = atomic_long_read(&h_cg->events[idx][HUGETLB_MAX]); > > - seq_printf(seq, "max %lu\n", max); > + seq_printf(seq, "max %ld\n", max); > > return 0; > } Well. We can't have a negative number of events. If we had an atomic_ulong_t, we'd be using that for ->events[]. Perhaps giving `max' an unsigned long type would better represent the intent in there.
