[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups |
Date: |
Mon, 30 Oct 2017 19:00:01 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 27/10/17 17:42, Philippe Mathieu-Daudé wrote:
> Hi Mark,
>
> On 10/25/2017 12:59 PM, Mark Cave-Ayland wrote:
>> This patchset aims to tidy-up the sparc32_dma code by improving the
>> modelling of the espdma/ledma devices using both QOM and the memory
>> API which didn't exist when the code was first written.
>>
>> The result is that it is now possible to remove both the iommu_opaque
>> and is_ledma workarounds from the code, and the code for wiring up
>> the espdma/ledma and respective devices is also a lot more readable.
>>
>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>
> The whole series:
>
> Tested-by: Philippe Mathieu-Daudé <address@hidden>
>
> If you don't accept my comments (or don't have time) about keeping
> "hw/sparc/sparc32_dma.h" generic and moving network/scsi parts in
> "hw/sparc/sun4m.h" you can still add to your series:
>
> Acked-by: Philippe Mathieu-Daudé <address@hidden>
Thanks for the review, I've added your R-B tags to the individual
patches. Note that while potentially I could move the network/scsi parts
to hw/sparc/sun4m.h I feel that it's a slightly better match for the
SPARC32 DMA container device to remain in sparc32_dma.c. So for these
patches I've just added your A-B tag.
> Also while testing your series on a Debian image, I noted your series
> results faster, I timed:
>
> master: 104s
> your series: 85s (>20% faster!)
Really? Is that for just this patchset or also with the v2 IOMMU
patchset applied on top? I can't immediately see how moving the logic
into sparc32_dma.c could make a difference here...
ATB,
Mark.
- [Qemu-devel] [PATCHv4 10/13] sparc32_dma: make lance device child of ledma device, (continued)
- [Qemu-devel] [PATCHv4 13/13] sparc32_dma: add len to esp/le DMA memory tracing, Mark Cave-Ayland, 2017/10/25
- Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups, Artyom Tarasenko, 2017/10/25
- Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups, no-reply, 2017/10/26
- Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups, Philippe Mathieu-Daudé, 2017/10/27
- Re: [Qemu-devel] [PATCHv4 00/13] sun4m: sparc32_dma tidy-ups,
Mark Cave-Ayland <=