Re: [Xen-devel] [RFC v3 07/13] tables.h: add linker table support

2016-08-11 Thread Borislav Petkov
On Fri, Aug 12, 2016 at 05:51:29AM +0200, Luis R. Rodriguez wrote: > OK I've added CONFIG_BUILD_AVOID_BITROT. What does that do? -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- ___ Xen-devel mailing list

[Xen-devel] [qemu-mainline test] 100421: tolerable FAIL - PUSHED

2016-08-11 Thread osstest service owner
flight 100421 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100421/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100397

[Xen-devel] [PATCH] xen: credit1: fix a race when picking initial pCPU for a vCPU

2016-08-11 Thread Dario Faggioli
In the Credit1 hunk of 9f358ddd69463 ("xen: Have schedulers revise initial placement") csched_cpu_pick() is called without taking the runqueue lock of the (temporary) pCPU that the vCPU has been assigned to (e.g., in XEN_DOMCTL_max_vcpus). However, although 'hidden' in the IS_RUNQ_IDLE() macro,

Re: [Xen-devel] [RFC v3 07/13] tables.h: add linker table support

2016-08-11 Thread Luis R. Rodriguez
On Tue, Aug 09, 2016 at 05:55:08AM +0200, Borislav Petkov wrote: > On Mon, Aug 08, 2016 at 05:05:39PM +0200, Luis R. Rodriguez wrote: > > > So how can I disable those table-* things from even getting built? Avoid > > > using table-y? But then everything declared table-y will be built > > >

Re: [Xen-devel] Scheduler regression in 4.7

2016-08-11 Thread Dario Faggioli
On Thu, 2016-08-11 at 16:42 +0100, Andrew Cooper wrote: > On 11/08/16 15:28, Dario Faggioli wrote: > > On Thu, 2016-08-11 at 14:39 +0100, Andrew Cooper wrote: > > > It will be IS_RUNQ_IDLE() which is the problem. > > > > > Ok, that does one step of list traversing (the runq). What I didn't > >

[Xen-devel] 答复: xen does not support the 8G large bar

2016-08-11 Thread Gaofeng (GaoFeng, Euler)
Hi Jan, Thanks for your reply. Qemu-xen seems that has problem for support 8G large Bar. I think this patch is not perfect: https://xenbits.xen.org/gitweb/?p=qemu-xen.git;a=commitdiff;h=aabc8530c7ba2be89e21463f051056ad7c255e6e Because I found upper 32bit bar of 8G large bar was not register.

