Since it is an abbreviation, SCSI should always be fully capitalized or full lower case, never Scsi. Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx> --- src/conf/domain_conf.c | 8 ++++---- src/conf/domain_conf.h | 2 +- src/conf/node_device_conf.c | 12 ++++++------ src/conf/node_device_conf.h | 2 +- src/node_device/node_device_udev.c | 4 ++-- src/qemu/qemu_capabilities.c | 12 ++++++------ src/qemu/qemu_cgroup.c | 4 ++-- src/qemu/qemu_command.c | 12 ++++++------ src/vmx/vmx.c | 2 +- 9 files changed, 29 insertions(+), 29 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 109705b..c59b4fb 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -4028,7 +4028,7 @@ out: } static int -virDomainHostdevSubsysScsiDefParseXML(xmlNodePtr node, +virDomainHostdevSubsysSCSIDefParseXML(xmlNodePtr node, virDomainHostdevDefPtr def) { int ret = -1; @@ -4230,7 +4230,7 @@ virDomainHostdevAssignAddress(virDomainXMLOptionPtr xmlopt, controller++; ret = virDomainControllerSCSINextUnit(def, - xmlopt->config.hasWideScsiBus ? + xmlopt->config.hasWideSCSIBus ? SCSI_WIDE_BUS_MAX_CONT_UNIT : SCSI_NARROW_BUS_MAX_CONT_UNIT, def->controllers[i]->idx); @@ -4352,7 +4352,7 @@ virDomainHostdevDefParseXMLSubsys(xmlNodePtr node, break; case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_SCSI: - if (virDomainHostdevSubsysScsiDefParseXML(sourcenode, def) < 0) + if (virDomainHostdevSubsysSCSIDefParseXML(sourcenode, def) < 0) goto error; break; @@ -4497,7 +4497,7 @@ virDomainDiskDefAssignAddress(virDomainXMLOptionPtr xmlopt, case VIR_DOMAIN_DISK_BUS_SCSI: def->info.type = VIR_DOMAIN_DEVICE_ADDRESS_TYPE_DRIVE; - if (xmlopt->config.hasWideScsiBus) { + if (xmlopt->config.hasWideSCSIBus) { /* For a wide SCSI bus we define the default mapping to be * 16 units per bus, 1 bus per controller, many controllers. * Unit 7 is the SCSI controller itself. Therefore unit 7 diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 37191a8..2fd2656 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -2196,7 +2196,7 @@ struct _virDomainDefParserConfig { virFreeCallback privFree; /* data */ - bool hasWideScsiBus; + bool hasWideSCSIBus; unsigned char macPrefix[VIR_MAC_PREFIX_BUFLEN]; }; diff --git a/src/conf/node_device_conf.c b/src/conf/node_device_conf.c index d86bbed..9e94092 100644 --- a/src/conf/node_device_conf.c +++ b/src/conf/node_device_conf.c @@ -645,7 +645,7 @@ out: } static int -virNodeDevCapScsiParseXML(xmlXPathContextPtr ctxt, +virNodeDevCapSCSIParseXML(xmlXPathContextPtr ctxt, virNodeDeviceDefPtr def, xmlNodePtr node, union _virNodeDevCapData *data) @@ -690,7 +690,7 @@ out: static int -virNodeDevCapScsiTargetParseXML(xmlXPathContextPtr ctxt, +virNodeDevCapSCSITargetParseXML(xmlXPathContextPtr ctxt, virNodeDeviceDefPtr def, xmlNodePtr node, union _virNodeDevCapData *data) @@ -718,7 +718,7 @@ out: static int -virNodeDevCapScsiHostParseXML(xmlXPathContextPtr ctxt, +virNodeDevCapSCSIHostParseXML(xmlXPathContextPtr ctxt, virNodeDeviceDefPtr def, xmlNodePtr node, union _virNodeDevCapData *data, @@ -1181,16 +1181,16 @@ virNodeDevCapsDefParseXML(xmlXPathContextPtr ctxt, ret = virNodeDevCapNetParseXML(ctxt, def, node, &caps->data); break; case VIR_NODE_DEV_CAP_SCSI_HOST: - ret = virNodeDevCapScsiHostParseXML(ctxt, def, node, + ret = virNodeDevCapSCSIHostParseXML(ctxt, def, node, &caps->data, create, virt_type); break; case VIR_NODE_DEV_CAP_SCSI_TARGET: - ret = virNodeDevCapScsiTargetParseXML(ctxt, def, node, &caps->data); + ret = virNodeDevCapSCSITargetParseXML(ctxt, def, node, &caps->data); break; case VIR_NODE_DEV_CAP_SCSI: - ret = virNodeDevCapScsiParseXML(ctxt, def, node, &caps->data); + ret = virNodeDevCapSCSIParseXML(ctxt, def, node, &caps->data); break; case VIR_NODE_DEV_CAP_STORAGE: ret = virNodeDevCapStorageParseXML(ctxt, def, node, &caps->data); diff --git a/src/conf/node_device_conf.h b/src/conf/node_device_conf.h index 26116a1..9b5d172 100644 --- a/src/conf/node_device_conf.h +++ b/src/conf/node_device_conf.h @@ -69,7 +69,7 @@ enum virNodeDevStorageCapFlags { VIR_NODE_DEV_CAP_STORAGE_HOTPLUGGABLE = (1 << 2), }; -enum virNodeDevScsiHostCapFlags { +enum virNodeDevSCSIHostCapFlags { VIR_NODE_DEV_CAP_FLAG_HBA_FC_HOST = (1 << 0), VIR_NODE_DEV_CAP_FLAG_HBA_VPORT_OPS = (1 << 1), }; diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c index 5d49968..8c4eb90 100644 --- a/src/node_device/node_device_udev.c +++ b/src/node_device/node_device_udev.c @@ -1141,7 +1141,7 @@ out: } static int -udevProcessScsiGeneric(struct udev_device *dev, +udevProcessSCSIGeneric(struct udev_device *dev, virNodeDeviceDefPtr def) { if (udevGetStringProperty(dev, @@ -1256,7 +1256,7 @@ static int udevGetDeviceDetails(struct udev_device *device, ret = udevProcessStorage(device, def); break; case VIR_NODE_DEV_CAP_SCSI_GENERIC: - ret = udevProcessScsiGeneric(device, def); + ret = udevProcessSCSIGeneric(device, def); break; default: VIR_ERROR(_("Unknown device type %d"), def->caps->type); diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c index c589a57..66e8448 100644 --- a/src/qemu/qemu_capabilities.c +++ b/src/qemu/qemu_capabilities.c @@ -1441,7 +1441,7 @@ static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsVfioPci[] = { { "bootindex", QEMU_CAPS_VFIO_PCI_BOOTINDEX }, }; -static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsScsiDisk[] = { +static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsSCSIDisk[] = { { "channel", QEMU_CAPS_SCSI_DISK_CHANNEL }, { "wwn", QEMU_CAPS_SCSI_DISK_WWN }, }; @@ -1464,7 +1464,7 @@ static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsUsbHost[] = { { "bootindex", QEMU_CAPS_USB_HOST_BOOTINDEX }, }; -static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsScsiGeneric[] = { +static struct virQEMUCapsStringFlags virQEMUCapsObjectPropsSCSIGeneric[] = { { "bootindex", QEMU_CAPS_DEVICE_SCSI_GENERIC_BOOTINDEX }, }; @@ -1509,8 +1509,8 @@ static struct virQEMUCapsObjectTypeProps virQEMUCapsObjectProps[] = { ARRAY_CARDINALITY(virQEMUCapsObjectPropsPciAssign) }, { "vfio-pci", virQEMUCapsObjectPropsVfioPci, ARRAY_CARDINALITY(virQEMUCapsObjectPropsVfioPci) }, - { "scsi-disk", virQEMUCapsObjectPropsScsiDisk, - ARRAY_CARDINALITY(virQEMUCapsObjectPropsScsiDisk) }, + { "scsi-disk", virQEMUCapsObjectPropsSCSIDisk, + ARRAY_CARDINALITY(virQEMUCapsObjectPropsSCSIDisk) }, { "ide-drive", virQEMUCapsObjectPropsIDEDrive, ARRAY_CARDINALITY(virQEMUCapsObjectPropsIDEDrive) }, { "PIIX4_PM", virQEMUCapsObjectPropsPixx4PM, @@ -1519,8 +1519,8 @@ static struct virQEMUCapsObjectTypeProps virQEMUCapsObjectProps[] = { ARRAY_CARDINALITY(virQEMUCapsObjectPropsUsbRedir) }, { "usb-host", virQEMUCapsObjectPropsUsbHost, ARRAY_CARDINALITY(virQEMUCapsObjectPropsUsbHost) }, - { "scsi-generic", virQEMUCapsObjectPropsScsiGeneric, - ARRAY_CARDINALITY(virQEMUCapsObjectPropsScsiGeneric) }, + { "scsi-generic", virQEMUCapsObjectPropsSCSIGeneric, + ARRAY_CARDINALITY(virQEMUCapsObjectPropsSCSIGeneric) }, { "i440FX-pcihost", virQEMUCapsObjectPropsI440FXPciHost, ARRAY_CARDINALITY(virQEMUCapsObjectPropsI440FXPciHost) }, { "q35-pcihost", virQEMUCapsObjectPropsQ35PciHost, diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c index a97f184..48a51f0 100644 --- a/src/qemu/qemu_cgroup.c +++ b/src/qemu/qemu_cgroup.c @@ -200,7 +200,7 @@ qemuSetupHostUsbDeviceCgroup(virUSBDevicePtr dev ATTRIBUTE_UNUSED, } static int -qemuSetupHostScsiDeviceCgroup(virSCSIDevicePtr dev ATTRIBUTE_UNUSED, +qemuSetupHostSCSIDeviceCgroup(virSCSIDevicePtr dev ATTRIBUTE_UNUSED, const char *path, void *opaque) { @@ -301,7 +301,7 @@ qemuSetupHostdevCGroup(virDomainObjPtr vm, goto cleanup; if (virSCSIDeviceFileIterate(scsi, - qemuSetupHostScsiDeviceCgroup, + qemuSetupHostSCSIDeviceCgroup, vm) < 0) goto cleanup; diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index aa1a3db..0f546f5 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_command.c @@ -688,7 +688,7 @@ static int qemuAssignDeviceDiskAliasFixed(virDomainDiskDefPtr disk) } static int -qemuSetScsiControllerModel(virDomainDefPtr def, +qemuSetSCSIControllerModel(virDomainDefPtr def, virQEMUCapsPtr qemuCaps, int *model) { @@ -760,7 +760,7 @@ qemuAssignDeviceDiskAliasCustom(virDomainDefPtr def, virDomainDeviceFindControllerModel(def, &disk->info, VIR_DOMAIN_CONTROLLER_TYPE_SCSI); - if ((qemuSetScsiControllerModel(def, qemuCaps, &controllerModel)) < 0) + if ((qemuSetSCSIControllerModel(def, qemuCaps, &controllerModel)) < 0) return -1; } @@ -1422,7 +1422,7 @@ int qemuDomainAssignSpaprVIOAddresses(virDomainDefPtr def, for (i = 0; i < def->ncontrollers; i++) { model = def->controllers[i]->model; if (def->controllers[i]->type == VIR_DOMAIN_CONTROLLER_TYPE_SCSI) { - if (qemuSetScsiControllerModel(def, qemuCaps, &model) < 0) + if (qemuSetSCSIControllerModel(def, qemuCaps, &model) < 0) goto cleanup; } @@ -4407,7 +4407,7 @@ qemuBuildDriveDevStr(virDomainDefPtr def, controllerModel = virDomainDeviceFindControllerModel(def, &disk->info, VIR_DOMAIN_CONTROLLER_TYPE_SCSI); - if ((qemuSetScsiControllerModel(def, qemuCaps, &controllerModel)) < 0) + if ((qemuSetSCSIControllerModel(def, qemuCaps, &controllerModel)) < 0) goto error; if (controllerModel == VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSILOGIC) { @@ -4812,7 +4812,7 @@ qemuBuildControllerDevStr(virDomainDefPtr domainDef, switch (def->type) { case VIR_DOMAIN_CONTROLLER_TYPE_SCSI: model = def->model; - if ((qemuSetScsiControllerModel(domainDef, qemuCaps, &model)) < 0) + if ((qemuSetSCSIControllerModel(domainDef, qemuCaps, &model)) < 0) return NULL; switch (model) { @@ -5838,7 +5838,7 @@ qemuBuildSCSIHostdevDevStr(virDomainDefPtr def, model = virDomainDeviceFindControllerModel(def, dev->info, VIR_DOMAIN_CONTROLLER_TYPE_SCSI); - if (qemuSetScsiControllerModel(def, qemuCaps, &model) < 0) + if (qemuSetSCSIControllerModel(def, qemuCaps, &model) < 0) goto error; if (model == VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LSILOGIC) { diff --git a/src/vmx/vmx.c b/src/vmx/vmx.c index 8fb2a93..7cc135d 100644 --- a/src/vmx/vmx.c +++ b/src/vmx/vmx.c @@ -522,7 +522,7 @@ VIR_ENUM_IMPL(virVMXControllerModelSCSI, VIR_DOMAIN_CONTROLLER_MODEL_SCSI_LAST, */ virDomainDefParserConfig virVMXDomainDefParserConfig = { - .hasWideScsiBus = true, + .hasWideSCSIBus = true, .macPrefix = {0x00, 0x0c, 0x29}, }; -- 1.8.5.3 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list