Re: [PATCH] ACPICA: make acpi_load_table() return table index

From: "Rafael J. Wysocki" <rafael@kernel.org>
To: Nikolaus Voss <nv@vosn.de>
Cc: "Schmauss, Erik" <erik.schmauss@intel.com>,
	"Shevchenko, Andriy" <andriy.shevchenko@intel.com>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	"Moore, Robert" <robert.moore@intel.com>,
	Len Brown <lenb@kernel.org>,
	Jacek Anaszewski <jacek.anaszewski@gmail.com>,
	Pavel Machek <pavel@ucw.cz>, Dan Murphy <dmurphy@ti.com>,
	"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
	"devel@acpica.org" <devel@acpica.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ACPICA: make acpi_load_table() return table index
Date: Thu, 26 Sep 2019 21:26:18 +0200	[thread overview]
Message-ID: <CAJZ5v0g=onKCidHMLNWuYFCuhcQGmtDDp5QSGiHu1jhoDRuhWQ@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.20.1909262043380.13592@fox.voss.local>

On Thu, Sep 26, 2019 at 8:44 PM Nikolaus Voss <nv@vosn.de> wrote:
>
> On Thu, 26 Sep 2019, Schmauss, Erik wrote:
> >> -----Original Message-----
> >> From: linux-acpi-owner@vger.kernel.org <linux-acpi-owner@vger.kernel.org>
> >> On Behalf Of Shevchenko, Andriy
> >> Sent: Thursday, September 26, 2019 9:35 AM
> >> To: Schmauss, Erik <erik.schmauss@intel.com>
> >> Cc: Nikolaus Voss <nikolaus.voss@loewensteinmedical.de>; Rafael J. Wysocki
> >> <rjw@rjwysocki.net>; Moore, Robert <robert.moore@intel.com>; Len Brown
> >> <lenb@kernel.org>; Jacek Anaszewski <jacek.anaszewski@gmail.com>; Pavel
> >> Machek <pavel@ucw.cz>; Dan Murphy <dmurphy@ti.com>; linux-
> >> acpi@vger.kernel.org; devel@acpica.org; linux-kernel@vger.kernel.org;
> >> nv@vosn.de
> >> Subject: Re: [PATCH] ACPICA: make acpi_load_table() return table index
> >>
> >> On Thu, Sep 26, 2019 at 07:09:05PM +0300, Schmauss, Erik wrote:
> >>>> -----Original Message-----
> >>>> From: Nikolaus Voss <nikolaus.voss@loewensteinmedical.de>
> >>>> Sent: Thursday, September 12, 2019 1:08 AM
> >>>> To: Shevchenko, Andriy <andriy.shevchenko@intel.com>; Schmauss, Erik
> >>>> <erik.schmauss@intel.com>; Rafael J. Wysocki <rjw@rjwysocki.net>;
> >>>> Moore, Robert <robert.moore@intel.com>
> >>>> Cc: Len Brown <lenb@kernel.org>; Jacek Anaszewski
> >>>> <jacek.anaszewski@gmail.com>; Pavel Machek <pavel@ucw.cz>; Dan
> >>>> Murphy <dmurphy@ti.com>; linux-acpi@vger.kernel.org;
> >>>> devel@acpica.org; linux- kernel@vger.kernel.org; nv@vosn.de;
> >>>> Nikolaus Voss <nikolaus.voss@loewensteinmedical.de>
> >>>> Subject: [PATCH] ACPICA: make acpi_load_table() return table index
> >>>>
> >>> Hi Nikolaus,
> >>>
> >>>> For unloading an ACPI table, it is necessary to provide the index of the table.
> >>>> The method intended for dynamically loading or hotplug addition of
> >>>> tables, acpi_load_table(), should provide this information via an
> >>>> optional pointer to the loaded table index.
> >>>
> >>> We'll take this patch for ACPICA upstream
> >>
> >> Erik,
> >>
> > Hi Andy,
> >
> >> how about to have also counterpart to acpi_load_table() which will do what it's
> >> done now in acpi_configfs.c via acpi_tb_*() API?
> >
> > I should have given more details. We decided to add this extra parameter
> > in AcpiLoadTable and we're going to create an AcpiUnloadTable function
> > that will take table index to unload the table (basically the
> > acpi_tb_unload..). Once we do this, you can use table indices with
> > AcpiUnloadTable and AcpiLoadTable.
>
> that's even better news.
>
> Rafael, shall I prepare anything?

I don't think so.  I'm expecting to get a proper fix from the upstream
through the normal process.

Thanks,
Rafael