[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v8 0/8] memory: prevent dma-reentracy issues
From: |
Thomas Huth |
Subject: |
Re: [PATCH v8 0/8] memory: prevent dma-reentracy issues |
Date: |
Tue, 25 Apr 2023 11:46:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.0 |
On 21/04/2023 16.27, Alexander Bulekov wrote:
v7 -> v8:
- Disable reentrancy checks for bcm2835_property's iomem (Patch 7)
- Cache DeviceState* in the MemoryRegion to avoid dynamic cast for
each MemoryRegion access. (Patch 1)
- Make re-entrancy fatal for debug-builds (Patch 8)
Hi Alexander,
I just put your series into a run with the gitlab-CI and it seems this now
introduced another failure in one of the avocado tests:
https://gitlab.com/thuth/qemu/-/jobs/4171448248#L318
The "IbmPrep40pMachine.test_openbios_and_netbsd" test is failing now.
You can reproduce it manually quite easily:
wget
https://archive.netbsd.org/pub/NetBSD-archive/NetBSD-7.1.2/iso/NetBSD-7.1.2-prep.iso
./qemu-system-ppc -nographic -M 40p -boot d -cdrom NetBSD-7.1.2-prep.iso
Without your patches, this prints out "NetBSD/prep BOOT, Revision 1.9" in
the console, but with your patches, the message does not appear anymore.
Could you please have a look?
Thanks,
Thomas
- [PATCH v8 2/8] async: Add an optional reentrancy guard to the BH API, (continued)
- [PATCH v8 2/8] async: Add an optional reentrancy guard to the BH API, Alexander Bulekov, 2023/04/21
- [PATCH v8 5/8] memory: Allow disabling re-entrancy checking per-MR, Alexander Bulekov, 2023/04/21
- [PATCH v8 4/8] hw: replace most qemu_bh_new calls with qemu_bh_new_guarded, Alexander Bulekov, 2023/04/21
- [PATCH v8 6/8] lsi53c895a: disable reentrancy detection for script RAM, Alexander Bulekov, 2023/04/21
- [PATCH v8 7/8] bcm2835_property: disable reentrancy detection for iomem, Alexander Bulekov, 2023/04/21
- [PATCH v8 8/8] memory: abort on re-entrancy in debug builds, Alexander Bulekov, 2023/04/21
- Re: [PATCH v8 0/8] memory: prevent dma-reentracy issues,
Thomas Huth <=