From 776fe81ba5335e6f220a039ef613e1e33a71b4f0 Mon Sep 17 00:00:00 2001 From: David Gibson Date: Wed, 9 Sep 2015 02:03:04 +0200 Subject: spapr: Reduce advertised max LUNs for spapr_vscsi Patchwork-id: 67711 O-Subject: [RHEL7.2 qemu-kvm-rhev PATCH] spapr: Reduce advertised max LUNs for spapr_vscsi Bugzilla: 1260464 RH-Acked-by: Miroslav Rezanina RH-Acked-by: Laurent Vivier RH-Acked-by: Thomas Huth The implementation of the PAPR paravirtual SCSI adapter currently allows up to 32 LUNs (max_lun == 31). However the adapter isn't really designed to support lots of devices - the PowerVM implementation only ever puts one disk per vSCSI controller. More specifically, the Linux guest side vscsi driver (the only one we really care about) is hardcoded to allow a maximum of 8 LUNs. So, reduce the number of LUNs on the qemu side to 8, so we don't falsely advertise that more LUNs can work. Signed-off-by: David Gibson Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1260464 Upstream: Posted, not merged. Trivial enough applying downstream only shouldn't be too bad. Brew: http://brewweb.devel.redhat.com/brew/taskinfo?taskID=9812826 Signed-off-by: David Gibson Signed-off-by: Miroslav Rezanina --- hw/scsi/spapr_vscsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c index b00edf7..b88695c 100644 --- a/hw/scsi/spapr_vscsi.c +++ b/hw/scsi/spapr_vscsi.c @@ -1194,7 +1194,7 @@ static const struct SCSIBusInfo vscsi_scsi_info = { .tcq = true, .max_channel = 7, /* logical unit addressing format */ .max_target = 63, - .max_lun = 31, + .max_lun = 7, .transfer_data = vscsi_transfer_data, .complete = vscsi_command_complete, -- 2.5.5