Re: [PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-11-15 Thread Philippe Mathieu-Daudé
On 10/11/22 21:50, Stefan Hajnoczi wrote: Preventing this class of bugs is important but QEMU is currently frozen for the 7.2 release. I'm a little concerned about regressions in a patch series that changes core device emulation code. I'm waiting for Alex's MemTxRequesterType field addition in

Re: [PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-11-10 Thread Peter Maydell
On Thu, 10 Nov 2022 at 20:51, Stefan Hajnoczi wrote: > > Preventing this class of bugs is important but QEMU is currently > frozen for the 7.2 release. I'm a little concerned about regressions > in a patch series that changes core device emulation code. > > I'll review the series on Monday and if

Re: [PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-11-10 Thread Michael S. Tsirkin
On Thu, Nov 10, 2022 at 03:50:51PM -0500, Stefan Hajnoczi wrote: > Preventing this class of bugs is important but QEMU is currently > frozen for the 7.2 release. I'm a little concerned about regressions > in a patch series that changes core device emulation code. > > I'll review the series on Mond

Re: [PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-11-10 Thread Stefan Hajnoczi
Preventing this class of bugs is important but QEMU is currently frozen for the 7.2 release. I'm a little concerned about regressions in a patch series that changes core device emulation code. I'll review the series on Monday and if anyone has strong opinions on whether to merge this into 7.2, ple

Re: [PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-11-07 Thread Alexander Bulekov
On 221028 1516, Alexander Bulekov wrote: > These patches aim to solve two types of DMA-reentrancy issues: > > 1.) mmio -> dma -> mmio case > To solve this, we track whether the device is engaged in io by > checking/setting a flag within APIs used for MMIO access. > > 2.) bh -> dma write -> mmio c

[PATCH v3 0/7] memory: prevent dma-reentracy issues

2022-10-28 Thread Alexander Bulekov
These patches aim to solve two types of DMA-reentrancy issues: 1.) mmio -> dma -> mmio case To solve this, we track whether the device is engaged in io by checking/setting a flag within APIs used for MMIO access. 2.) bh -> dma write -> mmio case This case is trickier, since we dont have a generic