pc-bios/s390-ccw/virtio-blkdev: Remove virtio_assume_scsi()
commit5447de2619050a0a4dd480b97f88a9b58da360d1
authorThomas Huth <thuth@redhat.com>
Mon, 4 Jul 2022 11:18:56 +0000 (4 13:18 +0200)
committerThomas Huth <thuth@redhat.com>
Wed, 6 Jul 2022 16:34:13 +0000 (6 18:34 +0200)
tree7bcc682d4de8f6c46ea967e49f06424f2b4f029a
parentbbf615f7b707f009ef8e757d170902ad33b90644
pc-bios/s390-ccw/virtio-blkdev: Remove virtio_assume_scsi()

The virtio_assume_scsi() function is very questionable: First, it
is only called for virtio-blk, and not for virtio-scsi, so the naming
is already quite confusing. Second, it is called if we detected a
"invalid" IPL disk, trying to fix it by blindly setting a sector
size of 512. This of course won't work in most cases since disks
might have a different sector size for a reason.

Thus let's remove this strange function now. The calling code can
also be removed completely, since there is another spot in main.c
that does "IPL_assert(virtio_ipl_disk_is_valid(), ...)" to make
sure that we do not try to IPL from an invalid device.

Message-Id: <20220704111903.62400-6-thuth@redhat.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
pc-bios/s390-ccw/virtio-blkdev.c
pc-bios/s390-ccw/virtio.h