qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 1/2] hw/ide: reset: cancel async DMA operation before rese


From: Simon Rowe
Subject: Re: [PATCH v2 1/2] hw/ide: reset: cancel async DMA operation before resetting state
Date: Thu, 5 Oct 2023 10:49:44 +0000

On Wednesday, 6 September 2023 Fiona Ebner <f.ebner@proxmox.com> wrote:

 

> If there is a pending DMA operation during ide_bus_reset(), the fact
> that the IDEState is already reset before the operation is canceled
> can be problematic. In particular, ide_dma_cb() might be called and
> then use the reset IDEState which contains the signature after the
> reset. When used to construct the IO operation this leads to
> ide_get_sector() returning 0 and nsector being 1. This is particularly
> bad, because a write command will thus destroy the first sector which
> often contains a partition table or similar.

 

Tested-by: simon.rowe@nutanix.com

 


reply via email to

[Prev in Thread] Current Thread [Next in Thread]