target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

[PATCH 0/2] target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

Richard Henderson posted 2 patches 4 years, 8 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210226162022.375654-1-richard.henderson@linaro.org

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>, Richard Henderson <richard.henderson@linaro.org>

target/i386/tcg/fpu_helper.c | 65 +++++++++++++++++++++++-------------
1 file changed, 41 insertions(+), 24 deletions(-)

Expand all Fold all

[PATCH 0/2] target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

Posted by Richard Henderson 4 years, 8 months ago

As discussed during review of Claudio's "i386 cleanup" patch set.


r~


Richard Henderson (2):
  target/i386: Rename helper_fldt, helper_fstt
  target/i386: Split out do_fsave, do_frstor, do_fxsave, do_fxrstor

 target/i386/tcg/fpu_helper.c | 65 +++++++++++++++++++++++-------------
 1 file changed, 41 insertions(+), 24 deletions(-)

-- 
2.25.1

Re: [PATCH 0/2] target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

Posted by Claudio Fontana 4 years, 8 months ago

On 2/26/21 5:20 PM, Richard Henderson wrote:
> As discussed during review of Claudio's "i386 cleanup" patch set.
> 
> 
> r~
> 
> 
> Richard Henderson (2):
>   target/i386: Rename helper_fldt, helper_fstt
>   target/i386: Split out do_fsave, do_frstor, do_fxsave, do_fxrstor
> 
>  target/i386/tcg/fpu_helper.c | 65 +++++++++++++++++++++++-------------
>  1 file changed, 41 insertions(+), 24 deletions(-)
> 
Reviewed-by: Claudio Fontana <cfontana@suse.de>
Tested-by: Claudio Fontana <cfontana@suse.de>

Re: [PATCH 0/2] target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

Posted by Paolo Bonzini 4 years, 8 months ago

On 26/02/21 17:59, Claudio Fontana wrote:
> On 2/26/21 5:20 PM, Richard Henderson wrote:
>> As discussed during review of Claudio's "i386 cleanup" patch set.
>>
>>
>> r~
>>
>>
>> Richard Henderson (2):
>>    target/i386: Rename helper_fldt, helper_fstt
>>    target/i386: Split out do_fsave, do_frstor, do_fxsave, do_fxrstor
>>
>>   target/i386/tcg/fpu_helper.c | 65 +++++++++++++++++++++++-------------
>>   1 file changed, 41 insertions(+), 24 deletions(-)
>>
> Reviewed-by: Claudio Fontana <cfontana@suse.de>
> Tested-by: Claudio Fontana <cfontana@suse.de>
> 

Claudio, can you merge them in your patch set?

Thanks,

Paolo

Re: [PATCH 0/2] target/i386: Fix cpu_x86_{fsave,frstor,fxsave,fxrstor}

Posted by Claudio Fontana 4 years, 8 months ago

On 2/26/21 6:12 PM, Paolo Bonzini wrote:
> On 26/02/21 17:59, Claudio Fontana wrote:
>> On 2/26/21 5:20 PM, Richard Henderson wrote:
>>> As discussed during review of Claudio's "i386 cleanup" patch set.
>>>
>>>
>>> r~
>>>
>>>
>>> Richard Henderson (2):
>>>    target/i386: Rename helper_fldt, helper_fstt
>>>    target/i386: Split out do_fsave, do_frstor, do_fxsave, do_fxrstor
>>>
>>>   target/i386/tcg/fpu_helper.c | 65 +++++++++++++++++++++++-------------
>>>   1 file changed, 41 insertions(+), 24 deletions(-)
>>>
>> Reviewed-by: Claudio Fontana <cfontana@suse.de>
>> Tested-by: Claudio Fontana <cfontana@suse.de>
>>
> 
> Claudio, can you merge them in your patch set?
> 
> Thanks,
> 
> Paolo
> 

Yep, doing that.

Ciao,

Claudio