[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/3] target-i386: Use C struct for xsave area
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/3] target-i386: Use C struct for xsave area layout, offsets & sizes |
Date: |
Wed, 11 May 2016 16:19:22 -0300 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Dec 04, 2015 at 01:00:52PM -0200, Eduardo Habkost wrote:
> target-i386/cpu.c:ext_save_area uses magic numbers for the xsave
> area offets and sizes, and target-i386/kvm.c:kvm_{put,get}_xsave()
> uses offset macros and bit manipulation to access the xsave area.
> This series changes both to use C structs for those operations.
>
> I still need to figure out a way to write unit tests for the new
> code. Maybe I will just copy and paste the new and old functions,
> and test them locally (checking if they give the same results
> when translating blobs of random bytes).
Paolo, Richard,
I plan to include this in my next pull request (next Monday,
assuming v2.6.0 will be already tagged). If you have any
objections, please let me know.
--
Eduardo
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 0/3] target-i386: Use C struct for xsave area layout, offsets & sizes,
Eduardo Habkost <=