RE: [PATCH v8 09/21] x86/resctrl: Remove a newline to avoid confusing the code move script

[Shaopeng Tan (Fujitsu)]

Hello 
> The resctrl filesystem code will shortly be moved to /fs/. This involves splitting
> all the existing files, with some functions remaining under arch/x86, and others
> moving to fs/resctrl.
> 
> To make this reproducible, a python script does the heavy lif^W
> copy-and-paste. This involves some clunky parsing of C code.
Is "lif^W" a typo?

Best regards,
Shaopeng TAN

> The parser gets confused by the newline after this #ifdef.
> Just remove it.
> 
> Signed-off-by: James Morse <james.morse@xxxxxxx>
> Tested-by: Carl Worth <carl@xxxxxxxxxxxxxxxxxxxxxx> # arm64
> Tested-by: Shaopeng Tan <tan.shaopeng@xxxxxxxxxxxxxx>
> Tested-by: Peter Newman <peternewman@xxxxxxxxxx>
> Tested-by: Amit Singh Tomar <amitsinght@xxxxxxxxxxx> # arm64
> Tested-by: Shanker Donthineni <sdonthineni@xxxxxxxxxx> # arm64
> Tested-by: Babu Moger <babu.moger@xxxxxxx>
> Reviewed-by: Shaopeng Tan <tan.shaopeng@xxxxxxxxxxxxxx>
> Reviewed-by: Tony Luck <tony.luck@xxxxxxxxx>
> Reviewed-by: Reinette Chatre <reinette.chatre@xxxxxxxxx>
> Reviewed-by: Fenghua Yu <fenghuay@xxxxxxxxxx>
> ---
>  arch/x86/kernel/cpu/resctrl/rdtgroup.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/x86/kernel/cpu/resctrl/rdtgroup.c
> b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
> index aa0bc57e1c7f..d61b7137f443 100644
> --- a/arch/x86/kernel/cpu/resctrl/rdtgroup.c
> +++ b/arch/x86/kernel/cpu/resctrl/rdtgroup.c
> @@ -899,7 +899,6 @@ static int rdtgroup_rmid_show(struct kernfs_open_file
> *of,  }
> 
>  #ifdef CONFIG_PROC_CPU_RESCTRL
> -
>  /*
>   * A task can only be part of one resctrl control group and of one monitor
>   * group which is associated to that control group.
> --
> 2.20.1