From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752441AbeD3IEc (ORCPT ); Mon, 30 Apr 2018 04:04:32 -0400 Received: from cloudserver094114.home.pl ([79.96.170.134]:63539 "EHLO cloudserver094114.home.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752142AbeD3IEb (ORCPT ); Mon, 30 Apr 2018 04:04:31 -0400 From: "Rafael J. Wysocki" To: Ard Biesheuvel Cc: lenb@kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, arnd@arndb.de, broonie@kernel.org, lorenzo.pieralisi@arm.com, bill.fletcher@linaro.org, linux-arm-kernel@lists.infradead.org, graeme.gregory@linaro.org Subject: Re: [PATCH v2] ACPI / button: make module loadable when booted in non-ACPI mode Date: Mon, 30 Apr 2018 10:04:16 +0200 Message-ID: <2286238.nKNuTerCUV@aspire.rjw.lan> In-Reply-To: <20180423091656.26257-1-ard.biesheuvel@linaro.org> References: <20180423091656.26257-1-ard.biesheuvel@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Monday, April 23, 2018 11:16:56 AM CEST Ard Biesheuvel wrote: > Modules such as nouveau.ko and i915.ko have a link time dependency on > acpi_lid_open(), and due to its use of acpi_bus_register_driver(), > the button.ko module that provides it is only loadable when booted in > ACPI mode. However, the ACPI button driver can be built into the core > kernel as well, in which case the dependency can always be satisfied, > and the dependent modules can be loaded regardless of whether the > system was booted in ACPI mode or not. > > So let's fix this asymmetry by making the ACPI button driver loadable > as a module even if not booted in ACPI mode, so it can provide the > acpi_lid_open() symbol in the same way as when built into the kernel. > > Signed-off-by: Ard Biesheuvel > --- > v2: invert acpi_disabled logic and move comment into __acpi_bus_register_driver > > Could we perhaps get this into -stable as well? It is not a classic > regression, but it completely breaks, e.g., Fedora when booting in > DT mode on an ARM system. > > drivers/acpi/button.c | 24 +++++++++++++++++++- > 1 file changed, 23 insertions(+), 1 deletion(-) > > diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c > index e1eee7a60fad..f33242e4fe6c 100644 > --- a/drivers/acpi/button.c > +++ b/drivers/acpi/button.c > @@ -635,4 +635,26 @@ module_param_call(lid_init_state, > NULL, 0644); > MODULE_PARM_DESC(lid_init_state, "Behavior for reporting LID initial state"); > > -module_acpi_driver(acpi_button_driver); > +static int __acpi_bus_register_driver(struct acpi_driver *driver) > +{ > + /* > + * Modules such as nouveau.ko and i915.ko have a link time dependency > + * on acpi_lid_open(), and would therefore not be loadable on ACPI > + * capable kernels booted in non-ACPI mode if we use the ordinary > + * acpi_bus_[un]register_driver routines here (which only work when > + * booted in ACPI mode) and build this driver as a module. So provide > + * our own versions instead. > + */ > + if (acpi_disabled) > + return 0; > + return acpi_bus_register_driver(driver); > +} > + > +static void __acpi_bus_unregister_driver(struct acpi_driver *driver) > +{ > + if (!acpi_disabled) > + acpi_bus_unregister_driver(driver); > +} > + > +module_driver(acpi_button_driver, __acpi_bus_register_driver, > + __acpi_bus_unregister_driver); > Applied (with some minor modifications) and merged into 4.17-rc3, thanks!