Subject: Re: [ckrm-tech] [PATCH 6/7] BC: kernel memory (core)
From: Dave Hansen <haveblue@us.ibm.com>
To: Kirill Korotaev <dev@sw.ru>
Cc: Rik van Riel <riel@redhat.com>,
       CKRM-Tech <ckrm-tech@lists.sourceforge.net>, Andi Kleen <ak@suse.de>,
       Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
       Christoph Hellwig <hch@infradead.org>, Andrey Savochkin <saw@sw.ru>,
       devel@openvz.org, Matt Helsley <matthltc@us.ibm.com>,
       Alexey Dobriyan <adobriyan@mail.ru>, Oleg Nesterov <oleg@tv-sign.ru>,
       Alan Cox <alan@lxorguk.ukuu.org.uk>, Pavel Emelianov <xemul@openvz.org>
In-Reply-To: <44FC1AAF.8040504@sw.ru>
References: <44F45045.70402@sw.ru>  <44F45601.9060807@sw.ru>
	 <1156883382.5408.153.camel@localhost.localdomain>  <44FC1AAF.8040504@sw.ru>
Content-Type: text/plain
Date: Tue, 05 Sep 2006 09:21:34 -0700
Message-Id: <1157473294.3186.2.camel@localhost.localdomain>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 631
Lines: 22

On Mon, 2006-09-04 at 16:23 +0400, Kirill Korotaev wrote:
> 
> are you suggesting the code like this:
> >>+#ifdef CONFIG_BEANCOUNTERS
> >>+       union {
> >>+               struct beancounter      *page_bc;
> >>+       };
> >>+#endif
> >> }; 
> 
> #define page_bc(page)   ((page)->page_bc) 

That would be a bit better.  Although, I think the "page_" bit is also
superfluous.

-- Dave

-
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/