From: "Vishal Moola (Oracle)" <vishal.moola@gmail.com>
To: Dave Hansen <dave.hansen@intel.com>
Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, x86@kernel.org,
"Mike Rapoport (Microsoft)" <rppt@kernel.org>,
Dave Hansen <dave.hansen@linux.intel.com>,
"Matthew Wilcox (Oracle)" <willy@infradead.org>,
akpm@linux-foundation.org, Andy Lutomirski <luto@kernel.org>,
Peter Zijlstra <peterz@infradead.org>,
Thomas Gleixner <tglx@kernel.org>, Ingo Molnar <mingo@redhat.com>,
Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [PATCH v6 0/4] Convert x86/mm/pat to generic page table apis
Date: Mon, 2 Mar 2026 12:27:21 -0800 [thread overview]
Message-ID: <aaXyqZpenNthBTq4@fedora> (raw)
In-Reply-To: <bf7d3f8b-8cd0-420b-b418-8b45bcfbcfcc@intel.com>
On Mon, Mar 02, 2026 at 09:33:31AM -0800, Dave Hansen wrote:
> On 2/27/26 16:44, Vishal Moola (Oracle) wrote:
> > On Fri, Feb 27, 2026 at 02:24:33PM -0800, Dave Hansen wrote:
> >> On 2/27/26 12:07, Vishal Moola (Oracle) wrote:
> >>> Do you have any more comments for this series? If not, could you please
> >>> take this through the x86 tree. Thanks 🙂
> >>
> >> From a 2-second glance, this doesn't have normal comment style and isn't
> >> using imperative voice. Quite a few "we's". Looks like it needs some
> >> more eyeballs.
> >
> > Ok, my mistake on the comment style. I'll fix that.
> >
> > Sorry I'm not the greatest at imperative voice, I'll pass all my comments
> > through AI to convert them all to imperative voice.
> >
> > In regards to more eyeballs, I've realized I didn't cc ALL x86
> > maintainers. +CC the rest of the x86 maintainers and we'll hope
> > someone else looks at it as well.
>
> I was actually kinda hoping you'd be able to actively go out and find
> some reviewers other than the maintainers.
I'd appreciate if you could give me more guidance on this. Who am I
supposed to go, if not the people from ./get_maintainers? I don't know
the x86 folks, I interact more with mm.
You've been giving valuable feedback to ensure the stylistic choices
meet your expectations. I have mentioned I'll have another version
*trying* to meet those stylistic expectations.
Mike has been very active in reviewing the ptdesc work (thanks!), and
he's provided valuable review about the code itself. I don't know about
you, but I trust his expertise on the topic of page tables.
So, I don't even know what you want from the "more eyeballs."
Do you want me to email a bunch of people unfamiliar with x86 and tell
them to catch what you (or any other specialized individual) would
catch? That doesn't sound productive *to me*.
This is on the x86 mailing list. And honestly, this current iteration is
simple enough that anyone specialized and interested could have taken a
look at it and to provide some feedback.
Please do let me know. I'd love to work with you not against you, but
that's difficult when you don't provide any guidance/pointers about the
issues you have with the patchset (or process?).
next prev parent reply other threads:[~2026-03-02 20:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-19 2:03 Vishal Moola (Oracle)
2026-02-19 2:03 ` [PATCH v6 1/4] x86/mm/pat: Convert pte code to use " Vishal Moola (Oracle)
2026-02-19 2:03 ` [PATCH v6 2/4] x86/mm/pat: Convert pmd " Vishal Moola (Oracle)
2026-02-19 2:03 ` [PATCH v6 3/4] x86/mm/pat: Convert populate_pgd() " Vishal Moola (Oracle)
2026-02-19 2:03 ` [PATCH v6 4/4] x86/mm/pat: Convert split_large_page() to use ptdescs Vishal Moola (Oracle)
2026-02-20 10:25 ` [PATCH v6 0/4] Convert x86/mm/pat to generic page table apis Mike Rapoport
2026-02-27 20:07 ` Vishal Moola (Oracle)
2026-02-27 22:24 ` Dave Hansen
2026-02-28 0:44 ` Vishal Moola (Oracle)
2026-03-02 17:33 ` Dave Hansen
2026-03-02 20:27 ` Vishal Moola (Oracle) [this message]
2026-03-02 20:55 ` Dave Hansen
2026-03-02 22:43 ` Vishal Moola (Oracle)
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=aaXyqZpenNthBTq4@fedora \
--to=vishal.moola@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=bp@alien8.de \
--cc=dave.hansen@intel.com \
--cc=dave.hansen@linux.intel.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=luto@kernel.org \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=rppt@kernel.org \
--cc=tglx@kernel.org \
--cc=willy@infradead.org \
--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