Re: [PATCH v2] arch/sparc: fix unused variable warning

From: Alex Shi

Date: Wed Jan 07 2026 - 23:26:30 EST




On 2026/1/7 22:46, Matthew Wilcox wrote:
On Wed, Jan 07, 2026 at 06:41:44PM +0800,alexs@xxxxxxxxxx wrote:
From: Alex Shi<alexs@xxxxxxxxxx>

arch/sparc/mm/init_64.c: In function 'arch_hugetlb_valid_size':
arch/sparc/mm/init_64.c:361:24: warning: variable 'hv_pgsz_idx' set but not used [-Wunused-but-set-variable]
361 | unsigned short hv_pgsz_idx;
| ^~~~~~~~~~~
Looks like it's been unused since introduction in c7d9f77d33a7 ?

As such, why cc all these people who have absolutely nothing to do with
it? Use your brain, not a stupid script.

Thanks for the feedback, Matthew. You're right—the script was too aggressive for such a localized fix.

I used scripts/get_maintainer.pl without enough manual filtering. Are there specific parameters or workflows you recommend to keep the CC list lean for trivial cleanups like this?

I'll be more surgical with the CC list on the next version.

Thanks
Alex