On 11:01 Wed 17 Mar 2021, Randy Dunlap wrote:
On 3/17/21 10:54 AM, Reinette Chatre wrote:Hi Bhaskar,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.
Bhaskar, Pretty much all of your patches need to have improved Subject: lines. The file name that is being modified should not be at the end of the Subject.
Noted, ...I was trying to feed the owner of the file as much as possible , so the person can struggle less to understand and hover,considering the load they might have on their shoulder.
[1] https://lore.kernel.org/lkml/[email protected]/ On 3/17/2021 1:40 AM, Bhaskar Chowdhury wrote:s/derefence/dereference/ Signed-off-by: Bhaskar Chowdhury <[email protected]> --- arch/x86/kernel/cpu/resctrl/pseudo_lock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/cpu/resctrl/pseudo_lock.c b/arch/x86/kernel/cpu/resctrl/pseudo_lock.c index e916646adc69..43990a882b36 100644 --- a/arch/x86/kernel/cpu/resctrl/pseudo_lock.c +++ b/arch/x86/kernel/cpu/resctrl/pseudo_lock.c @@ -1307,7 +1307,7 @@ int rdtgroup_pseudo_lock_create(struct rdtgroup *rdtgrp) * If the thread does not get on the CPU for whatever * reason and the process which sets up the region is * interrupted then this will leave the thread in runnable - * state and once it gets on the CPU it will derefence + * state and once it gets on the CPU it will dereference * the cleared, but not freed, plr struct resulting in an * empty pseudo-locking loop. */ -- 2.30.2Reinette-- ~Randy
signature.asc
Description: PGP signature

