From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 5/8] percpu: add __percpu sparse annotations to x86 Date: Tue, 26 Jan 2010 11:17:52 +0900 Message-ID: <4B5E50D0.8050705@kernel.org> References: <1264432935-10453-1-git-send-email-tj@kernel.org> <1264432935-10453-6-git-send-email-tj@kernel.org> <4B5E401C.1050605@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: linux-kernel@vger.kernel.org, axboe@kernel.dk, rusty@rustcorp.com.au, akpm@linux-foundation.org, ebiederm@xmission.com, tytso@mit.edu, Trond.Myklebust@netapp.com, aelder@sgi.com, hch@infradead.org, viro@zeniv.linux.org.uk, davem@davemloft.net, netdev@vger.kernel.org, x86@kernel.org, mingo@redhat.com, fweisbec@gmail.com, dan.j.williams@intel.com, borislav.petkov@amd.com, ying.huang@intel.com, lenb@kernel.org, neilb@suse.de, cl@linux-foundation.org, Thomas Gleixner To: "H. Peter Anvin" Return-path: In-Reply-To: <4B5E401C.1050605@zytor.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 01/26/2010 10:06 AM, H. Peter Anvin wrote: > On 01/25/2010 07:22 AM, Tejun Heo wrote: >> >> In arch/x86/kernel/cpu/common.c, per_cpu(exception_stacks, cpu) is >> replaced with &per_cpu(exception_stacks[0], cpu) which is equivalent >> except that it allows per_cpu() macro to correctly drop percpu >> designation during sparse pass. >> > > Same comment as Frederic's about this... this kind of uglification > really is unacceptable and needs to be addressed. Hmm... I don't think it's too bad but if you don't like it that much, probably the right thing to do is just leave it there and let sparse whine about it. There are only three in-kernel instances at this point, so it's not that big a deal. Thanks. -- tejun