From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gary R Hook Subject: Re: [PATCHv2 0/2] Add new SEV commands Date: Thu, 31 May 2018 07:43:13 -0500 Message-ID: <8210dfee-3370-743a-6e9e-96df0ceadf11@amd.com> References: <20180530162819.4xbwb3jbbx5dey24@gondor.apana.org.au> <79caff3c-1002-9fea-1ab6-954062435633@amd.com> <20180531064534.ldi2ysuhh2omex5d@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Janakarajan Natarajan , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, Tom Lendacky , "David S . Miller" , Brijesh Singh , Borislav Petkov , Paolo Bonzini To: Herbert Xu , Gary R Hook Return-path: In-Reply-To: <20180531064534.ldi2ysuhh2omex5d@gondor.apana.org.au> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org On 05/31/2018 01:45 AM, Herbert Xu wrote: > On Wed, May 30, 2018 at 01:44:12PM -0500, Gary R Hook wrote: >> On 05/30/2018 11:28 AM, Herbert Xu wrote: >>> On Fri, May 25, 2018 at 03:23:28PM -0500, Janakarajan Natarajan wrote: >>>> This patchset adds two new SEV commands, introduced in SEV API v0.15 >>>> and v0.16 respectively. >>>> >>>> * DOWNLOAD_FIRMWARE allows the SEV firmware to be updated if a blob newer >>>> than or similar to the exisiting build is available. >>>> >>>> * GET_ID allows to query for a unique ID that can be used to retrieve the >>>> Chip Endorsment Key (CEK) public key signed by the AMD SEV Signing Key >>>> (ASK). >>>> >>>> v1->v2: >>>> * Added cover letter. >>>> * Misc changes based on Boris' feedback. >>>> >>>> Janakarajan Natarajan (2): >>>> crypto: ccp: Add DOWNLOAD_FIRMWARE SEV command >>>> crypto: ccp: Add GET_ID SEV command >>>> >>>> drivers/crypto/ccp/psp-dev.c | 143 ++++++++++++++++++++++++++++++++++++++++--- >>>> drivers/crypto/ccp/psp-dev.h | 4 ++ >>>> include/linux/psp-sev.h | 23 +++++++ >>>> include/uapi/linux/psp-sev.h | 12 ++++ >>>> 4 files changed, 172 insertions(+), 10 deletions(-) >>> >>> All applied. Thanks. >> >> Huh. Without a Reviewed-by or an Acked-by? > > These patches have been out there for weeks. If you were going > to object you should have done so by now. Oh, yes, agreed. This was a question about protocol, not the patches themselves. But if you're happy, I'm happy. Thanks.