Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.1/patches/2232513/?format=api
{ "id": 2232513, "url": "http://patchwork.ozlabs.org/api/1.1/patches/2232513/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260504-feat-mte4-v5-9-232a648e63c6@gmail.com/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/1.1/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": "" }, "msgid": "<20260504-feat-mte4-v5-9-232a648e63c6@gmail.com>", "date": "2026-05-04T15:50:42", "name": "[v5,09/15] target/arm: load on canonical tag loads ext bits", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "664b430f8411684a23cfbd6ac5cb786bfc71e6c5", "submitter": { "id": 91863, "url": "http://patchwork.ozlabs.org/api/1.1/people/91863/?format=api", "name": "Gabriel Brookman", "email": "brookmangabriel@gmail.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260504-feat-mte4-v5-9-232a648e63c6@gmail.com/mbox/", "series": [ { "id": 502688, "url": "http://patchwork.ozlabs.org/api/1.1/series/502688/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=502688", "date": "2026-05-04T15:50:33", "name": "target/arm: add support for MTE4", "version": 5, "mbox": "http://patchwork.ozlabs.org/series/502688/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2232513/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2232513/checks/", "tags": {}, "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=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=ZvOQttqB;\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=lists1p.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)" ], "Received": [ "from lists1p.gnu.org (lists1p.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 4g8R4v3vLKz1yJ0\n\tfor <incoming@patchwork.ozlabs.org>; Tue, 05 May 2026 01:53:19 +1000 (AEST)", "from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists1p.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1wJvZv-0000zV-7S; Mon, 04 May 2026 11:51:27 -0400", "from eggs.gnu.org ([2001:470:142:3::10])\n by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <brookmangabriel@gmail.com>)\n id 1wJvZt-0000z4-NL\n for qemu-devel@nongnu.org; Mon, 04 May 2026 11:51:25 -0400", "from mail-qt1-x830.google.com ([2607:f8b0:4864:20::830])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)\n (Exim 4.90_1) (envelope-from <brookmangabriel@gmail.com>)\n id 1wJvZr-0006E2-OQ\n for qemu-devel@nongnu.org; Mon, 04 May 2026 11:51:25 -0400", "by mail-qt1-x830.google.com with SMTP id\n d75a77b69052e-50d7c12e48eso35475781cf.1\n for <qemu-devel@nongnu.org>; Mon, 04 May 2026 08:51:23 -0700 (PDT)", "from [192.168.1.164] ([2600:1009:a021:c665:5296:905f:3e4a:eb90])\n by smtp.gmail.com with ESMTPSA id\n d75a77b69052e-51040931552sm99599011cf.12.2026.05.04.08.51.20\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Mon, 04 May 2026 08:51:21 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20251104; t=1777909883; x=1778514683; darn=nongnu.org;\n h=cc:to:in-reply-to:references:message-id:content-transfer-encoding\n :mime-version:subject:date:from:from:to:cc:subject:date:message-id\n :reply-to; bh=MG1iwnko83uMuVMCUIM9KDLK/vF8JVyEE/TaS9GVUps=;\n b=ZvOQttqBpltWABpGJZWbBcp4yxDvrGNoCAsseCAiVLSYumqyZstWFo3Eais5hzZZcc\n WYz6EnMZSBiSmga62pZRSqDZP/CgW0xEJg6+lWKCE6Gn87AZJ9hmPwGJSxUkZndXuPjX\n ReXwbsKDLBOfZo31p9nSNXePR+UjyoP1MZJIGJh+7u0eh/ZqwHhNVCWy/UJz9r0cn2Wu\n 4EjXWY/H/SBIywjrO5+NvAa/o5uBuZTnr2L0avc9ltYuBoLLS1CbfQEo9QQe6HI3RKFj\n behE0h9vwHeYF9sw213zyquBM0tgAOrRoPQDu9wRxtEiJLRdej32akO74K5fzg3u0U1p\n Hd0g==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1777909883; x=1778514683;\n h=cc:to:in-reply-to:references:message-id:content-transfer-encoding\n :mime-version:subject:date:from:x-gm-gg:x-gm-message-state:from:to\n :cc:subject:date:message-id:reply-to;\n bh=MG1iwnko83uMuVMCUIM9KDLK/vF8JVyEE/TaS9GVUps=;\n b=JXK2mX6yoHWR8/bSr/uoVJ16eKFkNFPPZESka1U2whb/lYIFUDuNydQOAbKADjCYrV\n Krbe8BJ+rbcbz17RFP26P7xSolGDWbpBV1HZxXxz5X9Hgxu7mn3HNKZl5IEG7tk4j0yt\n tNxB6jYrtPD0BXIBFnmxDkznLPSeJHVRuKD02FurUuZyLEoPW9/DkHR+X7Qh2cFcdLVU\n pLt/YxIXzF6j6rCzcpm8pQWTCgi/T3ZUlX1h8H3w/WgWpw/38yT16jmk0EnphCbU1CYL\n jo5EWUPp11wFQZyZ8oPEqjbK7zwqcHAvwSE4Lu88/9KSJXycz27s/53SXU+MfXuyU2Ca\n d9uA==", "X-Gm-Message-State": "AOJu0YzKowe3VeoIGrIIBhZGvGPkNKeQcyDyT6jaGuS7b9Pfjp1c2rnh\n fqayXj7RSegoU8/WGU9eQjpmI6ixEQDk+2Gt8b9V5czZ1njZjP26x6CqkG8jzRLe", "X-Gm-Gg": "AeBDiest5Pps7+HF/cNs6uNwiGlEn8S7oCnSuZbM+usT2oZNm3yjUR57BOp8/m4osCO\n hL64Eo+aRYLCPCuqxsdNu6EY8P6WMrgZat5/Ox3os83NNVALFMipZmwjb3fl6q52zb5Q4/YWCfC\n 1xAgptJtytA3eulcN/rgG+SMRZkK+6IO8SCXK3L+N+fPH+pzfktoV7Nk4LY17o6i6nwvEASQMgN\n i7PkQ+VmuSAprjCRaOuB435F3Z1Eg/INfNE1Oh/ODq82Iux7hlpD7DPA8U1plUjaSsFS+PKDMzf\n mbVfvK3WKOSSKs0JEqLXIhyqrbl1hqoRRV7bPejfo04SCY78r/Y+QVcDbUTpi/EGCQNgvAGua6X\n NtgrXeKifCA3LVNh0EEwsZbBRq5++tuN3Y+0MnRRJgykA79Owsfyha4WaOmx2UCx5JLAm+CXPsL\n f7yAas7/IYX0QTS2W2rj3/1imjUg9t4AOy1nYAFTBJW3Hu/DitBqE=", "X-Received": "by 2002:a05:622a:1f8c:b0:50f:b7b3:2ec4 with SMTP id\n d75a77b69052e-5130534b3bbmr803141cf.25.1777909881921;\n Mon, 04 May 2026 08:51:21 -0700 (PDT)", "From": "Gabriel Brookman <brookmangabriel@gmail.com>", "Date": "Mon, 04 May 2026 11:50:42 -0400", "Subject": "[PATCH v5 09/15] target/arm: load on canonical tag loads ext bits", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "Message-Id": "<20260504-feat-mte4-v5-9-232a648e63c6@gmail.com>", "References": "<20260504-feat-mte4-v5-0-232a648e63c6@gmail.com>", "In-Reply-To": "<20260504-feat-mte4-v5-0-232a648e63c6@gmail.com>", "To": "qemu-devel@nongnu.org", "Cc": "Peter Maydell <peter.maydell@linaro.org>,\n Gustavo Romero <gustavo.romero@linaro.org>,\n Richard Henderson <richard.henderson@linaro.org>, qemu-arm@nongnu.org,\n Laurent Vivier <laurent@vivier.eu>,\n Gabriel Brookman <brookmangabriel@gmail.com>, Helge Deller <deller@gmx.de>,\n Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>,\n Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>", "X-Mailer": "b4 0.15.2", "X-Developer-Signature": "v=1; a=ed25519-sha256; t=1777909867; l=5681;\n i=brookmangabriel@gmail.com; s=20251009; h=from:subject:message-id;\n bh=Ws2Tqe13fBijY+IHknD2CyebZv1in1ArUctH8Hio2sA=;\n b=oz8eYjE5Vhv9ZTARi40/Hp4hw9BTP367GpMsUgRMbeoVgKXesDUDsrV4q+eHUAoXCN10U4XKi\n TjEmXWkKoGZCOfw5lW6i1VHoDaYhtfJGVF+zbu/NgSZ3rybx9vBpuSW", "X-Developer-Key": "i=brookmangabriel@gmail.com; a=ed25519;\n pk=m9TtPDal6WzoHNnQiHHKf8dTrv3DUCPUUTujuo8vNrw=", "Received-SPF": "pass client-ip=2607:f8b0:4864:20::830;\n envelope-from=brookmangabriel@gmail.com; helo=mail-qt1-x830.google.com", "X-Spam_score_int": "-20", "X-Spam_score": "-2.1", "X-Spam_bar": "--", "X-Spam_report": "(-2.1 / 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, FREEMAIL_FROM=0.001,\n RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,\n 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": "Loading tags from canonically tagged regions should use the canonical\ntags (extension bits), not allocation tags. See AArch64_MemTagRead().\n\nSigned-off-by: Gabriel Brookman <brookmangabriel@gmail.com>\n---\n target/arm/tcg/helper-a64-defs.h | 4 ++--\n target/arm/tcg/mte_helper.c | 42 +++++++++++++++++++++++++++++++++++++---\n target/arm/tcg/translate-a64.c | 4 ++--\n 3 files changed, 43 insertions(+), 7 deletions(-)", "diff": "diff --git a/target/arm/tcg/helper-a64-defs.h b/target/arm/tcg/helper-a64-defs.h\nindex 3c3c5dddb7..c08c8c3991 100644\n--- a/target/arm/tcg/helper-a64-defs.h\n+++ b/target/arm/tcg/helper-a64-defs.h\n@@ -102,14 +102,14 @@ DEF_HELPER_FLAGS_3(mte_check, TCG_CALL_NO_WG, i64, env, i32, i64)\n DEF_HELPER_FLAGS_3(mte_check_zva, TCG_CALL_NO_WG, i64, env, i32, i64)\n DEF_HELPER_FLAGS_3(irg, TCG_CALL_NO_RWG, i64, env, i64, i64)\n DEF_HELPER_FLAGS_4(addsubg, TCG_CALL_NO_RWG_SE, i64, env, i64, s32, i32)\n-DEF_HELPER_FLAGS_3(ldg, TCG_CALL_NO_WG, i64, env, i64, i64)\n+DEF_HELPER_FLAGS_4(ldg, TCG_CALL_NO_WG, i64, env, i64, i64, i32)\n DEF_HELPER_FLAGS_3(stg, TCG_CALL_NO_WG, void, env, i64, i64)\n DEF_HELPER_FLAGS_3(stg_parallel, TCG_CALL_NO_WG, void, env, i64, i64)\n DEF_HELPER_FLAGS_2(stg_stub, TCG_CALL_NO_WG, void, env, i64)\n DEF_HELPER_FLAGS_3(st2g, TCG_CALL_NO_WG, void, env, i64, i64)\n DEF_HELPER_FLAGS_3(st2g_parallel, TCG_CALL_NO_WG, void, env, i64, i64)\n DEF_HELPER_FLAGS_2(st2g_stub, TCG_CALL_NO_WG, void, env, i64)\n-DEF_HELPER_FLAGS_2(ldgm, TCG_CALL_NO_WG, i64, env, i64)\n+DEF_HELPER_FLAGS_3(ldgm, TCG_CALL_NO_WG, i64, env, i64, i32)\n DEF_HELPER_FLAGS_3(stgm, TCG_CALL_NO_WG, void, env, i64, i64)\n DEF_HELPER_FLAGS_3(stzgm_tags, TCG_CALL_NO_WG, void, env, i64, i64)\n \ndiff --git a/target/arm/tcg/mte_helper.c b/target/arm/tcg/mte_helper.c\nindex d35e3ef04d..8ed31ca2d8 100644\n--- a/target/arm/tcg/mte_helper.c\n+++ b/target/arm/tcg/mte_helper.c\n@@ -304,7 +304,7 @@ int load_tag1(uint64_t ptr, uint8_t *mem)\n return extract32(*mem, ofs, 4);\n }\n \n-uint64_t HELPER(ldg)(CPUARMState *env, uint64_t ptr, uint64_t xt)\n+uint64_t HELPER(ldg)(CPUARMState *env, uint64_t ptr, uint64_t xt, uint32_t mtx)\n {\n int mmu_idx = arm_env_mmu_index(env);\n uint8_t *mem;\n@@ -317,6 +317,11 @@ uint64_t HELPER(ldg)(CPUARMState *env, uint64_t ptr, uint64_t xt)\n /* Load if page supports tags. */\n if (mem) {\n rtag = load_tag1(ptr, mem);\n+ } else {\n+ uint64_t bit55 = extract64(ptr, 55, 1);\n+ if (mtx) {\n+ rtag = 0xF * bit55;\n+ }\n }\n \n return address_with_allocation_tag(xt, rtag);\n@@ -458,17 +463,19 @@ void HELPER(st2g_stub)(CPUARMState *env, uint64_t ptr)\n }\n }\n \n-uint64_t HELPER(ldgm)(CPUARMState *env, uint64_t ptr)\n+uint64_t HELPER(ldgm)(CPUARMState *env, uint64_t ptr, uint32_t mtx)\n {\n int mmu_idx = arm_env_mmu_index(env);\n uintptr_t ra = GETPC();\n int gm_bs = env_archcpu(env)->gm_blocksize;\n int gm_bs_bytes = 4 << gm_bs;\n void *tag_mem;\n- uint64_t ret;\n+ uint64_t ret, canonical_tag_val;\n int shift;\n+ bool bit55;\n \n ptr = QEMU_ALIGN_DOWN(ptr, gm_bs_bytes);\n+ bit55 = extract64(ptr, 55, 1);\n \n /* Trap if accessing an invalid page. */\n tag_mem = allocation_tag_mem(env, mmu_idx, ptr, MMU_DATA_LOAD,\n@@ -476,6 +483,35 @@ uint64_t HELPER(ldgm)(CPUARMState *env, uint64_t ptr)\n \n /* The tag is squashed to zero if the page does not support tags. */\n if (!tag_mem) {\n+ /* Load canonical value if mtx is set (untagged memory region) */\n+ if (mtx) {\n+ canonical_tag_val = -(uint64_t)bit55;\n+ switch (gm_bs) {\n+ case 3:\n+ /* 32 bytes -> 2 tags -> 8 result bits */\n+ ret = (uint8_t)canonical_tag_val;\n+ break;\n+ case 4:\n+ /* 64 bytes -> 4 tags -> 16 result bits */\n+ ret = (uint16_t)canonical_tag_val;\n+ break;\n+ case 5:\n+ /* 128 bytes -> 8 tags -> 32 result bits */\n+ ret = (uint32_t)canonical_tag_val;\n+ break;\n+ case 6:\n+ /* 256 bytes -> 16 tags -> 64 result bits */\n+ return canonical_tag_val;\n+ default:\n+ /*\n+ * CPU configured with unsupported/invalid gm blocksize.\n+ * This is detected early in arm_cpu_realizefn.\n+ */\n+ g_assert_not_reached();\n+ }\n+ shift = extract64(ptr, LOG2_TAG_GRANULE, 4) * 4;\n+ return ret << shift;\n+ }\n return 0;\n }\n \ndiff --git a/target/arm/tcg/translate-a64.c b/target/arm/tcg/translate-a64.c\nindex cd86178d56..5fbc54de4b 100644\n--- a/target/arm/tcg/translate-a64.c\n+++ b/target/arm/tcg/translate-a64.c\n@@ -4711,7 +4711,7 @@ static bool trans_LDGM(DisasContext *s, arg_ldst_tag *a)\n tcg_rt = cpu_reg(s, a->rt);\n \n if (s->ata[0]) {\n- gen_helper_ldgm(tcg_rt, tcg_env, addr);\n+ gen_helper_ldgm(tcg_rt, tcg_env, addr, tcg_constant_i32(s->mtx));\n } else {\n MMUAccessType acc = MMU_DATA_LOAD;\n int size = 4 << s->gm_blocksize;\n@@ -4746,7 +4746,7 @@ static bool trans_LDG(DisasContext *s, arg_ldst_tag *a)\n tcg_gen_andi_i64(addr, addr, -TAG_GRANULE);\n tcg_rt = cpu_reg(s, a->rt);\n if (s->ata[0]) {\n- gen_helper_ldg(tcg_rt, tcg_env, addr, tcg_rt);\n+ gen_helper_ldg(tcg_rt, tcg_env, addr, tcg_rt, tcg_constant_i32(s->mtx));\n } else {\n /*\n * Tag access disabled: we must check for aborts on the load\n", "prefixes": [ "v5", "09/15" ] }