On Tue, Dec 6, 2016 at 5:40 PM, Jamie Strandboge
wrote:
> I forgot to reiterate: the above is true *unless* there is another
> non-DAC, non-
> MAC kernel mediation (eg, does the kernel only allow modifying the 'comm'
> value
> of its own threads? If so, then the rule would
On 06.12.2016 14:12, Roman Mohr wrote:
> On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik
> wrote:
>
>> On 25.11.2016 14:38, Roman Mohr wrote:
>>
>>
> [...]
>
>
>>>
>>> 4) There libvirt domain description is not versioned
>>>
>>> I would expect that every time I update a
On Tue, Dec 06, 2016 at 01:03:29PM +0100, Viktor Mihajlovski wrote:
With kernel 3.18 (since commit 3e32cb2e0a12b6915056ff04601cf1bb9b44f967)
the "unlimited" value for cgroup memory limits has changed once again as
its byte value is now computed from a page counter.
The new "unlimited" value
On Tue, Dec 06, 2016 at 06:30:47PM +0100, Andrea Bolognani wrote:
> On Fri, 2016-12-02 at 15:18 +1100, David Gibson wrote:
> > > So, would the PCIe Root Bus in a pseries guest behave
> > > differently than the one in a q35 or mach-virt guest?
> >
> > Yes. I had a long discussion with BenH and
On Mon, Dec 05, 2016 at 03:54:49PM -0500, Laine Stump wrote:
> (Sorry for any duplicates. I sent it from the wrong address the first time)
>
> On 12/01/2016 11:18 PM, David Gibson wrote:
> > On Fri, Nov 25, 2016 at 02:46:21PM +0100, Andrea Bolognani wrote:
> > > On Wed, 2016-11-23 at 16:00 +1100,
For volume processing in virStorageBackendUpdateVolTargetInfo to get
the capacity commit id 'a760ba3a7' added the ability to probe a volume
that didn't list a target format. Unfortunately, the code used the
virStorageSource (e.g. target->type - virStorageType) rather than
virStorageVolDef (e.g.
If the voldef type is VIR_STORAGE_VOL_BLOCK, then as long as the
format is known, let's allow the probe to happen - gets a truer value
and the same probe/update would be allowed for the same volume defined
in a domain.
Signed-off-by: John Ferlan
---
Details in each patch - essentially trying to get "more correct" values.
John Ferlan (2):
storage: Fix virStorageBackendUpdateVolTargetInfo type check
storage: Allow probe of volume capacity for BLOCK type
src/storage/storage_backend.c| 12
src/storage/storage_backend.h
On 11/28/2016 04:55 AM, Jiri Denemark wrote:
...
+static virCPUCompareResult
+virCPUs390Compare(virCPUDefPtr host ATTRIBUTE_UNUSED,
+ virCPUDefPtr cpu ATTRIBUTE_UNUSED,
+ bool failMessages ATTRIBUTE_UNUSED)
The indentation is a bit off here.
+{
+return
On 12/06/2016 06:10 AM, Olga Krishtal wrote:
> On 06/12/16 02:59, John Ferlan wrote:
>>
>> On 12/02/2016 12:09 PM, Olga Krishtal wrote:
>>> On 20/09/16 23:30, John Ferlan wrote:
On 07/14/2016 01:13 PM, Olga Krishtal wrote:
> This patch supports pool and volume management within
On Tue, Dec 06, 2016 at 12:30:15PM -0500, John Ferlan wrote:
> Add VIR_DOMAIN_{BLOCK_IOTUNE|TUNABLE_BLKDEV}_GROUP_NAME parameters and
> the description.
>
> Signed-off-by: John Ferlan
> ---
> Changes| 2 +-
> Virt.xs| 2 ++
>
On Fri, 2016-12-02 at 15:18 +1100, David Gibson wrote:
> > So, would the PCIe Root Bus in a pseries guest behave
> > differently than the one in a q35 or mach-virt guest?
>
> Yes. I had a long discussion with BenH and got a somewhat better idea
> about this.
Sorry, but I'm afraid you're going
Add VIR_DOMAIN_{BLOCK_IOTUNE|TUNABLE_BLKDEV}_GROUP_NAME parameters and
the description.
Signed-off-by: John Ferlan
---
Changes| 2 +-
Virt.xs| 2 ++
lib/Sys/Virt/Domain.pm | 5 +
3 files changed, 8 insertions(+), 1 deletion(-)
diff --git
Currently, in case if we have 4 extra attached disks, then for each disk
we need to call 'glfs_init' (over network) and friends which could be costly.
Additionally snapshot(external) scenario will further complex the situation.
This patch maintain a cache of glfs objects per volume, hence the
Let the metadata for storage driver access to remote and local volumes
be cleaned by its respective driver *Deinit methods.
This will be used in the next patch, which will implement a connection
cache for/in gluster protocol driver.
Signed-off-by: Prasanna Kumar Kalever
Currently, the Host object looks like
struct _virStorageNetHostDef {
char *name;
char *port;
int transport; /* virStorageNetHostTransport */
char *socket; /* path to unix socket */
}
We don't actually need a 'name' and 'port' if the transport type is unix
domain
Currently, each among virStorageFileGetMetadataRecurse,
qemuSecurityChownCallback, qemuDomainSnapshotPrepareDiskExternal and
qemuDomainSnapshotCreateSingleDiskActive makes calls to virStorageFileInit
and friends for simple operations like stat, read headers, chown and etc.
This patch
v4: Address review comments from Peter on v3
Patch 1: changes to fix test failures
(qemuargv2xmltest, qemuxml2argvtest, qemuxml2xmltest and
virstoragetest)
Patch 2: fix hung in qemuxml2argvtest
Patch 3 v3: Split this Patch to 2 parts as below
Patch 3 v4: don't free metadata used for
On Mon, Dec 5, 2016 at 8:08 PM, Peter Krempa wrote:
> On Mon, Dec 05, 2016 at 18:55:19 +0530, Prasanna Kumar Kalever wrote:
>> Currently, in case if we have 4 extra attached disks, then for each disk
>> we need to call 'glfs_init' (over network) and friends which could be
On Mon, Dec 5, 2016 at 7:34 PM, Peter Krempa wrote:
> On Mon, Dec 05, 2016 at 18:55:18 +0530, Prasanna Kumar Kalever wrote:
>> Currently, each among virStorageFileGetMetadataRecurse,
>> qemuSecurityChownCallback, qemuDomainSnapshotPrepareDiskExternal and
>>
On Mon, Dec 5, 2016 at 7:25 PM, Peter Krempa wrote:
> On Mon, Dec 05, 2016 at 18:55:17 +0530, Prasanna Kumar Kalever wrote:
>
> qemuargv2xmltest, qemuxml2argvtest, qemuxml2xmltest, virstoragetest fail
> after this patch. I did not bother checking further or see the case.
On Tue, Dec 06, 2016 at 04:38:20PM +0100, Andrea Bolognani wrote:
> On Tue, 2016-12-06 at 13:04 +, Daniel P. Berrange wrote:
> > On Tue, Dec 06, 2016 at 01:58:50PM +0100, Peter Krempa wrote:
> > >
> > > On Tue, Dec 06, 2016 at 12:49:21 +, Daniel Berrange wrote:
> > > >
> > > > When
On Tue, 2016-12-06 at 10:17 -0600, Jamie Strandboge wrote:
> On Mon, 2016-12-05 at 17:30 +0100, Christian Ehrhardt wrote:
> >
> > On Mon, Dec 5, 2016 at 12:21 PM, intrigeri
> > wrote:
> >
> > >
> > >
> > > + @{PROC}/@{pid}/task/@{tid}/comm rw,
> > >
> > Hi,
> >
On Tue, Dec 06, 2016 at 10:18:15 -0600, Eric Blake wrote:
> On 12/06/2016 10:07 AM, Eric Blake wrote:
> > On 12/06/2016 10:00 AM, Peter Krempa wrote:
> >> Qemu 2.8.0+ changes arguments structure for blockdev-add in the effort
> >> to make it finally stable. Since libvirt recently added the
On Mon, 2016-12-05 at 17:30 +0100, Christian Ehrhardt wrote:
> On Mon, Dec 5, 2016 at 12:21 PM, intrigeri
> wrote:
>
> >
> > + @{PROC}/@{pid}/task/@{tid}/comm rw,
> >
>
> Hi,
> we have used the following for now that we planned to submit soon:
> owner
On 12/06/2016 10:07 AM, Eric Blake wrote:
> On 12/06/2016 10:00 AM, Peter Krempa wrote:
>> Qemu 2.8.0+ changes arguments structure for blockdev-add in the effort
>> to make it finally stable. Since libvirt recently added the detection of
>> gluster debug support relying on the old syntax we need
On 12/06/2016 10:00 AM, Peter Krempa wrote:
> Qemu 2.8.0+ changes arguments structure for blockdev-add in the effort
> to make it finally stable. Since libvirt recently added the detection of
> gluster debug support relying on the old syntax we need to add the new
> as well.
> ---
>
Qemu 2.8.0+ changes arguments structure for blockdev-add in the effort
to make it finally stable. Since libvirt recently added the detection of
gluster debug support relying on the old syntax we need to add the new
as well.
---
src/qemu/qemu_capabilities.c | 1 +
Hi,
Sorry to bother you again, but I still have no comment about using
openvswitch statistics command to retrieve vhost-user interface
statistics.
Regards,
On Fri, Nov 18, 2016 at 11:51:12PM +0100, Mehdi Abaakouk wrote:
This new version removes the virstat.c from locale configuration.
It
On Tue, Dec 06, 2016 at 16:38:20 +0100, Andrea Bolognani wrote:
> On Tue, 2016-12-06 at 13:04 +, Daniel P. Berrange wrote:
> > On Tue, Dec 06, 2016 at 01:58:50PM +0100, Peter Krempa wrote:
> > >
> > > On Tue, Dec 06, 2016 at 12:49:21 +, Daniel Berrange wrote:
> > > >
> > > > When redoing
On Tue, 2016-12-06 at 13:04 +, Daniel P. Berrange wrote:
> On Tue, Dec 06, 2016 at 01:58:50PM +0100, Peter Krempa wrote:
> >
> > On Tue, Dec 06, 2016 at 12:49:21 +, Daniel Berrange wrote:
> > >
> > > When redoing the website we deleted the libvirtLogo.png file
> > > not remembering that
Jamie Strandboge:
> This rule would allow any confined guest to change the 'comm' value of any
> task
> on the system, if the system otherwise allowed it.
Right. Fixed with the 'owner' prefix in my v2 patch, as suggested
by Christian.
Cheers,
--
intrigeri
--
libvir-list mailing list
https://bugzilla.redhat.com/show_bug.cgi?id=1369281
---
examples/apparmor/libvirt-qemu | 1 +
1 file changed, 1 insertion(+)
diff --git a/examples/apparmor/libvirt-qemu b/examples/apparmor/libvirt-qemu
index 11381d4df0..10d2ac958c 100644
--- a/examples/apparmor/libvirt-qemu
+++
If the monitor doesn't hold a reference to the domain object
the object may be destroyed before the monitor actually stops.
---
src/lxc/lxc_monitor.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/lxc/lxc_monitor.c b/src/lxc/lxc_monitor.c
index d828d52..de63f9e 100644
---
This function will be needed by the QEMU driver in an upcoming
patch. Additionally, removed a useless empty line.
Signed-off-by: Marc Hartmayer
---
src/conf/domain_conf.c | 3 +--
src/conf/domain_conf.h | 5 +
src/libvirt_private.syms | 1 +
3 files changed,
Disk->info is not live updatable so add a check for this. Otherwise
libvirt reports success even though no data was updated.
Signed-off-by: Marc Hartmayer
Reviewed-by: Bjoern Walk
Reviewed-by: Boris Fiuczynski
Disk->info is not live updatable so add a check for this. Otherwise
libvirt reports success even though no data was updated.
Marc Hartmayer (2):
conf: Make virDomainDeviceInfoAddressIsEqual() public
qemu: qemuDomainDiskChangeSupported: Add missing 'address' check
src/conf/domain_conf.c |
Adjust the spacing a bit in order to generate 'cleaner' looking output.
This matches what virDomainMemoryStats does and it creates text/code boxes
in order to list each of the stats for each category.
Signed-off-by: John Ferlan
---
src/libvirt-domain.c | 222
The Makefile.am adjustment works for me - although it's not my preferred
area of the environment to touch, so if there's a "better" way - let me know.
The GetAllDomainStats current output is just an unreadable mass. This cleans
it up a bit creating text/code boxes for the various stats.
The
Adjust the spacing so that the code examples are display in the code/text
box rather than just as paragraph text.
Signed-off-by: John Ferlan
---
src/libvirt-domain.c | 51 ++-
1 file changed, 26 insertions(+), 25 deletions(-)
When changing one of the src/libvirt-*.c files to alter the docs, the
adjusted files weren't being built. Added them into APIBUILD_STAMP and
then added that to the html/index.html rule which is used for the
$(apihtml_generated) generated rule.
Also, for clean we can remove the html/*.html files
On Fri, Nov 25, 2016 at 4:34 PM, Michal Privoznik
wrote:
> On 25.11.2016 14:38, Roman Mohr wrote:
>
>
[...]
> >
> > 4) There libvirt domain description is not versioned
> >
> > I would expect that every time I update a domainxml (update from third
> > party entity), or an
On Tue, Dec 06, 2016 at 01:58:50PM +0100, Peter Krempa wrote:
> On Tue, Dec 06, 2016 at 12:49:21 +, Daniel Berrange wrote:
> > When redoing the website we deleted the libvirtLogo.png file
> > not remembering that the test driver screenshot API impl
> > relied on it.
> >
> > Rather than having
On Tue, Dec 06, 2016 at 12:49:21 +, Daniel Berrange wrote:
> When redoing the website we deleted the libvirtLogo.png file
> not remembering that the test driver screenshot API impl
> relied on it.
>
> Rather than having the test driver use the logo as a side
> effect, give it its own
When redoing the website we deleted the libvirtLogo.png file
not remembering that the test driver screenshot API impl
relied on it.
Rather than having the test driver use the logo as a side
effect, give it its own dedicated image to use. This is
installed in /usr/share/libvirt/test-screenshot.png
On Tue, Dec 06, 2016 at 10:32:48AM +0100, David Hildenbrand wrote:
[...]
> >
> > I would like to hear from libvirt developers what they think. I
> > still don't know what they plan to use the type=static expansion
> > results for.
> >
> > >
> > > How long is the static expansion on a recent
On 05.12.2016 17:31, Martin Kletzander wrote:
> On Mon, Dec 05, 2016 at 11:31:46AM +0100, Michal Privoznik wrote:
>> v2 of:
>>
>> https://www.redhat.com/archives/libvir-list/2016-November/msg01456.html
>>
>
> I feel like this can go in if the nits are solved (fixed or explained).
> So conditional
[...]
> --- a/src/libvirt-domain.c
> +++ b/src/libvirt-domain.c
> @@ -11195,24 +11195,26 @@ virConnectGetDomainCapabilities(virConnectPtr conn,
> *
> * VIR_DOMAIN_STATS_PERF: Return perf event statistics.
> * The typed parameter keys are in this format:
> - * "perf.cmt" - the usage of l3
On 06.12.2016 13:10, Peter Krempa wrote:
> On Tue, Dec 06, 2016 at 13:01:48 +0100, Viktor Mihajlovski wrote:
>> On 06.12.2016 12:33, Peter Krempa wrote:
>>> On Fri, Nov 25, 2016 at 14:53:07 +0530, Nitesh Konkar wrote:
>
> [...]
>
>>> Additionally we historically treat the "all bits set" cpu map
On 30.11.2016 14:22, Viktor Mihajlovski wrote:
> On 30.11.2016 13:20, Martin Kletzander wrote:
>> On Mon, Nov 28, 2016 at 06:03:36PM +0100, Viktor Mihajlovski
>> wrote:
>>> With kernel 3.18 (since commit
>>> 3e32cb2e0a12b6915056ff04601cf1bb9b44f967) the "unlimited" value
>>> for cgroup memory
On Tue, Dec 06, 2016 at 11:11:09AM +0100, Michal Privoznik wrote:
On 05.12.2016 16:39, Martin Kletzander wrote:
On Mon, Dec 05, 2016 at 11:31:52AM +0100, Michal Privoznik wrote:
This module will be used to track:
pairs. It will be important to know these mappings
On Tue, Dec 06, 2016 at 13:01:48 +0100, Viktor Mihajlovski wrote:
> On 06.12.2016 12:33, Peter Krempa wrote:
> > On Fri, Nov 25, 2016 at 14:53:07 +0530, Nitesh Konkar wrote:
[...]
> > Additionally we historically treat the "all bits set" cpu map as
> > that there is no pinning configured. This
With kernel 3.18 (since commit 3e32cb2e0a12b6915056ff04601cf1bb9b44f967)
the "unlimited" value for cgroup memory limits has changed once again as
its byte value is now computed from a page counter.
The new "unlimited" value reported by the cgroup fs is therefore 2**51-1
pages which is
On 06.12.2016 12:33, Peter Krempa wrote:
> On Fri, Nov 25, 2016 at 14:53:07 +0530, Nitesh Konkar wrote:
>> Currently when the vcpu placement is static and cpuset is not
>> specified, CPU Affinity under virsh emulatorpin shows 0..CPUMAX.
>> This patchset will result in display of only online CPU's
On 11/25/2016 02:57 PM, Viktor Mihajlovski wrote:
> If the cpuset cgroup controller is disabled in /etc/libvirt/qemu.conf
> QEMU virtual machines can in principle use all host CPUs, even if they
> are hot plugged, if they have no explicit CPU affinity defined.
>
> However, there's libvirt code
On Fri, Nov 25, 2016 at 14:53:07 +0530, Nitesh Konkar wrote:
> Currently when the vcpu placement is static
> and cpuset is not specified, CPU Affinity
> under virsh emulatorpin shows 0..CPUMAX. This
> patchset will result in display of only
> online CPU's under CPU Affinity on linux.
>
>
On 06/12/16 02:59, John Ferlan wrote:
On 12/02/2016 12:09 PM, Olga Krishtal wrote:
On 20/09/16 23:30, John Ferlan wrote:
On 07/14/2016 01:13 PM, Olga Krishtal wrote:
This patch supports pool and volume management within Virtuozzo Storage.
Virtuozzo Storage is a highly-available distributed
On Tue, Dec 06, 2016 at 15:52:08 +0530, Nitesh Konkar wrote:
> With current perf framework, this patch adds support and documentation
> for branch instructions perf event.
>
> Signed-off-by: Nitesh Konkar
> ---
> docs/formatdomain.html.in | 6
With current perf framework, this patch adds support and documentation
for branch instructions perf event.
Signed-off-by: Nitesh Konkar
---
docs/formatdomain.html.in | 6
docs/schemas/domaincommon.rng | 1 +
On 05.12.2016 17:25, Martin Kletzander wrote:
> On Mon, Dec 05, 2016 at 11:31:56AM +0100, Michal Privoznik wrote:
>> So far the NSS module looks up only hostnames as provided by
>> guests themselves. However, there are some cases where this is
>> not enough: e.g. when there's a fresh new guest
On 05.12.2016 16:39, Martin Kletzander wrote:
> On Mon, Dec 05, 2016 at 11:31:52AM +0100, Michal Privoznik wrote:
>> This module will be used to track:
>>
>>
>>
>> pairs. It will be important to know these mappings without
>> libvirt connection (that is from a JSON
Am 06.12.2016 um 00:57 schrieb Eduardo Habkost:
On Mon, Dec 05, 2016 at 07:18:47PM +0100, David Hildenbrand wrote:
Am 02.12.2016 um 22:18 schrieb Eduardo Habkost:
The query-cpu-model-expand QMP command needs at least one static
model, to allow the "static" expansion mode to be implemented.
On Tue, Dec 06, 2016 at 10:47:51AM +0800, xieying...@huawei.com wrote:
> From: Yingtai Xie
>
> virDomainGetBlockIoTune() will be called before setBlockIoTune, as a result,
> it leads to a failure if set flags with VIR_DOMAIN_AFFECT_LIVE |
> VIR_DOMAIN_AFFECT_CONFIG which
On Fri, Nov 18, 2016 at 12:27:49PM +0530, Nehal J Wani wrote:
> Since a94f0c5c qemu supports '-vga virtio'.
> Libvirt also supports it since 21373feb.
>
> This patch enables libvirt to parse the qemu-argv:
>
> virsh domxml-from-native qemu-argv <(echo '/usr/bin/qemu-system-x86_64 -vga
>
On Thu, Dec 01, 2016 at 01:47:57PM +0100, Pino Toscano wrote:
When generating a new configuration for dnsmasq, add except-interface
for both lo and lo0: this should make sure the configuration works,
and is always the same for every supported OS.
Adjust the data of networkxml2conftest to
On Tue, Dec 06, 2016 at 10:47:51 +0800, xieying...@huawei.com wrote:
> From: Yingtai Xie
>
> virDomainGetBlockIoTune() will be called before setBlockIoTune, as a result,
> it leads to a failure if set flags with VIR_DOMAIN_AFFECT_LIVE |
> VIR_DOMAIN_AFFECT_CONFIG which is
66 matches
Mail list logo