ratelimit is a new option in bus lock handling. Need to add it in
buslock.rst.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Documentation/x86/buslock.rst | 23 +++
1 file changed, 23 insertions(+)
diff --git a/Documentation/x86/buslock.rst b/Documentation/x86
Since bus lock rate limit changes the split_lock_detect parameter,
update the documentation for the change.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Documentation/admin-guide/kernel-parameters.txt | 8
1 file changed, 8 insertions(+)
diff --git a/Documentation/admin-guide
system rate of bus locks drops below the threshold.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
arch/x86/kernel/cpu/intel.c | 43 +++--
1 file changed, 41 insertions(+), 2 deletions(-)
diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kernel/cpu/intel.c
Add buslock.rst to explain bus lock problem and how to detect and
handle it.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Documentation/x86/buslock.rst | 110 ++
1 file changed, 110 insertions(+)
create mode 100644 Documentation/x86/buslock.rst
diff
bus lock into warn and fatal
patch set and this rate limit patch set:
https://lore.kernel.org/lkml/871rca6dbp@nanos.tec.linutronix.de/
Fenghua Yu (4):
Documentation/x86: Add buslock.rst
x86/bus_lock: Set rate limit for bus lock
Documentation/admin-guide: Change doc for bus lock ratelimit
Hi, Thomas,
On Mon, Apr 12, 2021 at 09:15:08AM +0200, Thomas Gleixner wrote:
> On Sat, Apr 03 2021 at 01:04, Fenghua Yu wrote:
> > On Sat, Mar 20, 2021 at 01:42:52PM +0100, Thomas Gleixner wrote:
> >> On Fri, Mar 19 2021 at 22:19, Fenghua Yu wrote:
> >> And even with th
Hi, Shuah,
On Wed, Apr 07, 2021 at 04:46:38PM -0600, Shuah Khan wrote:
> On 4/7/21 1:57 PM, Fenghua Yu wrote:
> > Change a few printed messages to report test progress more clearly.
> Thank you. Applied to linux-kseftest next branch for 5.13-rc1
Great! I pull the next patch and te
Change a few printed messages to report test progress more clearly.
Add a missing "\n" at the end of one printed message.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
Change log:
v2:
- Add "Pass:" and "Fail:" sub-strings back (Shuah).
This is a fol
Hi, Shuah,
On Wed, Apr 07, 2021 at 08:33:23AM -0600, Shuah Khan wrote:
> On 4/5/21 6:52 PM, Fenghua Yu wrote:
> > - ksft_print_msg("%s cache miss rate within %d%%\n",
> > - ret ? "Fail:" : "Pass:", max_diff_percent);
> > +
A few printed messages contain pass/fail strings which should be shown
in test results. Remove the pass/fail strings in the messages to avoid
confusion.
Add "\n" at the end of one printed message.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
This is a follow-up patch
Hi, Thomas,
On Sat, Mar 20, 2021 at 01:42:52PM +0100, Thomas Gleixner wrote:
> On Fri, Mar 19 2021 at 22:19, Fenghua Yu wrote:
> > On Fri, Mar 19, 2021 at 10:30:50PM +0100, Thomas Gleixner wrote:
> >> > +if (sscanf(arg, "ratelimit:%d", &ratel
Hi, Thomas,
On Sat, Mar 20, 2021 at 02:57:52PM +0100, Thomas Gleixner wrote:
> On Sat, Mar 20 2021 at 02:01, Thomas Gleixner wrote:
>
> > On Fri, Mar 19 2021 at 21:50, Tony Luck wrote:
> >>> What is the justifucation for making this rate limit per UID and not
> >>> per task, per process or syst
On Fri, Apr 02, 2021 at 02:04:16PM -0600, Shuah Khan wrote:
> On 4/2/21 12:18 PM, Fenghua Yu wrote:
> > Hi, Shuah,
> >
> > On Fri, Apr 02, 2021 at 12:17:17PM -0600, Shuah Khan wrote:
> > > On 3/26/21 1:45 PM, Fenghua Yu wrote:
> > > > Hi, Shuah,
> >
Hi, Shuah,
On Fri, Apr 02, 2021 at 12:17:17PM -0600, Shuah Khan wrote:
> On 3/26/21 1:45 PM, Fenghua Yu wrote:
> > Hi, Shuah,
> >
> > On Wed, Mar 17, 2021 at 02:22:34AM +, Fenghua Yu wrote:
> > > This patch set has several miscellaneous fixes to resctrl selft
The following commit has been merged into the x86/splitlock branch of tip:
Commit-ID: ebca17707e38f2050b188d837bd4646b29a1b0c2
Gitweb:
https://git.kernel.org/tip/ebca17707e38f2050b188d837bd4646b29a1b0c2
Author:Fenghua Yu
AuthorDate:Mon, 22 Mar 2021 13:53:25
Committer
The following commit has been merged into the x86/splitlock branch of tip:
Commit-ID: ebb1064e7c2e90b56e4d40ab154ef9796060a1c3
Gitweb:
https://git.kernel.org/tip/ebb1064e7c2e90b56e4d40ab154ef9796060a1c3
Author:Fenghua Yu
AuthorDate:Mon, 22 Mar 2021 13:53:24
Committer
The following commit has been merged into the x86/splitlock branch of tip:
Commit-ID: f21d4d3b97a8603567e5d4250bd75e8ebbd520af
Gitweb:
https://git.kernel.org/tip/f21d4d3b97a8603567e5d4250bd75e8ebbd520af
Author:Fenghua Yu
AuthorDate:Mon, 22 Mar 2021 13:53:23
Committer
Hi, Shuah,
On Wed, Mar 17, 2021 at 02:22:34AM +, Fenghua Yu wrote:
> This patch set has several miscellaneous fixes to resctrl selftest tool
> that are easily visible to user. V1 had fixes to CAT test and CMT test
> but they were dropped in V2 because having them here made the
700-68460-1-git-send-email-fenghua...@intel.com/]
Fenghua Yu (3):
x86/cpufeatures: Enumerate #DB for bus lock detection
x86/bus_lock: Handle #DB for bus lock
Documentation/admin-guide: Change doc for split_lock_detect parameter
.../admin-guide/kernel-parameters.txt | 2
Since #DB for bus lock detect changes the split_lock_detect parameter,
update the documentation for the changes.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
Acked-by: Randy Dunlap
---
Change Log:
v6:
- Remove the ratelimit info which will be released later in a ratelimit
specific patch
r's choice.
The CPU feature flag to be shown in /proc/cpuinfo will be "bus_lock_detect".
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Change Log:
v6:
- Fix wording issues in the commit message (Thomas).
v5:
- Add "Both breakpoint and bus lock can trigger an #DB t
k in userspace clears DR6[11]. To avoid
confusion in identifying #DB, #DB handler sets the bit to 1 before
returning to the interrupted task.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Change Log:
v6:
- Split the v5 patch set into two patch sets: the one for warn and fatal
and the one
Hi, Thomas,
On Fri, Mar 19, 2021 at 10:30:50PM +0100, Thomas Gleixner wrote:
> On Sat, Mar 13 2021 at 05:49, Fenghua Yu wrote:
> > Change Log:
> > v5:
> > Address all comments from Thomas:
> > - Merge patch 2 and patch 3 into one patch so all "split_lock_detect=
On Fri, Mar 19, 2021 at 09:35:39PM +0100, Thomas Gleixner wrote:
> On Sat, Mar 13 2021 at 05:49, Fenghua Yu wrote:
> > A bus lock is acquired though either split locked access to
>
> s/though/through/
> either a
> > Some CPUs have ability to notify the kernel by an #DB tr
Create .gitignore to hold the test file resctrl_tests generated after
compiling.
Suggested-by: Shuah Khan
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools/testing/selftests/resctrl/.gitignore | 2 ++
1 file changed, 2 insertions(+)
create
()
function. Adding these wrapper functions makes it easier to add any logic
to each individual test.
Please note that this is a preparatory patch for the next one and no
functional changes are intended.
Suggested-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
else return
< 0 on error.
Reported-by: Dan Carpenter
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 41 +++
1 file changed, 23 insertions(+), 18 deletions(-)
diff --git a/tools/testing/selftests/resctrl/resctrl_val.c
therwise, no_of_bits is equal to half of the maximum number of bits in
the cbm mask.
Please note that CMT test is still hard coded to 5 bits. It will change in
subsequent patches that change CMT test.
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl
lue and
imc_mbm value is within 5% (randomly chosen value) of imc_mbm value. If the
difference is greater than 5% of imc_mbm value, treat it is a failure.
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/mba_test.c | 22 +-
tools/testing/se
rt MBA or MBA is disabled
Suggested-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v6:
- Replace "cat" by CAT_STR and so on (Babu).
tools/testing/selftests/resctrl/cat_test.c| 3 ---
tools/testing/selftests/resctrl/mba_test.c| 3 ---
tools/t
ause these variables are defined globally in two .c files
but are not declared as extern.
To fix issues for the global variables, declare them as extern.
Chang Log:
- Split this patch from v4's patch 1 (Shuah).
Reported-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
directory represents an iMC counter, else, skip the
directory.
Reported-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 22 +--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/tools
equest() would mount resctrl
FS to check if mba is enabled on the platform or not and finds that the H/W
doesn't support mba and hence will return false to run_mba_test(). This in
turn makes the main() function return without unmounting resctrl FS.
Tested-by: Babu Moger
Signed-off-by: Fengh
variables.
Reported-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Define long_mask, cbm_mask, count_of_bits etc as static variables
(Shuah).
- Split this patch into patch 2 and 3 (Shuah).
tools/testing/selftests/resctrl/cat_test.c | 10 +--
selection by taking the provided/default CPU into
account and ensuring a sibling that is a different CPU is selected.
Tested-by: Babu Moger
Signed-off-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Move from v4's patch 8 to this patch as the fix patch should be first
-off-by: Fenghua Yu
---
Change Log:
v5:
- Move from v4's patch 9 to this patch as the fix patch should be first
(Shuah).
tools/testing/selftests/resctrl/resctrl_tests.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/resctrl/resctrl_tests.c
b/
an
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Remove is_cat() etc functions and directly call strncmp() to check
the features (Shuah).
tools/testing/selftests/resctrl/cache.c | 8 +++
tools/testing/selftests/resctrl/cat_test.c| 2 +-
tools/testing/sel
Add the config file for test dependencies.
Suggested-by: Shuah Khan
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools/testing/selftests/resctrl/config | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 tools/testing/selftests/resctrl
l
Suggested-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/README| 4 +--
tools/testing/selftests/resctrl/cache.c | 4 +--
.../resctrl/{cqm_test.c => cmt_test.c}| 20 +++---
tools/testing/selftests/resctrl/
show_cache_info() functions are defined separately in CAT and CMT
tests. But the functions are same for the tests and unnecessary
to be defined separately. Share the function by the tests.
Suggested-by: Shuah Khan
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this
at
/proc/mounts
Steps 2 and 3 will fail if the platform does not support resctrl file
system. So, there is no need to check for them if step 1 fails.
Fix this by returning immediately if the platform does not support
resctrl file system.
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
Call kselftest APIs instead of using printf() to log test results
for cleaner code and better future extension.
Suggested-by: Shuah Khan
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v6:
- Capitalize the first letter in printed msg (Babu).
v5:
- Add this patch (Shuah
test and enable _FORTIFY_SOURCE gcc check to catch any future buffer
overflow errors.
Reported-by: David Binderman
Suggested-by: David Binderman
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Move from v4's patch 11 to patch 1 so the fix patch should be first
(Shuah).
too
From: Reinette Chatre
Add a missing newline to the printed help text to improve readability.
Tested-by: Babu Moger
Signed-off-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Remove the "notok" fix part because the API change fixes it already.
tools/testing
lftest/cover.1589835155.git.sai.praneeth.prak...@intel.com/]
v2:
1. Dropped changes to CAT test and CMT test as they will be posted in a later
series.
2. Added several other fixes
[v1:
https://lore.kernel.org/linux-kselftest/cover.1583657204.git.sai.praneeth.prak...@intel.com/]
Fenghua Yu (19):
selftest
on_features has
mbm__bytes
Please note that only L3_CAT, L3_CMT, MBA and MBM are supported. CDP and L2
variants can be added later.
Reported-by: Reinette Chatre
Tested-by: Babu Moger
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl.h | 6 ++-
tools/testing/selftests/resc
at:
https://lore.kernel.org/lkml/1595021700-68460-1-git-send-email-fenghua...@intel.com/]
Fenghua Yu (3):
x86/cpufeatures: Enumerate #DB for bus lock detection
x86/bus_lock: Handle #DB for bus lock
Documentation/admin-guide: Change doc for split_lock_detect parameter
.../admin-guide/k
k in userspace clears DR6[11]. To avoid
confusion in identifying #DB, #DB handler sets the bit to 1 before
returning to the interrupted task.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Change Log:
v5:
Address all comments from Thomas:
- Merge patch 2 and patch 3 into one patch so all &
Since #DB for bus lock detect changes the split_lock_detect parameter,
update the documentation for the changes.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
Acked-by: Randy Dunlap
---
Change Log:
v5:
- Remove N < HZ/2 check info in the doc (Thomas).
v4:
- Fix a ratelimit wording issue
s choice.
The CPU feature flag to be shown in /proc/cpuinfo will be "bus_lock_detect".
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
Change Log:
v5:
- Add "Both breakpoint and bus lock can trigger an #DB trap..." in the
commit message (Thomas).
arch/x86/include/asm
Hi, Babu,
On Fri, Mar 12, 2021 at 01:08:11PM -0600, Babu Moger wrote:
> Hi Fenghua, Thanks for the patches.
> Sanity tested them on AMD systems. Appears to work fine.
> Few minor comments in few patches.
> Tested-by: Babu Moger
I will add Tested-by: Babu Moger in the series and address your
comm
Hi, Babu,
On Fri, Mar 12, 2021 at 01:12:35PM -0600, Babu Moger wrote:
> > - printf("# dmesg: %s", line);
> > + ksft_print_msg("dmesg: %s", line);
> > if (strstr(line, "resctrl:"))
> > - printf("# dmesg: %s", line);
> > +
Hi, Babu,
On Fri, Mar 12, 2021 at 01:09:50PM -0600, Babu Moger wrote:
> > - if (strcmp(resctrl_val, "mba") == 0)
> > + if (!strncmp(resctrl_val, MBA_STR, sizeof(MBA_STR)))
> > sprintf(schema, "%s%d%c%s", "MB:", resource_id, '=',
> > schemata);
> I see there are few other references
Hi, Babu,
On Fri, Mar 12, 2021 at 01:08:31PM -0600, Babu Moger wrote:
> > From: Fenghua Yu
> > Taking a closer look at the usage of these variables reveals that these
> > variables are used only locally to functions such as cqm_resctrl_val()
>
> %s/ locally to functions/
Add the config file for test dependencies.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools/testing/selftests/resctrl/config | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 tools/testing/selftests/resctrl/config
diff --git a
equest() would mount resctrl
FS to check if mba is enabled on the platform or not and finds that the H/W
doesn't support mba and hence will return false to run_mba_test(). This in
turn makes the main() function return without unmounting resctrl FS.
Signed-off-by: Fenghua Yu
---
tools/
()
function. Adding these wrapper functions makes it easier to add any logic
to each individual test.
Please note that this is a preparatory patch for the next one and no
functional changes are intended.
Suggested-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
.../testing/selftests/resctrl
lue and
imc_mbm value is within 5% (randomly chosen value) of imc_mbm value. If the
difference is greater than 5% of imc_mbm value, treat it is a failure.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/mba_test.c | 22 +-
tools/testing/selftests/resctrl/
therwise, no_of_bits is equal to half of the maximum number of bits in
the cbm mask.
Please note that CMT test is still hard coded to 5 bits. It will change in
subsequent patches that change CMT test.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/cat_test.c | 5 -
too
rt MBA or MBA is disabled
Suggested-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/cat_test.c| 3 ---
tools/testing/selftests/resctrl/mba_test.c| 3 ---
tools/testing/selftests/resctrl/mbm_test.c| 3 ---
.../testing/selftests/res
at
/proc/mounts
Steps 2 and 3 will fail if the platform does not support resctrl file
system. So, there is no need to check for them if step 1 fails.
Fix this by returning immediately if the platform does not support
resctrl file system.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests
on_features has
mbm__bytes
Please note that only L3_CAT, L3_CMT, MBA and MBM are supported. CDP and L2
variants can be added later.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl.h | 6 ++-
tools/testing/selftests/resctrl/r
Create .gitignore to hold the test file resctrl_tests generated after
compiling.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools/testing/selftests/resctrl/.gitignore | 2 ++
1 file changed, 2 insertions(+)
create mode 100644 tools/testing
ause these variables are defined globally in two .c files
but are not declared as extern.
To fix issues for the global variables, declare them as extern.
Chang Log:
- Split this patch from v4's patch 1 (Shuah).
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/se
eturn 0, else return
< 0 on error.
Reported-by: Dan Carpenter
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 41 +++
1 file changed, 23 insertions(+), 18 deletions(-)
diff --git a/tools/testing/selftests/resctrl/resctrl_val.c
b/tools/testing/sel
directory represents an iMC counter, else, skip the
directory.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 22 +--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/resctrl/resctr
variables.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Define long_mask, cbm_mask, count_of_bits etc as static variables
(Shuah).
- Split this patch into patch 2 and 3 (Shuah).
tools/testing/selftests/resctrl/cat_test.c | 10 +-
tools/testing/selftes
Checking resctrl features call strcmp() to compare feature strings
(e.g. "mba", "cat" etc). The checkings are error prone and don't have
good coding style. Define the constant strings in macros and call
strncmp() to solve the potential issues.
Suggested-by: Shuah Khan
selection by taking the provided/default CPU into
account and ensuring a sibling that is a different CPU is selected.
Signed-off-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Move from v4's patch 8 to this patch as the fix patch should be first
(Shuah).
tools/testing
show_cache_info() functions are defined separately in CAT and CMT
tests. But the functions are same for the tests and unnecessary
to be defined separately. Share the function by the tests.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools
l
Suggested-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/README| 4 +--
tools/testing/selftests/resctrl/cache.c | 4 +--
.../resctrl/{cqm_test.c => cmt_test.c}| 20 +++---
tools/testing/selftests/resctrl/resctrl.h | 6 ++---
...
invalid options.
Fix this by adding them to the list of characters that getopt() recognizes
as valid arguments. Please note that the main() function already has the
logic to deal with the values passed as part of these arguments and hence
no changes are needed there.
Signed-off-by: Fenghua Yu
s.
2. Added several other fixes
[v1:
https://lore.kernel.org/linux-kselftest/cover.1583657204.git.sai.praneeth.prak...@intel.com/]
Fenghua Yu (19):
selftests/resctrl: Enable gcc checks to detect buffer overflows
selftests/resctrl: Fix compilation issues for global variables
selftests/resctr
From: Reinette Chatre
Add a missing newline to the printed help text to improve readability.
Signed-off-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Remove the "notok" fix part because the API change fixes it already.
tools/testing/selftests/resctrl/resct
test and enable _FORTIFY_SOURCE gcc check to catch any future buffer
overflow errors.
Reported-by: David Binderman
Suggested-by: David Binderman
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Move from v4's patch 11 to patch 1 so the fix patch should be first
(Shuah).
tools/testing/selftests/re
Call kselftest APIs instead of using printf() to log test results
for cleaner code and better future extension.
Suggested-by: Shuah Khan
Signed-off-by: Fenghua Yu
---
Change Log:
v5:
- Add this patch (Shuah)
tools/testing/selftests/resctrl/cat_test.c| 37 +++
tools/testing
Hi, Jean,
On Wed, Feb 24, 2021 at 11:19:27AM +0100, Jean-Philippe Brucker wrote:
> Hi Fenghua,
>
> [Trimmed the Cc list]
>
> On Mon, Jul 13, 2020 at 04:48:03PM -0700, Fenghua Yu wrote:
> > When a new mm is created, its PASID should be cleared, i.e. the PASID is
> > in
The following commit has been merged into the x86/urgent branch of tip:
Commit-ID: 8acf417805a5f5c69e9ff66f14cab022c2755161
Gitweb:
https://git.kernel.org/tip/8acf417805a5f5c69e9ff66f14cab022c2755161
Author:Fenghua Yu
AuthorDate:Mon, 01 Feb 2021 19:00:07
Committer
Add Alder Lake mobile processor to CPU list to enumerate and enable the
split lock feature.
Signed-off-by: Fenghua Yu
Reviewed-by: Tony Luck
---
arch/x86/kernel/cpu/intel.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kernel/cpu/intel.c
index
Hi, Dear X86 Maintainers,
>On Mon, Jan 04, 2021 at 07:42:28PM +0000, Fenghua Yu wrote:
>
> On Tue, Nov 24, 2020 at 08:52:41PM +, Fenghua Yu wrote:
> > A bus lock [1] is acquired through either split locked access to
> > writeback (WB) memory or any locked access to n
On Mon, Jan 25, 2021 at 02:52:09PM -0700, Shuah Khan wrote:
> On 1/25/21 1:47 PM, Fenghua Yu wrote:
> > On Mon, Nov 30, 2020 at 08:19:53PM +, Fenghua Yu wrote:
> > > This patch set has several miscellaneous fixes to resctrl selftest tool
> > > that are easily visib
Hi, Shuah,
On Mon, Nov 30, 2020 at 08:19:53PM +, Fenghua Yu wrote:
> This patch set has several miscellaneous fixes to resctrl selftest tool
> that are easily visible to user. V1 had fixes to CAT test and CMT test
> but they were dropped in V2 because having them here made the
On Wed, Jan 13, 2021 at 07:33:33AM +0530, Bhaskar Chowdhury wrote:
> s/kernlfs/kernfs/
> s/@mon_data_kn/@mon_data_kn:/
May change the message to describe the problems like:
Fix typo "kernlfs" and add missing ":" to match with other comments.
>
> Signed-off-by: Bhaskar Chowdhury
> ---
> Changes
The following commit has been merged into the x86/urgent branch of tip:
Commit-ID: ae28d1aae48a1258bd09a6f707ebb4231d79a761
Gitweb:
https://git.kernel.org/tip/ae28d1aae48a1258bd09a6f707ebb4231d79a761
Author:Fenghua Yu
AuthorDate:Thu, 17 Dec 2020 14:31:18 -08:00
Committer
The following commit has been merged into the x86/urgent branch of tip:
Commit-ID: a0195f314a25582b38993bf30db11c300f4f4611
Gitweb:
https://git.kernel.org/tip/a0195f314a25582b38993bf30db11c300f4f4611
Author:Fenghua Yu
AuthorDate:Thu, 17 Dec 2020 14:31:19 -08:00
Committer
Hi, Dear X86 Maintainers,
On Tue, Nov 24, 2020 at 08:52:41PM +, Fenghua Yu wrote:
> A bus lock [1] is acquired through either split locked access to
> writeback (WB) memory or any locked access to non-WB memory. This is
> typically >1000 cycles slower than an atomic operation with
rt MBA or MBA is disabled
Suggested-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/cat_test.c| 3 ---
tools/testing/selftests/resctrl/mba_test.c| 3 ---
tools/testing/selftests/resctrl/mbm_test.c| 3 ---
.../testing/selftests/res
()
function. Adding these wrapper functions makes it easier to add any logic
to each individual test.
Please note that this is a preparatory patch for the next one and no
functional changes are intended.
Suggested-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
.../testing/selftests/resctrl
equest() would mount resctrl
FS to check if mba is enabled on the platform or not and finds that the H/W
doesn't support mba and hence will return false to run_mba_test(). This in
turn makes the main() function return without unmounting resctrl FS.
Signed-off-by: Fenghua Yu
---
tools/
therwise, no_of_bits is equal to half of the maximum number of bits in
the cbm mask.
Please note that CMT test is still hard coded to 5 bits. It will change in
subsequent patches that change CMT test.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/cat_test.c | 5 -
too
test and enable _FORTIFY_SOURCE gcc check to catch any future buffer
overflow errors.
Reported-by: David Binderman
Suggested-by: David Binderman
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/Makefile | 2 +-
tools/testing/selftests/resctrl/cmt_test.c | 2 +-
2 files changed, 2 insertions(
lue and
imc_mbm value is within 5% (randomly chosen value) of imc_mbm value. If the
difference is greater than 5% of imc_mbm value, treat it is a failure.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/mba_test.c | 20 +++-
tools/testing/selftests/resctrl/
on_features has
mbm__bytes
Please note that only L3_CAT, L3_CMT, MBA and MBM are supported. CDP and L2
variants can be added later.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl.h | 6 ++-
tools/testing/selftests/resctrl/r
at
/proc/mounts
Steps 2 and 3 will fail if the platform does not support resctrl file
system. So, there is no need to check for them if step 1 fails.
Fix this by returning immediately if the platform does not support
resctrl file system.
Signed-off-by: Fenghua Yu
---
tools/testing/selftests
platform doesn't support resctrl file system.
Fix this by checking for the return value of check_resctrlfs_support() in
the main function. If resctrl file system isn't supported on the platform
then exit the test suite gracefully without attempting to run any of
resctrl unit tests.
Signed-off
eturn 0, else return
< 0 on error.
Reported-by: Dan Carpenter
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 41 +++
1 file changed, 23 insertions(+), 18 deletions(-)
diff --git a/tools/testing/selftests/resctrl/resctrl_val.c
b/tools/testing/sel
Checking resctrl features call strcmp() to compare feature strings
(e.g. "mba", "cat" etc). The checkings are error prone and don't have
good coding style. Define the constant strings in macros and call
strncmp() to solve the potential issues.
Suggested-by: Shuah Khan
[v1:
https://lore.kernel.org/linux-kselftest/cover.1583657204.git.sai.praneeth.prak...@intel.com/]
Fenghua Yu (15):
selftests/resctrl: Fix compilation issues for global variables
selftests/resctrl: Clean up resctrl features check
selftests/resctrl: Rename CQM test as CMT test
selftests/resctrl:
directory represents an iMC counter, else, skip the
directory.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/resctrl_val.c | 22 +--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/resctrl/resctr
variables to the functions where they are used.
To fix issues for other global variables (e.g: bm_pid, ppid, llc_occup_path
and is_amd) that are used across .c files, declare them as extern.
Reported-by: Reinette Chatre
Signed-off-by: Fenghua Yu
---
tools/testing/selftests/resctrl/cat_te
1 - 100 of 1001 matches
Mail list logo
