[Qemu-commits] [qemu/qemu] c4ce4c: scsi: Document intentional fall through in scsi_re...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: c4ce4c4b1f28095947d72abb0fb7de6c8d8efdb3
      
https://github.com/qemu/qemu/commit/c4ce4c4b1f28095947d72abb0fb7de6c8d8efdb3
  Author: Markus Armbruster <address@hidden>
  Date:   2014-05-16 (Fri, 16 May 2014)

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

  Log Message:
  -----------
  scsi: Document intentional fall through in scsi_req_length()

For clarity, and to hush up Coverity.

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


  Commit: 91e7fcca4743cf694eb0c8e7a8d938cf359b5bd8
      
https://github.com/qemu/qemu/commit/91e7fcca4743cf694eb0c8e7a8d938cf359b5bd8
  Author: Markus Armbruster <address@hidden>
  Date:   2014-05-16 (Fri, 16 May 2014)

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

  Log Message:
  -----------
  virtio-scsi: Plug memory leak on virtio_scsi_push_event() error path

Spotted by Coverity.

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


  Commit: f2564d88fe7949bdb5ca4ad652282c28b3da6210
      
https://github.com/qemu/qemu/commit/f2564d88fe7949bdb5ca4ad652282c28b3da6210
  Author: Jeff Cody <address@hidden>
  Date:   2014-05-20 (Tue, 20 May 2014)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  block: iscsi build fix if LIBISCSI_FEATURE_IOVECTOR is not defined

Commit b03c380 introduced the function
iscsi_allocationmap_is_allocated(), however it is only used within a
code block that is conditionally compiled.  This produces a warning
(error with -werror) of "defined but not used" for the the function, if
LIBISCSI_FEATURE_IOVECTOR is not defined.

This wraps iscsi_allocationmap_is_allocated() in the same conditional.

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


  Commit: 5a7733b0b728bb4900bdeed12fef22651ce0ab58
      
https://github.com/qemu/qemu/commit/5a7733b0b728bb4900bdeed12fef22651ce0ab58
  Author: Olaf Hering <address@hidden>
  Date:   2014-05-20 (Tue, 20 May 2014)

  Changed paths:
    M hw/scsi/megasas.c

  Log Message:
  -----------
  megasas: remove buildtime strings

Using __DATE__ or __TIME__ in binary pkgs changes the checksum of
compiled binaries if they get rebuilt, even if there are no other
source changes.  Replace the dynamic strings with some equally
informative static strings.

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


  Commit: 65903a8b0807dbe2983910060f5754d27762faed
      
https://github.com/qemu/qemu/commit/65903a8b0807dbe2983910060f5754d27762faed
  Author: Peter Maydell <address@hidden>
  Date:   2014-05-22 (Thu, 22 May 2014)

  Changed paths:
    M block/iscsi.c
    M hw/scsi/megasas.c
    M hw/scsi/scsi-bus.c
    M hw/scsi/virtio-scsi.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/bonzini/scsi-next' into staging

* remotes/bonzini/scsi-next:
  megasas: remove buildtime strings
  block: iscsi build fix if LIBISCSI_FEATURE_IOVECTOR is not defined
  virtio-scsi: Plug memory leak on virtio_scsi_push_event() error path
  scsi: Document intentional fall through in scsi_req_length()

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


Compare: https://github.com/qemu/qemu/compare/ca8c0fab9504...65903a8b0807