Skip to content

Split region permissions into M-mode and SU-mode

Himanshu Chauhan opensbi 2 versions – New

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest9 patches 0 repliesNew
  2. v19 patches 0 repliesSuperseded

Changes in v2 from the cover letter

1. Fixed the grammatical mistakes in the commit log.
2. Removed the white space new line from sbi_domain.h
3. Added enforce permission bit[6] details to documentation

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9include: sbi: Fine grain the permissions for M and SU modesinclude/sbi/sbi_domain.h+35 -50 replies
2/9lib: sbi: Use finer permission semantics for address validationlib/sbi/sbi_domain.c+16 -70 replies
3/9lib: sbi: Add permissions for the firmware start till endlib/sbi/sbi_domain.c+2 -10 replies
4/9lib: sbi: Use finer permission sematics to decide on PMP bitslib/sbi/sbi_hart.c+11 -50 replies
5/9lib: sbi: Modify the boot time region flag printslib/sbi/sbi_domain.c+15 -50 replies
6/9lib: utils: Use SU-{R/W/X} flags for region permissions during parsinglib/utils/fdt/fdt_domain.c+3 -30 replies
7/9lib: utils: Disallow non-root domains from adding M-mode regionsinclude/sbi/sbi_domain.h lib/utils/fdt/fdt_domain.c+18 -00 replies
8/9lib: utils: Add M-mode {R/W} flags to the MMIO regionslib/utils/ipi/aclint_mswi.c lib/utils/irqchip/aplic.c lib/utils/irqchip/imsic.c +1 more+21 -70 replies
9/9docs: Update domain's region permissions and requirementsdocs/domain_support.md+10 -21 reply

Reviews trailers per patch

All 9 patches have a Reviewed-by or Acked-by.

PatchAnup Patel
SeriesT-b
1/9R-b
2/9R-b
3/9R-b
4/9R-b
5/9R-b
6/9R-b
7/9R-b
8/9R-b
9/9R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/9] Split region permissions into M-mode and SU-mode

This is first series in the series of forthcoming patches that would add support for different access permission for different sections of OpenSBI (RO, RX, RW). Eventually, this would lead to support of smepmp extension.

This patch series splits the current R/W/X permissions to M-mode R/W/X and SU-mode R/W/X. The split is in such a way that current permission semantics are not broken

Changes in v2:
1. Fixed the grammatical mistakes in the commit log.
2. Removed the white space new line from sbi_domain.h
3. Added enforce permission bit[6] details to documentation

Himanshu Chauhan (9):
include: sbi: Fine grain the permissions for M and SU modes
lib: sbi: Use finer permission semantics for address validation
lib: sbi: Add permissions for the firmware start till end
lib: sbi: Use finer permission sematics to decide on PMP bits
lib: sbi: Modify the boot time region flag prints
lib: utils: Use SU-{R/W/X} flags for region permissions during parsing
lib: utils: Disallow non-root domains from adding M-mode regions
lib: utils: Add M-mode {R/W} flags to the MMIO regions
docs: Update domain's region permissions and requirements

 docs/domain_support.md          | 12 +++++++--
 include/sbi/sbi_domain.h        | 44 +++++++++++++++++++++++++++----
 lib/sbi/sbi_domain.c            | 46 +++++++++++++++++++++++----------
 lib/sbi/sbi_hart.c              | 16 ++++++++----
 lib/utils/fdt/fdt_domain.c      | 20 +++++++++++---
 lib/utils/ipi/aclint_mswi.c     |  4 ++-
 lib/utils/irqchip/aplic.c       |  4 ++-
 lib/utils/irqchip/imsic.c       |  4 ++-
 lib/utils/timer/aclint_mtimer.c | 16 +++++++++---
 9 files changed, 131 insertions(+), 35 deletions(-)
Signature · 1 line
2.34.1