Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 04:56, wrote: > All existing commands ignore the parameter so this does > not break the ABI. Does it not? What about the debug mode clobbering of hypercall argument registers? I think such length indicators need to be part of the newly added

Re: [Xen-devel] [V5 2/4] x86/xsaves: enable xsaves/xrstors/xsavec in xen

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 10:14, wrote: > On Fri, Oct 09, 2015 at 01:13:11AM -0600, Jan Beulich wrote: >> >>> On 09.10.15 at 07:49, wrote: >> > On Mon, Sep 28, 2015 at 10:03:12AM -0600, Jan Beulich wrote: >> >> >>> On 21.09.15 at 13:33,

[Xen-devel] [linux-linus test] 62721: regressions - trouble: broken/fail/pass

2015-10-09 Thread osstest service owner
flight 62721 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/62721/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-multivcpu 3 host-install(3)broken REGR. vs. 59254 test-amd64-i386-xl

Re: [Xen-devel] [PATCH v1 3/4] XENVER_build_id: Provide ld-embedded build-ids

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 22:56 -0400, Konrad Rzeszutek Wilk wrote: > @@ -1633,6 +1633,8 @@ int xc_sysctl(xc_interface *xch, struct xen_sysctl > *sysctl); > > int xc_version(xc_interface *xch, int cmd, void *arg); > > +int xc_version_len(xc_interface *xch, int cmd, void *arg, size_t len); If we

