* [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
@ 2020-04-12 14:23 Jarkko Sakkinen
2020-04-14 15:12 ` shuah
0 siblings, 1 reply; 6+ messages in thread
From: Jarkko Sakkinen @ 2020-04-12 14:23 UTC (permalink / raw)
To: Shuah Khan
Cc: linux-kselftest, linux-integrity, Jarkko Sakkinen,
Nikita Sobolev, Tadeusz Struk, open list
This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
The original comment was neither reviewed nor tested. Thus, this the
*only* possible action to take.
Cc: Nikita Sobolev <Nikita.Sobolev@synopsys.com>
Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
---
tools/testing/selftests/tpm2/test_smoke.sh | 13 ++-----------
tools/testing/selftests/tpm2/test_space.sh | 9 +--------
2 files changed, 3 insertions(+), 19 deletions(-)
diff --git a/tools/testing/selftests/tpm2/test_smoke.sh b/tools/testing/selftests/tpm2/test_smoke.sh
index b630c7b5950a..8155c2ea7ccb 100755
--- a/tools/testing/selftests/tpm2/test_smoke.sh
+++ b/tools/testing/selftests/tpm2/test_smoke.sh
@@ -1,17 +1,8 @@
#!/bin/bash
# SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
-self.flags = flags
-# Kselftest framework requirement - SKIP code is 4.
-ksft_skip=4
-
-
-if [ -f /dev/tpm0 ] ; then
- python -m unittest -v tpm2_tests.SmokeTest
- python -m unittest -v tpm2_tests.AsyncTest
-else
- exit $ksft_skip
-fi
+python -m unittest -v tpm2_tests.SmokeTest
+python -m unittest -v tpm2_tests.AsyncTest
CLEAR_CMD=$(which tpm2_clear)
if [ -n $CLEAR_CMD ]; then
diff --git a/tools/testing/selftests/tpm2/test_space.sh b/tools/testing/selftests/tpm2/test_space.sh
index 180b469c53b4..a6f5e346635e 100755
--- a/tools/testing/selftests/tpm2/test_space.sh
+++ b/tools/testing/selftests/tpm2/test_space.sh
@@ -1,11 +1,4 @@
#!/bin/bash
# SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause)
-# Kselftest framework requirement - SKIP code is 4.
-ksft_skip=4
-
-if [ -f /dev/tpmrm0 ] ; then
- python -m unittest -v tpm2_tests.SpaceTest
-else
- exit $ksft_skip
-fi
+python -m unittest -v tpm2_tests.SpaceTest
--
2.25.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
2020-04-12 14:23 [PATCH] Revert "Kernel selftests: tpm2: check for tpm support" Jarkko Sakkinen
@ 2020-04-14 15:12 ` shuah
2020-04-14 19:37 ` Jarkko Sakkinen
0 siblings, 1 reply; 6+ messages in thread
From: shuah @ 2020-04-14 15:12 UTC (permalink / raw)
To: Jarkko Sakkinen
Cc: linux-kselftest, linux-integrity, Nikita Sobolev, Tadeusz Struk,
open list, shuah
On 4/12/20 8:23 AM, Jarkko Sakkinen wrote:
> This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
>
> The original comment
commit I assume.
was neither reviewed nor tested. Thus, this the
> *only* possible action to take.
>
Bummer. I will apply this right away.
> Cc: Nikita Sobolev <Nikita.Sobolev@synopsys.com>
> Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> ---
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
2020-04-14 15:12 ` shuah
@ 2020-04-14 19:37 ` Jarkko Sakkinen
2020-04-14 19:38 ` Jarkko Sakkinen
0 siblings, 1 reply; 6+ messages in thread
From: Jarkko Sakkinen @ 2020-04-14 19:37 UTC (permalink / raw)
To: shuah
Cc: linux-kselftest, linux-integrity, Nikita Sobolev, Tadeusz Struk,
open list
On Tue, Apr 14, 2020 at 09:12:05AM -0600, shuah wrote:
> On 4/12/20 8:23 AM, Jarkko Sakkinen wrote:
> > This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
> >
> > The original comment
>
> commit I assume.
>
> was neither reviewed nor tested. Thus, this the
> > *only* possible action to take.
> >
>
> Bummer. I will apply this right away.
No worries, and fine with the change as long as I'm able to
review and try it out :-) Sorry for the dull tone!
/Jarkko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
2020-04-14 19:37 ` Jarkko Sakkinen
@ 2020-04-14 19:38 ` Jarkko Sakkinen
2020-04-14 20:52 ` shuah
0 siblings, 1 reply; 6+ messages in thread
From: Jarkko Sakkinen @ 2020-04-14 19:38 UTC (permalink / raw)
To: shuah
Cc: linux-kselftest, linux-integrity, Nikita Sobolev, Tadeusz Struk,
open list
On Tue, Apr 14, 2020 at 10:37:25PM +0300, Jarkko Sakkinen wrote:
> On Tue, Apr 14, 2020 at 09:12:05AM -0600, shuah wrote:
> > On 4/12/20 8:23 AM, Jarkko Sakkinen wrote:
> > > This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
> > >
> > > The original comment
> >
> > commit I assume.
> >
> > was neither reviewed nor tested. Thus, this the
> > > *only* possible action to take.
> > >
> >
> > Bummer. I will apply this right away.
>
> No worries, and fine with the change as long as I'm able to
> review and try it out :-) Sorry for the dull tone!
I mean refined change after this revert has been applied.
/Jarkko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
2020-04-14 19:38 ` Jarkko Sakkinen
@ 2020-04-14 20:52 ` shuah
2020-04-16 17:01 ` Jarkko Sakkinen
0 siblings, 1 reply; 6+ messages in thread
From: shuah @ 2020-04-14 20:52 UTC (permalink / raw)
To: Jarkko Sakkinen
Cc: linux-kselftest, linux-integrity, Nikita Sobolev, Tadeusz Struk,
open list, shuah
On 4/14/20 1:38 PM, Jarkko Sakkinen wrote:
> On Tue, Apr 14, 2020 at 10:37:25PM +0300, Jarkko Sakkinen wrote:
>> On Tue, Apr 14, 2020 at 09:12:05AM -0600, shuah wrote:
>>> On 4/12/20 8:23 AM, Jarkko Sakkinen wrote:
>>>> This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
>>>>
>>>> The original comment
>>>
>>> commit I assume.
>>>
>>> was neither reviewed nor tested. Thus, this the
>>>> *only* possible action to take.
>>>>
>>>
>>> Bummer. I will apply this right away.
>>
>> No worries, and fine with the change as long as I'm able to
>> review and try it out :-) Sorry for the dull tone!
>
I usually wait for test authors to do the review. This patch
was sent a while back and I ended up picking it up. It stayed
in next for a while. It is fixed now.
> I mean refined change after this revert has been applied.
>
Of course. I applied your revert to for my next pull request
to linux-kselftest next
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Revert "Kernel selftests: tpm2: check for tpm support"
2020-04-14 20:52 ` shuah
@ 2020-04-16 17:01 ` Jarkko Sakkinen
0 siblings, 0 replies; 6+ messages in thread
From: Jarkko Sakkinen @ 2020-04-16 17:01 UTC (permalink / raw)
To: shuah
Cc: linux-kselftest, linux-integrity, Nikita Sobolev, Tadeusz Struk,
open list
On Tue, Apr 14, 2020 at 02:52:12PM -0600, shuah wrote:
> On 4/14/20 1:38 PM, Jarkko Sakkinen wrote:
> > On Tue, Apr 14, 2020 at 10:37:25PM +0300, Jarkko Sakkinen wrote:
> > > On Tue, Apr 14, 2020 at 09:12:05AM -0600, shuah wrote:
> > > > On 4/12/20 8:23 AM, Jarkko Sakkinen wrote:
> > > > > This reverts commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc.
> > > > >
> > > > > The original comment
> > > >
> > > > commit I assume.
> > > >
> > > > was neither reviewed nor tested. Thus, this the
> > > > > *only* possible action to take.
> > > > >
> > > >
> > > > Bummer. I will apply this right away.
> > >
> > > No worries, and fine with the change as long as I'm able to
> > > review and try it out :-) Sorry for the dull tone!
> >
>
> I usually wait for test authors to do the review. This patch
> was sent a while back and I ended up picking it up. It stayed
> in next for a while. It is fixed now.
>
> > I mean refined change after this revert has been applied.
> >
> Of course. I applied your revert to for my next pull request
> to linux-kselftest next
>
> thanks,
> -- Shuah
Thanks.
/Jarkko
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-04-16 17:01 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-12 14:23 [PATCH] Revert "Kernel selftests: tpm2: check for tpm support" Jarkko Sakkinen
2020-04-14 15:12 ` shuah
2020-04-14 19:37 ` Jarkko Sakkinen
2020-04-14 19:38 ` Jarkko Sakkinen
2020-04-14 20:52 ` shuah
2020-04-16 17:01 ` Jarkko Sakkinen
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).