2021-03-17 20:33:55

by Reinette Chatre

[permalink] [raw]
Subject: Re: [PATCH] kernel: cpu: resctrl: Minor typo fix in the file pseudo_lock.c

Hi Bhaskar,

On 3/17/2021 12:33 PM, Bhaskar Chowdhury wrote:
> On 10:54 Wed 17 Mar 2021, Reinette Chatre wrote:
>> Thank you very much for catching this typo.
>>
>> My feedback [1] to a previous patch from you applies here also. The
>> prefix should be "x86/resctrl:" for contributions to this area.
>>
> Thanks for the heads up!

This was the third time I mention this to you.

> Do you want another revision of it, or will you
> take it as it is???

If accepted I am not the one who will merge this patch so having a
version that is ready for merge (all feedback addressed) would be best.

> I am noting down your point ,so ,I might pull out next
> time I intend to sent out something to this specific area.

Please do note that my comment was not specific to resctrl. This prefix
style is used by x86 area.

Reinette


2021-03-17 22:23:20

by Bhaskar Chowdhury

[permalink] [raw]
Subject: Re: [PATCH] kernel: cpu: resctrl: Minor typo fix in the file pseudo_lock.c

On 13:31 Wed 17 Mar 2021, Reinette Chatre wrote:
>Hi Bhaskar,
>
>On 3/17/2021 12:33 PM, Bhaskar Chowdhury wrote:
>> On 10:54 Wed 17 Mar 2021, Reinette Chatre wrote:
>>> Thank you very much for catching this typo.
>>>
>>> My feedback [1] to a previous patch from you applies here also. The
>>> prefix should be "x86/resctrl:" for contributions to this area.
>>>
>> Thanks for the heads up!
>
>This was the third time I mention this to you.
>
Meh ...heck..that means something more important than your words occupied my
mind ...anyway...

>> Do you want another revision of it, or will you
>> take it as it is???
>
>If accepted I am not the one who will merge this patch so having a
>version that is ready for merge (all feedback addressed) would be best.
>
Thanks for the explanations.

>> I am noting down your point ,so ,I might pull out next
>> time I intend to sent out something to this specific area.
>
>Please do note that my comment was not specific to resctrl. This prefix
>style is used by x86 area.
>
I do understand.
>Reinette


Attachments:
(No filename) (1.05 kB)
signature.asc (499.00 B)
Download all attachments