Re: [PATCH] x86-64: make pda's cpunumber and nodenumber unsigned

* Jan Beulich <[email protected]> wrote:

> This generally allows better code to be generated, since the zero- 
> extension during 32-bit operations comes for free (needed when the 
> result is used as array index or similar), whereas sign extension must 
> be done explicitly and frequently requires a one byte larger 
> instruction due to the necessary rex64 prefix.

good catch! Applied your patch to x86.git - queued it up for v2.6.25. I 
bet there are tons of other instances where we use signed instead of 
unsigned and get worse code generation.

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] x86-64: make pda's cpunumber and nodenumber unsigned
    • From: "Jan Beulich" <[email protected]>

• References:
  • [PATCH] x86-64: make pda's cpunumber and nodenumber unsigned
    • From: "Jan Beulich" <[email protected]>

• Prev by Date: Re: [PATCH] i386: hard_{en,dis}able_TSC can be static
• Next by Date: Re: [PATCH] x86-64: honor notify_die() returning NOTIFY_STOP
• Previous by thread: [PATCH] x86-64: make pda's cpunumber and nodenumber unsigned
• Next by thread: Re: [PATCH] x86-64: make pda's cpunumber and nodenumber unsigned
• Index(es):
  • Date
  • Thread

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