From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Phillips Subject: Re: rename *MEMALLOC flags Date: Sun, 13 Aug 2006 17:06:59 -0700 Message-ID: <44DFBEA3.5070305@google.com> References: <20060812141415.30842.78695.sendpatchset@lappy> <20060812141445.30842.47336.sendpatchset@lappy> <44DDE8B6.8000900@garzik.org> <1155395201.13508.44.camel@lappy> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Jeff Garzik , linux-mm@kvack.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Indan Zupancic , Evgeniy Polyakov , Rik van Riel , David Miller Return-path: Received: from smtp-out.google.com ([216.239.45.12]:51354 "EHLO smtp-out.google.com") by vger.kernel.org with ESMTP id S1751438AbWHNAH5 (ORCPT ); Sun, 13 Aug 2006 20:07:57 -0400 To: Peter Zijlstra In-Reply-To: <1155395201.13508.44.camel@lappy> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Peter Zijlstra wrote: >Jeff Garzik in his infinite wisdom spake thusly: >>Peter Zijlstra wrote: >> >>>Index: linux-2.6/include/linux/gfp.h >>>=================================================================== >>>--- linux-2.6.orig/include/linux/gfp.h 2006-08-12 12:56:06.000000000 +0200 >>>+++ linux-2.6/include/linux/gfp.h 2006-08-12 12:56:09.000000000 +0200 >>>@@ -46,6 +46,7 @@ struct vm_area_struct; >>> #define __GFP_ZERO ((__force gfp_t)0x8000u)/* Return zeroed page on success */ >>> #define __GFP_NOMEMALLOC ((__force gfp_t)0x10000u) /* Don't use emergency reserves */ >>> #define __GFP_HARDWALL ((__force gfp_t)0x20000u) /* Enforce hardwall cpuset memory allocs */ >>>+#define __GFP_MEMALLOC ((__force gfp_t)0x40000u) /* Use emergency reserves */ >> >>This symbol name has nothing to do with its purpose. The entire area of >>code you are modifying could be described as having something to do with >>'memalloc'. >> >>GFP_EMERGENCY or GFP_USE_RESERVES or somesuch would be a far better >>symbol name. >> >>I recognize that is matches with GFP_NOMEMALLOC, but that doesn't change >>the situation anyway. In fact, a cleanup patch to rename GFP_NOMEMALLOC >>would be nice. > > I'm rather bad at picking names, but here goes: > > PF_MEMALLOC -> PF_EMERGALLOC > __GFP_NOMEMALLOC -> __GFP_NOEMERGALLOC > __GFP_MEMALLOC -> __GFP_EMERGALLOC > > Is that suitable and shall I prepare patches? Or do we want more ppl to > chime in and have a few more rounds? MEMALLOC is the name Linus chose to name exactly the reserve from which we are allocating. Perhaps that was just Linus being denser than jgarzik and not realizing that he should have called it EMERGALLOC right from the start. BUT since Linus did call it MEMALLOC, we should too. Or just email Linus and tell him how much better EMERGALLOC rolls off the tongue, and could we please change all occurances of MEMALLOC to EMERGALLOC. Then don't read your email for a week ;-) Inventing a new name for an existing thing is very poor taste on grounds of grepability alone. Regards, Daniel