Kernel KVM virtualization development
 help / color / mirror / Atom feed
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


  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