| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 26 Jan 2015 13:56:38 +0200
From: Octavian Purdila <octavian.purdila@...el.com>
To: Lv Zheng <lv.zheng@...el.com>
Cc: "Rafael J. Wysocki" <rafael.j.wysocki@...el.com>,
Len Brown <len.brown@...el.com>, Lv Zheng <zetalog@...il.com>,
lkml <linux-kernel@...r.kernel.org>,
"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
Tony Luck <tony.luck@...el.com>,
Fenghua Yu <fenghua.yu@...el.com>, linux-pci@...r.kernel.org,
linux-ia64@...r.kernel.org
Subject: Re: [RFC PATCH 1/2] ACPI: Introduce acpi_unload_parent_table() usages
in Linux kernel.
On Mon, Jan 26, 2015 at 10:58 AM, Lv Zheng <lv.zheng@...el.com> wrote:
> ACPICA has implemented acpi_unload_parent_table() which can exactly replace
> the acpi_get_id()/acpi_unload_table_id() implemented in Linux kernel. The
> acpi_unload_parent_table() has been unit tested in ACPICA simulation
> environment.
>
> This patch can also help to reduce the source code differences between
> Linux and ACPICA.
>
> Signed-off-by: Lv Zheng <lv.zheng@...el.com>
> Acked-by: Bjorn Helgaas <bhelgaas@...gle.com>
> Cc: Octavian Purdila <octavian.purdila@...el.com>
> Cc: Tony Luck <tony.luck@...el.com>
> Cc: Fenghua Yu <fenghua.yu@...el.com>
> Cc: linux-pci@...r.kernel.org
> Cc: linux-ia64@...r.kernel.org
I can confirm that acpi_unload_parent_table() works on my DLN2 setup.
Tested-by: Octavian Purdila <octavian.purdila@...el.com>
> diff --git a/drivers/acpi/acpica/nsxfobj.c b/drivers/acpi/acpica/nsxfobj.c
> index dae9401..51a8329 100644
> --- a/drivers/acpi/acpica/nsxfobj.c
> +++ b/drivers/acpi/acpica/nsxfobj.c
> @@ -53,50 +53,6 @@ ACPI_MODULE_NAME("nsxfobj")
>
> /*******************************************************************************
> *
> - * FUNCTION: acpi_get_id
> - *
> - * PARAMETERS: Handle - Handle of object whose id is desired
> - * ret_id - Where the id will be placed
> - *
> - * RETURN: Status
> - *
> - * DESCRIPTION: This routine returns the owner id associated with a handle
> - *
> - ******************************************************************************/
> -acpi_status acpi_get_id(acpi_handle handle, acpi_owner_id * ret_id)
> -{
> - struct acpi_namespace_node *node;
> - acpi_status status;
> -
> - /* Parameter Validation */
> -
> - if (!ret_id) {
> - return (AE_BAD_PARAMETER);
> - }
> -
> - status = acpi_ut_acquire_mutex(ACPI_MTX_NAMESPACE);
> - if (ACPI_FAILURE(status)) {
> - return (status);
> - }
> -
> - /* Convert and validate the handle */
> -
> - node = acpi_ns_validate_handle(handle);
> - if (!node) {
> - (void)acpi_ut_release_mutex(ACPI_MTX_NAMESPACE);
> - return (AE_BAD_PARAMETER);
> - }
> -
> - *ret_id = node->owner_id;
> -
> - status = acpi_ut_release_mutex(ACPI_MTX_NAMESPACE);
> - return (status);
> -}
> -
> -ACPI_EXPORT_SYMBOL(acpi_get_id)
> -
> -/*******************************************************************************
> - *
> * FUNCTION: acpi_get_type
> *
> * PARAMETERS: handle - Handle of object whose type is desired
> diff --git a/drivers/acpi/acpica/tbxface.c b/drivers/acpi/acpica/tbxface.c
> index 9520ae1..0f9dd80 100644
> --- a/drivers/acpi/acpica/tbxface.c
> +++ b/drivers/acpi/acpica/tbxface.c
> @@ -265,52 +265,6 @@ ACPI_EXPORT_SYMBOL(acpi_get_table_header)
>
> /*******************************************************************************
> *
> - * FUNCTION: acpi_unload_table_id
> - *
> - * PARAMETERS: id - Owner ID of the table to be removed.
> - *
> - * RETURN: Status
> - *
> - * DESCRIPTION: This routine is used to force the unload of a table (by id)
> - *
> - ******************************************************************************/
> -acpi_status acpi_unload_table_id(acpi_owner_id id)
> -{
> - int i;
> - acpi_status status = AE_NOT_EXIST;
> -
> - ACPI_FUNCTION_TRACE(acpi_unload_table_id);
> -
> - status = acpi_ut_acquire_mutex(ACPI_MTX_INTERPRETER);
> - if (ACPI_FAILURE(status)) {
> - return_ACPI_STATUS(status);
> - }
> -
> - /* Find table in the global table list */
> - for (i = 0; i < acpi_gbl_root_table_list.current_table_count; ++i) {
> - if (id != acpi_gbl_root_table_list.tables[i].owner_id) {
> - continue;
> - }
> - /*
> - * Delete all namespace objects owned by this table. Note that these
> - * objects can appear anywhere in the namespace by virtue of the AML
> - * "Scope" operator. Thus, we need to track ownership by an ID, not
> - * simply a position within the hierarchy
> - */
> - acpi_tb_delete_namespace_by_owner(i);
> - status = acpi_tb_release_owner_id(i);
> - acpi_tb_set_table_loaded_flag(i, FALSE);
> - break;
> - }
> -
> - (void)acpi_ut_release_mutex(ACPI_MTX_INTERPRETER);
> - return_ACPI_STATUS(status);
> -}
> -
> -ACPI_EXPORT_SYMBOL(acpi_unload_table_id)
> -
> -/*******************************************************************************
> - *
> * FUNCTION: acpi_get_table_with_size
> *
> * PARAMETERS: signature - ACPI signature of needed table
> diff --git a/drivers/pci/hotplug/sgi_hotplug.c b/drivers/pci/hotplug/sgi_hotplug.c
> index bada2099..c32fb78 100644
> --- a/drivers/pci/hotplug/sgi_hotplug.c
> +++ b/drivers/pci/hotplug/sgi_hotplug.c
> @@ -475,7 +475,7 @@ static int disable_slot(struct hotplug_slot *bss_hotplug_slot)
> struct slot *slot = bss_hotplug_slot->private;
> struct pci_dev *dev, *temp;
> int rc;
> - acpi_owner_id ssdt_id = 0;
> + acpi_handle ssdt_hdl = NULL;
>
> /* Acquire update access to the bus */
> mutex_lock(&sn_hotplug_mutex);
> @@ -522,7 +522,7 @@ static int disable_slot(struct hotplug_slot *bss_hotplug_slot)
> if (ACPI_SUCCESS(ret) &&
> (adr>>16) == (slot->device_num + 1)) {
> /* retain the owner id */
> - acpi_get_id(chandle, &ssdt_id);
> + ssdt_hdl = chandle;
>
> ret = acpi_bus_get_device(chandle,
> &device);
> @@ -547,12 +547,13 @@ static int disable_slot(struct hotplug_slot *bss_hotplug_slot)
> pci_unlock_rescan_remove();
>
> /* Remove the SSDT for the slot from the ACPI namespace */
> - if (SN_ACPI_BASE_SUPPORT() && ssdt_id) {
> + if (SN_ACPI_BASE_SUPPORT() && ssdt_hdl) {
> acpi_status ret;
> - ret = acpi_unload_table_id(ssdt_id);
> + ret = acpi_unload_parent_table(ssdt_hdl);
> if (ACPI_FAILURE(ret)) {
> - printk(KERN_ERR "%s: acpi_unload_table_id failed (0x%x) for id %d\n",
> - __func__, ret, ssdt_id);
> + acpi_handle_err(ssdt_hdl,
> + "%s: acpi_unload_parent_table failed (0x%x)\n",
> + __func__, ret);
> /* try to continue on */
> }
> }
> diff --git a/include/acpi/acpixf.h b/include/acpi/acpixf.h
> index 5ba7846..5dd21bc 100644
> --- a/include/acpi/acpixf.h
> +++ b/include/acpi/acpixf.h
> @@ -891,12 +891,6 @@ ACPI_APP_DEPENDENT_RETURN_VOID(ACPI_PRINTF_LIKE(1)
> ACPI_GLOBAL(u8, acpi_gbl_permanent_mmap);
>
> ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> - acpi_get_id(acpi_handle object,
> - acpi_owner_id * out_type))
> -
> -ACPI_EXTERNAL_RETURN_STATUS(acpi_status acpi_unload_table_id(acpi_owner_id id))
> -
> -ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> acpi_get_table_with_size(acpi_string signature,
> u32 instance,
> struct acpi_table_header
> --
> 1.7.10
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists