From: "Edgecombe, Rick P" <rick.p.edgecombe@intel.com>
To: "pbonzini@redhat.com" <pbonzini@redhat.com>,
"Hansen, Dave" <dave.hansen@intel.com>,
"seanjc@google.com" <seanjc@google.com>,
"kas@kernel.org" <kas@kernel.org>
Cc: "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
"x86@kernel.org" <x86@kernel.org>
Subject: Re: Process for doing early internal type review on the list
Date: Thu, 27 Aug 2026 00:54:05 +0000 [thread overview]
Message-ID: <575b25a59231cfb7bc5fbaf4bbe45bfedad1e3a9.camel@intel.com> (raw)
In-Reply-To: <827b213a9482b417abb1d172438f0fdbfecd4c7c.camel@intel.com>
Arg, fix Kiryl's email address.
On Wed, 2026-08-26 at 16:59 -0700, Rick Edgecombe wrote:
> 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 prev parent reply other threads:[~2026-08-27 0:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-26 23:59 Process for doing early internal type review on the list Edgecombe, Rick P
2026-08-27 0:54 ` Edgecombe, Rick P [this message]
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=575b25a59231cfb7bc5fbaf4bbe45bfedad1e3a9.camel@intel.com \
--to=rick.p.edgecombe@intel.com \
--cc=dave.hansen@intel.com \
--cc=kas@kernel.org \
--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