<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel.git/tools/testing/selftests/powerpc/pmu, branch linux-5.10.y</title>
<subtitle>Hosts the 0x221E linux distro kernel.
</subtitle>
<id>https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.10.y</id>
<link rel='self' href='https://git.0xinfinity.dev/distro/kernel.git/atom?h=linux-5.10.y'/>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/'/>
<updated>2021-07-20T14:05:43Z</updated>
<entry>
<title>selftests/powerpc: Fix "no_handler" EBB selftest</title>
<updated>2021-07-20T14:05:43Z</updated>
<author>
<name>Athira Rajeev</name>
</author>
<published>2021-05-25T13:51:42Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=b7f4423c7de87893ce5f4e9ca559ad50f9986e4c'/>
<id>urn:sha1:b7f4423c7de87893ce5f4e9ca559ad50f9986e4c</id>
<content type='text'>
[ Upstream commit 45677c9aebe926192e59475b35a1ff35ff2d4217 ]

The "no_handler_test" in ebb selftests attempts to read the PMU
registers twice via helper function "dump_ebb_state". First dump is
just before closing of event and the second invocation is done after
closing of the event. The original intention of second
dump_ebb_state was to dump the state of registers at the end of
the test when the counters are frozen. But this will be achieved
with the first call itself since sample period is set to low value
and PMU will be frozen by then. Hence patch removes the
dump which was done before closing of the event.

Reported-by: Shirisha Ganta &lt;shirisha.ganta1@ibm.com&gt;
Signed-off-by: Athira Rajeev &lt;atrajeev@linux.vnet.ibm.com&gt;
Tested-by: Nageswara R Sastry &lt;rnsastry@linux.ibm.com &lt;mailto:rnsastry@linux.ibm.com&gt;&gt;
Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/1621950703-1532-2-git-send-email-atrajeev@linux.vnet.ibm.com
Signed-off-by: Sasha Levin &lt;sashal@kernel.org&gt;
</content>
</entry>
<entry>
<title>selftests/powerpc: Skip L3 bank test on older CPUs</title>
<updated>2020-09-08T12:24:08Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-08-19T01:57:25Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=4871a10b7b5f6b0632bff229884dad1cb1e8dc37'/>
<id>urn:sha1:4871a10b7b5f6b0632bff229884dad1cb1e8dc37</id>
<content type='text'>
This is a test of specific piece of logic in isa207-common.c, which is
only used on Power8 or later. So skip it on older CPUs.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200819015727.1977134-7-mpe@ellerman.id.au
</content>
</entry>
<entry>
<title>selftests/powerpc: Include asm/cputable.h from utils.h</title>
<updated>2020-09-08T12:23:59Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-08-19T01:57:22Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=178282a054dced1a08a9683d41ac08cbace2b2fe'/>
<id>urn:sha1:178282a054dced1a08a9683d41ac08cbace2b2fe</id>
<content type='text'>
utils.h provides have_hwcap() and have_hwcap2() which check for a
feature bit. Those bits are defined in asm/cputable.h, so include it
in utils.h so users of utils.h don't have to do it manually.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200819015727.1977134-4-mpe@ellerman.id.au
</content>
</entry>
<entry>
<title>selftests/powerpc: Skip vmx/vsx/tar/etc tests on older CPUs</title>
<updated>2020-08-03T12:12:41Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-08-03T02:07:19Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=872d11bca9c29ed19595c993b9f552ffe9b63dcb'/>
<id>urn:sha1:872d11bca9c29ed19595c993b9f552ffe9b63dcb</id>
<content type='text'>
Some of our tests use VSX or newer VMX instructions, so need to be
skipped on older CPUs to avoid SIGILL'ing.

Similarly TAR was added in v2.07, and the PMU event used in the stcx
fail test only works on Power8 or later.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200803020719.96114-1-mpe@ellerman.id.au
</content>
</entry>
<entry>
<title>selftests/powerpc: Add test of stack expansion logic</title>
<updated>2020-07-29T11:02:11Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-07-24T09:25:24Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=c9938a9dac95be7650218cdd8e9d1f882e7b5691'/>
<id>urn:sha1:c9938a9dac95be7650218cdd8e9d1f882e7b5691</id>
<content type='text'>
We have custom stack expansion checks that it turns out are extremely
badly tested and contain bugs, surprise. So add some tests that
exercise the code and capture the current boundary conditions.

The signal test currently fails on 64-bit kernels because the 2048
byte allowance for the signal frame is too small, we will fix that in
a subsequent patch.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200724092528.1578671-1-mpe@ellerman.id.au
</content>
</entry>
<entry>
<title>selftests/powerpc: Run per_event_excludes test on Power8 or later</title>
<updated>2020-07-21T14:01:23Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-07-16T12:21:42Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=9d1ebe9a98c1d7bf7cfbe1dba0052230c042ecdb'/>
<id>urn:sha1:9d1ebe9a98c1d7bf7cfbe1dba0052230c042ecdb</id>
<content type='text'>
The per_event_excludes test wants to run on Power8 or later. But
currently it checks that AT_BASE_PLATFORM *equals* power8, which means
it only runs on Power8.

