Re: [PATCH v5 0/7] mips/kvm: Support FPU & SIMD (MSA) in MIPS KVM guests

James Hogan
2016-02-05T10:46:25Z

[-- Attachment #1: Type: text/plain, Size: 2424 bytes --]

On Thu, Feb 04, 2016 at 10:04:11AM +0000, Leon Alrae wrote:
> On 03/02/16 17:16, James Hogan wrote:
> > Here's a v5 refresh of my FPU/MSA patchset for v2.6. Thanks to all who
> > have taken the time to review it so far.
> >
> > This patchset primarily adds support for FPU and MIPS SIMD Architecture
> > (MSA) in MIPS KVM guests to QEMU. It depends on Linux v4.1, specifically
> > my KVM patchset to add the corresponding hypervisor support to KVM
> > ("[PATCH 00/20] MIPS: KVM: Guest FPU & SIMD (MSA) support").
> >
> > All comments welcome.
> >
> > Changes in v5:
> > - Rebase on master (fixed use of uint64 in patch 5).
> > - Use restore_fp_status(env) in patch 6.
> > - Restore MSA FP state using restore_msa_fp_status(env) in patch 7
> > (Leon).
> >
> > Changes in v4:
> > - Rebase on master (dropped patch 1 & 2).
> >
> > Changes in v3 (patch 6 only):
> > - Fix big endian (the pointer passed to the kernel must be for the
> > actual 32-bit value, not a temporary 64-bit value, otherwise on big
> > endian systems the kernel will only interpret the upper half).
> >
> > Changes in v2:
> > - Moved most of patch 7 and updates to linux-headers/linux/kvm.h from
> > patches 8 and 9 into a new patch 1, which is purely for reference
> > (Paolo).
> > - Add the changes to MIPS_CP0_{32,64} macros from v1 patch 7 to patch 2,
> > since the rest of that patch is now unnecessary and the change is
> > along the same lines as patch 2 (not added Leon's Reviewed-by to this
> > patch due to that non-reviewed change).
> > - Fix line wrapping of kvm_mips_get_one_reg() calls from Config4 and
> > Config5 in patch 5 (Leon).
> > - Change (1 << x) to (1U << x) in important places in patch 5, 8 & 9 to
> > avoid compiler undefined behaviour (Leon).
> >
> > James Hogan (7):
> > mips/kvm: Remove a couple of noisy DPRINTFs
> > mips/kvm: Implement PRid CP0 register
> > mips/kvm: Implement Config CP0 registers
> > mips/kvm: Support unsigned KVM registers
> > mips/kvm: Support signed 64-bit KVM registers
> > mips/kvm: Support FPU in MIPS KVM guests
> > mips/kvm: Support MSA in MIPS KVM guests
> >
> > target-mips/kvm.c | 387 ++++++++++++++++++++++++++++++++++++++++++++++++++++--
> > 1 file changed, 374 insertions(+), 13 deletions(-)
> >
>
> I've applied the series to my target-mips queue, thanks.

Thanks Leon!

Cheers
James

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]