Commit dde94626 authored by Peter Mitsis's avatar Peter Mitsis Committed by Benjamin Cabé
Browse files

arch: tweak xtensa_mem_kernel_has_access() API 



Adds 'const' to address pointer as its memory contents
do not change.

Signed-off-by: default avatarPeter Mitsis <peter.mitsis@intel.com>
parent 84d2f7f4

arch/xtensa/core/mpu.c

+1 −1
Original line number Diff line number Diff line
@@ -1078,7 +1078,7 @@ out: 
	return ret;

}



bool xtensa_mem_kernel_has_access(void *addr, size_t size, int write)

bool xtensa_mem_kernel_has_access(const void *addr, size_t size, int write)

{

	uintptr_t aligned_addr;

	size_t aligned_size, addr_offset;

arch/xtensa/core/ptables.c

+1 −1
Original line number Diff line number Diff line
@@ -1112,7 +1112,7 @@ static int mem_buffer_validate(const void *addr, size_t size, int write, int rin 
	return ret;

}



bool xtensa_mem_kernel_has_access(void *addr, size_t size, int write)

bool xtensa_mem_kernel_has_access(const void *addr, size_t size, int write)

{

	return mem_buffer_validate(addr, size, write, XTENSA_MMU_KERNEL_RING) == 0;

}

arch/xtensa/core/syscall_helper.c

+1 −1
Original line number Diff line number Diff line
@@ -115,7 +115,7 @@ size_t arch_user_string_nlen(const char *s, size_t maxsize, int *err_arg) 
	 * For MPU systems, this would simply results in access errors

	 * and the exception handler will terminate the thread.

	 */

	if (!xtensa_mem_kernel_has_access((void *)s, maxsize, 0)) {

	if (!xtensa_mem_kernel_has_access(s, maxsize, 0)) {

		/*

		 * API says we need to set err_arg to -1 if there are

		 * any errors.

arch/xtensa/include/xtensa_internal.h

+1 −1
Original line number Diff line number Diff line
@@ -72,7 +72,7 @@ void xtensa_userspace_enter(k_thread_entry_t user_entry, 
 *

 * @return False if the permissions don't match.

 */

bool xtensa_mem_kernel_has_access(void *addr, size_t size, int write);

bool xtensa_mem_kernel_has_access(const void *addr, size_t size, int write);



/**

 * @}

CRA Git | Maintained and supported by SUSTech CRA and CCSE