On Tue, Apr 13, 2021 at 03:40:56PM +0200, Vitaly Kuznetsov wrote: > Siddharth Chandrasekaran <sidcha@xxxxxxxxx> writes: > > @@ -0,0 +1,140 @@ > > +/* SPDX-License-Identifier: GPL-2.0 */ > > + > > +#ifndef __KVM_FPU_H_ > > +#define __KVM_FPU_H_ > > + > > +#include <asm/fpu/api.h> > > + > > +typedef u32 __attribute__((vector_size(16))) sse128_t; > > Post-patch we seem to have two definitions of 'sse128_t': > > $ git grep sse128_t > HEAD~3:arch/x86/kvm/fpu.h:typedef u32 __attribute__((vector_size(16))) sse128_t; > HEAD~3:arch/x86/kvm/fpu.h:#define __sse128_u union { sse128_t vec; u64 as_u64[2]; u32 as_u32[4]; } > HEAD~3:arch/x86/kvm/fpu.h:static inline void _kvm_read_sse_reg(int reg, sse128_t *data) > HEAD~3:arch/x86/kvm/fpu.h:static inline void _kvm_write_sse_reg(int reg, const sse128_t *data) > HEAD~3:arch/x86/kvm/fpu.h:static inline void kvm_read_sse_reg(int reg, sse128_t *data) > HEAD~3:arch/x86/kvm/fpu.h:static inline void kvm_write_sse_reg(int reg, const sse128_t *data) > HEAD~3:arch/x86/kvm/kvm_emulate.h:typedef u32 __attribute__((vector_size(16))) sse128_t; > HEAD~3:arch/x86/kvm/kvm_emulate.h: char valptr[sizeof(sse128_t)]; > HEAD~3:arch/x86/kvm/kvm_emulate.h: sse128_t vec_val; > > Should the one from kvm_emulate.h go away? Yes, removed. ~ Sid. Amazon Development Center Germany GmbH Krausenstr. 38 10117 Berlin Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B Sitz: Berlin Ust-ID: DE 289 237 879