From: Hanjun Guo <hanjun.guo@linaro.org>
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Jiang Liu <jiang.liu@linux.intel.com>,
Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>,
Sudeep Holla <Sudeep.Holla@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Tony Luck <tony.luck@intel.com>, "H. Peter Anvin" <hpa@zytor.com>,
Lv Zheng <lv.zheng@intel.com>,
linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linaro-acpi@lists.linaro.org, Hanjun Guo <hanjun.guo@linaro.com>
Subject: Re: [PATCH v2 2/3] ACPI / processor: Convert apic_id to phys_id to make it arch agnostic
Date: Thu, 27 Nov 2014 13:29:55 +0800 [thread overview]
Message-ID: <5476B6D3.9080700@linaro.org> (raw)
In-Reply-To: <1417064853-27905-3-git-send-email-hanjun.guo@linaro.com>
Hi Rafael,
On 2014-11-27 13:07, Hanjun Guo wrote:
> apic_id in MADT table is the CPU hardware id which identify
> it self in the system for x86 and ia64, OSPM will use it for
> SMP init to map APIC ID to logical cpu number in the early
> boot, when the DSDT/SSDT (ACPI namespace) is scanned later, the
> ACPI processor driver is probed and the driver will use acpi_id
> in DSDT to get the apic_id, then map to the logical cpu number
> which is needed by the processor driver.
>
> Before ACPI 5.0, only x86 and ia64 were supported in ACPI spec,
> so apic_id is used both in arch code and ACPI core which is
> pretty fine. Since ACPI 5.0, ARM is supported by ACPI and
> APIC is not available on ARM, this will confuse people when
> apic_id is both used by x86 and ARM in one function.
>
> So convert apic_id to phys_id (which is the original meaning)
> in ACPI processor dirver to make it arch agnostic, but leave the
> arch dependent code unchanged, no functional change.
>
> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.com>
Sorry, my bad, not linaro.com but linaro.org, and patch 3
has the same problem, if you need resend those patch, please
let me know.
Thanks
Hanjun
WARNING: multiple messages have this Message-ID (diff)
From: hanjun.guo@linaro.org (Hanjun Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/3] ACPI / processor: Convert apic_id to phys_id to make it arch agnostic
Date: Thu, 27 Nov 2014 13:29:55 +0800 [thread overview]
Message-ID: <5476B6D3.9080700@linaro.org> (raw)
In-Reply-To: <1417064853-27905-3-git-send-email-hanjun.guo@linaro.com>
Hi Rafael,
On 2014-11-27 13:07, Hanjun Guo wrote:
> apic_id in MADT table is the CPU hardware id which identify
> it self in the system for x86 and ia64, OSPM will use it for
> SMP init to map APIC ID to logical cpu number in the early
> boot, when the DSDT/SSDT (ACPI namespace) is scanned later, the
> ACPI processor driver is probed and the driver will use acpi_id
> in DSDT to get the apic_id, then map to the logical cpu number
> which is needed by the processor driver.
>
> Before ACPI 5.0, only x86 and ia64 were supported in ACPI spec,
> so apic_id is used both in arch code and ACPI core which is
> pretty fine. Since ACPI 5.0, ARM is supported by ACPI and
> APIC is not available on ARM, this will confuse people when
> apic_id is both used by x86 and ARM in one function.
>
> So convert apic_id to phys_id (which is the original meaning)
> in ACPI processor dirver to make it arch agnostic, but leave the
> arch dependent code unchanged, no functional change.
>
> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.com>
Sorry, my bad, not linaro.com but linaro.org, and patch 3
has the same problem, if you need resend those patch, please
let me know.
Thanks
Hanjun
next prev parent reply other threads:[~2014-11-27 5:32 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-27 5:07 [PATCH v2 0/3] cleanups for converting apic_id to phys_id to make it arch agnostic Hanjun Guo
2014-11-27 5:07 ` Hanjun Guo
2014-11-27 5:07 ` [PATCH v2 1/3] ACPI / processor: Update the comments in processor.h Hanjun Guo
2014-11-27 5:07 ` Hanjun Guo
2014-11-27 5:07 ` [PATCH v2 2/3] ACPI / processor: Convert apic_id to phys_id to make it arch agnostic Hanjun Guo
2014-11-27 5:07 ` Hanjun Guo
2014-11-27 5:29 ` Hanjun Guo [this message]
2014-11-27 5:29 ` Hanjun Guo
2014-11-27 5:07 ` [PATCH v2 3/3] ACPI / processor: Rename acpi_(un)map_lsapic() to acpi_(un)map_cpu() Hanjun Guo
2014-11-27 5:07 ` Hanjun Guo
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=5476B6D3.9080700@linaro.org \
--to=hanjun.guo@linaro.org \
--cc=Lorenzo.Pieralisi@arm.com \
--cc=Sudeep.Holla@arm.com \
--cc=hanjun.guo@linaro.com \
--cc=hpa@zytor.com \
--cc=jiang.liu@linux.intel.com \
--cc=linaro-acpi@lists.linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=lv.zheng@intel.com \
--cc=mark.rutland@arm.com \
--cc=rjw@rjwysocki.net \
--cc=tony.luck@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.