sysadmin/ipxe
1
0
Fork 0
mirror of https://github.com/ipxe/ipxe synced 2025-12-16 09:32:33 +03:00
Code Issues Packages Projects Releases Wiki Activity

[uaccess] Add explicit casts to and from userptr_t where needed

Browse Source 
Allow for the possibility of userptr_t becoming a pointer type by
adding explicit casts where necessary.

Signed-off-by: Michael Brown <mcb30@ipxe.org>
This commit is contained in:
Michael Brown
2025-04-20 13:39:32 +01:00
parent 63d27c6311
commit 71174e19d8
6 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/arch/x86/include/librm.h
View File

@@ -96,10 +96,10 @@ UACCESS_INLINE ( librm, phys_to_user ) ( unsigned long phys_addr ) {
* identity-mapped.
*/
if ( sizeof ( physaddr_t ) > sizeof ( uint32_t ) )
return phys_addr;
return ( ( userptr_t ) phys_addr );
/* In a 32-bit build, subtract virt_offset */
return ( phys_addr - virt_offset );
return ( ( userptr_t ) ( phys_addr - virt_offset ) );
}
/**
@@ -111,7 +111,7 @@ UACCESS_INLINE ( librm, phys_to_user ) ( unsigned long phys_addr ) {
*/
static inline __always_inline unsigned long
UACCESS_INLINE ( librm, user_to_phys ) ( userptr_t userptr, off_t offset ) {
unsigned long addr = ( userptr + offset );
unsigned long addr = ( ( unsigned long ) ( userptr + offset ) );
/* In a 64-bit build, any virtual address in the low 4GB is
* directly usable as a physical address, since the low 4GB is

2
src/arch/x86/interface/pcbios/memtop_umalloc.c
View File

@@ -45,7 +45,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
#define EM_ALIGN ( 4 * 1024 )
/** Equivalent of NOWHERE for user pointers */
#define UNOWHERE ( ~UNULL )
#define UNOWHERE ( ( userptr_t ) ~( ( intptr_t ) 0 ) )
/** An external memory block */
struct external_memory {

2
src/drivers/infiniband/golan.c
View File

@@ -487,7 +487,7 @@ static inline int golan_provide_pages ( struct golan *golan , uint32_t pages
next_page_addr += GOLAN_PAGE_SIZE ) {
addr = next_page_addr;
if (GOLAN_PAGE_MASK & user_to_phys(addr, 0)) {
DBGC (golan ,"Addr not Page alligned [%lx %lx]\n", user_to_phys(addr, 0), addr);
DBGC (golan ,"Addr not Page alligned [%lx]\n", user_to_phys(addr, 0));
}
mailbox->mblock.data[j] = USR_2_BE64_BUS(addr);
}

4
src/include/ipxe/linux/linux_uaccess.h
View File

@@ -43,7 +43,7 @@ UACCESS_INLINE ( linux, user_to_phys ) ( userptr_t userptr, off_t offset ) {
* virtual address will suffice for the purpose of determining
* alignment.
*/
return ( userptr + offset );
return ( ( unsigned long ) ( userptr + offset ) );
}
/**
@@ -56,7 +56,7 @@ static inline __always_inline userptr_t
UACCESS_INLINE ( linux, phys_to_user ) ( physaddr_t phys_addr ) {
/* For symmetry with the stub user_to_phys() */
return phys_addr;
return ( ( userptr_t ) phys_addr );
}
static inline __always_inline userptr_t

4
src/include/ipxe/uaccess.h
View File

@@ -224,12 +224,12 @@ trivial_memchr_user ( userptr_t buffer, off_t offset, int c, size_t len ) {
static inline __always_inline userptr_t
UACCESS_INLINE ( flat, phys_to_user ) ( unsigned long phys_addr ) {
return phys_addr;
return ( ( userptr_t ) phys_addr );
}
static inline __always_inline unsigned long
UACCESS_INLINE ( flat, user_to_phys ) ( userptr_t userptr, off_t offset ) {
return ( userptr + offset );
return ( ( unsigned long ) ( userptr + offset ) );
}
static inline __always_inline userptr_t

2
src/interface/efi/efi_umalloc.c
View File

@@ -36,7 +36,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
*/
/** Equivalent of NOWHERE for user pointers */
#define UNOWHERE ( ~UNULL )
#define UNOWHERE ( ( userptr_t ) ~( ( intptr_t ) 0 ) )
/**
* Reallocate external memory