hw/esp.c does not properly deal with TEST_UNIT_READY in NetBSD/sparc

Bug #1014099 reported by Martin Husemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Fix Released
Undecided
Unassigned

Bug Description

The NetBSD ncr53c9x.c driver does a TEST_UNIT_READY command with SELATN but dma disabled sometimes (early during bus enumeration). This is fine, as the command will not produce nor consume any data, and works on real hardware.

However, the qemu emulation does not allow this (for reasons I don't understand).

The change below fixes the problem.

Revision history for this message
Martin Husemann (ub71a5bwcb7k-6kfa0n-b69y0hv8hgzd) wrote :

Guess I understand the code now - so here is a working version - though it may be considered slightly hackish

Revision history for this message
Stefan Hajnoczi (stefanha) wrote : Re: [Qemu-devel] [Bug 1014099] Re: hw/esp.c does not properly deal with TEST_UNIT_READY in NetBSD/sparc

On Sat, Jun 16, 2012 at 5:50 PM, Martin Husemann <email address hidden> wrote:
> ** Patch added: "esp.c.patch"
>   https://bugs.launchpad.net/bugs/1014099/+attachment/3192643/+files/esp.c.patch

Please see this on how to contribute patches to QEMU:
http://wiki.qemu.org/Contribute/SubmitAPatch

Stefan

Revision history for this message
Martin Husemann (ub71a5bwcb7k-6kfa0n-b69y0hv8hgzd) wrote :

Patch removed, as it was bogus and your workflow is weird, so I'll post a better patch to the devel list

Revision history for this message
Thomas Huth (th-huth) wrote :

Has this problem been fixed in 2012, so that we could close this ticket now? Or is there still something left to do?

Changed in qemu:
status: New → Incomplete
Revision history for this message
Martin Husemann (ub71a5bwcb7k-6kfa0n-b69y0hv8hgzd) wrote :

Yes. Just to make sure I tested qemu 2.8 against an old disk image from 2012 and it boots fine w/o any complaints during the device probes.

Changed in qemu:
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.