mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg@ziepe.ca>
To: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
Cc: "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
	"linux-coco@lists.linux.dev" <linux-coco@lists.linux.dev>,
	"drdeeglaze@gmail.com" <drdeeglaze@gmail.com>,
	"James.Bottomley@hansenpartnership.com"
	<James.Bottomley@hansenpartnership.com>,
	"aik@amd.com" <aik@amd.com>,
	"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
	"Xing, Cedric" <cedric.xing@intel.com>,
	"alistair.francis@wdc.com" <alistair.francis@wdc.com>,
	"linux-api@vger.kernel.org" <linux-api@vger.kernel.org>,
	"Reshetova, Elena" <elena.reshetova@intel.com>,
	"kas@kernel.org" <kas@kernel.org>,
	"ankita@nvidia.com" <ankita@nvidia.com>,
	"jiri@resnulli.us" <jiri@resnulli.us>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Jiang, Dave" <dave.jiang@intel.com>,
	"Quigley, David" <david.quigley@intel.com>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	"x86@kernel.org" <x86@kernel.org>,
	"jpiotrowski@nvidia.com" <jpiotrowski@nvidia.com>,
	"suzuki.poulose@arm.com" <suzuki.poulose@arm.com>,
	"lukas@wunner.de" <lukas@wunner.de>,
	"Vivi, Rodrigo" <rodrigo.vivi@intel.com>,
	"Fang, Peter" <peter.fang@intel.com>,
	"jic23@kernel.org" <jic23@kernel.org>,
	"ksinyuk@kernel.org" <ksinyuk@kernel.org>,
	"sameo@meta.com" <sameo@meta.com>,
	"sathyanarayanan.kuppuswamy@linux.intel.com"
	<sathyanarayanan.kuppuswamy@linux.intel.com>,
	"qinkun@google.com" <qinkun@google.com>,
	"pgonda@google.com" <pgonda@google.com>
Subject: Re: [PATCH RFC 20/20] attestation: Add the TDX family implementation
Date: Tue, 10 Nov 2026 09:11:14 -0400	[thread overview]
Message-ID: <20261110131114.GV663069@ziepe.ca> (raw)
In-Reply-To: <4c75b369b68c98c5c35e296895b6935342dc7dc1.camel@intel.com>

On Tue, Nov 10, 2026 at 01:30:16AM +0000, Edgecombe, Rick P wrote:
> On Mon, 2026-11-09 at 19:42 -0400, Jason Gunthorpe wrote:
> > > So we have this SGX based quote format that is fixed because it already
> > > exists
> > > in the wild. Going forward we are moving to DICE, but that is a standard
> > > format.
> > > So also somewhat fixed. I guess the question is: is there some combo of bits
> > > in
> > > each blob we could check that could deterministically distinguish between
> > > the
> > > two? To do this we would need to analyze the quote formats.
> > 
> > You guys really didn't make it discoverable when you added DICE? :( :(
> 
> Like influence the DICE spec to interop with legacy SGX based attestation? I'm
> out of my depth to try to say how possible that is.

No, I mean just have the SGX firmware tell the OS if it produced a
DICE or legacy blob :(

> I'm also under the impression that the quote was not designed for local
> processing in the guest. But rather remote attestation which, usually involves a
> helper service that processes the quote. I've never been clear on why that was
> desired. But it probably is part of the reason attestation doesn't fit this use
> case perfectly.

Even the remote helper needs to tell the two cases apart so it can
parse them

Again if you expected the user to pre-tell the helper that this is a
legacy vs DICE VM then that's exactly the bad user experiance I want
to avoid.

> > > If there isn't, I'm not sure what we could do at this point. There is a
> > > library
> > > Intel publishes (right Peter?) that can verify the quotes. I'd guess it must
> > > have some way to distinguish between the two. Can you check how it does it
> > > Peter?
> > 
> > See this is exactly what I'm talking about, I don't want to repeatedly
> > get into this situation down the road over the next decade.
> 
> I'm not sure if I'm missing something. New incompatible standards seem pretty
> normal? Sometimes there is a container to specify which type of blob, and
> sometimes it is just discoverable by clues in the bits.

Yes, completely normal. The thing that generates the data should have
an unambiguous contract with the OS on what kind of data it will
generate.

If you want to change it, thats fine, but the OS participates in the
change and is aware of it.

> I think the thing that is a bit dicey (no pun) with this, is that there is kind
> of a backwards compatibility adjacent change for systems giving the new DICE
> quotes. This came up in in both discussions with Dave/x86 and Sean/KVM. The
> reason why it was considered grumble-ok in the end, was because the full
> verification depends on updates to the library for each new platform due to
> needing to be aware of new keys. So to actually attest even SGX quotes on a new
> platform, library updates are needed to whatever tool or service is being used.
> Since the library needs to be updated in any case, it can also understand the
> new formats without any impact to the user.

I don't know what you guys have done here. There are IETF standards
for all of this and well defined container formats. Sure to parse
every bit of the token you might need a new library, but you should at
least be able to get a signature verified and parse the generic
information reliably.

If your plan is TDX releases are ABI incompatible with the verifiers,
then thats a horrible plan and *much* worse than what ARM is doing,
for instance.

But it sounds like your library can somehow parse the blob and figure
it out, so maybe it is just a matter of describing how that works.

> But this didn't take into account this generic API, local quote processing use
> case. Which is kind of a surprise and I'm actually still not 100% clear on how
> it will be used.

It is not local quote processing. This is labeling data properly so a
verifier is possible without the user having to manually and directly
inform the system what versions of Intel CPUs and TDX modules it
should expect to have.

> I think we would have to circle back with everyone that pushed for the host
> controlled quote type. Let's see if the library can distinguish them already
> too. This attestctl tool... is that something that could consume a library like
> this? Or something that would have a no dependency implementation?

I would hope for no dependency, the idea would be to provide
references for anyone wanting to make verifier software, and a tool
for the admin that wants to debug/understand their systems..

As well as a point to have a strict injection of sanity into what
people are doing so we don't get repeated ABI breaking instances in
future.

Jason

  reply	other threads:[~2026-11-10 13:11 UTC|newest]

Thread overview: 208+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-15 14:50 [PATCH RFC 00/20] attestation: Add a common subsystem and TDX support Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 01/20] ctlv: Add the schema language and generator Jiri Pirko
2026-10-20 22:31   ` Jonathan Cameron
2026-10-21 13:23     ` Jiri Pirko
2026-10-27  8:34     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 02/20] ctlv: Add the UAPI wire format and family API Jiri Pirko
2026-10-19 23:29   ` Jason Gunthorpe
2026-10-20 12:26     ` Jiri Pirko
2026-10-20 23:00   ` Jonathan Cameron
2026-10-20 23:39     ` Jonathan Cameron
2026-10-21 13:54     ` Jiri Pirko
2026-10-26 18:55   ` Yeoreum Yun
2026-10-27  8:02     ` Jiri Pirko
2026-10-27 14:15       ` Jason Gunthorpe
2026-10-27 14:48         ` Yeoreum Yun
2026-10-27 15:17           ` Jason Gunthorpe
2026-10-27 15:55             ` Yeoreum Yun
2026-11-05 13:11             ` Jiri Pirko
2026-11-05 13:51               ` Yeoreum Yun
2026-11-05 16:53                 ` Yeoreum Yun
2026-10-15 14:50 ` [PATCH RFC 03/20] ctlv: Add the root family specification and generated files Jiri Pirko
2026-10-19 23:29   ` Jason Gunthorpe
2026-10-20 12:28     ` Jiri Pirko
2026-10-20 12:55       ` Jason Gunthorpe
2026-10-20 14:07         ` Jiri Pirko
2026-10-20 23:30   ` Jonathan Cameron
2026-10-21 16:25     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 04/20] Documentation: ctlv: Add userspace ABI Jiri Pirko
2026-10-19 23:29   ` Jason Gunthorpe
2026-10-20 14:06     ` Jiri Pirko
2026-10-21  0:11   ` Jonathan Cameron
2026-10-22  9:01     ` Jiri Pirko
2026-10-23 19:23       ` Jonathan Cameron
2026-10-29 18:53   ` Sergii Dmytruk
2026-11-03 10:30     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 05/20] ctlv: Prepare message input handling Jiri Pirko
2026-10-21  0:43   ` Jonathan Cameron
2026-10-22  8:27     ` Jiri Pirko
2026-10-22 18:39     ` Jason Gunthorpe
2026-10-23 12:55       ` Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-27 10:17     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 06/20] ctlv: Prepare blob descriptor validation Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 07/20] ctlv: Prepare schema validation Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 08/20] ctlv: Prepare message output handling Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-11-02 19:19     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 09/20] ctlv: Prepare VA blob handling Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 10/20] ctlv: Prepare event queue support Jiri Pirko
2026-10-21  1:12   ` Jonathan Cameron
2026-10-22  9:04     ` Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-26 14:45     ` Jiri Pirko
2026-10-27 12:38   ` Yeoreum Yun
2026-11-02 12:49     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 11/20] ctlv: Prepare runtime introspection Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-26 14:52     ` Jiri Pirko
2026-10-26 16:12       ` Jason Gunthorpe
2026-10-30  9:40         ` Jiri Pirko
2026-10-27 10:21     ` Jiri Pirko
2026-10-30  9:53     ` Jiri Pirko
2026-11-03  1:05       ` Jason Gunthorpe
2026-11-03  9:29         ` Jiri Pirko
2026-11-04 17:57           ` Jason Gunthorpe
2026-10-15 14:50 ` [PATCH RFC 12/20] ctlv: Add device and operation dispatch Jiri Pirko
2026-10-21  1:25   ` Jonathan Cameron
2026-10-22  9:05     ` Jiri Pirko
2026-10-22 18:43     ` Jason Gunthorpe
2026-10-23 12:58       ` Jiri Pirko
2026-10-23 14:26         ` Jason Gunthorpe
2026-10-26 13:48           ` Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-27 13:38     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 13/20] Documentation: ctlv: Describe how to write a family Jiri Pirko
2026-10-19 23:29   ` Jason Gunthorpe
2026-10-21 14:23     ` Jiri Pirko
2026-10-21 14:49       ` Jason Gunthorpe
2026-10-21 16:24         ` Jiri Pirko
2026-10-21 16:48           ` Jason Gunthorpe
2026-10-22  8:22             ` Jiri Pirko
2026-10-22 19:21               ` Jason Gunthorpe
2026-10-23 13:01                 ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 14/20] attestation: Add the base family specification and generated files Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-29 10:04     ` Jiri Pirko
2026-10-29 18:43       ` Jason Gunthorpe
2026-10-29 19:38         ` Jiri Pirko
2026-10-30 15:02     ` Jiri Pirko
2026-10-30 15:05     ` Jiri Pirko
2026-11-04 17:54       ` Jason Gunthorpe
2026-11-05 12:35         ` Jiri Pirko
2026-11-05 13:52     ` Jiri Pirko
2026-11-05 14:02     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 15/20] attestation: Prepare provider and instance core Jiri Pirko
2026-10-19 23:29   ` Jason Gunthorpe
2026-10-21 13:04     ` Jiri Pirko
2026-10-29 18:50   ` Sergii Dmytruk
2026-11-04  8:22     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 16/20] attestation: Prepare measurement register journal Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 17/20] attestation: Add CTLV operation handling Jiri Pirko
2026-11-04 19:11   ` Yeoreum Yun
2026-11-04 19:46     ` Jason Gunthorpe
2026-11-05 12:32       ` Jiri Pirko
2026-11-05 13:52         ` Yeoreum Yun
2026-11-06  8:27           ` Jiri Pirko
2026-11-06  8:57             ` Yeoreum Yun
2026-11-06 14:25               ` Jiri Pirko
2026-11-06 14:36                 ` Yeoreum Yun
2026-11-06 16:03                   ` Jiri Pirko
2026-11-06 16:19                     ` Yeoreum Yun
2026-11-07 11:01                       ` Yeoreum Yun
2026-11-07 17:25                       ` Jiri Pirko
2026-11-07 18:22                         ` Yeoreum Yun
2026-11-09  8:20                           ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 18/20] tdx-guest: Expose the native attestation service API Jiri Pirko
2026-10-26 14:07   ` Jason Gunthorpe
2026-10-27 10:03     ` Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 19/20] attestation: Add the TDX family specification and generated files Jiri Pirko
2026-10-15 14:50 ` [PATCH RFC 20/20] attestation: Add the TDX family implementation Jiri Pirko
2026-10-22 22:07   ` Edgecombe, Rick P
2026-10-23 14:12     ` Jason Gunthorpe
2026-10-23 19:40       ` Edgecombe, Rick P
2026-10-23 22:00         ` Jason Gunthorpe
2026-11-02 19:02           ` Edgecombe, Rick P
2026-11-03  0:58             ` Jason Gunthorpe
2026-11-06 21:47               ` Peter Fang
2026-11-09 17:12                 ` Jason Gunthorpe
2026-11-09 23:20                   ` Edgecombe, Rick P
2026-11-09 23:42                     ` Jason Gunthorpe
2026-11-10  1:30                       ` Edgecombe, Rick P
2026-11-10 13:11                         ` Jason Gunthorpe [this message]
2026-10-23 18:15     ` Jiri Pirko
2026-10-23 19:41       ` Edgecombe, Rick P
2026-10-16 23:43 ` [PATCH RFC 00/20] attestation: Add a common subsystem and TDX support Jason Gunthorpe
2026-10-17 18:16   ` Dionna Glaze
2026-10-17 19:06     ` Jarkko Sakkinen
2026-10-17 19:08       ` Jarkko Sakkinen
2026-10-17 19:22         ` Jarkko Sakkinen
2026-10-17 20:55       ` Jason Gunthorpe
2026-10-17 21:14         ` Jarkko Sakkinen
2026-10-17 20:34     ` Jason Gunthorpe
2026-10-17 20:55       ` Jarkko Sakkinen
2026-10-17 23:10         ` Jason Gunthorpe
2026-10-18 23:53           ` Dionna Glaze
2026-10-19 12:08             ` Jiri Pirko
2026-10-19 16:10               ` Dionna Glaze
2026-10-19 22:56                 ` Jason Gunthorpe
2026-10-21  8:33                   ` Jiri Pirko
2026-10-21 14:12                     ` Mimi Zohar
2026-10-21 14:35                       ` Jason Gunthorpe
2026-10-21 19:05                         ` Mimi Zohar
2026-10-22 10:54                           ` Jiri Pirko
2026-10-22 11:24                             ` Mimi Zohar
2026-10-22 18:51                               ` Jason Gunthorpe
2026-10-22 19:19                                 ` Jarkko Sakkinen
2026-10-22 19:23                                   ` Jason Gunthorpe
2026-10-22 19:40                                     ` Jarkko Sakkinen
2026-10-22 19:40                                     ` James Bottomley
2026-10-22 12:26                         ` James Bottomley
2026-10-22 13:11                           ` Jason Gunthorpe
2026-10-22 14:07                             ` James Bottomley
2026-10-22 19:19                               ` Jason Gunthorpe
2026-10-22 19:37                                 ` Mimi Zohar
2026-10-22 19:45                                   ` Jason Gunthorpe
2026-10-22 19:52                                     ` James Bottomley
2026-10-22 20:43                                       ` Mimi Zohar
2026-10-20  9:26                 ` Jiri Pirko
2026-10-20 13:45                 ` Mimi Zohar
2026-10-20 14:06                   ` Jason Gunthorpe
2026-10-20 15:22                     ` Mimi Zohar
2026-10-20 16:11                       ` Yeoreum Yun
2026-10-20 16:52                         ` Jason Gunthorpe
2026-10-21 12:23                           ` Mimi Zohar
2026-10-21 13:18                             ` Jarkko Sakkinen
2026-10-21 13:50                               ` Yeoreum Yun
2026-10-21 13:51                             ` Jason Gunthorpe
2026-10-21 14:31                               ` Mimi Zohar
2026-10-21 14:39                                 ` Jason Gunthorpe
2026-10-21 12:48                         ` James Bottomley
2026-10-21 13:55                           ` Jason Gunthorpe
2026-10-19 12:40             ` Jarkko Sakkinen
2026-10-19 23:11             ` Jason Gunthorpe
2026-10-20 12:49       ` James Bottomley
2026-10-20 13:13         ` Jason Gunthorpe
2026-10-22 10:30 ` Aneesh Kumar K.V
2026-10-22 15:38   ` Jiri Pirko
2026-10-22 16:42     ` Jason Gunthorpe
2026-10-23  8:04     ` Aneesh Kumar K.V
2026-10-23 11:32       ` Jiri Pirko
2026-10-23 14:20       ` Jason Gunthorpe
2026-10-23 15:28         ` Aneesh Kumar K.V
2026-10-23 15:42           ` Jason Gunthorpe
2026-10-23 19:13             ` Jonathan Cameron
2026-11-03  8:38         ` Jiri Pirko
2026-10-27 18:43 ` Dave Jiang
2026-10-27 19:36   ` Jason Gunthorpe
2026-10-27 23:09     ` Dave Jiang
2026-10-27 23:46       ` Jason Gunthorpe
2026-10-28 19:16         ` Dave Jiang
2026-10-29 14:09           ` Jason Gunthorpe
2026-10-29  9:32         ` Jiri Pirko
2026-10-29  9:30       ` Jiri Pirko
2026-10-29 10:07   ` Jiri Pirko
2026-10-29  7:37 ` Nikolay Borisov
2026-10-29 14:16   ` Jason Gunthorpe
2026-10-29 18:43 ` Sergii Dmytruk
2026-11-03 10:10   ` Jiri Pirko

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=20261110131114.GV663069@ziepe.ca \
    --to=jgg@ziepe.ca \
    --cc=James.Bottomley@hansenpartnership.com \
    --cc=aik@amd.com \
    --cc=alistair.francis@wdc.com \
    --cc=ankita@nvidia.com \
    --cc=cedric.xing@intel.com \
    --cc=dave.hansen@linux.intel.com \
    --cc=dave.jiang@intel.com \
    --cc=david.quigley@intel.com \
    --cc=drdeeglaze@gmail.com \
    --cc=elena.reshetova@intel.com \
    --cc=jic23@kernel.org \
    --cc=jiri@resnulli.us \
    --cc=jpiotrowski@nvidia.com \
    --cc=kas@kernel.org \
    --cc=ksinyuk@kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-coco@lists.linux.dev \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lukas@wunner.de \
    --cc=peter.fang@intel.com \
    --cc=pgonda@google.com \
    --cc=qinkun@google.com \
    --cc=rick.p.edgecombe@intel.com \
    --cc=rodrigo.vivi@intel.com \
    --cc=sameo@meta.com \
    --cc=sathyanarayanan.kuppuswamy@linux.intel.com \
    --cc=suzuki.poulose@arm.com \
    --cc=x86@kernel.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

all inboxes | Powered by JetHome®