From nobody Wed Dec 17 04:03:16 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=linux.microsoft.com ARC-Seal: i=1; a=rsa-sha256; t=1754577800; cv=none; d=zohomail.com; s=zohoarc; b=RjtcjOZWeP8/Bc+uag2JhAH27NdN50CgXQVf4ZQW7+fokHHg+BH0nQK5ZvPaHQu26+ZgHbS+FNH9etDEXKH6O+8qV6//KESZMIlWplkDmTlB28RsD4t1aRo2imngJUL9YRhSfKvR39NmWPRKjoItQf/70LPlzUs8AnaA2ZJsu9Q= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1754577800; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=qIAQxkUlxMHjIAo0ZhNfyDqS2DKve41IsvEjvKNBakw=; b=JvpD6ooe0gEI5UVXSjBpWFdJPZzqCA//lX1clGkbtJFbC/Ew/dOi3YaxLelw/C4Np2TaKCGNp2IQEUZe4R2IGJlnC5YJR14BGmQY1kmFDYoQRzuPoilVqmNufsMKWWZTEiBiiNy55bcL/iBZ+x6EwJnc0FTzCqRfRw19+jOXvbc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1754577800144946.2323021333765; Thu, 7 Aug 2025 07:43:20 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1uk1pc-0003Pk-OO; Thu, 07 Aug 2025 10:43:02 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1uk1oP-0000Hh-J4 for qemu-devel@nongnu.org; Thu, 07 Aug 2025 10:41:48 -0400 Received: from linux.microsoft.com ([13.77.154.182]) by eggs.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1uk1oM-00012i-UX for qemu-devel@nongnu.org; Thu, 07 Aug 2025 10:41:44 -0400 Received: from localhost.localdomain (unknown [167.220.208.72]) by linux.microsoft.com (Postfix) with ESMTPSA id 8AB87201BC82; Thu, 7 Aug 2025 07:41:37 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 8AB87201BC82 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1754577702; bh=qIAQxkUlxMHjIAo0ZhNfyDqS2DKve41IsvEjvKNBakw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=Q/THkAaaDGkM+3H2ooDViaoBAWirzafHtjyGrEcKMEAtxMt2lyHAXCvbbU6phAMb6 j8zmTyaTodtwctyHXpyjkz5YhsSh6DJMaPMfzoA+3mSxPx1+r8i4KB9umQzfza0Jwz UF/7PLimJ0Rkt80ZF2fzbjv+YLRJuYHUan1O6fGc= From: Magnus Kulke To: qemu-devel@nongnu.org Cc: Eric Blake , Eduardo Habkost , "Michael S. Tsirkin" , Markus Armbruster , Magnus Kulke , Paolo Bonzini , Richard Henderson , Phil Dennis-Jordan , Marcel Apfelbaum , =?UTF-8?q?Alex=20Benn=C3=A9e?= , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , Magnus Kulke , Cornelia Huck , Zhao Liu , Thomas Huth , Yanan Wang , Cameron Esfahani , Wei Liu , Wei Liu , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Roman Bolshakov , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= Subject: [PATCH v3 16/26] target/i386/mshv: Implement mshv_arch_put_registers() Date: Thu, 7 Aug 2025 16:39:41 +0200 Message-Id: <20250807143951.1154713-17-magnuskulke@linux.microsoft.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20250807143951.1154713-1-magnuskulke@linux.microsoft.com> References: <20250807143951.1154713-1-magnuskulke@linux.microsoft.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=13.77.154.182; envelope-from=magnuskulke@linux.microsoft.com; helo=linux.microsoft.com X-Spam_score_int: -19 X-Spam_score: -2.0 X-Spam_bar: -- X-Spam_report: (-2.0 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @linux.microsoft.com) X-ZM-MESSAGEID: 1754577801585124100 Content-Type: text/plain; charset="utf-8" Write CPU register state to MSHV vCPUs. Various mapping functions to prepare the payload for the HV call have been implemented. Signed-off-by: Magnus Kulke --- include/system/mshv.h | 15 +++ target/i386/mshv/mshv-cpu.c | 239 ++++++++++++++++++++++++++++++++++++ 2 files changed, 254 insertions(+) diff --git a/include/system/mshv.h b/include/system/mshv.h index 6ca82d367b..ec41e62315 100644 --- a/include/system/mshv.h +++ b/include/system/mshv.h @@ -81,6 +81,20 @@ typedef struct MshvMsiControl { #define mshv_msi_via_irqfd_enabled() mshv_enabled() =20 /* cpu */ +typedef struct MshvFPU { + uint8_t fpr[8][16]; + uint16_t fcw; + uint16_t fsw; + uint8_t ftwx; + uint8_t pad1; + uint16_t last_opcode; + uint64_t last_ip; + uint64_t last_dp; + uint8_t xmm[16][16]; + uint32_t mxcsr; + uint32_t pad2; +} MshvFPU; + typedef enum MshvVmExit { MshvVmExitIgnore =3D 0, MshvVmExitShutdown =3D 1, @@ -90,6 +104,7 @@ typedef enum MshvVmExit { void mshv_init_cpu_logic(void); int mshv_create_vcpu(int vm_fd, uint8_t vp_index, int *cpu_fd); void mshv_remove_vcpu(int vm_fd, int cpu_fd); +int mshv_configure_vcpu(const CPUState *cpu, const MshvFPU *fpu, uint64_t = xcr0); int mshv_get_standard_regs(CPUState *cpu); int mshv_get_special_regs(CPUState *cpu); int mshv_run_vcpu(int vm_fd, CPUState *cpu, hv_message *msg, MshvVmExit *e= xit); diff --git a/target/i386/mshv/mshv-cpu.c b/target/i386/mshv/mshv-cpu.c index 53b6722af4..dddb2da428 100644 --- a/target/i386/mshv/mshv-cpu.c +++ b/target/i386/mshv/mshv-cpu.c @@ -71,6 +71,35 @@ static enum hv_register_name SPECIAL_REGISTER_NAMES[17] = =3D { HV_X64_REGISTER_APIC_BASE, }; =20 +static enum hv_register_name FPU_REGISTER_NAMES[26] =3D { + HV_X64_REGISTER_XMM0, + HV_X64_REGISTER_XMM1, + HV_X64_REGISTER_XMM2, + HV_X64_REGISTER_XMM3, + HV_X64_REGISTER_XMM4, + HV_X64_REGISTER_XMM5, + HV_X64_REGISTER_XMM6, + HV_X64_REGISTER_XMM7, + HV_X64_REGISTER_XMM8, + HV_X64_REGISTER_XMM9, + HV_X64_REGISTER_XMM10, + HV_X64_REGISTER_XMM11, + HV_X64_REGISTER_XMM12, + HV_X64_REGISTER_XMM13, + HV_X64_REGISTER_XMM14, + HV_X64_REGISTER_XMM15, + HV_X64_REGISTER_FP_MMX0, + HV_X64_REGISTER_FP_MMX1, + HV_X64_REGISTER_FP_MMX2, + HV_X64_REGISTER_FP_MMX3, + HV_X64_REGISTER_FP_MMX4, + HV_X64_REGISTER_FP_MMX5, + HV_X64_REGISTER_FP_MMX6, + HV_X64_REGISTER_FP_MMX7, + HV_X64_REGISTER_FP_CONTROL_STATUS, + HV_X64_REGISTER_XMM_CONTROL_STATUS, +}; + int mshv_set_generic_regs(int cpu_fd, hv_register_assoc *assocs, size_t n_= regs) { struct mshv_vp_registers input =3D { @@ -292,8 +321,218 @@ int mshv_load_regs(CPUState *cpu) return 0; } =20 +static inline void populate_hv_segment_reg(SegmentCache *seg, + hv_x64_segment_register *hv_reg) +{ + uint32_t flags =3D seg->flags; + + hv_reg->base =3D seg->base; + hv_reg->limit =3D seg->limit; + hv_reg->selector =3D seg->selector; + hv_reg->segment_type =3D (flags >> DESC_TYPE_SHIFT) & 0xF; + hv_reg->non_system_segment =3D (flags & DESC_S_MASK) !=3D 0; + hv_reg->descriptor_privilege_level =3D (flags >> DESC_DPL_SHIFT) & 0x3; + hv_reg->present =3D (flags & DESC_P_MASK) !=3D 0; + hv_reg->reserved =3D 0; + hv_reg->available =3D (flags & DESC_AVL_MASK) !=3D 0; + hv_reg->_long =3D (flags >> DESC_L_SHIFT) & 0x1; + hv_reg->_default =3D (flags >> DESC_B_SHIFT) & 0x1; + hv_reg->granularity =3D (flags & DESC_G_MASK) !=3D 0; +} + +static inline void populate_hv_table_reg(const struct SegmentCache *seg, + hv_x64_table_register *hv_reg) +{ + memset(hv_reg, 0, sizeof(*hv_reg)); + + hv_reg->base =3D seg->base; + hv_reg->limit =3D seg->limit; +} + +static int set_special_regs(const CPUState *cpu) +{ + X86CPU *x86cpu =3D X86_CPU(cpu); + CPUX86State *env =3D &x86cpu->env; + int cpu_fd =3D mshv_vcpufd(cpu); + struct hv_register_assoc assocs[ARRAY_SIZE(SPECIAL_REGISTER_NAMES)]; + size_t n_regs =3D ARRAY_SIZE(SPECIAL_REGISTER_NAMES); + int ret; + + /* set names */ + for (size_t i =3D 0; i < n_regs; i++) { + assocs[i].name =3D SPECIAL_REGISTER_NAMES[i]; + } + populate_hv_segment_reg(&env->segs[R_CS], &assocs[0].value.segment); + populate_hv_segment_reg(&env->segs[R_DS], &assocs[1].value.segment); + populate_hv_segment_reg(&env->segs[R_ES], &assocs[2].value.segment); + populate_hv_segment_reg(&env->segs[R_FS], &assocs[3].value.segment); + populate_hv_segment_reg(&env->segs[R_GS], &assocs[4].value.segment); + populate_hv_segment_reg(&env->segs[R_SS], &assocs[5].value.segment); + populate_hv_segment_reg(&env->tr, &assocs[6].value.segment); + populate_hv_segment_reg(&env->ldt, &assocs[7].value.segment); + + populate_hv_table_reg(&env->gdt, &assocs[8].value.table); + populate_hv_table_reg(&env->idt, &assocs[9].value.table); + + assocs[10].value.reg64 =3D env->cr[0]; + assocs[11].value.reg64 =3D env->cr[2]; + assocs[12].value.reg64 =3D env->cr[3]; + assocs[13].value.reg64 =3D env->cr[4]; + assocs[14].value.reg64 =3D cpu_get_apic_tpr(x86cpu->apic_state); + assocs[15].value.reg64 =3D env->efer; + assocs[16].value.reg64 =3D cpu_get_apic_base(x86cpu->apic_state); + + ret =3D mshv_set_generic_regs(cpu_fd, assocs, n_regs); + if (ret < 0) { + error_report("failed to set special registers"); + return -1; + } + + return 0; +} + +static int set_fpu(int cpu_fd, const struct MshvFPU *regs) +{ + struct hv_register_assoc assocs[ARRAY_SIZE(FPU_REGISTER_NAMES)]; + union hv_register_value *value; + size_t fp_i; + union hv_x64_fp_control_status_register *ctrl_status; + union hv_x64_xmm_control_status_register *xmm_ctrl_status; + int ret; + size_t n_regs =3D ARRAY_SIZE(FPU_REGISTER_NAMES); + + /* first 16 registers are xmm0-xmm15 */ + for (size_t i =3D 0; i < 16; i++) { + assocs[i].name =3D FPU_REGISTER_NAMES[i]; + value =3D &assocs[i].value; + memcpy(&value->reg128, ®s->xmm[i], 16); + } + + /* next 8 registers are fp_mmx0-fp_mmx7 */ + for (size_t i =3D 16; i < 24; i++) { + assocs[i].name =3D FPU_REGISTER_NAMES[i]; + fp_i =3D (i - 16); + value =3D &assocs[i].value; + memcpy(&value->reg128, ®s->fpr[fp_i], 16); + } + + /* last two registers are fp_control_status and xmm_control_status */ + assocs[24].name =3D FPU_REGISTER_NAMES[24]; + value =3D &assocs[24].value; + ctrl_status =3D &value->fp_control_status; + ctrl_status->fp_control =3D regs->fcw; + ctrl_status->fp_status =3D regs->fsw; + ctrl_status->fp_tag =3D regs->ftwx; + ctrl_status->reserved =3D 0; + ctrl_status->last_fp_op =3D regs->last_opcode; + ctrl_status->last_fp_rip =3D regs->last_ip; + + assocs[25].name =3D FPU_REGISTER_NAMES[25]; + value =3D &assocs[25].value; + xmm_ctrl_status =3D &value->xmm_control_status; + xmm_ctrl_status->xmm_status_control =3D regs->mxcsr; + xmm_ctrl_status->xmm_status_control_mask =3D 0; + xmm_ctrl_status->last_fp_rdp =3D regs->last_dp; + + ret =3D mshv_set_generic_regs(cpu_fd, assocs, n_regs); + if (ret < 0) { + error_report("failed to set fpu registers"); + return -1; + } + + return 0; +} + +static int set_xc_reg(int cpu_fd, uint64_t xcr0) +{ + int ret; + struct hv_register_assoc assoc =3D { + .name =3D HV_X64_REGISTER_XFEM, + .value.reg64 =3D xcr0, + }; + + ret =3D mshv_set_generic_regs(cpu_fd, &assoc, 1); + if (ret < 0) { + error_report("failed to set xcr0"); + return -errno; + } + return 0; +} + +static int set_cpu_state(const CPUState *cpu, const MshvFPU *fpu_regs, + uint64_t xcr0) +{ + int ret; + int cpu_fd =3D mshv_vcpufd(cpu); + + ret =3D set_standard_regs(cpu); + if (ret < 0) { + return ret; + } + ret =3D set_special_regs(cpu); + if (ret < 0) { + return ret; + } + ret =3D set_fpu(cpu_fd, fpu_regs); + if (ret < 0) { + return ret; + } + ret =3D set_xc_reg(cpu_fd, xcr0); + if (ret < 0) { + return ret; + } + return 0; +} + +/* + * TODO: populate topology info: + * + * X86CPU *x86cpu =3D X86_CPU(cpu); + * CPUX86State *env =3D &x86cpu->env; + * X86CPUTopoInfo *topo_info =3D &env->topo_info; + */ +int mshv_configure_vcpu(const CPUState *cpu, const struct MshvFPU *fpu, + uint64_t xcr0) +{ + int ret; + + ret =3D set_cpu_state(cpu, fpu, xcr0); + if (ret < 0) { + error_report("failed to set cpu state"); + return -1; + } + + return 0; +} + +static int put_regs(const CPUState *cpu) +{ + X86CPU *x86cpu =3D X86_CPU(cpu); + CPUX86State *env =3D &x86cpu->env; + MshvFPU fpu =3D {0}; + int ret; + + memset(&fpu, 0, sizeof(fpu)); + + ret =3D mshv_configure_vcpu(cpu, &fpu, env->xcr0); + if (ret < 0) { + error_report("failed to configure vcpu"); + return ret; + } + + return 0; +} + int mshv_arch_put_registers(const CPUState *cpu) { + int ret; + + ret =3D put_regs(cpu); + if (ret < 0) { + error_report("Failed to put registers"); + return -1; + } + error_report("unimplemented"); abort(); } --=20 2.34.1