linux-kernel - Re: [PATCH] tpm: allow TCPA log area to be empty

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <49C2A39F-C9ED-4620-82C9-C0919DA3F2CD@fb.com>
Date:   Tue, 27 Mar 2018 00:07:14 +0000
From:   Sahil Rihan <srihan@...com>
To:     Jarkko Sakkinen <jarkko.sakkinen@...ux.intel.com>,
        Jason Gunthorpe <jgg@...pe.ca>
CC:     "linux-integrity@...r.kernel.org" <linux-integrity@...r.kernel.org>,
        "linux-security-module@...r.kernel.org" 
        <linux-security-module@...r.kernel.org>,
        Alexei Starovoitov <ast@...com>,
        "stable@...r.kernel.org" <stable@...r.kernel.org>,
        Peter Huewe <peterhuewe@....de>, Arnd Bergmann <arnd@...db.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] tpm: allow TCPA log area to be empty

> On 3/22/18, 7:20 AM, "Jarkko Sakkinen" <jarkko.sakkinen@...ux.intel.com> wrote:
>
>   On Tue, 2018-03-20 at 22:29 +0000, Sahil Rihan wrote:
>   > > On 3/19/18, 2:25 PM, "Jarkko Sakkinen" <jarkko.sakkinen@...ux.intel.com>
>   > > wrote:
>   > >     
>   > >    Sahil, would you care to try if this fixes the issue that you are
>   > >    having or not?
>   > 
>   > Yes, we can test this out. Might take a day or two.
>    
>   Awesome, thank you. I'll add
>    
>   Reported-by: Sahil Rihan <srihan@...com>
>    
>   if that is OK for you?

Jarkko, 

We applied your patch to a 4.11 kernel (unfortunately not the latest upstream) and were able to confirm that the char device is now being created correctly. The sysfs enabled node is also present.

And yes, please add the "Reported-by" line.

Thanks,
Sahil