]> git.ipfire.org Git - thirdparty/qemu.git/commit
mac_dbdma: Remove leftover `dma_memory_unmap` calls
authorMattias Nissler <mnissler@rivosinc.com>
Mon, 16 Sep 2024 17:57:08 +0000 (10:57 -0700)
committerMichael Tokarev <mjt@tls.msk.ru>
Wed, 25 Sep 2024 18:05:07 +0000 (21:05 +0300)
commit73f5d5bfb7b1f53c830bdd41cc20aefe12ab4827
treea43ecbb2be46a2870467e2d4da4951107f649502
parent97fa3d7fccb1975a33caf011dfd83aba437608d9
mac_dbdma: Remove leftover `dma_memory_unmap` calls

These were passing a NULL buffer pointer unconditionally, which happens
to behave in a mostly benign way (except for the chance of an excess
memory region unref and a bounce buffer leak). Per the function comment,
this was never meant to be accepted though, and triggers an assertion
with the "softmmu: Support concurrent bounce buffers" change.

Given that the code in question never sets up any mappings, just remove
the unnecessary dma_memory_unmap calls along with the DBDMA_io struct
fields that are now entirely unused.

Signed-off-by: Mattias Nissler <mnissler@rivosinc.com>
Message-Id: <20240916175708.1829059-1-mnissler@rivosinc.com>
Fixes: be1e343995 ("macio: switch over to new byte-aligned DMA helpers")
Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
(cherry picked from commit 2d0a071e625d7234e8c5623b7e7bf445e1bef72c)
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
hw/ide/macio.c
include/hw/ppc/mac_dbdma.h