Re: [PATCH v3 3/3] Parallels: implemented domainAttachDevice

[Maxim Nestratov <mnestratov@xxxxxxxxxxxxx>]

27.03.2015 15:34, Alexander Burluka пишет:
> That function uses domainAttachDeviceFlags
>
> Signed-off-by: Alexander Burluka <aburluka@xxxxxxxxxxxxx>
> ---
>   src/parallels/parallels_driver.c |    7 +++++++
>   1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c
> index 8058903..632f614 100644
> --- a/src/parallels/parallels_driver.c
> +++ b/src/parallels/parallels_driver.c
> @@ -1068,6 +1068,12 @@ static int parallelsDomainAttachDeviceFlags(virDomainPtr dom, const char *xml,
>       return ret;
>   }
>   
> +static int parallelsDomainAttachDevice(virDomainPtr dom, const char *xml)
> +{
> +    return parallelsDomainAttachDeviceFlags(dom, xml,
> +                                            VIR_DOMAIN_AFFECT_LIVE);
> +}
> +
Since you've started to demand VIR_DOMAIN_AFFECT_CONFIG it is necessary 
here too.
>   static virHypervisorDriver parallelsDriver = {
>       .name = "Parallels",
>       .connectOpen = parallelsConnectOpen,            /* 0.10.0 */
> @@ -1102,6 +1108,7 @@ static virHypervisorDriver parallelsDriver = {
>       .domainDefineXMLFlags = parallelsDomainDefineXMLFlags, /* 1.2.12 */
>       .domainUndefine = parallelsDomainUndefine, /* 1.2.10 */
>       .domainUndefineFlags = parallelsDomainUndefineFlags, /* 1.2.10 */
> +    .domainAttachDevice = parallelsDomainAttachDevice, /* 1.2.15 */
>       .domainAttachDeviceFlags = parallelsDomainAttachDeviceFlags, /* 1.2.15 */
>       .domainIsActive = parallelsDomainIsActive, /* 1.2.10 */
>       .connectDomainEventRegisterAny = parallelsConnectDomainEventRegisterAny, /* 1.2.10 */

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list