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

Groups > linux.kernel > #1264915 > unrolled thread

Re: [PATCH 08/10] tpm: seal/unseal for TPM 2.0

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 08/10] tpm: seal/unseal for TPM 2.0 Jeremiah Mahler <jmmahler@gmail.com> - 2015-11-07 20:00 +0100
    Re: [PATCH 08/10] tpm: seal/unseal for TPM 2.0 Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> - 2015-11-07 22:50 +0100

#1264915 — Re: [PATCH 08/10] tpm: seal/unseal for TPM 2.0

Jarkko,

On Fri, Oct 16, 2015 at 09:40:27PM +0300, Jarkko Sakkinen wrote:
> Added tpm_trusted_seal() and tpm_trusted_unseal() API for sealing
> trusted keys.
> 
> This patch implements basic sealing and unsealing functionality for
> TPM 2.0:
> 
> * Seal with a parent key using a 20 byte auth value.
> * Unseal with a parent key using a 20 byte auth value.
> 
> Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> ---
>  drivers/char/tpm/tpm-interface.c |  76 ++++++++++++
>  drivers/char/tpm/tpm.h           |  15 ++-
>  drivers/char/tpm/tpm2-cmd.c      | 250 ++++++++++++++++++++++++++++++++++++++-
>  include/keys/trusted-type.h      |   2 +-
>  include/linux/tpm.h              |  26 ++++
>  5 files changed, 366 insertions(+), 3 deletions(-)
> 

This patch adds a lot of code but doesn't use any of it until later
patches.  This can make using 'git bisect' more difficult because it
will point to the wrong patches.  If this were my patch series I would
combine this patch with the later ones that actually start using the
code.

[...]

-- 
- Jeremiah Mahler
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1264938

On Sat, Nov 07, 2015 at 10:58:03AM -0800, Jeremiah Mahler wrote:
> Jarkko,
> 
> On Fri, Oct 16, 2015 at 09:40:27PM +0300, Jarkko Sakkinen wrote:
> > Added tpm_trusted_seal() and tpm_trusted_unseal() API for sealing
> > trusted keys.
> > 
> > This patch implements basic sealing and unsealing functionality for
> > TPM 2.0:
> > 
> > * Seal with a parent key using a 20 byte auth value.
> > * Unseal with a parent key using a 20 byte auth value.
> > 
> > Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > ---
> >  drivers/char/tpm/tpm-interface.c |  76 ++++++++++++
> >  drivers/char/tpm/tpm.h           |  15 ++-
> >  drivers/char/tpm/tpm2-cmd.c      | 250 ++++++++++++++++++++++++++++++++++++++-
> >  include/keys/trusted-type.h      |   2 +-
> >  include/linux/tpm.h              |  26 ++++
> >  5 files changed, 366 insertions(+), 3 deletions(-)
> > 
> 
> This patch adds a lot of code but doesn't use any of it until later
> patches.  This can make using 'git bisect' more difficult because it
> will point to the wrong patches.  If this were my patch series I would
> combine this patch with the later ones that actually start using the
> code.
> 
> [...]

I do agree with you that it'd had been a good idea. Thanks for the
feedback.

> -- 
> - Jeremiah Mahler
> --

/Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web