From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx169.postini.com [74.125.245.169]) by kanga.kvack.org (Postfix) with SMTP id 29D4D6B13F0 for ; Tue, 31 Jan 2012 04:59:10 -0500 (EST) From: Rusty Russell Subject: Re: [PATCH 1/3] percpu: use ZERO_SIZE_PTR / ZERO_OR_NULL_PTR In-Reply-To: <20120130181639.GJ3355@google.com> References: <1327912654-8738-1-git-send-email-dmitry.antipov@linaro.org> <20120130171558.GB3355@google.com> <20120130174256.GF3355@google.com> <20120130175434.GG3355@google.com> <20120130180224.GH3355@google.com> <20120130181639.GJ3355@google.com> Date: Tue, 31 Jan 2012 15:18:54 +1030 Message-ID: <87obtkjx3d.fsf@rustcorp.com.au> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Sender: owner-linux-mm@kvack.org List-ID: To: Tejun Heo , Christoph Lameter Cc: Dmitry Antipov , linux-mm@kvack.org, linux-kernel@vger.kernel.org, patches@linaro.org, linaro-dev@lists.linaro.org On Mon, 30 Jan 2012 10:16:39 -0800, Tejun Heo wrote: > On Mon, Jan 30, 2012 at 12:12:18PM -0600, Christoph Lameter wrote: > > > I thought it didn't. I rememer thinking about this and determining > > > that NULL can't be allocated for dynamic addresses. Maybe I'm > > > imagining things. Anyways, if it can return NULL for valid > > > allocation, it is a bug and should be fixed. > > > > I dont see anything that would hinder an arbitrary value to be returned. > > NULL is also used for the failure case. Definitely a bug. > > Given the address translation we do and kernel image layout, I don't > think this can happen on x86. It may theoretically possible on other > archs tho. Anyways, yeah, this one needs improving. I tried setting the lower bit on all percpu ptrs, but since non-dynamic percpu vars could have odd alignments, that fails in general. > > > We don't have returned addr >= PAGE_SIZE guarantee yet but I'm fairly > > > sure that's the only acceptable direction if we want any improvement > > > in this area. > > > > The ZERO_SIZE_PTR patch would not make the situation that much worse. > > I'm not objecting to marking zero-sized allocations per-se. I'm > saying the patch is pointless at this point. It doesn't contribute > anything while giving the illusion of better error checking than we > actually do. Let's do it when it can actually work. Disagree: This patch works. It allows zero-size per-cpu allocs, like the other allocators. Nor does it fail in practice. We should do better, but the perfect is the enemy of the good. Cheers, Rusty. -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org