Blame SOURCES/kvm-pc-bios-s390-ccw-Do-not-bail-out-early-if-not-findin.patch

a19a21
From 6f44767aeda52048e7c9ee4b5fcc30353c71cbc1 Mon Sep 17 00:00:00 2001
a19a21
From: Thomas Huth <thuth@redhat.com>
a19a21
Date: Fri, 9 Oct 2020 10:08:45 -0400
a19a21
Subject: [PATCH 09/14] pc-bios/s390-ccw: Do not bail out early if not finding
a19a21
 a SCSI disk
a19a21
a19a21
RH-Author: Thomas Huth <thuth@redhat.com>
a19a21
Message-id: <20201009100849.264994-6-thuth@redhat.com>
a19a21
Patchwork-id: 98599
a19a21
O-Subject: [RHEL-8.4.0 qemu-kvm PATCH 5/9] pc-bios/s390-ccw: Do not bail out early if not finding a SCSI disk
a19a21
Bugzilla: 1846975
a19a21
RH-Acked-by: Jens Freimann <jfreimann@redhat.com>
a19a21
RH-Acked-by: David Hildenbrand <david@redhat.com>
a19a21
RH-Acked-by: Cornelia Huck <cohuck@redhat.com>
a19a21
a19a21
In case the user did not specify a boot device, we want to continue
a19a21
looking for other devices if there are no valid SCSI disks on a virtio-
a19a21
scsi controller. As a first step, do not panic in this case and let
a19a21
the control flow carry the error to the upper functions instead.
a19a21
a19a21
Message-Id: <20200806105349.632-6-thuth@redhat.com>
a19a21
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
a19a21
Signed-off-by: Thomas Huth <thuth@redhat.com>
a19a21
(cherry picked from commit 605751b5a5334e187761b0b8a8266a216897bf70)
a19a21
Signed-off-by: Danilo C. L. de Paula <ddepaula@redhat.com>
a19a21
---
a19a21
 pc-bios/s390-ccw/main.c          | 14 ++++++++++----
a19a21
 pc-bios/s390-ccw/s390-ccw.h      |  2 +-
a19a21
 pc-bios/s390-ccw/virtio-blkdev.c |  7 +++++--
a19a21
 pc-bios/s390-ccw/virtio-scsi.c   | 28 ++++++++++++++++++++--------
a19a21
 pc-bios/s390-ccw/virtio-scsi.h   |  2 +-
a19a21
 5 files changed, 37 insertions(+), 16 deletions(-)
a19a21
a19a21
diff --git a/pc-bios/s390-ccw/main.c b/pc-bios/s390-ccw/main.c
a19a21
index d6fd218074..456733fbee 100644
a19a21
--- a/pc-bios/s390-ccw/main.c
a19a21
+++ b/pc-bios/s390-ccw/main.c
a19a21
@@ -227,7 +227,7 @@ static void find_boot_device(void)
a19a21
     IPL_assert(found, "Boot device not found\n");
a19a21
 }
a19a21
 
a19a21
-static void virtio_setup(void)
a19a21
+static int virtio_setup(void)
a19a21
 {
a19a21
     VDev *vdev = virtio_get_device();
a19a21
     QemuIplParameters *early_qipl = (QemuIplParameters *)QIPL_ADDRESS;
a19a21
@@ -242,9 +242,14 @@ static void virtio_setup(void)
a19a21
         sclp_print("Network boot device detected\n");
a19a21
         vdev->netboot_start_addr = qipl.netboot_start_addr;
a19a21
     } else {
a19a21
-        virtio_blk_setup_device(blk_schid);
a19a21
+        int ret = virtio_blk_setup_device(blk_schid);
a19a21
+        if (ret) {
a19a21
+            return ret;
a19a21
+        }
a19a21
         IPL_assert(virtio_ipl_disk_is_valid(), "No valid IPL device detected");
a19a21
     }
a19a21
+
a19a21
+    return 0;
a19a21
 }
a19a21
 
a19a21
 static void ipl_boot_device(void)
a19a21
@@ -255,8 +260,9 @@ static void ipl_boot_device(void)
a19a21
         dasd_ipl(blk_schid, cutype); /* no return */
a19a21
         break;
