Re: [PATCH] acpi: Handle cpu_index greater than 256 properly in processor_core.c

Hi Venkatesh,

On Saturday 27 August 2005 02:07, Venkatesh Pallipadi wrote:
> Fix convert_acpiid_to_cpu function to handle cpu_index greater than 256. This 
> patch also prevents a warning in IA64 cross-compile of this file 
> (drivers/acpi/processor_core.c:517: warning: comparison is always false due 
> to limited range of data type).

Why don't you just change the datatype to "unsigned int" and 
the return failure value to NR_CPUS?

That reduces the code changes and leaves the code quite clear.
It should also reduce compiled code size by some bytes, but I'm not
sure about that one.


Regards

Ingo Oeser

Attachment: pgp77dSlpak7x.pgp
Description: PGP signature

---

• References:
  • [PATCH] acpi: Handle cpu_index greater than 256 properly in processor_core.c
    • From: Venkatesh Pallipadi <[email protected]>

• Prev by Date: Linux 2.6 context switching and posix threads performance question
• Next by Date: Re: Linux 2.6 context switching and posix threads performance question
• Previous by thread: [PATCH] acpi: Handle cpu_index greater than 256 properly in processor_core.c
• Next by thread: RE: [PATCH] acpi: Handle cpu_index greater than 256 properly in processor_core.c
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]