Re: [Xen-devel] [PATCH v1 1/4] xsm/libxl/xen_version: Add XSM for some of the xen_version commands.

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 22:56 -0400, Konrad Rzeszutek Wilk wrote: > The XENVER_[compile_info|changeset|commandline] are now > guarded by an XSM check. I can guess, but please explain/justify why this is the case for these here. > The rest: XENVER_[version|extraversion|capabilities| >

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 22:56 -0400, Konrad Rzeszutek Wilk wrote: > If the hypervisor is built with we will display it. > > Signed-off-by: Konrad Rzeszutek Wilk > --- > tools/libxl/libxl.c | 16 > tools/libxl/libxl_types.idl | 1 + >

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 06:42 +0200, Juergen Gross wrote: > Is it okay to remove all of the unused bindings? Please can you remind me what your motivation is here, just cleaning up detritus or are some of these bindings actively hindering work you are doing? Or are they known to be broken? If

Re: [Xen-devel] [RFC] Xen 4.6 Acknowledgements

2015-10-09 Thread Lars Kurth
> On 9 Oct 2015, at 10:09, Ian Campbell wrote: > > On Thu, 2015-10-08 at 18:04 +0100, Lars Kurth wrote: >> Hi, >> >> because we are now branching and opening master before the release, I >> have to make some changes to how I acknowledge Xen release contributions. >>

[Xen-devel] osstest: elbling0 bios boot order reset

2015-10-09 Thread Ian Jackson
FTR: 09:12 <@ijc> Diziet: elbling0 seems to have stopped pxe booting and is always booting from it's disk: 09:12 <@ijc> http://logs.test-lab.xenproject.org/osstest/results/host/elbling0.html 09:12 <@ijc>

Re: [Xen-devel] [V5 1/4] x86/xsaves: add basic definitions/helpers to support xsaves

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 10:17, wrote: > On Mon, Sep 28, 2015 at 03:01:50AM -0600, Jan Beulich wrote: >> >>> On 21.09.15 at 13:33, wrote: >> > +void save_xsave_states(struct vcpu *v, void *dest, unsigned int size) >> >> Iirc it was suggested

Re: [Xen-devel] [RFC] Xen 4.6 Acknowledgements

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 10:57 +0100, Lars Kurth wrote: > > On 9 Oct 2015, at 10:09, Ian Campbell wrote: > > > > On Thu, 2015-10-08 at 18:04 +0100, Lars Kurth wrote: > > > Hi, > > > > > > because we are now branching and opening master before the release, I > > > have to

Re: [Xen-devel] patch "x86/cpufreq: relocate the driver register function" breaks cpu hot(un)plug

2015-10-09 Thread Wang, Wei W
On 09/10/2015 04:01, Konrad Rzeszutek Wilk wrote: > On Fri, Oct 09, 2015 at 06:48:23PM +0200, Dario Faggioli wrote: > > Hey, > > > > As far as my bisection goes, commit > > 49388f11d512bb92706ce046643bfbb3c1d963c9 "x86/cpufreq: relocate the > > driver register function" prevents me from hot

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 09:06, wrote: >> > >>>On 08.10.2015 at 16:52 wrote: >> >>> On 07.10.15 at 19:02, wrote: >> > Q3: what to do about mappings of other domains' memory (i.e. grant and >> > foreign mappings). >> >Between two domains,

Re: [Xen-devel] [V5 1/4] x86/xsaves: add basic definitions/helpers to support xsaves

2015-10-09 Thread Shuai Ruan
On Mon, Sep 28, 2015 at 03:01:50AM -0600, Jan Beulich wrote: > >>> On 21.09.15 at 13:33, wrote: > > This patch add basic definitions/helpers which will be used in > > later patches. > > > > Signed-off-by: Shuai Ruan > > --- > > > > +void

Re: [Xen-devel] [V5 2/4] x86/xsaves: enable xsaves/xrstors/xsavec in xen

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 07:49, wrote: > On Mon, Sep 28, 2015 at 10:03:12AM -0600, Jan Beulich wrote: >> >>> On 21.09.15 at 13:33, wrote: >> > @@ -954,8 +975,13 @@ long arch_do_domctl( >> > v->arch.xcr0_accum = _xcr0_accum; >> >

Re: [Xen-devel] [xen-4.3-testing test] 62392: tolerable FAIL - PUSHED

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 09:51 +0100, Ian Campbell wrote: > > I'm considering manually forcing a rerun of xen-4.3-testing even though no > changes are pending because I think it would be nice to have an unbroken > chain of passes for test- Things were looking pretty quiet so I decided to do this,

Re: [Xen-devel] Regression: qemu crash of hvm domUs with spice (backtrace included)

2015-10-09 Thread Fabio Fantoni
Il 08/10/2015 17:58, Andreas Kinzler ha scritto: Is this still current? I made an interesting observation: I had no problems with SPICE and vanilla Xen 4.5.1 when using it on Gentoo with glibc 2.19/gcc 4.6.4. Segfaults started when I switched to glibc 2.20/gcc 4.9.3 - I did not change Xen

Re: [Xen-devel] [PATCH V7 3/7] libxl: add pvusb API

2015-10-09 Thread Chun Yan Liu
>>> On 10/1/2015 at 01:55 AM, in message <560c2204.9030...@citrix.com>, George Dunlap wrote: > On 25/09/15 03:11, Chunyan Liu wrote: > > Add pvusb APIs, including: > > - attach/detach (create/destroy) virtual usb controller. > > - attach/detach usb device > > -

Re: [Xen-devel] [V5 2/4] x86/xsaves: enable xsaves/xrstors/xsavec in xen

2015-10-09 Thread Shuai Ruan
On Fri, Oct 09, 2015 at 01:13:11AM -0600, Jan Beulich wrote: > >>> On 09.10.15 at 07:49, wrote: > > On Mon, Sep 28, 2015 at 10:03:12AM -0600, Jan Beulich wrote: > >> >>> On 21.09.15 at 13:33, wrote: > >> > @@ -954,8 +975,13 @@ long

Re: [Xen-devel] [PATCH v1] Add build-id to XENVER hypercall.

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 04:56, wrote: > However they also change the behavior of the existing hypercall > for XENVER_[compile_info|changeset|commandline] and make them > dom0 accessible. This is if XSM is built in or not (though with > XSM one can expose it to a guest if

[Xen-devel] [PATCH 4.5] ipxe fix for gcc5

2015-10-09 Thread Mark Pryor
Signed-off-by: Mark Pryor --- .../firmware/etherboot/patches/ipxe-ath9k-gcc5.patch | 20 .../etherboot/patches/ipxe-ath9k2-gcc5.patch | 20 tools/firmware/etherboot/patches/series | 2 ++ 3 files changed, 42

Re: [Xen-devel] [PATCH V7 5/7] xl: add pvusb commands

2015-10-09 Thread Chun Yan Liu
>>> On 10/2/2015 at 01:02 AM, in message <560d6733.4030...@citrix.com>, George Dunlap wrote: > On 25/09/15 03:11, Chunyan Liu wrote: > > Add pvusb commands: usb-ctrl-attach, usb-ctrl-detach, usb-list, > > usb-attach and usb-detach. > > > > To attach a usb device

Re: [Xen-devel] [PATCH seabios.git rel-1.7.5] fix release-1.7.5 for gcc5

2015-10-09 Thread Jan Beulich
>>> On 08.10.15 at 21:36, wrote: > Signed-off-by: Mark Pryor Without any description I cannot see what is being fixed here, or why there are _different_ comment changes on the inclusion of the same comment. Since I assume that whatever issue there was has

Re: [Xen-devel] [PATCH 4.5] ipxe fix for gcc5

2015-10-09 Thread Jan Beulich
>>> On 08.10.15 at 19:25, wrote: > Signed-off-by: Mark Pryor To be honest I'd prefer to take the upstream fix for this, i.e. what we also have in 4.6. Plus for submissions against stable releases, please see ./MAINTAINERS of the respective tree. Jan >

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-10-09 Thread Xu, Quan
>> >>> On 09.10.2015 at 15:18 wrote: > >>> On 09.10.15 at 09:06, wrote: > >> > >>>On 08.10.2015 at 16:52 wrote: > >> >>> On 07.10.15 at 19:02, wrote: > >> > __scheme B__ > >> > Q1: - when to take the references? > >>

Re: [Xen-devel] [PATCH v2 1/2] x86/mem-sharing: Bulk mem-sharing entire domains

2015-10-09 Thread Jan Beulich
>>> On 08.10.15 at 22:57, wrote: > --- a/xen/arch/x86/mm/mem_sharing.c > +++ b/xen/arch/x86/mm/mem_sharing.c > @@ -1293,6 +1293,37 @@ int relinquish_shared_pages(struct domain *d) > return rc; > } > > +static int bulk_share(struct domain *d, struct domain *cd,

Re: [Xen-devel] [PATCH] VT-d: don't suppress invalidation address write when it is zero

2015-10-09 Thread Zhang, Yang Z
Jan Beulich wrote on 2015-09-28: > GFN zero is a valid address, and hence may need invalidation done for > it just like for any other GFN. > > Signed-off-by: Jan Beulich Acked-by: Yang Zhang > --- > The comment right before the respective

[Xen-devel] [xen-4.4-testing test] 62730: trouble: blocked/broken/fail/pass

2015-10-09 Thread osstest service owner
flight 62730 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62730/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-raw 3 host-install(3) broken REGR. vs.

Re: [Xen-devel] [RFC OSSTEST v2] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 17:06 +0100, Ian Jackson wrote: > [...] Agree with that analysis, thanks. > In any case, I have tried this several times both with and without > eatmydata and it seems to work fine for me. Me too, today. > I may prepare a patch to make

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Boris Ostrovsky
On 10/09/2015 12:19 PM, Jan Beulich wrote: On 09.10.15 at 18:09, wrote: On 10/09/2015 11:11 AM, Jan Beulich wrote: On 09.10.15 at 16:00, wrote: On Fri, Oct 09, 2015 at 09:41:36AM -0400, Boris Ostrovsky wrote: On 10/09/2015 02:51 AM, Jan

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Haozhong Zhang
On Fri, Oct 09, 2015 at 11:37:06AM -0400, Boris Ostrovsky wrote: > On 10/09/2015 10:39 AM, Jan Beulich wrote: > On 09.10.15 at 15:41, wrote: > >>On 10/09/2015 02:51 AM, Jan Beulich wrote: > >>On 28.09.15 at 09:13, wrote: > When

Re: [Xen-devel] [RFC OSSTEST v2] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-09 Thread Ian Jackson
Ian Campbell writes ("[RFC OSSTEST v2] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder"): > Still RFC because of these sqlite errors > > DBD::SQLite::db do failed: UNIQUE constraint failed: jobs.flight, > jobs.job [for Statement "INSERT INTO jobs VALUES

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Boris Ostrovsky
On 10/09/2015 11:11 AM, Jan Beulich wrote: On 09.10.15 at 16:00, wrote: On Fri, Oct 09, 2015 at 09:41:36AM -0400, Boris Ostrovsky wrote: On 10/09/2015 02:51 AM, Jan Beulich wrote: On 28.09.15 at 09:13, wrote: When the TSC mode of a domain

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Boris Ostrovsky
On 10/09/2015 12:39 PM, Haozhong Zhang wrote: On Fri, Oct 09, 2015 at 11:37:06AM -0400, Boris Ostrovsky wrote: On 10/09/2015 10:39 AM, Jan Beulich wrote: On 09.10.15 at 15:41, wrote: On 10/09/2015 02:51 AM, Jan Beulich wrote: On 28.09.15 at 09:13,

[Xen-devel] [linux-mingo-tip-master test] 62728: regressions - FAIL

2015-10-09 Thread osstest service owner
flight 62728 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/62728/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 6 xen-boot fail REGR. vs. 60684

[Xen-devel] [PATCH OSSTEST v3] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-09 Thread Ian Campbell
And use this in standalone-generate-dump-flight-runvars. In general I don't think we are interested in the specific revision_* runvars when using this tool but when it matters this new behaviour can be avoided by setting AP_FETCH_PLACEHOLDERS=n. This is quicker even than using memoisation on the

Re: [Xen-devel] [PATCH v1] Add build-id to XENVER hypercall.

2015-10-09 Thread Andrew Cooper
On 09/10/15 09:17, Jan Beulich wrote: On 09.10.15 at 04:56, wrote: >> However they also change the behavior of the existing hypercall >> for XENVER_[compile_info|changeset|commandline] and make them >> dom0 accessible. This is if XSM is built in or not (though with >>

Re: [Xen-devel] [PATCH v1 3/4] XENVER_build_id: Provide ld-embedded build-ids

2015-10-09 Thread Andrew Cooper
On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: > @@ -367,6 +368,35 @@ DO(xen_version)(int cmd, XEN_GUEST_HANDLE_PARAM(void) > arg, > if ( copy_to_guest(arg, saved_cmdline, ARRAY_SIZE(saved_cmdline)) ) > return -EFAULT; > return 0; > + > +case XENVER_build_id: >

Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 13:29 +0100, Andrew Cooper wrote: > On 09/10/15 09:25, Jan Beulich wrote: > > > > > On 09.10.15 at 04:56, wrote: > > > All existing commands ignore the parameter so this does > > > not break the ABI. > > Does it not? What about the debug mode

Re: [Xen-devel] [PATCH v1] xSplice initial foundation patches.

2015-10-09 Thread Konrad Rzeszutek Wilk
On Fri, Oct 02, 2015 at 10:48:54AM -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Sep 16, 2015 at 05:01:11PM -0400, Konrad Rzeszutek Wilk wrote: > > > > *OK, what do you have?* > > > > They are located at a git tree: > > git://xenbits.xen.org/people/konradwilk/xen.git xsplice.v1 > > I've

Re: [Xen-devel] [PATCH 4.5] ipxe fix for gcc5

2015-10-09 Thread Wei Liu
On Thu, Oct 08, 2015 at 10:25:42AM -0700, Mark Pryor wrote: > Signed-off-by: Mark Pryor There is already a fix in tree for 4.6. You can request backporting commit 6596412d59bcde3d1a2473f341851f4c476fc9df Author: Konrad Rzeszutek Wilk Date: Mon Aug

Re: [Xen-devel] [qemu-mainline test] 62725: trouble: broken/fail/pass

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 10:33 +, osstest service owner wrote: > flight 62725 qemu-mainline real [real] > http://logs.test-lab.xenproject.org/osstest/logs/62725/ > > Failures and problems with tests :-( > > Tests which did not succeed and are blocking, > including tests which could not be run:

[Xen-devel] Intention to deprecate and remove some Python libxc bindings

2015-10-09 Thread Juergen Gross
Mostly for historical reasons Xen includes Python bindings for libxc. They have been used by xm/xend in the past but nowadays there is only one usage in the Xen source tree: pygrub is using xc.xeninfo(). I wrote a patch to remove all libxc Python bindings but xc.xeninfo() and got some feedback

Re: [Xen-devel] [PATCH OSSTEST v3 3/3] Create a flight to test OpenStack with xen-unstable and libvirt

2015-10-09 Thread Anthony PERARD
On Thu, Oct 08, 2015 at 05:42:56PM +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH OSSTEST v3 3/3] Create a flight to test > OpenStack with xen-unstable and libvirt"): > > On Tue, 2015-09-29 at 17:52 +0100, Ian Jackson wrote: > > > All these revision_FOO=master are rather odd. > > >

Re: [Xen-devel] [PATCH v3 7/9] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-10-09 Thread Julien Grall
On 08/10/15 19:36, Julien Grall wrote: > On 08/10/15 15:25, Ian Campbell wrote: >>> If the concern is the behavior is changed, I'm happy to rework this code >>> to keep exactly the same behavior. I.e any 32-bit write containing >>> a 0 byte will be ignored. This is not optimal but at least I'm not

Re: [Xen-devel] [PATCH v3 7/9] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 12:24 +0100, Julien Grall wrote: > I plan to fix it in patch #1 and request a backport for Xen 4.6 and Xen > 4.5. I can do a separate patch if we don't want the cleanup. I'm not quite sure what you are proposing but please put logical changes and cleanups into separate

[Xen-devel] [OSSTEST PATCH 3/3] smoke tests: Consider osstest revision when reusing builds

2015-10-09 Thread Ian Jackson
Build results done with one version of osstest are not necessarily reuseable with a different version of osstest. For example, the suite may have changed. The smoke tests try to reuse builds from xen-unstable but if osstest changes incompatibly, the smoke tests might repeatedly fail until a

[Xen-devel] [OSSTEST PATCH 2/3] sg-check-tested: Honour multiple branches (comma-separated)

2015-10-09 Thread Ian Jackson
No functional change with existing invocations. Signed-off-by: Ian Jackson --- sg-check-tested |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sg-check-tested b/sg-check-tested index 1a3afa3..43f2854 100755 --- a/sg-check-tested +++

Re: [Xen-devel] [PATCH v1 1/4] xsm/libxl/xen_version: Add XSM for some of the xen_version commands.

2015-10-09 Thread Andrew Cooper
On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: > The XENVER_[compile_info|changeset|commandline] are now > guarded by an XSM check. > > The rest: XENVER_[version|extraversion|capabilities| > parameters|get_features|page_size|guest_handle] behave > as before (no XSM check). > > We allow the

Re: [Xen-devel] [OSSTEST PATCH 2/3] sg-check-tested: Honour multiple branches (comma-separated)

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 12:48 +0100, Ian Jackson wrote: > No functional change with existing invocations. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v1] Add build-id to XENVER hypercall.

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 14:15, wrote: > On 09/10/15 09:17, Jan Beulich wrote: >> The more that the tool stack uses the two, and as we know >> tool stacks or parts thereof can live in unprivileged domains. > > I would argue than a fully unprivileged toolstack domain has no

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Haozhong Zhang
On Fri, Oct 09, 2015 at 12:51:32AM -0600, Jan Beulich wrote: > >>> On 28.09.15 at 09:13, wrote: > > When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation > > is used, the existing tsc_get_info() calculates elapsed_nsec by scaling > > the host TSC with a

Re: [Xen-devel] [OSSTEST PATCH v3 2/3] Testing cpupools: recipe for it and job definition

2015-10-09 Thread Ian Campbell
On Sat, 2015-10-03 at 02:39 +0200, Dario Faggioli wrote: > Signed-off-by: Dario Faggioli > --- > Cc: Ian Jackson > Cc: Ian Campbell > Cc: Juergen Gross This looks correct to me as it stands, but I

[Xen-devel] [qemu-mainline test] 62725: trouble: broken/fail/pass

2015-10-09 Thread osstest service owner
flight 62725 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/62725/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-multivcpu 3 host-install(3)broken REGR. vs.

[Xen-devel] [xen-unstable baseline-only test] 38140: regressions - FAIL

2015-10-09 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38140 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38140/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-pygrub 19 guest-start.2

Re: [Xen-devel] [PATCH v1 3/4] XENVER_build_id: Provide ld-embedded build-ids

2015-10-09 Thread Martin Pohlack
On 09.10.2015 04:56, Konrad Rzeszutek Wilk wrote: > @@ -367,6 +368,35 @@ DO(xen_version)(int cmd, XEN_GUEST_HANDLE_PARAM(void) > arg, > if ( copy_to_guest(arg, saved_cmdline, ARRAY_SIZE(saved_cmdline)) ) > return -EFAULT; > return 0; > + > +case XENVER_build_id:

[Xen-devel] [OSSTEST PATCH 1/3] Move get_harness_rev to Osstest from Osstest::Executive.

2015-10-09 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest.pm | 17 + Osstest/Executive.pm | 18 +- 2 files changed, 18 insertions(+), 17 deletions(-) diff --git a/Osstest.pm b/Osstest.pm index 4a763c6..969a2d0 100644

Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Andrew Cooper
On 09/10/15 09:25, Jan Beulich wrote: On 09.10.15 at 04:56, wrote: >> All existing commands ignore the parameter so this does >> not break the ABI. > Does it not? What about the debug mode clobbering of hypercall > argument registers? That is an implementation detail

Re: [Xen-devel] [PATCH 0/2] Bulk mem-share identical domains

2015-10-09 Thread Wei Liu
On Thu, Oct 08, 2015 at 03:07:19PM -0600, Tamas K Lengyel wrote: > In case you miss it, there is now soft-reset support which dumps all > > memory plus various states from one domain to another, and toolstack > > will take care of QEMU and various userspace bits. This might be useful > > to you? >

Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 14:46, wrote: > On Fri, 2015-10-09 at 13:29 +0100, Andrew Cooper wrote: >> On 09/10/15 09:25, Jan Beulich wrote: >> > > > > On 09.10.15 at 04:56, wrote: >> > > All existing commands ignore the parameter so this does >> > > not

Re: [Xen-devel] [OSSTEST PATCH v3 3/3] ts-logs-capture: include some cpupools info in the captured logs.

2015-10-09 Thread Ian Campbell
On Sat, 2015-10-03 at 02:39 +0200, Dario Faggioli wrote: > Signed-off-by: Dario Faggioli Acked-by: Ian Campbell There's probably no need for this to wait for the rest. > --- > Cc: Ian Jackson > Cc: Ian Campbell

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 15:41, wrote: > On 10/09/2015 02:51 AM, Jan Beulich wrote: > On 28.09.15 at 09:13, wrote: >>> When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation >>> is used, the existing tsc_get_info() calculates

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Haozhong Zhang
On Fri, Oct 09, 2015 at 12:31:53PM -0400, Boris Ostrovsky wrote: > On 10/09/2015 12:19 PM, Jan Beulich wrote: > On 09.10.15 at 18:09, wrote: > >>On 10/09/2015 11:11 AM, Jan Beulich wrote: > >>On 09.10.15 at 16:00, wrote: > On Fri,

[Xen-devel] [PATCH] x86/shadow: Fix missing newline in dprintk()

2015-10-09 Thread Andrew Cooper
to avoid console corruption. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Tim Deegan --- xen/arch/x86/mm/shadow/common.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/x86/mm/shadow/common.c

[Xen-devel] patch "x86/cpufreq: relocate the driver register function" breaks cpu hot(un)plug

2015-10-09 Thread Dario Faggioli
Hey, As far as my bisection goes, commit 49388f11d512bb92706ce046643bfbb3c1d963c9 "x86/cpufreq: relocate the driver register function" prevents me from hot unplugging pCPUs. Xen does not crash or anything, but dom0 is stalled. In fact, with current staging, here's what I see: root@Zhaman:~#

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-09 Thread Dario Faggioli
On Fri, 2015-10-09 at 14:05 +0100, Andrew Cooper wrote: > On 08/10/15 21:39, Dario Faggioli wrote: > > On Thu, 2015-10-08 at 16:20 +0100, Andrew Cooper wrote: > > > On 08/10/15 15:58, George Dunlap wrote: > > > > Generic scheduling code is called from interrupt contexts -- > > > > namely, > > > >

[Xen-devel] [OSSTEST PATCH] standalone-generate-dump-flight-runvars: Handle ^C properly

2015-10-09 Thread Ian Jackson
This is all mad. Signed-off-by: Ian Jackson --- standalone-generate-dump-flight-runvars | 24 1 file changed, 24 insertions(+) diff --git a/standalone-generate-dump-flight-runvars b/standalone-generate-dump-flight-runvars index

Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Andrew Cooper
On 09/10/15 13:46, Ian Campbell wrote: > On Fri, 2015-10-09 at 13:29 +0100, Andrew Cooper wrote: >> On 09/10/15 09:25, Jan Beulich wrote: >> On 09.10.15 at 04:56, wrote: All existing commands ignore the parameter so this does not break the ABI. >>> Does it

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Andrew Cooper
On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: > +rc = xc_version_len(ctx->xch, XENVER_build_id, _id, > BUILD_ID_LEN); > +if (rc > 0) { > +unsigned int i; > + > +info->build_id = (char *)malloc((rc * 2) + 1); > + > +for (i = 0; i < rc && (i + 1) * 2 <

[Xen-devel] [RFC OSSTEST v2] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-09 Thread Ian Campbell
And use this in standalone-generate-dump-flight-runvars. In general I don't think we are interested in the specific revision_* runvars when using this tool but it can be avoided using AP_FETCH_PLACEHOLDERS=n when calling standalone-generate-dump-flight-runvars. This is quicker even than using

Re: [Xen-devel] [PATCH v1] Add build-id to XENVER hypercall.

2015-10-09 Thread Konrad Rzeszutek Wilk
On Fri, Oct 09, 2015 at 01:15:42PM +0100, Andrew Cooper wrote: > On 09/10/15 09:17, Jan Beulich wrote: > On 09.10.15 at 04:56, wrote: > >> However they also change the behavior of the existing hypercall > >> for XENVER_[compile_info|changeset|commandline] and make

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-09 Thread Andrew Cooper
On 08/10/15 21:39, Dario Faggioli wrote: > On Thu, 2015-10-08 at 16:20 +0100, Andrew Cooper wrote: >> On 08/10/15 15:58, George Dunlap wrote: >>> Generic scheduling code is called from interrupt contexts -- >>> namely, >>> vcpu_wake() >> There are a lot of codepaths, but I cant see one which is

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 13:59 +0100, Andrew Cooper wrote: > On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: > > +rc = xc_version_len(ctx->xch, XENVER_build_id, _id, > > BUILD_ID_LEN); > > +if (rc > 0) { > > +unsigned int i; > > + > > +info->build_id = (char *)malloc((rc * 2)

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Andrew Cooper
On 09/10/15 14:06, Ian Campbell wrote: > On Fri, 2015-10-09 at 13:59 +0100, Andrew Cooper wrote: >> On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: >>> +rc = xc_version_len(ctx->xch, XENVER_build_id, _id, >>> BUILD_ID_LEN); >>> +if (rc > 0) { >>> +unsigned int i; >>> + >>> +

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 14:06 +0100, Ian Campbell wrote: > On Fri, 2015-10-09 at 13:59 +0100, Andrew Cooper wrote: > > On 09/10/15 03:56, Konrad Rzeszutek Wilk wrote: > > > +rc = xc_version_len(ctx->xch, XENVER_build_id, _id, > > > BUILD_ID_LEN); > > > +if (rc > 0) { > > > +unsigned

Re: [Xen-devel] [OSSTEST PATCH 3/3] smoke tests: Consider osstest revision when reusing builds

2015-10-09 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 3/3] smoke tests: Consider osstest revision when reusing builds"): > Probably a bad idea, but I wonder: would comparing the hostflags required > of the two build hosts take care of some of this? > > e.g. some random job I just pulled up: > >

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Boris Ostrovsky
On 10/09/2015 02:51 AM, Jan Beulich wrote: On 28.09.15 at 09:13, wrote: When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation is used, the existing tsc_get_info() calculates elapsed_nsec by scaling the host TSC with a ratio between guest TSC rate and

Re: [Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 22:56 -0400, Konrad Rzeszutek Wilk wrote: > If the hypervisor is built with we will display it. I think there is a word missing in this sentence. Perhaps "it" after "with", or better "a build_id" or "blah blah feature enabled". > @@ -5295,8 +5298,21 @@ const

Re: [Xen-devel] [OSSTEST PATCH 3/3] smoke tests: Consider osstest revision when reusing builds

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 14:22 +0100, Ian Jackson wrote: > > (is that worth mentioning? is it correct?) > > No. This reuse machinery does not consider the versions of anything - > except, now, osstest itself. This is because the other trees' > versions are supposed to be intercompatible. Ah, I'm

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Haozhong Zhang
On Fri, Oct 09, 2015 at 09:41:36AM -0400, Boris Ostrovsky wrote: > On 10/09/2015 02:51 AM, Jan Beulich wrote: > On 28.09.15 at 09:13, wrote: > >>When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation > >>is used, the existing tsc_get_info() calculates

Re: [Xen-devel] [PATCH v2 1/2] x86/mem-sharing: Bulk mem-sharing entire domains

2015-10-09 Thread Tamas K Lengyel
On Fri, Oct 9, 2015 at 1:51 AM, Jan Beulich wrote: > >>> On 08.10.15 at 22:57, wrote: > > --- a/xen/arch/x86/mm/mem_sharing.c > > +++ b/xen/arch/x86/mm/mem_sharing.c > > @@ -1293,6 +1293,37 @@ int relinquish_shared_pages(struct domain *d) > > return

Re: [Xen-devel] [PATCH OSSTEST v3] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-09 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v3] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder"): > And use this in standalone-generate-dump-flight-runvars. In general I > don't think we are interested in the specific revision_* runvars when > using this tool but when it

Re: [Xen-devel] [OSSTEST PATCH] standalone-generate-dump-flight-runvars: Handle ^C properly

2015-10-09 Thread Ian Jackson
Ian Jackson writes ("[OSSTEST PATCH] standalone-generate-dump-flight-runvars: Handle ^C properly"): > This is all mad. ... > +# I _think_ that that any signal which arrives before the assignment > +# to $SIG{} will definitely have caused our parent to vanish and us to > +# be reparented to pid 1

Re: [Xen-devel] [PATCH] x86/shadow: Fix missing newline in dprintk()

2015-10-09 Thread Tim Deegan
At 18:01 +0100 on 09 Oct (113710), Andrew Cooper wrote: > to avoid console corruption. > > Signed-off-by: Andrew Cooper Acked-by: Tim Deegan ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v2 1/2] x86/mem-sharing: Bulk mem-sharing entire domains

2015-10-09 Thread Tamas K Lengyel
On Fri, Oct 9, 2015 at 7:26 AM, Andrew Cooper wrote: > On 08/10/15 21:57, Tamas K Lengyel wrote: > > diff --git a/xen/arch/x86/mm/mem_sharing.c > b/xen/arch/x86/mm/mem_sharing.c > > index a95e105..4cdddb1 100644 > > --- a/xen/arch/x86/mm/mem_sharing.c > > +++

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Boris Ostrovsky
On 10/09/2015 12:51 PM, Haozhong Zhang wrote: On Fri, Oct 09, 2015 at 12:31:53PM -0400, Boris Ostrovsky wrote: On 10/09/2015 12:19 PM, Jan Beulich wrote: On 09.10.15 at 18:09, wrote: On 10/09/2015 11:11 AM, Jan Beulich wrote: On 09.10.15 at 16:00,

Re: [Xen-devel] [PATCH] libxc: remove superpages option for pv domains

2015-10-09 Thread Konrad Rzeszutek Wilk
On Fri, Oct 09, 2015 at 04:12:12PM +0100, Ian Campbell wrote: > On Thu, 2015-10-08 at 17:23 +0200, Juergen Gross wrote: > > The pv domain builder currently supports the additional flag > > "superpages" to build a pv domain with 2MB pages. This feature isn't > > being used by any component other

[Xen-devel] [libvirt test] 62727: tolerable FAIL - PUSHED

2015-10-09 Thread osstest service owner
flight 62727 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/62727/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-vhd 9 debian-di-installfail never pass test-armhf-armhf-libvirt-raw 9

Re: [Xen-devel] patch "x86/cpufreq: relocate the driver register function" breaks cpu hot(un)plug

2015-10-09 Thread Konrad Rzeszutek Wilk
On Fri, Oct 09, 2015 at 06:48:23PM +0200, Dario Faggioli wrote: > Hey, > > As far as my bisection goes, commit > 49388f11d512bb92706ce046643bfbb3c1d963c9 "x86/cpufreq: relocate the > driver register function" prevents me from hot unplugging pCPUs. > > Xen does not crash or anything, but dom0 is

Re: [Xen-devel] [OSSTEST PATCH 2/2] make-flight: create the vNUMA HVM test job

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-02 at 01:17 +0200, Dario Faggioli wrote: > make-flight | 36 ++-- FWIW the make-flight side of this looks fine to me. We discussed the save-restore allowances already. ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 16:35, wrote: > On Fri, Oct 09, 2015 at 12:51:32AM -0600, Jan Beulich wrote: >> >>> On 28.09.15 at 09:13, wrote: >> > When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation >> > is used, the existing

Re: [Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-09 Thread Ian Campbell
On Fri, 2015-10-09 at 08:38 -0600, Jan Beulich wrote: > > > > On 09.10.15 at 14:46, wrote: > > On Fri, 2015-10-09 at 13:29 +0100, Andrew Cooper wrote: > > > On 09/10/15 09:25, Jan Beulich wrote: > > > > > > > On 09.10.15 at 04:56, wrote: > > > > >

Re: [Xen-devel] [PATCH 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 19:45 +0100, Julien Grall wrote: > From Xen point of view, PSCI v0.2 and PSCI v1.0 are very similar. All > the PSCI calls used within Xen (PSCI_VERSION, CPU_ON, SYSTEM_OFF and > SYSTEM_RESET) behaves exactly the same. > > While there is no compatible string to represent PSCI

Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-09 Thread Jan Beulich
>>> On 09.10.15 at 16:00, wrote: > On Fri, Oct 09, 2015 at 09:41:36AM -0400, Boris Ostrovsky wrote: >> On 10/09/2015 02:51 AM, Jan Beulich wrote: >> On 28.09.15 at 09:13, wrote: >> >>When the TSC mode of a domain is TSC_MODE_DEFAULT and no

Re: [Xen-devel] [PATCH 2/2] xen/arm: Replace XEN_PSCI_* by PSCI_VERSION(major, minor)

2015-10-09 Thread Ian Campbell
On Thu, 2015-10-08 at 19:45 +0100, Julien Grall wrote: > It will avoid to introduce a new XEN_PSCI_* define every time we support > a new version of PSCI in Xen. > > Also fix the coding style in modified place. > > Signed-off-by: Julien Grall Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-09 Thread Julien Grall
Hi Ian, On 09/10/15 16:08, Ian Campbell wrote: > On Thu, 2015-10-08 at 19:45 +0100, Julien Grall wrote: >> From Xen point of view, PSCI v0.2 and PSCI v1.0 are very similar. All >> the PSCI calls used within Xen (PSCI_VERSION, CPU_ON, SYSTEM_OFF and >> SYSTEM_RESET) behaves exactly the same. >> >>