From: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
To: "kas@kernel.og" <kas@kernel.og>,
"pbonzini@redhat.com" <pbonzini@redhat.com>,
"Hansen, Dave" <dave.hansen@intel.com>,
"seanjc@google.com" <seanjc@google.com>
Cc: "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"x86@kernel.org" <x86@kernel.org>
Subject: Process for doing early internal type review on the list
Date: Wed, 26 Aug 2026 23:59:24 +0000 [thread overview]
Message-ID: <827b213a9482b417abb1d172438f0fdbfecd4c7c.camel@intel.com> (raw)
Sean has long been pushing TDX developers to review less internally and capture
everything on the list. For several reasons:
- Avoid thrash by hashing out things internally that are just overturned later.
- Capture all the design decisions in lore history.
- Help developers grow by getting comfortable working and making mistakes in
public.
- A strong personal conviction that through some confluence of intangibles,
working in public produces better outcomes for the kernel?
At this point we pretty much do all TDX KVM patch review externally. But there
were a couple straggling cases we ran into recently where not working in public
produced snags:
1. Far out things that are in POC stages (e.g. DICE, and next migration)
2. Things that are held internally to not overwhelm the list with too much TDX
stuff at once, while also continuing to make some progress on them. (TDX huge
pages)
In an off-list discussion, the idea came up to have something like RFC, but to
denote that the posting was for early "get it ready" type review. That it was
being shared only to capture discussion, but not garner attention. It could be
used for these straggling categories to do the review in public, but just off to
the side. It would be an optional thing instead of internal review. Not
required, just encouraged when possible. That kind of idea.
Sean suggested a "FUTURE" tag like "[PATCH FUTURE]". He didn't like "PREP"
because it could be confused for a patch that was preparatory for another patch.
I don't love FUTURE because is time focused, rather than about the purpose of
sending the patches. How about "PREVIEW"?
Dave, I'm wondering if you would want to have a similar process for tip targeted
TDX series?
Thanks,
Rick
next reply other threads:[~2026-08-26 23:59 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-26 23:59 Edgecombe, Rick P [this message]
2026-08-27 0:54 ` Process for doing early internal type review on the list Edgecombe, Rick P
2026-08-27 19:02 ` Sean Christopherson
2026-08-27 19:09 ` Borislav Petkov
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=827b213a9482b417abb1d172438f0fdbfecd4c7c.camel@intel.com \
--to=rick.p.edgecombe@intel.com \
--cc=dave.hansen@intel.com \
--cc=kas@kernel.og \
--cc=kvm@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=seanjc@google.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