Commit 9b019b0f authored by Lv Zheng's avatar Lv Zheng Committed by Rafael J. Wysocki
Browse files

ACPICA: Tables: Cleanup table handler invokers 

ACPICA commit 4551f51fa8ba33a977721c3b250cb70a309e3f23

Recently, we allows the table mutex to be held in both early and late stage
APIs. This patch further cleans up the related code to reduce redundant
code related to acpi_gbl_table_handler. Lv Zheng.

Link: https://github.com/acpica/acpica/commit/4551f51f


Tested-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarLv Zheng <lv.zheng@intel.com>
Signed-off-by: default avatarBob Moore <robert.moore@intel.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 96d07940

drivers/acpi/acpica/actables.h

+2 −0
Original line number Diff line number Diff line
@@ -132,6 +132,8 @@ acpi_tb_install_and_load_table(acpi_physical_address address, 


acpi_status acpi_tb_unload_table(u32 table_index);



void acpi_tb_notify_table(u32 event, void *table);



void acpi_tb_terminate(void);



acpi_status acpi_tb_delete_namespace_by_owner(u32 table_index);

drivers/acpi/acpica/tbdata.c

+29 −14
Original line number Diff line number Diff line
@@ -818,13 +818,9 @@ acpi_tb_load_table(u32 table_index, struct acpi_namespace_node *parent_node) 
		acpi_ev_update_gpes(owner_id);

	}



	/* Invoke table handler if present */



	if (acpi_gbl_table_handler) {

		(void)acpi_gbl_table_handler(ACPI_TABLE_EVENT_LOAD, table,

					     acpi_gbl_table_handler_context);

	}

	/* Invoke table handler */



	acpi_tb_notify_table(ACPI_TABLE_EVENT_LOAD, table);

	return_ACPI_STATUS(status);

}
@@ -894,15 +890,11 @@ acpi_status acpi_tb_unload_table(u32 table_index) 
		return_ACPI_STATUS(AE_NOT_EXIST);

	}



	/* Invoke table handler if present */

	/* Invoke table handler */



	if (acpi_gbl_table_handler) {

	status = acpi_get_table_by_index(table_index, &table);

	if (ACPI_SUCCESS(status)) {

			(void)acpi_gbl_table_handler(ACPI_TABLE_EVENT_UNLOAD,

						     table,

						     acpi_gbl_table_handler_context);

		}

		acpi_tb_notify_table(ACPI_TABLE_EVENT_UNLOAD, table);

	}



	/* Delete the portion of the namespace owned by this table */
@@ -918,3 +910,26 @@ acpi_status acpi_tb_unload_table(u32 table_index) 
}



ACPI_EXPORT_SYMBOL(acpi_tb_unload_table)



/*******************************************************************************

 *

 * FUNCTION:    acpi_tb_notify_table

 *

 * PARAMETERS:  event               - Table event

 *              table               - Validated table pointer

 *

 * RETURN:      None

 *

 * DESCRIPTION: Notify a table event to the users.

 *

 ******************************************************************************/



void acpi_tb_notify_table(u32 event, void *table)

{

	/* Invoke table handler if present */



	if (acpi_gbl_table_handler) {

		(void)acpi_gbl_table_handler(event, table,

					     acpi_gbl_table_handler_context);

	}

}

drivers/acpi/acpica/tbinstal.c

+2 −6
Original line number Diff line number Diff line
@@ -306,14 +306,10 @@ acpi_tb_install_standard_table(acpi_physical_address address, 
	acpi_tb_install_table_with_override(&new_table_desc, override,

					    table_index);



	/* Invoke table handler if present */

	/* Invoke table handler */



	(void)acpi_ut_release_mutex(ACPI_MTX_TABLES);

	if (acpi_gbl_table_handler) {

		(void)acpi_gbl_table_handler(ACPI_TABLE_EVENT_INSTALL,

					     new_table_desc.pointer,

					     acpi_gbl_table_handler_context);

	}

	acpi_tb_notify_table(ACPI_TABLE_EVENT_INSTALL, new_table_desc.pointer);

	(void)acpi_ut_acquire_mutex(ACPI_MTX_TABLES);



unlock_and_exit:

CRA Git | Maintained and supported by SUSTech CRA and CCSE