Re: [Qemu-devel] [PATCH] scsi-disk: Add support for port WWN and index descriptors in VPD page 83h

[Paolo Bonzini <pbonzini@redhat.com>]

Il 19/02/2014 17:28, Roland Dreier ha scritto:
> From: Roland Dreier <roland@purestorage.com>
>
> To make a VM more convincing to my application, it's useful to be able
> to add a port WWN and relative target port index to the descriptors
> returned for VPD page 83h.  Add device properties to allow setting
> these, and return them from INQUIRY commands.
>
> Signed-off-by: Roland Dreier <roland@purestorage.com>

I'm applying to scsi-next, thanks.

Paolo

> ---
>  hw/scsi/scsi-disk.c | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>
> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
> index a8d0f15ebe61..12013a6768a1 100644
> --- a/hw/scsi/scsi-disk.c
> +++ b/hw/scsi/scsi-disk.c
> @@ -75,6 +75,8 @@ struct SCSIDiskState
>      bool media_event;
>      bool eject_request;
>      uint64_t wwn;
> +    uint64_t port_wwn;
> +    uint16_t port_index;
>      uint64_t max_unmap_size;
>      QEMUBH *bh;
>      char *version;
> @@ -617,6 +619,24 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req, uint8_t *outbuf)
>                  stq_be_p(&outbuf[buflen], s->wwn);
>                  buflen += 8;
>              }
> +
> +            if (s->port_wwn) {
> +                outbuf[buflen++] = 0x61; // SAS / Binary
> +                outbuf[buflen++] = 0x93; // PIV / Target port / NAA
> +                outbuf[buflen++] = 0;    // reserved
> +                outbuf[buflen++] = 8;
> +                stq_be_p(&outbuf[buflen], s->port_wwn);
> +                buflen += 8;
> +            }
> +
> +            if (s->port_index) {
> +                outbuf[buflen++] = 0x61; // SAS / Binary
> +                outbuf[buflen++] = 0x94; // PIV / Target port / relative target port
> +                outbuf[buflen++] = 0;    // reserved
> +                outbuf[buflen++] = 4;
> +                stw_be_p(&outbuf[buflen + 2], s->port_index);
> +                buflen += 4;
> +            }
>              break;
>          }
>          case 0xb0: /* block limits */
> @@ -2536,6 +2556,8 @@ static Property scsi_hd_properties[] = {
>      DEFINE_PROP_BIT("dpofua", SCSIDiskState, features,
>                      SCSI_DISK_F_DPOFUA, false),
>      DEFINE_PROP_HEX64("wwn", SCSIDiskState, wwn, 0),
> +    DEFINE_PROP_HEX64("port_wwn", SCSIDiskState, port_wwn, 0),
> +    DEFINE_PROP_UINT16("port_index", SCSIDiskState, port_index, 0),
>      DEFINE_PROP_UINT64("max_unmap_size", SCSIDiskState, max_unmap_size,
>                         DEFAULT_MAX_UNMAP_SIZE),
>      DEFINE_BLOCK_CHS_PROPERTIES(SCSIDiskState, qdev.conf),
> @@ -2584,6 +2606,8 @@ static const TypeInfo scsi_hd_info = {
>  static Property scsi_cd_properties[] = {
>      DEFINE_SCSI_DISK_PROPERTIES(),
>      DEFINE_PROP_HEX64("wwn", SCSIDiskState, wwn, 0),
> +    DEFINE_PROP_HEX64("port_wwn", SCSIDiskState, port_wwn, 0),
> +    DEFINE_PROP_UINT16("port_index", SCSIDiskState, port_index, 0),
>      DEFINE_PROP_END_OF_LIST(),
>  };
>
> @@ -2647,6 +2671,8 @@ static Property scsi_disk_properties[] = {
>      DEFINE_PROP_BIT("dpofua", SCSIDiskState, features,
>                      SCSI_DISK_F_DPOFUA, false),
>      DEFINE_PROP_HEX64("wwn", SCSIDiskState, wwn, 0),
> +    DEFINE_PROP_HEX64("port_wwn", SCSIDiskState, port_wwn, 0),
> +    DEFINE_PROP_UINT16("port_index", SCSIDiskState, port_index, 0),
>      DEFINE_PROP_UINT64("max_unmap_size", SCSIDiskState, max_unmap_size,
>                         DEFAULT_MAX_UNMAP_SIZE),
>      DEFINE_PROP_END_OF_LIST(),
>