[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7367ec47-cc1b-4b5b-a406-90853a11774b@nvidia.com>
Date: Thu, 6 Mar 2025 18:44:17 -0800
From: Fenghua Yu <fenghuay@...dia.com>
To: James Morse <james.morse@....com>, x86@...nel.org,
linux-kernel@...r.kernel.org
Cc: Reinette Chatre <reinette.chatre@...el.com>,
Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>,
Borislav Petkov <bp@...en8.de>, H Peter Anvin <hpa@...or.com>,
Babu Moger <Babu.Moger@....com>, shameerali.kolothum.thodi@...wei.com,
D Scott Phillips OS <scott@...amperecomputing.com>,
carl@...amperecomputing.com, lcherian@...vell.com,
bobo.shaobowang@...wei.com, tan.shaopeng@...itsu.com,
baolin.wang@...ux.alibaba.com, Jamie Iles <quic_jiles@...cinc.com>,
Xin Hao <xhao@...ux.alibaba.com>, peternewman@...gle.com,
dfustini@...libre.com, amitsinght@...vell.com,
David Hildenbrand <david@...hat.com>, Rex Nie <rex.nie@...uarmicro.com>,
Dave Martin <dave.martin@....com>, Koba Ko <kobak@...dia.com>,
Shanker Donthineni <sdonthineni@...dia.com>
Subject: Re: [PATCH v7 48/49] fs/resctrl: Change internal.h's header guard
macros
Hi, James,
On 2/28/25 11:59, James Morse wrote:
> The code in internal.h was moved by a script. The script didn't know to
> change the header guard macros. Do that.
As for "The script ...", people may forget or don't know the context of
the script in the future.
Is it better to merge this patch into patch #45 to avoid future confusion?
> Signed-off-by: James Morse <james.morse@....com>
> ---
> Changes since v6:
> * This patch is new.
> ---
> fs/resctrl/internal.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/fs/resctrl/internal.h b/fs/resctrl/internal.h
> index f1763a13715f..ec3863d18f68 100644
> --- a/fs/resctrl/internal.h
> +++ b/fs/resctrl/internal.h
> @@ -1,6 +1,6 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> -#ifndef _ASM_X86_RESCTRL_INTERNAL_H
> -#define _ASM_X86_RESCTRL_INTERNAL_H
> +#ifndef _FS_RESCTRL_INTERNAL_H
> +#define _FS_RESCTRL_INTERNAL_H
>
> #include <linux/resctrl.h>
> #include <linux/sched.h>
> @@ -432,4 +432,4 @@ static inline int rdtgroup_pseudo_lock_create(struct rdtgroup *rdtgrp)
> static inline void rdtgroup_pseudo_lock_remove(struct rdtgroup *rdtgrp) { }
> #endif /* CONFIG_RESCTRL_FS_PSEUDO_LOCK */
>
> -#endif /* _ASM_X86_RESCTRL_INTERNAL_H */
> +#endif /* _FS_RESCTRL_INTERNAL_H */
Thanks.
-Fenghua
Powered by blists - more mailing lists