On 10/10/2012 11:23 PM, Eric Blake wrote:
Are you interested in writing patches to implement this new API, if
others agree with my alternate single-API signature?
actually, I started to write up my proposal with the signature you
suggested :-). A second thought was that having a function to
On 10/10/2012 10:52 PM, yue wrote:
hi, now i rebuild libvirtd, version 0.10.2. and define ceph secret
,define ceph storage-pool. all fine up to now.
i operate guest-os through virt-manager-GUI . i add a
rbd-file(qemu-img create before) from ceph-rbd-pool, click 'finish',it
pop error NoneType
how should i do? i configurate guest-os.xml ,add rbd disk manually. then
virsh start xpSP3
---rbd disk
disk type='network' device='disk'
driver name='qemu' type='raw' cache='writethrough' io='native'/
auth username='cloud'
secret type='ceph'
On 10/11/2012 06:06 AM, Eric Blake wrote:
On 10/10/2012 10:02 PM, Osier Yang wrote:
Are you interested in writing patches to implement this new API, if
others agree with my alternate single-API signature?
Agreed. A single-API and allocating the cpumap inside libvirt without
the need to known
Hi Claudio,
Sorry for my late response.
I have gone through Claudio's solution. It's good. But I think this is not a
common solution. There are two points:
1. This solution must use Pointerbyreference to encapsulate the Pointer. This
is not clean.
2. Libvirt provides virFree method. But a
Thanks for figuring out why it was not working for you initially!
Especially as this was hiding 2 bugs (one in libvirt documentation and one
in this patch).
Looks good, as for the very long function prototype lines, I sometimes
split them right after the return value:
void
very_long_foo(Foo
On Wed, Oct 10, 2012 at 10:39:53PM +0200, Jiri Denemark wrote:
On Wed, Oct 10, 2012 at 16:39:06 +0100, Daniel P. Berrange wrote:
On Tue, Oct 09, 2012 at 02:13:25PM +0200, Jiri Denemark wrote:
+virBufferAddLit(buf, usb-host);
+if (!dev-missing) {
+
On Thu, Oct 11, 2012 at 10:43:14AM +0800, li guang wrote:
在 2012-10-10三的 14:38 +0100,Daniel P. Berrange写道:
On Tue, Sep 25, 2012 at 03:45:24PM +0800, liguang wrote:
original migration did not aware of offline case,
so, try to support offline migration quietly
(did not disturb original
On Wed, Oct 10, 2012 at 16:40:59 +0100, Daniel P. Berrange wrote:
On Tue, Oct 09, 2012 at 02:13:27PM +0200, Jiri Denemark wrote:
@@ -12072,6 +12079,9 @@ virDomainHostdevSourceFormat(virBufferPtr buf,
policy = virDomainStartupPolicyTypeToString(def-startupPolicy);
On Thu, Oct 11, 2012 at 11:42:50AM +0200, Jiri Denemark wrote:
On Wed, Oct 10, 2012 at 16:40:59 +0100, Daniel P. Berrange wrote:
On Tue, Oct 09, 2012 at 02:13:27PM +0200, Jiri Denemark wrote:
@@ -12072,6 +12079,9 @@ virDomainHostdevSourceFormat(virBufferPtr buf,
policy =
On 2012年10月10日 23:34, Martin Kletzander wrote:
On 10/10/2012 01:14 PM, Osier Yang wrote:
There are two branches to specifiy the pinning policy for vcpus.
1) def-cpuset:
vcpu cpuset='0-10,^6'6/vcpu
2) def-cputune.vcpupins:
cputune
vcpupin vcpuid='1' cpuset='0-5'/
/cputune
Am 10.10.2012 16:20, schrieb Corey Bryant:
qmp_add_fd() gets an fd that was received over a socket with
SCM_RIGHTS and adds it to an fd set. This patch adds support
that will enable adding an fd that was inherited on the
command line to an fd set.
This patch also prevents removal of an fd
On 10/09/12 14:13, Jiri Denemark wrote:
Using VIR_DOMAIN_XML_MIGRATABLE flag, one can request domain's XML
configuration that is suitable for migration or save/restore. Such XML
may contain extra run-time stuff internal to libvirt and some default
configuration may be removed for better
On 10/11/2012 12:46 AM, Viktor Mihajlovski wrote:
On 10/10/2012 11:23 PM, Eric Blake wrote:
Are you interested in writing patches to implement this new API, if
others agree with my alternate single-API signature?
actually, I started to write up my proposal with the signature you
suggested
On 10/11/2012 01:01 AM, Viktor Mihajlovski wrote:
No problem to include flags for future extensions to be on the safe
side. However, the map will contain all CPUs with the bits representing
the online/offline state. If the intention is to quickly get the number
of offline/online CPUS the
On Wed, Oct 10, 2012 at 15:11:18 +0100, Daniel P. Berrange wrote:
On Wed, Oct 10, 2012 at 01:35:33PM +0200, Jiri Denemark wrote:
+h2a name=domainconfigDomain configuration/a/h2
+
+p
+ In case sanlock loses access to disk locks for some reason, it will
+ kill all
On 10/11/2012 02:27 PM, Eric Blake wrote:
That starts to feel rather complex.
well :-)
On the other hand, you have a point that returning maplen (number of
bytes allocated in the map) does NOT tell you how many cpus are online
without doing a population count of the result; worse, it
On Wed, Oct 10, 2012 at 16:35:05 +0100, Daniel P. Berrange wrote:
On Tue, Oct 09, 2012 at 02:13:24PM +0200, Jiri Denemark wrote:
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index fa5d1f4..3cb1193 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -386,6
On Thu, Oct 11, 2012 at 03:35:27PM +0200, Jiri Denemark wrote:
On Wed, Oct 10, 2012 at 16:35:05 +0100, Daniel P. Berrange wrote:
On Tue, Oct 09, 2012 at 02:13:24PM +0200, Jiri Denemark wrote:
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index fa5d1f4..3cb1193 100644
---
On 10/11/12 05:25, Eric Blake wrote:
On 10/09/2012 07:39 AM, Peter Krempa wrote:
The comment stated that you may call qemuDomainObjBeginJobWithDriver()
without passing qemud_driver to signalize it's not locked.
signalize is a valid spelling, but unusual; I'd write it 'signal'.
While ago, commit 32a9aac2e04c991340b66c855a1095e4e6445e54 introduced
a change of the folders used to keep information in user home
directories. As this can still cause problems, mainly with older
SELinux policies, make this change variable with new configuration
option '--disable-xdg'.
---
On Thu, Oct 11, 2012 at 04:31:10PM +0200, Martin Kletzander wrote:
While ago, commit 32a9aac2e04c991340b66c855a1095e4e6445e54 introduced
a change of the folders used to keep information in user home
directories. As this can still cause problems, mainly with older
SELinux policies, make this
On Tue, Oct 09, 2012 at 14:13:21 +0200, Jiri Denemark wrote:
USB devices can disappear without OS being mad about it, which makes
them ideal for startupPolicy so that a domain with USB devices can be
migrated to a host that lacks the USB devices. Moreover, this series
allows USB devices to be
On 10/10/2012 05:49 PM, Eric Blake wrote:
On 10/10/2012 08:20 AM, Corey Bryant wrote:
The first call to add an fd to an fd set was previously not
allowed to choose the fd set ID. The ID was generated as
the first available and ensuing calls could add more fds by
specifying the fd set ID.
On Thu, Oct 11, 2012 at 14:42:07 +0100, Daniel P. Berrange wrote:
On Thu, Oct 11, 2012 at 03:35:27PM +0200, Jiri Denemark wrote:
On Wed, Oct 10, 2012 at 16:35:05 +0100, Daniel P. Berrange wrote:
I was wondering if we should actually expose this in the
XML, as a read-only attribute,
---
tools/virsh-domain.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index 01550c7..1a9097f 100644
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -1468,13 +1468,13 @@ cmdBlockCopy(vshControl *ctl, const vshCmd
On 10/10/2012 06:01 PM, Eric Blake wrote:
On 10/10/2012 08:20 AM, Corey Bryant wrote:
qmp_add_fd() gets an fd that was received over a socket with
SCM_RIGHTS and adds it to an fd set. This patch adds support
that will enable adding an fd that was inherited on the
command line to an fd set.
On 09.10.2012 09:58, Li Zhang wrote:
There are some descriptions not right in PowerPC CPU model driver.
This patch is to fix them.
Signed-off-by: Li Zhang zhlci...@linux.vnet.ibm.com
---
src/cpu/cpu_powerpc.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git
On 09.10.2012 09:58, Li Zhang wrote:
CPU version can be got by PVR on PowerPC. So this PVR is defined in
the CPU data in cpuData structure.
Signed-off-by: Li Zhang zhlci...@linux.vnet.ibm.com
---
src/cpu/cpu.h |3 +++
src/cpu/cpu_ppc_data.h | 33
On Thu, Oct 11, 2012 at 11:41 AM, Christophe Fergeau
cferg...@redhat.com wrote:
Thanks for figuring out why it was not working for you initially!
Especially as this was hiding 2 bugs (one in libvirt documentation and one
in this patch).
Looks good, as for the very long function prototype
On 10/11/2012 07:25 AM, Kevin Wolf wrote:
Am 10.10.2012 16:20, schrieb Corey Bryant:
qmp_add_fd() gets an fd that was received over a socket with
SCM_RIGHTS and adds it to an fd set. This patch adds support
that will enable adding an fd that was inherited on the
command line to an fd set.
On 10/10/2012 06:31 PM, Eric Blake wrote:
On 10/10/2012 08:20 AM, Corey Bryant wrote:
This option can be used for passing file descriptors on the
command line. It mirrors the existing add-fd QMP command which
allows an fd to be passed to QEMU via SCM_RIGHTS and added to an
fd set.
This can
On 09.10.2012 09:58, Li Zhang wrote:
Currently, the CPU model driver is not implemented for PowerPC.
Host's CPU information is needed to exposed to guests' XML file some
time.
This patch is to implement the callback functions of CPU model driver.
Signed-off-by: Li Zhang
SIGINT hasn't been blocked, which could lead to losing it somewhere in
virDomainGetBlockJobInfo and not aborting the job.
---
tools/virsh-domain.c | 24 ++--
1 files changed, 18 insertions(+), 6 deletions(-)
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index
At Thu, 11 Oct 2012 08:37:23 +,
Benjamin Wang (gendwang) wrote:
Hi Claudio,
Sorry for my late response.
I have gone through Claudio's solution. It's good. But I think this is not
a common solution. There are two points:
1. This solution must use Pointerbyreference to encapsulate
On 10/11/2012 08:45 AM, Corey Bryant wrote:
Another missing validation check is for duplicate use. With the monitor
command, you ALWAYS have a unique fd (thanks to SCM_RIGHTS). But with
the command line, I can type 'qemu -add-fd fd=4,set=1 -add-fd
fd=4,set=2'. Oops - I've now corrupted
On 10/10/2012 04:31 PM, Eric Blake wrote:
Another missing validation check is for duplicate use. With the monitor
command, you ALWAYS have a unique fd (thanks to SCM_RIGHTS). But with
the command line, I can type 'qemu -add-fd fd=4,set=1 -add-fd
fd=4,set=2'. Oops - I've now corrupted your
On 10/11/2012 12:00 AM, yue wrote:
how should i do? i configurate guest-os.xml ,add rbd disk manually. then
virsh start xpSP3
That's the right way to do it.
---rbd disk
disk type='network' device='disk'
driver name='qemu' type='raw' cache='writethrough'
On 10/11/2012 10:04 AM, Eric Blake wrote:
Another idea: a hybrid approach - the _first_ -add-fd 4 directly adds 4
to the set, all other -add-fd 4 end up adding dup(4) instead (well,
fcntl(F_DUPFD_CLOEXEC), but you get the picture). That is, do the
duplicate scanning, and if there is no
---
configure.ac |2 +-
daemon/libvirtd.conf |4 ++--
daemon/stream.c|2 +-
docs/api.html.in |4 ++--
docs/apibuild.py
On 10/11/2012 11:55 AM, Eric Blake wrote:
On 10/11/2012 08:45 AM, Corey Bryant wrote:
Another missing validation check is for duplicate use. With the monitor
command, you ALWAYS have a unique fd (thanks to SCM_RIGHTS). But with
the command line, I can type 'qemu -add-fd fd=4,set=1 -add-fd
Add long-running jobs for save, dump. Add normal job for the
api maybe modify the domain.
Signed-off-by: Bamvor Jian Zhang bjzh...@suse.com
---
src/libxl/libxl_driver.c | 219 ++-
1 file changed, 158 insertions(+), 61 deletions(-)
diff --git
These two patches implement a lock for long-running
job(save, dump and migration) and normal job which will
return in short time but maybe affect the state of virtual
machine.
With these patches, the normal job could not execute while
the long-ruuning job or other normal job is running.
In order
This patch introduce a lock for protecting the long-running
api (save, dump, migration and so on) from the other api
which may update the status of the virtual machine.
Signed-off-by: Bamvor Jian Zhang bjzh...@suse.com
---
src/libxl/libxl_conf.h | 58 ++
src/libxl/libxl_driver.c | 446
On 10/11/2012 12:04 PM, Eric Blake wrote:
On 10/10/2012 04:31 PM, Eric Blake wrote:
Another missing validation check is for duplicate use. With the monitor
command, you ALWAYS have a unique fd (thanks to SCM_RIGHTS). But with
the command line, I can type 'qemu -add-fd fd=4,set=1 -add-fd
Well, so much for libvirt support for stateful IPv6 addresses (dhcp6).
Not with the current design/implementation.
The problem is UPD6 server port 547 or rather how it is used. I am not
sure what the technical details are yet nor if this is fixable, but as
the situation now stands, there
When startupPolicy set for a USB devices allows such device to be
missing, there was no way this could be detected from domain XML. With
this patch, libvirt emits a new missing='yes' attribute for such devices
when active domain XML is generated.
---
docs/schemas/domaincommon.rng | 8
On Oct 1, 2012, at 10:18 AM, Kyle Mestery kmest...@cisco.com wrote:
This series of commits has the end goal of allowing per-port data stored
in the Open vSwitch DB to be transported during live migration. This is
done by first providing a generic infrastructure for transporting network
data,
On 10/11/2012 05:06 PM, Kyle Mestery (kmestery) wrote:
On Oct 1, 2012, at 10:18 AM, Kyle Mestery kmest...@cisco.com wrote:
This series of commits has the end goal of allowing per-port data stored
in the Open vSwitch DB to be transported during live migration. This is
done by first providing a
On Oct 11, 2012, at 4:25 PM, Laine Stump la...@laine.org wrote:
On 10/11/2012 05:06 PM, Kyle Mestery (kmestery) wrote:
On Oct 1, 2012, at 10:18 AM, Kyle Mestery kmest...@cisco.com wrote:
This series of commits has the end goal of allowing per-port data stored
in the Open vSwitch DB to be
On 10/11/12 18:31, Ján Tomko wrote:
---
...
38 files changed, 66 insertions(+), 66 deletions(-)
...
diff --git a/examples/domain-events/events-python/event-test.py
b/examples/domain-events/events-python/event-test.py
index 3e314bf..27e74c4 100644
---
On 10/11/12 23:43, Peter Krempa wrote:
On 10/11/12 18:31, Ján Tomko wrote:
---
...
38 files changed, 66 insertions(+), 66 deletions(-)
...
diff --git a/examples/domain-events/events-python/event-test.py
b/examples/domain-events/events-python/event-test.py
index 3e314bf..27e74c4 100644
On 10/11/2012 02:56 PM, Jiri Denemark wrote:
When startupPolicy set for a USB devices allows such device to be
missing, there was no way this could be detected from domain XML. With
this patch, libvirt emits a new missing='yes' attribute for such devices
when active domain XML is generated.
On 10/11/2012 03:53 PM, Peter Krempa wrote:
# Due to the coarse granularity of schedular timeslices, if
# we ask for a sleep of 500ms in order to satisfy a timer, we
-# may return upto 1 schedular timeslice early. So even though
+# may return up to 1 schedular timeslice
On 10/09/2012 12:22 PM, Eric Blake wrote:
On 10/06/2012 01:20 PM, Doug Goldstein wrote:
Added support for retrieving the XML defining a specific interface via
the udev based backend to virInterface. Implement the following APIs
for the udev based backend:
* virInterfaceGetXMLDesc()
Note:
Hi Claudio,
Thanks for you informing about new JNA version. I try to use JNA provided
API. It works well. The updated code is as following:
public SchedParameter[] getSchedulerParameters() throws LibvirtException {
IntByReference nParams = new IntByReference();
On 10/11/2012 09:12 AM, Ján Tomko wrote:
SIGINT hasn't been blocked, which could lead to losing it somewhere in
virDomainGetBlockJobInfo and not aborting the job.
Indeed, looking at vshWatchJob, we do the same around other libvirt APIs
when waiting for Ctrl-C; the problem is that if SIGINT
On 10/11/2012 09:14 AM, Ján Tomko wrote:
---
tools/virsh-domain.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index 01550c7..1a9097f 100644
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -1468,13
On 09/15/2012 05:53 PM, Eric Blake wrote:
v6 of these patches is here:
https://www.redhat.com/archives/libvir-list/2012-April/msg01240.html
The 'drive-mirror' command not only missed qemu 1.1 (back when I
first posted these patches), but also missed qemu 1.2. Paolo
is still working on
On 09/28/2012 09:41 AM, Osier Yang wrote:
Upstream kernel introduced new sysfs knob merge_across_nodes to
specify if pages from different numa nodes can be merged. When set
to 0, only pages which physically reside in the memory area of
same NUMA node can be merged. When set to 1, pages from
These 3 elements conflicts with each other in either the doc
or the underlying codes are. This is to propse a solution.
Before writing any codes, I want to see if the principle is
correct, any advise is welcomed.
Current problems:
Problem 1:
The doc shouldn't simply say These settings are
61 matches
Mail list logo