qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] 769998: scsi-bus: prepare scsi_req_new for in


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 769998: scsi-bus: prepare scsi_req_new for introduction of...
Date: Tue, 19 Aug 2014 07:30:05 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 769998a1dbf338a0cdeab2acf8ef605bcf635e01
      
https://github.com/qemu/qemu/commit/769998a1dbf338a0cdeab2acf8ef605bcf635e01
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M hw/scsi/scsi-bus.c

  Log Message:
  -----------
  scsi-bus: prepare scsi_req_new for introduction of parse_cdb

The per-SCSIDevice parse_cdb callback must not be called if the
request will go through special SCSIReqOps, so detect the special
cases early enough.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: ff34c32ccc29fc23c11f0ef6c86257e23f9e3266
      
https://github.com/qemu/qemu/commit/ff34c32ccc29fc23c11f0ef6c86257e23f9e3266
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M hw/scsi/scsi-bus.c
    M include/hw/scsi/scsi.h

  Log Message:
  -----------
  scsi-bus: introduce parse_cdb in SCSIDeviceClass and SCSIBusInfo

These callbacks will let devices do their own request parsing, or
defer it to the bus.  If the bus does not provide an implementation,
in turn, fall back to the default parsing routine.

Swap the first two arguments to scsi_req_parse, and rename it to
scsi_req_parse_cdb, for consistency.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 592c3b289f77ee77d5bff25cb19326cc7f22a532
      
https://github.com/qemu/qemu/commit/592c3b289f77ee77d5bff25cb19326cc7f22a532
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M hw/scsi/scsi-disk.c

  Log Message:
  -----------
  scsi-block: extract scsi_block_is_passthrough

This will be used for both scsi_block_new_request and the scsi-block
implementation of parse_cdb.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 3e7e180ab3f970a4e5b64b5fafd57c48c98ee560
      
https://github.com/qemu/qemu/commit/3e7e180ab3f970a4e5b64b5fafd57c48c98ee560
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M hw/scsi/scsi-bus.c
    M hw/scsi/scsi-disk.c
    M hw/scsi/scsi-generic.c
    M include/hw/scsi/scsi.h

  Log Message:
  -----------
  scsi-block, scsi-generic: implement parse_cdb

The callback lets the bus provide the direction and transfer count
for passthrough commands, enabling passthrough of vendor-specific
commands.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 33cbb2c546594685131e771e2e0972418b9d0301
      
https://github.com/qemu/qemu/commit/33cbb2c546594685131e771e2e0972418b9d0301
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-07-29 (Tue, 29 Jul 2014)

  Changed paths:
    M hw/scsi/virtio-scsi.c

  Log Message:
  -----------
  virtio-scsi: implement parse_cdb

Enable passthrough of vendor-specific commands.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 76a6e1cc7cc3ad022e7159b37b291b75bc4615bf
      
https://github.com/qemu/qemu/commit/76a6e1cc7cc3ad022e7159b37b291b75bc4615bf
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M qom/object.c

  Log Message:
  -----------
  qom: object: delete properties before calling instance_finalize

This ensures that the children's unparent callback will still
have a usable parent.

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: bffc687d66604e68c14f6277b595b33966eb328a
      
https://github.com/qemu/qemu/commit/bffc687d66604e68c14f6277b595b33966eb328a
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M qom/object.c

  Log Message:
  -----------
  qom: object: move unparenting to the child property's release callback

This ensures that the unparent callback is called automatically
when the parent object is finalized.

Note that there's no need to keep a reference neither in
object_unparent nor in object_finalize_child_property.  The
reference held by the child property itself will do.

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 1dd79a237eb84a70c21b7a640f237ee603b0e74a
      
https://github.com/qemu/qemu/commit/1dd79a237eb84a70c21b7a640f237ee603b0e74a
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M hw/core/sysbus.c
    M include/hw/sysbus.h

  Log Message:
  -----------
  sysbus: remove unused function sysbus_del_io

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: ad37168cbdaff061fd7c37be57de4692bd8c2c50
      
https://github.com/qemu/qemu/commit/ad37168cbdaff061fd7c37be57de4692bd8c2c50
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M hw/display/vga.c
    M hw/display/vga_int.h

  Log Message:
  -----------
  vga: do not dynamically allocate chain4_alias

Instead, add a boolean variable to indicate the presence of the region.
This avoids a repeated malloc/free (later we can also avoid the
add_child/unparent by changing the offset/size of the alias).

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: eed79309502034d348880414e1dc156c0c4b196c
      
https://github.com/qemu/qemu/commit/eed79309502034d348880414e1dc156c0c4b196c
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M hw/net/dp8393x.c
    M hw/net/mcf_fec.c

  Log Message:
  -----------
  nic: do not destroy memory regions in cleanup functions

The memory regions should be destroyed in the unrealize function;
since these NICs are not even qdev-ified, they cannot be unplugged
and they do not have to do anything to destroy their memory regions.

Reviewed-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e3fb0ade83420a86464ee50c71f2daf5641cab10
      
