Mirrors/opensbi
1
0
Fork 1
mirror of https://github.com/riscv-software-src/opensbi.git synced 2025-08-24 15:31:22 +01:00
Code Issues Packages Projects Releases Wiki Activity

lib: sbi: Remove MENVCFG hart feature

Browse Source 
If a hart implements privileged spec v1.12 (or higher) then we can
safely assume that menvcfg CSR is present and we don't need MENVCFG
as a hart feature.

Signed-off-by: Anup Patel <apatel@ventanamicro.com>
Reviewed-by: Atish Patra <atishp@rivosinc.com>
This commit is contained in:
Anup Patel
2022-04-28 19:18:17 +05:30
committed by Anup Patel
parent dbc3d8f0ef
commit 97a17c2e5c
2 changed files with 14 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
include/sbi/sbi_hart.h
View File

@@ -32,12 +32,10 @@ enum sbi_hart_features {
SBI_HART_HAS_TIME = (1 << 1), SBI_HART_HAS_TIME = (1 << 1),
/** HART has AIA local interrupt CSRs */ /** HART has AIA local interrupt CSRs */
SBI_HART_HAS_AIA = (1 << 2), SBI_HART_HAS_AIA = (1 << 2),
/** HART has menvcfg CSR */
SBI_HART_HAS_MENVCFG = (1 << 3),
/** HART has mstateen CSR **/ /** HART has mstateen CSR **/
SBI_HART_HAS_SMSTATEEN = (1 << 4), SBI_HART_HAS_SMSTATEEN = (1 << 3),
/** HART has SSTC extension implemented in hardware */ /** HART has SSTC extension implemented in hardware */
SBI_HART_HAS_SSTC = (1 << 5), SBI_HART_HAS_SSTC = (1 << 4),
/** Last index of Hart features*/ /** Last index of Hart features*/
SBI_HART_HAS_LAST_FEATURE = SBI_HART_HAS_SSTC, SBI_HART_HAS_LAST_FEATURE = SBI_HART_HAS_SSTC,

32
lib/sbi/sbi_hart.c
View File

@@ -97,10 +97,8 @@ static void mstatus_init(struct sbi_scratch *scratch)
mstateen_val |= ((uint64_t)csr_read(CSR_MSTATEEN0H)) << 32; mstateen_val |= ((uint64_t)csr_read(CSR_MSTATEEN0H)) << 32;
#endif #endif
mstateen_val |= SMSTATEEN_STATEN; mstateen_val |= SMSTATEEN_STATEN;
if (sbi_hart_has_feature(scratch, SBI_HART_HAS_MENVCFG)) mstateen_val |= SMSTATEEN0_HSENVCFG;
mstateen_val |= SMSTATEEN0_HSENVCFG;
else
mstateen_val &= ~SMSTATEEN0_HSENVCFG;
if (sbi_hart_has_feature(scratch, SBI_HART_HAS_AIA)) if (sbi_hart_has_feature(scratch, SBI_HART_HAS_AIA))
mstateen_val |= (SMSTATEEN0_AIA | SMSTATEEN0_SVSLCT | mstateen_val |= (SMSTATEEN0_AIA | SMSTATEEN0_SVSLCT |
SMSTATEEN0_IMSIC); SMSTATEEN0_IMSIC);
@@ -113,7 +111,7 @@ static void mstatus_init(struct sbi_scratch *scratch)
#endif #endif
} }
if (sbi_hart_has_feature(scratch, SBI_HART_HAS_MENVCFG)) { if (sbi_hart_priv_version(scratch) >= SBI_HART_PRIV_VER_1_12) {
menvcfg_val = csr_read(CSR_MENVCFG); menvcfg_val = csr_read(CSR_MENVCFG);
/* /*
@@ -413,9 +411,6 @@ static inline char *sbi_hart_feature_id2string(unsigned long feature)
case SBI_HART_HAS_SSTC: case SBI_HART_HAS_SSTC:
fstr = "sstc"; fstr = "sstc";
break; break;
case SBI_HART_HAS_MENVCFG:
fstr = "menvcfg";
break;
case SBI_HART_HAS_SMSTATEEN: case SBI_HART_HAS_SMSTATEEN:
fstr = "smstateen"; fstr = "smstateen";
break; break;
@@ -616,13 +611,11 @@ __mhpm_skip:
(hfeatures->priv_version >= SBI_HART_PRIV_VER_1_10)) (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_10))
hfeatures->priv_version = SBI_HART_PRIV_VER_1_11; hfeatures->priv_version = SBI_HART_PRIV_VER_1_11;
/* Detect if hart has menvcfg CSR */ /* Detect if hart supports Priv v1.12 */
csr_read_allowed(CSR_MENVCFG, (unsigned long)&trap); csr_read_allowed(CSR_MENVCFG, (unsigned long)&trap);
if (!trap.cause && if (!trap.cause &&
(hfeatures->priv_version >= SBI_HART_PRIV_VER_1_11)) { (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_11))
hfeatures->priv_version = SBI_HART_PRIV_VER_1_12; hfeatures->priv_version = SBI_HART_PRIV_VER_1_12;
hfeatures->features |= SBI_HART_HAS_MENVCFG;
}
/* Counter overflow/filtering is not useful without mcounter/inhibit */ /* Counter overflow/filtering is not useful without mcounter/inhibit */
if (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_12) { if (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_12) {
@@ -644,20 +637,19 @@ __mhpm_skip:
hfeatures->features |= SBI_HART_HAS_AIA; hfeatures->features |= SBI_HART_HAS_AIA;
__aia_skip: __aia_skip:
/** /* Detect if hart supports stimecmp CSR(Sstc extension) */
* Detect if hart supports stimecmp CSR(Sstc extension) and menvcfg is if (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_12) {
* implemented.
*/
if (hfeatures->features & SBI_HART_HAS_MENVCFG) {
csr_read_allowed(CSR_STIMECMP, (unsigned long)&trap); csr_read_allowed(CSR_STIMECMP, (unsigned long)&trap);
if (!trap.cause) if (!trap.cause)
hfeatures->features |= SBI_HART_HAS_SSTC; hfeatures->features |= SBI_HART_HAS_SSTC;
} }
/* Detect if hart supports mstateen CSRs */ /* Detect if hart supports mstateen CSRs */
val = csr_read_allowed(CSR_MSTATEEN0, (unsigned long)&trap); if (hfeatures->priv_version >= SBI_HART_PRIV_VER_1_12) {
if (!trap.cause) val = csr_read_allowed(CSR_MSTATEEN0, (unsigned long)&trap);
hfeatures->features |= SBI_HART_HAS_SMSTATEEN; if (!trap.cause)
hfeatures->features |= SBI_HART_HAS_SMSTATEEN;
}
return; return;
} }