[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20230517191038.437a3750caa877cde58c054f@linux-foundation.org>
Date: Wed, 17 May 2023 19:10:38 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Kefeng Wang <wangkefeng.wang@...wei.com>
Cc: Mike Rapoport <rppt@...nel.org>, <linux-mm@...ck.org>,
David Hildenbrand <david@...hat.com>,
Oscar Salvador <osalvador@...e.de>,
"Rafael J. Wysocki" <rafael@...nel.org>,
Pavel Machek <pavel@....cz>, Len Brown <len.brown@...el.com>,
Luis Chamberlain <mcgrof@...nel.org>,
Kees Cook <keescook@...omium.org>,
Iurii Zaikin <yzaikin@...gle.com>,
<linux-kernel@...r.kernel.org>, <linux-pm@...r.kernel.org>,
<linux-fsdevel@...r.kernel.org>, <ying.huang@...el.com>
Subject: Re: [PATCH v2 08/13] mm: page_alloc: split out DEBUG_PAGEALLOC
On Thu, 18 May 2023 09:35:29 +0800 Kefeng Wang <wangkefeng.wang@...wei.com> wrote:
> > We're irritatingly inconsistent about whether there's an underscore.
> >
> > akpm:/usr/src/25> grep page_alloc mm/*c|wc -l
> > 49
> > akpm:/usr/src/25> grep pagealloc mm/*c|wc -l
> > 28
>
> All the 28 pagealloc naming is from DEBUG_PAGEALLOC feature, they chould
> be changed to page_alloc except the cmdline, but it will lead to long
> function name and don't gain too much advantage, so keep unchange?
Sure, it's probably not the worst thing in there. I was just having
a moan.
Powered by blists - more mailing lists