Received-SPF: pass (google.com: domain of linux-kernel+bounces-37510-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <39c4501e-4937-49de-b12b-742e6201df6f@intel.com>
Date: Wed, 24 Jan 2024 10:25:17 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] x86/resctrl: Fix unneeded variable warning reported by
 kernel test robot
Content-Language: en-US
To: Babu Moger <babu.moger@amd.com>, <corbet@lwn.net>, <fenghua.yu@intel.com>,
	<tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>,
	<dave.hansen@linux.intel.com>
CC: <x86@kernel.org>, <hpa@zytor.com>, <paulmck@kernel.org>,
	<rdunlap@infradead.org>, <tj@kernel.org>, <peterz@infradead.org>,
	<yanjiewtw@gmail.com>, <kim.phillips@amd.com>, <lukas.bulwahn@gmail.com>,
	<seanjc@google.com>, <jmattson@google.com>, <leitao@debian.org>,
	<jpoimboe@kernel.org>, <rick.p.edgecombe@intel.com>,
	<kirill.shutemov@linux.intel.com>, <jithu.joseph@intel.com>,
	<kai.huang@intel.com>, <kan.liang@linux.intel.com>,
	<daniel.sneddon@linux.intel.com>, <pbonzini@redhat.com>,
	<sandipan.das@amd.com>, <ilpo.jarvinen@linux.intel.com>,
	<peternewman@google.com>, <maciej.wieczor-retman@intel.com>,
	<linux-doc@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<eranian@google.com>
References: <202401241810.jbd8Ipa1-lkp@intel.com>
 <84128a3c83654493f637b8349153af10d69e2752.1706118776.git.babu.moger@amd.com>
