Re: [libvirt] [PATCH 1/2] conf: Support for Block Device IO Limits

Introducing a new iolimits element allowing to override certain properties of a guest block device like the physical and logical block size. This can be useful for platforms with 'non-standard' disk formats like S390 DASD with its 4K block size. Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com&gt; --- docs/formatdomain.html.in | 18 +++++++++++++++ docs/schemas/domaincommon.rng | 17 ++++++++++++++ src/conf/domain_conf.c | 49 +++++++++++++++++++++++++++++++++++++++++ src/conf/domain_conf.h | 5 ++++ 4 files changed, 89 insertions(+), 0 deletions(-)

+++ b/src/conf/domain_conf.c @@ -3398,6 +3398,8 @@ virDomainDiskDefParseXML(virCapsPtr caps, char *authUUID = NULL; char *usageType = NULL; char *tray = NULL; + char *logical_block_size = NULL; + char *physical_block_size = NULL; if (VIR_ALLOC(def) < 0) { virReportOOMError(); @@ -3409,6 +3411,9 @@ virDomainDiskDefParseXML(virCapsPtr caps, def->geometry.sectors = 0; def->geometry.trans = VIR_DOMAIN_DISK_TRANS_DEFAULT; + def->iolimits.logical_block_size = 0; + def->iolimits.physical_block_size = 0;

+ physical_block_size = + virXMLPropString(cur, "physical_block_size"); + if (physical_block_size && + virStrToLong_ui(physical_block_size, NULL, 0, + &def->iolimits.physical_block_size) < 0) { + virReportError(VIR_ERR_INTERNAL_ERROR, + _("invalid physical block size '%s'"), + physical_block_size);