a19a21
     case CU_TYPE_VIRTIO:
a19a21
-        virtio_setup();
a19a21
-        zipl_load(); /* no return */
a19a21
+        if (virtio_setup() == 0) {
a19a21
+            zipl_load(); /* no return */
a19a21
+        }
a19a21
         break;
a19a21
     default:
a19a21
         print_int("Attempting to boot from unexpected device type", cutype);
a19a21
diff --git a/pc-bios/s390-ccw/s390-ccw.h b/pc-bios/s390-ccw/s390-ccw.h
a19a21
index ae432c40b8..e7cf36eb91 100644
a19a21
--- a/pc-bios/s390-ccw/s390-ccw.h
a19a21
+++ b/pc-bios/s390-ccw/s390-ccw.h
a19a21
@@ -70,7 +70,7 @@ int sclp_read(char *str, size_t count);
a19a21
 unsigned long virtio_load_direct(ulong rec_list1, ulong rec_list2,
a19a21
                                  ulong subchan_id, void *load_addr);
a19a21
 bool virtio_is_supported(SubChannelId schid);
a19a21
-void virtio_blk_setup_device(SubChannelId schid);
a19a21
+int virtio_blk_setup_device(SubChannelId schid);
a19a21
 int virtio_read(ulong sector, void *load_addr);
a19a21
 u64 get_clock(void);
a19a21
 ulong get_second(void);
a19a21
diff --git a/pc-bios/s390-ccw/virtio-blkdev.c b/pc-bios/s390-ccw/virtio-blkdev.c
a19a21
index 11c56261ca..7d35050292 100644
a19a21
--- a/pc-bios/s390-ccw/virtio-blkdev.c
a19a21
+++ b/pc-bios/s390-ccw/virtio-blkdev.c
a19a21
@@ -263,9 +263,10 @@ uint64_t virtio_get_blocks(void)
a19a21
     return 0;
a19a21
 }
a19a21
 
a19a21
-void virtio_blk_setup_device(SubChannelId schid)
a19a21
+int virtio_blk_setup_device(SubChannelId schid)
a19a21
 {
a19a21
     VDev *vdev = virtio_get_device();
a19a21
+    int ret = 0;
a19a21
 
a19a21
     vdev->schid = schid;
a19a21
     virtio_setup_ccw(vdev);
a19a21
@@ -288,9 +289,11 @@ void virtio_blk_setup_device(SubChannelId schid)
a19a21
             "Config: CDB size mismatch");
a19a21
 
a19a21
         sclp_print("Using virtio-scsi.\n");
a19a21
-        virtio_scsi_setup(vdev);
a19a21
+        ret = virtio_scsi_setup(vdev);
a19a21
         break;
a19a21
     default:
a19a21
         panic("\n! No IPL device available !\n");
a19a21
     }
a19a21
+
a19a21
+    return ret;
a19a21
 }
a19a21
diff --git a/pc-bios/s390-ccw/virtio-scsi.c b/pc-bios/s390-ccw/virtio-scsi.c
a19a21
index 4fe4b9d261..88691edb89 100644
a19a21
--- a/pc-bios/s390-ccw/virtio-scsi.c
a19a21
+++ b/pc-bios/s390-ccw/virtio-scsi.c
a19a21
@@ -192,7 +192,12 @@ static bool scsi_read_capacity(VDev *vdev,
a19a21
 
a19a21
 /* virtio-scsi routines */
a19a21
 
a19a21
-static void virtio_scsi_locate_device(VDev *vdev)
a19a21
+/*
a19a21
+ * Tries to locate a SCSI device and and adds the information for the found
a19a21
+ * device to the vdev->scsi_device structure.
a19a21
+ * Returns 0 if SCSI device could be located, or a error code < 0 otherwise
a19a21
+ */
a19a21
+static int virtio_scsi_locate_device(VDev *vdev)
a19a21
 {
a19a21
     const uint16_t channel = 0; /* again, it's what QEMU does */
a19a21
     uint16_t target;
a19a21
@@ -218,7 +223,7 @@ static void virtio_scsi_locate_device(VDev *vdev)
a19a21
         IPL_check(sdev->channel == 0, "non-zero channel requested");
a19a21
         IPL_check(sdev->target <= vdev->config.scsi.max_target, "target# high");
a19a21
         IPL_check(sdev->lun <= vdev->config.scsi.max_lun, "LUN# high");
a19a21
-        return;
a19a21
+        return 0;
a19a21
     }
a19a21
 
a19a21
     for (target = 0; target <= vdev->config.scsi.max_target; target++) {
a19a21
@@ -245,18 +250,20 @@ static void virtio_scsi_locate_device(VDev *vdev)
a19a21
              */
a19a21
             sdev->lun = r->lun[0].v16[0]; /* it's returned this way */
a19a21
             debug_print_int("Have to use LUN", sdev->lun);
a19a21
-            return; /* we have to use this device */
a19a21
+            return 0; /* we have to use this device */
a19a21
         }
a19a21
         for (i = 0; i < luns; i++) {
a19a21
             if (r->lun[i].v64) {
a19a21
                 /* Look for non-zero LUN - we have where to choose from */
a19a21
                 sdev->lun = r->lun[i].v16[0];
a19a21
                 debug_print_int("Will use LUN", sdev->lun);
a19a21
-                return; /* we have found a device */
a19a21
+                return 0; /* we have found a device */
a19a21
             }
a19a21
         }
a19a21
     }
