Linux-EDAC Archive on lore.kernel.org
 help / color / Atom feed
* [PATCH] doc: Fix some typo errors in ras.rst
@ 2020-05-05 15:10 Waiman Long
  2020-05-05 15:32 ` Jonathan Corbet
  2020-05-05 15:48 ` Matthew Wilcox
  0 siblings, 2 replies; 7+ messages in thread
From: Waiman Long @ 2020-05-05 15:10 UTC (permalink / raw)
  To: Jonathan Corbet, Tony Luck, Borislav Petkov,
	Mauro Carvalho Chehab, James Morse, Robert Richter
  Cc: linux-edac, linux-doc, linux-kernel, Waiman Long

Fix typo errors.

Signed-off-by: Waiman Long <longman@redhat.com>
---
 Documentation/admin-guide/ras.rst | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/admin-guide/ras.rst b/Documentation/admin-guide/ras.rst
index 0310db624964..8b1803b2606f 100644
--- a/Documentation/admin-guide/ras.rst
+++ b/Documentation/admin-guide/ras.rst
@@ -156,10 +156,10 @@ the labels provided by the BIOS won't match the real ones.
 ECC memory
 ----------
 
-As mentioned on the previous section, ECC memory has extra bits to be
-used for error correction. So, on 64 bit systems, a memory module
-has 64 bits of *data width*, and 74 bits of *total width*. So, there are
-8 bits extra bits to be used for the error detection and correction
+As mentioned on the previous section, ECC memory has extra bits to
+be used for error correction. So, on 64 bit systems, a memory module
+has 64 bits of *data width*, and 72 bits of *total width*. So, there
+are 8 extra bits to be used for the error detection and correction
 mechanisms. Those extra bits are called *syndrome*\ [#f1]_\ [#f2]_.
 
 So, when the cpu requests the memory controller to write a word with
-- 
2.18.1


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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 15:10 [PATCH] doc: Fix some typo errors in ras.rst Waiman Long
@ 2020-05-05 15:32 ` Jonathan Corbet
  2020-05-05 15:50   ` Waiman Long
  2020-05-05 15:48 ` Matthew Wilcox
  1 sibling, 1 reply; 7+ messages in thread
From: Jonathan Corbet @ 2020-05-05 15:32 UTC (permalink / raw)
  To: Waiman Long
  Cc: Tony Luck, Borislav Petkov, Mauro Carvalho Chehab, James Morse,
	Robert Richter, linux-edac, linux-doc, linux-kernel

On Tue,  5 May 2020 11:10:49 -0400
Waiman Long <longman@redhat.com> wrote:

> Fix typo errors.
> 
> Signed-off-by: Waiman Long <longman@redhat.com>
> ---
>  Documentation/admin-guide/ras.rst | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/admin-guide/ras.rst b/Documentation/admin-guide/ras.rst
> index 0310db624964..8b1803b2606f 100644
> --- a/Documentation/admin-guide/ras.rst
> +++ b/Documentation/admin-guide/ras.rst
> @@ -156,10 +156,10 @@ the labels provided by the BIOS won't match the real ones.
>  ECC memory
>  ----------
>  
> -As mentioned on the previous section, ECC memory has extra bits to be
> -used for error correction. So, on 64 bit systems, a memory module
> -has 64 bits of *data width*, and 74 bits of *total width*. So, there are
> -8 bits extra bits to be used for the error detection and correction
> +As mentioned on the previous section, ECC memory has extra bits to
> +be used for error correction. So, on 64 bit systems, a memory module
> +has 64 bits of *data width*, and 72 bits of *total width*. So, there
> +are 8 extra bits to be used for the error detection and correction
>  mechanisms. Those extra bits are called *syndrome*\ [#f1]_\ [#f2]_.

So I had to work to figure out what the change was, since you didn't say
in the changelog and you refilled the paragraph.  But this looks like a
*factual* error, giving the wrong number of bits, right?  It seems like
the changelog should say that.  Do the people who know about this stuff
agree that the change is correct?

Thanks,

jon

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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 15:10 [PATCH] doc: Fix some typo errors in ras.rst Waiman Long
  2020-05-05 15:32 ` Jonathan Corbet
@ 2020-05-05 15:48 ` Matthew Wilcox
  2020-05-05 16:03   ` Borislav Petkov
  2020-05-06 16:19   ` Waiman Long
  1 sibling, 2 replies; 7+ messages in thread
From: Matthew Wilcox @ 2020-05-05 15:48 UTC (permalink / raw)
  To: Waiman Long
  Cc: Jonathan Corbet, Tony Luck, Borislav Petkov,
	Mauro Carvalho Chehab, James Morse, Robert Richter, linux-edac,
	linux-doc, linux-kernel

On Tue, May 05, 2020 at 11:10:49AM -0400, Waiman Long wrote:
> Fix typo errors.

By reformatting it, you've successfully obscured what typos you've fixed.
As a result I read the whole paragraph, and ...

>  ECC memory
>  ----------
>  
> -As mentioned on the previous section, ECC memory has extra bits to be
> -used for error correction. So, on 64 bit systems, a memory module
> -has 64 bits of *data width*, and 74 bits of *total width*. So, there are
> -8 bits extra bits to be used for the error detection and correction
> +As mentioned on the previous section, ECC memory has extra bits to

s/on/in/

> +be used for error correction. So, on 64 bit systems, a memory module
> +has 64 bits of *data width*, and 72 bits of *total width*.

Usually a 64-bit system refers to the width of a pointer.  Here, it's
referring to the width of the memory system, which is rather confusing.
How about "In the above example" instead of "So, on 64 bit systems".

> So, there
> +are 8 extra bits to be used for the error detection and correction
>  mechanisms. Those extra bits are called *syndrome*\ [#f1]_\ [#f2]_.

This would read better as:

The extra 8 bits which are used for error detection and correction
are referred to as the *syndrome*\ [#f1]_\ [#f2]_.


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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 15:32 ` Jonathan Corbet
@ 2020-05-05 15:50   ` Waiman Long
  0 siblings, 0 replies; 7+ messages in thread
From: Waiman Long @ 2020-05-05 15:50 UTC (permalink / raw)
  To: Jonathan Corbet
  Cc: Tony Luck, Borislav Petkov, Mauro Carvalho Chehab, James Morse,
	Robert Richter, linux-edac, linux-doc, linux-kernel

On 5/5/20 11:32 AM, Jonathan Corbet wrote:
> On Tue,  5 May 2020 11:10:49 -0400
> Waiman Long <longman@redhat.com> wrote:
>
>> Fix typo errors.
>>
>> Signed-off-by: Waiman Long <longman@redhat.com>
>> ---
>>   Documentation/admin-guide/ras.rst | 8 ++++----
>>   1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/Documentation/admin-guide/ras.rst b/Documentation/admin-guide/ras.rst
>> index 0310db624964..8b1803b2606f 100644
>> --- a/Documentation/admin-guide/ras.rst
>> +++ b/Documentation/admin-guide/ras.rst
>> @@ -156,10 +156,10 @@ the labels provided by the BIOS won't match the real ones.
>>   ECC memory
>>   ----------
>>   
>> -As mentioned on the previous section, ECC memory has extra bits to be
>> -used for error correction. So, on 64 bit systems, a memory module
>> -has 64 bits of *data width*, and 74 bits of *total width*. So, there are
>> -8 bits extra bits to be used for the error detection and correction
>> +As mentioned on the previous section, ECC memory has extra bits to
>> +be used for error correction. So, on 64 bit systems, a memory module
>> +has 64 bits of *data width*, and 72 bits of *total width*. So, there
>> +are 8 extra bits to be used for the error detection and correction
>>   mechanisms. Those extra bits are called *syndrome*\ [#f1]_\ [#f2]_.
> So I had to work to figure out what the change was, since you didn't say
> in the changelog and you refilled the paragraph.  But this looks like a
> *factual* error, giving the wrong number of bits, right?  It seems like
> the changelog should say that.  Do the people who know about this stuff
> agree that the change is correct?
>
> Thanks,
>
> jon
>
Yes, I change 74 bits to 72 bits and remove an extra "bits". Will send 
out v2 to clarify that.

Thanks,
Longman


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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 15:48 ` Matthew Wilcox
@ 2020-05-05 16:03   ` Borislav Petkov
  2020-05-06 16:21     ` Waiman Long
  2020-05-06 16:19   ` Waiman Long
  1 sibling, 1 reply; 7+ messages in thread
From: Borislav Petkov @ 2020-05-05 16:03 UTC (permalink / raw)
  To: Matthew Wilcox
  Cc: Waiman Long, Jonathan Corbet, Tony Luck, Mauro Carvalho Chehab,
	James Morse, Robert Richter, linux-edac, linux-doc, linux-kernel

On Tue, May 05, 2020 at 08:48:16AM -0700, Matthew Wilcox wrote:
> Usually a 64-bit system refers to the width of a pointer.  Here, it's
> referring to the width of the memory system, which is rather confusing.
> How about "In the above example" instead of "So, on 64 bit systems".

Yes, that should not talk about 64-bit systems but about the length of
the word the memory controller uses to create the ECC check bits out of.

That whole doc needs checking/fixing.

> The extra 8 bits which are used for error detection and correction
> are referred to as the *syndrome*\ [#f1]_\ [#f2]_.

Well, I know it as "The syndrome field uniquely identifies the failing
bit positions of a correctable ECC error." and depending on the error
severity you can have correctable and uncorrectable syndromes.

I don't think the syndrome is the ECC word but it might be, depending
on the ECC algorithm used or it might be that what I pasted above. That
would need deeper digging.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 15:48 ` Matthew Wilcox
  2020-05-05 16:03   ` Borislav Petkov
@ 2020-05-06 16:19   ` Waiman Long
  1 sibling, 0 replies; 7+ messages in thread
From: Waiman Long @ 2020-05-06 16:19 UTC (permalink / raw)
  To: Matthew Wilcox
  Cc: Jonathan Corbet, Tony Luck, Borislav Petkov,
	Mauro Carvalho Chehab, James Morse, Robert Richter, linux-edac,
	linux-doc, linux-kernel

On 5/5/20 11:48 AM, Matthew Wilcox wrote:
> On Tue, May 05, 2020 at 11:10:49AM -0400, Waiman Long wrote:
>> Fix typo errors.
> By reformatting it, you've successfully obscured what typos you've fixed.
> As a result I read the whole paragraph, and ...
>
>>   ECC memory
>>   ----------
>>   
>> -As mentioned on the previous section, ECC memory has extra bits to be
>> -used for error correction. So, on 64 bit systems, a memory module
>> -has 64 bits of *data width*, and 74 bits of *total width*. So, there are
>> -8 bits extra bits to be used for the error detection and correction
>> +As mentioned on the previous section, ECC memory has extra bits to
> s/on/in/
>
>> +be used for error correction. So, on 64 bit systems, a memory module
>> +has 64 bits of *data width*, and 72 bits of *total width*.
> Usually a 64-bit system refers to the width of a pointer.  Here, it's
> referring to the width of the memory system, which is rather confusing.
> How about "In the above example" instead of "So, on 64 bit systems".
>
>> So, there
>> +are 8 extra bits to be used for the error detection and correction
>>   mechanisms. Those extra bits are called *syndrome*\ [#f1]_\ [#f2]_.
> This would read better as:
>
> The extra 8 bits which are used for error detection and correction
> are referred to as the *syndrome*\ [#f1]_\ [#f2]_.
>
Thanks for the suggestion. Will incorporate that in v2.

-Longman


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

* Re: [PATCH] doc: Fix some typo errors in ras.rst
  2020-05-05 16:03   ` Borislav Petkov
@ 2020-05-06 16:21     ` Waiman Long
  0 siblings, 0 replies; 7+ messages in thread
From: Waiman Long @ 2020-05-06 16:21 UTC (permalink / raw)
  To: Borislav Petkov, Matthew Wilcox
  Cc: Jonathan Corbet, Tony Luck, Mauro Carvalho Chehab, James Morse,
	Robert Richter, linux-edac, linux-doc, linux-kernel

On 5/5/20 12:03 PM, Borislav Petkov wrote:
> On Tue, May 05, 2020 at 08:48:16AM -0700, Matthew Wilcox wrote:
>> Usually a 64-bit system refers to the width of a pointer.  Here, it's
>> referring to the width of the memory system, which is rather confusing.
>> How about "In the above example" instead of "So, on 64 bit systems".
> Yes, that should not talk about 64-bit systems but about the length of
> the word the memory controller uses to create the ECC check bits out of.
>
> That whole doc needs checking/fixing.

I am not knowledgeable enough to check/fix the whole document. This 
patch is motivated primarily by the incorrect "74 bits" total width that 
I saw. I will send a v2 to make the suggested edit. I will leave it to 
you to do the full document check.

Cheers,
Longman


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

end of thread, back to index

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-05 15:10 [PATCH] doc: Fix some typo errors in ras.rst Waiman Long
2020-05-05 15:32 ` Jonathan Corbet
2020-05-05 15:50   ` Waiman Long
2020-05-05 15:48 ` Matthew Wilcox
2020-05-05 16:03   ` Borislav Petkov
2020-05-06 16:21     ` Waiman Long
2020-05-06 16:19   ` Waiman Long

Linux-EDAC Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-edac/0 linux-edac/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-edac linux-edac/ https://lore.kernel.org/linux-edac \
		linux-edac@vger.kernel.org
	public-inbox-index linux-edac

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-edac


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git