From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936605AbdEWIid (ORCPT ); Tue, 23 May 2017 04:38:33 -0400 Received: from mx2.suse.de ([195.135.220.15]:40993 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S935886AbdEWIiT (ORCPT ); Tue, 23 May 2017 04:38:19 -0400 Subject: Re: [PATCH] mm: Define KB, MB, GB, TB in core VM To: Christoph Hellwig , Andrew Morton Cc: Anshuman Khandual , linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <20170522111742.29433-1-khandual@linux.vnet.ibm.com> <20170522141149.9ef84bb0713769f4af0383f0@linux-foundation.org> <20170523070227.GA27864@infradead.org> From: Vlastimil Babka Message-ID: <09a6bafa-5743-425e-8def-bd9219cd756c@suse.cz> Date: Tue, 23 May 2017 10:38:17 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <20170523070227.GA27864@infradead.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/23/2017 09:02 AM, Christoph Hellwig wrote: > On Mon, May 22, 2017 at 02:11:49PM -0700, Andrew Morton wrote: >> On Mon, 22 May 2017 16:47:42 +0530 Anshuman Khandual wrote: >> >>> There are many places where we define size either left shifting integers >>> or multiplying 1024s without any generic definition to fall back on. But >>> there are couples of (powerpc and lz4) attempts to define these standard >>> memory sizes. Lets move these definitions to core VM to make sure that >>> all new usage come from these definitions eventually standardizing it >>> across all places. >> >> Grep further - there are many more definitions and some may now >> generate warnings. >> >> Newly including mm.h for these things seems a bit heavyweight. I can't >> immediately think of a more appropriate place. Maybe printk.h or >> kernel.h. > > IFF we do these kernel.h is the right place. And please also add the > MiB & co variants for the binary versions right next to the decimal > ones. Those defined in the patch are binary, not decimal. Do we even need decimal ones? > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in > the body to majordomo@kvack.org. For more info on Linux MM, > see: http://www.linux-mm.org/ . > Don't email: email@kvack.org >