Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751752AbWHNAH6 (ORCPT ); Sun, 13 Aug 2006 20:07:58 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751745AbWHNAH6 (ORCPT ); Sun, 13 Aug 2006 20:07:58 -0400 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 DomainKey-Signature: a=rsa-sha1; s=beta; d=google.com; c=nofws; q=dns; h=received:message-id:date:from:user-agent: x-accept-language:mime-version:to:cc:subject:references:in-reply-to: content-type:content-transfer-encoding; b=TYkF7OHFf5oK/adARXZAdAFff2mrx5oRhJhv3RP4VdEaEHM+m8DpqS9mp2syrh6EU tgh7ja2NdVrE0jFa1xKeQ== Message-ID: <44DFBEA3.5070305@google.com> Date: Sun, 13 Aug 2006 17:06:59 -0700 From: Daniel Phillips User-Agent: Mozilla Thunderbird 1.0.8 (X11/20060502) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Peter Zijlstra 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 Subject: Re: rename *MEMALLOC flags References: <20060812141415.30842.78695.sendpatchset@lappy> <20060812141445.30842.47336.sendpatchset@lappy> <44DDE8B6.8000900@garzik.org> <1155395201.13508.44.camel@lappy> In-Reply-To: <1155395201.13508.44.camel@lappy> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2314 Lines: 55 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 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/