public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jarkko Sakkinen <jarkko@kernel.org>
To: Jonathan McDowell <noodles@earth.li>
Cc: James Bottomley <James.Bottomley@hansenpartnership.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Peter Huewe <peterhuewe@gmx.de>, Jason Gunthorpe <jgg@ziepe.ca>,
	David Howells <dhowells@redhat.com>,
	keyrings@vger.kernel.org, linux-integrity@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [GIT PULL] TPM DEVICE DRIVER: tpmdd-next-v6.18
Date: Mon, 6 Oct 2025 21:50:11 +0300	[thread overview]
Message-ID: <aOQPYwrGwZfP-GsV@kernel.org> (raw)
In-Reply-To: <aOP2DUj67yB0afUt@earth.li>

On Mon, Oct 06, 2025 at 06:02:05PM +0100, Jonathan McDowell wrote:
> On Mon, Oct 06, 2025 at 07:51:46PM +0300, Jarkko Sakkinen wrote:
> > My main issue preventing sending a new pull request is that weird list
> > of core TPM2 features that is claimed "not to be required" with zero
> > references. Especially it is contraditory claim that TPM2_CreatePrimary
> > would be optional feature as the whole chip standard is based on three
> > random seeds from which primary keys are templated and used as root
> > keys for other keys.
> 
> My understanding here is that the main specification about what's 
> "required" for TPMs to implement is from the PC Client Platform TPM 
> Profile. There's no specific server PTP (though there is talk about 
> creating one), so _most_ vendors just implement the PC Client PTP. It 
> doesn't mean a TPM that doesn't do so isn't TPM compliant, just not PC 
> Client PTP compliant.
> 
> Google have taken the approach in their Titan based TPM implementation 
> to avoid implementing features they don't need, to reduce attack 
> surface.

If it is an internal product, it does not qualify for as an argument but
good that you brought this detail out. I mean the action is the same
and debate is really what are correct preconditions for taking any
action.

So I'll cover the next trial for PR with:

1. I'll retain my existing commit with no changes.
2. In the cover letter I can address the details brough by Chris with
   the clause that Titan specific arguments are not basis for any
   decision making, as it is wrong and does not scale given that
   any possible company in this planet can have their own random
   incompatible partial TPM implementation (and most likely there
   are at least few of such).
3. I'll link Chris' message to the email. I.e. based on other
   claims in that mail we can already fully justify the action
   itself. I'm just knowingly ignoring that list of "incompatible"
   features, and remark that so that the process is transparent.

> 
> I'm not aware of anyone else who has done this.
> 
> J.
> 
> -- 
> ... You non-conformists are all alike.

BR, Jarkko

  reply	other threads:[~2025-10-06 18:50 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-05 15:47 [GIT PULL] TPM DEVICE DRIVER: tpmdd-next-v6.18 Jarkko Sakkinen
2025-10-05 18:09 ` Linus Torvalds
2025-10-06 11:58   ` Jarkko Sakkinen
2025-10-06 14:12     ` Jarkko Sakkinen
2025-10-06 14:18       ` Jarkko Sakkinen
2025-10-06 14:30         ` Jarkko Sakkinen
2025-10-06 14:33       ` James Bottomley
2025-10-06 16:51         ` Jarkko Sakkinen
2025-10-06 16:57           ` Jarkko Sakkinen
2025-10-07 14:32             ` Jarkko Sakkinen
2025-10-07 14:38               ` Jarkko Sakkinen
2025-10-06 17:02           ` Jonathan McDowell
2025-10-06 18:50             ` Jarkko Sakkinen [this message]
2025-10-05 18:24 ` Linus Torvalds
2025-10-06 12:33   ` Jarkko Sakkinen
2025-10-06 21:40     ` Jonathan McDowell
2025-10-06 22:09       ` Linus Torvalds
2025-10-06 23:11       ` James Bottomley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aOQPYwrGwZfP-GsV@kernel.org \
    --to=jarkko@kernel.org \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=dhowells@redhat.com \
    --cc=jgg@ziepe.ca \
    --cc=keyrings@vger.kernel.org \
    --cc=linux-integrity@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=noodles@earth.li \
    --cc=peterhuewe@gmx.de \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox