[smart] fixup: mmap support (#8154)

Signed-off-by: Shell <smokewood@qq.com>
This commit is contained in:
Shell 2023-10-20 13:28:20 +08:00 committed by GitHub
parent 07b23ecbb1
commit 1d678e5596
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 47 additions and 44 deletions

View File

@ -30,14 +30,12 @@ struct mem_desc
struct rt_varea varea; struct rt_varea varea;
}; };
enum rt_hw_mmu_prot_t { #define RT_HW_MMU_PROT_READ 1
RT_HW_MMU_PROT_READ, #define RT_HW_MMU_PROT_WRITE 2
RT_HW_MMU_PROT_WRITE, #define RT_HW_MMU_PROT_EXECUTE 4
RT_HW_MMU_PROT_EXECUTE, #define RT_HW_MMU_PROT_KERNEL 8
RT_HW_MMU_PROT_KERNEL, #define RT_HW_MMU_PROT_USER 16
RT_HW_MMU_PROT_USER, #define RT_HW_MMU_PROT_CACHE 32
RT_HW_MMU_PROT_CACHE,
};
#define MMU_AF_SHIFT 10 #define MMU_AF_SHIFT 10
#define MMU_SHARED_SHIFT 8 #define MMU_SHARED_SHIFT 8
@ -142,7 +140,7 @@ static inline void *rt_hw_mmu_kernel_v2p(void *v_addr)
* @param prot protect that will be added * @param prot protect that will be added
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -163,7 +161,7 @@ rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot
* @param prot protect that will be removed * @param prot protect that will be removed
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -185,7 +183,7 @@ rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot)
* @param prot protect that will be test * @param prot protect that will be test
* @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE * @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE
*/ */
rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, rt_base_t prot)
{ {
rt_bool_t rc; rt_bool_t rc;
switch (prot) switch (prot)

View File

@ -101,14 +101,12 @@ struct mem_desc
*/ */
#define ARCH_MAP_FAILED ((void *)-1) #define ARCH_MAP_FAILED ((void *)-1)
enum rt_hw_mmu_prot_t { #define RT_HW_MMU_PROT_READ 1
RT_HW_MMU_PROT_READ, #define RT_HW_MMU_PROT_WRITE 2
RT_HW_MMU_PROT_WRITE, #define RT_HW_MMU_PROT_EXECUTE 4
RT_HW_MMU_PROT_EXECUTE, #define RT_HW_MMU_PROT_KERNEL 8
RT_HW_MMU_PROT_KERNEL, #define RT_HW_MMU_PROT_USER 16
RT_HW_MMU_PROT_USER, #define RT_HW_MMU_PROT_CACHE 32
RT_HW_MMU_PROT_CACHE,
};
int rt_hw_mmu_ioremap_init(struct rt_aspace *aspace, void *v_address, size_t size); int rt_hw_mmu_ioremap_init(struct rt_aspace *aspace, void *v_address, size_t size);
void rt_hw_init_mmu_table(struct mem_desc *mdesc, rt_uint32_t size); void rt_hw_init_mmu_table(struct mem_desc *mdesc, rt_uint32_t size);
@ -142,7 +140,7 @@ void rt_hw_mmu_pgtbl_delete(void *pgtbl);
* @param prot protect that will be removed * @param prot protect that will be removed
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -176,7 +174,7 @@ rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot)
* @param prot protect that will be added * @param prot protect that will be added
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -199,7 +197,7 @@ rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot
* @param prot protect that will be test * @param prot protect that will be test
* @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE * @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE
*/ */
rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, rt_base_t prot)
{ {
rt_bool_t rc = 0; rt_bool_t rc = 0;
switch (prot) switch (prot)

View File

@ -111,14 +111,12 @@ void mmu_set_pagetable(rt_ubase_t addr);
void mmu_enable_user_page_access(); void mmu_enable_user_page_access();
void mmu_disable_user_page_access(); void mmu_disable_user_page_access();
enum rt_hw_mmu_prot_t { #define RT_HW_MMU_PROT_READ 1
RT_HW_MMU_PROT_READ, #define RT_HW_MMU_PROT_WRITE 2
RT_HW_MMU_PROT_WRITE, #define RT_HW_MMU_PROT_EXECUTE 4
RT_HW_MMU_PROT_EXECUTE, #define RT_HW_MMU_PROT_KERNEL 8
RT_HW_MMU_PROT_KERNEL, #define RT_HW_MMU_PROT_USER 16
RT_HW_MMU_PROT_USER, #define RT_HW_MMU_PROT_CACHE 32
RT_HW_MMU_PROT_CACHE,
};
/** /**
* @brief Remove permission from attribution * @brief Remove permission from attribution
@ -127,7 +125,7 @@ enum rt_hw_mmu_prot_t {
* @param prot protect that will be removed * @param prot protect that will be removed
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -146,7 +144,7 @@ rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot)
* @param prot protect that will be added * @param prot protect that will be added
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
@ -165,7 +163,7 @@ rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot
* @param prot protect that will be test * @param prot protect that will be test
* @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE * @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE
*/ */
rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, rt_base_t prot)
{ {
rt_bool_t rc = 0; rt_bool_t rc = 0;
switch (prot) switch (prot)

View File

@ -204,6 +204,7 @@ void handle_user(rt_size_t scause, rt_size_t stval, rt_size_t sepc, struct rt_hw
if (fault_op) if (fault_op)
{ {
rt_base_t saved_stat;
lwp = lwp_self(); lwp = lwp_self();
struct rt_aspace_fault_msg msg = { struct rt_aspace_fault_msg msg = {
.fault_op = fault_op, .fault_op = fault_op,
@ -211,10 +212,13 @@ void handle_user(rt_size_t scause, rt_size_t stval, rt_size_t sepc, struct rt_hw
.fault_vaddr = (void *)stval, .fault_vaddr = (void *)stval,
}; };
__asm__ volatile ("csrrsi %0, sstatus, 2":"=r"(saved_stat));
if (lwp && rt_aspace_fault_try_fix(lwp->aspace, &msg)) if (lwp && rt_aspace_fault_try_fix(lwp->aspace, &msg))
{ {
__asm__ volatile ("csrw sstatus, %0"::"r"(saved_stat));
return; return;
} }
__asm__ volatile ("csrw sstatus, %0"::"r"(saved_stat));
} }
LOG_E("[FATAL ERROR] Exception %ld:%s\n", id, get_exception_msg(id)); LOG_E("[FATAL ERROR] Exception %ld:%s\n", id, get_exception_msg(id));
LOG_E("scause:0x%p,stval:0x%p,sepc:0x%p\n", scause, stval, sepc); LOG_E("scause:0x%p,stval:0x%p,sepc:0x%p\n", scause, stval, sepc);

View File

@ -94,14 +94,12 @@ void mmu_set_pagetable(rt_ubase_t addr);
void mmu_enable_user_page_access(); void mmu_enable_user_page_access();
void mmu_disable_user_page_access(); void mmu_disable_user_page_access();
enum rt_hw_mmu_prot_t { #define RT_HW_MMU_PROT_READ 1
RT_HW_MMU_PROT_READ, #define RT_HW_MMU_PROT_WRITE 2
RT_HW_MMU_PROT_WRITE, #define RT_HW_MMU_PROT_EXECUTE 4
RT_HW_MMU_PROT_EXECUTE, #define RT_HW_MMU_PROT_KERNEL 8
RT_HW_MMU_PROT_KERNEL, #define RT_HW_MMU_PROT_USER 16
RT_HW_MMU_PROT_USER, #define RT_HW_MMU_PROT_CACHE 32
RT_HW_MMU_PROT_CACHE,
};
/** /**
* @brief Remove permission from attribution * @brief Remove permission from attribution
@ -110,13 +108,14 @@ enum rt_hw_mmu_prot_t {
* @param prot protect that will be removed * @param prot protect that will be removed
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
/* remove write permission for user */ /* remove write permission for user */
case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER: case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER:
attr &= ~PTE_W; attr &= ~PTE_W;
break;
default: default:
RT_ASSERT(0); RT_ASSERT(0);
} }
@ -130,13 +129,14 @@ rt_inline size_t rt_hw_mmu_attr_rm_perm(size_t attr, enum rt_hw_mmu_prot_t prot)
* @param prot protect that will be added * @param prot protect that will be added
* @return size_t returned attribution * @return size_t returned attribution
*/ */
rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, rt_base_t prot)
{ {
switch (prot) switch (prot)
{ {
/* add write permission for user */ /* add write permission for user */
case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER: case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER:
attr |= PTE_W; attr |= PTE_W;
break;
default: default:
RT_ASSERT(0); RT_ASSERT(0);
} }
@ -150,7 +150,7 @@ rt_inline size_t rt_hw_mmu_attr_add_perm(size_t attr, enum rt_hw_mmu_prot_t prot
* @param prot protect that will be test * @param prot protect that will be test
* @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE * @return rt_bool_t RT_TRUE if the prot is allowed, otherwise RT_FALSE
*/ */
rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, enum rt_hw_mmu_prot_t prot) rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, rt_base_t prot)
{ {
rt_bool_t rc = 0; rt_bool_t rc = 0;
switch (prot) switch (prot)
@ -158,6 +158,7 @@ rt_inline rt_bool_t rt_hw_mmu_attr_test_perm(size_t attr, enum rt_hw_mmu_prot_t
/* test write permission for user */ /* test write permission for user */
case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER: case RT_HW_MMU_PROT_WRITE | RT_HW_MMU_PROT_USER:
rc = !!(attr & PTE_W); rc = !!(attr & PTE_W);
break;
default: default:
RT_ASSERT(0); RT_ASSERT(0);
} }

View File

@ -201,6 +201,7 @@ void handle_user(rt_size_t scause, rt_size_t stval, rt_size_t sepc, struct rt_hw
if (fault_op) if (fault_op)
{ {
rt_base_t saved_stat;
lwp = lwp_self(); lwp = lwp_self();
struct rt_aspace_fault_msg msg = { struct rt_aspace_fault_msg msg = {
.fault_op = fault_op, .fault_op = fault_op,
@ -208,10 +209,13 @@ void handle_user(rt_size_t scause, rt_size_t stval, rt_size_t sepc, struct rt_hw
.fault_vaddr = (void *)stval, .fault_vaddr = (void *)stval,
}; };
__asm__ volatile ("csrrsi %0, sstatus, 2":"=r"(saved_stat));
if (lwp && rt_aspace_fault_try_fix(lwp->aspace, &msg)) if (lwp && rt_aspace_fault_try_fix(lwp->aspace, &msg))
{ {
__asm__ volatile ("csrw sstatus, %0"::"r"(saved_stat));
return; return;
} }
__asm__ volatile ("csrw sstatus, %0"::"r"(saved_stat));
} }
LOG_E("[FATAL ERROR] Exception %ld:%s\n", id, get_exception_msg(id)); LOG_E("[FATAL ERROR] Exception %ld:%s\n", id, get_exception_msg(id));
LOG_E("scause:0x%p,stval:0x%p,sepc:0x%p\n", scause, stval, sepc); LOG_E("scause:0x%p,stval:0x%p,sepc:0x%p\n", scause, stval, sepc);