On Wed, Jan 24, 2024 at 10:25:17AM -0800, Reinette Chatre wrote:
> This can be confusing since it implies that the patch you mention
> introduces the issue but instead the variable has been unneeded since
> the original:
> 92bd5a139033 ("x86/resctrl: Add interface to write mbm_total_bytes_config")
What I said. :)
> To help clarify you can mention this order of events and also add an
> appropriate "Fixes:" tag.
>
> > cocci warnings: (new ones prefixed by >>)
> >>> arch/x86/kernel/cpu/resctrl/rdtgroup.c:1621:5-8: Unneeded variable: "ret". Return " 0" on line 1655
> >
> > Fix the warning by removing the variable "ret" and returning 0 directly.
>
> cocci warning was spot on*. This fix is not just a change to "make a
> warning go away" but instead fixing an actual problem.
> It can just be "Remove the unneeded variable and return 0 directly".
I'll fix all up before applying.
> * I'll add a private setup with the goal to catch these earlier.
Except that it doesn't fire with the patch that added the code. It looks
like the cocci script needs adjustment...
Thx.
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
