From: "Zheng, Lv" <lv.zheng@intel.com>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
ACPI Devel Maling List <linux-acpi@vger.kernel.org>
Cc: "Moore, Robert" <robert.moore@intel.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] ACPICA: Drop Linux-specific waking vector functions
Date: Mon, 4 Jan 2016 07:33:55 +0000 [thread overview]
Message-ID: <1AE640813FDE7649BE1B193DEA596E883BB1AB15@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <1672897.3KviH2SFad@vostro.rjw.lan>
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 10889 bytes --]
Hi, Rafael
Acke-by: Lv Zheng <lv.zheng@intel.com>
One more improvement is:
After applying this patch, I can still detect the following diff blocks using ACPICA's divergence checking utility:
diff -E -b -w -B -rpuN linux-acpica/drivers/acpi/acpica/hwxfsleep.c acpica-linuxized/drivers/acpi/acpica/hwxfsleep.c
--- linux-acpica/drivers/acpi/acpica/hwxfsleep.c 2016-01-04 15:16:31.000000000 +0800
+++ acpica-linuxized/drivers/acpi/acpica/hwxfsleep.c 2016-01-04 15:17:19.000000000 +0800
@@ -90,9 +90,9 @@ static struct acpi_sleep_functions acpi_
*
* PARAMETERS: facs - Pointer to FACS table
* physical_address - 32-bit physical address of ACPI real mode
- * entry point.
+ * entry point
* physical_address64 - 64-bit physical address of ACPI protected
- * mode entry point.
+ * entry point
*
* RETURN: Status
*
@@ -141,9 +140,9 @@ acpi_hw_set_firmware_waking_vector(struc
* FUNCTION: acpi_set_firmware_waking_vector
*
* PARAMETERS: physical_address - 32-bit physical address of ACPI real mode
- * entry point.
+ * entry point
* physical_address64 - 64-bit physical address of ACPI protected
- * mode entry point.
+ * entry point
*
* RETURN: Status
*
This seems to be added recently in ACPICA upstream.
Hope you can help to reduce this in the same patch.
Thanks in advance.
Best regards
-Lv
> From: Rafael J. Wysocki [mailto:rjw@rjwysocki.net]
> Sent: Sunday, January 3, 2016 9:51 AM
> Subject: [PATCH] ACPICA: Drop Linux-specific waking vector functions
>
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
>
> Commit f06147f9fbf1 (ACPICA: Hardware: Enable firmware waking vector
> for both 32-bit and 64-bit FACS) added three functions that aren't
> present in upstream ACPICA, acpi_hw_set_firmware_waking_vectors(),
> acpi_set_firmware_waking_vectors() and
> acpi_set_firmware_waking_vector64(),
> to allow Linux to use the previously existing API for setting the
> platform firmware waking vector.
>
> However, that wasn't necessary, since the ACPI sleep support code
> in Linux can be modified to use the upstream ACPICA's API easily
> and the additional functions may be dropped which reduces the code
> size and puts the kernel's ACPICA code more in line with the upstream.
>
> Make the changes as per the above.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> drivers/acpi/acpica/hwxfsleep.c | 88 ++++++----------------------------------
> drivers/acpi/sleep.c | 4 -
> drivers/acpi/sleep.h | 6 ++
> include/acpi/acpixf.h | 10 ----
> 4 files changed, 24 insertions(+), 84 deletions(-)
>
> Index: linux-pm/drivers/acpi/acpica/hwxfsleep.c
> ================================================================
> ===
> --- linux-pm.orig/drivers/acpi/acpica/hwxfsleep.c
> +++ linux-pm/drivers/acpi/acpica/hwxfsleep.c
> @@ -52,9 +52,9 @@ ACPI_MODULE_NAME("hwxfsleep")
> /* Local prototypes */
> #if (!ACPI_REDUCED_HARDWARE)
> static acpi_status
> -acpi_hw_set_firmware_waking_vectors(struct acpi_table_facs *facs,
> - acpi_physical_address physical_address,
> - acpi_physical_address physical_address64);
> +acpi_hw_set_firmware_waking_vector(struct acpi_table_facs *facs,
> + acpi_physical_address physical_address,
> + acpi_physical_address physical_address64);
> #endif
>
> static acpi_status acpi_hw_sleep_dispatch(u8 sleep_state, u32 function_id);
> @@ -79,16 +79,14 @@ static struct acpi_sleep_functions acpi_
>
> /*
> * These functions are removed for the ACPI_REDUCED_HARDWARE case:
> - * acpi_set_firmware_waking_vectors
> * acpi_set_firmware_waking_vector
> - * acpi_set_firmware_waking_vector64
> * acpi_enter_sleep_state_s4bios
> */
>
> #if (!ACPI_REDUCED_HARDWARE)
>
> /***************************************************************
> ****************
> *
> - * FUNCTION: acpi_hw_set_firmware_waking_vectors
> + * FUNCTION: acpi_hw_set_firmware_waking_vector
> *
> * PARAMETERS: facs - Pointer to FACS table
> * physical_address - 32-bit physical address of ACPI real mode
> @@ -103,11 +101,11 @@ static struct acpi_sleep_functions acpi_
>
> ****************************************************************
> **************/
>
> static acpi_status
> -acpi_hw_set_firmware_waking_vectors(struct acpi_table_facs *facs,
> - acpi_physical_address physical_address,
> - acpi_physical_address physical_address64)
> +acpi_hw_set_firmware_waking_vector(struct acpi_table_facs *facs,
> + acpi_physical_address physical_address,
> + acpi_physical_address physical_address64)
> {
> - ACPI_FUNCTION_TRACE(acpi_hw_set_firmware_waking_vectors);
> + ACPI_FUNCTION_TRACE(acpi_hw_set_firmware_waking_vector);
>
>
> /*
> @@ -140,7 +138,7 @@ acpi_hw_set_firmware_waking_vectors(stru
>
>
> /***************************************************************
> ****************
> *
> - * FUNCTION: acpi_set_firmware_waking_vectors
> + * FUNCTION: acpi_set_firmware_waking_vector
> *
> * PARAMETERS: physical_address - 32-bit physical address of ACPI real
> mode
> * entry point.
> @@ -154,79 +152,23 @@ acpi_hw_set_firmware_waking_vectors(stru
>
> ****************************************************************
> **************/
>
> acpi_status
> -acpi_set_firmware_waking_vectors(acpi_physical_address physical_address,
> - acpi_physical_address physical_address64)
> +acpi_set_firmware_waking_vector(acpi_physical_address physical_address,
> + acpi_physical_address physical_address64)
> {
>
> - ACPI_FUNCTION_TRACE(acpi_set_firmware_waking_vectors);
> + ACPI_FUNCTION_TRACE(acpi_set_firmware_waking_vector);
>
> if (acpi_gbl_FACS) {
> - (void)acpi_hw_set_firmware_waking_vectors(acpi_gbl_FACS,
> - physical_address,
> - physical_address64);
> + (void)acpi_hw_set_firmware_waking_vector(acpi_gbl_FACS,
> + physical_address,
> + physical_address64);
> }
>
> return_ACPI_STATUS(AE_OK);
> }
>
> -ACPI_EXPORT_SYMBOL(acpi_set_firmware_waking_vectors)
> -
> -
> /***************************************************************
> ****************
> - *
> - * FUNCTION: acpi_set_firmware_waking_vector
> - *
> - * PARAMETERS: physical_address - 32-bit physical address of ACPI real
> mode
> - * entry point.
> - *
> - * RETURN: Status
> - *
> - * DESCRIPTION: Sets the 32-bit firmware_waking_vector field of the FACS
> - *
> -
> ****************************************************************
> **************/
> -acpi_status acpi_set_firmware_waking_vector(u32 physical_address)
> -{
> - acpi_status status;
> -
> - ACPI_FUNCTION_TRACE(acpi_set_firmware_waking_vector);
> -
> - status = acpi_set_firmware_waking_vectors((acpi_physical_address)
> - physical_address, 0);
> -
> - return_ACPI_STATUS(status);
> -}
> -
> ACPI_EXPORT_SYMBOL(acpi_set_firmware_waking_vector)
>
> -#if ACPI_MACHINE_WIDTH == 64
> -
> /***************************************************************
> ****************
> - *
> - * FUNCTION: acpi_set_firmware_waking_vector64
> - *
> - * PARAMETERS: physical_address - 64-bit physical address of ACPI
> protected
> - * mode entry point.
> - *
> - * RETURN: Status
> - *
> - * DESCRIPTION: Sets the 64-bit X_firmware_waking_vector field of the FACS,
> if
> - * it exists in the table. This function is intended for use with
> - * 64-bit host operating systems.
> - *
> -
> ****************************************************************
> **************/
> -acpi_status acpi_set_firmware_waking_vector64(u64 physical_address)
> -{
> - acpi_status status;
> -
> - ACPI_FUNCTION_TRACE(acpi_set_firmware_waking_vector64);
> -
> - status = acpi_set_firmware_waking_vectors(0,
> - (acpi_physical_address)
> - physical_address);
> -
> - return_ACPI_STATUS(status);
> -}
> -
> -ACPI_EXPORT_SYMBOL(acpi_set_firmware_waking_vector64)
> -#endif
>
> /***************************************************************
> ****************
> *
> * FUNCTION: acpi_enter_sleep_state_s4bios
> Index: linux-pm/include/acpi/acpixf.h
> ================================================================
> ===
> --- linux-pm.orig/include/acpi/acpixf.h
> +++ linux-pm/include/acpi/acpixf.h
> @@ -863,17 +863,9 @@ ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> ACPI_EXTERNAL_RETURN_STATUS(acpi_status acpi_leave_sleep_state(u8
> sleep_state))
>
> ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status
> - acpi_set_firmware_waking_vectors
> + acpi_set_firmware_waking_vector
> (acpi_physical_address physical_address,
> acpi_physical_address physical_address64))
> -ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status
> - acpi_set_firmware_waking_vector(u32
> -
> physical_address))
> -#if ACPI_MACHINE_WIDTH == 64
> -ACPI_HW_DEPENDENT_RETURN_STATUS(acpi_status
> - acpi_set_firmware_waking_vector64(u64
> -
> physical_address))
> -#endif
> /*
> * ACPI Timer interfaces
> */
> Index: linux-pm/drivers/acpi/sleep.c
> ================================================================
> ===
> --- linux-pm.orig/drivers/acpi/sleep.c
> +++ linux-pm/drivers/acpi/sleep.c
> @@ -61,7 +61,7 @@ static int acpi_sleep_prepare(u32 acpi_s
> if (acpi_state == ACPI_STATE_S3) {
> if (!acpi_wakeup_address)
> return -EFAULT;
> - acpi_set_firmware_waking_vector(acpi_wakeup_address);
> + acpi_set_waking_vector(acpi_wakeup_address);
>
> }
> ACPI_FLUSH_CPU_CACHE();
> @@ -410,7 +410,7 @@ static void acpi_pm_finish(void)
> acpi_leave_sleep_state(acpi_state);
>
> /* reset firmware waking vector */
> - acpi_set_firmware_waking_vector((acpi_physical_address) 0);
> + acpi_set_waking_vector(0);
>
> acpi_target_sleep_state = ACPI_STATE_S0;
>
> Index: linux-pm/drivers/acpi/sleep.h
> ================================================================
> ===
> --- linux-pm.orig/drivers/acpi/sleep.h
> +++ linux-pm/drivers/acpi/sleep.h
> @@ -6,3 +6,9 @@ extern struct list_head acpi_wakeup_devi
> extern struct mutex acpi_device_lock;
>
> extern void acpi_resume_power_resources(void);
> +
> +static inline acpi_status acpi_set_waking_vector(u32 wakeup_address)
> +{
> + return acpi_set_firmware_waking_vector(
> + (acpi_physical_address)wakeup_address, 0);
> +}
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
next prev parent reply other threads:[~2016-01-04 7:34 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-03 1:51 [PATCH] ACPICA: Drop Linux-specific waking vector functions Rafael J. Wysocki
2016-01-04 7:33 ` Zheng, Lv [this message]
2016-01-04 13:28 ` Rafael J. Wysocki
2016-01-05 3:10 ` Zheng, Lv
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1AE640813FDE7649BE1B193DEA596E883BB1AB15@SHSMSX101.ccr.corp.intel.com \
--to=lv.zheng@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rjw@rjwysocki.net \
--cc=robert.moore@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).