Am 09.05.2011 16:12, schrieb Alexander Graf: > On 05/08/2011 09:10 PM, Jan Kiszka wrote: >> Hi Alex, >> >> I've seen crashes caused by ahci_check_cmd_bh unregistering a NULL bh. >> It looks like ahci_dma_set_inactive can a called while there is already >> a bh hanging around. Patch below cures the issue, but I have no clue if >> such an invocation order is valid at all. > > It's certainly guest triggerable, so yes, let's check here. > > Acked-by: Alexander Graf <ag...@suse.de>
Yes, the change makes sense to me. Please resend this as a proper patch, Jan. However, I still think Jan's question is valid: Is the AHCI emulation supposed to run multiple DMA requests at once using the core.c functions? I'd find it surprising if this actually worked well. Kevin
