From: Fenghua Yu <fenghua.yu@intel.com>
To: James Morse <james.morse@arm.com>
Cc: linux-doc@vger.kernel.org, x86@kernel.org,
Jonathan Corbet <corbet@lwn.net>,
Reinette Chatre <reinette.chatre@intel.com>,
Babu Moger <Babu.Moger@amd.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 4/4] Documentation: x86: fix some typos
Date: Fri, 7 Jun 2019 14:29:24 -0700 [thread overview]
Message-ID: <20190607212923.GD143433@romley-ivt3.sc.intel.com> (raw)
In-Reply-To: <20190607151409.15476-5-james.morse@arm.com>
On Fri, Jun 07, 2019 at 04:14:09PM +0100, James Morse wrote:
> These are all obvious typos.
>
> Signed-off-by: James Morse <james.morse@arm.com>
> ---
> Documentation/x86/resctrl_ui.rst | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/x86/resctrl_ui.rst b/Documentation/x86/resctrl_ui.rst
> index 866b66aa289b..5368cedfb530 100644
> --- a/Documentation/x86/resctrl_ui.rst
> +++ b/Documentation/x86/resctrl_ui.rst
> @@ -40,7 +40,7 @@ mount options are:
> Enable the MBA Software Controller(mba_sc) to specify MBA
> bandwidth in MBps
>
> -L2 and L3 CDP are controlled seperately.
> +L2 and L3 CDP are controlled separately.
>
> RDT features are orthogonal. A particular system may support only
> monitoring, only control, or both monitoring and control. Cache
> @@ -118,7 +118,7 @@ related to allocation:
> Corresponding region is pseudo-locked. No
> sharing allowed.
>
> -Memory bandwitdh(MB) subdirectory contains the following files
> +Memory bandwidth(MB) subdirectory contains the following files
> with respect to allocation:
>
> "min_bandwidth":
> @@ -209,7 +209,7 @@ All groups contain the following files:
> CPUs to/from this group. As with the tasks file a hierarchy is
> maintained where MON groups may only include CPUs owned by the
> parent CTRL_MON group.
> - When the resouce group is in pseudo-locked mode this file will
> + When the resource group is in pseudo-locked mode this file will
> only be readable, reflecting the CPUs associated with the
> pseudo-locked region.
>
> @@ -380,7 +380,7 @@ where L2 external is 10GBps (hence aggregate L2 external bandwidth is
> 240GBps) and L3 external bandwidth is 100GBps. Now a workload with '20
> threads, having 50% bandwidth, each consuming 5GBps' consumes the max L3
> bandwidth of 100GBps although the percentage value specified is only 50%
> -<< 100%. Hence increasing the bandwidth percentage will not yeild any
> +<< 100%. Hence increasing the bandwidth percentage will not yield any
> more bandwidth. This is because although the L2 external bandwidth still
> has capacity, the L3 external bandwidth is fully used. Also note that
> this would be dependent on number of cores the benchmark is run on.
> @@ -398,7 +398,7 @@ In order to mitigate this and make the interface more user friendly,
> resctrl added support for specifying the bandwidth in MBps as well. The
> kernel underneath would use a software feedback mechanism or a "Software
> Controller(mba_sc)" which reads the actual bandwidth using MBM counters
> -and adjust the memowy bandwidth percentages to ensure::
> +and adjust the memory bandwidth percentages to ensure::
>
> "actual bandwidth < user specified bandwidth".
>
> --
> 2.20.1
>
Acked-by: Fenghua Yu <fenghua.yu@intel.com>
Thanks.
-Fenghua
next prev parent reply other threads:[~2019-06-07 21:38 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-07 15:14 [PATCH 0/4] Documentation: x86: resctrl_ui.txt fixes and clarification James Morse
2019-06-07 15:14 ` [PATCH 1/4] Documentation: x86: Contiguous cbm isn't all X86 James Morse
2019-06-07 21:24 ` Fenghua Yu
2019-06-07 15:14 ` [PATCH 2/4] Documentation: x86: Remove cdpl2 unspported statement and fix capitalisation James Morse
2019-06-07 21:26 ` Fenghua Yu
2019-06-07 15:14 ` [PATCH 3/4] Documentation: x86: Clarify MBA takes MB as referring to mba_sc James Morse
2019-06-07 21:28 ` Fenghua Yu
2019-06-07 15:14 ` [PATCH 4/4] Documentation: x86: fix some typos James Morse
2019-06-07 21:29 ` Fenghua Yu [this message]
2019-06-20 20:16 ` [PATCH 0/4] Documentation: x86: resctrl_ui.txt fixes and clarification 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=20190607212923.GD143433@romley-ivt3.sc.intel.com \
--to=fenghua.yu@intel.com \
--cc=Babu.Moger@amd.com \
--cc=corbet@lwn.net \
--cc=james.morse@arm.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=reinette.chatre@intel.com \
--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
Powered by JetHome