Fix it to check for the ISA 2.07 feature, which will be set on Power8
and later CPUs.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200716122142.3776261-1-mpe@ellerman.id.au
</content>
</entry>
<entry>
<title>Merge branch 'fixes' into next</title>
<updated>2020-07-18T12:43:55Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-07-18T12:43:55Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=ef9f7cfaa50025068c69cdee8efccee12d9fabfc'/>
<id>urn:sha1:ef9f7cfaa50025068c69cdee8efccee12d9fabfc</id>
<content type='text'>
Merge our fixes branch, primarily to bring in the ebb selftests build
fix and the pkey fix, which is a dependency for some future work.
</content>
</entry>
<entry>
<title>selftests/powerpc: Purge extra count_pmc() calls of ebb selftests</title>
<updated>2020-07-08T12:09:31Z</updated>
<author>
<name>Desnes A. Nunes do Rosario</name>
</author>
<published>2020-06-26T16:47:37Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=3337bf41e0dd70b4064cdf60acdfcdc2d050066c'/>
<id>urn:sha1:3337bf41e0dd70b4064cdf60acdfcdc2d050066c</id>
<content type='text'>
An extra count on ebb_state.stats.pmc_count[PMC_INDEX(pmc)] is being per-
formed when count_pmc() is used to reset PMCs on a few selftests. This
extra pmc_count can occasionally invalidate results, such as the ones from
cycles_test shown hereafter. The ebb_check_count() failed with an above
the upper limit error due to the extra value on ebb_state.stats.pmc_count.

Furthermore, this extra count is also indicated by extra PMC1 trace_log on
the output of the cycle test (as well as on pmc56_overflow_test):

==========
   ...
   [21]: counter = 8
   [22]: register SPRN_MMCR0 = 0x0000000080000080
   [23]: register SPRN_PMC1  = 0x0000000080000004
   [24]: counter = 9
   [25]: register SPRN_MMCR0 = 0x0000000080000080
   [26]: register SPRN_PMC1  = 0x0000000080000004
   [27]: counter = 10
   [28]: register SPRN_MMCR0 = 0x0000000080000080
   [29]: register SPRN_PMC1  = 0x0000000080000004
&gt;&gt; [30]: register SPRN_PMC1  = 0x000000004000051e
PMC1 count (0x280000546) above upper limit 0x2800003e8 (+0x15e)
[FAIL] Test FAILED on line 52
failure: cycles
==========

Signed-off-by: Desnes A. Nunes do Rosario &lt;desnesn@linux.ibm.com&gt;
Tested-by: Sachin Sant &lt;sachinp@linux.vnet.ibm.com&gt;
Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200626164737.21943-1-desnesn@linux.ibm.com
</content>
</entry>
<entry>
<title>selftests/powerpc: Fix build failure in ebb tests</title>
<updated>2020-06-26T02:53:09Z</updated>
<author>
<name>Harish</name>
</author>
<published>2020-06-25T16:57:21Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=896066aa0685af3434637998b76218c2045142a8'/>
<id>urn:sha1:896066aa0685af3434637998b76218c2045142a8</id>
<content type='text'>
We use OUTPUT directory as TMPOUT for checking no-pie option.

Since commit f2f02ebd8f38 ("kbuild: improve cc-option to clean up all
temporary files") when building powerpc/ from selftests directory, the
OUTPUT directory points to powerpc/pmu/ebb/ and gets removed when
checking for -no-pie option in try-run routine, subsequently build
fails with the following:

  $ make -C powerpc
  ...
  TARGET=ebb; BUILD_TARGET=$OUTPUT/$TARGET; mkdir -p $BUILD_TARGET; make OUTPUT=$BUILD_TARGET -k -C $TARGET all
  make[2]: Entering directory '/home/linux-master/tools/testing/selftests/powerpc/pmu/ebb'
  make[2]: *** No rule to make target 'Makefile'.
  make[2]: Failed to remake makefile 'Makefile'.
  make[2]: *** No rule to make target 'ebb.c', needed by '/home/linux-master/tools/testing/selftests/powerpc/pmu/ebb/reg_access_test'.
  make[2]: *** No rule to make target 'ebb_handler.S', needed by '/home/linux-master/tools/testing/selftests/powerpc/pmu/ebb/reg_access_test'.
  make[2]: *** No rule to make target 'trace.c', needed by '/home/linux-master/tools/testing/selftests/powerpc/pmu/ebb/reg_access_test'.
  make[2]: *** No rule to make target 'busy_loop.S', needed by '/home/linux-master/tools/testing/selftests/powerpc/pmu/ebb/reg_access_test'.
  make[2]: Target 'all' not remade because of errors.

Fix this by adding a suffix to the OUTPUT directory so that the
failure is avoided.

Fixes: 9686813f6e9d ("selftests/powerpc: Fix try-run when source tree is not writable")
Signed-off-by: Harish &lt;harish@linux.ibm.com&gt;
[mpe: Mention that commit that triggered the breakage]
Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200625165721.264904-1-harish@linux.ibm.com
</content>
</entry>
<entry>
<title>selftests/powerpc: Add a test of counting larx/stcx</title>
<updated>2020-05-15T01:58:55Z</updated>
<author>
<name>Michael Ellerman</name>
</author>
<published>2020-04-26T11:44:10Z</published>
<link rel='alternate' type='text/html' href='https://git.0xinfinity.dev/distro/kernel.git/commit/?id=7481cad4747303442209bc5dba2f56c3afcea07d'/>
<id>urn:sha1:7481cad4747303442209bc5dba2f56c3afcea07d</id>
<content type='text'>
This is based on the count_instructions test.

However this one also counts the number of failed stcx's, and in
conjunction with knowing the size of the stcx loop, can calculate the
total number of instructions executed even in the face of
non-deterministic stcx failures.

Signed-off-by: Michael Ellerman &lt;mpe@ellerman.id.au&gt;
Link: https://lore.kernel.org/r/20200426114410.3917383-1-mpe@ellerman.id.au
</content>
</entry>
</feed>
