| Message ID | 20231107211307.14279-1-zhaimingbing@cmss.chinamobile.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [V2] mm:hugetlb_cgroup: Optimize variable initialization | expand |
On 11/7/23 13:13, zhaimingbing wrote: > Initialize 'max' with 'unsigned long' instead of 'long'. > There is no variable initialization here, so the $Subject does not match what is being changed. Also there is no description of Why this change is needed. > Signed-off-by: zhaimingbing <zhaimingbing@cmss.chinamobile.com> > --- > mm/hugetlb_cgroup.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/hugetlb_cgroup.c b/mm/hugetlb_cgroup.c > index dedd2edb0..d4ca2ac55 100644 > --- a/mm/hugetlb_cgroup.c > +++ b/mm/hugetlb_cgroup.c > @@ -685,7 +685,7 @@ static char *mem_fmt(char *buf, int size, unsigned long hsize) > static int __hugetlb_events_show(struct seq_file *seq, bool local) > { > int idx; > - long max; > + unsigned long max; > struct cftype *cft = seq_cft(seq); > struct hugetlb_cgroup *h_cg = hugetlb_cgroup_from_css(seq_css(seq)); >
diff --git a/mm/hugetlb_cgroup.c b/mm/hugetlb_cgroup.c index dedd2edb0..d4ca2ac55 100644 --- a/mm/hugetlb_cgroup.c +++ b/mm/hugetlb_cgroup.c @@ -685,7 +685,7 @@ static char *mem_fmt(char *buf, int size, unsigned long hsize) static int __hugetlb_events_show(struct seq_file *seq, bool local) { int idx; - long max; + unsigned long max; struct cftype *cft = seq_cft(seq); struct hugetlb_cgroup *h_cg = hugetlb_cgroup_from_css(seq_css(seq));
Initialize 'max' with 'unsigned long' instead of 'long'. Signed-off-by: zhaimingbing <zhaimingbing@cmss.chinamobile.com> --- mm/hugetlb_cgroup.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)