Re: [PATCH 17/39] clobber rbx in putuser_64.S

From: Glauber Costa
Date: Mon Jun 30 2008 - 17:03:06 EST


Andi Kleen wrote:
Glauber Costa <gcosta@xxxxxxxxxx> writes:

Instead of clobbering r8, clobber rbx, which is the i386 way.

Note rbx is callee saved on 64bit, so using that one means
the surrounding function always has to save explicitely.
Not the case with r8.

There's a reason it is the way it is.

-Andi
Right. Thanks for pointing this out.
However, r8 is not available for i386. We could use %ax, but it
holds part of the data for the call itself.

But for this case, I think we can come up with a macro that selects the appropriate register for each of them. Should be easy to do now that the code is merged.

Many thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/