diff options
| author | Alexander Bulekov | 2021-03-15 15:05:12 +0100 |
|---|---|---|
| committer | Paolo Bonzini | 2021-03-16 19:30:30 +0100 |
| commit | 7cac7fea708a1ca46f16c4e816d88b40da755de1 (patch) | |
| tree | e4ab1fa4ce4e02f73491c4b0c1757138277e8918 /softmmu/memory.c | |
| parent | fuzz: configure a sparse-mem device, by default (diff) | |
| download | qemu-7cac7fea708a1ca46f16c4e816d88b40da755de1.tar.gz qemu-7cac7fea708a1ca46f16c4e816d88b40da755de1.tar.xz qemu-7cac7fea708a1ca46f16c4e816d88b40da755de1.zip | |
fuzz: move some DMA hooks
For the sparse-mem device, we want the fuzzer to populate entire DMA
reads from sparse-mem, rather than hooking into the individual MMIO
memory_region_dispatch_read operations. Otherwise, the fuzzer will treat
each sequential read separately (and populate it with a separate
pattern). Work around this by rearranging some DMA hooks. Since the
fuzzer has it's own logic to skip accidentally writing to MMIO regions,
we can call the DMA cb, outside the flatview_translate loop.
Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
Reviewed-by: Darren Kenny <darren.kenny@oracle.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'softmmu/memory.c')
| -rw-r--r-- | softmmu/memory.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/softmmu/memory.c b/softmmu/memory.c index 9db47b7db6..c4730ec47a 100644 --- a/softmmu/memory.c +++ b/softmmu/memory.c @@ -1440,7 +1440,6 @@ MemTxResult memory_region_dispatch_read(MemoryRegion *mr, unsigned size = memop_size(op); MemTxResult r; - fuzz_dma_read_cb(addr, size, mr); if (!memory_region_access_valid(mr, addr, size, false, attrs)) { *pval = unassigned_mem_read(mr, addr, size); return MEMTX_DECODE_ERROR; |