Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756098AbcLOL3G (ORCPT ); Thu, 15 Dec 2016 06:29:06 -0500 Received: from mx2.suse.de ([195.135.220.15]:43091 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755521AbcLOL3C (ORCPT ); Thu, 15 Dec 2016 06:29:02 -0500 Date: Thu, 15 Dec 2016 12:28:56 +0100 From: Jean Delvare To: Andy Shevchenko Cc: Eric Biederman , linux-kernel@vger.kernel.org, Mika Westerberg , kexec@lists.infradead.org Subject: Re: [PATCH v1 2/2] firmware: dmi_scan: Pass dmi_entry_point to kexec'ed kernel Message-ID: <20161215122856.7d24b7a8@endymion> In-Reply-To: <20161202195416.58953-3-andriy.shevchenko@linux.intel.com> References: <20161202195416.58953-1-andriy.shevchenko@linux.intel.com> <20161202195416.58953-3-andriy.shevchenko@linux.intel.com> Organization: SUSE Linux X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.31; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2386 Lines: 71 Hi Andy, On Fri, 2 Dec 2016 21:54:16 +0200, Andy Shevchenko wrote: > Until now kexec'ed kernel has no clue where to look for DMI entry point. > > Pass it via kernel command line parameter in the same way as it's done for ACPI > RSDP. I am no kexec expert but this confuses me. Shouldn't the second kernel have access to the EFI systab as the first kernel does? It includes many more pointers than just ACPI and DMI tables, and it would seem inconvenient to have to pass all these addresses individually explicitly. Adding Eric to Cc for his opinion. > > Signed-off-by: Andy Shevchenko > --- > Documentation/admin-guide/kernel-parameters.txt | 5 +++++ > drivers/firmware/dmi_scan.c | 14 ++++++++++++++ > 2 files changed, 19 insertions(+) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index be2d6d0..94f219f 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -843,6 +843,11 @@ > The filter can be disabled or changed to another > driver later using sysfs. > > + dmi_entry_point= [DMI,EFI,KEXEC] > + Pass the DMI entry point to the kernel, mostly used > + on machines running EFI runtime service to boot the > + second kernel for kdump. > + > drm_kms_helper.edid_firmware=[:][,[:]] > Broken monitors, graphic adapters, KVMs and EDIDless > panels may send no or incorrect EDID data sets. > diff --git a/drivers/firmware/dmi_scan.c b/drivers/firmware/dmi_scan.c > index b88def6..215843f 100644 > --- a/drivers/firmware/dmi_scan.c > +++ b/drivers/firmware/dmi_scan.c > @@ -595,8 +595,22 @@ static int __init dmi_smbios3_present(const u8 *buf) > return 1; > } > > +#ifdef CONFIG_KEXEC > +static unsigned long dmi_entry_point; > +static int __init setup_dmi_entry_point(char *arg) > +{ > + return kstrtoul(arg, 16, &dmi_entry_point); > +} > +early_param("dmi_entry_point", setup_dmi_entry_point); > +#endif > + > static resource_size_t __init dmi_get_entry_point(void) > { > +#ifdef CONFIG_KEXEC > + if (dmi_entry_point) > + return dmi_entry_point; > +#endif > + > if (efi_enabled(EFI_CONFIG_TABLES)) { > /* > * According to the DMTF SMBIOS reference spec v3.0.0, it is -- Jean Delvare SUSE L3 Support