From: Peter Zijlstra Subject: Re: [PATCH 1/9] mm: add generic adaptive large memory allocation APIs Date: Fri, 14 May 2010 10:03:34 +0200 Message-ID: <1273824214.5605.3625.camel@twins> References: <1273744285-8128-1-git-send-email-xiaosuo@gmail.com> <1273756816.5605.3547.camel@twins> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Cc: akpm@linux-foundation.org, Hoang-Nam Nguyen , Christoph Raisch , Roland Dreier , Sean Hefty , Hal Rosenstock , Divy Le Ray , "James E.J. Bottomley" , Theodore Ts'o , Andreas Dilger , Alexander Viro , Paul Menage , Li Zefan , linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-scsi@vger.kernel.org, linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, containers@lists.linux-foundation.org, Eric Dumazet , Tetsuo Handa To: Changli Gao Return-path: In-Reply-To: Sender: owner-linux-mm@kvack.org List-Id: linux-ext4.vger.kernel.org On Thu, 2010-05-13 at 22:08 +0800, Changli Gao wrote: > > NAK, I really utterly dislike that inatomic argument. The alloc side > > doesn't function in atomic context either. Please keep the thing > > symmetric in that regards. > > >=20 > There are some users, who release memory in atomic context. for > example: fs/file.c: fdmem.=20 urgh, but yeah, aside from not using vmalloc to allocate fd tables one needs to deal with this. But if that is the only one, I'd let them do the workqueue thing that's already there. If there really are more people wanting to do this, then maybe add: kvfree_atomic(). -- 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