From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Stefan Berger" Subject: Re: ioctl API for vTPM driver Date: Sun, 6 Mar 2016 11:21:42 -0500 Message-ID: <201603061621.u26GLuQK014398@d03av04.boulder.ibm.com> References: <20160306120537.GA22702@intel.com><20160306121103.GA23480@intel.com> <20160306122816.GA24077@intel.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1376215373662174564==" Return-path: In-Reply-To: <20160306122816.GA24077-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: tpmdd-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Jarkko Sakkinen Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: tpmdd-devel@lists.sourceforge.net --===============1376215373662174564== Content-Type: multipart/alternative; boundary="=_alternative 0059E35185257F6E_=" --=_alternative 0059E35185257F6E_= Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="US-ASCII" Jarkko Sakkinen wrote on 03/06/2016=20 07:28:16 AM: > From: Jarkko Sakkinen > To: Stefan Berger > Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org > Date: 03/06/2016 07:29 AM > Subject: Re: [tpmdd-devel] ioctl API for vTPM driver >=20 > On Sun, Mar 06, 2016 at 02:11:03PM +0200, Jarkko Sakkinen wrote: > > On Sun, Mar 06, 2016 at 02:05:37PM +0200, Jarkko Sakkinen wrote: > > > Hi > > >=20 > > > Some follow-up question that have popped up while I've started to > > > write a test program for this feature mainly about ioctl API. It's > > > better to be extremly cautious here because we will be stuck with=20 this > > > ioctl forever. > > >=20 > > > 1. Why the ioctl name is VTPM=5FNEW=5FDEV but the struct name is > > > vtpm=5Fnew=5Fpair? It would be better if they both were either > > > VTPM=5FNEW=5FDEV and vtpm=5Fnew=5Fdev or alternatively VTPM=5FNEW= =5FPAIR > > > and vtpm=5Fnew=5Fpair. Renamed it to vtpm=5Fnew=5Fdev. > > > 2. Is 'pair' or 'tuple' a better term? > > > 3. Where is the documentation for the ioctl? I don't think I can > > > merge this to my next branch before it exists. I'll add some later today or tomorrow. > > > 4. I have forgotten why the major and minor numbers were returned. > > > My guess is that they were returned so that a container could > > > replicate the device? This is one reason why documentation is > > > mandatory. Yes, it's there so that inside a container a device with that major and=20 minor number can be created and device cgroups be setup. > >=20 > > 5. Is there any particular reason why 'tpm=5Fdev=5Fnum' couldn't simply= be > > 'dev=5Fnum'? Renamed to 'tpm=5Fnum' Updated my git repo. >=20 > I think you should still send a patch set after documentation has been > completed and we have addressed these items to the following mailing > lists: >=20 > * linux-kernel > * linux-api > * linux-doc > * tpmdd The whole patchset as 'v5' ? >=20 > Now these patches have circled only in the tpmdd mailing list which > is not sufficient. I just realized this recently that the relevant > mailing lists were completely missing (while starting to write a > test program). >=20 > For patches up to TPM=5FCHIP=5FFLAG=5FVIRTUAL you could downloaded patches > from my repository before posting the patch set since they include > also my reviewed/tested-by's. Ok, will sync. Stefan >=20 > Thanks. >=20 > /Jarkko >=20 >=20 ---------------------------------------------------------------------------= --- > =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F= =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F > tpmdd-devel mailing list > tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org > https://lists.sourceforge.net/lists/listinfo/tpmdd-devel >=20 --=_alternative 0059E35185257F6E_= Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset="US-ASCII" Jarkko Sakkinen <jarkko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> wrote on 03/06/2016 07:28:16 AM:

> From: Jarkko Sakkinen <jark= ko.sakkinen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> To: = Stefan Berger <stefanb-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
> Cc: tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> Date: 03/06/2016 07:29 AM
= > Subject: Re: [tpmdd-devel] ioctl API for vTPM driver
>
> On Sun, Mar 06, 2016= at 02:11:03PM +0200, Jarkko Sakkinen wrote:
> > On Sun, Mar 06, 2= 016 at 02:05:37PM +0200, Jarkko Sakkinen wrote:
> > > Hi
>= ; > >
> > > Some follow-up question that have popped up = while I've started to
> > > write a test program for this feature mainly about ioc= tl API.  It's
> > > better to be extremly cautious here becau= se we will be stuck with this
> > > ioctl forever.
> > >
> > = > 1. Why the ioctl name is VTPM=5FNEW=5FDEV but the struct name is
> > >    vtpm=5Fnew=5Fpair? It would be better if = they both were either
> > >    VTPM=5FNEW=5FDEV and vtpm=5Fnew= =5Fdev or alternatively VTPM=5FNEW=5FPAIR
> > >    and vtpm=5Fnew=5Fpair.

Renamed it to vtpm=5Fnew=5Fdev.

> > > 2. Is 'pair' or 'tuple' a bett= er term?
> > > 3. Where is the d= ocumentation for the ioctl? I don't think I can
> > >    merge this to my = next branch before it exists.

I'll ad= d some later today or tomorrow.

> = > > 4. I have forgotten why the major and minor numbers were returned.
> > >    My guess is that they were returne= d so that a container could
> > >    replicate the device? This= is one reason why documentation is
> > >    mandatory.
<= br>Yes, it's there so that inside a container a device with that major and minor number can be created and device cgroups be setup= .

> >
> > 5. Is there= any particular reason why 'tpm=5Fdev=5Fnum' couldn't simply be
> >    'dev=5Fnum'?

Renamed to 'tpm=5Fnum' Updated my git repo.
=
>
> I think you should still send a patch set = after documentation has been
> completed and we have addressed these items to the following m= ailing
> lists:
>
> * linux-kernel
> * linux-api> * linux-doc
> * tpmdd

Th= e whole patchset as 'v5' ?

>
&= gt; Now these patches have circled only in the tpmdd mailing list which
= > is not sufficient. I just realized this recently that the relevant
= > mailing lists were completely missing (while starting to write a
&g= t; test program).
>
> For patches up to TPM=5FCHIP=5FFLAG=5FVI= RTUAL you could downloaded patches
> from my repository before postin= g the patch set since they include
> also my reviewed/tested-by's.

Ok, will sync.

<= font size=3D2>    Stefan

&g= t;
> Thanks.
>
> /Jarkko
>
> -------------= -----------------------------------------------------------------
> = =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F= =5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F=5F
> = tpmdd-devel mailing list
> tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> = https://lists.sourceforge.net/lists/listinfo/tpmdd= -devel
>

--=_alternative 0059E35185257F6E_=-- --===============1376215373662174564== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline ------------------------------------------------------------------------------ --===============1376215373662174564== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ tpmdd-devel mailing list tpmdd-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org https://lists.sourceforge.net/lists/listinfo/tpmdd-devel --===============1376215373662174564==--