[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] Fix up pxe boot
From: |
Glauber Costa |
Subject: |
[Qemu-devel] Re: [PATCH] Fix up pxe boot |
Date: |
Tue, 2 Sep 2008 12:22:19 -0300 |
On Tue, Sep 2, 2008 at 12:20 PM, Avi Kivity <address@hidden> wrote:
> Glauber Costa wrote:
>>
>> On Tue, Sep 2, 2008 at 5:39 AM, Avi Kivity <address@hidden> wrote:
>>
>>>
>>> Glauber Costa wrote:
>>>
>>>>
>>>> diff --git a/target-i386/op_helper.c b/target-i386/op_helper.c
>>>> index 0b5fdc0..433aa3f 100644
>>>> --- a/target-i386/op_helper.c
>>>> +++ b/target-i386/op_helper.c
>>>> @@ -600,7 +600,7 @@ do {\
>>>> #define PUSHL(ssp, sp, sp_mask, val)\
>>>> {\
>>>> sp -= 4;\
>>>> - stl_kernel((ssp) + (sp & (sp_mask)), (val));\
>>>> + stl_kernel((uint32_t)((ssp) + (sp & (sp_mask))), (uint32_t)(val));\
>>>> }
>>>>
>>>>
>>>
>>> Surly it is better to push this into the underlying virtual->physical
>>> translation functions, so it applies everywhere?
>>>
>>> btw, the cast is wrong for x86-64, so it must be qualified for 32-bit
>>> operating modes.
>>>
>>
>> The tests were all done with x86_64. This is a PUSHL macro, so it's
>> 32-bit anyway.
>> A x86_64-only PUSHQ seems to do the right thing.
>>
>>
>
> Right.
>
> It's still odd to see this in an op helper rather than in somewhere generic.
>
I'll take a closer look today to see if I can improve it better.
> --
> error compiling committee.c: too many arguments to function
>
>
--
Glauber Costa.
"Free as in Freedom"
http://glommer.net
"The less confident you are, the more serious you have to act."
- [Qemu-devel] [PATCH] Fix up pxe boot, Glauber Costa, 2008/09/01
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Avi Kivity, 2008/09/02
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Glauber Costa, 2008/09/02
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Avi Kivity, 2008/09/02
- [Qemu-devel] Re: [PATCH] Fix up pxe boot,
Glauber Costa <=
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Glauber Costa, 2008/09/03
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Avi Kivity, 2008/09/07
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Glauber Costa, 2008/09/08
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Avi Kivity, 2008/09/09
- [Qemu-devel] Re: [PATCH] Fix up pxe boot, Anthony Liguori, 2008/09/09