public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arch: x86: kernel: Adjust the words to suit sentences in the file vmlinux.lds.S
@ 2021-03-09  3:41 Bhaskar Chowdhury
  2021-03-09  5:10 ` Randy Dunlap
  0 siblings, 1 reply; 3+ messages in thread
From: Bhaskar Chowdhury @ 2021-03-09  3:41 UTC (permalink / raw)
  To: tglx, mingo, bp, x86, hpa, keescook, jpoimboe, jroedel, hjl.tools,
	nivedita, unixbhaskar, inglorion, linux-kernel
  Cc: rdunlap



s/percpu/per CPU/
s/baremetal/bare metal/

Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
---
 arch/x86/kernel/vmlinux.lds.S | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
index efd9e9ea17f2..592a44ad13b1 100644
--- a/arch/x86/kernel/vmlinux.lds.S
+++ b/arch/x86/kernel/vmlinux.lds.S
@@ -217,7 +217,7 @@ SECTIONS

 #if defined(CONFIG_X86_64) && defined(CONFIG_SMP)
 	/*
-	 * percpu offsets are zero-based on SMP.  PERCPU_VADDR() changes the
+	 * per CPU offsets are zero-based on SMP.  PERCPU_VADDR() changes the
 	 * output PHDR, so the next output section - .init.text - should
 	 * start another segment - init.
 	 */
@@ -262,7 +262,7 @@ SECTIONS
 	/*
 	 * start address and size of operations which during runtime
 	 * can be patched with virtualization friendly instructions or
-	 * baremetal native ones. Think page table operations.
+	 * bare metal native ones. Think page table operations.
 	 * Details in paravirt_types.h
 	 */
 	. = ALIGN(8);
--
2.30.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arch: x86: kernel: Adjust the words to suit sentences in the file vmlinux.lds.S
  2021-03-09  3:41 [PATCH] arch: x86: kernel: Adjust the words to suit sentences in the file vmlinux.lds.S Bhaskar Chowdhury
@ 2021-03-09  5:10 ` Randy Dunlap
  2021-03-09  5:40   ` Bhaskar Chowdhury
  0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2021-03-09  5:10 UTC (permalink / raw)
  To: Bhaskar Chowdhury, tglx, mingo, bp, x86, hpa, keescook, jpoimboe,
	jroedel, hjl.tools, nivedita, inglorion, linux-kernel

On 3/8/21 7:41 PM, Bhaskar Chowdhury wrote:
> 
> 
> s/percpu/per CPU/
> s/baremetal/bare metal/
> 
> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>

Yeah, "bare metal" is a little better than "baremetal".

OTOH, "percpu" is a kernel construct's name. It doesn't have to
be proper English IMHO. (2 cents worth)

> ---
>  arch/x86/kernel/vmlinux.lds.S | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
> index efd9e9ea17f2..592a44ad13b1 100644
> --- a/arch/x86/kernel/vmlinux.lds.S
> +++ b/arch/x86/kernel/vmlinux.lds.S
> @@ -217,7 +217,7 @@ SECTIONS
> 
>  #if defined(CONFIG_X86_64) && defined(CONFIG_SMP)
>  	/*
> -	 * percpu offsets are zero-based on SMP.  PERCPU_VADDR() changes the
> +	 * per CPU offsets are zero-based on SMP.  PERCPU_VADDR() changes the
>  	 * output PHDR, so the next output section - .init.text - should
>  	 * start another segment - init.
>  	 */
> @@ -262,7 +262,7 @@ SECTIONS
>  	/*
>  	 * start address and size of operations which during runtime
>  	 * can be patched with virtualization friendly instructions or
> -	 * baremetal native ones. Think page table operations.
> +	 * bare metal native ones. Think page table operations.
>  	 * Details in paravirt_types.h
>  	 */
>  	. = ALIGN(8);
> --


-- 
~Randy


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arch: x86: kernel: Adjust the words to suit sentences in the file vmlinux.lds.S
  2021-03-09  5:10 ` Randy Dunlap
@ 2021-03-09  5:40   ` Bhaskar Chowdhury
  0 siblings, 0 replies; 3+ messages in thread
From: Bhaskar Chowdhury @ 2021-03-09  5:40 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: tglx, mingo, bp, x86, hpa, keescook, jpoimboe, jroedel, hjl.tools,
	nivedita, inglorion, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1478 bytes --]

On 21:10 Mon 08 Mar 2021, Randy Dunlap wrote:
>On 3/8/21 7:41 PM, Bhaskar Chowdhury wrote:
>>
>>
>> s/percpu/per CPU/
>> s/baremetal/bare metal/
>>
>> Signed-off-by: Bhaskar Chowdhury <unixbhaskar@gmail.com>
>
>Yeah, "bare metal" is a little better than "baremetal".
>
>OTOH, "percpu" is a kernel construct's name. It doesn't have to
>be proper English IMHO. (2 cents worth)
>

My bad ....disregard...pls...will be careful ...

>> ---
>>  arch/x86/kernel/vmlinux.lds.S | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
>> index efd9e9ea17f2..592a44ad13b1 100644
>> --- a/arch/x86/kernel/vmlinux.lds.S
>> +++ b/arch/x86/kernel/vmlinux.lds.S
>> @@ -217,7 +217,7 @@ SECTIONS
>>
>>  #if defined(CONFIG_X86_64) && defined(CONFIG_SMP)
>>  	/*
>> -	 * percpu offsets are zero-based on SMP.  PERCPU_VADDR() changes the
>> +	 * per CPU offsets are zero-based on SMP.  PERCPU_VADDR() changes the
>>  	 * output PHDR, so the next output section - .init.text - should
>>  	 * start another segment - init.
>>  	 */
>> @@ -262,7 +262,7 @@ SECTIONS
>>  	/*
>>  	 * start address and size of operations which during runtime
>>  	 * can be patched with virtualization friendly instructions or
>> -	 * baremetal native ones. Think page table operations.
>> +	 * bare metal native ones. Think page table operations.
>>  	 * Details in paravirt_types.h
>>  	 */
>>  	. = ALIGN(8);
>> --
>
>
>--
>~Randy
>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-03-09  5:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-09  3:41 [PATCH] arch: x86: kernel: Adjust the words to suit sentences in the file vmlinux.lds.S Bhaskar Chowdhury
2021-03-09  5:10 ` Randy Dunlap
2021-03-09  5:40   ` Bhaskar Chowdhury

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox