From patchwork Tue Oct 25 10:40:26 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Paolo Bonzini X-Patchwork-Id: 121657 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [140.186.70.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 5A1F6B6F9D for ; Tue, 25 Oct 2011 23:50:07 +1100 (EST) Received: from localhost ([::1]:37855 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RIgS3-00059C-PA for incoming@patchwork.ozlabs.org; Tue, 25 Oct 2011 08:49:55 -0400 Received: from eggs.gnu.org ([140.186.70.92]:55465) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RIeRZ-0000TX-RC for qemu-devel@nongnu.org; Tue, 25 Oct 2011 06:41:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RIeRS-0006Lb-P5 for qemu-devel@nongnu.org; Tue, 25 Oct 2011 06:41:15 -0400 Received: from mail-wy0-f173.google.com ([74.125.82.173]:51141) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RIeRR-0006JE-1C for qemu-devel@nongnu.org; Tue, 25 Oct 2011 06:41:09 -0400 Received: by mail-wy0-f173.google.com with SMTP id 15so379276wyh.4 for ; Tue, 25 Oct 2011 03:41:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=sender:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references; bh=sJhKuDWGu490PRqWzn7xSo4L3WI5ZGodrXpnlwr40Xg=; b=fa53UZ59YIzg2CzfG5UkyNV4rD1sMFQC/MbTRlhiCRHi8W0FtiX2CLdnjmbmY3vleo R7XiDRMec0DevwTGxJ32LI0DBN6aAwqjHjEpz0hpQDIb6PyWje5sjaOTN7Lnr++9h23m X1JTzFsW0AndLBFsmdlnOwxr/Pyn/71i7HWXc= Received: by 10.227.199.5 with SMTP id eq5mr2882939wbb.2.1319539268288; Tue, 25 Oct 2011 03:41:08 -0700 (PDT) Received: from localhost.localdomain (93-34-199-98.ip51.fastwebnet.it. [93.34.199.98]) by mx.google.com with ESMTPS id fr4sm15677349wbb.0.2011.10.25.03.41.07 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 25 Oct 2011 03:41:07 -0700 (PDT) From: Paolo Bonzini To: qemu-devel@nongnu.org Date: Tue, 25 Oct 2011 12:40:26 +0200 Message-Id: <1319539241-26436-19-git-send-email-pbonzini@redhat.com> X-Mailer: git-send-email 1.7.6 In-Reply-To: <1319539241-26436-1-git-send-email-pbonzini@redhat.com> References: <1319539241-26436-1-git-send-email-pbonzini@redhat.com> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 74.125.82.173 Cc: kwolf@redhat.com Subject: [Qemu-devel] [PATCH v2 18/33] scsi: allow arbitrary LUNs X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org This only requires changes in two places: in SCSIBus, we need to look for a free LUN if somebody creates a device with a pre-existing scsi-id but the default LUN (-1, meaning "search for a free spot"); in vSCSI, we need to actually parse the LUN according to the SCSI spec. For vSCSI, max_target/max_lun are set according to the logical unit addressing format in SAM. Signed-off-by: Paolo Bonzini --- hw/esp.c | 3 ++- hw/lsi53c895a.c | 3 ++- hw/scsi-bus.c | 48 ++++++++++++++++++++++++++++++++---------------- hw/scsi.h | 3 ++- hw/spapr_vscsi.c | 39 +++++++++++++++++++++++++++++++++++---- hw/usb-msd.c | 3 ++- 6 files changed, 75 insertions(+), 24 deletions(-) diff --git a/hw/esp.c b/hw/esp.c index aad290f..5742bb3 100644 --- a/hw/esp.c +++ b/hw/esp.c @@ -724,7 +724,8 @@ void esp_init(target_phys_addr_t espaddr, int it_shift, static const struct SCSIBusInfo esp_scsi_info = { .tcq = false, - .ndev = ESP_MAX_DEVS, + .max_target = ESP_MAX_DEVS, + .max_lun = 7, .transfer_data = esp_transfer_data, .complete = esp_command_complete, diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index c15f167..d26e442 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -2085,7 +2085,8 @@ static int lsi_scsi_uninit(PCIDevice *d) static const struct SCSIBusInfo lsi_scsi_info = { .tcq = true, - .ndev = LSI_MAX_DEVS, + .max_target = LSI_MAX_DEVS, + .max_lun = 0, /* LUN support is buggy */ .transfer_data = lsi_transfer_data, .complete = lsi_command_complete, diff --git a/hw/scsi-bus.c b/hw/scsi-bus.c index a2d57a7..cec06db 100644 --- a/hw/scsi-bus.c +++ b/hw/scsi-bus.c @@ -17,7 +17,7 @@ static struct BusInfo scsi_bus_info = { .get_fw_dev_path = scsibus_get_fw_dev_path, .props = (Property[]) { DEFINE_PROP_UINT32("scsi-id", SCSIDevice, id, -1), - DEFINE_PROP_UINT32("lun", SCSIDevice, lun, 0), + DEFINE_PROP_UINT32("lun", SCSIDevice, lun, -1), DEFINE_PROP_END_OF_LIST(), }, }; @@ -37,26 +37,42 @@ static int scsi_qdev_init(DeviceState *qdev, DeviceInfo *base) SCSIDevice *dev = DO_UPCAST(SCSIDevice, qdev, qdev); SCSIDeviceInfo *info = DO_UPCAST(SCSIDeviceInfo, qdev, base); SCSIBus *bus = DO_UPCAST(SCSIBus, qbus, dev->qdev.parent_bus); - SCSIDevice *olddev; + SCSIDevice *d; int rc = -1; - if (dev->id == -1) { - int id; - for (id = 0; id < bus->info->ndev; id++) { - if (!scsi_device_find(bus, id, 0)) { - dev->id = id; - break; - } - } - } - if (dev->id >= bus->info->ndev) { + if (dev->id != -1 && dev->id > bus->info->max_target) { error_report("bad scsi device id: %d", dev->id); goto err; } - olddev = scsi_device_find(bus, dev->id, dev->lun); - if (olddev && dev->lun == olddev->lun) { - qdev_free(&olddev->qdev); + if (dev->id == -1) { + int id = -1; + if (dev->lun == -1) { + dev->lun = 0; + } + do { + d = scsi_device_find(bus, ++id, dev->lun); + } while (d && d->lun == dev->lun && id <= bus->info->max_target); + if (id > bus->info->max_target) { + error_report("no free target"); + goto err; + } + dev->id = id; + } else if (dev->lun == -1) { + int lun = -1; + do { + d = scsi_device_find(bus, dev->id, ++lun); + } while (d && d->lun == lun && lun < bus->info->max_lun); + if (lun > bus->info->max_lun) { + error_report("no free lun"); + goto err; + } + dev->lun = lun; + } else { + d = scsi_device_find(bus, dev->id, dev->lun); + if (dev->lun == d->lun && dev != d) { + qdev_free(&d->qdev); + } } dev->info = info; @@ -115,7 +131,7 @@ int scsi_bus_legacy_handle_cmdline(SCSIBus *bus) int res = 0, unit; loc_push_none(&loc); - for (unit = 0; unit < bus->info->ndev; unit++) { + for (unit = 0; unit < bus->info->max_target; unit++) { dinfo = drive_get(IF_SCSI, bus->busnr, unit); if (dinfo == NULL) { continue; diff --git a/hw/scsi.h b/hw/scsi.h index add3d2d..401d8d3 100644 --- a/hw/scsi.h +++ b/hw/scsi.h @@ -98,7 +98,8 @@ struct SCSIDeviceInfo { }; struct SCSIBusInfo { - int tcq, ndev; + int tcq; + int max_target, max_lun; void (*transfer_data)(SCSIRequest *req, uint32_t arg); void (*complete)(SCSIRequest *req, uint32_t arg); void (*cancel)(SCSIRequest *req); diff --git a/hw/spapr_vscsi.c b/hw/spapr_vscsi.c index c941f28..334b2e6 100644 --- a/hw/spapr_vscsi.c +++ b/hw/spapr_vscsi.c @@ -131,9 +131,39 @@ static void vscsi_put_req(vscsi_req *req) static SCSIDevice *vscsi_device_find(SCSIBus *bus, uint64_t srp_lun, int *lun) { - /* XXX Figure that one out properly ! This is crackpot */ - int id = (srp_lun >> 56) & 0x7f; - *lun = (srp_lun >> 48) & 0xff; + int channel = 0, id = 0; + +retry: + switch (srp_lun >> 62) { + case 0: + if ((srp_lun >> 56) != 0) { + channel = (srp_lun >> 56) & 0x3f; + id = (srp_lun >> 48) & 0xff; + srp_lun <<= 16; + goto retry; + } + *lun = (srp_lun >> 48) & 0xff; + break; + + case 1: + *lun = (srp_lun >> 48) & 0x3fff; + break; + case 2: + channel = (srp_lun >> 53) & 0x7; + id = (srp_lun >> 56) & 0x3f; + *lun = (srp_lun >> 48) & 0x1f; + break; + case 3: + *lun = -1; + return NULL; + default: + abort(); + } + + if (channel) { + *lun = -1; + return NULL; + } return scsi_device_find(bus, id, *lun); } @@ -862,7 +892,8 @@ static int vscsi_do_crq(struct VIOsPAPRDevice *dev, uint8_t *crq_data) static const struct SCSIBusInfo vscsi_scsi_info = { .tcq = true, - .ndev = VSCSI_REQ_LIMIT, + .max_target = 63, /* logical unit addressing format */ + .max_lun = 31, .transfer_data = vscsi_transfer_data, .complete = vscsi_command_complete, diff --git a/hw/usb-msd.c b/hw/usb-msd.c index 2c047fa..1a0815a 100644 --- a/hw/usb-msd.c +++ b/hw/usb-msd.c @@ -497,7 +497,8 @@ static void usb_msd_password_cb(void *opaque, int err) static const struct SCSIBusInfo usb_msd_scsi_info = { .tcq = false, - .ndev = 1, + .max_target = 0, + .max_lun = 0, .transfer_data = usb_msd_transfer_data, .complete = usb_msd_command_complete,