Re: [PATCH v2 07/11] arch/x86: Initialize the resource functions that are different

From: Moger, Babu
Date: Tue Oct 09 2018 - 16:45:36 EST




On 10/09/2018 12:18 PM, Reinette Chatre wrote:
> Hi Babu,
>
> On 10/5/2018 1:55 PM, Moger, Babu wrote:
>> diff --git a/arch/x86/kernel/cpu/rdt_ctrlmondata.c b/arch/x86/kernel/cpu/rdt_ctrlmondata.c
>> index 812cc5c5e39e..ee3e8389d8d2 100644
>> --- a/arch/x86/kernel/cpu/rdt_ctrlmondata.c
>> +++ b/arch/x86/kernel/cpu/rdt_ctrlmondata.c
>> @@ -64,7 +64,7 @@ static bool bw_validate(char *buf, unsigned long *data, struct rdt_resource *r)
>> return true;
>> }
>>
>> -int parse_bw(struct rdt_parse_data *data, struct rdt_resource *r,
>> +int parse_bw_intel(struct rdt_parse_data *data, struct rdt_resource *r,
>> struct rdt_domain *d)
>> {
>> unsigned long bw_val;
>>
>
> Just a small nit - could you please check that the alignment matches the
> open parenthesis as you have done with all the other similar changes in
> this patch?

Ok. Sure. will take care of it. Thanks.

>
> Reinette
>