[virtio] scsi: Correctly handle command completion

Correct a bug in how ExecuteCommandSync called IrqRingUpdate that would
cause the scan worker thread to wait forever.

virtio ring::IrqRingUpdate will invoke a passed-in functor on descriptor
chains completed by the device. However if no descriptor chains were
completed, it would not call the functor. (I had assumed at first skim
that it would have waited instead).

Modify ExecuteCommandSync to keep invoking IrqRingUpdate with 5 mS steps
until the command completes with either an error or success. Later we will
wire up interrupt mode support, to avoid this 5 mS polling looping.

ZX-2314

Tested:
1) Booted on GCE, dm dump displayed the virtio-scsi target at 1-0.

2) Still boots on QEMU (qemu masked the error by synchronously processing
   control commands).

            [00:02.0] pid=2006 /boot/driver/bus-pci.so
               <00:02.0> pid=2884 /boot/driver/bus-pci.proxy.so
                  [virtio-scsi] pid=2884 /boot/driver/virtio.so
                     [scsi-disk-1-1] pid=2884 /boot/driver/virtio.so

Both QEMU and GCE tests need the LUN encoding fix included (see 239313) for
tests to work.

Change-Id: I9a0d8f5860f11cf1f4b900ebe3c5c4cf76a060cd
1 file changed
tree: a7cc391186522a11dd694c5e73e2ffda400f9cc4
  1. bootloader/
  2. docs/
  3. kernel/
  4. make/
  5. prebuilt/
  6. public/
  7. scripts/
  8. system/
  9. third_party/
  10. .clang-format
  11. .clang-tidy
  12. .dir-locals.el
  13. .gitignore
  14. AUTHORS
  15. LICENSE
  16. MAINTAINERS
  17. makefile
  18. navbar.md
  19. PATENTS
  20. README.md
README.md

Zircon

Zircon is the core platform that powers the Fuchsia OS. Zircon is composed of a microkernel (source in kernel/...) as well as a small set of userspace services, drivers, and libraries (source in system/...) necessary for the system to boot, talk to hardware, load userspace processes and run them, etc. Fuchsia builds a much larger OS on top of this foundation.

The canonical Zircon Git repository is located at: https://fuchsia.googlesource.com/zircon

The Zircon Kernel provides syscalls to manage processes, threads, virtual memory, inter-process communication, waiting on object state changes, and locking (via futexes).

Currently there are some temporary syscalls that have been used for early bringup work, which will be going away in the future as the long term syscall API/ABI surface is finalized. The expectation is that there will be about 100 syscalls.

Zircon syscalls are generally non-blocking. The wait_one, wait_many port_wait and thread sleep being the notable exceptions.

This page is a non-comprehensive index of the zircon documentation.