From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B60EAC79F99 for ; Mon, 7 Sep 2026 14:51:01 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists1p.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1x3ag6-0000BY-Bf; Mon, 07 Sep 2026 10:50:34 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1x3afp-00005Y-2K for qemu-arm@nongnu.org; Mon, 07 Sep 2026 10:50:19 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1x3afh-00016b-OW for qemu-arm@nongnu.org; Mon, 07 Sep 2026 10:50:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1788792605; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=CyZinKkuErv6IRIgwBItnKcoa51Rv4dpxls5PY7GuNA=; b=OXZ2hB+/yb8YN4aBHZtLL/0lk69qYJhSySns8DFr9v9KmCwpbg094WitPYuFpoX08TrqGT hnyNoB8eXlqtgLuwi/fxFpL8J/Rx+U9S/6tIBUTGS+bUVWsBc1WEjMgyGMeqd04sk/iiyC ojpEgr0xVR/4YElyI5ynR5ZZYyRgq2M= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-532-mUZ4yEdDO9O1i3-NkLSEjg-1; Mon, 07 Sep 2026 10:50:03 -0400 X-MC-Unique: mUZ4yEdDO9O1i3-NkLSEjg-1 X-Mimecast-MFC-AGG-ID: mUZ4yEdDO9O1i3-NkLSEjg_1788792602 Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-48589603501so2123558f8f.3 for ; Mon, 07 Sep 2026 07:50:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788792602; x=1789397402; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:content-language:subject:reply-to:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=CyZinKkuErv6IRIgwBItnKcoa51Rv4dpxls5PY7GuNA=; b=kxoIDUEMoz3sz6/jyf17Iq1HjrJYza/SSUAc1NPW2nh01M1joyHJGSazhaOB5+lLXN CnwyiTO3psyYyGn97ZY6ASgM9fTz4qIH8trcf2dPmjs6H83BYqmC9md+JuN4by67J02X j93JC429zeq7lVN22+V1L5mHS6hLAm13iVkRjfsPgF/MwwZlz0vMSTJKuNS+F+panaBf lOaH9o3mJBEol81xipoNU36uW3EBZpwJWKP4pk8Ts1EurVorNYQ4vTVPZJwuyOIYZow7 VuwTrH9Qlc9nx1gtUh66B5NB+AWVHDCiUmnIPK1DItNf/xdNbMX2vTq118n/Tv8ZBwUi 3A8A== X-Forwarded-Encrypted: i=1; AKwUvBykzseSb4DOqt0zdrHjW/HsX7S7VuV3ldltcPCPd56MIGntRvQ/jOVPhh0LZQLDkJI4NJ7KIaqAPw==@nongnu.org X-Gm-Message-State: AFuF++nu+pNCIGkLh17b764MkZgnrAYY90n7lcIQBuqEUJh/ZS9ENVM2 EgCVn6cLT4xGX1AQIE8fz+QY/TdH2fySeCibeGtcjcbOYvqJ5R38Bfd8+AdGh79pt4/IpEVazmN 1dwOzoRC4uzrMe+l8ijCprXC+iBDLHtOSV0GqYRK0kiSScFBXXAuGyg== X-Gm-Gg: AYBFou2vmIde0Znhl/QsFSO0AiJyZgqdMqYKCwSu/P+wclzyW08VhrJXGtxgqz6GJXI Vrd/nzmeRZINV5hPQUW8OkWUyXuz5rICdRL9ccpRUCFNbmdCxLuZTAjcH6cCo7XGgOYjjdhUrY8 8aCrntrz8wHZqE2fZB7s+eT5lDqo4OGKerDTfvDsg9WJjTb+Md3Nxh7LTHk4BC/iWgU+biNCoMS b2alMyovygM3vO/Lll4blMZ7eY/T5i62r7uzmckV91bfWw5pBgQafXgXl/7et++Lszp/x0v6S6x KyME2FjXLPSzk2f0aHhRZTyz+g9fpJtQWSD1d6xOf9O/P3Jnt4Hx+atA25CLU1YIk53toVvLGiR 7MzX1AgW0eqghOp5Uz96C2etoPH0UX+scIuiXbFT9KLvDDIU0 X-Received: by 2002:a5d:64e5:0:b0:485:8c16:a351 with SMTP id ffacd0b85a97d-4858c16a7a7mr23415492f8f.41.1788792601891; Mon, 07 Sep 2026 07:50:01 -0700 (PDT) X-Received: by 2002:a5d:64e5:0:b0:485:8c16:a351 with SMTP id ffacd0b85a97d-4858c16a7a7mr23415435f8f.41.1788792601264; Mon, 07 Sep 2026 07:50:01 -0700 (PDT) Received: from ?IPV6:2a01:e0a:f0e:9070:527b:9dff:feef:3874? ([2a01:e0a:f0e:9070:527b:9dff:feef:3874]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-485885be01bsm28499462f8f.31.2026.09.07.07.49.59 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 07 Sep 2026 07:50:00 -0700 (PDT) Message-ID: <2f430034-626e-40a1-89fb-266ce037bb1f@redhat.com> Date: Mon, 7 Sep 2026 16:49:58 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [RFC v5 12/28] hw/arm/smmuv3: Tag IOTLB cache keys with SEC_SID To: Mostafa Saleh , Tao Tang Cc: Peter Maydell , qemu-devel@nongnu.org, qemu-arm@nongnu.org, Chen Baozi , Pierrick Bouvier , =?UTF-8?Q?Philippe_Mathieu-Daud=C3=A9?= , Chao Liu , Jim MacArthur References: <20260813161515.2788900-1-tangtao1634@phytium.com.cn> <20260813162512.2807281-2-tangtao1634@phytium.com.cn> From: Eric Auger In-Reply-To: X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: YuAwZKL3cnP4uy04IS5j8pr0LSeyigbK1I6iothAkYQ_1788792602 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Received-SPF: pass client-ip=170.10.133.124; envelope-from=eric.auger@redhat.com; helo=us-smtp-delivery-124.mimecast.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, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-arm@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: eric.auger@redhat.com Errors-To: qemu-arm-bounces+qemu-arm=archiver.kernel.org@nongnu.org Sender: qemu-arm-bounces+qemu-arm=archiver.kernel.org@nongnu.org On 9/1/26 4:11 PM, Mostafa Saleh wrote: > On Fri, Aug 14, 2026 at 12:25:09AM +0800, Tao Tang wrote: >> To prevent aliasing between translations controlled through the Secure and >> Non-secure programming interfaces, the IOTLB lookup key must incorporate >> SEC_SID. >> >> This commit: >> - expands SMMUIOTLBKey with SEC_SID field for cache key differentiation > I still feel that it's better to have a separate IOTLB for the secure > world, as it should never mix with the non-secure one; as I commented > on the last version: > https://lore.kernel.org/qemu-devel/aaGuGuevX8HFqx0x@google.com/ > > Then all the functions can be re-used and it is just a matter > of passing the right instance. > > No strong opinion though, this approach should work also, so it is up > to Eric. I don't have a strong opinion either. I am just curious about what the implementation will become once we add further support for StreamWorld. Will we be able to keep separate IOTLBs or will it make more sense to have a unified IOTLB? Thanks Eric > > Thanks, > Mostafa > >> - extends SMMUIOTLBPageInvInfo with SEC_SID for invalidation filtering >> - updates all IOTLB invalidation helpers (smmu_iotlb_inv_iova, >> smmu_iotlb_inv_ipa, smmu_iotlb_inv_asid_vmid, smmu_iotlb_inv_vmid, >> smmu_iotlb_inv_vmid_s1) to accept and filter by SEC_SID >> - plumbs SEC_SID through smmuv3_range_inval for TLB invalidation >> - enhances trace events to include SEC_SID for better debugging >> >> This ensures that IOTLB entries decoded through the Secure and Non-secure >> programming interfaces are distinct, preventing cache aliasing across >> SEC_SID namespaces. >> >> Signed-off-by: Tao Tang >> --- >> hw/arm/smmu-common.c | 110 +++++++++++++++++++++++------------ >> hw/arm/smmu-internal.h | 2 + >> hw/arm/smmuv3.c | 47 ++++++++++----- >> hw/arm/trace-events | 20 +++---- >> include/hw/arm/smmu-common.h | 32 +++++++--- >> 5 files changed, 140 insertions(+), 71 deletions(-) >> >> diff --git a/hw/arm/smmu-common.c b/hw/arm/smmu-common.c >> index 317cfafded2..3d4b6b3a287 100644 >> --- a/hw/arm/smmu-common.c >> +++ b/hw/arm/smmu-common.c >> @@ -95,7 +95,7 @@ static guint smmu_iotlb_key_hash(gconstpointer v) >> >> /* Jenkins hash */ >> a = b = c = JHASH_INITVAL + sizeof(*key); >> - a += key->asid + key->vmid + key->level + key->tg; >> + a += key->asid + key->vmid + key->level + key->tg + key->sec_sid; >> b += extract64(key->iova, 0, 32); >> c += extract64(key->iova, 32, 32); >> >> @@ -111,14 +111,15 @@ static gboolean smmu_iotlb_key_equal(gconstpointer v1, gconstpointer v2) >> >> return (k1->asid == k2->asid) && (k1->iova == k2->iova) && >> (k1->level == k2->level) && (k1->tg == k2->tg) && >> - (k1->vmid == k2->vmid); >> + (k1->vmid == k2->vmid) && (k1->sec_sid == k2->sec_sid); >> } >> >> SMMUIOTLBKey smmu_get_iotlb_key(int asid, int vmid, uint64_t iova, >> - uint8_t tg, uint8_t level) >> + uint8_t tg, uint8_t level, >> + SMMUSecSID sec_sid) >> { >> SMMUIOTLBKey key = {.asid = asid, .vmid = vmid, .iova = iova, >> - .tg = tg, .level = level}; >> + .tg = tg, .level = level, .sec_sid = sec_sid}; >> >> return key; >> } >> @@ -126,7 +127,8 @@ SMMUIOTLBKey smmu_get_iotlb_key(int asid, int vmid, uint64_t iova, >> static SMMUTLBEntry *smmu_iotlb_lookup_all_levels(SMMUState *bs, >> SMMUTransCfg *cfg, >> SMMUTransTableInfo *tt, >> - hwaddr iova) >> + hwaddr iova, >> + SMMUSecSID sec_sid) >> { >> uint8_t tg = (tt->granule_sz - 10) / 2; >> uint8_t inputsize = 64 - tt->tsz; >> @@ -140,7 +142,7 @@ static SMMUTLBEntry *smmu_iotlb_lookup_all_levels(SMMUState *bs, >> SMMUIOTLBKey key; >> >> key = smmu_get_iotlb_key(cfg->asid, cfg->s2cfg.vmid, >> - iova & ~mask, tg, level); >> + iova & ~mask, tg, level, sec_sid); >> entry = g_hash_table_lookup(bs->iotlb, &key); >> if (entry) { >> break; >> @@ -156,6 +158,7 @@ static SMMUTLBEntry *smmu_iotlb_lookup_all_levels(SMMUState *bs, >> * @cfg: Configuration of the translation >> * @tt: Translation table info (granule and tsz) >> * @iova: IOVA address to lookup >> + * @sec_sid: StreamID Security state >> * >> * returns a valid entry on success, otherwise NULL. >> * In case of nested translation, tt can be updated to include >> @@ -163,11 +166,12 @@ static SMMUTLBEntry *smmu_iotlb_lookup_all_levels(SMMUState *bs, >> * the IOVA granule. >> */ >> SMMUTLBEntry *smmu_iotlb_lookup(SMMUState *bs, SMMUTransCfg *cfg, >> - SMMUTransTableInfo *tt, hwaddr iova) >> + SMMUTransTableInfo *tt, hwaddr iova, >> + SMMUSecSID sec_sid) >> { >> SMMUTLBEntry *entry = NULL; >> >> - entry = smmu_iotlb_lookup_all_levels(bs, cfg, tt, iova); >> + entry = smmu_iotlb_lookup_all_levels(bs, cfg, tt, iova, sec_sid); >> /* >> * For nested translation also try the s2 granule, as the TLB will insert >> * it if the size of s2 tlb entry was smaller. >> @@ -175,18 +179,20 @@ SMMUTLBEntry *smmu_iotlb_lookup(SMMUState *bs, SMMUTransCfg *cfg, >> if (!entry && (cfg->stage == SMMU_NESTED) && >> (cfg->s2cfg.granule_sz != tt->granule_sz)) { >> tt->granule_sz = cfg->s2cfg.granule_sz; >> - entry = smmu_iotlb_lookup_all_levels(bs, cfg, tt, iova); >> + entry = smmu_iotlb_lookup_all_levels(bs, cfg, tt, iova, sec_sid); >> } >> >> if (entry) { >> cfg->iotlb_hits++; >> - trace_smmu_iotlb_lookup_hit(cfg->asid, cfg->s2cfg.vmid, iova, >> + trace_smmu_iotlb_lookup_hit(sec_sid, cfg->asid, >> + cfg->s2cfg.vmid, iova, >> cfg->iotlb_hits, cfg->iotlb_misses, >> 100 * cfg->iotlb_hits / >> (cfg->iotlb_hits + cfg->iotlb_misses)); >> } else { >> cfg->iotlb_misses++; >> - trace_smmu_iotlb_lookup_miss(cfg->asid, cfg->s2cfg.vmid, iova, >> + trace_smmu_iotlb_lookup_miss(sec_sid, cfg->asid, >> + cfg->s2cfg.vmid, iova, >> cfg->iotlb_hits, cfg->iotlb_misses, >> 100 * cfg->iotlb_hits / >> (cfg->iotlb_hits + cfg->iotlb_misses)); >> @@ -194,7 +200,8 @@ SMMUTLBEntry *smmu_iotlb_lookup(SMMUState *bs, SMMUTransCfg *cfg, >> return entry; >> } >> >> -void smmu_iotlb_insert(SMMUState *bs, SMMUTransCfg *cfg, SMMUTLBEntry *new) >> +void smmu_iotlb_insert(SMMUState *bs, SMMUTransCfg *cfg, SMMUTLBEntry *new, >> + SMMUSecSID sec_sid) >> { >> SMMUIOTLBKey *key = g_new0(SMMUIOTLBKey, 1); >> uint8_t tg = (new->granule - 10) / 2; >> @@ -204,9 +211,9 @@ void smmu_iotlb_insert(SMMUState *bs, SMMUTransCfg *cfg, SMMUTLBEntry *new) >> } >> >> *key = smmu_get_iotlb_key(cfg->asid, cfg->s2cfg.vmid, new->entry.iova, >> - tg, new->level); >> - trace_smmu_iotlb_insert(cfg->asid, cfg->s2cfg.vmid, new->entry.iova, >> - tg, new->level); >> + tg, new->level, sec_sid); >> + trace_smmu_iotlb_insert(sec_sid, cfg->asid, cfg->s2cfg.vmid, >> + new->entry.iova, tg, new->level); >> g_hash_table_insert(bs->iotlb, key, new); >> } >> >> @@ -223,26 +230,29 @@ static gboolean smmu_hash_remove_by_asid_vmid(gpointer key, gpointer value, >> SMMUIOTLBKey *iotlb_key = (SMMUIOTLBKey *)key; >> >> return (SMMU_IOTLB_ASID(*iotlb_key) == info->asid) && >> - (SMMU_IOTLB_VMID(*iotlb_key) == info->vmid); >> + (SMMU_IOTLB_VMID(*iotlb_key) == info->vmid) && >> + (SMMU_IOTLB_SEC_SID(*iotlb_key) == info->sec_sid); >> } >> >> static gboolean smmu_hash_remove_by_vmid(gpointer key, gpointer value, >> gpointer user_data) >> { >> - int vmid = *(int *)user_data; >> + SMMUIOTLBPageInvInfo *info = (SMMUIOTLBPageInvInfo *)user_data; >> SMMUIOTLBKey *iotlb_key = (SMMUIOTLBKey *)key; >> >> - return SMMU_IOTLB_VMID(*iotlb_key) == vmid; >> + return (SMMU_IOTLB_VMID(*iotlb_key) == info->vmid) && >> + (SMMU_IOTLB_SEC_SID(*iotlb_key) == info->sec_sid); >> } >> >> static gboolean smmu_hash_remove_by_vmid_s1(gpointer key, gpointer value, >> gpointer user_data) >> { >> - int vmid = *(int *)user_data; >> + SMMUIOTLBPageInvInfo *info = (SMMUIOTLBPageInvInfo *)user_data; >> SMMUIOTLBKey *iotlb_key = (SMMUIOTLBKey *)key; >> >> - return (SMMU_IOTLB_VMID(*iotlb_key) == vmid) && >> - (SMMU_IOTLB_ASID(*iotlb_key) >= 0); >> + return (SMMU_IOTLB_VMID(*iotlb_key) == info->vmid) && >> + (SMMU_IOTLB_ASID(*iotlb_key) >= 0) && >> + (SMMU_IOTLB_SEC_SID(*iotlb_key) == info->sec_sid); >> } >> >> static gboolean smmu_hash_remove_by_asid_vmid_iova(gpointer key, gpointer value, >> @@ -259,6 +269,9 @@ static gboolean smmu_hash_remove_by_asid_vmid_iova(gpointer key, gpointer value, >> if (info->vmid >= 0 && info->vmid != SMMU_IOTLB_VMID(iotlb_key)) { >> return false; >> } >> + if (info->sec_sid != SMMU_IOTLB_SEC_SID(iotlb_key)) { >> + return false; >> + } >> return ((info->iova & ~entry->addr_mask) == entry->iova) || >> ((entry->iova & ~info->mask) == info->iova); >> } >> @@ -278,6 +291,9 @@ static gboolean smmu_hash_remove_by_vmid_ipa(gpointer key, gpointer value, >> if (info->vmid != SMMU_IOTLB_VMID(iotlb_key)) { >> return false; >> } >> + if (info->sec_sid != SMMU_IOTLB_SEC_SID(iotlb_key)) { >> + return false; >> + } >> return ((info->iova & ~entry->addr_mask) == entry->iova) || >> ((entry->iova & ~info->mask) == info->iova); >> } >> @@ -323,13 +339,17 @@ void smmu_configs_inv_sdev(SMMUState *s, SMMUDevice *sdev) >> } >> >> void smmu_iotlb_inv_iova(SMMUState *s, int asid, int vmid, dma_addr_t iova, >> - uint8_t tg, uint64_t num_pages, uint8_t ttl) >> + uint8_t tg, uint64_t num_pages, uint8_t ttl, >> + SMMUSecSID sec_sid) >> { >> /* if tg is not set we use 4KB range invalidation */ >> uint8_t granule = tg ? tg * 2 + 10 : 12; >> >> + trace_smmu_iotlb_inv_iova(sec_sid, asid, iova); >> + >> if (ttl && (num_pages == 1) && (asid >= 0)) { >> - SMMUIOTLBKey key = smmu_get_iotlb_key(asid, vmid, iova, tg, ttl); >> + SMMUIOTLBKey key = smmu_get_iotlb_key(asid, vmid, iova, >> + tg, ttl, sec_sid); >> >> if (g_hash_table_remove(s->iotlb, &key)) { >> return; >> @@ -343,7 +363,8 @@ void smmu_iotlb_inv_iova(SMMUState *s, int asid, int vmid, dma_addr_t iova, >> SMMUIOTLBPageInvInfo info = { >> .asid = asid, .iova = iova, >> .vmid = vmid, >> - .mask = (num_pages * 1 << granule) - 1}; >> + .mask = (num_pages * 1 << granule) - 1, >> + .sec_sid = sec_sid}; >> >> g_hash_table_foreach_remove(s->iotlb, >> smmu_hash_remove_by_asid_vmid_iova, >> @@ -355,13 +376,15 @@ void smmu_iotlb_inv_iova(SMMUState *s, int asid, int vmid, dma_addr_t iova, >> * in Stage-1 invalidation ASID = -1, means don't care. >> */ >> void smmu_iotlb_inv_ipa(SMMUState *s, int vmid, dma_addr_t ipa, uint8_t tg, >> - uint64_t num_pages, uint8_t ttl) >> + uint64_t num_pages, uint8_t ttl, >> + SMMUSecSID sec_sid) >> { >> uint8_t granule = tg ? tg * 2 + 10 : 12; >> int asid = -1; >> >> if (ttl && (num_pages == 1)) { >> - SMMUIOTLBKey key = smmu_get_iotlb_key(asid, vmid, ipa, tg, ttl); >> + SMMUIOTLBKey key = smmu_get_iotlb_key(asid, vmid, ipa, >> + tg, ttl, sec_sid); >> >> if (g_hash_table_remove(s->iotlb, &key)) { >> return; >> @@ -371,34 +394,47 @@ void smmu_iotlb_inv_ipa(SMMUState *s, int vmid, dma_addr_t ipa, uint8_t tg, >> SMMUIOTLBPageInvInfo info = { >> .iova = ipa, >> .vmid = vmid, >> - .mask = (num_pages << granule) - 1}; >> + .mask = (num_pages << granule) - 1, >> + .sec_sid = sec_sid}; >> >> g_hash_table_foreach_remove(s->iotlb, >> smmu_hash_remove_by_vmid_ipa, >> &info); >> } >> >> -void smmu_iotlb_inv_asid_vmid(SMMUState *s, int asid, int vmid) >> +void smmu_iotlb_inv_asid_vmid(SMMUState *s, int asid, int vmid, >> + SMMUSecSID sec_sid) >> { >> SMMUIOTLBPageInvInfo info = { >> .asid = asid, >> .vmid = vmid, >> + .sec_sid = sec_sid, >> }; >> >> - trace_smmu_iotlb_inv_asid_vmid(asid, vmid); >> + trace_smmu_iotlb_inv_asid_vmid(sec_sid, asid, vmid); >> g_hash_table_foreach_remove(s->iotlb, smmu_hash_remove_by_asid_vmid, &info); >> } >> >> -void smmu_iotlb_inv_vmid(SMMUState *s, int vmid) >> +void smmu_iotlb_inv_vmid(SMMUState *s, int vmid, SMMUSecSID sec_sid) >> { >> - trace_smmu_iotlb_inv_vmid(vmid); >> - g_hash_table_foreach_remove(s->iotlb, smmu_hash_remove_by_vmid, &vmid); >> + SMMUIOTLBPageInvInfo info = { >> + .vmid = vmid, >> + .sec_sid = sec_sid, >> + }; >> + >> + trace_smmu_iotlb_inv_vmid(sec_sid, vmid); >> + g_hash_table_foreach_remove(s->iotlb, smmu_hash_remove_by_vmid, &info); >> } >> >> -void smmu_iotlb_inv_vmid_s1(SMMUState *s, int vmid) >> +void smmu_iotlb_inv_vmid_s1(SMMUState *s, int vmid, SMMUSecSID sec_sid) >> { >> - trace_smmu_iotlb_inv_vmid_s1(vmid); >> - g_hash_table_foreach_remove(s->iotlb, smmu_hash_remove_by_vmid_s1, &vmid); >> + SMMUIOTLBPageInvInfo info = { >> + .vmid = vmid, >> + .sec_sid = sec_sid, >> + }; >> + >> + trace_smmu_iotlb_inv_vmid_s1(sec_sid, vmid); >> + g_hash_table_foreach_remove(s->iotlb, smmu_hash_remove_by_vmid_s1, &info); >> } >> >> /* VMSAv8-64 Translation */ >> @@ -919,7 +955,7 @@ SMMUTLBEntry *smmu_translate(SMMUState *bs, SMMUTransCfg *cfg, dma_addr_t addr, >> tt_combined.tsz = tt->tsz; >> } >> >> - cached_entry = smmu_iotlb_lookup(bs, cfg, &tt_combined, addr); >> + cached_entry = smmu_iotlb_lookup(bs, cfg, &tt_combined, addr, sec_sid); >> if (cached_entry) { >> if ((flag & IOMMU_WO) && !(cached_entry->entry.perm & >> cached_entry->parent_perm & IOMMU_WO)) { >> @@ -938,7 +974,7 @@ SMMUTLBEntry *smmu_translate(SMMUState *bs, SMMUTransCfg *cfg, dma_addr_t addr, >> g_free(cached_entry); >> return NULL; >> } >> - smmu_iotlb_insert(bs, cfg, cached_entry); >> + smmu_iotlb_insert(bs, cfg, cached_entry, sec_sid); >> return cached_entry; >> } >> >> diff --git a/hw/arm/smmu-internal.h b/hw/arm/smmu-internal.h >> index 004abd58bca..ce68d4b5813 100644 >> --- a/hw/arm/smmu-internal.h >> +++ b/hw/arm/smmu-internal.h >> @@ -144,12 +144,14 @@ static inline int pgd_concat_idx(int start_level, int granule_sz, >> >> #define SMMU_IOTLB_ASID(key) ((key).asid) >> #define SMMU_IOTLB_VMID(key) ((key).vmid) >> +#define SMMU_IOTLB_SEC_SID(key) ((key).sec_sid) >> >> typedef struct SMMUIOTLBPageInvInfo { >> int asid; >> int vmid; >> uint64_t iova; >> uint64_t mask; >> + SMMUSecSID sec_sid; >> } SMMUIOTLBPageInvInfo; >> >> #endif >> diff --git a/hw/arm/smmuv3.c b/hw/arm/smmuv3.c >> index cc5d3ab696c..087112ba4b6 100644 >> --- a/hw/arm/smmuv3.c >> +++ b/hw/arm/smmuv3.c >> @@ -634,6 +634,17 @@ static int decode_ste(SMMUv3State *s, SMMUTransCfg *cfg, >> goto bad_ste; >> } >> >> + /* >> + * Keep the SEC_SID-to-StreamWorld approximation used by the IOTLB key >> + * one-to-one until the other Secure translation regimes are modeled. >> + */ >> + if (sec_sid == SMMU_SEC_SID_S && STE_CFG_S1_TRANSLATE(config) && >> + STE_STRW(ste) != 0) { >> + qemu_log_mask(LOG_UNIMP, >> + "SMMUv3 Secure StreamWorld is not implemented\n"); >> + goto bad_ste; >> + } >> + >> if (STAGE2_SUPPORTED(s)) { >> /* VMID is considered even if s2 is disabled. */ >> cfg->s2cfg.vmid = STE_S2VMID(ste); >> @@ -1317,7 +1328,8 @@ static void smmuv3_inv_notifiers_iova(SMMUState *s, int asid, int vmid, >> } >> } >> >> -static void smmuv3_range_inval(SMMUState *s, Cmd *cmd, SMMUStage stage) >> +static void smmuv3_range_inval(SMMUState *s, Cmd *cmd, SMMUStage stage, >> + SMMUSecSID sec_sid) >> { >> dma_addr_t end, addr = CMD_ADDR(cmd); >> uint8_t type = CMD_TYPE(cmd); >> @@ -1342,12 +1354,13 @@ static void smmuv3_range_inval(SMMUState *s, Cmd *cmd, SMMUStage stage) >> } >> >> if (!tg) { >> - trace_smmuv3_range_inval(vmid, asid, addr, tg, 1, ttl, leaf, stage); >> + trace_smmuv3_range_inval(sec_sid, vmid, asid, addr, >> + tg, 1, ttl, leaf, stage); >> smmuv3_inv_notifiers_iova(s, asid, vmid, addr, tg, 1, stage); >> if (stage == SMMU_STAGE_1) { >> - smmu_iotlb_inv_iova(s, asid, vmid, addr, tg, 1, ttl); >> + smmu_iotlb_inv_iova(s, asid, vmid, addr, tg, 1, ttl, sec_sid); >> } else { >> - smmu_iotlb_inv_ipa(s, vmid, addr, tg, 1, ttl); >> + smmu_iotlb_inv_ipa(s, vmid, addr, tg, 1, ttl, sec_sid); >> } >> return; >> } >> @@ -1364,13 +1377,15 @@ static void smmuv3_range_inval(SMMUState *s, Cmd *cmd, SMMUStage stage) >> uint64_t mask = dma_aligned_pow2_mask(addr, end, 64); >> >> num_pages = (mask + 1) >> granule; >> - trace_smmuv3_range_inval(vmid, asid, addr, tg, num_pages, >> - ttl, leaf, stage); >> - smmuv3_inv_notifiers_iova(s, asid, vmid, addr, tg, num_pages, stage); >> + trace_smmuv3_range_inval(sec_sid, vmid, asid, addr, tg, >> + num_pages, ttl, leaf, stage); >> + smmuv3_inv_notifiers_iova(s, asid, vmid, addr, tg, >> + num_pages, stage); >> if (stage == SMMU_STAGE_1) { >> - smmu_iotlb_inv_iova(s, asid, vmid, addr, tg, num_pages, ttl); >> + smmu_iotlb_inv_iova(s, asid, vmid, addr, tg, >> + num_pages, ttl, sec_sid); >> } else { >> - smmu_iotlb_inv_ipa(s, vmid, addr, tg, num_pages, ttl); >> + smmu_iotlb_inv_ipa(s, vmid, addr, tg, num_pages, ttl, sec_sid); >> } >> addr += mask + 1; >> } >> @@ -1521,9 +1536,9 @@ static int smmuv3_cmdq_consume(SMMUv3State *s, Error **errp) >> vmid = CMD_VMID(&cmd); >> } >> >> - trace_smmuv3_cmdq_tlbi_nh_asid(asid); >> + trace_smmuv3_cmdq_tlbi_nh_asid(sec_sid, asid); >> smmu_inv_notifiers_all(&s->smmu_state); >> - smmu_iotlb_inv_asid_vmid(bs, asid, vmid); >> + smmu_iotlb_inv_asid_vmid(bs, asid, vmid, sec_sid); >> if (!smmuv3_accel_issue_inv_cmd(s, &cmd, NULL, errp)) { >> cmd_error = SMMU_CERROR_ILL; >> break; >> @@ -1545,8 +1560,8 @@ static int smmuv3_cmdq_consume(SMMUv3State *s, Error **errp) >> */ >> if (STAGE2_SUPPORTED(s)) { >> vmid = CMD_VMID(&cmd); >> - trace_smmuv3_cmdq_tlbi_nh(vmid); >> - smmu_iotlb_inv_vmid_s1(bs, vmid); >> + trace_smmuv3_cmdq_tlbi_nh(sec_sid, vmid); >> + smmu_iotlb_inv_vmid_s1(bs, vmid, sec_sid); >> break; >> } >> QEMU_FALLTHROUGH; >> @@ -1566,7 +1581,7 @@ static int smmuv3_cmdq_consume(SMMUv3State *s, Error **errp) >> cmd_error = SMMU_CERROR_ILL; >> break; >> } >> - smmuv3_range_inval(bs, &cmd, SMMU_STAGE_1); >> + smmuv3_range_inval(bs, &cmd, SMMU_STAGE_1, SMMU_SEC_SID_NS); >> if (!smmuv3_accel_issue_inv_cmd(s, &cmd, NULL, errp)) { >> cmd_error = SMMU_CERROR_ILL; >> break; >> @@ -1583,7 +1598,7 @@ static int smmuv3_cmdq_consume(SMMUv3State *s, Error **errp) >> >> trace_smmuv3_cmdq_tlbi_s12_vmid(vmid); >> smmu_inv_notifiers_all(&s->smmu_state); >> - smmu_iotlb_inv_vmid(bs, vmid); >> + smmu_iotlb_inv_vmid(bs, vmid, SMMU_SEC_SID_NS); >> break; >> } >> case SMMU_CMD_TLBI_S2_IPA: >> @@ -1595,7 +1610,7 @@ static int smmuv3_cmdq_consume(SMMUv3State *s, Error **errp) >> * As currently only either s1 or s2 are supported >> * we can reuse same function for s2. >> */ >> - smmuv3_range_inval(bs, &cmd, SMMU_STAGE_2); >> + smmuv3_range_inval(bs, &cmd, SMMU_STAGE_2, SMMU_SEC_SID_NS); >> break; >> case SMMU_CMD_ATC_INV: >> { >> diff --git a/hw/arm/trace-events b/hw/arm/trace-events >> index a166b79c8ef..6a8716e8041 100644 >> --- a/hw/arm/trace-events >> +++ b/hw/arm/trace-events >> @@ -19,16 +19,16 @@ smmu_ptw_page_pte(int stage, int level, uint64_t iova, uint64_t baseaddr, uint6 >> smmu_ptw_block_pte(int stage, int level, uint64_t baseaddr, uint64_t pteaddr, uint64_t pte, uint64_t iova, uint64_t gpa, int bsize_mb) "stage=%d level=%d base@=0x%"PRIx64" pte@=0x%"PRIx64" pte=0x%"PRIx64" iova=0x%"PRIx64" block address = 0x%"PRIx64" block size = %d MiB" >> smmu_get_pte(uint64_t baseaddr, int index, uint64_t pteaddr, uint64_t pte) "baseaddr=0x%"PRIx64" index=0x%x, pteaddr=0x%"PRIx64", pte=0x%"PRIx64 >> smmu_iotlb_inv_all(void) "IOTLB invalidate all" >> -smmu_iotlb_inv_asid_vmid(int asid, int vmid) "IOTLB invalidate asid=%d vmid=%d" >> -smmu_iotlb_inv_vmid(int vmid) "IOTLB invalidate vmid=%d" >> -smmu_iotlb_inv_vmid_s1(int vmid) "IOTLB invalidate vmid=%d" >> -smmu_iotlb_inv_iova(int asid, uint64_t addr) "IOTLB invalidate asid=%d addr=0x%"PRIx64 >> +smmu_iotlb_inv_asid_vmid(int sec_sid, int asid, int vmid) "IOTLB invalidate sec_sid=%d asid=%d vmid=%d" >> +smmu_iotlb_inv_vmid(int sec_sid, int vmid) "IOTLB invalidate sec_sid=%d vmid=%d" >> +smmu_iotlb_inv_vmid_s1(int sec_sid, int vmid) "IOTLB invalidate S1 sec_sid=%d vmid=%d" >> +smmu_iotlb_inv_iova(int sec_sid, int asid, uint64_t addr) "IOTLB invalidate sec_sid=%d asid=%d addr=0x%"PRIx64 >> smmu_configs_inv_sid_range(uint32_t start, uint32_t end) "Config cache INV SID range from 0x%x to 0x%x" >> smmu_config_cache_inv(uint32_t sid) "Config cache INV for sid=0x%x" >> smmu_inv_notifiers_mr(const char *name) "iommu mr=%s" >> -smmu_iotlb_lookup_hit(int asid, int vmid, uint64_t addr, uint32_t hit, uint32_t miss, uint32_t p) "IOTLB cache HIT asid=%d vmid=%d addr=0x%"PRIx64" hit=%d miss=%d hit rate=%d" >> -smmu_iotlb_lookup_miss(int asid, int vmid, uint64_t addr, uint32_t hit, uint32_t miss, uint32_t p) "IOTLB cache MISS asid=%d vmid=%d addr=0x%"PRIx64" hit=%d miss=%d hit rate=%d" >> -smmu_iotlb_insert(int asid, int vmid, uint64_t addr, uint8_t tg, uint8_t level) "IOTLB ++ asid=%d vmid=%d addr=0x%"PRIx64" tg=%d level=%d" >> +smmu_iotlb_lookup_hit(int sec_sid, int asid, int vmid, uint64_t addr, uint32_t hit, uint32_t miss, uint32_t p) "IOTLB cache HIT sec_sid=%d asid=%d vmid=%d addr=0x%"PRIx64" hit=%d miss=%d hit rate=%d" >> +smmu_iotlb_lookup_miss(int sec_sid, int asid, int vmid, uint64_t addr, uint32_t hit, uint32_t miss, uint32_t p) "IOTLB cache MISS sec_sid=%d asid=%d vmid=%d addr=0x%"PRIx64" hit=%d miss=%d hit rate=%d" >> +smmu_iotlb_insert(int sec_sid, int asid, int vmid, uint64_t addr, uint8_t tg, uint8_t level) "IOTLB ++ sec_sid=%d asid=%d vmid=%d addr=0x%"PRIx64" tg=%d level=%d" >> >> # smmuv3.c >> smmuv3_read_mmio(uint64_t addr, uint64_t val, unsigned size, uint32_t r) "addr: 0x%"PRIx64" val:0x%"PRIx64" size: 0x%x(%d)" >> @@ -57,10 +57,10 @@ smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" >> smmuv3_cmdq_cfgi_cd(uint32_t sid) "sid=0x%x" >> smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache HIT for sid=0x%x (hits=%d, misses=%d, hit rate=%d)" >> smmuv3_config_cache_miss(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache MISS for sid=0x%x (hits=%d, misses=%d, hit rate=%d)" >> -smmuv3_range_inval(int vmid, int asid, uint64_t addr, uint8_t tg, uint64_t num_pages, uint8_t ttl, bool leaf, int stage) "vmid=%d asid=%d addr=0x%"PRIx64" tg=%d num_pages=0x%"PRIx64" ttl=%d leaf=%d stage=%d" >> -smmuv3_cmdq_tlbi_nh(int vmid) "vmid=%d" >> +smmuv3_range_inval(int sec_sid, int vmid, int asid, uint64_t addr, uint8_t tg, uint64_t num_pages, uint8_t ttl, bool leaf, int stage) "sec_sid=%d vmid=%d asid=%d addr=0x%"PRIx64" tg=%d num_pages=0x%"PRIx64" ttl=%d leaf=%d stage=%d" >> +smmuv3_cmdq_tlbi_nh(int sec_sid, int vmid) "sec_sid=%d vmid=%d" >> smmuv3_cmdq_tlbi_nsnh(void) "" >> -smmuv3_cmdq_tlbi_nh_asid(int asid) "asid=%d" >> +smmuv3_cmdq_tlbi_nh_asid(int sec_sid, int asid) "sec_sid=%d asid=%d" >> smmuv3_cmdq_tlbi_s12_vmid(int vmid) "vmid=%d" >> smmuv3_notify_flag_add(const char *iommu) "ADD SMMUNotifier node for iommu mr=%s" >> smmuv3_notify_flag_del(const char *iommu) "DEL SMMUNotifier node for iommu mr=%s" >> diff --git a/include/hw/arm/smmu-common.h b/include/hw/arm/smmu-common.h >> index 0c5718ea684..8e971c28093 100644 >> --- a/include/hw/arm/smmu-common.h >> +++ b/include/hw/arm/smmu-common.h >> @@ -152,6 +152,17 @@ typedef struct SMMUIOTLBKey { >> int vmid; >> uint8_t tg; >> uint8_t level; >> + /* >> + * We currently model one StreamWorld per SEC_SID, giving the approximate >> + * mapping: >> + * >> + * SMMU_SEC_SID_NS -> NS-EL1 >> + * SMMU_SEC_SID_S -> Secure >> + * >> + * SEC_SID is not architecturally equivalent to StreamWorld. Extend this >> + * key when additional translation regimes are implemented. >> + */ >> + SMMUSecSID sec_sid; >> } SMMUIOTLBKey; >> >> typedef struct SMMUConfigKey { >> @@ -250,19 +261,24 @@ SMMUDevice *smmu_find_sdev(SMMUState *s, uint32_t sid); >> #define SMMU_IOTLB_MAX_SIZE 256 >> >> SMMUTLBEntry *smmu_iotlb_lookup(SMMUState *bs, SMMUTransCfg *cfg, >> - SMMUTransTableInfo *tt, hwaddr iova); >> -void smmu_iotlb_insert(SMMUState *bs, SMMUTransCfg *cfg, SMMUTLBEntry *entry); >> + SMMUTransTableInfo *tt, hwaddr iova, >> + SMMUSecSID sec_sid); >> +void smmu_iotlb_insert(SMMUState *bs, SMMUTransCfg *cfg, SMMUTLBEntry *entry, >> + SMMUSecSID sec_sid); >> SMMUIOTLBKey smmu_get_iotlb_key(int asid, int vmid, uint64_t iova, >> - uint8_t tg, uint8_t level); >> + uint8_t tg, uint8_t level, SMMUSecSID sec_sid); >> SMMUConfigKey smmu_get_config_key(SMMUDevice *sdev, SMMUSecSID sec_sid); >> void smmu_iotlb_inv_all(SMMUState *s); >> -void smmu_iotlb_inv_asid_vmid(SMMUState *s, int asid, int vmid); >> -void smmu_iotlb_inv_vmid(SMMUState *s, int vmid); >> -void smmu_iotlb_inv_vmid_s1(SMMUState *s, int vmid); >> +void smmu_iotlb_inv_asid_vmid(SMMUState *s, int asid, int vmid, >> + SMMUSecSID sec_sid); >> +void smmu_iotlb_inv_vmid(SMMUState *s, int vmid, SMMUSecSID sec_sid); >> +void smmu_iotlb_inv_vmid_s1(SMMUState *s, int vmid, SMMUSecSID sec_sid); >> void smmu_iotlb_inv_iova(SMMUState *s, int asid, int vmid, dma_addr_t iova, >> - uint8_t tg, uint64_t num_pages, uint8_t ttl); >> + uint8_t tg, uint64_t num_pages, uint8_t ttl, >> + SMMUSecSID sec_sid); >> void smmu_iotlb_inv_ipa(SMMUState *s, int vmid, dma_addr_t ipa, uint8_t tg, >> - uint64_t num_pages, uint8_t ttl); >> + uint64_t num_pages, uint8_t ttl, >> + SMMUSecSID sec_sid); >> void smmu_configs_inv_sid_range(SMMUState *s, SMMUSIDRange sid_range); >> void smmu_configs_inv_sdev(SMMUState *s, SMMUDevice *sdev); >> /* Unmap the range of all the notifiers registered to any IOMMU mr */ >> -- >> 2.34.1 >>