On Sat, Feb 21, 2026 at 06:16:40PM +0800, Tao Tang wrote: > This is a non-functional preparation step that adds storage for > resolved security state in SMMUTLBEntry. >
I am not sure about this, when I added stage-2 I re-used the same TLB instance as it might be used for nesting, and we can have end-to-end cached entires. For secure state, I think it’s cleaner to instantiate a new TLB, as both states can never mix, and in this case we can re-use the same functions without adding the secure bit in the TLB. Thanks, Mostafa > Together with the earlier commits that added NSCFG handling and > PTE NS/NSTable helpers, the plumbing is complete and we can now > refactor the PTW flow to handle Secure state. > > Signed-off-by: Tao Tang <[email protected]> > --- > include/hw/arm/smmu-common.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/hw/arm/smmu-common.h b/include/hw/arm/smmu-common.h > index bd88e599c77..b0a02e12fe6 100644 > --- a/include/hw/arm/smmu-common.h > +++ b/include/hw/arm/smmu-common.h > @@ -91,6 +91,7 @@ typedef struct SMMUTLBEntry { > uint8_t level; > uint8_t granule; > IOMMUAccessFlags parent_perm; > + SMMUSecSID sec_sid; > } SMMUTLBEntry; > > /* Stage-2 configuration. */ > -- > 2.34.1 > >
