- 13 4月, 2016 16 次提交
-
-
由 Andrea Bolognani 提交于
virt-host-validate, just like virt-login-shell, doesn't make sense on Windows, so we should avoid building it. Make the tool optional and build it by default on all platforms except Windows, erroring out if the user attempts to build it anyway.
-
由 Andrea Bolognani 提交于
Instead of having separate handling for programs and man pages, deal with both in the same place.
-
由 Peter Krempa 提交于
-
由 Peter Krempa 提交于
Now that the mirror parsing code is not crammed in the main disk parser we can employ better coding style.
-
由 Peter Krempa 提交于
Changes are indentation and 'cleanup' label instead of 'error'.
-
由 Peter Krempa 提交于
Rest of the fields of the iotune data structure did not check for malformed integers. Use the previously defined macro to extract them which will simplify the code and add error reporting.
-
由 Peter Krempa 提交于
Since the structure was pre-initialized to 0 we don't need to set every single member to 0 if it's not present in the XML. Additionally if we put the name of the field into the error message the code can be simplified using a macro to parse the members.
-
由 Peter Krempa 提交于
Since this function isn't doing any cleanup, the label is not necessary.
-
由 Peter Krempa 提交于
-
由 Peter Krempa 提交于
Also simplify the code by switching to a for loop.
-
由 Peter Krempa 提交于
All the fields were initialized to 0.
-
由 Peter Krempa 提交于
-
由 Maxim Nestratov 提交于
-
由 Maxim Nestratov 提交于
No need to remember connection name and have corresponding domain type to keep backward compatibility with former 'parallels' driver. It is enough to be able to accept 'parallels' uri and domain types. Signed-off-by: NMaxim Nestratov <mnestratov@virtuozzo.com>
-
由 Maxim Nestratov 提交于
'vz' goes first now to make clients like virt-manager choose 'vz' instead of 'parallels' Signed-off-by: NMaxim Nestratov <mnestratov@virtuozzo.com>
-
由 Maxim Nestratov 提交于
Signed-off-by: NMaxim Nestratov <mnestratov@virtuozzo.com>
-
- 12 4月, 2016 14 次提交
-
-
由 Nikolay Shirokovskiy 提交于
we don't need to allocate macstr at all as it is an array and already has the the space it needs. Signed-off-by: NNikolay Shirokovskiy <nshirokovskiy@virtuozzo.com>
-
由 Ján Tomko 提交于
-
由 Ján Tomko 提交于
Return early if there is nothing to do.
-
由 Ján Tomko 提交于
If we encounter a video device with primary=yes, we insert it at def->videos[0]. There is no need to record this in a separate variable, just check if there already is a primary video at def->videos[0].
-
由 Ján Tomko 提交于
We call VIR_INSERT_ELEMENT_INPLACE either with 0 (for primary video) or def->nvideos (for the rest). Use a variable with more semantic name, since j is usually used for iterating.
-
由 Ján Tomko 提交于
We start with both i and def->nvideos at 0 and increment both after every successful iteration. Use i directly, instead of passing the def->nvideos value through j.
-
由 Ján Tomko 提交于
Commit 119cd06e started setting the primary bool for the first user-specified video even if user omitted the 'primary' attribute. However this was done before the addition of the implicit device. This broke startup of transient qemu domains with no <video>: https://bugzilla.redhat.com/show_bug.cgi?id=1325757 Move this default to virDomainDefPostParseInternal, after the addition of the implicit video device, to catch the implicit video as well.
-
由 Ján Tomko 提交于
Separate parsing of the XML from auto-generating the device.
-
由 Andrea Bolognani 提交于
Being consistent is nice, especially when it comes to defining our regular expression, where using single quotes instead of double quotes allows us to leave out a few backslashes. Changing this required altering a few error messages. The only remaining use of double quotes is one where they are actually required for the check to work.
-
由 Andrea Bolognani 提交于
-
由 Andrea Bolognani 提交于
-
由 Nikolay Shirokovskiy 提交于
Quite straigthforward as vz sdk memory setting function makes just what we want to that is set "amount of physical memory allocated to a domain". 'useflags' is introduced for non flag function implementation. We can't just use combination of flags like "live | config" or we fail for inactive domains. Other combinations have drawbacks too. Signed-off-by: NNikolay Shirokovskiy <nshirokovskiy@virtuozzo.com>
-
由 Nikolay Shirokovskiy 提交于
Actually this is not pure refactoring. Part of common code is replaced with virDomainObjUpdateModificationImpact and this a good replacement. It includes removed check of inactive domain and active flags set. Additionally we resolve current flag in accordance with current state of domain. Thus it becames possible to attach/detach devices for inactive domains if this flag is set. Signed-off-by: NNikolay Shirokovskiy <nshirokovskiy@virtuozzo.com>
- 11 4月, 2016 9 次提交
-
-
由 Pavel Hrdina 提交于
Commit dc98a5bc refactored the code a lot and forget about checking if listen attribute is specified. This ensures that listen attribute and first listen element are compared only if both exist. Signed-off-by: NPavel Hrdina <phrdina@redhat.com>
-
-
由 Pavel Hrdina 提交于
fdstream.c: In function 'virFDStreamWrite': fdstream.c:390:29: error: logical 'or' of equal expressions [-Werror=logical-op] if (errno == EAGAIN || errno == EWOULDBLOCK) { ^~ Fedora rawhide now uses gcc 6.0 and there is a bug with -Wlogical-op producing false warnings. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69602 Use GCC pragma push/pop and ignore -Wlogical-op for GCC that supports push/pop pragma and also has this bug. Signed-off-by: NPavel Hrdina <phrdina@redhat.com>
-
由 Pavel Hrdina 提交于
Signed-off-by: NPavel Hrdina <phrdina@redhat.com>
-
由 Martin Kletzander 提交于
Commit c3bd0019 forgot to cleanup after itself. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1325043Signed-off-by: NMartin Kletzander <mkletzan@redhat.com>
-
由 Peter Krempa 提交于
Replace the nonsensical debug statement by adding the expected event code into the existing debug statement. Since the monitor code always notifies the agent on guest reboot/shutdown even if that was not initiated by the agent the warning emitted later is bogus and pollutes the logs in such cases. Delete it and keep just the original debug message where this info can be inferred.
-
由 Andrea Bolognani 提交于
Simply checking whether the cgroup name appears somewhere inside /proc/self/cgroup is enough most of the time, but there are some corner cases that require a more mindful parsing.
-
由 Andrea Bolognani 提交于
The check is supposed to stop users from trying to compile virt-login-shell on Windows by erroring out during the configure phase; however, there are two flaws in it: * the value of "x$with_win" is compared to "yes" instead of "xyes" (note the "x" in the first string) * "test" is not being used, so the script will actually try to run a command called "x$with_win" instead of performing string comparison This patch fixes both issues.
-
由 Erik Skultety 提交于
Commit e72667bd tried to fix the incorrect file ordering in po/POTFILES.in except it didn't, since the sort was run using locale en_US instead of using locale C which is the default in libvirt. So this patch hopefully fixes it for good. Signed-off-by: NErik Skultety <eskultet@redhat.com>
-
- 10 4月, 2016 1 次提交
-
-
由 Erik Skultety 提交于
When it comes to a situation that a new translatable file needs to be added into the list of files, an automatically generated patch is proposed during syntax-check. However, the diff was made against a sorted list of files and the same sorted list of files + the new file that actually needs to be added into the list. Since we do not keep POTFILES sorted, the proposed patch thus can't be applied...most of the time - depending on the context. Signed-off-by: NErik Skultety <eskultet@redhat.com>
-