From: Jarkko Sakkinen Subject: Re: [PATCH v2] tpm: Move Linux RNG connection to hwrng Date: Fri, 17 Nov 2017 15:28:53 +0200 Message-ID: <20171117132853.cmr3wmsc3biedv7p@linux.intel.com> References: <20171031200503.GC18578@ziepe.ca> <20171105110506.usxmuzrvcjvxahr6@linux.intel.com> <20171106022704.GD26011@ziepe.ca> <20171107160404.GG21466@ziepe.ca> <20171114143421.3jxn6fzvgp2xmyk6@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: PrasannaKumar Muralidharan , Stefan Berger , linux-integrity@vger.kernel.org, David Howells , Herbert Xu , Dmitry Kasatkin , open list , "open list:INTEGRITY MEASUREMENT ARCHITECTURE (IMA)" , "open list:KEYS-TRUSTED" , "open list:HARDWARE RANDOM NUMBER GENERATOR CORE" , James Morris , Matt Mackall , David Safford , Mimi Zohar , "Serge E. Hallyn" To: Jason Gunthorpe Return-path: Received: from mga03.intel.com ([134.134.136.65]:36395 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934046AbdKQN3A (ORCPT ); Fri, 17 Nov 2017 08:29:00 -0500 Content-Disposition: inline In-Reply-To: <20171114143421.3jxn6fzvgp2xmyk6@linux.intel.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Tue, Nov 14, 2017 at 04:34:21PM +0200, Jarkko Sakkinen wrote: > On Tue, Nov 07, 2017 at 09:04:04AM -0700, Jason Gunthorpe wrote: > > On Tue, Nov 07, 2017 at 08:50:44AM +0530, PrasannaKumar Muralidharan wrote: > > > > > I am assuming you are talking about the following patches - using > > > struct tpm_chip instead of chip number and this patch. > > > > yes > > > > > I won't be able to test if struct tpm_chip usage as I don't have > > > multiple tpm hw in one machine. In case of tpm rng changes I can test > > > only the lifecycle of tpm rng device. Is that enough? I feel my test > > > will be limited. Please provide your thoughts on this. > > > > That is certainly better than no testing. > > > > Jason > > WFM too. > > Tested-by: Jarkko Sakkinen > Reviewed-by: Jarkko Sakkinen > > /Jarkko I applied these commits. Please check that everything is correct as I had to do manual work with the 2nd commit. /Jarkko