Re: [PATCH] lib: utils: fdt_domain: add support for root domain region inheritance
Anup Patel <[email protected]>
| Newsgroups | org.infradead.lists.opensbi |
|---|---|
| Message-ID | <CAAhSdy0QLe3jjtdAZDigyd9Lizk=AegU3r-pSstQsdFj_MmB4A@mail.gmail.com> |
On Tue, May 12, 2026 at 1:53 PM Yu-Chien Peter Lin <[email protected]> wrote: > > Hi Anup, > > Thanks for suggetion, I'd like to double-check before > sending v2 patch. > > On Sun, May 10, 2026 at 11:34:49AM +0530, Anup Patel wrote: > > On Fri, Mar 27, 2026 at 10:04 AM Yu-Chien Peter Lin > > <[email protected]> wrote: > > > > > > Add the "root-regions" property in domain device-tree nodes to > > > allow domains to inherit all regions from the root domain. This > > > simplifies configuration for domains that need access to most > > > root domain regions with only minor exclusions or additions. > > > > Overall, this is a good approach but I suggest renaming this property > > to "root-regions-inheritance" which takes the following possible values: > > > > 1) "all" : Inherit all regions from the root domain > > 2) "su-only": Inherit regions accessible to S-mode or U-mode from the > > root domain > > 1. Do you mean: > a) any root regions that have any SU permission bits set regardless of the > M‑mode permission bits, or > b) only regions that satisfy SBI_DOMAIN_MEMREGION_SU_ONLY_ACCESS(flags)? Actually, none of these. Apologies for the confusion. I meant the current policy of copying select regions from root domain to non-root domain which is only copy regions which have no SU permissions set. This means the following possible values: 1) "all" : Inherit all regions from the root domain 2) "m-only": Inherit regions of root domain not having any SU permissions set (current policy) which also covers firmware regions. 3) "none": Inherit no regions from the root domain > 2. For firmware regions marked with SBI_DOMAIN_MEMREGION_FW, should those always > be inherited, even when "root-regions-inheritance" is "none" or "su-only"? Yes, FW regions should always be inherited because these are for OpenSBI itself and OpenSBI is always resident in background. Regards, Anup -- opensbi mailing list [email protected] http://lists.infradead.org/mailman/listinfo/opensbi