Re: [PATCH 01/12] libxl: Move detection of autoballoon to libxl_conf

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Michal Privoznik wrote:
> On 30.08.2013 23:46, Jim Fehlig wrote:
>   
>> Detecting whether or not to autoballoon is configuration related,
>> so move the code to libxl_conf.
>>
>> Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxx>
>> ---
>>  src/libxl/libxl_conf.c   | 22 ++++++++++++++++++++++
>>  src/libxl/libxl_conf.h   |  3 +++
>>  src/libxl/libxl_driver.c | 25 +------------------------
>>  3 files changed, 26 insertions(+), 24 deletions(-)
>>
>> diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c
>> index 8129c7f..f8937a4 100644
>> --- a/src/libxl/libxl_conf.c
>> +++ b/src/libxl/libxl_conf.c
>> @@ -978,6 +978,28 @@ error:
>>      return -1;
>>  }
>>  
>> +bool
>> +libxlGetAutoballoonConf(libxlDriverPrivatePtr driver)
>> +{
>> +    const libxl_version_info *info;
>> +    regex_t regex;
>> +    int ret;
>> +
>> +    info = libxl_get_version_info(driver->ctx);
>> +    if (!info)
>> +        return true; /* default to on */
>> +
>> +    ret = regcomp(&regex,
>> +            "(^| )dom0_mem=((|min:|max:)[0-9]+[bBkKmMgG]?,?)+($| )",
>> +            REG_NOSUB | REG_EXTENDED);
>> +    if (ret)
>> +        return true;
>>     
>
> Pre-existing, but if we fail to compile the regex, shouldn't we error out?
>   

I think so, especially since patch 7 makes similar changes, e.g. bailing
out on libxl_get_version_info() failures.

I'll send a separate patch to fix this pre-existing issue.

Regards,
Jim

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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]