[v1] vz: build fix

[libvirt] [PATCH] vz: build fix

Posted by Nikolay Shirokovskiy 6 years ago

Broken by [1] commit - trailing comma instead of semicolon. Fortunately
the issue did not get sneak in released 4.2 version. Note that uriSchemes
for parallelsConnectDriver should not be allocated on stack.

[1] 8e4f9a27: "driver: declare supported URI schemes in virConnectDriver struct"
---
 src/vz/vz_driver.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
index 4c30ee1..a9ee773 100644
--- a/src/vz/vz_driver.c
+++ b/src/vz/vz_driver.c
@@ -4163,7 +4163,11 @@ static virStateDriver vzStateDriver = {
 
 /* Parallels domain type backward compatibility*/
 static virHypervisorDriver parallelsHypervisorDriver;
-static virConnectDriver parallelsConnectDriver;
+static virConnectDriver parallelsConnectDriver = {
+    .localOnly = true,
+    .uriSchemes = (const char *[]){ "parallels", NULL },
+    .hypervisorDriver = &parallelsHypervisorDriver,
+};
 
 /**
  * vzRegister:
@@ -4186,9 +4190,6 @@ vzRegister(void)
     /* Backward compatibility with Parallels domain type */
     parallelsHypervisorDriver = vzHypervisorDriver;
     parallelsHypervisorDriver.name = "Parallels";
-    parallelsConnectDriver = vzConnectDriver;
-    parallelsConnectDriver.hypervisorDriver = &parallelsHypervisorDriver;
-    parallelsConnectDriver.uriSchemes = (const char *[]){ "parallels", NULL },
     if (virRegisterConnectDriver(&parallelsConnectDriver, true) < 0)
         return -1;
 
-- 
1.8.3.1

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

Re: [libvirt] [PATCH] vz: build fix

Posted by Daniel P. Berrangé 6 years ago

On Wed, Apr 18, 2018 at 04:45:31PM +0300, Nikolay Shirokovskiy wrote:
> Broken by [1] commit - trailing comma instead of semicolon. Fortunately
> the issue did not get sneak in released 4.2 version. Note that uriSchemes
> for parallelsConnectDriver should not be allocated on stack.
> 
> [1] 8e4f9a27: "driver: declare supported URI schemes in virConnectDriver struct"
> ---
>  src/vz/vz_driver.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


> 
> diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
> index 4c30ee1..a9ee773 100644
> --- a/src/vz/vz_driver.c
> +++ b/src/vz/vz_driver.c
> @@ -4163,7 +4163,11 @@ static virStateDriver vzStateDriver = {
>  
>  /* Parallels domain type backward compatibility*/
>  static virHypervisorDriver parallelsHypervisorDriver;
> -static virConnectDriver parallelsConnectDriver;
> +static virConnectDriver parallelsConnectDriver = {
> +    .localOnly = true,
> +    .uriSchemes = (const char *[]){ "parallels", NULL },
> +    .hypervisorDriver = &parallelsHypervisorDriver,
> +};
>  
>  /**
>   * vzRegister:
> @@ -4186,9 +4190,6 @@ vzRegister(void)
>      /* Backward compatibility with Parallels domain type */
>      parallelsHypervisorDriver = vzHypervisorDriver;
>      parallelsHypervisorDriver.name = "Parallels";
> -    parallelsConnectDriver = vzConnectDriver;
> -    parallelsConnectDriver.hypervisorDriver = &parallelsHypervisorDriver;
> -    parallelsConnectDriver.uriSchemes = (const char *[]){ "parallels", NULL },
>      if (virRegisterConnectDriver(&parallelsConnectDriver, true) < 0)
>          return -1;
>  
> -- 
> 1.8.3.1
> 
> --
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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

Re: [libvirt] [PATCH] vz: build fix

Posted by Nikolay Shirokovskiy 6 years ago


On 18.04.2018 17:03, Daniel P. Berrangé wrote:
> On Wed, Apr 18, 2018 at 04:45:31PM +0300, Nikolay Shirokovskiy wrote:
>> Broken by [1] commit - trailing comma instead of semicolon. Fortunately
>> the issue did not get sneak in released 4.2 version. Note that uriSchemes
>> for parallelsConnectDriver should not be allocated on stack.
>>
>> [1] 8e4f9a27: "driver: declare supported URI schemes in virConnectDriver struct"
>> ---
>>  src/vz/vz_driver.c | 9 +++++----
>>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> 
> 
>>
>> diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
>> index 4c30ee1..a9ee773 100644
>> --- a/src/vz/vz_driver.c
>> +++ b/src/vz/vz_driver.c
>> @@ -4163,7 +4163,11 @@ static virStateDriver vzStateDriver = {
>>  
>>  /* Parallels domain type backward compatibility*/
>>  static virHypervisorDriver parallelsHypervisorDriver;
>> -static virConnectDriver parallelsConnectDriver;
>> +static virConnectDriver parallelsConnectDriver = {
>> +    .localOnly = true,
>> +    .uriSchemes = (const char *[]){ "parallels", NULL },
>> +    .hypervisorDriver = &parallelsHypervisorDriver,
>> +};
>>  
>>  /**
>>   * vzRegister:
>> @@ -4186,9 +4190,6 @@ vzRegister(void)
>>      /* Backward compatibility with Parallels domain type */
>>      parallelsHypervisorDriver = vzHypervisorDriver;
>>      parallelsHypervisorDriver.name = "Parallels";
>> -    parallelsConnectDriver = vzConnectDriver;
>> -    parallelsConnectDriver.hypervisorDriver = &parallelsHypervisorDriver;
>> -    parallelsConnectDriver.uriSchemes = (const char *[]){ "parallels", NULL },
>>      if (virRegisterConnectDriver(&parallelsConnectDriver, true) < 0)
>>          return -1;
>>  
>> -- 
>> 1.8.3.1
>>
>> --
>> libvir-list mailing list
>> libvir-list@redhat.com
>> https://www.redhat.com/mailman/listinfo/libvir-list
> 
> Regards,
> Daniel
> 

I'll resend soon. It turns out patch breaks 'make check'.

Nikolay

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