https://github.com/qemu/qemu/commit/e3fb0ade83420a86464ee50c71f2daf5641cab10
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-17 (Sun, 17 Aug 2014)

  Changed paths:
    M ioport.c

  Log Message:
  -----------
  ioport: split deletion and destruction

Of the two functions portio_list_del and portio_list_destroy,
the latter is just freeing a memory area.  However, portio_list_del
is the logical equivalent of memory_region_del_subregion so
destruction of memory regions does not belong there.

Actually, neither of these APIs are in use; portio is mostly used by
ISA devices or VGAs, and neither of these is currently hot-unpluggable.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: d8d95814609e89e5438a3318a647ec322fc4ff16
      
https://github.com/qemu/qemu/commit/d8d95814609e89e5438a3318a647ec322fc4ff16
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M docs/memory.txt
    M hw/display/vga.c
    M hw/i386/kvmvapic.c
    M hw/mips/gt64xxx_pci.c
    M hw/misc/omap_gpmc.c
    M hw/misc/vfio.c
    M hw/ppc/ppc4xx_devs.c
    M ioport.c
    M memory.c

  Log Message:
  -----------
  memory: convert memory_region_destroy to object_unparent

Explicitly call object_unparent in the few places where we
will re-create the memory region.  If the memory region is
simply being destroyed as part of device teardown, let QOM
handle it.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 469b046ead0671932ff3af8d6f95045b19b186ef
      
https://github.com/qemu/qemu/commit/469b046ead0671932ff3af8d6f95045b19b186ef
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M backends/hostmem.c
    M hw/audio/ac97.c
    M hw/audio/es1370.c
    M hw/audio/intel-hda.c
    M hw/block/nvme.c
    M hw/block/pflash_cfi01.c
    M hw/block/pflash_cfi02.c
    M hw/char/serial-pci.c
    M hw/i386/kvm/pci-assign.c
    M hw/ide/ahci.c
    M hw/ide/cmd646.c
    M hw/ide/piix.c
    M hw/ide/via.c
    M hw/ipack/tpci200.c
    M hw/misc/ivshmem.c
    M hw/misc/pci-testdev.c
    M hw/misc/vfio.c
    M hw/net/e1000.c
    M hw/net/eepro100.c
    M hw/net/ne2000.c
    M hw/net/pcnet-pci.c
    M hw/net/rtl8139.c
    M hw/net/stellaris_enet.c
    M hw/net/vmxnet3.c
    M hw/pci-bridge/pci_bridge_dev.c
    M hw/pci/msix.c
    M hw/pci/pci.c
    M hw/pci/pci_bridge.c
    M hw/pci/pcie_host.c
    M hw/pci/shpc.c
    M hw/scsi/esp-pci.c
    M hw/scsi/lsi53c895a.c
    M hw/scsi/megasas.c
    M hw/scsi/vmw_pvscsi.c
    M hw/tpm/tpm_tis.c
    M hw/usb/hcd-uhci.c
    M hw/virtio/virtio-pci.c
    M hw/watchdog/wdt_i6300esb.c
    M hw/xen/xen_pt.c
    M hw/xen/xen_pt_msi.c
    M include/exec/memory.h
    M memory.c

  Log Message:
  -----------
  memory: remove memory_region_destroy

The function is empty after the previous patch, so remove it.

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: c54779f9623ba189fc79f249f8ee1309b73320db
      
https://github.com/qemu/qemu/commit/c54779f9623ba189fc79f249f8ee1309b73320db
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M hw/tpm/tpm_tis.c

  Log Message:
  -----------
  tpm_tis: remove instance_finalize callback

It is never used, since ISA device are not hot-unpluggable.

Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 401cf7fdc4b9f1971ee74927511391a66af0627e
      
https://github.com/qemu/qemu/commit/401cf7fdc4b9f1971ee74927511391a66af0627e
  Author: Peter Crosthwaite <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M hw/core/loader.c

  Log Message:
  -----------
  loader: Abstract away ref to memory region names

Use the function provided rather than spying on the struct.

Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 83234bf2fa4b28feb2accee823c56f2188eed48c
      
https://github.com/qemu/qemu/commit/83234bf2fa4b28feb2accee823c56f2188eed48c
  Author: Peter Crosthwaite <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec: Abstract away ref to memory region names

Use the function provided rather than spying on the struct.

Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 5d546d4b65a3751172286e299528de15c5f05576
      
https://github.com/qemu/qemu/commit/5d546d4b65a3751172286e299528de15c5f05576
  Author: Peter Crosthwaite <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M include/exec/memory.h
    M memory.c

  Log Message:
  -----------
  memory: constify memory_region_name

It doesn't change the MR and some prospective call sites will have
const MRs at hand.

Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 3fb18b4da7a545321e38b4a59f38995630a8d752
      
https://github.com/qemu/qemu/commit/3fb18b4da7a545321e38b4a59f38995630a8d752
  Author: Peter Crosthwaite <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M memory.c

  Log Message:
  -----------
  memory: Use memory_region_name for name access

Despite being local to memory.c, use the helper function. This prepares
support for fully QOMifiying the name field of MR (which will remove
this state from MR completely).

Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: b0225c2c0d89200a29dc3d0b59d2e87a79cbaeb8
      
