Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762857AbZDHUhB (ORCPT ); Wed, 8 Apr 2009 16:37:01 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756761AbZDHUgv (ORCPT ); Wed, 8 Apr 2009 16:36:51 -0400 Received: from blaine.gmane.org ([80.91.229.8]:44434 "EHLO hugh.gmane.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755783AbZDHUgu (ORCPT ); Wed, 8 Apr 2009 16:36:50 -0400 From: Andreas Schwab To: public-alan-jenkins-cCz0Lq7MMjm9FHfhHBbuYA@hugh.gmane.org Cc: Larry Finger , Rusty Russell , Arjan van de Ven , LKML , "Rafael J. Wysocki" , wireless Subject: Re: Regression in 2.6.30-rc1 since commit acae0515 - wireless broken References: <49DC2DF5.3010603@lwfinger.net> <49DC31EA.8050208@linux.intel.com> <200904081631.09150.rusty@rustcorp.com.au> <49DCB976.4020200@lwfinger.net> <9b2b86520904081006m6e6b5276v834c3565b0f6f4c7@mail.gmail.com> X-Yow: .. does your DRESSING ROOM have enough ASPARAGUS? Date: Wed, 08 Apr 2009 22:05:21 +0200 In-Reply-To: <9b2b86520904081006m6e6b5276v834c3565b0f6f4c7@mail.gmail.com> (Alan Jenkins's message of "Wed, 8 Apr 2009 18:06:18 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.92 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-TMDA-Confirmed: Wed, 08 Apr 2009 22:36:32 +0200 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1562 Lines: 45 Alan Jenkins writes: > I agree that commit looks like it should be a no-op though :-). But it isn't. Andreas. --- Fix try_then_request_module to use waiting __request_module again. Signed-off-by: Andreas Schwab --- include/linux/kmod.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Index: linux-2.6.30-rc1/include/linux/kmod.h =================================================================== --- linux-2.6.30-rc1.orig/include/linux/kmod.h 2009-04-08 12:47:54.000000000 +0200 +++ linux-2.6.30-rc1/include/linux/kmod.h 2009-04-08 17:39:35.000000000 +0200 @@ -34,7 +34,7 @@ extern int __request_module(bool wait, c #define request_module(mod...) __request_module(true, mod) #define request_module_nowait(mod...) __request_module(false, mod) #define try_then_request_module(x, mod...) \ - ((x) ?: (__request_module(false, mod), (x))) + ((x) ?: (__request_module(true, mod), (x))) #else static inline int request_module(const char *name, ...) { return -ENOSYS; } static inline int request_module_nowait(const char *name, ...) { return -ENOSYS; } -- Andreas Schwab, schwab@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different." -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/