Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1592374 > unrolled thread

Re: [PATCH v3 0/3] Locality support for tpm_crb

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v3 0/3] Locality support for tpm_crb Jerry Snitselaar <dev@snitselaar.org> - 2017-03-04 02:40 +0100
    Re: [PATCH v3 0/3] Locality support for tpm_crb Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-03-06 16:40 +0100
      RE: [PATCH v3 0/3] Locality support for tpm_crb "Wei, Gang" <gang.wei@intel.com> - 2017-03-07 17:50 +0100
        Re: [PATCH v3 0/3] Locality support for tpm_crb Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-03-07 19:40 +0100
    Re: [PATCH v3 0/3] Locality support for tpm_crb Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2017-03-07 12:00 +0100

#1592374 — Re: [PATCH v3 0/3] Locality support for tpm_crb

Jarkko Sakkinen @ 2017-02-08 11:11 GMT:

> The tpm_crb driver should follow the policy of reserving and
> relinquishing the locality it uses when multiple localities are used,
> like when TXT is another locality.
>
> [This is a resend of v3 as this has been asked from me for a few times
>  now and there wasn't anything obvious to fix.]
>
> Jarkko Sakkinen (3):
>   tpm_crb: map locality registers
>   tpm_crb: encapsulate crb_wait_for_reg_32
>   tpm_crb: request and relinquish locality 0
>
>  drivers/char/tpm/tpm_crb.c | 185 +++++++++++++++++++++++++++++++++------------
>  1 file changed, 137 insertions(+), 48 deletions(-)

Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>

[toc] | [next] | [standalone]

#1593479

On Fri, Mar 03, 2017 at 06:27:19PM -0700, Jerry Snitselaar wrote:
> 
> Jarkko Sakkinen @ 2017-02-08 11:11 GMT:
> 
> > The tpm_crb driver should follow the policy of reserving and
> > relinquishing the locality it uses when multiple localities are used,
> > like when TXT is another locality.
> >
> > [This is a resend of v3 as this has been asked from me for a few times
> >  now and there wasn't anything obvious to fix.]
> >
> > Jarkko Sakkinen (3):
> >   tpm_crb: map locality registers
> >   tpm_crb: encapsulate crb_wait_for_reg_32
> >   tpm_crb: request and relinquish locality 0
> >
> >  drivers/char/tpm/tpm_crb.c | 185 +++++++++++++++++++++++++++++++++------------
> >  1 file changed, 137 insertions(+), 48 deletions(-)
> 
> Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>

Jimmy, can you give these patches Tested-by? I remember that
you said that they are working for you.

/Jarkko

[toc] | [prev] | [next] | [standalone]

#1594426

On Mon, Mar 06, 2017 at 11:35 PM +0800, Jarkko Sakkinen wrote:
> On Fri, Mar 03, 2017 at 06:27:19PM -0700, Jerry Snitselaar wrote:
> >
> > Jarkko Sakkinen @ 2017-02-08 11:11 GMT:
> >
> > > The tpm_crb driver should follow the policy of reserving and
> > > relinquishing the locality it uses when multiple localities are
> used,
> > > like when TXT is another locality.
> > >
> > > [This is a resend of v3 as this has been asked from me for a few
> times
> > >  now and there wasn't anything obvious to fix.]
> > >
> > > Jarkko Sakkinen (3):
> > >   tpm_crb: map locality registers
> > >   tpm_crb: encapsulate crb_wait_for_reg_32
> > >   tpm_crb: request and relinquish locality 0
> > >
> > >  drivers/char/tpm/tpm_crb.c | 185
> +++++++++++++++++++++++++++++++++------------
> > >  1 file changed, 137 insertions(+), 48 deletions(-)
> >
> > Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>
> 
> Jimmy, can you give these patches Tested-by? I remember that
> you said that they are working for you.

Sure. They did work for me.

Jimmy

[toc] | [prev] | [next] | [standalone]

#1594510

On Tue, Mar 07, 2017 at 04:41:04PM +0000, Wei, Gang wrote:
> On Mon, Mar 06, 2017 at 11:35 PM +0800, Jarkko Sakkinen wrote:
> > On Fri, Mar 03, 2017 at 06:27:19PM -0700, Jerry Snitselaar wrote:
> > >
> > > Jarkko Sakkinen @ 2017-02-08 11:11 GMT:
> > >
> > > > The tpm_crb driver should follow the policy of reserving and
> > > > relinquishing the locality it uses when multiple localities are
> > used,
> > > > like when TXT is another locality.
> > > >
> > > > [This is a resend of v3 as this has been asked from me for a few
> > times
> > > >  now and there wasn't anything obvious to fix.]
> > > >
> > > > Jarkko Sakkinen (3):
> > > >   tpm_crb: map locality registers
> > > >   tpm_crb: encapsulate crb_wait_for_reg_32
> > > >   tpm_crb: request and relinquish locality 0
> > > >
> > > >  drivers/char/tpm/tpm_crb.c | 185
> > +++++++++++++++++++++++++++++++++------------
> > > >  1 file changed, 137 insertions(+), 48 deletions(-)
> > >
> > > Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>
> > 
> > Jimmy, can you give these patches Tested-by? I remember that
> > you said that they are working for you.
> 
> Sure. They did work for me.
> 
> Jimmy

Thank you!

/Jarkko

[toc] | [prev] | [next] | [standalone]

#1594116

On Fri, Mar 03, 2017 at 06:27:19PM -0700, Jerry Snitselaar wrote:
> 
> Jarkko Sakkinen @ 2017-02-08 11:11 GMT:
> 
> > The tpm_crb driver should follow the policy of reserving and
> > relinquishing the locality it uses when multiple localities are used,
> > like when TXT is another locality.
> >
> > [This is a resend of v3 as this has been asked from me for a few times
> >  now and there wasn't anything obvious to fix.]
> >
> > Jarkko Sakkinen (3):
> >   tpm_crb: map locality registers
> >   tpm_crb: encapsulate crb_wait_for_reg_32
> >   tpm_crb: request and relinquish locality 0
> >
> >  drivers/char/tpm/tpm_crb.c | 185 +++++++++++++++++++++++++++++++++------------
> >  1 file changed, 137 insertions(+), 48 deletions(-)
> 
> Reviewed-by: Jerry Snitselaar <jsnitsel@redhat.com>

I decided to push these to my tree and linux-next as I've fixed
reported issues from v2 to get coverage.

/Jarkko

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web