a19a21
-    panic("\n! Cannot locate virtio-scsi device !\n");
a19a21
+
a19a21
+    sclp_print("Warning: Could not locate a usable virtio-scsi device\n");
a19a21
+    return -ENODEV;
a19a21
 }
a19a21
 
a19a21
 int virtio_scsi_read_many(VDev *vdev,
a19a21
@@ -320,17 +327,20 @@ static void scsi_parse_capacity_report(void *data,
a19a21
     }
a19a21
 }
a19a21
 
a19a21
-void virtio_scsi_setup(VDev *vdev)
a19a21
+int virtio_scsi_setup(VDev *vdev)
a19a21
 {
a19a21
     int retry_test_unit_ready = 3;
a19a21
     uint8_t data[256];
a19a21
     uint32_t data_size = sizeof(data);
a19a21
     ScsiInquiryEvpdPages *evpd = &scsi_inquiry_evpd_pages_response;
a19a21
     ScsiInquiryEvpdBl *evpd_bl = &scsi_inquiry_evpd_bl_response;
a19a21
-    int i;
a19a21
+    int i, ret;
a19a21
 
a19a21
     vdev->scsi_device = &default_scsi_device;
a19a21
-    virtio_scsi_locate_device(vdev);
a19a21
+    ret = virtio_scsi_locate_device(vdev);
a19a21
+    if (ret < 0) {
a19a21
+        return ret;
a19a21
+    }
a19a21
 
a19a21
     /* We have to "ping" the device before it becomes readable */
a19a21
     while (!scsi_test_unit_ready(vdev)) {
a19a21
@@ -415,4 +425,6 @@ void virtio_scsi_setup(VDev *vdev)
a19a21
     }
a19a21
     scsi_parse_capacity_report(data, &vdev->scsi_last_block,
a19a21
                                (uint32_t *) &vdev->scsi_block_size);
a19a21
+
a19a21
+    return 0;
a19a21
 }
a19a21
diff --git a/pc-bios/s390-ccw/virtio-scsi.h b/pc-bios/s390-ccw/virtio-scsi.h
a19a21
index 4c4f4bbc31..4b14c2c2f9 100644
a19a21
--- a/pc-bios/s390-ccw/virtio-scsi.h
a19a21
+++ b/pc-bios/s390-ccw/virtio-scsi.h
a19a21
@@ -67,7 +67,7 @@ static inline bool virtio_scsi_response_ok(const VirtioScsiCmdResp *r)
a19a21
         return r->response == VIRTIO_SCSI_S_OK && r->status == CDB_STATUS_GOOD;
a19a21
 }
a19a21
 
a19a21
-void virtio_scsi_setup(VDev *vdev);
a19a21
+int virtio_scsi_setup(VDev *vdev);
a19a21
 int virtio_scsi_read_many(VDev *vdev,
a19a21
                           ulong sector, void *load_addr, int sec_num);
a19a21
 
a19a21
-- 
a19a21
2.27.0
a19a21