mirror of https://github.com/xqemu/xqemu.git
pc-bios/s390-ccw Really big EAV ECKD DASD handling
For EAV ECKD DASD, the cylinder count will have the magic value 0xfffeU. Therefore, use the block number to test for valid eckd addresses instead. Signed-off-by: Eugene (jno) Dvurechenski <jno@linux.vnet.ibm.com> Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> Reviewed-by: David Hildenbrand <dahi@linux.vnet.ibm.com> Signed-off-by: Jens Freimann <jfrei@linux.vnet.ibm.com> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
This commit is contained in:
parent
b0885f7599
commit
f04db28b86
|
@ -93,25 +93,6 @@ static inline void verify_boot_info(BootInfo *bip)
|
|||
"Bad block size in zIPL section of the 1st record.");
|
||||
}
|
||||
|
||||
static bool eckd_valid_address(BootMapPointer *p)
|
||||
{
|
||||
const uint64_t cylinder = p->eckd.cylinder
|
||||
+ ((p->eckd.head & 0xfff0) << 12);
|
||||
const uint64_t head = p->eckd.head & 0x000f;
|
||||
|
||||
if (head >= virtio_get_heads()
|
||||
|| p->eckd.sector > virtio_get_sectors()
|
||||
|| p->eckd.sector <= 0) {
|
||||
return false;
|
||||
}
|
||||
|
||||
if (!virtio_guessed_disk_nature() && cylinder >= virtio_get_cylinders()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
static block_number_t eckd_block_num(BootMapPointer *p)
|
||||
{
|
||||
const uint64_t sectors = virtio_get_sectors();
|
||||
|
@ -126,6 +107,24 @@ static block_number_t eckd_block_num(BootMapPointer *p)
|
|||
return block;
|
||||
}
|
||||
|
||||
static bool eckd_valid_address(BootMapPointer *p)
|
||||
{
|
||||
const uint64_t head = p->eckd.head & 0x000f;
|
||||
|
||||
if (head >= virtio_get_heads()
|
||||
|| p->eckd.sector > virtio_get_sectors()
|
||||
|| p->eckd.sector <= 0) {
|
||||
return false;
|
||||
}
|
||||
|
||||
if (!virtio_guessed_disk_nature() &&
|
||||
eckd_block_num(p) >= virtio_get_blocks()) {
|
||||
return false;
|
||||
}
|
||||
|
||||
return true;
|
||||
}
|
||||
|
||||
static block_number_t load_eckd_segments(block_number_t blk, uint64_t *address)
|
||||
{
|
||||
block_number_t block_nr;
|
||||
|
|
|
@ -340,11 +340,6 @@ int virtio_get_block_size(void)
|
|||
return blk_cfg.blk_size << blk_cfg.physical_block_exp;
|
||||
}
|
||||
|
||||
uint16_t virtio_get_cylinders(void)
|
||||
{
|
||||
return blk_cfg.geometry.cylinders;
|
||||
}
|
||||
|
||||
uint8_t virtio_get_heads(void)
|
||||
{
|
||||
return blk_cfg.geometry.heads;
|
||||
|
@ -355,6 +350,12 @@ uint8_t virtio_get_sectors(void)
|
|||
return blk_cfg.geometry.sectors;
|
||||
}
|
||||
|
||||
uint64_t virtio_get_blocks(void)
|
||||
{
|
||||
return blk_cfg.capacity /
|
||||
(virtio_get_block_size() / VIRTIO_SECTOR_SIZE);
|
||||
}
|
||||
|
||||
void virtio_setup_block(struct subchannel_id schid)
|
||||
{
|
||||
struct vq_info_block info;
|
||||
|
|
|
@ -192,9 +192,9 @@ extern bool virtio_disk_is_scsi(void);
|
|||
extern bool virtio_disk_is_eckd(void);
|
||||
extern bool virtio_ipl_disk_is_valid(void);
|
||||
extern int virtio_get_block_size(void);
|
||||
extern uint16_t virtio_get_cylinders(void);
|
||||
extern uint8_t virtio_get_heads(void);
|
||||
extern uint8_t virtio_get_sectors(void);
|
||||
extern uint64_t virtio_get_blocks(void);
|
||||
extern int virtio_read_many(ulong sector, void *load_addr, int sec_num);
|
||||
|
||||
#define VIRTIO_SECTOR_SIZE 512
|
||||
|
|
Loading…
Reference in New Issue