[libvirt] [rebased][PATCH 1/4] add pci-bridge controller type

Ján Tomko jtomko at redhat.com
Tue Feb 5 16:53:30 UTC 2013


On 01/30/13 03:30, liguang wrote:
> @@ -4493,6 +4494,8 @@ virDomainControllerDefParseXML(xmlNodePtr node,
>          goto error;
>  
>      switch (def->type) {
> +    case VIR_DOMAIN_CONTROLLER_TYPE_PCIBRIDGE:
> +        break;
>      case VIR_DOMAIN_CONTROLLER_TYPE_VIRTIO_SERIAL: {
>          char *ports = virXMLPropString(node, "ports");
>          if (ports) {

This hunk doesn't do anything and can be dropped.

> diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
> index 9a9e0b1..a93562d 100644
> --- a/src/conf/domain_conf.h
> +++ b/src/conf/domain_conf.h
> @@ -661,6 +661,7 @@ enum virDomainControllerType {
>      VIR_DOMAIN_CONTROLLER_TYPE_VIRTIO_SERIAL,
>      VIR_DOMAIN_CONTROLLER_TYPE_CCID,
>      VIR_DOMAIN_CONTROLLER_TYPE_USB,
> +    VIR_DOMAIN_CONTROLLER_TYPE_PCIBRIDGE,
>  
>      VIR_DOMAIN_CONTROLLER_TYPE_LAST
>  };
> 
VIR_DOMAIN_CONTROLLER_TYPE_PCI_BRIDGE would be nicer. Or maybe just
VIR_DOMAIN_CONTROLLER_TYPE_PCI would do.

Jan




More information about the libvir-list mailing list