ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-01-31 00:24:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

platform_drv: use guard pages for DMA buffers

Browse Source 
genodelabs/genode#5002
This commit is contained in:
Johannes Schlatow 2023-11-02 12:43:54 +01:00
parent d018cb62f3
commit 196133c582
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
repos/os/src/drivers/platform/dma_allocator.cc
View File

@ -51,7 +51,12 @@ addr_t Dma_allocator::_alloc_dma_addr(addr_t const phys_addr,
if (size_align_log2 > 24) /* 16 MB */ if (size_align_log2 > 24) /* 16 MB */
size_align_log2 = 24; size_align_log2 = 24;
return _dma_alloc.alloc_aligned(size, size_align_log2).convert<addr_t>( /* add guard page */
size_t guarded_size = size;
if (_use_guard_page)
guarded_size += 0x1000; /* 4 kB */
return _dma_alloc.alloc_aligned(guarded_size, size_align_log2).convert<addr_t>(
[&] (void *ptr) { return (addr_t)ptr; }, [&] (void *ptr) { return (addr_t)ptr; },
[&] (Alloc_error err) -> addr_t { [&] (Alloc_error err) -> addr_t {
switch (err) { switch (err) {
@ -60,6 +65,7 @@ addr_t Dma_allocator::_alloc_dma_addr(addr_t const phys_addr,
case Alloc_error::DENIED: case Alloc_error::DENIED:
error("Could not allocate DMA area of size: ", size, error("Could not allocate DMA area of size: ", size,
" alignment: ", size_align_log2, " alignment: ", size_align_log2,
" size with guard page: ", guarded_size,
" total avail: ", _dma_alloc.avail(), " total avail: ", _dma_alloc.avail(),
" (error: ", err, ")"); " (error: ", err, ")");
break; break;

1
repos/os/src/drivers/platform/dma_allocator.h
View File

@ -61,6 +61,7 @@ class Driver::Dma_allocator
Allocator & _md_alloc; Allocator & _md_alloc;
bool _remapping; bool _remapping;
bool const _use_guard_page { true };
Allocator_avl _dma_alloc { &_md_alloc }; Allocator_avl _dma_alloc { &_md_alloc };
Registry<Dma_buffer> _registry { }; Registry<Dma_buffer> _registry { };