From: Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <84128a3c83654493f637b8349153af10d69e2752.1706118776.git.babu.moger@amd.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NytpeCtpUXJFM2k2MGdZNURvWVYvUDNBaEYzR3JwZndZZktEaU5nMmZhM2sy?=
 =?utf-8?B?a1EvSUJPaDh5OGlqekY3ZE8wYmVTZW5BK2pya1dhcmVLRHNFbVRGWmpYOHVr?=
 =?utf-8?B?Z0JZSlk1bFU1Mmh0T1ZYd1JFU1Q1eGhhVElJOEpscWlNaytwYXc3dTNCMzB6?=
 =?utf-8?B?UFdjaG0rNHpONXJJVDFTM1NWSStPVmowSmkzYlFMYWRMU3lqUW5rUWN0OGFM?=
 =?utf-8?B?Q1FKa0QyM3BadUw3Wlovem0rNUMxcmRKTUdwWU52L1VvUTMwOXpwWUhTU1Vw?=
 =?utf-8?B?ZTBFd3hGQ0ZEc2FvRHYyRVNSeEFCcTBmUklac1YwUDh1T28wdS9lNmJsVUha?=
 =?utf-8?B?Y3B0V1lPNDlZbjNqYVc2VXZINnV3bm4weFRkemUraGxxTnc4VE1ZaitZTUN4?=
 =?utf-8?B?WFJ5dXBwZ3hmQU5kRjRoTGMxbTR4TVBaV0RrbDNZblpWQkIyUURzYlR4TG5j?=
 =?utf-8?B?UzdOL3A5ckFLTVJ3aUFFdE0xaTNBNnJCMWVRZ3ptQ2YzVWJjQ1ljbXpwMkxn?=
 =?utf-8?B?TWFGdlQ1eGdycm9maUZxVkhpK2VkdlBCdW9YN1hZUjBFcndudXltamk2QU53?=
 =?utf-8?B?RWYzKzMwYzBHYklYYnBUTGxFajhCYWhxM0h4RXlYOVJrbndhYjUweUdqdjlL?=
 =?utf-8?B?Yzg5TFcrUkRNd2tBaFptTndFbTYyTVJHd05kbnJWdG5DSXZOR1FlWlVKM3Q1?=
 =?utf-8?B?eE41QnNsZmx4eFloZ3hFazc3UUdYVDNOL3lKZ1hkOUswekc0N0VOVU5ZVGV6?=
 =?utf-8?B?eUFtNmNGL3NiYlA1WkpuUDFNbEwzRVNpUmhVZk5CYSt6TFRicWdqbmhsNHlW?=
 =?utf-8?B?Z01OTncrZW1RSVVwMzlHSTlML05oaWd5Y2lMK1IrMGlzaFhyVU5udk45RGRK?=
 =?utf-8?B?N3ZnSFJLWnBzK1pheWlRNHJlL0tGSDlQNHhSeC9nVlp0TTJPU0lUTnFtL21E?=
 =?utf-8?B?VVlTdVc0RHJmQXBDajFERmRsOFpQbWo1NURoWkhNUWo2Qlk5WDJ1TFRQL0Nh?=
 =?utf-8?B?cGx2VzJRR0pmZnk0YmRDN21rMnJyanJlc2l5cUN1RVB3L0lNZjd1eGp3VG51?=
 =?utf-8?B?K3ZtcmNocDdyVGxpUk5yVnd0UjdTaHJ5bHRmNllzY0RjOWxpTVM2RVZxaGMx?=
 =?utf-8?B?dDc4L3NpTFNmMzltaTVCTnlOTFIzSWlJVlduQWgvdjA2SGkvTm16YWdRUDVL?=
 =?utf-8?B?TWo1V0xBMDdRdzdSYmRUV3pob2Q0QWc4QlE1ejVIZTBNRUdvOWI2V2tERzhK?=
 =?utf-8?B?aFdoMkk4d05ndjhNUkRDcy84NUhuSTZhQlFHMHFqL1pKUS9DTCtHbmdBbElv?=
 =?utf-8?B?V3hib2dYR2lsbUkyemtPallDZHRPSkJNRVRNejhVYVJuem9uVjJOZkpUSDBv?=
 =?utf-8?B?bnpMdlZiVnBUQVordFpVMkdzcnd5eGxGajNmek04TEtleVkvSFIxaUJlVUYz?=
 =?utf-8?B?SDR2ZzcrTEVIRktxbXMvVGQ3dUoxZk9hR21ZZElZQjhpZHVrZE5CN2N5aFg0?=
 =?utf-8?B?S0M3SmpXYXJqNFhRTFBnSjFENlllRG81bGJheklQK2hPQU81MGlGMHZsSUtx?=
 =?utf-8?B?cVM0RitvOXNpaXN3N21SQ2krdXBoNlkxRzlpQmNwb2ZNd0RhY3ltVjduMUpI?=
 =?utf-8?B?QWdCQzhRcnBqN2RVYWpHcmY3Tlo5b1RzNFhGaThCZWF3alJjK1Z6N3k1ZGNL?=
 =?utf-8?B?Z0dzb1NWaWFZVmgxRVVXbk9kbVM2UW1rNDNScms5ZnhXMTROYW1uVTlGTm1D?=
 =?utf-8?B?MDEvUEszcW9jVG9qZVE0ZkJoQXhGRXMzaWl2dGROaUJCbEoyeStBejd3VmJr?=
 =?utf-8?B?UE83Z3ZHcndOWi9ucW8xRnZrMzA1WEkrS0ZEUmRHbGl6Qk9DNnoxNllVWmNE?=
 =?utf-8?B?dytCbzYvTjZHVmUvb3ptc0pLWFhlUmVUcEpJSzFHaG92SzNmNnZqNHdlRFhx?=
 =?utf-8?B?dG5lays4amc4MytMK2lNakE3YzZuVkcxMitOSktZMHZNZytDbjJ4OFR3d0tj?=
 =?utf-8?B?TkFWSThMM3dBempIL3hmbVlSWExkdDFKWStwbjJmOVEya2pUejJTWUxRL3o3?=
 =?utf-8?B?anB4T243cElORVg2dG41bCtWUDJqMHg3VDRlMzA5OWd3Q1V0cUtQYlpicXd3?=
 =?utf-8?B?TmEwaGphTFZGZWQwU2MycEhkOVdRRHlWdmRUVVhvN2V0aUpFa3Y5S3k2M1hE?=
 =?utf-8?B?RkE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e6d7ac80-1fba-4029-a21a-08dc1d09d226
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jan 2024 18:25:19.8981
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: zIJjYi/3sdjeP/lh9HN7R1jo0ASl877KBQVp2A3qhiF3l2B7JNh7pmfjC5h2Fcq2pYoRYz0FTcKOtl5a1Z8uEeXw9C8fueOxTssbBLGEXb8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR11MB8319
X-OriginatorOrg: intel.com

Hi Babu,

Thank you for fixing this so promptly.

I think the subject can just be:
"x86/resctrl: Remove unneeded variable"

On 1/24/2024 9:52 AM, Babu Moger wrote:
> kernel test robot reported the following warning after the commit
> 54e35eb8611c ("x86/resctrl: Read supported bandwidth sources from CPUID").

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")

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".

> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202401241810.jbd8Ipa1-lkp@intel.com/
> Signed-off-by: Babu Moger <babu.moger@amd.com>

Reinette

* I'll add a private setup with the goal to catch these earlier.