From: David Hildenbrand <david@redhat.com>
To: Christophe Leroy <christophe.leroy@csgroup.eu>,
Maxwell Bland <mbland@motorola.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
"linux-efi@vger.kernel.org" <linux-efi@vger.kernel.org>,
"mst@redhat.com" <mst@redhat.com>,
"catalin.marinas@arm.com" <catalin.marinas@arm.com>,
"dave.hansen@linux.intel.com" <dave.hansen@linux.intel.com>,
"ast@kernel.org" <ast@kernel.org>,
"linux@armlinux.org.uk" <linux@armlinux.org.uk>,
"linux-mm@kvack.org" <linux-mm@kvack.org>,
"ryabinin.a.a@gmail.com" <ryabinin.a.a@gmail.com>,
"guoren@kernel.org" <guoren@kernel.org>,
"sdf@google.com" <sdf@google.com>,
"yonghong.song@linux.dev" <yonghong.song@linux.dev>,
"wuqiang.matt@bytedance.com" <wuqiang.matt@bytedance.com>,
"agordeev@linux.ibm.com" <agordeev@linux.ibm.com>,
"vincenzo.frascino@arm.com" <vincenzo.frascino@arm.com>,
"will@kernel.org" <will@kernel.org>,
"ardb@kernel.org" <ardb@kernel.org>,
"michael.christie@oracle.com" <michael.christie@oracle.com>,
"quic_nprakash@quicinc.com" <quic_nprakash@quicinc.com>,
"linux-arch@vger.kernel.org" <linux-arch@vger.kernel.org>,
"hch@infradead.o rg" <hch@infradead.org>,
"arnd@arndb.de" <arnd@arndb.de>,
"daniel@iogearbox.net" <daniel@iogearbox.net>,
"john.fastabend@gmail.com" <john.fastabend@gmail.com>,
"andrii@kernel.org" <andrii@kernel.org>,
"kasan-dev@googlegroups.com" <kasan-dev@googlegroups.com>,
"aneesh.kumar@kernel.org" <aneesh.kumar@kernel.org>,
"urezki@gmail.com" <urezki@gmail.com>,
"samitolvanen@google.com" <samitolvanen@google.com>,
"zlim.lnx@gmail.com" <zlim.lnx@gmail.com>,
"naveen.n.rao@linux.ibm.com" <naveen.n.rao@linux.ibm.com>,
"dennis@kernel.org" <dennis@kernel.org>,
"borntraeger@linux.ibm.com" <borntraeger@linux.ibm.com>,
"cl@linux.com" <cl@linux.com>,
"aou@eecs.berkeley.edu" <aou@eecs.berkeley.edu>,
"ryan.roberts@arm.com" <ryan.roberts@arm.com>,
"gor@linux.ibm.com" <gor@linux.ibm.com>,
"linux-s390@vger.kernel.org" <linux-s390@vger.kernel.org>,
"hca@linux.ibm.com" <hca@linux.ibm.com>,
"npiggin@gmail.com" <npiggin@gmail.com>,
"kpsingh@kernel.org" <kpsingh@kernel.org>,
"meted@linux.ibm.com" <meted@linux.ibm.c om>,
"quic_pkondeti@quicinc.com" <quic_pkondeti@quicinc.com>,
"paul.walmsley@sifive.com" <paul.walmsley@sifive.com>,
"surenb@google.com" <surenb@google.com>,
"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
"dvyukov@google.com" <dvyukov@google.com>,
"andreyknvl@gmail.com" <andreyknvl@gmail.com>,
"haoluo@google.com" <haoluo@google.com>,
"brauner@kernel.org" <brauner@kernel.org>,
"mjguzik@gmail.com" <mjguzik@gmail.com>,
"lstoakes@gmail.com" <lstoakes@gmail.com>,
"song@kernel.org" <song@kernel.org>,
"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
"muchun.song@linux.dev" <muchun.song@linux.dev>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"awheeler@motorola.com" <awheeler@motorola.com>,
"martin.lau@linux.dev" <martin.lau@linux.dev>,
"linux-riscv@lists.infradead.org"
<linux-riscv@lists.infradead.org>,
"palmer@dabbelt.com" <palmer@dabbelt.com>,
"svens@linux.ibm.com" <svens@linux.ibm.com>,
"jolsa@kernel.org" <jolsa@kernel.org>,
"tj@kernel.org" <tj@kern el.org>,
"glider@google.com" <glider@google.com>,
"bpf@vger.kernel.org" <bpf@vger.kernel.org>,
"rick.p.edgecombe@intel.com" <rick.p.edgecombe@intel.com>,
"linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>
Subject: Re: [PATCH 2/4] mm: pgalloc: support address-conditional pmd allocation
Date: Wed, 21 Feb 2024 10:27:06 +0100 [thread overview]
Message-ID: <cf5409c3-254a-459b-8969-429db2ec6439@redhat.com> (raw)
In-Reply-To: <838a05f0-568d-481d-b826-d2bb61908ace@csgroup.eu>
On 21.02.24 08:13, Christophe Leroy wrote:
>
>
> Le 20/02/2024 à 21:32, Maxwell Bland a écrit :
>> [Vous ne recevez pas souvent de courriers de mbland@motorola.com. Découvrez pourquoi ceci est important à https://aka.ms/LearnAboutSenderIdentification ]
>>
>> While other descriptors (e.g. pud) allow allocations conditional on
>> which virtual address is allocated, pmd descriptor allocations do not.
>> However, adding support for this is straightforward and is beneficial to
>> future kernel development targeting the PMD memory granularity.
>>
>> As many architectures already implement pmd_populate_kernel in an
>> address-generic manner, it is necessary to roll out support
>> incrementally. For this purpose a preprocessor flag,
>
> Is it really worth it ? It is only 48 call sites that need to be
> updated. It would avoid that processor flag and avoid introducing that
> pmd_populate_kernel_at() in kernel core.
+1, let's avoid that if possible.
--
Cheers,
David / dhildenb
next prev parent reply other threads:[~2024-02-21 11:04 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-20 20:32 [PATCH 0/4] arm64: mm: support dynamic vmalloc/pmd configuration Maxwell Bland
2024-02-20 20:32 ` [PATCH 1/4] mm/vmalloc: allow arch-specific vmalloc_node overrides Maxwell Bland
2024-02-21 5:43 ` Christoph Hellwig
2024-02-21 7:38 ` Christophe Leroy
2024-02-21 6:59 ` Christophe Leroy
2024-02-21 17:19 ` Maxwell Bland
2024-02-20 20:32 ` [PATCH 2/4] mm: pgalloc: support address-conditional pmd allocation Maxwell Bland
2024-02-21 7:13 ` Christophe Leroy
2024-02-21 9:27 ` David Hildenbrand [this message]
2024-02-21 15:54 ` [External] " Maxwell Bland
2024-02-20 20:32 ` [PATCH 3/4] arm64: separate code and data virtual memory allocation Maxwell Bland
2024-02-21 7:20 ` Christophe Leroy
2024-02-20 20:32 ` [PATCH 4/4] arm64: dynamic enforcement of pmd-level PXNTable Maxwell Bland
2024-02-21 7:32 ` [PATCH 0/4] arm64: mm: support dynamic vmalloc/pmd configuration Christophe Leroy
2024-02-21 17:57 ` Maxwell Bland
2024-02-21 14:50 ` Conor Dooley
2024-02-21 15:42 ` [External] " Maxwell Bland
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=cf5409c3-254a-459b-8969-429db2ec6439@redhat.com \
--to=david@redhat.com \
--cc=agordeev@linux.ibm.com \
--cc=andrii@kernel.org \
--cc=aneesh.kumar@kernel.org \
--cc=aou@eecs.berkeley.edu \
--cc=ardb@kernel.org \
--cc=arnd@arndb.de \
--cc=ast@kernel.org \
--cc=borntraeger@linux.ibm.com \
--cc=catalin.marinas@arm.com \
--cc=christophe.leroy@csgroup.eu \
--cc=cl@linux.com \
--cc=daniel@iogearbox.net \
--cc=dave.hansen@linux.intel.com \
--cc=dennis@kernel.org \
--cc=gor@linux.ibm.com \
--cc=guoren@kernel.org \
--cc=hca@linux.ibm.com \
--cc=hch@infradead.org \
--cc=john.fastabend@gmail.com \
--cc=kasan-dev@googlegroups.com \
--cc=kpsingh@kernel.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linux-s390@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=mbland@motorola.com \
--cc=meted@linux.ibm.c \
--cc=michael.christie@oracle.com \
--cc=mst@redhat.com \
--cc=naveen.n.rao@linux.ibm.com \
--cc=npiggin@gmail.com \
--cc=quic_nprakash@quicinc.com \
--cc=ryabinin.a.a@gmail.com \
--cc=ryan.roberts@arm.com \
--cc=samitolvanen@google.com \
--cc=sdf@google.com \
--cc=urezki@gmail.com \
--cc=vincenzo.frascino@arm.com \
--cc=will@kernel.org \
--cc=wuqiang.matt@bytedance.com \
--cc=yonghong.song@linux.dev \
--cc=zlim.lnx@gmail.com \
/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;
as well as URLs for NNTP newsgroup(s).