From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: linux-next: manual merge of the rr tree with the sound tree Date: Tue, 13 Jul 2010 13:51:54 +0200 Message-ID: <201007131351.54246.arnd@arndb.de> References: <20100713121348.c9e76987.sfr@canb.auug.org.au> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from moutng.kundenserver.de ([212.227.17.8]:54335 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751400Ab0GMLv6 (ORCPT ); Tue, 13 Jul 2010 07:51:58 -0400 In-Reply-To: <20100713121348.c9e76987.sfr@canb.auug.org.au> Sender: linux-next-owner@vger.kernel.org List-ID: To: Stephen Rothwell Cc: Rusty Russell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Takashi Iwai On Tuesday 13 July 2010, Stephen Rothwell wrote: > Overlapping additions. I fixed it up (see below) and carry the fix as necessary. > > I was not sure if the kparam_(un)block_sysfs_write() calls needed to be > surrounded by the (un)lock_kernel() calls or vice verse. Either way should be fine, since the BKL does not suffer from AB-BA deadlocks with mutexes. I would generally recommend making the BKL the outermost lock to make it easier to conver to a mutex. That is how you did it too. Thanks, Arnd