From mboxrd@z Thu Jan 1 00:00:00 1970 From: ebiederm@xmission.com (Eric W. Biederman) Subject: Re: [PATCH 01/11] sysctl: Separate the binary sysctl logic into it's own file. Date: Fri, 06 Nov 2009 04:55:23 -0800 Message-ID: References: <200911061004.01007.borntraeger@de.ibm.com> <200911061310.02785.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from out01.mta.xmission.com ([166.70.13.231]:59361 "EHLO out01.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752110AbZKFMzZ (ORCPT ); Fri, 6 Nov 2009 07:55:25 -0500 In-Reply-To: <200911061310.02785.arnd@arndb.de> (Arnd Bergmann's message of "Fri\, 6 Nov 2009 13\:10\:02 +0100") Sender: linux-arch-owner@vger.kernel.org List-ID: To: Arnd Bergmann Cc: Christian Borntraeger , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org Arnd Bergmann writes: > On Friday 06 November 2009, Eric W. Biederman wrote: >> Christian Borntraeger writes: >> >> > Am Freitag 06 November 2009 01:41:44 schrieb Eric W. Biederman: >> >> From: Eric W. Biederman >> >> >> >> In preparation for more invasive cleanups separate the core >> >> binary sysctl logic into it's own file. >> >> >> >> Signed-off-by: Eric W. Biederman >> > >> > Hmm, with your patches on Linus git I get the following on s390: >> > >> > kernel/sysctl_binary.c: In function 'SYSC_sysctl': >> > kernel/sysctl_binary.c:126: error: implicit declaration of function >> > 'lock_kernel' >> > kernel/sysctl_binary.c:129: error: implicit declaration of function >> > 'unlock_kernel' >> >> Bah. A missing smp_lock.h. > > BTW, there is a patch in the kill-the-BKL tree to push down the BKL > further into the sysctl handlers. It may be a good idea to put that > into your tree, or to redo the same thing there differently, since > you already have a patch series touching this area. Thanks for the info. The primary proc path already doesn't need the lock_kernel(). My next patch winds up killing the entire binary path and rebuilding on top of /proc/sys. Which removes that lock_kernel(). Which I think elegantly solves all of the sysctl BKL lock issues. Which is probably why I missed the compilation failure. Eric