KekenoBSD/src
3
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

linuxkpi: Fix set_memory_*

Browse Source 
set_memory_* is currently implemented using PHYS_TO_DMAP but not all
architectures have a DMAP.  Looking at how this function is used the
given address isn't physical but virtual so the PHYS_TO_DMAP call can
simply be removed.

Also cast numpages before shifting it to avoid overflow.

Reviewed by:	kib, markj
MFC after:	3 days
Differential Revision:	https://reviews.freebsd.org/D45057
This commit is contained in:
Tijl Coosemans
2024-05-03 15:27:29 +02:00
parent 95032b58a1
commit 7206f7c619
1 changed files with 6 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sys/compat/linuxkpi/common/include/asm/set_memory.h
+6 -15
View File
@@ -34,26 +34,20 @@
static inline int
set_memory_uc(unsigned long addr, int numpages)
{
vm_offset_t va;
vm_size_t len;
va = PHYS_TO_DMAP(addr);
len = numpages << PAGE_SHIFT;
return (-pmap_change_attr(va, len, VM_MEMATTR_UNCACHEABLE));
len = (vm_size_t)numpages << PAGE_SHIFT;
return (-pmap_change_attr(addr, len, VM_MEMATTR_UNCACHEABLE));
}
static inline int
set_memory_wc(unsigned long addr, int numpages)
{
#ifdef VM_MEMATTR_WRITE_COMBINING
vm_offset_t va;
vm_size_t len;
va = PHYS_TO_DMAP(addr);
len = numpages << PAGE_SHIFT;
return (-pmap_change_attr(va, len, VM_MEMATTR_WRITE_COMBINING));
len = (vm_size_t)numpages << PAGE_SHIFT;
return (-pmap_change_attr(addr, len, VM_MEMATTR_WRITE_COMBINING));
#else
return (set_memory_uc(addr, numpages));
#endif
@@ -62,13 +56,10 @@ set_memory_wc(unsigned long addr, int numpages)
static inline int
set_memory_wb(unsigned long addr, int numpages)
{
vm_offset_t va;
vm_size_t len;
va = PHYS_TO_DMAP(addr);
len = numpages << PAGE_SHIFT;
return (-pmap_change_attr(va, len, VM_MEMATTR_WRITE_BACK));
len = (vm_size_t)numpages << PAGE_SHIFT;
return (-pmap_change_attr(addr, len, VM_MEMATTR_WRITE_BACK));
}
static inline int