[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <10338CF38D9A684EA38E1CF7D8D3411B045BA207@dggeml533-mbs.china.huawei.com>
Date: Wed, 25 Mar 2020 02:10:40 +0000
From: "yezhenyu (A)" <yezhenyu2@...wei.com>
To: Steven Rostedt <rostedt@...dmis.org>
CC: "will@...nel.org" <will@...nel.org>,
"mark.rutland@....com" <mark.rutland@....com>,
"catalin.marinas@....com" <catalin.marinas@....com>,
"aneesh.kumar@...ux.ibm.com" <aneesh.kumar@...ux.ibm.com>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"npiggin@...il.com" <npiggin@...il.com>,
"peterz@...radead.org" <peterz@...radead.org>,
"arnd@...db.de" <arnd@...db.de>, "maz@...nel.org" <maz@...nel.org>,
"suzuki.poulose@....com" <suzuki.poulose@....com>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"yuzhao@...gle.com" <yuzhao@...gle.com>,
"Dave.Martin@....com" <Dave.Martin@....com>,
"steven.price@....com" <steven.price@....com>,
"broonie@...nel.org" <broonie@...nel.org>,
"Guohanjun (Hanjun Guo)" <guohanjun@...wei.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-arch@...r.kernel.org" <linux-arch@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
"arm@...nel.org" <arm@...nel.org>,
Xiexiangyou <xiexiangyou@...wei.com>,
"Zengtao (B)" <prime.zeng@...ilicon.com>,
Zhangshaokun <zhangshaokun@...ilicon.com>
Subject: RE: [RFC PATCH v4 4/6] mm: Add page table level flags to vm_flags
Hi Steve,
On Wed, 25 Mar 2020 2:45, Steve wrote:
> On Tue, 24 Mar 2020 21:45:32 +0800
> Zhenyu Ye <yezhenyu2@...wei.com> wrote:
>
> > --- a/include/trace/events/mmflags.h
> > +++ b/include/trace/events/mmflags.h
> > @@ -130,6 +130,16 @@ IF_HAVE_PG_IDLE(PG_idle, "idle" )
> > #define IF_HAVE_VM_SOFTDIRTY(flag,name)
> > #endif
> >
> > +#ifdef CONFIG_64BIT
> > +#define IF_HAVE_VM_LEVEL_PUD(flag,name) {flag, name}
> > +#define IF_HAVE_VM_LEVEL_PMD(flag,name) {flag, name}
> > +#define IF_HAVE_VM_LEVEL_PTE(flag,name) {flag, name}
> > +#else
> > +#define IF_HAVE_VM_LEVEL_PUD(flag,name)
> > +#define IF_HAVE_VM_LEVEL_PMD(flag,name)
> > +#define IF_HAVE_VM_LEVEL_PTE(flag,name)
> > +#endif
> > +
> > #define __def_vmaflag_names \
> > {VM_READ, "read" }, \
> > {VM_WRITE, "write" }, \
> > @@ -161,7 +171,10 @@ IF_HAVE_VM_SOFTDIRTY(VM_SOFTDIRTY,
> "softdirty" ) \
> > {VM_MIXEDMAP, "mixedmap" }, \
> > {VM_HUGEPAGE, "hugepage" }, \
> > {VM_NOHUGEPAGE, "nohugepage" }, \
> > - {VM_MERGEABLE, "mergeable" } \
> > + {VM_MERGEABLE, "mergeable" }, \
> > +IF_HAVE_VM_LEVEL_PUD(VM_LEVEL_PUD, "pud-level" ), \
> > +IF_HAVE_VM_LEVEL_PMD(VM_LEVEL_PMD, "pmd-level" ), \
> > +IF_HAVE_VM_LEVEL_PTE(VM_LEVEL_PTE, "pte-level" ) \
> >
>
> Have you tested this on 32bit? It looks like you'll get empty commas there.
> Perhaps the defines need to be:
>
> #ifdef CONFIG_64BIT
> #define IF_HAVE_VM_LEVEL_PUD(flag,name) {flag, name},
> #define IF_HAVE_VM_LEVEL_PMD(flag,name) {flag, name},
> #define IF_HAVE_VM_LEVEL_PTE(flag,name) {flag, name}
> #else
> #define IF_HAVE_VM_LEVEL_PUD(flag,name)
> #define IF_HAVE_VM_LEVEL_PMD(flag,name)
> #define IF_HAVE_VM_LEVEL_PTE(flag,name)
> #endif
>
> And leave out the commas in the list.
>
> -- Steve
Thanks for your review. I will fix this in next version, if I could still use vm_flags
at that time :).
Zhenyu
Powered by blists - more mailing lists