On Wed, Feb 25, 2015 at 11:38:46AM +0100, Raymond Durand wrote:
Thanks.
Is it possible to enable/disable which parameters are triggered by Nova on
Libvirt? ie.
-device virtio-balloon
-pci-device isa-serial
No, these are a standard part of Nova
Regards,
Daniel
--
|: http://berrange.com
Zhe Peng wrote:
$vmm-get_all_domain_stats
Sys::Virt::GET_ALL_STATS_ACTIVE
Sys::Virt::GET_ALL_STATS_INACTIVE
Sys::Virt::GET_ALL_STATS_OTHER
Sys::Virt::GET_ALL_STATS_PAUSED
Sys::Virt::GET_ALL_STATS_PERSISTENT
Sys::Virt::GET_ALL_STATS_RUNNING
Sys::Virt::GET_ALL_STATS_SHUTOFF
Ok, thanks.
So as soon as a device is directly assigned to a VM, setrlimit() or
prlimit() are applied to the qemu-kvm process to avoid paging at the host
level, right?
and if there is not enough physical RAM, is the device simply not connected
or is the VM halted?
And the VM itself can page
Ok thanks.
Regards,
2015-03-05 20:45 GMT+01:00 Daniel P. Berrange berra...@redhat.com:
On Wed, Feb 25, 2015 at 11:38:46AM +0100, Raymond Durand wrote:
Thanks.
Is it possible to enable/disable which parameters are triggered by Nova
on
Libvirt? ie.
-device virtio-balloon
-pci-device
Zhe Peng wrote:
Signed-off-by: Hao Liu h...@redhat.com
---
scripts/domain/275-open-graphics-fd.t | 76
+++
1 file changed, 76 insertions(+)
create mode 100644 scripts/domain/275-open-graphics-fd.t
diff --git a/scripts/domain/275-open-graphics-fd.t
On 03/05/2015 09:01 AM, Ján Tomko wrote:
This can be used to disable the ROM.
---
docs/formatdomain.html.in | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
ACK
diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 6276a61..050476f 100644
---
On 03/05/2015 07:45 AM, Daniel P. Berrange wrote:
On Mon, Jan 26, 2015 at 12:08:46AM +0530, Nehal J Wani wrote:
Define helper function virDomainInterfaceFree, which allows
the upper layer application to free the domain interface object
conveniently.
The API is going to provide multiple
Add qemuDomainGetIOThreadPin to handle getting the CPU Affinity for
one specific IOThread
Add qemuDomainPinIOThread to handle setting the cpuset for a specific
IOThread
Signed-off-by: John Ferlan jfer...@redhat.com
---
include/libvirt/libvirt-domain.h | 9 ++
src/qemu/qemu_driver.c
https://bugzilla.redhat.com/show_bug.cgi?id=1135491
More or less a virtual copy of the existing virDomainVcpuPin{Add|Del} API's.
NB: The IOThreads implementation reused the virDomainVcpuPinDefPtr
since it provided everything necessary - an id and a map for each
thread id configured.
Add the 'iothreadsinfo' command to display IOThread Info data. Allow for
[--live] or [--config] options in order to display live or config data
for an active domain.
$ virsh iothreadsinfo --help
NAME
iothreadsinfo - view domain IOThreads
SYNOPSIS
iothreadsinfo domain [--config]
Add virDomainGetIOThreadPin to fetch the pinned CPU affinity map
for one IOThread.
Add virDomainPinIOThread to allow setting the CPU affinity for a
specific IOThread.
Signed-off-by: John Ferlan jfer...@redhat.com
---
include/libvirt/libvirt-domain.h | 10 +++
src/driver-hypervisor.h |
Depending on the flags passed, either attempt to return the active/live
IOThread data for the domain or the config data.
The active/live path will call into the Monitor in order to get the
IOThread data and then correlate the thread_id's returned from the
monitor to the currently running
v3:
http://www.redhat.com/archives/libvir-list/2015-February/msg00593.html
Changes since v3: (from code reviews)
- Updated to use 1.2.14 not 1.2.13
- Change struct from virDomainIOThreadsInfo to virDomainIOThreadInfo
- Remove the resources and nresources from virDomainIOThreadInfo
and of
Implement the remote plumbing to get an IOThread's pinned CPU data or
to allow pinning an IOThread to a specific CPU set
Signed-off-by: John Ferlan jfer...@redhat.com
---
daemon/remote.c | 49 ++
src/remote/remote_driver.c | 62
https://bugzilla.redhat.com/show_bug.cgi?id=1135491
$ virsh iothread --help
NAME
iothreadpin - control or query domain IOThread affinity
SYNOPSIS
iothreadpin domain iothread [--cpulist string] [--config] [--live]
[--current]
DESCRIPTION
Pin domain IOThreads to host physical
On Thu, Mar 05, 2015 at 07:45:51AM -0500, John Ferlan wrote:
+unsigned char *cpumap; /* CPU map for thread */
+int cpumaplen; /* cpumap size */
+size_t nresources; /* count of resources using
IOThread */
+char
On Thu, Mar 05, 2015 at 12:05:16 +0100, Michal Privoznik wrote:
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
For this patch this isn't entirely true,
On Thu, Mar 05, 2015 at 12:05:17 +0100, Michal Privoznik wrote:
This is going to be needed later, when some functions needs to avoid
calling multiple times at once. It will work like this:
1) gain the object list mutex
2) find the object to work on
3) do the work
4) release the mutex
As
On Tue, Feb 17, 2015 at 04:03:54PM -0500, John Ferlan wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1135491
Add the libvirt API infrastructure to support setting IOThread data.
For now this is the pinned CPU information, but eventually will also
support hot plug add/del
The API will
On Thu, Mar 05, 2015 at 12:05:17 +0100, Michal Privoznik wrote:
This is going to be needed later, when some functions needs to avoid
calling multiple times at once. It will work like this:
1) gain the object list mutex
2) find the object to work on
3) do the work
4) release the mutex
As
This can be used to disable the ROM.
---
docs/formatdomain.html.in | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 6276a61..050476f 100644
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -4265,7
On Thu, Mar 05, 2015 at 05:21:41PM +0100, Ján Tomko wrote:
On Thu, Mar 05, 2015 at 07:45:51AM -0500, John Ferlan wrote:
+unsigned char *cpumap; /* CPU map for thread */
+int cpumaplen; /* cpumap size */
+size_t nresources;
On Wed, Mar 04, 2015 at 08:34:26PM +, Daniel P. Berrange wrote:
On Thu, Feb 19, 2015 at 10:19:22PM +0100, Marek Marczykowski-Górecki wrote:
On Thu, Feb 19, 2015 at 01:45:52PM -0700, Jim Fehlig wrote:
Marek Marczykowski-Górecki wrote:
Xen have feature of having device model in
On Thu, Mar 05, 2015 at 07:51:33AM -0500, John Ferlan wrote:
+for (i = 0; i targetDef-iothreads; i++) {
+if (VIR_ALLOC(info_ret[i]) 0)
+goto cleanup;
+
+/* IOThreads being counting at 1 */
+info_ret[i]-iothread_id = i + 1;
+
+if
On Thu, Feb 19, 2015 at 07:59:39AM -0500, John Ferlan wrote:
Support the libvirt_virDomainSetIOThreads method using code that mimics
the existing libvirt_virDomainPinVcpuFlags method
The following is a sample session assuming guest 'iothr-gst' has IOThreads
configured (it's currently
Michal Privoznik wrote:
On 04.03.2015 01:09, Jim Fehlig wrote:
This series contains a few simple changes to the libxl driver.
Patches 1 and 2 were included in
https://www.redhat.com/archives/libvir-list/2015-February/msg00611.html
To make an upcoming V2 of that series easier to review,
On Wed, Mar 04, 2015 at 17:17:06 +0100, Pavel Hrdina wrote:
Using this macro will ensure that the value stored in domain def will
never be greater than VIR_DOMAIN_MEMORY_PARAM_UNLIMITED.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/util/virutil.h | 4
1 file changed, 4
On Wed, Mar 04, 2015 at 17:17:05 +0100, Pavel Hrdina wrote:
Commit message is too sparse.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1146539
Also I doubt that this on itself resolves this bug.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
tools/virsh-domain.c | 62
The case will validate the getAllDomainStats API in class virConnect
---
repos/virconn/connection_getAllDomainStats.py | 528 ++
1 file changed, 528 insertions(+)
create mode 100644 repos/virconn/connection_getAllDomainStats.py
diff --git
The testing case will validate the getAllDomainStats API in class virConnect
jiahu (2):
Add connection_getAllDomainStats test case to linux_domain.conf
Add connection_getAllDomainStats test case
cases/linux_domain.conf | 6 +
---
cases/linux_domain.conf | 6 ++
1 file changed, 6 insertions(+)
diff --git a/cases/linux_domain.conf b/cases/linux_domain.conf
index a5ada35..d3d6aa5 100644
--- a/cases/linux_domain.conf
+++ b/cases/linux_domain.conf
@@ -34,6 +34,12 @@ domain:start
guestname
$defaultname
On Thu, Mar 05, 2015 at 12:05:13 +0100, Michal Privoznik wrote:
This is practically copy of qemuDomObjEndAPI. The reason why is
it so widely available is to avoid code duplication, since the
function is going to be called from our bridge driver, test
driver and parallels driver too.
On 03.03.2015 15:40, Stefan Berger wrote:
Pass the TPM file descriptor to QEMU via command line.
Instead of passing /dev/tpm0 we now pass /dev/fdset/10 and the additional
parameters -add-fd set=10,fd=20.
This addresses the use case when QEMU is started with non-root privileges
and QEMU
On Thu, Mar 05, 2015 at 13:26:09 +0100, Peter Krempa wrote:
Use correct package names too as they differ.
---
libvirt.spec.in | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
ACK
Jirka
--
libvir-list mailing list
libvir-list@redhat.com
On Thu, Mar 05, 2015 at 12:05:02 +0100, Michal Privoznik wrote:
This API will be used in the future to call passed callback over
each network object in the list. It's slightly different to its
virDomainObjListForEach counterpart, because virDomainObjList
uses a hash table to store domain
We have something like pvpanic device. However, in some cases it does
not have any address assigned, in which case we produce this ugly XML
(still valid though):
devices
emulator/usr/bin/qemu/emulator
...
panic
/panic
/devices
Lets format pvpanic/ instead.
Signed-off-by:
On 04.03.2015 10:06, Peter Krempa wrote:
Since the APIs support just one element per namespace and while
modifying an element all duplicates would be removed, let's do this
right away in the post parse callback.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1190590
---
On Thu, Mar 05, 2015 at 12:05:04 +0100, Michal Privoznik wrote:
An accessor following pattern laid out by virDomainObjList* APIs.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
ACK,
Peter
signature.asc
Description: Digital signature
--
libvir-list mailing list
On Thu, Mar 05, 2015 at 12:05:03 +0100, Michal Privoznik wrote:
An accessor following pattern laid out by virDomainObjList* APIs.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 38 ++
src/conf/network_conf.h | 8
On Wed, Mar 04, 2015 at 17:17:07 +0100, Pavel Hrdina wrote:
There was a mess in the way how we store unlimited value for memory
limits and how we handled values provided by user. Internally there
were two possible ways how to store unlimited value: as 0 value or as
$dom-set_metadata
$dom-get_metadata
only have title and destription
element not support in libvirt now
Signed-off-by: Zhe Peng zp...@redhat.com
---
scripts/domain/500-metadata.t | 72 +++
1 file changed, 72 insertions(+)
create mode 100644
Signed-off-by: Hao Liu h...@redhat.com
---
scripts/domain/275-open-graphics-fd.t | 76 +++
1 file changed, 76 insertions(+)
create mode 100644 scripts/domain/275-open-graphics-fd.t
diff --git a/scripts/domain/275-open-graphics-fd.t
The first one is to truncate the memory limit to
VIR_DOMAIN_MEMORY_PARAM_UNLIMITED if the value is greater and the second
one is to decide whether the memory limit is set or not, unlimited means
that it's not set.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/libvirt_private.syms | 2
s/host/hypervisor/ to match the wording used by the C binding.
https://bugzilla.redhat.com/show_bug.cgi?id=1198518
---
libvirt-override-api.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Pushed as trivial.
diff --git a/libvirt-override-api.xml b/libvirt-override-api.xml
index
There was a mess in the way how we store unlimited value for memory
limits and how we handled values provided by user. Internally there
were two possible ways how to store unlimited value: as 0 value or as
VIR_DOMAIN_MEMORY_PARAM_UNLIMITED. Because we chose to store memory
limits as unsigned
The first patch rewrites virsh memtune command to accept 0 as valid value
instead of ignoring it.
In the second patch I'll introduce a simple helper to crop the *_limit values
to VIR_DOMAIN_MEMORY_PARAM_UNLIMITED as its used several times while reading
values from cgroups and also a helper to
On Thu, Mar 05, 2015 at 12:05:12 +0100, Michal Privoznik wrote:
Later we can turn APIs to lock the object if needed instead of
relying on caller to mutually exclude itself (probably done by
locking a big lock anyway).
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
On Thu, Mar 05, 2015 at 15:03:22 +0100, Michal Privoznik wrote:
On 04.03.2015 10:06, Peter Krempa wrote:
Since the APIs support just one element per namespace and while
modifying an element all duplicates would be removed, let's do this
right away in the post parse callback.
Resolves:
On Thu, Mar 05, 2015 at 16:20:55 +0100, Jiri Denemark wrote:
On Thu, Mar 05, 2015 at 13:26:09 +0100, Peter Krempa wrote:
Use correct package names too as they differ.
---
libvirt.spec.in | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
ACK
Pushed; Thanks.
Peter
On Thu, Mar 05, 2015 at 12:05:14 +0100, Michal Privoznik wrote:
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
Signed-off-by: Michal Privoznik
On Thu, Mar 05, 2015 at 02:59:52PM +0100, Michal Privoznik wrote:
We have something like pvpanic device. However, in some cases it does
not have any address assigned, in which case we produce this ugly XML
(still valid though):
devices
emulator/usr/bin/qemu/emulator
...
On Thu, Mar 05, 2015 at 12:05:15 +0100, Michal Privoznik wrote:
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
Signed-off-by: Michal Privoznik
On Tue, Feb 24, 2015 at 04:34:25PM +0100, Michal Privoznik wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1195660
There's been a bug report appearing on the qemu-devel list, that
libvirt is unable to pass spaces in disk serial number [1]. Not only
our RNG schema forbids that, the code is
Later we can turn APIs to lock the object if needed instead of
relying on caller to mutually exclude itself (probably done by
locking a big lock anyway).
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/network/bridge_driver.c | 57
An accessor following pattern laid out by virDomainObjList* APIs.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 26 ++
src/conf/network_conf.h | 5 +
src/libvirt_private.syms | 1 +
3 files changed, 32 insertions(+)
diff --git
Ta-Da! Finally the commit you've been waiting for. This is yet
one of the bottlenecks. Each API has to go through the list of
network objects to find the correct one to work on. But currently
it's done in suboptimal way: every single network object is
locked, and then compared. If found, it's
On Thu, Mar 05, 2015 at 10:08:45AM +0100, Peter Krempa wrote:
On Wed, Mar 04, 2015 at 17:17:05 +0100, Pavel Hrdina wrote:
Commit message is too sparse.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1146539
Also I doubt that this on itself resolves this bug.
Well, not directly
On 03/04/2015 12:24 PM, Ján Tomko wrote:
On Tue, Feb 17, 2015 at 04:03:50PM -0500, John Ferlan wrote:
Add virDomainGetIOThreadsInfo in order to return a list of
virDomainIOThreadsInfoPtr structures which list the IOThread ID,
the CPU Affinity map, and associated resources for each IOThread
On 03/04/2015 12:45 PM, Ján Tomko wrote:
On Tue, Feb 17, 2015 at 04:03:52PM -0500, John Ferlan wrote:
Depending on the flags passed, either attempt to return the active/live
IOThread data for the domain or the config data.
The active/live path will call into the Monitor in order to get the
On 03/04/2015 12:48 PM, Ján Tomko wrote:
On Tue, Feb 17, 2015 at 04:03:53PM -0500, John Ferlan wrote:
Add the 'iothreads' command to display IOThread Info data. Allow for
[--live] or [--config] options in order to display live or config data
for an active domain.
An active domain may
This API will be used in the future to call passed callback over
each network object in the list. It's slightly different to its
virDomainObjListForEach counterpart, because virDomainObjList
uses a hash table to store domain object, while virNetworkObjList
uses an array.
Signed-off-by: Michal
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/network/bridge_driver.c | 334
1 file changed, 148 insertions(+), 186 deletions(-)
diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 2a61991..1878833 100644
---
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/test/test_driver.c | 64 ++
1 file changed, 12 insertions(+), 52 deletions(-)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 9591b7c..2bfe0ad 100644
---
Explicitly request that virNumaNodeIsAvailable not be inlined.
This fixes the test suite when building with clang (3.5.1).
---
This only leaves the mysterious check-protocol failure.
And too large stack frame size when building the tests with -O0.
src/internal.h | 10 ++
Yet another version. This time with:
1) Peter's review worked in
2) Even more patches, that allow even more parallelism. With my
test program [1], I've been able to go from 56s to 23s.
1: https://www.redhat.com/archives/libvir-list/2015-February/msg01214.html
Michal Privoznik (24):
The API will iterate over the list of network object and remove
desired ones from it.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 31 +++
src/conf/network_conf.h | 3 +++
src/libvirt_private.syms | 1 +
3 files changed, 35
Well, if 'everywhere' is defined as that part of the driver code
that serves virNetwork* APIs. Again, we lower layers already have
their locks, so there's no point doing big lock.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/test/test_driver.c | 56
This patch alone does not make much sense, I know. But it
prepares ground for next patch which when looking up a network in
the object list will not lock each network separately when
accessing its definition. Therefore we must have all the places
changing network definition lock the list.
An accessor following pattern laid out by virDomainObjList* APIs.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 38 ++
src/conf/network_conf.h | 8
src/libvirt_private.syms | 1 +
3 files changed, 47
This patch fixed following issues:
1) Hot unplug won't work for a VM without OS. Create a working VM
instead.
2) Avoid using multicast MAC address.
Signed-off-by: Zhe Peng zp...@redhat.com
---
scripts/domain/200-disk-hotplug.t | 5 +++--
scripts/domain/205-disk-hotplug-ordering.t | 3
$vmm-get_all_domain_stats
Sys::Virt::GET_ALL_STATS_ACTIVE
Sys::Virt::GET_ALL_STATS_INACTIVE
Sys::Virt::GET_ALL_STATS_OTHER
Sys::Virt::GET_ALL_STATS_PAUSED
Sys::Virt::GET_ALL_STATS_PERSISTENT
Sys::Virt::GET_ALL_STATS_RUNNING
Sys::Virt::GET_ALL_STATS_SHUTOFF
Sys::Virt::GET_ALL_STATS_TRANSIENT
Use correct package names too as they differ.
---
libvirt.spec.in | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/libvirt.spec.in b/libvirt.spec.in
index 12bd17c..d85bd4e 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -108,7 +108,7 @@
%define with_storage_iscsi
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/parallels/parallels_network.c | 29
Now that the network driver lock is ash heap of history,
we can use more of networkObjFromNetwork().
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/network/bridge_driver.c | 33 +
1 file changed, 5 insertions(+), 28 deletions(-)
diff --git
Well, one day this will be self-locking object, but not today.
But lets prepare the code for that! Moreover,
virNetworkObjListFree() is no longer needed, so turn it into
virNetworkObjListDispose().
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
cfg.mk| 1 -
So far it's just a structure which happens to have 'Obj' in its
name, but otherwise it not related to virObject at all. No
reference counting, not virObjectLock(), nothing.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
cfg.mk| 2 -
src/conf/network_conf.c
This patch turns both virNetworkObjFindByUUID() and
virNetworkObjFindByName() to return an referenced object so that
even if caller unlocks it, it's for sure that object won't
disappear meanwhile. Especially if the object (in general) is
locked and unlocked during the caller run.
Moreover, this
Now that we have fine grained locks, there's no need to lock the
whole driver. We can rely on self-locking APIs.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/network/bridge_driver.c | 49 +++--
1 file changed, 7 insertions(+), 42
Every API that touches internal structure of the object must lock
the object first. Not every API that has the object as an
argument needs to do that though. Some APIs just pass the object
to lower layers which, however, must lock the object then. Look
at the code, you'll get my meaning soon.
This is practically copy of qemuDomObjEndAPI. The reason why is
it so widely available is to avoid code duplication, since the
function is going to be called from our bridge driver, test
driver and parallels driver too.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
While in previous commits there were some places that relied on
the big lock, in this file there's no such place and the big
driver lock can be dropped completely. Yay!
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/parallels/parallels_network.c | 33
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/parallels/parallels_network.c | 66 +++
1 file changed, 12 insertions(+), 54 deletions(-)
diff --git a/src/parallels/parallels_network.c
b/src/parallels/parallels_network.c
index e1c6040..6b53518
Now that all the code uses accessors, don't expose the structure
anyway.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 7 +++
src/conf/network_conf.h | 6 --
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/conf/network_conf.c
So far, this is pure code replacement. But once we introduce
reference counting to virNetworkObj this will be more handy as
there'll be only one function to change: virNetworkObjEndAPI().
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/test/test_driver.c | 42
This is going to be needed later, when some functions needs to avoid
calling multiple times at once. It will work like this:
1) gain the object list mutex
2) find the object to work on
3) do the work
4) release the mutex
As an example of such function is virNetworkAssignDef(). The other
use case
On Thu, Mar 05, 2015 at 12:05:52PM +0100, Ján Tomko wrote:
Explicitly request that virNumaNodeIsAvailable not be inlined.
This fixes the test suite when building with clang (3.5.1).
Huh, so clang will inline functions, even if they are exported
in the .so library ? Is there some clang compiler
26.02.2015 18:56, Peter Krempa пишет:
On Thu, Feb 26, 2015 at 15:17:18 +0100, Michal Privoznik wrote:
In order to hide the object internals (and use just accessors
everywhere), lets store a pointer to the object, instead of object
itself.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
On Mon, Feb 23, 2015 at 03:38:29PM +, James Chapman wrote:
Adding functionality to libvirt that will allow it
query the ethtool interface for the availability
of certain NIC HW offload features
Here is an example of the feature XML definition:
device
On 04.03.2015 01:09, Jim Fehlig wrote:
This series contains a few simple changes to the libxl driver.
Patches 1 and 2 were included in
https://www.redhat.com/archives/libvir-list/2015-February/msg00611.html
To make an upcoming V2 of that series easier to review, I removed the
mostly
On Mon, Jan 26, 2015 at 12:08:46AM +0530, Nehal J Wani wrote:
Define helper function virDomainInterfaceFree, which allows
the upper layer application to free the domain interface object
conveniently.
The API is going to provide multiple methods by flags, e.g.
* Query guest agent
*
On Thu, Mar 05, 2015 at 12:05:08 +0100, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/parallels/parallels_network.c | 66
+++
1 file changed, 12 insertions(+), 54 deletions(-)
ACK
Peter
signature.asc
Description:
On Thu, Mar 05, 2015 at 12:05:05 +0100, Michal Privoznik wrote:
The API will iterate over the list of network object and remove
desired ones from it.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/network_conf.c | 31 +++
On 04.03.2015 01:09, Jim Fehlig wrote:
In the old days of a global driver lock, it was necessary to unlock
the driver after a domain restore operation. When the global lock
was removed from the driver, some remnants were left behind in
libxlDomainRestoreFlags. Remove this unneeded (and
On Thu, Mar 05, 2015 at 12:05:06 +0100, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/network/bridge_driver.c | 334
1 file changed, 148 insertions(+), 186 deletions(-)
ACK,
Peter
signature.asc
On Thu, Mar 05, 2015 at 12:05:07 +0100, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/test/test_driver.c | 64
++
1 file changed, 12 insertions(+), 52 deletions(-)
ACK,
Peter
signature.asc
Description:
Last commit unconditionally included a linux-specific header.
Do not do that.
---
Pushed as trivial. Forgot to send the mail.
src/util/virnetdev.c | 4
src/util/virnetdev.h | 1 -
2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
On Thu, Mar 05, 2015 at 11:44:07 +0100, Peter Krempa wrote:
Use correct package names too as they differ.
---
libvirt.spec.in | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
Self NACK. The two libraries are available only on x86_64.
v2 comming soon.
Peter
signature.asc
Use correct package names too as they differ.
---
libvirt.spec.in | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/libvirt.spec.in b/libvirt.spec.in
index 12bd17c..ce792e6 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -108,7 +108,7 @@
%define
On 03/04/2015 01:00 PM, Ján Tomko wrote:
On Tue, Feb 17, 2015 at 04:03:54PM -0500, John Ferlan wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1135491
Add the libvirt API infrastructure to support setting IOThread data.
For now this is the pinned CPU information, but eventually will also
1 - 100 of 108 matches
Mail list logo