linux-kernel - Re: [PATCH AUTOSEL 5.5 04/66] Kernel selftests: tpm2: check for tpm support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200416131234.GA65786@linux.intel.com>
Date:   Thu, 16 Apr 2020 16:12:34 +0300
From:   Jarkko Sakkinen <jarkko.sakkinen@...ux.intel.com>
To:     shuah <shuah@...nel.org>
Cc:     Sasha Levin <sashal@...nel.org>, linux-kernel@...r.kernel.org,
        stable@...r.kernel.org,
        Nikita Sobolev <Nikita.Sobolev@...opsys.com>,
        Shuah Khan <skhan@...uxfoundation.org>,
        linux-kselftest@...r.kernel.org
Subject: Re: [PATCH AUTOSEL 5.5 04/66] Kernel selftests: tpm2: check for tpm
 support

On Tue, Apr 14, 2020 at 10:04:40AM -0600, shuah wrote:
> On 3/2/20 7:45 PM, Sasha Levin wrote:
> > From: Nikita Sobolev <Nikita.Sobolev@...opsys.com>
> > 
> > [ Upstream commit b32694cd0724d4ceca2c62cc7c3d3a8d1ffa11fc ]
> > 
> > tpm2 tests set fails if there is no /dev/tpm0 and /dev/tpmrm0
> > supported. Check if these files exist before run and mark test as
> > skipped in case of absence.
> > 
> > Signed-off-by: Nikita Sobolev <Nikita.Sobolev@...opsys.com>
> > Signed-off-by: Shuah Khan <skhan@...uxfoundation.org>
> > Signed-off-by: Sasha Levin <sashal@...nel.org>
> > ---
> >   tools/testing/selftests/tpm2/test_smoke.sh | 13 +++++++++++--
> >   tools/testing/selftests/tpm2/test_space.sh |  9 ++++++++-
> >   2 files changed, 19 insertions(+), 3 deletions(-)
> > 
> 
> Let's drop this unless it already made it into stables.
> Jarkko Sakkinen found regressions and send in a revert
> for this patch.
> 
> thanks,
> -- Shuah

Thanks, can ack this.

/Jarkko