Re: [PATCH v10 13/21] ACPI / processor: Introduce phys_cpuid_t for CPU hardware ID

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2015年03月12日 17:51, Lorenzo Pieralisi wrote:
On Wed, Mar 11, 2015 at 12:39:39PM +0000, Hanjun Guo wrote:
CPU hardware ID (phys_id) is defined as u32 in structure acpi_processor,
but phys_id is used as int in acpi processor driver, so it will lead to
some inconsistence for the drivers.

Furthermore, to cater for ACPI arch ports that implement 64 bits CPU
ids a generic CPU physical id type is required.

So introduce typedef u32 phys_cpuid_t for x86 and ia64, and introduce
a macro CPU_PHYS_ID_INVALID as (u32)(-1), use phys_cpuid_t when phys_id
defined in acpi processor driver, and replace CPU_PHYS_ID_INVALID as -1
for phys_id, this will solve the inconsistence in acpi processor driver,
and will prepare for the ACPI on ARM64 for the 64 bit CPU hardware ID
in the following patch.

Nit: Commit log above is stale (since you moved the phys_cpuid_t to
common declaration for all archs that do not define PHYS_CPUID_INVALID
and the macro name itself changed) so you should rewrite it before

Oops, I will send a updated patch only for this one, good catch.

asking to merge it. Furthermore you should not describe how you implemented
the change, you should describe why.

I'm prefer writing the commit log as following way (not always) :)
 - why the patch is needed
 - what the patch implements
 - How the patch is implemented


Other than that:

Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>

Thanks!

Hanjun
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux