From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755205AbYDHN6c (ORCPT ); Tue, 8 Apr 2008 09:58:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752314AbYDHN6Y (ORCPT ); Tue, 8 Apr 2008 09:58:24 -0400 Received: from fk-out-0910.google.com ([209.85.128.189]:49205 "EHLO fk-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752289AbYDHN6Y (ORCPT ); Tue, 8 Apr 2008 09:58:24 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version:content-type:content-disposition:in-reply-to:user-agent; b=Vvi3391bSSBLVVm4sjlazxNpNLFIZWdMvYqA6YHh8DTAw969nk+Xld4xEDyEJyM9RhXwOVTDF0jk74t1CwD8bu0zsy9n7YhZ5d/4yvc76UdOJmyDtkht6yfmeAJBxgMPZJRfvpp2Nnsa2I4rRh4aFc6RErPPqm1MlHo+d7nDVJc= Date: Tue, 8 Apr 2008 17:57:42 +0400 From: Cyrill Gorcunov To: Andi Kleen Cc: Ingo Molnar , "H. Peter Anvin" , LKML , Yinghai Lu Subject: Re: bootmem allocator Message-ID: <20080408135742.GA7656@cvg> References: <20080407185613.GD9211@cvg> <20080407190904.GH12292@elte.hu> <87iqytqwl7.fsf@basil.nowhere.org> <20080408080446.GA12308@elte.hu> <20080408081206.GL16647@one.firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20080408081206.GL16647@one.firstfloor.org> User-Agent: Mutt/1.5.16 (2007-06-09) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [Andi Kleen - Tue, Apr 08, 2008 at 10:12:06AM +0200] | On Tue, Apr 08, 2008 at 10:04:46AM +0200, Ingo Molnar wrote: | > | > * Andi Kleen wrote: | > | > > > hm, bootmem allocator is supposed to clear memory. We have a couple | > > > of places that rely on that. | > > | > > I was actually considering to change that for the GB pages hugetlbfs | > > patchkit, because memset for 1G is a little slow and not needed (will | > > be cleared later anyways) and it might be a problem for very large | > > systems with a lot of such pages at boot. | > | > changing the default behavior of bootmem alloc to be non-clearing is a | > really bad idea that will only cause unrobustness. The proper approach | > is to add an _opt-in_ API that does not clear memory | | I was considering that too, but we have so many weird variants of bootmem | with opt in and opt out and even combinations of both now that the whole thing | is starting to look really pear shaped (I admit I added some of them | in the past myself but I'm not proud). Would be a great project for | someone to consolidate that all a bit. | | -Andi | OK, thanks to all of you - Cyrill -