https://github.com/qemu/qemu/commit/b0225c2c0d89200a29dc3d0b59d2e87a79cbaeb8
  Author: Peter Crosthwaite <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M include/exec/memory.h
    M memory.c

  Log Message:
  -----------
  memory: Use canonical path component as the name

Rather than having the name as separate state. This prepares support
for creating a MemoryRegion dynamically (i.e. without
memory_region_init() and friends) and the MemoryRegion still getting
a usable name.

Signed-off-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: f54bb15f9d373877954e44db3a8bb368aff45b42
      
https://github.com/qemu/qemu/commit/f54bb15f9d373877954e44db3a8bb368aff45b42
  Author: Paolo Bonzini <address@hidden>
  Date:   2014-08-18 (Mon, 18 Aug 2014)

  Changed paths:
    M memory.c

  Log Message:
  -----------
  mtree: remove write-only field

ml->printed is never set to true.

Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 0e4a77370594c91dd126f9872893ed473374cc72
      
https://github.com/qemu/qemu/commit/0e4a77370594c91dd126f9872893ed473374cc72
  Author: Peter Maydell <address@hidden>
  Date:   2014-08-19 (Tue, 19 Aug 2014)

  Changed paths:
    M backends/hostmem.c
    M docs/memory.txt
    M exec.c
    M hw/audio/ac97.c
    M hw/audio/es1370.c
    M hw/audio/intel-hda.c
    M hw/block/nvme.c
    M hw/block/pflash_cfi01.c
    M hw/block/pflash_cfi02.c
    M hw/char/serial-pci.c
    M hw/core/loader.c
    M hw/core/sysbus.c
    M hw/display/vga.c
    M hw/display/vga_int.h
    M hw/i386/kvm/pci-assign.c
    M hw/i386/kvmvapic.c
    M hw/ide/ahci.c
    M hw/ide/cmd646.c
    M hw/ide/piix.c
    M hw/ide/via.c
    M hw/ipack/tpci200.c
    M hw/mips/gt64xxx_pci.c
    M hw/misc/ivshmem.c
    M hw/misc/omap_gpmc.c
    M hw/misc/pci-testdev.c
    M hw/misc/vfio.c
    M hw/net/dp8393x.c
    M hw/net/e1000.c
    M hw/net/eepro100.c
    M hw/net/mcf_fec.c
    M hw/net/ne2000.c
    M hw/net/pcnet-pci.c
    M hw/net/rtl8139.c
    M hw/net/stellaris_enet.c
    M hw/net/vmxnet3.c
    M hw/pci-bridge/pci_bridge_dev.c
    M hw/pci/msix.c
    M hw/pci/pci.c
    M hw/pci/pci_bridge.c
    M hw/pci/pcie_host.c
    M hw/pci/shpc.c
    M hw/ppc/ppc4xx_devs.c
    M hw/scsi/esp-pci.c
    M hw/scsi/lsi53c895a.c
    M hw/scsi/megasas.c
    M hw/scsi/scsi-bus.c
    M hw/scsi/scsi-disk.c
    M hw/scsi/scsi-generic.c
    M hw/scsi/virtio-scsi.c
    M hw/scsi/vmw_pvscsi.c
    M hw/tpm/tpm_tis.c
    M hw/usb/hcd-uhci.c
    M hw/virtio/virtio-pci.c
    M hw/watchdog/wdt_i6300esb.c
    M hw/xen/xen_pt.c
    M hw/xen/xen_pt_msi.c
    M include/exec/memory.h
    M include/hw/scsi/scsi.h
    M include/hw/sysbus.h
    M ioport.c
    M memory.c
    M qom/object.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging

SCSI changes that enable sending vendor-specific commands via virtio-scsi.

Memory changes for QOMification and automatic tracking of MR lifetime.

# gpg: Signature made Mon 18 Aug 2014 13:03:09 BST using RSA key ID 9B4D86F2
# gpg: Good signature from "Paolo Bonzini <address@hidden>"
# gpg:                 aka "Paolo Bonzini <address@hidden>"

* remotes/bonzini/tags/for-upstream:
  mtree: remove write-only field
  memory: Use canonical path component as the name
  memory: Use memory_region_name for name access
  memory: constify memory_region_name
  exec: Abstract away ref to memory region names
  loader: Abstract away ref to memory region names
  tpm_tis: remove instance_finalize callback
  memory: remove memory_region_destroy
  memory: convert memory_region_destroy to object_unparent
  ioport: split deletion and destruction
  nic: do not destroy memory regions in cleanup functions
  vga: do not dynamically allocate chain4_alias
  sysbus: remove unused function sysbus_del_io
  qom: object: move unparenting to the child property's release callback
  qom: object: delete properties before calling instance_finalize
  virtio-scsi: implement parse_cdb
  scsi-block, scsi-generic: implement parse_cdb
  scsi-block: extract scsi_block_is_passthrough
  scsi-bus: introduce parse_cdb in SCSIDeviceClass and SCSIBusInfo
  scsi-bus: prepare scsi_req_new for introduction of parse_cdb

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/8e6e2c2ae7a8...0e4a77370594

reply via email to

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