From: bilbao@vt.edu
To: Carlos Bilbao <bilbao@vt.edu>,
tglx@linutronix.de, mingo@redhat.com, peterz@infradead.org,
bp@alien8.de, x86@kernel.org, hpa@zytor.com,
linux-kernel@vger.kernel.org, Jonathan Corbet <corbet@lwn.net>
Subject: Re: [PATCH] Fixed typo in Documentation/x86/x86_64/5level-paging.rst
Date: Tue, 27 Apr 2021 13:28:29 -0400 [thread overview]
Message-ID: <2599991.mvXUDI8C0e@iron-maiden> (raw)
In-Reply-To: <871raveno6.fsf@meer.lwn.net>
Hello Jon, thanks a lot for your feedback, it was instructive. I attach changelog and the patch as plain text below.
I fix two typos in the documentation (Documentation/x86/x86_64/5level-paging.rst), changing 'paing' for
'paging' and using the right verbal form for plural on 'some vendors offer'.
Signed-off-by: Carlos Bilbao <bilbao@vt.edu>
---
Documentation/x86/x86_64/5level-paging.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Documentation/x86/x86_64/5level-paging.rst b/Documentation/x86/x86_64/5level-paging.rst
index 44856417e6a5..b792bbdc0b01 100644
--- a/Documentation/x86/x86_64/5level-paging.rst
+++ b/Documentation/x86/x86_64/5level-paging.rst
@@ -6,9 +6,9 @@
Overview
========
-Original x86-64 was limited by 4-level paing to 256 TiB of virtual address
+Original x86-64 was limited by 4-level paging to 256 TiB of virtual address
space and 64 TiB of physical address space. We are already bumping into
-this limit: some vendors offers servers with 64 TiB of memory today.
+this limit: some vendors offer servers with 64 TiB of memory today.
To overcome the limitation upcoming hardware will introduce support for
5-level paging. It is a straight-forward extension of the current page
--
2.25.1
On Tuesday, April 27, 2021 11:45:45 AM EDT Jonathan Corbet wrote:
> Carlos Bilbao <bilbao@vt.edu> writes:
> > Signed-off-by: Carlos Bilbao <bilbao@vt.edu>
> > ---
> >
> > Documentation/x86/x86_64/5level-paging.rst | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/x86/x86_64/5level-paging.rst
> > b/Documentation/x86/x86_64/5level-paging.rst
> > index 44856417e6a5..b792bbdc0b01 100644
> > --- a/Documentation/x86/x86_64/5level-paging.rst
> > +++ b/Documentation/x86/x86_64/5level-paging.rst
> > @@ -6,9 +6,9 @@
> >
> > Overview
> > ========
> >
> > -Original x86-64 was limited by 4-level paing to 256 TiB of virtual
> > address
> > +Original x86-64 was limited by 4-level paging to 256 TiB of virtual
> > address>
> > space and 64 TiB of physical address space. We are already bumping into
> >
> > -this limit: some vendors offers servers with 64 TiB of memory today.
> > +this limit: some vendors offer servers with 64 TiB of memory today.
>
> So this seems like a good change, but I need to make a couple of
> requests:
>
> - Please include a changelog, even with relatively simple patches like
> this.
>
> - Patches should be sent as plain text, inline in the mail - not as
> attachments and *certainly* not as HTML. Have a look at
> Documentation/process/email-clients.rst if you need some guidance on
> configuring your email setup.
>
> Thanks,
>
> jon
next prev parent reply other threads:[~2021-04-27 17:28 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CACbHsvSta2hogK0E4AKCxsUF3bFrdQTAA-mp2MvO6Q7+rQuKCg@mail.gmail.com>
2021-04-27 15:45 ` [PATCH] Fixed typo in Documentation/x86/x86_64/5level-paging.rst Jonathan Corbet
2021-04-27 17:28 ` bilbao [this message]
2021-04-27 19:07 ` Jonathan Corbet
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2599991.mvXUDI8C0e@iron-maiden \
--to=bilbao@vt.edu \
--cc=bp@alien8.de \
--cc=corbet@lwn.net \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).