Commit dd454839 authored by Jan Höppner's avatar Jan Höppner Committed by Jens Axboe

s390/dasd: Fix error handling during online processing

It is possible that the CCW commands for reading volume and extent pool
information are not supported, either by the storage server (for
dedicated DASDs) or by z/VM (for virtual devices, such as MDISKs).

As a command reject will occur in such a case, the current error
handling leads to a failing online processing and thus the DASD can't be
used at all.

Since the data being read is not essential for an fully operational
DASD, the error handling can be removed. Information about the failing
command is sent to the s390dbf debug feature.

Fixes: c729696b ("s390/dasd: Recognise data for ESE volumes")
Cc: <stable@vger.kernel.org> # 5.3
Reported-by: default avatarFrank Heimes <frank.heimes@canonical.com>
Signed-off-by: default avatarJan Höppner <hoeppner@linux.ibm.com>
Signed-off-by: default avatarStefan Haberland <sth@linux.ibm.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent bdf20073
...@@ -1553,8 +1553,8 @@ static int dasd_eckd_read_vol_info(struct dasd_device *device) ...@@ -1553,8 +1553,8 @@ static int dasd_eckd_read_vol_info(struct dasd_device *device)
if (rc == 0) { if (rc == 0) {
memcpy(&private->vsq, vsq, sizeof(*vsq)); memcpy(&private->vsq, vsq, sizeof(*vsq));
} else { } else {
dev_warn(&device->cdev->dev, DBF_EVENT_DEVID(DBF_WARNING, device->cdev,
"Reading the volume storage information failed with rc=%d\n", rc); "Reading the volume storage information failed with rc=%d", rc);
} }
if (useglobal) if (useglobal)
...@@ -1737,8 +1737,8 @@ static int dasd_eckd_read_ext_pool_info(struct dasd_device *device) ...@@ -1737,8 +1737,8 @@ static int dasd_eckd_read_ext_pool_info(struct dasd_device *device)
if (rc == 0) { if (rc == 0) {
dasd_eckd_cpy_ext_pool_data(device, lcq); dasd_eckd_cpy_ext_pool_data(device, lcq);
} else { } else {
dev_warn(&device->cdev->dev, DBF_EVENT_DEVID(DBF_WARNING, device->cdev,
"Reading the logical configuration failed with rc=%d\n", rc); "Reading the logical configuration failed with rc=%d", rc);
} }
dasd_sfree_request(cqr, cqr->memdev); dasd_sfree_request(cqr, cqr->memdev);
...@@ -2020,14 +2020,10 @@ dasd_eckd_check_characteristics(struct dasd_device *device) ...@@ -2020,14 +2020,10 @@ dasd_eckd_check_characteristics(struct dasd_device *device)
dasd_eckd_read_features(device); dasd_eckd_read_features(device);
/* Read Volume Information */ /* Read Volume Information */
rc = dasd_eckd_read_vol_info(device); dasd_eckd_read_vol_info(device);
if (rc)
goto out_err3;
/* Read Extent Pool Information */ /* Read Extent Pool Information */
rc = dasd_eckd_read_ext_pool_info(device); dasd_eckd_read_ext_pool_info(device);
if (rc)
goto out_err3;
/* Read Device Characteristics */ /* Read Device Characteristics */
rc = dasd_generic_read_dev_chars(device, DASD_ECKD_MAGIC, rc = dasd_generic_read_dev_chars(device, DASD_ECKD_MAGIC,
...@@ -5663,14 +5659,10 @@ static int dasd_eckd_restore_device(struct dasd_device *device) ...@@ -5663,14 +5659,10 @@ static int dasd_eckd_restore_device(struct dasd_device *device)
dasd_eckd_read_features(device); dasd_eckd_read_features(device);
/* Read Volume Information */ /* Read Volume Information */
rc = dasd_eckd_read_vol_info(device); dasd_eckd_read_vol_info(device);
if (rc)
goto out_err2;
/* Read Extent Pool Information */ /* Read Extent Pool Information */
rc = dasd_eckd_read_ext_pool_info(device); dasd_eckd_read_ext_pool_info(device);
if (rc)
goto out_err2;
/* Read Device Characteristics */ /* Read Device Characteristics */
rc = dasd_generic_read_dev_chars(device, DASD_ECKD_MAGIC, rc = dasd_generic_read_dev_chars(device, DASD_ECKD_MAGIC,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment