>>> On 09.12.16 at 16:27, <jbeul...@suse.com> wrote:
>>>> On 09.12.16 at 16:19, <jbeul...@suse.com> wrote:
>>>>> On 09.12.16 at 15:56, <andrew.coop...@citrix.com> wrote:
>>> It is a layering violation to remove the host_has_* part of the check
>>> from the emulator.
>>
>> This is from a very, very abstract perspective. The fact that
>> host_and_vcpu_*() and vcpu_*() are the same in the harness
>> should allow for them to be used interchangeably when only the
>> harness is affected.
>
> I should add that in a subsequent patch adding emulation of
> LDDQU (an SSE3 insn) this is going to be done by slightly
> extending the existing MOVDQ{U,A} etc block, i.e. using an
> SSE2 insn. If I was to follow your directions, I'd have to add
> separate host_has(sse2) and vcpu_has(sse3) checks. That's
> just going to become insane. I have just the vcpu check in
> there, and I really hope you won't force me to uglify that.
And one more which did stall. Meanwhile I've submitted patches
falling into the above category, which should help making a
judgment here.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
