There are a lot of code try to get the pte flags repeatedly, why
not save the result and reuse it in the following code? It can help
to save some CPU cycles and make the code cleaner, no?
I am not sure if this is the right direction, just change one place.
Signed-off-by: Liang Li
---
xen/arch/x
flight 102337 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102337/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 102292
test-amd64-i386-xl-qemuu-
flight 102332 libvirt real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102332/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-armhf-armhf-libvirt-qcow2 14 guest-start/debian.repeat fail REGR. vs.
102283
Regressions which a
Tuesday, November 8, 2016, 8:59:46 PM, you wrote:
> The following changes since commit 207faf24c58859f5240f66bf6decc33b87a1776e:
> Merge remote-tracking branch 'pm215/tags/pull-target-arm-20161107' into
> staging (2016-11-07 14:02:15 +)
> are available in the git repository at:
> git
>>> On 16.11.16 at 17:42, wrote:
> I assume that temporary limiting PVHv2 Dom0 support to systems with 1 IO APIC
> only is not going to be accepted, right?
Well, as long as it's experimental (and the respective code clearly
marked with fixme annotations) that would be acceptable imo.
Jan
>>> On 16.11.16 at 17:49, wrote:
> On Fri, Nov 11, 2016 at 03:04:49AM -0700, Jan Beulich wrote:
>> >>> On 10.11.16 at 18:21, wrote:
>> > On Thu, Nov 10, 2016 at 04:20:34PM +0100, Roger Pau Monné wrote:
>> >> > 0a:11.4 Ethernet controller: Intel Corporation 82576 Virtual Function
>> >> > (rev 01)
>>> On 16.11.16 at 19:02, wrote:
> On Fri, Nov 11, 2016 at 09:53:49AM -0700, Jan Beulich wrote:
>> >>> On 29.10.16 at 10:59, wrote:
>> > --- a/xen/arch/x86/domain_build.c
>> > +++ b/xen/arch/x86/domain_build.c
>> > @@ -191,10 +191,8 @@ struct vcpu *__init alloc_dom0_vcpu0(struct domain
> *dom0)
>>> On 17.11.16 at 06:21, wrote:
>> From: Jan Beulich [mailto:jbeul...@suse.com]
>> Sent: Wednesday, November 16, 2016 9:34 PM
>> >>> On 16.11.16 at 14:01, wrote:
>> > On 16/11/16 12:53, Jan Beulich wrote:
>> > On 16.11.16 at 13:31, wrote:
>> >>> This reduces the net complexity of CPUID han
>>> On 16.11.16 at 18:07, wrote:
> On 16/11/16 15:20, Jan Beulich wrote:
> On 16.11.16 at 13:31, wrote:
>>> @@ -3676,6 +3671,12 @@ void hvm_cpuid(unsigned int input, unsigned int
>>> *eax, unsigned int *ebx,
>>> if ( !(hvm_pae_enabled(v) || hvm_long_mode_enabled(v)) )
>>>
On Wed, Nov 16, Boris Ostrovsky wrote:
> Unfortunately I haven't been able to trigger NUMA balancing
> so while I tested this in general I am not sure I actually
> exercised the code path.
Thanks for the patch!
Would be nice to actually test the code path which caused the initial
addition of VM_
>>> On 17.11.16 at 09:24, wrote:
> There are a lot of code try to get the pte flags repeatedly, why
> not save the result and reuse it in the following code? It can help
> to save some CPU cycles and make the code cleaner, no?
>
> I am not sure if this is the right direction, just change one plac
Konrad Rzeszutek Wilk writes ("[PATCH v1 1/7] ts-xen-build: Enable livepatch."):
> Livepatch compiles and works on x86/ARM{32|64} so we can
> unconditionaly enable it.
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lis
Konrad Rzeszutek Wilk writes ("[PATCH v1 3/7] ts-xen-build: Install livepatch
regressions tests."):
> That come with the Xen git tree (see xen/test/).
I think this and the "build them" patch should be combined.
> +buildcmd_stamped_logged(600, 'xen', 'tests-install', '',< $dokconfig;
Can you
Konrad Rzeszutek Wilk writes ("[PATCH v1 4/7] OssTest: Add target_cmd_root_rc
which returns return code."):
> All the different target_cmd_* end up calling tcmdex
> which has the unfortunate side-effect of calling 'die' if
> the SSH sessions results in any return code not zero.
...
> Also we make
On 17/11/16 05:10, Tian, Kevin wrote:
>> From: Andrew Cooper [mailto:andrew.coop...@citrix.com]
>> Sent: Wednesday, November 16, 2016 9:42 PM
>>
>> On 26/10/16 14:15, Andrew Cooper wrote:
>>> The x86_segment enumeration matches hardware SReg encoding, which can be
>>> used
>>> to calculate the app
I am no longer in a postion to be a maintainer of the Xen subsystem.
Signed-off-by: David Vrabel
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 2a58eea..32a36da 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -13196,7 +13196,6 @@ F: drivers/
On 17/11/16 13:12, David Vrabel wrote:
> I am no longer in a postion to be a maintainer of the Xen subsystem.
>
> Signed-off-by: David Vrabel
Acked-by: Juergen Gross
Thank you very much for your work
Juergen
> ---
> MAINTAINERS | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/MAINT
I am no longer in a position to be a Xen maintainer. Andrew has
kindly volunteered to continue maintainance of the KEXEC subsystem.
EVENT CHANNELS (FIFO-BASED ABI) will be maintained by the "Other"
hypervisor maintainers.
Signed-off-by: David Vrabel
Cc: Andrew Cooper
---
MAINTAINERS | 8 +-
Konrad Rzeszutek Wilk writes ("[PATCH v1 5/7] ts-livepatch: Initial
test-cases."):
> There are 32 test-cases in there. Before we run
> any of them we verify that the payload files are present
> in /usr/lib/debug.
>
> If so we go through all of the test-cases.
> +my @livepatch_files = ("xen_hello
On 17/11/16 12:17, David Vrabel wrote:
> I am no longer in a position to be a Xen maintainer. Andrew has
> kindly volunteered to continue maintainance of the KEXEC subsystem.
> EVENT CHANNELS (FIFO-BASED ABI) will be maintained by the "Other"
> hypervisor maintainers.
>
> Signed-off-by: David Vrab
Konrad Rzeszutek Wilk writes ("[PATCH v1 6/7] sg-run-job: Add the
test-livepatch."):
> +proc need-hosts/test-livepatch {} { return host }
> +proc run-job/test-livepatch {} {
> +run-ts . = ts-livepatch host
+run-ts . = ts-livepatch + host
Please. I think that will make a prettier testi
Konrad Rzeszutek Wilk writes ("[PATCH v1 7/7] make-flight/mfi-common: Add them
in the matrix."):
> Signed-off-by: Konrad Rzeszutek Wilk
Can you accompany this with a diff of the output of
standalone-generate-dump-flight-runvars
before and after ?
You probably want
AP_FETCH_PLACEHOLDERS=y e
Konrad Rzeszutek Wilk writes ("[PATCH v1] OSSTest test-harness for
livepatches."):
> With this test-harness I am able to run the regressions tests on livepatches
> on Xen 4.8. I only used the standalone tests. The incantations was:
Thanks.
> There are still some TODOs such as:
> - Test this on
On 14/11/16 13:52, Geliang Tang wrote:
> Use builtin_pci_driver() helper to simplify the code.
>
> Signed-off-by: Geliang Tang
Applied to xen/tip.git for-linus-4.10
Juergen
> ---
> drivers/xen/platform-pci.c | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/drive
flight 102329 qemu-mainline real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102329/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-libvirt-vhd 9 debian-di-installfail REGR. vs. 101909
test-amd64-amd64-
On 14/11/16 12:12, David Vrabel wrote:
> From: Seth Forshee
>
> Mounting proc in user namespace containers fails if the xenbus
> filesystem is mounted on /proc/xen because this directory fails
> the "permanently empty" test. proc_create_mount_point() exists
> specifically to create such mountpoin
flight 68054 distros-debian-wheezy real [real]
http://osstest.xs.citrite.net/~osstest/testlogs/logs/68054/
Perfect :-)
All tests in this flight passed as required
baseline version:
flight 68024
jobs:
build-amd64 pass
build-armh
> >>> On 17.11.16 at 09:24, wrote:
> > There are a lot of code try to get the pte flags repeatedly, why not
> > save the result and reuse it in the following code? It can help to
> > save some CPU cycles and make the code cleaner, no?
> >
> > I am not sure if this is the right direction, just chan
Hi Peter and Stefan, can you merge this pull request? We want to
backport this patch once it is merged.
Thanks
Wei.
On Tue, Nov 08, 2016 at 11:59:46AM -0800, Stefano Stabellini wrote:
> The following changes since commit 207faf24c58859f5240f66bf6decc33b87a1776e:
>
> Merge remote-tracking branc
On 11/17/2016 07:17 AM, Juergen Gross wrote:
> On 17/11/16 13:12, David Vrabel wrote:
>> I am no longer in a postion to be a maintainer of the Xen subsystem.
>>
>> Signed-off-by: David Vrabel
> Acked-by: Juergen Gross
>
> Thank you very much for your work
Regretfully,
Acked-by: Boris Ostrovsky
On Thu, Nov 17, 2016 at 10:56:50AM +0100, Sander Eikelenboom wrote:
>
> Tuesday, November 8, 2016, 8:59:46 PM, you wrote:
>
> > The following changes since commit 207faf24c58859f5240f66bf6decc33b87a1776e:
>
> > Merge remote-tracking branch 'pm215/tags/pull-target-arm-20161107' into
> > stagin
On Tue, Nov 15, 2016 at 11:18:39AM +0100, Cédric Bosdonnat wrote:
> From: Cédric Bosdonnat
>
> These functions should be used to log messages when the domain
> id is known. libxl__log will now prepend the log message by
> "Domain %PRIu32:" if the domain id is a valid one.
>
> This aims at helpin
flight 102343 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102343/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 102292
test-amd64-i386-xl-qemuu-
On Tue, Nov 15, 2016 at 11:18:42AM +0100, Cédric Bosdonnat wrote:
> Use LOG*D functions to output the domain ID in logs as much as
> possible. This will help consumer code sorting the logs by
> domain.
>
> This commit includes all LOG* to LOG*D changes where the domain
> ID is not just a domid var
On 11/16/2016 05:08 AM, Jan Beulich wrote:
On 15.11.16 at 21:51, wrote:
>> On 11/15/2016 03:44 PM, Andrew Cooper wrote:
>>> On 15/11/2016 20:39, Boris Ostrovsky wrote:
On 11/15/2016 02:45 PM, Andrew Cooper wrote:
> On 15/11/16 19:34, Boris Ostrovsky wrote:
>> In addition to runni
Change since V2:
1) Update motivation for Xen vIOMMU - 288 vcpus support part
Add descriptor about plan of increasing vcpu from 128 to 255 and
dependency between X2APIC and interrupt remapping.
2) Update 3.1 New vIOMMU hypercall interface
Change vIOMMU hypercall fro
Add a test to check that Live Patch operations cannot be called from an
unprivileged domain.
Signed-off-by: Ross Lagerwall
---
docs/all-tests.dox | 2 +
include/xen/sysctl.h| 220
include/xtf/hypercall.h | 6 +
On 11/17/2016 06:28 AM, Olaf Hering wrote:
> On Wed, Nov 16, Boris Ostrovsky wrote:
>
>> Unfortunately I haven't been able to trigger NUMA balancing
>> so while I tested this in general I am not sure I actually
>> exercised the code path.
> Thanks for the patch!
>
> Would be nice to actually test t
On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote:
> On Tue, Nov 15, 2016 at 11:18:39AM +0100, Cédric Bosdonnat wrote:
> > From: Cédric Bosdonnat
> >
> > These functions should be used to log messages when the domain
> > id is known. libxl__log will now prepend the log message by
> > "Domain %PRIu
On Thu, Nov 17, Boris Ostrovsky wrote:
> On 11/17/2016 06:28 AM, Olaf Hering wrote:
> > ERROR: "__mpol_dup" [drivers/xen/xen-gntdev.ko] undefined!
> > ERROR: "get_task_policy" [drivers/xen/xen-gntdev.ko] undefined!
> I just built 4.4.11 with this patch applied and haven't had any problems.
Are th
On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote:
> On Tue, Nov 15, 2016 at 11:18:42AM +0100, Cédric Bosdonnat wrote:
> > Use LOG*D functions to output the domain ID in logs as much as
> > possible. This will help consumer code sorting the logs by
> > domain.
> >
> > This commit includes all LOG*
On Thu, Nov 17, 2016 at 10:01:15AM -0500, Boris Ostrovsky wrote:
> On 11/16/2016 05:08 AM, Jan Beulich wrote:
> On 15.11.16 at 21:51, wrote:
> >> On 11/15/2016 03:44 PM, Andrew Cooper wrote:
> >>> On 15/11/2016 20:39, Boris Ostrovsky wrote:
> On 11/15/2016 02:45 PM, Andrew Cooper wrote:
Konrad,
Thanks for the review comments. Got one quick question below.
On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote:
+int svm_avic_init_vmcb(struct vcpu *v)
> +{
> +paddr_t ma;
> +u32 apic_id_reg;
> +struct arch_svm_struct *s = &v->arch.hvm_svm;
> +struct vmcb_struct *vmcb = s-
On 11/17/2016 11:01 AM, Olaf Hering wrote:
> On Thu, Nov 17, Boris Ostrovsky wrote:
>
>> On 11/17/2016 06:28 AM, Olaf Hering wrote:
>>> ERROR: "__mpol_dup" [drivers/xen/xen-gntdev.ko] undefined!
>>> ERROR: "get_task_policy" [drivers/xen/xen-gntdev.ko] undefined!
>> I just built 4.4.11 with this pat
On Thu, Nov 17, 2016 at 05:01:08PM +0100, Cedric Bosdonnat wrote:
> On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote:
> > On Tue, Nov 15, 2016 at 11:18:39AM +0100, Cédric Bosdonnat wrote:
> > > From: Cédric Bosdonnat
> > >
> > > These functions should be used to log messages when the domain
> > >
On Thu, Nov 17, 2016 at 05:03:03PM +0100, Cedric Bosdonnat wrote:
> On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote:
> > On Tue, Nov 15, 2016 at 11:18:42AM +0100, Cédric Bosdonnat wrote:
> > > Use LOG*D functions to output the domain ID in logs as much as
> > > possible. This will help consumer co
On 17/11/16 15:40, Ross Lagerwall wrote:
> +typedef struct {
> +union {
> +void *p;
> +uint64_t __attribute__((aligned(8))) q;
> +};
> +} guest_handle_64;
> +
> +#define XEN_SYSCTL_INTERFACE_VERSION 0x000E
/sigh
This is going to require some rethinking. We cannot have
Cedric Bosdonnat writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D
function families."):
> On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote:
> > You should be able to use ao->domid here, right?
>
> Oh indeed. I discovered this one after writing this patch and forgot
> about it here. Co
Cedric Bosdonnat writes ("Re: [PATCH 04/35] libxl.c: switch to LOG*D use"):
> On Thu, 2016-11-17 at 14:50 +, Wei Liu wrote:
> > The only thing I don't understand is why you had patch 2-4 for libxl.c.
> > These three patches can probably be merged into one patch.
>
> That was to ease the review
Hi,
I would like a new repo on xenbits to hold the configuration of our
OpenStack CI loop. Its name could be:
git://xenbits.xen.org/openstack-ci-loop-config.git
Right now, the repo is at
https://github.com/citrix-openstack/os-ext-testing
The repo contain everything needed if someone wanted to red
Anthony PERARD writes ("Request for a repo on xenbits for the XenProject
Openstack CI Loop"):
> I would like a new repo on xenbits to hold the configuration of our
> OpenStack CI loop. Its name could be:
> git://xenbits.xen.org/openstack-ci-loop-config.git
I am happy with this. But there may be
On Thu, Nov 17, 2016 at 04:29:35PM +, Ian Jackson wrote:
> Cedric Bosdonnat writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D
> function families."):
> > On Thu, 2016-11-17 at 14:41 +, Wei Liu wrote:
> > > You should be able to use ao->domid here, right?
> >
> > Oh indeed. I di
Wei Liu writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function
families."):
> Oh, yes. I misunderstood. In that case libxl_is_stubdom could be handy.
How about this: check whether the ao is a nested ao, and if so, fish
the domid out of its parent ?
Ian.
__
On Thu, Nov 17, 2016 at 04:36:54PM +, Ian Jackson wrote:
> Wei Liu writes ("Re: [PATCH 01/35] libxl: add LIBXL_LOGD_* and LOG*D function
> families."):
> > Oh, yes. I misunderstood. In that case libxl_is_stubdom could be handy.
>
> How about this: check whether the ao is a nested ao, and if s
On Thu, Nov 17, 2016 at 04:34:16PM +, Ian Jackson wrote:
> Anthony PERARD writes ("Request for a repo on xenbits for the XenProject
> Openstack CI Loop"):
> > I would like a new repo on xenbits to hold the configuration of our
> > OpenStack CI loop. Its name could be:
> > git://xenbits.xen.org
Konrad,
On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote:
+
> +/* NOTE: Current max index allowed for physical APIC ID table is 255 */
> +#define AVIC_PHY_APIC_ID_MAX0xFF
> +
> +#define AVIC_DOORBELL 0xc001011b
> +#define AVIC_HPA_MASK ~((0xFFFULL << 52) || 0xFFF)
> +#defin
flight 102338 xen-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102338/
Failures :-/ but no regressions.
Tests which are failing intermittently (not blocking):
test-armhf-armhf-libvirt-xsm 6 xen-boot fail in 102318 pass in 102338
test-armhf-armhf-xl-cubietruck 1
flight 102355 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/102355/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf 2048ab4b3ec966183eb87d93d9cdab05fe9cc840
baseline version:
ovmf e242cdfb307a6dfe2c0f7
On Thu, Nov 17, 2016 at 10:05:58AM -0600, Suravee Suthikulpanit wrote:
> Konrad,
>
> Thanks for the review comments. Got one quick question below.
>
> On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote:
> > > +int svm_avic_init_vmcb(struct vcpu *v)
> > > > +{
> > > > +paddr_t ma;
> > > > +u32
On Thu, Nov 17, 2016 at 10:55:52AM -0600, Suravee Suthikulpanit wrote:
> Konrad,
>
> On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote:
> > > +
> > > > +/* NOTE: Current max index allowed for physical APIC ID table is 255 */
> > > > +#define AVIC_PHY_APIC_ID_MAX0xFF
> > > > +
> > > > +#define AVI
On Mon, 14 Nov 2016, Julien Grall wrote:
> Hi Stefano,
>
> On 11/11/2016 13:55, Stefano Stabellini wrote:
> > On Fri, 11 Nov 2016, Julien Grall wrote:
> > > Hi Stefano,
> > >
> > > On 11/11/2016 02:24, Stefano Stabellini wrote:
> > > > On Thu, 10 Nov 2016, Julien Grall wrote:
> > > > > (CC Stefan
On Wed, 16 Nov 2016, Andrii Anisov wrote:
> > For example, take a look at PVCalls which is entirely based on data
> > copies:
> >
> > http://marc.info/?l=xen-devel&m=147639616310487
> >
> >
> > I have already shown that it performs better than netfront/netback on
> > x86 in this blog post:
> >
> >
From: Cedric Bosdonnat
These functions should be used to log messages when the domain
id is known. libxl__log will now prepend the log message by
"Domain %PRIu32:" if the domain id is a valid one.
This aims at helping consumers filter logs on domain IDs.
Signed-off-by: Cédric Bosdonnat
---
v
From: Cedric Bosdonnat
Use LOG*D functions to output the domain ID in logs as much as
possible. This will help consumer code sorting the logs by
domain.
libxl.c changes have been split into 3 commits to help review
them and isolate more instances that could be problematic.
This commit, only cha
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo_qdisk.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/libxl/libxl_colo_qdisk.c b/tools/libxl/libxl_colo_qdisk.c
ind
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/libxl/libxl_colo.h b/tools/libxl/libxl_colo.h
index f0e438e..682275
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_linux.c | 23 ++-
1 file changed, 14 insertions(+), 9 deletions(-)
diff --git a/tools/libxl/libxl_linux.c b/tools/libxl/libxl_l
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_device.c | 70 --
1 file changed, 37 insertions(+), 33 deletions(-)
diff --git a/tools/libxl/libxl_devi
From: Cedric Bosdonnat
Use LOG*D functions to output the domain ID in logs as much as
possible. This will help consumer code sorting the logs by
domain.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_bootloader.c | 84 --
1 file changed, 49 insert
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_no_colo.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/libxl/libxl_no_colo.c b/tools/libxl/libxl_no_colo.c
index 15
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_nic.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/tools/libxl/libxl_nic.c b/tools/libxl/libxl_nic.c
index 220a028..
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_netbuffer.c | 43 ---
1 file changed, 24 insertions(+), 19 deletions(-)
diff --git a/tools/libxl/libxl_netb
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_netbsd.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/tools/libxl/libxl_netbsd.c b/tools/libxl/libxl_netbsd.c
index
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo_nic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/libxl/libxl_colo_nic.c b/tools/libxl/libxl_colo_nic.c
index 5e0
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_freebsd.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tools/libxl/libxl_freebsd.c b/tools/libxl/libxl_freebsd.c
inde
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_create.c | 119 +++--
1 file changed, 60 insertions(+), 59 deletions(-)
diff --git a/tools/libxl/libxl_crea
From: Cedric Bosdonnat
Use LOG*D functions to output the domain ID in logs as much as
possible. This will help consumer code sorting the logs by
domain.
This commit, only changes LOG*() into LOG*D() and adds a domid
parameter. The message of these LOG* calls has been altered to
remove the domain
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo_proxy.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/tools/libxl/libxl_colo_proxy.c b/tools/l
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_save_callout.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/libxl/libxl_save_callout.c b/tools/libxl/libxl_save
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo_restore.c | 57
1 file changed, 28 insertions(+), 29 deletions(-)
diff --git a/tools/libxl/libxl_colo
Hey all,
Here is v2 addressing Wei's comments on patch #1. The 3 libxl.c
patches haven't been merged, but the commit message of the first
one has been slightly rewritten to help understanding the reason
of the split.
Cedric Bosdonnat (35):
libxl: add LIBXL_LOGD_* and LOG*D function families.
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_dom_suspend.c | 45 -
1 file changed, 22 insertions(+), 23 deletions(-)
diff --git a/tools/libxl/libxl_dom_
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_colo_save.c | 49 ++-
1 file changed, 25 insertions(+), 24 deletions(-)
diff --git a/tools/libxl/libxl_colo
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_psr.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tools/libxl/libxl_psr.c b/tools/libxl/libxl_psr.c
index 78618
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_internal.c | 29 ++---
1 file changed, 14 insertions(+), 15 deletions(-)
diff --git a/tools/libxl/libxl_internal.c b/tools/
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_checkpoint_device.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/tools/libxl/libxl_checkpoint_device.c
b/tool
From: Cedric Bosdonnat
Use LOG*D functions to output the domain ID in logs as much as
possible. This will help consumer code sorting the logs by
domain.
This commit includes all LOG* to LOG*D changes where the domain
ID is not just a domid variable.
We want the domain ID provided to the LOG*D f
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_dm.c | 111 ++---
1 file changed, 59 insertions(+), 52 deletions(-)
diff --git a/tools/libxl/libxl_dm.c
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_qmp.c | 56 -
1 file changed, 28 insertions(+), 28 deletions(-)
diff --git a/tools/libxl/libxl_qmp.
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_usb.c | 57 ++---
1 file changed, 30 insertions(+), 27 deletions(-)
diff --git a/tools/libxl/libxl_usb.
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_x86.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tools/libxl/libxl_x86.c b/tools/libxl/libxl_x86.c
index e1844c8..5da
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_vtpm.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/libxl/libxl_vtpm.c b/tools/libxl/libxl_vtpm.c
index 8588569
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_stream_write.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/tools/libxl/libxl_stream_write.c b/tools/libxl/libxl_s
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_vnuma.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/libxl/libxl_vnuma.c b/tools/libxl/libxl_vnuma.c
index db22799..8ec
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_dom_save.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/tools/libxl/libxl_dom_save.c b/tools/
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_xshelp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/libxl/libxl_xshelp.c b/tools/libxl/libxl_xshelp.c
index b3bac
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_pci.c | 153 +---
1 file changed, 79 insertions(+), 74 deletions(-)
diff --git a/tools/libxl/libxl_pci.
From: Cedric Bosdonnat
Use LOG*D logging functions where possible instead of the LOG* ones.
Signed-off-by: Cédric Bosdonnat
---
tools/libxl/libxl_remus.c | 27 ++-
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/tools/libxl/libxl_remus.c b/tools/libxl/li
Hi Stefano,
On 17/11/2016 11:26, Stefano Stabellini wrote:
On Mon, 14 Nov 2016, Julien Grall wrote:
On 11/11/2016 13:55, Stefano Stabellini wrote:
On Fri, 11 Nov 2016, Julien Grall wrote:
On 11/11/2016 02:24, Stefano Stabellini wrote:
On Thu, 10 Nov 2016, Julien Grall wrote:
(CC Stefano and
On 11/17/16 11:18, Konrad Rzeszutek Wilk wrote:
On Thu, Nov 17, 2016 at 10:05:58AM -0600, Suravee Suthikulpanit wrote:
Konrad,
Thanks for the review comments. Got one quick question below.
On 10/12/16 15:02, Konrad Rzeszutek Wilk wrote:
+int svm_avic_init_vmcb(struct vcpu *v)
+{
+paddr
1 - 100 of 138 matches
Mail list logo
