Change the permissible minimum value of nodesuspend duration time
to 60 seconds. If option is less than the value, reports error.
Update virsh help and manpage the infomation.
---
src/util/virnodesuspend.c |2 +-
tools/virsh.c |2 +-
tools/virsh.pod |3 ++-
3 file
From: Tang Chen
Introduce 2 APIs to support hypervisor threads pin in qemu driver.
1) qemudDomainPinHypervisorFlags: setup hypervisor threads pin info.
2) qemudDomainGetHypervisorPinInfo: get all hypervisor threads pin info.
They are similar to qemudDomainPinVcpuFlags and qemudDomainGetVc
From: Tang Chen
Modify vcpupin command to support hypervisor threads pin.
1) add "--hypervisor" option to get hypervisor threads info.
2) add "--hypervisor cpuset" to set hypervisor threads to specified cpuset.
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
tests/vcpupin |6
From: Tang Chen
Introduce 2 APIs to support hypervisor threads in remote driver.
1) remoteDomainPinHypervisorFlags: call driver api, such as
qemudDomainPinHypervisorFlags.
2) remoteDomainGetHypervisorPinInfo: call driver api, such as
qemudDomainGetHypervisorPinInfo.
They are similar to
From: Tang Chen
Introduce 2 APIs for client to use.
1) virDomainPinHypervisorFlags: call remote driver api, such as
remoteDomainPinHypervisorFlags.
2) virDomainGetHypervisorPinInfo: call remote driver api, such as
remoteDomainGetHypervisorPinInfo.
Signed-off-by: Tang Chen
Signed-off-b
From: Tang Chen
Introduce 2 APIs to support hypervisor threads pin.
1) virDomainHypervisorPinAdd: setup hypervisor threads pin with a given
cpumap string.
2) virDomainHypervisorPinDel: remove all hypervisor threads pin.
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/conf/doma
Users can use vcpupin command to bind a vcpu thread to a specific physical cpu.
But besides vcpu threads, there are alse some other threads created by qemu
(known as hypervisor threads) that could not be explicitly bound to physical
cpus.
The first 3 patches are from Wen Congyang, which implemen
From: Tang Chen
Hypervisor threads should also be pinned by sched_setaffinity(), just
the same as vcpu threads.
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/qemu/qemu_process.c | 54 +++
1 file changed, 54 insertions(+)
diff --git a/src
From: Tang Chen
vcpu threads pin are implemented using sched_setaffinity(), but
not controlled by cgroup. This patch does the following things:
1) enable cpuset cgroup
2) reflect all the vcpu threads pin info to cgroup
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/libvirt_pr
From: Tang Chen
This patch adds a new xml element ,
and also the parser functions, docs, and tests.
hypervisorpin means pinning hypervisor threads, and cpuset = '1'
means pinning all hypervisor threads to cpu 1.
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
docs/schemas/domaincommon.rng
From: Tang Chen
Introduce qemuSetupCgroupHypervisorPin() function to add hypervisor threads
pin info to cpuset cgroup, the same as vcpupin.
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/qemu/qemu_cgroup.c | 36
src/qemu/qemu_cgroup.h |1 +
2
From: Wen Congyang
Create a new cgroup and move all hypervisor threads to the new cgroup.
And then we can do the other things:
1. limit only vcpu usage rather than the whole qemu
2. limit for hypervisor threads(include vhost-net threads)
Signed-off-by: Wen Congyang
Signed-off-by: Tang Chen
Sig
From: Wen Congyang
Introduce a new API to move tasks of one controller from a cgroup to another
cgroup
Signed-off-by: Wen Congyang
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/libvirt_private.syms |2 +
src/util/cgroup.c| 111 ++
From: Wen Congyang
Introduce the function virCgroupForHypervisor() to create sub directory
for hypervisor thread(include I/O thread, vhost-net thread)
Signed-off-by: Wen Congyang
Signed-off-by: Tang Chen
Signed-off-by: Hu Tao
---
src/libvirt_private.syms |1 +
src/util/cgroup.c|
On 07/24/2012 08:09 AM, Kevin Wolf wrote:
Am 23.07.2012 15:07, schrieb Corey Bryant:
Corey Bryant (6):
qemu-char: Add MSG_CMSG_CLOEXEC flag to recvmsg
qapi: Introduce add-fd, remove-fd, query-fdsets
monitor: Clean up fd sets on monitor disconnect
block: Convert open calls to qemu_o
On 07/24/2012 08:07 AM, Kevin Wolf wrote:
Am 23.07.2012 15:08, schrieb Corey Bryant:
When qemu_open is passed a filename of the "/dev/fdset/nnn"
format (where nnn is the fdset ID), an fd with matching access
mode flags will be searched for within the specified monitor
fd set. If the fd is fou
On 07/24/2012 06:32 PM, Kyla Zhang wrote:
From: Xiaoqiang Hu
v2: Add skip block for qemu only and 120s timeout for test
v1: Add tests for block job lifecyle and the test flow is as follows:
create 50M qed img with qed backing img->
block pull->abort block job->resume block pull->set block job
On 2012年07月25日 01:22, Laine Stump wrote:
On 07/24/2012 01:09 AM, Osier Yang wrote:
On 2012年07月24日 04:53, Laine Stump wrote:
On 07/20/2012 10:25 AM, Osier Yang wrote:
src/conf/virobjectlist.c: Add virNetworkMatch to filter the networks;
and virNetworkList to iterate over all the networks with t
On 07/24/2012 08:05 AM, Guannan Ren wrote:
> On 07/20/2012 10:25 PM, Osier Yang wrote:
>> WITH_INTERFACE is not defined, it should be WITH_NETCF there to load
>> the interface driver.
>>
>> Eric posted patch weeks ago to resolve the problems in the whole
>> build system, but it's not finalised yet:
On 07/24/2012 08:14 AM, Guannan Ren wrote:
> On 07/24/2012 06:32 PM, Kyla Zhang wrote:
>> From: Xiaoqiang Hu
>>
>> add tests for network interface transaction: interface_change_begin,
>> interface_change_commit and interface_change_rollback
>> ---
>> .../networks/110-interface-change-transaction
On 07/24/2012 08:08 AM, Martin Kletzander wrote:
> No check for conn->uri being NULL in virAuthGetConfigFilePath (valid
> state) made the client segfault. This happens for example with these
> settings:
> - no virtualbox driver installed (modifies conn->uri)
> - no default URI set (VIRSH_DEFAULT_
On 07/24/2012 02:50 AM, Jan Tomko wrote:
> ---
> tools/virsh.pod |5 -
> 1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index b4a3d5c..6ecf6ce 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -1366,7 +1366,10 @@ appropriate
On 07/21/2012 07:02 AM, Doug Goldstein wrote:
> When using libnl use the variables pkg-config provides incase there are
s/incase/in case/
> additional libraries or CFLAGS required to build it. Specificially if
s/Specificially/Specifically/
> the libnl headers are not directly in /usr/include.
>
On 07/21/2012 02:51 PM, Matthias Bolte wrote:
> Print the actual unknown type name instead of for AnyType objects.
> ---
> src/esx/esx_vi.c |2 +-
> src/esx/esx_vi_types.c | 36 +---
> src/esx/esx_vi_types.h |1 +
> 3 files changed, 27 insertions(+)
On 07/21/2012 09:08 AM, Jim Meyering wrote:
> Eric Blake wrote:
> ...
>> This [change-set] LOOKS huge, but is really just a couple of 'git mv'
>> commands and a change to nodeinfotest.c. Scroll to the bottom for
>> the real change; I've shortened boring parts of this mail to get past
>> the 100k m
On 07/24/2012 07:48 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> virNWFilterGetIpAddrForIfname and virNWFilterDelIpAddrForIfname
> do not exist, so remove them from libvirt_nwfilter.syms
>
> Signed-off-by: Daniel P. Berrange
> ---
> src/libvirt_nwfilter.syms |2 --
> 1 fi
On 07/24/2012 09:48 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> virNWFilterGetIpAddrForIfname and virNWFilterDelIpAddrForIfname
> do not exist, so remove them from libvirt_nwfilter.syms
>
> Signed-off-by: Daniel P. Berrange
> ---
> src/libvirt_nwfilter.syms |2 --
> 1 file
On 07/24/2012 03:26 PM, Matthias Bolte wrote:
> 2012/7/23 Laine Stump :
>> On 07/20/2012 05:20 PM, Ata E Husain Bohra wrote:
>>> Add following routines to esx_interface_driver:
>>> esxNumOfInterfaces,
>>> esxNumOfDefinedInterfaces,
>>> esxListInterfaces,
>>> esxListDefinedInterfaces
On 07/24/2012 08:38 AM, Guannan Ren wrote:
> On 07/23/2012 11:10 PM, Eric Blake wrote:
>> On 07/20/2012 07:47 AM, Guannan Ren wrote:
>>> libvirt asks the duration time is above 60 seconds in code
>>>
>>> src/util/virnodesuspend.c:
>>> define MIN_TIME_REQ_FOR_SUSPEND 60 /* in seconds */
>>>
>>>
On 07/24/2012 09:48 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"
>
> During refactoring of code, it has proved common to forget to
> remove old symbols from the .syms file. While the Win32 linker
> will complain about this, the Linux ELF linker does not. The
> new test case validates
On 07/23/2012 03:05 PM, Jim Meyering wrote:
> Eric Blake wrote:
>> Any time we have a string with no % passed through gettext, a
>> translator can inject a % to cause a stack overread. When there
>> is nothing to format, it's easier to ask for a string that cannot
>> be used as a formatter, by usi
On 07/23/2012 09:08 PM, Guannan Ren wrote:
> On 07/23/2012 11:10 PM, Eric Blake wrote:
>> On 07/20/2012 07:47 AM, Guannan Ren wrote:
>>> libvirt asks the duration time is above 60 seconds in code
>>>
>>> -specified by the 'duration' parameter.
>>> +specified by the 'duration' parameter. The duratio
2012/7/23 Laine Stump :
> On 07/20/2012 05:20 PM, Ata E Husain Bohra wrote:
>> Add following routines to esx_interface_driver:
>> esxNumOfInterfaces,
>> esxNumOfDefinedInterfaces,
>> esxListInterfaces,
>> esxListDefinedInterfaces,
>> esxInterfaceLookupByMACString,
>> esxInte
On 07/24/2012 01:09 AM, Osier Yang wrote:
> On 2012年07月24日 04:53, Laine Stump wrote:
>> On 07/20/2012 10:25 AM, Osier Yang wrote:
>>> src/conf/virobjectlist.c: Add virNetworkMatch to filter the networks;
>>> and virNetworkList to iterate over all the networks with the filter.
>>>
>>> src/conf/virob
On Tue, Jul 24, 2012 at 9:06 AM, Martin Kletzander wrote:
> The 'domdisplay' command didn't properly evaluate '--include-password'
> option.
> ---
> v2:
> - switched to using 'flags'
> - kept the second condition in place
> - fixed indentation after virAsprintf
>
> tools/virsh.c | 11 +++
For objects with a subtype 'type' attribute, when the _new_from_xml
function was called, the 'type' attribute was forcefully set to the
right value rather than checking that the passed-in value matches
the type of the subclass we are trying to instantiate. This commit
changes this, and returns NULL
On 07/24/2012 04:49 PM, Peter Krempa wrote:
> On 07/24/12 16:06, Martin Kletzander wrote:
>> The 'domdisplay' command didn't properly evaluate '--include-password'
>> option.
>> ---
>> v2:
>> - switched to using 'flags'
>> - kept the second condition in place
>> - fixed indentation after virA
On 07/24/12 16:06, Martin Kletzander wrote:
The 'domdisplay' command didn't properly evaluate '--include-password'
option.
---
v2:
- switched to using 'flags'
- kept the second condition in place
- fixed indentation after virAsprintf
tools/virsh.c | 11 ---
1 files changed, 8
On 07/24/2012 06:32 PM, Kyla Zhang wrote:
From: Xiaoqiang Hu
add tests for network interface transaction: interface_change_begin,
interface_change_commit and interface_change_rollback
---
.../networks/110-interface-change-transaction.t| 81
1 files changed, 81 inse
No check for conn->uri being NULL in virAuthGetConfigFilePath (valid
state) made the client segfault. This happens for example with these
settings:
- no virtualbox driver installed (modifies conn->uri)
- no default URI set (VIRSH_DEFAULT_CONNECT_URI="",
LIBVIRT_DEFAULT_URI="", uri_default="")
On 07/20/2012 10:25 PM, Osier Yang wrote:
WITH_INTERFACE is not defined, it should be WITH_NETCF there to load
the interface driver.
Eric posted patch weeks ago to resolve the problems in the whole
build system, but it's not finalised yet:
https://www.redhat.com/archives/libvir-list/2012-June/m
The 'domdisplay' command didn't properly evaluate '--include-password'
option.
---
v2:
- switched to using 'flags'
- kept the second condition in place
- fixed indentation after virAsprintf
tools/virsh.c | 11 ---
1 files changed, 8 insertions(+), 3 deletions(-)
diff --git a/tools/v
From: "Daniel P. Berrange"
virNWFilterGetIpAddrForIfname and virNWFilterDelIpAddrForIfname
do not exist, so remove them from libvirt_nwfilter.syms
Signed-off-by: Daniel P. Berrange
---
src/libvirt_nwfilter.syms |2 --
1 file changed, 2 deletions(-)
diff --git a/src/libvirt_nwfilter.syms b
From: "Daniel P. Berrange"
During refactoring of code, it has proved common to forget to
remove old symbols from the .syms file. While the Win32 linker
will complain about this, the Linux ELF linker does not. The
new test case validates that every symbol listed in the .syms
file actually exists i
On 2012年07月24日 17:53, Peter Krempa wrote:
On 07/11/12 16:27, Osier Yang wrote:
We see the requirement for allowing to build the pool while pool-create
/pool-create-as/pool-start often in either upstream list or bugzilla,
so this patch introduces the flags virStoragePoolCreateFlags for
both virSt
We see the requirement for allowing to build the pool while pool-create
/pool-create-as/pool-start often in either upstream list or bugzilla,
so this patch introduces the flags virStoragePoolCreateFlags for
both virStoragePoolCreate and virStoragePoolCreateXML.
VIR_STORAGE_POOL_CREATE_WITH_BUILD a
On 2012年07月24日 17:53, Peter Krempa wrote:
On 07/11/12 16:27, Osier Yang wrote:
We see the requirement for allowing to build the pool while pool-create
/pool-create-as/pool-start often in either upstream list or bugzilla,
so this patch introduces the flags virStoragePoolCreateFlags for
both virSt
From: "Daniel P. Berrange"
Rename the lxc_driver_t struct typedef to virLXCDriver to more
closely follow normal libvirt naming conventions
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_conf.c|4 +-
src/lxc/lxc_conf.h| 14 ---
src/lxc/lxc_driver.c | 101 +
From: "Daniel P. Berrange"
In the socket event handler for the RPC client we must deal
with read/write events, before checking for EOF, otherwise
we might close the socket before we've read & acted upon the
last RPC messages
Signed-off-by: Daniel P. Berrange
---
src/rpc/virnetclient.c | 20 +
From: "Daniel P. Berrange"
For consistency all the APIs in the lxc_domain.c file should
have a virLXCDomain prefix in their name
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_domain.c | 14 +++---
src/lxc/lxc_domain.h |8
src/lxc/lxc_driver.c |2 +-
src/lxc/lx
From: "Daniel P. Berrange"
This defines a new RPC protocol to be used between the LXC
controller and the libvirtd LXC driver. There is only a
single RPC message defined thus far, an asynchronous "EXIT"
event that is emitted just before the LXC controller process
exits. This provides the LXC drive
From: "Daniel P. Berrange"
Update the gendispatch.pl script to get a little closer to
being able to generate code for the LXC monitor, by passing
in the struct prefix separately from the procedure prefix.
Also allow method names using virCapitalLetters instead
of vir_underscore_separator
Signed-
From: "Daniel P. Berrange"
For consistency all the APIs in the lxc_process.c file should
have a virLXCProcess prefix in their name
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_driver.c | 24 +++---
src/lxc/lxc_process.c | 216 -
src/lxc/
This is a series which makes it possible to properly handle
reboots in the LXC driver. The lxc_controller can detect
if the container asked for a reboot by looking at the exit
status of the init process & checking for SIGHUP as the
termination signal. The fun is that the lxc controller cannot
actua
From: "Daniel P. Berrange"
The reboot() syscall is allowed by new kernels for LXC containers.
The LXC controller can detect whether a reboot was requested
(instead of a normal shutdown) by looking at the "init" process
exit status. If a reboot was triggered, the exit status will
record SIGHUP as
From: "Daniel P. Berrange"
Check whether the reboot() system call is virtualized, and if
it is, then allow the container to keep CAP_SYS_REBOOT.
Based on an original patch by Serge Hallyn
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_container.c | 88
From: "Daniel P. Berrange"
Update the remote driver to use the virNetClient close callback
to trigger the virConnectPtr close callbacks
Signed-off-by: Daniel P. Berrange
---
src/remote/remote_driver.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/remote/remote_d
From: "Daniel P. Berrange"
Use a driver close callback to trigger shutdown of the
events demo program
Signed-off-by: Daniel P. Berrange
---
examples/domain-events/events-c/event-test.c | 33 --
1 file changed, 31 insertions(+), 2 deletions(-)
diff --git a/examples/do
From: "Daniel P. Berrange"
Move the code that handles the LXC monitor out of the
lxc_process.c file and into lxc_monitor.{c,h}
Signed-off-by: Daniel P. Berrange
---
po/POTFILES.in|1 +
src/Makefile.am |1 +
src/lxc/lxc_domain.h |4 +-
src/lxc/lxc_monitor.c | 163 +++
From: "Daniel P. Berrange"
Update the LXC driver to use the virNetClient APIs for
connecting to the libvirt_lxc monitor, instead of the
low-level socket APIs. This is a step towards running
a full RPC protocol with libvirt_lxc
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_domain.c |3
From: "Daniel P. Berrange"
Update all LXC code to use virReportError instead of the custom
lxcError macro
Signed-off-by: Daniel P. Berrange
---
src/lxc/lxc_conf.c | 10 +-
src/lxc/lxc_conf.h |5 -
src/lxc/lxc_container.c | 30 ++---
src/lxc/lxc_controller.c | 58
From: "Daniel P. Berrange"
Currently if the keepalive timer triggers, the 'markClose'
flag is set on the virNetClient. A controlled shutdown will
then be performed. If an I/O error occurs during read or
write of the connection an error is raised back to the
caller, but the connection isn't marked
From: "Daniel P. Berrange"
Define new virConnect{Register,Unregister}CloseCallback() public APIs
which allows registering/unregistering a callback to be invoked when
the connection to a hypervisor is closed. The callback is provided
with the reason for the close, which may be 'error', 'eof' or
'k
Update of this series
https://www.redhat.com/archives/libvir-list/2012-July/msg00917.html
Most significant change is adding a separate API to allow the
callback to be unregistered
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
From: "Daniel P. Berrange"
Allow detection of socket close in virNetClient via an callback
function, triggered on any condition that causes the socket to
be close.
Signed-off-by: Daniel P. Berrange
---
src/libvirt_private.syms |2 +-
src/rpc/virnetclient.c | 35
On 07/24/2012 06:12 AM, Eric Blake wrote:
On 07/23/2012 03:58 AM, Wen Congyang wrote:
libvirt-daemon-driver-XXX should be depent on only when with_driver_modules
s/depent on/a dependency/g
is 1.
libvirt-daemon-driver-libxl should be depent on only when with_libxl is 1.
libvirt-daemon-driver-
Hi Daniel,
I am unable to start VM
Lbvirt Version :
Compiled against library: libvir 0.9.13
Using library: libvir 0.9.13
Using API: QEMU 0.9.13
Running hypervisor: QEMU 1.0.0
Here is my command:
LC_ALL=C PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/sbin:/usr/sbin:/sbin
HOME=/home/root USER=roo
Am 23.07.2012 15:08, schrieb Corey Bryant:
> When qemu_open is passed a filename of the "/dev/fdset/nnn"
> format (where nnn is the fdset ID), an fd with matching access
> mode flags will be searched for within the specified monitor
> fd set. If the fd is found, a dup of the fd will be returned
>
Am 23.07.2012 15:07, schrieb Corey Bryant:
> Corey Bryant (6):
> qemu-char: Add MSG_CMSG_CLOEXEC flag to recvmsg
> qapi: Introduce add-fd, remove-fd, query-fdsets
> monitor: Clean up fd sets on monitor disconnect
> block: Convert open calls to qemu_open
> block: Convert close calls to qem
On Tue, Jul 24, 2012 at 1:53 PM, Christophe Fergeau wrote:
> GVirConfigObject does not implement GInitable.
I don't think that would break API/ABI to implement it (provided you
don't return GError in existing _new() functions), but the bindings
could use that information to check return value. Bu
On Tue, Jul 24, 2012 at 12:55:52PM +0200, Marc-André Lureau wrote:
> On Tue, Jul 24, 2012 at 12:24 PM, Daniel P. Berrange
> wrote:
> > Should we instead be using g_return_if_fail() so you get a dianostic,
> > on the basis that this is programmer mistake ?
>
> That was my feeling too. Although I t
On Tue, Jul 24, 2012 at 12:24 PM, Daniel P. Berrange
wrote:
> Should we instead be using g_return_if_fail() so you get a dianostic,
> on the basis that this is programmer mistake ?
That was my feeling too. Although I think returning NULL is not a real
problem, especailly if the object implements
From: Xiaoqiang Hu
v2: Add skip block for qemu only and 120s timeout for test
v1: Add tests for block job lifecyle and the test flow is as follows:
create 50M qed img with qed backing img->
block pull->abort block job->resume block pull->set block job speed->
wait to finish
---
scripts/qemu/300
From: Xiaoqiang Hu
add tests for network interface transaction: interface_change_begin,
interface_change_commit and interface_change_rollback
---
.../networks/110-interface-change-transaction.t| 81
1 files changed, 81 insertions(+), 0 deletions(-)
create mode 100644
On Wed, Jul 18, 2012 at 03:39:39PM +0200, Christophe Fergeau wrote:
> For objects with a subtype 'type' attribute, when the _new_from_xml
> function was called, the 'type' attribute was forcefully set to the
> right value rather than checking that the passed-in value matches
> the type of the subcl
Please squash this patch into patch 10.
--
Thanks,
Hu Tao
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
On Tue, Jul 10, 2012 at 05:17:45PM +0800, tangchen wrote:
> Introduce 2 APIs to support hypervisor threads pin in qemu driver.
> 1) qemudDomainPinHypervisorFlags: setup hypervisor threads pin info.
> 2) qemudDomainGetHypervisorPinInfo: get all hypervisor threads pin info.
> They are similar
On 07/11/12 16:27, Osier Yang wrote:
We see the requirement for allowing to build the pool while pool-create
/pool-create-as/pool-start often in either upstream list or bugzilla,
so this patch introduces the flags virStoragePoolCreateFlags for
both virStoragePoolCreate and virStoragePoolCreateXML
On 07/11/12 16:27, Osier Yang wrote:
Substitute 2 spaces with 4 spaces instead.
ACK, just whitespace change.
Peter
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
ping?
Christophe
On Wed, Jul 18, 2012 at 03:39:39PM +0200, Christophe Fergeau wrote:
> For objects with a subtype 'type' attribute, when the _new_from_xml
> function was called, the 'type' attribute was forcefully set to the
> right value rather than checking that the passed-in value matches
> th
Ping.
Patch 1/3 is just about the indention fix.
We tries to start the pool while creating a transicient pool,
if the pool target is not existed yet, we must fail on starting,
and thus we see many users raise up the problem on either list
or bugzilla. Patch 2/3 and 3/3 are to fix the problem
* virsh-domain-monitor.c: Add domMonitoringCmds
* virsh-domain.c: Add domManagementCmds
* virsh-host.c: Add hostAndHypervisorCmds
* virsh-interface.c: Add ifaceCmds
* virsh-network.c: Add networkCmds
* virsh-nodedev.c: Add nodedevCmds
* virsh-nwfilter.c: Add nwfilterCmds
* virsh-pool.c: Add storage
Commands in host group moved from virsh.c to virsh-host.c,
* virsh.c: Remove commands in host group.
* virsh-host.c: New file, filled with commands in host group
---
tools/virsh-host.c | 820
tools/virsh.c | 799 +
Commands to manage network filter are moved from virsh.c to virsh-nwfilter.c,
with a few helpers for network filter command use.
* virsh.c: Remove network filter commands and a few helpers.
(vshCommandOptNWFilter, and vshCommandOptNWFilterBy)
* virsh-nwfilter.c: New file, filled with ne
Commands to manage secret are moved from virsh.c to virsh-secret.c,
with a few helpers for secret command use.
* virsh.c: Remove secret commands and helper (vshCommandOptSecret).
* virsh-secret.c: New file, filled with secret commands and its helper.
---
tools/virsh-secret.c | 358 ++
Commands to manage network are moved from virsh.c to virsh-network.c,
with a few helpers for network command use.
* virsh.c: Remove network commands and a few helpers.
* virsh-network.c: New file, filled with network commands its helpers.
---
tools/virsh-network.c | 688 +
Commands in node device group moved from virsh.c to virsh-nodedev.c,
* virsh.c: Remove commands in node device group.
* virsh-nodedev.c: New file, filled with commands in node device group
---
tools/virsh-nodedev.c | 383 +
tools/virsh.c |
This will avoid many conflicts if moving the codes for each command
group into separate files in later patches.
---
tools/virsh.c | 967 -
1 files changed, 483 insertions(+), 484 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c
index
There are conflicts after splitting 2/11 and 3/11, so post the whole
set again.
v1 - v2:
* split 2/11, and 3/11 for the patches were too big. However, the new
3/13 is still quite big. I will send it individualy after compression.
This splits virsh.c by the command groups, except 'virsh itse
---
tools/virsh.pod |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/tools/virsh.pod b/tools/virsh.pod
index b4a3d5c..6ecf6ce 100644
--- a/tools/virsh.pod
+++ b/tools/virsh.pod
@@ -1366,7 +1366,10 @@ appropriate limit is adjusted if supported by the
hypervisor. LXC an
On 07/24/2012 09:58 AM, Daniel P. Berrange wrote:
> On Tue, Jul 24, 2012 at 09:16:38AM +0200, Martin Kletzander wrote:
>> On 07/23/2012 09:42 PM, Daniel P. Berrange wrote:
>>> On Mon, Jul 23, 2012 at 02:25:23PM -0500, Doug Goldstein wrote:
On Mon, Jul 23, 2012 at 1:51 PM, Martin Kletzander
>
On Tue, Jul 10, 2012 at 05:17:08PM +0800, tangchen wrote:
> Introduce 2 APIs to support hypervisor threads pin.
> 1) virDomainHypervisorPinAdd: setup hypervisor threads pin with a given
> cpumap string.
> 2) virDomainHypervisorPinDel: remove all hypervisor threads pin.
>
> Signed-off-by:
On Mon, Jul 23, 2012 at 02:37:42PM -0600, Eric Blake wrote:
> Any time we have a string with no % passed through gettext, a
> translator can inject a % to cause a stack overread. When there
> is nothing to format, it's easier to ask for a string that cannot
> be used as a formatter, by using a tri
On Tue, Jul 24, 2012 at 09:16:38AM +0200, Martin Kletzander wrote:
> On 07/23/2012 09:42 PM, Daniel P. Berrange wrote:
> > On Mon, Jul 23, 2012 at 02:25:23PM -0500, Doug Goldstein wrote:
> >> On Mon, Jul 23, 2012 at 1:51 PM, Martin Kletzander
> >> wrote:
> >>> The 'domdisplay' command didn't prop
On 07/23/2012 09:42 PM, Eric Blake wrote:
> On 07/23/2012 12:51 PM, Martin Kletzander wrote:
>> The 'domdisplay' command didn't properly evaluate '--include-password'
>> option.
>> ---
>> tools/virsh.c | 35 +++
>> 1 files changed, 23 insertions(+), 12 deletions(-
On 07/23/2012 09:42 PM, Daniel P. Berrange wrote:
> On Mon, Jul 23, 2012 at 02:25:23PM -0500, Doug Goldstein wrote:
>> On Mon, Jul 23, 2012 at 1:51 PM, Martin Kletzander
>> wrote:
>>> The 'domdisplay' command didn't properly evaluate '--include-password'
>>> option.
>>> ---
>>> tools/virsh.c |
Eric Blake wrote:
...
> This [change-set] LOOKS huge, but is really just a couple of 'git mv'
> commands and a change to nodeinfotest.c. Scroll to the bottom for
> the real change; I've shortened boring parts of this mail to get past
> the 100k mail cap enforced by the list.
If you set git diff's
Eric Blake wrote:
> Any time we have a string with no % passed through gettext, a
> translator can inject a % to cause a stack overread. When there
> is nothing to format, it's easier to ask for a string that cannot
> be used as a formatter, by using a trivial "%s" format instead.
>
> In the past,
On Tue, Jul 10, 2012 at 05:14:42PM +0800, tangchen wrote:
> vcpu threads pin are implemented using sched_setaffinity(), but not
> controlled by cgroup.
> This patch does the following things:
> 1) enable cpuset cgroup
I think it's better to make this part and adding of
virCgroupGetCpusetCpus/
99 matches
Mail list logo