Re: [Xen-devel] [V4 PATCH 2/2] mips/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-11 Thread Dave Young
On 08/10/16 at 05:09pm, Hidehiro Kawai wrote: > Daniel Walker reported problems which happens when > crash_kexec_post_notifiers kernel option is enabled > (https://lkml.org/lkml/2015/6/24/44). > > In that case, smp_send_stop() is called before entering kdump routines > which assume other CPUs are

Re: [Xen-devel] [V4 PATCH 1/2] x86/panic: Replace smp_send_stop() with kdump friendly version in panic path

2016-08-11 Thread Dave Young
Hi Hidehiro Thanks for the update. On 08/10/16 at 05:09pm, Hidehiro Kawai wrote: > Daniel Walker reported problems which happens when > crash_kexec_post_notifiers kernel option is enabled > (https://lkml.org/lkml/2015/6/24/44). > > In that case, smp_send_stop() is called before entering kdump

[Xen-devel] dependences for backporting to 4.6 [was: Re: [PATCH 2/3] xen: Have schedulers revise initial placement]

2016-08-11 Thread Dario Faggioli
On Fri, 2016-08-05 at 07:24 -0600, Jan Beulich wrote: > I'd really like to have those backported, but I have to ask one > of you to identify which prereq-s are needed on 4.6 and 4.5 > (I'll revert them from 4.5 right away, but I'll wait for an osstest > flight to confirm the same issue exists on

Re: [Xen-devel] [PATCH 2/3] xen: Have schedulers revise initial placement

2016-08-11 Thread Dario Faggioli
On Thu, 2016-08-11 at 16:51 +0100, Andrew Cooper wrote: > On 11/08/16 15:59, Dario Faggioli wrote: > >  > > Which, I think needs at least this hunk (from 6b53bb4ab3c9  "sched: > > better handle (not) inserting idle vCPUs in runqueues"): > > > > diff --git a/xen/common/schedule.c

[Xen-devel] [ovmf baseline-only test] 66965: regressions - FAIL

2016-08-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66965 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66965/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-xsm 5 xen-build

[Xen-devel] [PATCH 3/3] Significant changes to decision making; some new roles and minor changes

2016-08-11 Thread Lars Kurth
Added RTC Policy Added +2 ... -2 scheme for votes Clarified lazy consensus Added Informal Votes/Surveys Added Project Team Leadership role and Decision making Changed Project Wide Decision making: per project based scheme Clarified scope of Decision making Modified sections which have

[Xen-devel] [PATCH 1/3] Code motion changes to make real patches easier to read

2016-08-11 Thread Lars Kurth
Added TOC Re-arranged sections compared to previous version of document Added new anchors where needed Split Roles section into two sections The actual content was not changed (with the exception of minor typos that I spotted) Signed-off-by: Lars Kurth ---

[Xen-devel] [PATCH 2/3] Added comment sections to highight problem areas

2016-08-11 Thread Lars Kurth
These are marked by - ... - blocks that will be removed in the published version Signed-off-by: Lars Kurth --- governance.pandoc

[Xen-devel] [PATCH 0/3] Significant changes to Xen Project Governance (governance.html)

2016-08-11 Thread Lars Kurth
I made some significant proposed changes to governance.html based on a number of issues that were raised in a number of surveys last year, and via other means, as well as in the recent discussions related to governance.html changes (the issue of too many committers in XAPI and XAPI being able

Re: [Xen-devel] [Very RFC PATCH] Livepatch - initial ARM64/32 support.

2016-08-11 Thread Konrad Rzeszutek Wilk
On Thu, Aug 11, 2016 at 12:05:33PM -0700, Stefano Stabellini wrote: > On Thu, 11 Aug 2016, Julien Grall wrote: > > On 09/08/2016 06:18, Konrad Rzeszutek Wilk wrote: > > > Hey! > > > > Hi Konrad, > > > > > Over the last couple of months in my spare time I was playing > > > with making livepatch

[Xen-devel] [xen-unstable test] 100419: regressions - FAIL

2016-08-11 Thread osstest service owner
flight 100419 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100419/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-cubietruck 16 guest-start.2 fail REGR. vs. 100408 Regressions which

[Xen-devel] [linux-3.18 baseline-only test] 66964: regressions - FAIL

2016-08-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66964 linux-3.18 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66964/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 16

Re: [Xen-devel] [PATCH] xen: handle inbound migration of VMs without ioreq server pages

2016-08-11 Thread Stefano Stabellini
On Tue, 9 Aug 2016, Paul Durrant wrote: > > -Original Message- > > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > > Sent: 09 August 2016 16:19 > > To: Paul Durrant > > Cc: xen-de...@lists.xenproject.org; qemu-de...@nongnu.org; Stefano > > Stabellini > > Subject: Re: [PATCH] xen:

[Xen-devel] [seabios test] 100418: tolerable FAIL - PUSHED

2016-08-11 Thread osstest service owner
flight 100418 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/100418/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100366

Re: [Xen-devel] [PATCH v3 19/19] mini-os: repair build system

2016-08-11 Thread Samuel Thibault
Juergen Gross, on Thu 11 Aug 2016 11:18:22 +0200, wrote: > The build system of Mini-OS is using different settings for arch/*/* > than for the rest of the tree. The main reasons are that Config.mk is > included only conditionally in the top level Makefile, while minios.mk > isn't included by the

[Xen-devel] [qemu-mainline test] 100410: regressions - FAIL

2016-08-11 Thread osstest service owner
flight 100410 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100410/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-arndale 15 guest-start/debian.repeat fail REGR. vs. 100397 Regressions

Re: [Xen-devel] [Very RFC PATCH] Livepatch - initial ARM64/32 support.

2016-08-11 Thread Stefano Stabellini
On Thu, 11 Aug 2016, Julien Grall wrote: > On 09/08/2016 06:18, Konrad Rzeszutek Wilk wrote: > > Hey! > > Hi Konrad, > > > Over the last couple of months in my spare time I was playing > > with making livepatch work with ARM64 (using the FoundationModel > > simulator) and I finally got it

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

2016-08-11 Thread Stefano Stabellini
On Thu, 11 Aug 2016, Lars Kurth wrote: > On 11/08/2016 01:51, "Stefano Stabellini" wrote: > > >> +Developer's Certificate of Origin > >> +- > >> + > >> +All patches to the Xen Project code base must include the the line > >

Re: [Xen-devel] [Very RFC PATCH] Livepatch - initial ARM64/32 support.

2016-08-11 Thread Julien Grall
On 09/08/2016 06:18, Konrad Rzeszutek Wilk wrote: Hey! Hi Konrad, Over the last couple of months in my spare time I was playing with making livepatch work with ARM64 (using the FoundationModel simulator) and I finally got it working tonight. Congrats! Sending out the patches just in

Re: [Xen-devel] [Very RFC PATCH 2/3] insn: Add arch64_insn_gen_branch_imm to generate branch

2016-08-11 Thread Julien Grall
Hi Konrad, On 09/08/2016 06:18, Konrad Rzeszutek Wilk wrote: We copied from Linux code (v4.7) the code that generates the unconditional branch instructions. This is mostly from Linux commit c0cafbae20d2878883ec3c06d6ea30ff38a6bf92 "arm64: introduce aarch64_insn_gen_branch_reg()" However the

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

2016-08-11 Thread Stefano Stabellini
On Thu, 11 Aug 2016, George Dunlap wrote: > On 11/08/16 01:51, Stefano Stabellini wrote: > > On Wed, 10 Aug 2016, Lars Kurth wrote: > >> COPYING file: > >> The motivation of this change is to make it easier for new > >> contributors to conduct a license and patent review, WITHOUT > >> changing any

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-11 Thread Boris Ostrovsky
On 08/11/2016 02:19 PM, Andrew Cooper wrote: > On 11/08/16 19:16, Boris Ostrovsky wrote: >> On 08/11/2016 12:40 PM, Wei Liu wrote: >>> On Thu, Aug 04, 2016 at 05:06:51PM -0400, Boris Ostrovsky wrote: Load ACPI modules into guest space Signed-off-by: Boris Ostrovsky

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Ian Jackson
Andrew Cooper writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for getting output"): > On 11/08/16 18:51, Wei Liu wrote: > > I'm pretty out of idea here. > > Because of XTF's behaviour (waiting for xenconsoled to catch up), you > know for certain that once `xl create` has finished,

Re: [Xen-devel] [PATCH RFC 06/12] xen/x86: populate PVHv2 Dom0 physical memory map

2016-08-11 Thread Andrew Cooper
On 05/08/16 10:40, Roger Pau Monne wrote: > On Thu, Aug 04, 2016 at 07:43:39PM +0100, Andrew Cooper wrote: >> On 02/08/16 10:19, Roger Pau Monne wrote: >>> On Fri, Jul 29, 2016 at 08:04:12PM +0100, Andrew Cooper wrote: On 29/07/16 17:29, Roger Pau Monne wrote: > +/* Calculate the biggest

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-11 Thread Andrew Cooper
On 11/08/16 19:16, Boris Ostrovsky wrote: > On 08/11/2016 12:40 PM, Wei Liu wrote: >> On Thu, Aug 04, 2016 at 05:06:51PM -0400, Boris Ostrovsky wrote: >>> Load ACPI modules into guest space >>> >>> Signed-off-by: Boris Ostrovsky >>> --- >>> v2: >>> * New patch, loosely

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-11 Thread Boris Ostrovsky
On 08/11/2016 12:40 PM, Wei Liu wrote: > On Thu, Aug 04, 2016 at 05:06:51PM -0400, Boris Ostrovsky wrote: >> Load ACPI modules into guest space >> >> Signed-off-by: Boris Ostrovsky >> --- >> v2: >> * New patch, loosely based on Shannon's ARM patch >> >>

Re: [Xen-devel] [PATCH] replace bogus -ENOSYS uses

2016-08-11 Thread Andrew Cooper
On 09/08/16 11:40, Jan Beulich wrote: > --- a/xen/arch/x86/cpu/mtrr/main.c > +++ b/xen/arch/x86/cpu/mtrr/main.c > @@ -332,7 +332,7 @@ int mtrr_add_page(unsigned long base, un > if ((type == MTRR_TYPE_WRCOMB) && !have_wrcomb()) { > printk(KERN_WARNING >

Re: [Xen-devel] [PATCH v2 22/23] libxl/acpi: Build ACPI tables for HVMlite guests

2016-08-11 Thread Boris Ostrovsky
On 08/11/2016 12:36 PM, Wei Liu wrote: > >> diff --git a/tools/libxl/libxl_x86_acpi.c b/tools/libxl/libxl_x86_acpi.c >> new file mode 100644 >> index 000..88e69e2 >> --- /dev/null >> +++ b/tools/libxl/libxl_x86_acpi.c > I only skimmed this file. I think it's mostly code movement and believe >

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 11:50:44AM -0600, Tamas K Lengyel wrote: > On Thu, Aug 11, 2016 at 11:17 AM, Wei Liu wrote: > > On Wed, Aug 10, 2016 at 09:00:15AM -0600, Tamas K Lengyel wrote: > >> Currently setting altp2mhvm=1 in the domain configuration allows access to > >> the >

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Andrew Cooper
On 11/08/16 18:51, Wei Liu wrote: > On Thu, Aug 11, 2016 at 06:17:31PM +0100, Ian Jackson wrote: >> Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for >> getting output"): >>> On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: This is perverse. Why not just open

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 06:17:31PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for getting > output"): > > On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: > > > This is perverse. Why not just open it O_CREAT|O_RDONLY ? If Python > >

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

2016-08-11 Thread Tamas K Lengyel
On Thu, Aug 11, 2016 at 11:17 AM, Wei Liu wrote: > On Wed, Aug 10, 2016 at 09:00:15AM -0600, Tamas K Lengyel wrote: >> Currently setting altp2mhvm=1 in the domain configuration allows access to >> the >> altp2m interface for both in-guest and external privileged tools. This

[Xen-devel] [linux-4.1 baseline-only test] 66963: tolerable trouble: blocked/broken/fail/pass

2016-08-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66963 linux-4.1 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66963/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-ovmf-amd64 3 host-install(3)

Re: [Xen-devel] [PATCH 5/7] x86emul: don't special case fetching unsigned 8-bit immediates

2016-08-11 Thread Andrew Cooper
On 11/08/16 17:44, Jan Beulich wrote: On 11.08.16 at 18:32, wrote: >> On 11/08/16 13:06, Jan Beulich wrote: >>> @@ -2893,7 +2894,6 @@ x86_emulate( >>> goto swint; >>> >>> case 0xcd: /* int imm8 */ >>> -src.val = insn_fetch_type(uint8_t); >>>

Re: [Xen-devel] [PATCH 1/7] x86emul: don't special case fetching the immediate of PUSH

2016-08-11 Thread Andrew Cooper
On 11/08/16 14:26, Jan Beulich wrote: On 11.08.16 at 14:58, wrote: >> On 11/08/16 13:03, Jan Beulich wrote: >>> These immediates follow the standard patterns in all modes, so they're >>> better fetched by the generic source operand handling code. >>> >>> To

Re: [Xen-devel] [PATCH 7/7] x86emul: introduce SrcImm16

2016-08-11 Thread Andrew Cooper
On 11/08/16 13:07, Jan Beulich wrote: > ... and use it for RET, LRET, and ENTER processing to limit the amount > of "manual" insn bytes fetching. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 06:21:55PM +0100, Andrew Cooper wrote: > On 11/08/16 18:17, Ian Jackson wrote: > > Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for > > getting output"): > >> On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: > >>> This is perverse. Why not

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Andrew Cooper
On 11/08/16 18:17, Ian Jackson wrote: > Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for getting > output"): >> On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: >>> This is perverse. Why not just open it O_CREAT|O_RDONLY ? If Python >>> can't do that then

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

2016-08-11 Thread Wei Liu
On Wed, Aug 10, 2016 at 09:00:15AM -0600, Tamas K Lengyel wrote: > Currently setting altp2mhvm=1 in the domain configuration allows access to the > altp2m interface for both in-guest and external privileged tools. This poses > a problem for use-cases where only external access should be allowed,

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Ian Jackson
Wei Liu writes ("Re: [XTF PATCH v2] xtf-runner: support two modes for getting output"): > On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: > > This is perverse. Why not just open it O_CREAT|O_RDONLY ? If Python > > can't do that then unconditionally opening it O_CREAT|O_RDWR would

Re: [Xen-devel] [PATCH v2 20/25] arm/altp2m: Add altp2m paging mechanism.

2016-08-11 Thread Sergej Proskurin
Hi Julien, On 08/11/2016 10:47 AM, Julien Grall wrote: > > > On 10/08/2016 11:32, Sergej Proskurin wrote: >> Hi Julien, > > Hello Sergej, > > [...] > >> switch ( fsc ) >> { >> +case FSC_FLT_TRANS: >> +{ >> +if ( altp2m_active(d) ) >> +

Re: [Xen-devel] [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 05:17:56PM +0100, Ian Jackson wrote: > Running XTF in osstest is likely to produce failures where multiple > steps fail interestingly. We would like to prefer to report, and > bisect, earlier steps. > > This series does that. > > Wei, NB, this has a conflict with the XTF

Re: [Xen-devel] [PATCH 3/3] tools: add config parameter for maximum memory of xenstore domain

2016-08-11 Thread Wei Liu
On Mon, Aug 08, 2016 at 10:28:29AM +0200, Juergen Gross wrote: > Add a parameter to xencommons configuration file for specifying the > maximum memory size of the xenstore domain. > > Signed-off-by: Juergen Gross Acked-by: Wei Liu

Re: [Xen-devel] [PATCH 1/3] tools: add --maxmem parameter to init-xenstore-domain

2016-08-11 Thread Wei Liu
On Mon, Aug 08, 2016 at 10:28:27AM +0200, Juergen Gross wrote: > Add a parameter to specify the maximum memory size of the xenstore > domain. In case the xenstore domain supports ballooning it will be > capable to adjust its own size according to its memory needs. > > The maximum memory size can

Re: [Xen-devel] [PATCH 2/3] stubdom: add CONFIG_BALLOON to xenstore config

2016-08-11 Thread Wei Liu
On Mon, Aug 08, 2016 at 10:28:28AM +0200, Juergen Gross wrote: > Compile xenstore stubdom with ballooning support. > > Signed-off-by: Juergen Gross Acked-by: Wei Liu ___ Xen-devel mailing list

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 05:27:56PM +0100, Ian Jackson wrote: > Wei Liu writes ("[XTF PATCH v2] xtf-runner: support two modes for getting > output"): > > We need two modes for getting output: > ... > > +logfile = open(fn, "rb") > > +except IOError as e: > > +# Create file if it

[Xen-devel] [xen-unstable-smoke test] 100417: tolerable all pass - PUSHED

2016-08-11 Thread osstest service owner
flight 100417 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100417/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl

Re: [Xen-devel] [PATCH 5/7] x86emul: don't special case fetching unsigned 8-bit immediates

2016-08-11 Thread Jan Beulich
>>> On 11.08.16 at 18:32, wrote: > On 11/08/16 13:06, Jan Beulich wrote: >> @@ -2893,7 +2894,6 @@ x86_emulate( >> goto swint; >> >> case 0xcd: /* int imm8 */ >> -src.val = insn_fetch_type(uint8_t); >> swint_type = x86_swint_int; >>

Re: [Xen-devel] [PATCH 6/7] x86emul: use DstEax where possible

2016-08-11 Thread Andrew Cooper
On 11/08/16 13:06, Jan Beulich wrote: > While it avoids just a few instructions, we should nevertheless make > use of generic code as much as possible. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper

Re: [Xen-devel] [PATCH v2 23/23] libxc/xc_dom_core: Copy ACPI tables to guest space

2016-08-11 Thread Wei Liu
On Thu, Aug 04, 2016 at 05:06:51PM -0400, Boris Ostrovsky wrote: > Load ACPI modules into guest space > > Signed-off-by: Boris Ostrovsky > --- > v2: > * New patch, loosely based on Shannon's ARM patch > > tools/libxc/xc_dom_core.c | 92 >

Re: [Xen-devel] [PATCH v2 22/23] libxl/acpi: Build ACPI tables for HVMlite guests

2016-08-11 Thread Wei Liu
On Thu, Aug 04, 2016 at 05:06:50PM -0400, Boris Ostrovsky wrote: [...] > > distclean: clean > $(RM) -f xenlight.pc.in xlutil.pc.in > diff --git a/tools/libxl/libxl_arch.h b/tools/libxl/libxl_arch.h > index 34a853c..7c6536b 100644 > --- a/tools/libxl/libxl_arch.h > +++

Re: [Xen-devel] [PATCH v2 19/23] libxl/acpi: Add ACPI e820 entry

2016-08-11 Thread Wei Liu
On Thu, Aug 04, 2016 at 05:06:47PM -0400, Boris Ostrovsky wrote: > Add entry for ACPI tables created for PVHv2 guests to e820 map. > > Signed-off-by: Boris Ostrovsky > --- > v2: > * Deal with multiple acpi_modules. > > tools/libxl/libxl_dom.c | 8 >

Re: [Xen-devel] [PATCH v2 18/23] libxc/libxl: Allow multiple ACPI modules

2016-08-11 Thread Wei Liu
On Thu, Aug 04, 2016 at 05:06:46PM -0400, Boris Ostrovsky wrote: > Provide ability to load multiple ACPI modules. Thie feature is needed > by PVHv2 guests and will be used in subsequent patches. > > We assume that PVHv2 guests do not load their ACPI modules specified > in the configuration file.

Re: [Xen-devel] [PATCH 5/7] x86emul: don't special case fetching unsigned 8-bit immediates

2016-08-11 Thread Andrew Cooper
On 11/08/16 13:06, Jan Beulich wrote: > @@ -2893,7 +2894,6 @@ x86_emulate( > goto swint; > > case 0xcd: /* int imm8 */ > -src.val = insn_fetch_type(uint8_t); > swint_type = x86_swint_int; > swint: > rc = inject_swint(swint_type, src.val, I would be

Re: [Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Ian Jackson
Wei Liu writes ("[XTF PATCH v2] xtf-runner: support two modes for getting output"): > We need two modes for getting output: ... > +logfile = open(fn, "rb") > +except IOError as e: > +# Create file if it doesn't exist > +if e.errno == 2: > +logfile =

[Xen-devel] [OSSTEST PATCH 1/7] duration_estimator: Introduce $duration_duration_qtxt and @d_d_args

2016-08-11 Thread Ian Jackson
This is going to make it easier to, conditionally, make this query more complicated. No functional change. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Osstest/Executive.pm

[Xen-devel] [OSSTEST PATCH 5/7] sg-report-flight: Fix misleading value in call to duration_estimator

2016-08-11 Thread Ian Jackson
The third argument is $debug. If it's falseish, a default subref is used instead. Passing '' is confusing; pass undef instead. No functional change. Signed-off-by: Ian Jackson --- sg-report-flight | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [OSSTEST PATCH 7/7] sg-report-flight: Report earlier, earlier step failures

2016-08-11 Thread Ian Jackson
The order of results reported by sg-report-flight determines which testid the bisector will try to work on first. It also determines the order in which failures are shown in the email reports. We currently sort them by the duration estimate (for each failure's containing job). We should prefer

[Xen-devel] [OSSTEST PATCH 2/7] duration_estimator: Reformat inner SQL query with more newlines

2016-08-11 Thread Ian Jackson
This is going to make the next patch, which adds more complication, easier to read. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/Osstest/Executive.pm b/Osstest/Executive.pm index

[Xen-devel] [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection

2016-08-11 Thread Ian Jackson
Running XTF in osstest is likely to produce failures where multiple steps fail interestingly. We would like to prefer to report, and bisect, earlier steps. This series does that. Wei, NB, this has a conflict with the XTF pre series that you have taken over. The conflict is not trivial, but

[Xen-devel] [OSSTEST PATCH 6/7] sg-report-flight: Fix a missing newline in a DEBUG message

2016-08-11 Thread Ian Jackson
Signed-off-by: Ian Jackson --- sg-report-flight | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sg-report-flight b/sg-report-flight index 47ecf07..d1acb60 100755 --- a/sg-report-flight +++ b/sg-report-flight @@ -569,7 +569,7 @@ sub

[Xen-devel] [OSSTEST PATCH 4/7] duration_estimator: Be able to estimate job duration up to a particular step

2016-08-11 Thread Ian Jackson
If this is passed, we are interested only in the duration up to and including the specified test step. (If the specified test step is not present or didn't have a recorded finish, we look at the whole job.) Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 23

[Xen-devel] [OSSTEST PATCH 3/7] duration_estimator: Reorganise inner SQL query to use WITH

2016-08-11 Thread Ian Jackson
We are going to, sometimes, want to ask more complicated questions about the steps in the job of interest. No functional change, except perhaps to query optimisation. Signed-off-by: Ian Jackson --- Osstest/Executive.pm | 13 + 1 file changed, 9

Re: [Xen-devel] [PATCH] gnttab: Add gntdev device mappings for FreeBSD

2016-08-11 Thread Wei Liu
On Thu, Aug 04, 2016 at 06:23:51PM +0530, Akshay Jaggi wrote: > Add grant table userspace device mappings for > FreeBSD (enables support for qdisk backend > on FreeBSD Dom0). > > Signed-off-by: Akshay Jaggi > --- > tools/include/xen-sys/FreeBSD/gntdev.h | 118

Re: [Xen-devel] [PATCH] gnttab: Add gntdev device mappings for FreeBSD

2016-08-11 Thread Wei Liu
On Sun, Aug 07, 2016 at 03:48:44PM +0530, Akshay Jaggi wrote: > > > +int fd = xgt->fd, rc; > > > +struct ioctl_gntdev_set_max_grants max_grants = { .count = count }; > > > + > > > +rc = ioctl(fd, IOCTL_GNTDEV_SET_MAX_GRANTS, _grants); > > > +if (rc) { > > > > Coding style, the

Re: [Xen-devel] [PATCH v2 23/25] arm/altp2m: Extend libxl to activate altp2m on ARM.

2016-08-11 Thread Wei Liu
Sorry for the late reply. On Tue, Aug 02, 2016 at 04:07:53PM +0200, Sergej Proskurin wrote: > Hi Wei, [...] > >> @@ -901,8 +903,8 @@ static void initiate_domain_create(libxl__egc *egc, > >> > >> if (d_config->c_info.type == LIBXL_DOMAIN_TYPE_HVM && > >>

Re: [Xen-devel] [PATCH 2/3] xen: Have schedulers revise initial placement

2016-08-11 Thread Andrew Cooper
On 11/08/16 15:59, Dario Faggioli wrote: > On Fri, 2016-08-05 at 07:24 -0600, Jan Beulich wrote: >> I'd really like to have those backported, but I have to ask one >> of you to identify which prereq-s are needed on 4.6 and 4.5 >> (I'll revert them from 4.5 right away, but I'll wait for an osstest

Re: [Xen-devel] Scheduler regression in 4.7

2016-08-11 Thread Andrew Cooper
On 11/08/16 15:28, Dario Faggioli wrote: > On Thu, 2016-08-11 at 14:39 +0100, Andrew Cooper wrote: >> On 11/08/16 14:24, George Dunlap wrote: >>> On 11/08/16 12:35, Andrew Cooper wrote: The actual cause is _csched_cpu_pick() falling over LIST_POISON, which happened to occur at the

Re: [Xen-devel] Save/Restore is not working properly

2016-08-11 Thread Wei Liu
On Wed, Aug 10, 2016 at 02:24:09PM +0100, George Dunlap wrote: > On Wed, Aug 10, 2016 at 12:11 PM, Roger Pau Monné > wrote: > > On Sun, Aug 07, 2016 at 07:51:14PM +0430, Cendrin Sa wrote: > >> Hi, > >> I was searching a way to clone a machine using both memory and disk > >>

Re: [Xen-devel] [PATCH v3 19/19] mini-os: repair build system

2016-08-11 Thread Wei Liu
On Thu, Aug 11, 2016 at 11:18:22AM +0200, Juergen Gross wrote: > The build system of Mini-OS is using different settings for arch/*/* > than for the rest of the tree. The main reasons are that Config.mk is > included only conditionally in the top level Makefile, while minios.mk > isn't included by

[Xen-devel] [ovmf test] 100413: all pass - PUSHED

2016-08-11 Thread osstest service owner
flight 100413 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100413/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 926059304e8377fc37bb848d06d9419f419d93ff baseline version: ovmf

[Xen-devel] [xen-unstable test] 100408: tolerable FAIL - PUSHED

2016-08-11 Thread osstest service owner
flight 100408 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100408/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-amd64-rumpuserxen 6 xen-buildfail like 100377 build-i386-rumpuserxen

Re: [Xen-devel] [PATCH 4/7] x86emul: fold SrcImmByte fetching

2016-08-11 Thread Andrew Cooper
On 11/08/16 15:09, Jan Beulich wrote: On 11.08.16 at 15:41, wrote: >> On 11/08/16 13:05, Jan Beulich wrote: >>> There's no need for having identical code spelled out twice. >>> >>> Signed-off-by: Jan Beulich >>> >>> ---

Re: [Xen-devel] [PATCH 2/3] xen: Have schedulers revise initial placement

2016-08-11 Thread Dario Faggioli
On Fri, 2016-08-05 at 07:24 -0600, Jan Beulich wrote: > I'd really like to have those backported, but I have to ask one > of you to identify which prereq-s are needed on 4.6 and 4.5 > (I'll revert them from 4.5 right away, but I'll wait for an osstest > flight to confirm the same issue exists on

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

2016-08-11 Thread Jan Beulich
>>> On 11.08.16 at 16:37, wrote: > On Aug 11, 2016 06:02, "Jan Beulich" wrote: >> >> >>> On 10.08.16 at 17:00, wrote: >> > @@ -5238,18 +5238,19 @@ static int do_altp2m_op( >> > goto out; >> > } >> > >> > -

Re: [Xen-devel] [PATCH V3] common/vm_event: synchronize vCPU state in vm_event_resume()

2016-08-11 Thread Tamas K Lengyel
On Aug 11, 2016 03:29, "Razvan Cojocaru" wrote: > > Vm_event_vcpu_pause() needs to use vcpu_pause_nosync() in order > for the current vCPU to not get stuck. A consequence of this is > that the custom vm_event response handlers will not always see > the real vCPU state

Re: [Xen-devel] [PATCH v2 01/25] arm/altp2m: Add first altp2m HVMOP stubs.

2016-08-11 Thread Tamas K Lengyel
On Aug 11, 2016 02:18, "Julien Grall" wrote: > > Hello Tamas, > > > On 10/08/2016 16:49, Tamas K Lengyel wrote: >> >> On Aug 10, 2016 03:52, "Julien Grall" > > wrote: >>> >>> On 09/08/2016 21:16, Tamas K Lengyel wrote:

Re: [Xen-devel] [PATCH v2 2/2] x86/altp2m: allow specifying external-only use-case

2016-08-11 Thread Tamas K Lengyel
On Aug 11, 2016 06:02, "Jan Beulich" wrote: > > >>> On 10.08.16 at 17:00, wrote: > > @@ -5238,18 +5238,19 @@ static int do_altp2m_op( > > goto out; > > } > > > > -if ( (rc = xsm_hvm_altp2mhvm_op(XSM_TARGET, d)) ) > > +if (

Re: [Xen-devel] Scheduler regression in 4.7

2016-08-11 Thread Dario Faggioli
On Thu, 2016-08-11 at 14:39 +0100, Andrew Cooper wrote: > On 11/08/16 14:24, George Dunlap wrote: > > On 11/08/16 12:35, Andrew Cooper wrote: > > > The actual cause is _csched_cpu_pick() falling over LIST_POISON, > > > which > > > happened to occur at the same time as a domain was shutting > > >

[Xen-devel] [XTF PATCH v2] xtf-runner: support two modes for getting output

2016-08-11 Thread Wei Liu
We need two modes for getting output: 1. Use console directly with newer (>=4.8) Xen 2. Use log files for older Xen This patch implements both. The default behaviour is to use the console mode. Signed-off-by: Wei Liu --- v2: delete auto selection mode, squash all patches

Re: [Xen-devel] [PATCH v4 06/19] x86/boot/reloc: create generic alloc and copy functions

2016-08-11 Thread Jan Beulich
>>> On 11.08.16 at 16:12, wrote: On 06.08.16 at 01:04, wrote: >> --- a/xen/arch/x86/boot/reloc.c >> +++ b/xen/arch/x86/boot/reloc.c >> @@ -32,60 +32,69 @@ typedef unsigned int u32; >> >> static u32 alloc; >> >> -static void

Re: [Xen-devel] [PATCH v4 08/19] x86/boot/reloc: Rename some variables and rearrange code a bit

2016-08-11 Thread Jan Beulich
>>> On 06.08.16 at 01:04, wrote: > Replace mbi with mbi_out and mbi_old with mbi_in and rearrange code > a bit to make it more readable. Additionally, this way multiboot (v1) > protocol implementation and future multiboot2 protocol implementation > will use the same

Re: [Xen-devel] [PATCH v4 06/19] x86/boot/reloc: create generic alloc and copy functions

2016-08-11 Thread Jan Beulich
>>> On 06.08.16 at 01:04, wrote: > --- a/xen/arch/x86/boot/reloc.c > +++ b/xen/arch/x86/boot/reloc.c > @@ -32,60 +32,69 @@ typedef unsigned int u32; > > static u32 alloc; > > -static void *reloc_mbi_struct(void *old, unsigned int bytes) > +static u32 alloc_mem(u32

[Xen-devel] [xen-unstable-smoke test] 100415: tolerable all pass - PUSHED

2016-08-11 Thread osstest service owner
flight 100415 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100415/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl

Re: [Xen-devel] [PATCH 4/7] x86emul: fold SrcImmByte fetching

2016-08-11 Thread Jan Beulich
>>> On 11.08.16 at 15:41, wrote: > On 11/08/16 13:05, Jan Beulich wrote: >> There's no need for having identical code spelled out twice. >> >> Signed-off-by: Jan Beulich >> >> --- a/xen/arch/x86/x86_emulate/x86_emulate.c >> +++

Re: [Xen-devel] [PATCH] Xen: remove -fshort-wchar gcc flag

2016-08-11 Thread Jan Beulich
>>> On 11.08.16 at 16:01, wrote: > On Thursday, August 11, 2016 6:51:33 AM CEST Jan Beulich wrote: >> >>> On 11.08.16 at 14:39, wrote: >> > A previous patch added the --no-wchar-size-warning to the Makefile to >> > avoid this harmless warning: >> > >> >

Re: [Xen-devel] [PATCH] Xen: remove -fshort-wchar gcc flag

2016-08-11 Thread Arnd Bergmann
On Thursday, August 11, 2016 6:51:33 AM CEST Jan Beulich wrote: > >>> On 11.08.16 at 14:39, wrote: > > A previous patch added the --no-wchar-size-warning to the Makefile to > > avoid this harmless warning: > > > > arm-linux-gnueabi-ld: warning: drivers/xen/efi.o uses 2-byte

Re: [Xen-devel] [PATCH v4 05/19] x86/boot: call reloc() using stdcall calling convention

2016-08-11 Thread Jan Beulich
>>> On 06.08.16 at 01:04, wrote: > Current reloc() call method makes confusion and does not scale > well for more arguments. And patch adding multiboot2 protocol > support have to pass 3 arguments instead of 2. Hence, move reloc() > call to stdcall calling convention.

Re: [Xen-devel] [PATCH v4 04/19] x86/boot/reloc: reduce assembly usage as much as possible

2016-08-11 Thread Jan Beulich
>>> On 06.08.16 at 01:04, wrote: > Next patch will leave just required jmp instruction > in xen/x86/boot/reloc.c. I can't make sense of this now, and it'll get even more problematic for archaeologists if the two patches don't get committed one right after the other.

Re: [Xen-devel] [PATCH 4/7] x86emul: fold SrcImmByte fetching

2016-08-11 Thread Andrew Cooper
On 11/08/16 13:05, Jan Beulich wrote: > There's no need for having identical code spelled out twice. > > Signed-off-by: Jan Beulich > > --- a/xen/arch/x86/x86_emulate/x86_emulate.c > +++ b/xen/arch/x86/x86_emulate/x86_emulate.c > @@ -1979,9 +1979,12 @@ x86_emulate( >

Re: [Xen-devel] [PATCH 3/7] x86emul: all push flavors are data moves

2016-08-11 Thread Andrew Cooper
On 11/08/16 13:04, Jan Beulich wrote: > Make all paths leading to the "push" label have the Mov flag set, and > ASSERT() that to be the case. For the opcode FF group the adjustment is > benign for the paths not leading to "push", as they all set dst.type to > OP_NONE > > Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH v4 03/19] x86/boot: create *.lnk files with linker script

2016-08-11 Thread Jan Beulich
>>> On 06.08.16 at 01:04, wrote: > Newer GCC (e.g. gcc version 5.1.1 20150618 (Red Hat 5.1.1-4) (GCC)) does > some code optimizations by creating data sections (e.g. jump addresses > for C switch/case are calculated using data in .rodata section). This > thing is not

Re: [Xen-devel] Scheduler regression in 4.7

2016-08-11 Thread Andrew Cooper
On 11/08/16 14:24, George Dunlap wrote: > On 11/08/16 12:35, Andrew Cooper wrote: >> Hello, >> >> XenServer testing has discovered a regression from recent changes in >> staging-4.7. >> >> The actual cause is _csched_cpu_pick() falling over LIST_POISON, which >> happened to occur at the same time

Re: [Xen-devel] [PATCH] Remove ambiguities in the COPYING file; add CONTRIBUTING file

2016-08-11 Thread George Dunlap
On 11/08/16 12:05, Lars Kurth wrote: > > > On 11/08/2016 10:56, "George Dunlap" wrote: > >>> +GPL v2 License >>> +-- >>> + >>> +/* >>> + * >>> + * >>> + * >>> + * >>> + * Copyright (C) >>> + * >>> + * This program is free software; you can

  1   2   3   >