27.03.2015 14:25, Alexander Burluka пишет:
Parallels Cloud Server supports block devices and virtual NIC
live attachment. I implemented that function for block devices so
OpenStack volume attachment is now works.
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
The first four patches only cleanup the flags checking in our APIs by
introducing new macros to check exclusive flags and requirements.
Patch 5/7 uses the new macros to do better flags checking for
virDomainSetvcpusFlags API.
Patch 6/7 introduces macro to check virsh options requirements.
The
On 03/27/2015 02:25 PM, Alexander Burluka wrote:
Parallels Cloud Server supports block devices and virtual NIC
live attachment. I implemented that function for block devices so
OpenStack volume attachment is now works.
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
tools/virsh.h | 52
1 file changed, 52 insertions(+)
diff --git a/tools/virsh.h b/tools/virsh.h
index df2ea7f..fde20ef 100644
--- a/tools/virsh.h
+++ b/tools/virsh.h
@@ -474,4 +474,56 @@ char
Commint 0473b45cc introduced new function virNetlinkDelLink, but in
it's counterpart for non-linux platform there should be ATTRIBUTE_UNUSED
instead of ATTRIBUTE_UNSUPPORTED.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
Pushed under build-breaker rule.
src/util/virnetlink.c | 2 +-
1
This patchset implements disk device attachment and allows
OpenStack to attach volumes to Parallels-driven instances.
Parallels Cloud Server SDK supports live attachment of disk devices
and virtual interfaces cards.
Alexander Burluka (3):
Parallels: remove disk serial number check
Parallels:
Parallels Cloud Server supports block devices and virtual NIC
live attachment. I implemented that function for block devices so
OpenStack volume attachment is now works.
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
src/parallels/parallels_driver.c | 51
OpenStack needs disk serial number setup because
nova boot --block-device-mapping command generates that param in
libvirt xml. I took QEMU libvirt driver behavior as a base.
QEMU driver skips inability to set serial and continues work.
So Parallels driver will ignore this param too and let domain
That function uses domainAttachDeviceFlags
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
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
On Thu, Mar 26, 2015 at 11:49:28AM -0400, Don Dutile wrote:
On 03/26/2015 07:03 AM, Ján Tomko wrote:
On Thu, Mar 26, 2015 at 12:48:13AM -0400, Wei Huang wrote:
Current libvirt can only handle up to 1024 thread siblings when it
s/1024 thread siblings/1023 bytes/
reads Linux sysfs
https://bugzilla.redhat.com/show_bug.cgi?id=1206479
From our manual of virsh and qemu side code, we know this
value must be power of 2, so instead of let qemu output error,
we can add a check when we file this value in qemuDomainBlockCopy.
Signed-off-by: Luyao Huang lhu...@redhat.com
---
We already have a check for this, just add a jump to cleanup and change to
use vshError instead of virReportError.
Signed-off-by: Luyao Huang lhu...@redhat.com
---
tools/virsh-domain.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tools/virsh-domain.c
On Fri, Mar 27, 2015 at 05:16:41PM +0800, Shanzhi Yu wrote:
Blockcopy to non-file destination is not supported according the code while
a 'goto endjob' is missed after check the destination which lead qemu try
to do drive-mirror.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1206406
Blockcopy to non-file destination is not supported according the code while
a 'goto endjob' is missed after check the destination which lead qemu try
to do drive-mirror.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1206406
Signed-off-by: Shanzhi Yu s...@redhat.com
---
On Thu, Mar 26, 2015 at 08:36:34PM +0100, Jiri Denemark wrote:
On Thu, Mar 26, 2015 at 11:37:28 -0400, John Ferlan wrote:
On 03/26/2015 08:43 AM, Ján Tomko wrote:
---
generator.py | 4 ++--
libvirt-override-api.xml | 2 +-
libvirt-override.c | 10
On Thu, Mar 26, 2015 at 11:42:12AM -0400, John Ferlan wrote:
On 03/26/2015 08:46 AM, Ján Tomko wrote:
---
Changes | 1 +
Virt.xs | 4 ++--
2 files changed, 3 insertions(+), 2 deletions(-)
There's also examples/iothreadsinfo.pl that needs to be updated - not
only to have the
From: Luyao Huang lhu...@redhat.com
We will ignore --maximum option when only use setvcpus with
this option, like this (this error is another issue):
# virsh setvcpus test3 --maximum 10
error: Failed to create controller cpu for group: No such file or directory
this is because we do not set
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/libvirt-domain-snapshot.c | 11 +++
src/qemu/qemu_driver.c| 9 +++--
2 files changed, 6 insertions(+), 14 deletions(-)
diff --git a/src/libvirt-domain-snapshot.c b/src/libvirt-domain-snapshot.c
index 0d5c5e8..9d43f54
Similar to VIR_EXLUSIVE_FLAGS, it will error out if flag requirement is
not met.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/internal.h | 43 +++
1 file changed, 43 insertions(+)
diff --git a/src/internal.h b/src/internal.h
index
Now that we have macros for exclusive flags and flag requirements we can
use them to cleanup the code for setvcpus and error out for all wrong
flag combination.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/libvirt-domain.c | 12 +++-
src/qemu/qemu_driver.c | 14 --
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/libvirt-domain-snapshot.c | 45 ++
src/libvirt-domain.c | 276 ++---
src/qemu/qemu_driver.c | 10 +-
src/storage/storage_backend_disk.c | 10 +-
Inspired by commit 7e437ee7 that introduced similar macros for virsh
commands so we don't have to repeat the same code all over.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/internal.h | 44
1 file changed, 44 insertions(+)
diff --git
That function uses domainAttachDeviceFlags
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
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
Parallels Cloud Server supports block devices and virtual NIC
live attachment. I implemented that function for block devices so
OpenStack volume attachment is now works.
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
src/parallels/parallels_driver.c | 65
This patchset implements disk device attachment and allows
OpenStack to attach volumes to Parallels-driven instances.
Parallels Cloud Server SDK supports live attachment of disk devices
and virtual interfaces cards.
Alexander Burluka (3):
Parallels: remove disk serial number check
Parallels:
OpenStack needs disk serial number setup because
nova boot --block-device-mapping command generates that param in
libvirt xml. I took QEMU libvirt driver behavior as a base.
QEMU driver skips inability to set serial and continues work.
So Parallels driver will ignore this param too and let domain
27.03.2015 15:34, Alexander Burluka пишет:
That function uses domainAttachDeviceFlags
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
src/parallels/parallels_driver.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/src/parallels/parallels_driver.c
That function uses domainAttachDeviceFlags
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
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
Parallels Cloud Server supports block devices and virtual NIC
live attachment. I implemented that function for block devices so
OpenStack volume attachment is now works.
Signed-off-by: Alexander Burluka aburl...@parallels.com
---
src/parallels/parallels_driver.c | 65
OpenStack needs disk serial number setup because
nova boot --block-device-mapping command generates that param in
libvirt xml. I took QEMU libvirt driver behavior as a base.
QEMU driver skips inability to set serial and continues work.
So Parallels driver will ignore this param too and let domain
This patchset implements disk device attachment and allows
OpenStack to attach volumes to Parallels-driven instances.
Parallels Cloud Server SDK supports live attachment of disk devices
and virtual interfaces cards.
Alexander Burluka (3):
Parallels: remove disk serial number check
Parallels:
On Mon, Mar 23, 2015 at 10:14:31PM +0530, mad Engineer wrote:
hello All,
I know the issue is related to libvirt,but i dont know
where to ask.
The libvirt mailing list is the place to ask libvirt questions. I have
CCed it.
i have centos 6.6 running KVM as compute node in
27.03.2015 15:59, Alexander Burluka пишет:
This patchset implements disk device attachment and allows
OpenStack to attach volumes to Parallels-driven instances.
Parallels Cloud Server SDK supports live attachment of disk devices
and virtual interfaces cards.
Alexander Burluka (3):
Parallels:
Use virBitmapNewCopy instead of a combination of virBitmapNew and
virBitmapCopy.
---
src/qemu/qemu_driver.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 58fa3c6..71bbf6e 100644
--- a/src/qemu/qemu_driver.c
+++
The function is used to set cpuset.cpus in various other helpers.
---
src/qemu/qemu_cgroup.c | 14 +++---
src/qemu/qemu_cgroup.h | 2 +-
src/qemu/qemu_driver.c | 4 ++--
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c
Use the default or auto cpuset if they are provided for IOThreads.
---
src/qemu/qemu_cgroup.c | 29 +
1 file changed, 17 insertions(+), 12 deletions(-)
diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c
index bc7632f..be02ede 100644
---
The automatic cpuset can be stored along with automatic nodeset and it
does not have to be recreated when used.
---
src/qemu/qemu_cgroup.c | 21 +
src/qemu/qemu_domain.c | 1 +
src/qemu/qemu_domain.h | 3 +++
src/qemu/qemu_process.c | 11 +++
4 files changed, 16
Since commit a39f69d2b libvirt would fail to start a VM if the default cpu set
was specified and individual vcpus were pinned to cpus outside of that cpuset.
Peter Krempa (8):
qemu: cgroup: Store auto cpuset instead of re-creating it on demand
qemu: cgroup: Refactor setup for IOThread cgroups
Two places would call to qemuPrepareCpumap() with priv-autoNodeset to
convert it to a cpuset. Remove the function and use the prepared cpuset
automatically.
---
src/qemu/qemu_cgroup.c | 18 +++
src/qemu/qemu_cgroup.h | 3 +-
src/qemu/qemu_process.c | 82
On 03/27/2015 06:13 AM, Pavel Hrdina wrote:
Commint 0473b45cc introduced new function virNetlinkDelLink, but in
it's counterpart for non-linux platform there should be ATTRIBUTE_UNUSED
instead of ATTRIBUTE_UNSUPPORTED.
Derp. Sorry about that, and thanks for fixing it!
--
libvir-list mailing
On Thu, Mar 26, 2015 at 15:40:36 +0100, Ján Tomko wrote:
---
tools/virsh-domain.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
ACK,
Peter
signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
The function doesn't make sense. There's a simpler way to achieve the
same.
---
src/qemu/qemu_cgroup.c | 16
src/qemu/qemu_cgroup.h | 4
src/qemu/qemu_driver.c | 8 +++-
3 files changed, 3 insertions(+), 25 deletions(-)
diff --git a/src/qemu/qemu_cgroup.c
When the default cpuset or automatic numa placement is used libvirt
would place the whole parent cgroup in the specified cpuset. This then
disallowed to re-pin the vcpus to a different cpu.
This patch pins only the vcpu threads to the default cpuset and thus
allows to re-pin them later.
The
The function doesn't make sense. There's a simpler way to achieve the
same.
---
src/qemu/qemu_cgroup.c | 15 ---
src/qemu/qemu_cgroup.h | 4
src/qemu/qemu_driver.c | 5 +
3 files changed, 1 insertion(+), 23 deletions(-)
diff --git a/src/qemu/qemu_cgroup.c
When getting info on NUMA parameters for domain,
virCgroupGetCpusetMems() may be called. However, as of 43b67f2e
the call is guarded by check if memory controller is present.
Even though it may be not obvious instantly, NUMA parameters are
stored under cpuset controller. Therefore the check needs
Throughout our code, the virCgroupController enum is used in two ways.
First as an index to an array of cgroup controllers:
struct virCgroup {
char *path;
struct virCgroupController controllers[VIR_CGROUP_CONTROLLER_LAST];
};
Second way is that when calling virCgroupNew() a bitmask of
When creating new internal representation of cgroups, all passed
arguments are logged. Well, except for two: pid and pointer for
return value. Lets log them too.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/util/vircgroup.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
The heart of the patchset are the last two patches. Long story
short, if a domain is configured to be pinned onto a set of NUMA
nodes stricly, we use both CGroups and numa_set_membind(). While
we can change the former later on a user's wish, we can't change
the latter. Therefore, any subsequent
The function has no argument named @name rather than @path
instead. The comment is, however, referring to @name while it
should have been referring to @path really.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/util/vircgroup.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Once upon a time, there was a little domain. And the domain was pinned
onto a NUMA node and hasn't fully allocated its memory:
memory unit='KiB'2355200/memory
currentMemory unit='KiB'1048576/currentMemory
numatune
memory mode='strict' nodeset='0'/
/numatune
Oh little me, said the
Like we are doing in qemu driver
($COMMIT_HASH_TO_BE_FILLED_DURING_PUSHING), lets call
virNumaSetupMemoryPolicy() only if really needed. Problem is, if
we numa_set_membind() child, there's no way to change it from the
daemon afterwards. So any later attempts to change the pinning
will fail. But in
On Fri, Mar 27, 2015 at 04:15:50PM +0100, Michal Privoznik wrote:
On 25.03.2015 16:00, Pavel Hrdina wrote:
Pavel Hrdina (2):
Revert qemucaps2xmltest: fix test to successfully run without kvm
support
tests: introduce qemucaps2xmlmock
tests/Makefile.am| 7 +++
On Fri, Mar 27, 2015 at 10:54:26 +0800, zhang bo wrote:
1 When we connect libvirt with URI qemu+ssh, it uses 'nc' command to
connect to libvirt-sock.
# virsh -c qemu+ssh://root@9.61.1.74/system list
Password: //ask users to input passwords here.
IdName State
Commit id '471e1c4e' only considered updating the pool if the extended
partition was removed. As it turns out removing a primary partition
would also need to update the freeExtent list otherwise the following
sequence would fail (assuming a fresh disk pool for /dev/sde of 500M):
$ virsh
https://bugzilla.redhat.com/show_bug.cgi?id=1206521
If the backend driver updates the pool available and/or allocation values,
then the storage_driver VolCreateXML, VolCreateXMLFrom, and VolDelete APIs
should not change the value; otherwise, it will appear as if the values
were doubled for each
https://bugzilla.redhat.com/show_bug.cgi?id=1073305
When creating a volume in a pool, the creation allows the 'capacity'
value to be larger than the available space in the pool. As long as
the 'allocation' value will fit in the space, the volume will be created.
However, resizing the volume
On 25.03.2015 16:00, Pavel Hrdina wrote:
Pavel Hrdina (2):
Revert qemucaps2xmltest: fix test to successfully run without kvm
support
tests: introduce qemucaps2xmlmock
tests/Makefile.am| 7 +++
tests/qemucaps2xmlmock.c | 33 +
Patch 1 3 are bz based, while Patch 2 was determined while working on
Patch 3. Details in each commit message
John Ferlan (3):
storage: Fix issues in storageVolResize
storage: Need to update freeExtent at delete primary partition
storage: Don't duplicate efforts of backend driver
On 03/27/2015 11:13 AM, Eric Blake wrote:
On 03/27/2015 07:12 AM, Peter Krempa wrote:
Since commit a39f69d2b libvirt would fail to start a VM if the default cpu
set
was specified and individual vcpus were pinned to cpus outside of that
cpuset.
Peter Krempa (8):
qemu: cgroup: Store
Commit id '2dbfa716' exposed virCgroupDetectMountsFromFile, but did not
add the corresponding entry in the #else /* !VIR_CGROUP_SUPPORTED */
section of the module.
Signed-off-by: John Ferlan jfer...@redhat.com
---
src/util/vircgroup.c | 11 +++
1 file changed, 11 insertions(+)
diff
On 03/27/2015 07:12 AM, Peter Krempa wrote:
Since commit a39f69d2b libvirt would fail to start a VM if the default cpu set
was specified and individual vcpus were pinned to cpus outside of that cpuset.
Peter Krempa (8):
qemu: cgroup: Store auto cpuset instead of re-creating it on demand
Commit id 'ba1dfc5' added virCgroupSetCpusetMemoryMigrate and
virCgroupGetCpusetMemoryMigrate, but did not add the corresponding
entry points into the #else /* !VIR_CGROUP_SUPPORTED */ section
Signed-off-by: John Ferlan jfer...@redhat.com
---
Not pushing as a build break since I'm not 100% it
On Wed, Mar 25, 2015 at 02:08:33PM -0600, Jim Fehlig wrote:
This small series of patches fixes some issues wrt domain destroy in
the libxl driver. The primary motivation for this work is to
prevent locking the virDomainObj during long running destroy operations
on large memory domains.
Konrad Rzeszutek Wilk wrote:
On Wed, Mar 25, 2015 at 02:08:33PM -0600, Jim Fehlig wrote:
This small series of patches fixes some issues wrt domain destroy in
the libxl driver. The primary motivation for this work is to
prevent locking the virDomainObj during long running destroy
[redirecting to libvirt; a continuation of
https://www.redhat.com/archives/libvir-list/2015-February/msg01227.html]
On 03/27/2015 02:43 PM, Pavel Raiskup wrote:
[+cc back libtool bug; as fixing automake does not seem to be enough]
Hmm. How hard is it to change ARFLAGS to 'cr' instead of the
On 03/26/2015 08:46 AM, Ján Tomko wrote:
These depend on the rename being pushed to libvirt.git:
(first four patches of:)
https://www.redhat.com/archives/libvir-list/2015-March/msg01325.html
Ján Tomko (2):
Adapt to rename of virDomainIOThreadsInfoFree to
On 03/19/2015 09:15 AM, Pavel Hrdina wrote:
From: Nehal J Wani nehaljw.k...@gmail.com
examples/Makefile.am:
* Add new file domipaddrs.py
examples/README:
* Add documentation for the python example
libvirt-override-api.xml:
* Add new symbol for virDomainInterfacesAddresses
On 03/26/2015 08:43 AM, Ján Tomko wrote:
These depend on the rename being pushed to libvirt.git:
(first four patches of:)
https://www.redhat.com/archives/libvir-list/2015-March/msg01325.html
Ján Tomko (2):
Rename virDomainIOThreadsInfoFree to virDomainIOThreadInfoFree
Rename
On Fri, Mar 27, 2015 at 11:01:23AM +0100, Pavel Hrdina wrote:
Similar to VIR_EXLUSIVE_FLAGS, it will error out if flag requirement is
not met.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/internal.h | 43 +++
1 file changed, 43
On Fri, Mar 27, 2015 at 11:01:22AM +0100, Pavel Hrdina wrote:
Inspired by commit 7e437ee7 that introduced similar macros for virsh
commands so we don't have to repeat the same code all over.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/internal.h | 44
On Fri, Mar 27, 2015 at 11:01:27AM +0100, Pavel Hrdina wrote:
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
tools/virsh.h | 52
1 file changed, 52 insertions(+)
diff --git a/tools/virsh.h b/tools/virsh.h
index df2ea7f..fde20ef
I'm running libvirt with qemu-kvm underneath.
My network stack has the eth0 interface which is bridged to a br0 interface.
Usually, when a VM starts up, a new vnet interface is created as well by
libvirt. That vnet interface has a matching HWaddr to the VM that was spun up
with it, so I'm
On 03/27/2015 10:41 AM, John Ferlan wrote:
Commit id '2dbfa716' exposed virCgroupDetectMountsFromFile, but did not
add the corresponding entry in the #else /* !VIR_CGROUP_SUPPORTED */
section of the module.
Signed-off-by: John Ferlan jfer...@redhat.com
---
src/util/vircgroup.c | 11
On Fri, Mar 27, 2015 at 11:13:07AM -0500, Jeff Nelson wrote:
On Fri, Mar 27, 2015 at 11:01:23AM +0100, Pavel Hrdina wrote:
Similar to VIR_EXLUSIVE_FLAGS, it will error out if flag requirement is
not met.
Signed-off-by: Pavel Hrdina phrd...@redhat.com
---
src/internal.h | 43
74 matches
Mail list logo