Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2196588/?format=api
{ "id": 2196588, "url": "http://patchwork.ozlabs.org/api/patches/2196588/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260215112543.4817-4-mohamed@unpredictable.fr/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260215112543.4817-4-mohamed@unpredictable.fr>", "list_archive_url": null, "date": "2026-02-15T11:25:32", "name": "[v10,03/14] hw/intc: arm_gicv3_hvf: save/restore Apple GIC state", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "43aacefbdcc922be072d580635016ef1e7425992", "submitter": { "id": 91318, "url": "http://patchwork.ozlabs.org/api/people/91318/?format=api", "name": "Mohamed Mediouni", "email": "mohamed@unpredictable.fr" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260215112543.4817-4-mohamed@unpredictable.fr/mbox/", "series": [ { "id": 492211, "url": "http://patchwork.ozlabs.org/api/series/492211/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=492211", "date": "2026-02-15T11:25:32", "name": "HVF: Add support for platform vGIC and nested virtualisation", "version": 10, "mbox": "http://patchwork.ozlabs.org/series/492211/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2196588/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2196588/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=unpredictable.fr header.i=@unpredictable.fr\n header.a=rsa-sha256 header.s=sig1 header.b=By/V5sJ8;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)" ], "Received": [ "from lists.gnu.org (lists.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fDNsM1twBz1xpY\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 15 Feb 2026 22:26:47 +1100 (AEDT)", "from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1vraGU-0005I8-1m; Sun, 15 Feb 2026 06:26:14 -0500", "from eggs.gnu.org ([2001:470:142:3::10])\n by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <mohamed@unpredictable.fr>)\n id 1vraGJ-0005Ea-UV\n for qemu-devel@nongnu.org; Sun, 15 Feb 2026 06:26:05 -0500", "from p-west2-cluster1-host2-snip4-10.eps.apple.com ([57.103.68.23]\n helo=outbound.mr.icloud.com)\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <mohamed@unpredictable.fr>)\n id 1vraGG-0007wV-3N\n for qemu-devel@nongnu.org; Sun, 15 Feb 2026 06:26:02 -0500", "from outbound.mr.icloud.com (unknown [127.0.0.2])\n by p00-icloudmta-asmtp-us-west-2a-100-percent-0 (Postfix) with ESMTPS id\n 5C041180029F; Sun, 15 Feb 2026 11:25:55 +0000 (UTC)", "from localhost.localdomain (unknown [17.57.152.38])\n by p00-icloudmta-asmtp-us-west-2a-100-percent-0 (Postfix) with ESMTPSA id\n D88C518000A9; Sun, 15 Feb 2026 11:25:52 +0000 (UTC)" ], "Dkim-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=unpredictable.fr;\n s=sig1; t=1771154756; x=1773746756;\n bh=a4Lipn9YKiDG3oI91EQCLLzeKcwiCgC5chTtbPEFiwg=;\n h=From:To:Subject:Date:Message-ID:MIME-Version:x-icloud-hme;\n b=By/V5sJ8NrzpoBU0um90W1h9eZNYaEjD2g7JBuqcXYZPUeKP2rfZbOg9qXIETxeiFUfz6txBE5gGiaCTRLhN2QSdmKTOYgo0WnN4ICe1MFWKWUMAyGb08p2SlgrcFv6PII3nAanu9KGWNMP+7kjn32MjQbdDJdZFP3szDwjB/1xExZbDH6K6FRoax83mTXTIZQTjoATezHfaYba1WdSqbd+r/pmcItolM1DAktUcNDX1CcPB6tYhkjiG1Alwu5RP4Nd8dzMgSV0Wy7Udv5H/002pWVobRvVkf2T8fMbxR/KnBOxiySrPO1g/Wq3FygReIIpofbXUF6wKxYRgy8TvNg==", "mail-alias-created-date": "1752046281608", "From": "Mohamed Mediouni <mohamed@unpredictable.fr>", "To": "qemu-devel@nongnu.org", "Cc": "qemu-arm@nongnu.org, Peter Maydell <peter.maydell@linaro.org>,\n Mads Ynddal <mads@ynddal.dk>, Roman Bolshakov <rbolshakov@ddn.com>,\n Phil Dennis-Jordan <phil@philjordan.eu>, Alexander Graf <agraf@csgraf.de>,\n Paolo Bonzini <pbonzini@redhat.com>, Cameron Esfahani <dirty@apple.com>,\n Mohamed Mediouni <mohamed@unpredictable.fr>", "Subject": "[PATCH v10 03/14] hw/intc: arm_gicv3_hvf: save/restore Apple GIC\n state", "Date": "Sun, 15 Feb 2026 12:25:32 +0100", "Message-ID": "<20260215112543.4817-4-mohamed@unpredictable.fr>", "X-Mailer": "git-send-email 2.50.1", "In-Reply-To": "<20260215112543.4817-1-mohamed@unpredictable.fr>", "References": "<20260215112543.4817-1-mohamed@unpredictable.fr>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-Proofpoint-ORIG-GUID": "sVPmFnz0CfZeUQW-ny49LK_T8l93UhEv", "X-Authority-Info-Out": "v=2.4 cv=aKv9aL9m c=1 sm=1 tr=0 ts=6991ad44\n cx=c_apl:c_apl_out:c_pps a=9OgfyREA4BUYbbCgc0Y0oA==:117\n a=9OgfyREA4BUYbbCgc0Y0oA==:17 a=HzLeVaNsDn8A:10 a=VkNPw1HP01LnGYTKEx00:22\n a=Mpw57Om8IfrbqaoTuvik:22 a=GgsMoib0sEa3-_RKJdDe:22 a=kNOF8fv34t_zTB2Y_VIA:9", "X-Proofpoint-GUID": "sVPmFnz0CfZeUQW-ny49LK_T8l93UhEv", "X-Proofpoint-Spam-Details-Enc": "AW1haW4tMjYwMjE1MDA5MiBTYWx0ZWRfX7t+s+qI6noXG\n 6cpaz1r2oJGW9NznRtLGXODwE2K912uWqKdgteqBeq78eo2w1EcThtSrxh0fik/en3oQYrUJxn/\n oSlwAqPUkPnAYOLEMJXXOI1BR7x1X84U9+Uz4s9/pWH6GBl61o3DVcP5JwSLzR4UqgEJdjYYyAq\n Jf73zrCHcadx6RD4Dmt5awOLe3Sot9e1K6ma5UhiHViP3lVpDVQiYheHd8NWy8MWMdX6c0oUHnT\n fe02Xddv8ZDaSz9qlweKL1NWI50JsQ0VxTDMqiYih3mcwd+rBtbX5iOrGCPudIUrMqONQPTsoUC\n pYpuR+b0G/O6+/YAND55lzm26VEc8hG5mXhuRUeqxHI52FcML4DlmojpX5e0pA=", "X-Proofpoint-Virus-Version": "vendor=baseguard\n engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.51,FMLib:17.12.100.49\n definitions=2026-02-15_04,2026-02-13_01,2025-10-01_01", "X-Proofpoint-Spam-Details": "rule=notspam policy=default score=0 phishscore=0\n mlxscore=0 adultscore=0 bulkscore=0 spamscore=0 suspectscore=0\n lowpriorityscore=0 malwarescore=0 mlxlogscore=975 clxscore=1030\n classifier=spam authscore=0 adjust=0 reason=mlx scancount=1\n engine=8.22.0-2601150000 definitions=main-2602150092", "X-JNJ": "\n AAAAAAABTApmkbUcupuYfOh/eA0Cj93XyABfAdxQpbcfuAGVdgjLFXlxUE+wOdewcMyOvKV/NuPUOMmSBIVjxIQzNAkadCALE5whaX0KDNHKBEXdKFpKz+7XtXktozFDDwcGzBPLZ9eGyDLh2L1bBqyRtarBdeqVaIvO19Bjy3uo760nnYwFzsKLvI+nJmGVU0viJgFuafR3FiuHwW5GEEZ7gEMBY1LLMRSZFWJcpjfgCd9P0qIOeSPjNzArzQx9aFzTueFmv/L+IUvLAOMBUlSu5BjQWx6+p997CFR3uHvxJlE2oepGJ9NQsBYO9kz6PbO6zZCtCyoqDIerC1mhZNK/dGJ/VrUo3LwYP+6tGjIWdS5UImBZbAUOdfOOQr8DTUUP3uWSrfPUIdH+4WDKppnojUBqmYUU9o8OnQtLUkU78V2Fgf9SU9Z1Xpb/6AdFf0B2u4x+7JwKoW/1wzX+Ro8rdMXxIjnRQJTyQn0YkBvdfIRuDFAe0A1Nf3MGprMft4msgUrlOJH6emULQXRPLaU+v2Mukv11VBYOAesA4ordN8ULGNmrtTERp0q0fn+6ukA/YC11WRdlAHMIDzwN8rXrAt4PWnha0lG77mhp5itLyX4bbEJLRhnnIigQBcxxErzmbXJMc2ESNuNGoJxuB/G+S6yCbz+u/muR75/mRjiLnSnel4mFk7DKk2Fin3Ue/RlIr2VvZcqBhEhX79HShAQOUiCLuKbDTPtxX2RWSlKKeTnKPW/NqJaY6rQSAsN9MeLCv0qgDE3W4aPH", "Received-SPF": "pass client-ip=57.103.68.23;\n envelope-from=mohamed@unpredictable.fr; helo=outbound.mr.icloud.com", "X-Spam_score_int": "-27", "X-Spam_score": "-2.8", "X-Spam_bar": "--", "X-Spam_report": "(-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,\n DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,\n RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001,\n RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001,\n 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": "qemu development <qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>", "List-Archive": "<https://lists.nongnu.org/archive/html/qemu-devel>", "List-Post": "<mailto:qemu-devel@nongnu.org>", "List-Help": "<mailto:qemu-devel-request@nongnu.org?subject=help>", "List-Subscribe": "<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org" }, "content": "On HVF, some of the GIC state is in an opaque Apple-provided structure.\n\nSave/restore that state to be able to save/restore VMs that use the hardware GIC.\n\nSigned-off-by: Mohamed Mediouni <mohamed@unpredictable.fr>\n---\n hw/intc/arm_gicv3_common.c | 1 +\n hw/intc/arm_gicv3_hvf.c | 91 ++++++++++++++++++++++++++++--\n hw/intc/arm_gicv3_hvf_stub.c | 25 ++++++++\n hw/intc/meson.build | 1 +\n include/hw/intc/arm_gicv3_common.h | 3 +\n 5 files changed, 117 insertions(+), 4 deletions(-)\n create mode 100644 hw/intc/arm_gicv3_hvf_stub.c", "diff": "diff --git a/hw/intc/arm_gicv3_common.c b/hw/intc/arm_gicv3_common.c\nindex 9200671c7a..9c3fb2f4bf 100644\n--- a/hw/intc/arm_gicv3_common.c\n+++ b/hw/intc/arm_gicv3_common.c\n@@ -305,6 +305,7 @@ static const VMStateDescription vmstate_gicv3 = {\n .subsections = (const VMStateDescription * const []) {\n &vmstate_gicv3_gicd_no_migration_shift_bug,\n &vmstate_gicv3_gicd_nmi,\n+ &vmstate_gicv3_hvf,\n NULL\n }\n };\ndiff --git a/hw/intc/arm_gicv3_hvf.c b/hw/intc/arm_gicv3_hvf.c\nindex d6a46b7d53..9e4d974cd1 100644\n--- a/hw/intc/arm_gicv3_hvf.c\n+++ b/hw/intc/arm_gicv3_hvf.c\n@@ -13,6 +13,7 @@\n #include \"qemu/error-report.h\"\n #include \"qemu/module.h\"\n #include \"system/runstate.h\"\n+#include \"migration/vmstate.h\"\n #include \"system/hvf.h\"\n #include \"system/hvf_int.h\"\n #include \"hvf_arm.h\"\n@@ -30,8 +31,13 @@ struct HVFARMGICv3Class {\n \n typedef struct HVFARMGICv3Class HVFARMGICv3Class;\n \n-/* This is reusing the GICv3State typedef from ARM_GICV3_ITS_COMMON */\n-DECLARE_OBJ_CHECKERS(GICv3State, HVFARMGICv3Class,\n+typedef struct HVFGICv3State {\n+ GICv3State gicv3_state;\n+ uint32_t size;\n+ void *state;\n+} HVFGICv3State;\n+\n+DECLARE_OBJ_CHECKERS(HVFGICv3State, HVFARMGICv3Class,\n HVF_GICV3, TYPE_HVF_GICV3);\n \n /*\n@@ -656,7 +662,7 @@ static const ARMCPRegInfo gicv3_cpuif_reginfo[] = {\n static void hvf_gicv3_realize(DeviceState *dev, Error **errp)\n {\n ERRP_GUARD();\n- GICv3State *s = HVF_GICV3(dev);\n+ GICv3State *s = (GICv3State *)HVF_GICV3(dev);\n HVFARMGICv3Class *kgc = HVF_GICV3_GET_CLASS(s);\n int i;\n \n@@ -703,6 +709,83 @@ static void hvf_gicv3_realize(DeviceState *dev, Error **errp)\n }\n }\n \n+/*\n+ * HVF doesn't have a way to save the RDIST pending tables\n+ * to guest memory, only to an opaque data structure.\n+ */\n+static bool gicv3_is_hvf(void *opaque)\n+{\n+ return hvf_enabled() && hvf_irqchip_in_kernel();\n+}\n+\n+static int hvf_gic_opaque_state_save(void* opaque)\n+{\n+ HVFGICv3State* gic = opaque;\n+ hv_gic_state_t gic_state;\n+ hv_return_t err;\n+ size_t size;\n+\n+ gic_state = hv_gic_state_create();\n+ if (gic_state == NULL) {\n+ error_report(\"hvf: vgic: failed to create hv_gic_state_create.\");\n+ return 1;\n+ }\n+ err = hv_gic_state_get_size(gic_state, &size);\n+ gic->size = size;\n+ if (err != HV_SUCCESS) {\n+ error_report(\"hvf: vgic: failed to get GIC state size.\");\n+ return 1;\n+ }\n+ gic->state = g_malloc(gic->size);\n+ err = hv_gic_state_get_data(gic_state, gic->state);\n+ if (err != HV_SUCCESS) {\n+ error_report(\"hvf: vgic: failed to get GIC state.\");\n+ return 1;\n+ }\n+ return 0;\n+}\n+\n+static int hvf_gic_opaque_state_free(void* opaque)\n+{\n+ HVFGICv3State* gic = opaque;\n+ free(gic->state);\n+ return 0;\n+}\n+\n+static int hvf_gic_opaque_state_restore(void* opaque, int version_id)\n+{\n+ HVFGICv3State* gic = opaque;\n+ hv_return_t err;\n+ if (!gic->size) {\n+ return 0;\n+ }\n+ err = hv_gic_set_state(gic->state, gic->size);\n+ if (err != HV_SUCCESS) {\n+ error_report(\"hvf: vgic: failed to restore GIC state.\");\n+ return 1;\n+ }\n+ return 0;\n+}\n+\n+const VMStateDescription vmstate_gicv3_hvf = {\n+ .name = \"arm_gicv3/hvf_gic_state\",\n+ .version_id = 1,\n+ .minimum_version_id = 1,\n+ .needed = gicv3_is_hvf,\n+ .pre_save = hvf_gic_opaque_state_save,\n+ .post_save = hvf_gic_opaque_state_free,\n+ .post_load = hvf_gic_opaque_state_restore,\n+ .version_id = 1,\n+ .minimum_version_id = 1,\n+ .fields = (const VMStateField[]) {\n+ VMSTATE_UINT32(size, HVFGICv3State),\n+ VMSTATE_VBUFFER_ALLOC_UINT32(state,\n+ HVFGICv3State, 0, 0,\n+ size),\n+ VMSTATE_END_OF_LIST()\n+ },\n+};\n+\n static void hvf_gicv3_class_init(ObjectClass *klass, const void *data)\n {\n DeviceClass *dc = DEVICE_CLASS(klass);\n@@ -722,7 +805,7 @@ static void hvf_gicv3_class_init(ObjectClass *klass, const void *data)\n static const TypeInfo hvf_arm_gicv3_info = {\n .name = TYPE_HVF_GICV3,\n .parent = TYPE_ARM_GICV3_COMMON,\n- .instance_size = sizeof(GICv3State),\n+ .instance_size = sizeof(HVFGICv3State),\n .class_init = hvf_gicv3_class_init,\n .class_size = sizeof(HVFARMGICv3Class),\n };\ndiff --git a/hw/intc/arm_gicv3_hvf_stub.c b/hw/intc/arm_gicv3_hvf_stub.c\nnew file mode 100644\nindex 0000000000..a587332c7c\n--- /dev/null\n+++ b/hw/intc/arm_gicv3_hvf_stub.c\n@@ -0,0 +1,25 @@\n+/* SPDX-License-Identifier: GPL-2.0-or-later */\n+/*\n+ * ARM Generic Interrupt Controller using HVF platform support stub\n+ *\n+ * Copyright (c) 2026 Mohamed Mediouni\n+ *\n+ */\n+#include \"qemu/osdep.h\"\n+#include \"hw/intc/arm_gicv3_common.h\"\n+#include \"migration/vmstate.h\"\n+#include \"qemu/typedefs.h\"\n+\n+static bool needed_never(void *opaque)\n+{\n+ return false;\n+}\n+\n+const VMStateDescription vmstate_gicv3_hvf = {\n+ .name = \"arm_gicv3/hvf_gic_state\",\n+ .version_id = 1,\n+ .minimum_version_id = 1,\n+ .needed = needed_never,\n+ .version_id = 1,\n+ .minimum_version_id = 1,\n+};\ndiff --git a/hw/intc/meson.build b/hw/intc/meson.build\nindex b7baf8a0f6..c6de2d9d00 100644\n--- a/hw/intc/meson.build\n+++ b/hw/intc/meson.build\n@@ -43,6 +43,7 @@ arm_common_ss.add(when: 'CONFIG_ARM_GICV3', if_true: files('arm_gicv3_cpuif.c'))\n specific_ss.add(when: 'CONFIG_ARM_GIC_KVM', if_true: files('arm_gic_kvm.c'))\n specific_ss.add(when: ['CONFIG_WHPX', 'TARGET_AARCH64'], if_true: files('arm_gicv3_whpx.c'))\n specific_ss.add(when: ['CONFIG_HVF', 'CONFIG_ARM_GICV3'], if_true: files('arm_gicv3_hvf.c'))\n+specific_ss.add(when: ['CONFIG_HVF', 'CONFIG_ARM_GICV3'], if_false: files('arm_gicv3_hvf_stub.c'))\n specific_ss.add(when: ['CONFIG_ARM_GIC_KVM', 'TARGET_AARCH64'], if_true: files('arm_gicv3_kvm.c', 'arm_gicv3_its_kvm.c'))\n arm_common_ss.add(when: 'CONFIG_ARM_V7M', if_true: files('armv7m_nvic.c'))\n specific_ss.add(when: 'CONFIG_GRLIB', if_true: files('grlib_irqmp.c'))\ndiff --git a/include/hw/intc/arm_gicv3_common.h b/include/hw/intc/arm_gicv3_common.h\nindex 9adcab0a0c..dd49fe1b9d 100644\n--- a/include/hw/intc/arm_gicv3_common.h\n+++ b/include/hw/intc/arm_gicv3_common.h\n@@ -339,4 +339,7 @@ void gicv3_init_irqs_and_mmio(GICv3State *s, qemu_irq_handler handler,\n */\n const char *gicv3_class_name(void);\n \n+/* HVF vGIC-specific state: stubbed out on a build with HVF disabled. s*/\n+extern const VMStateDescription vmstate_gicv3_hvf;\n+\n #endif\n", "prefixes": [ "v10", "03/14" ] }