[Xen-devel] [libvirt test] 58899: regressions - FAIL

2015-06-26 Thread osstest service user
flight 58899 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/58899/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-libvirt-xsm 11 guest-start fail REGR. vs. 58842 Regressions which are reg

[Xen-devel] [xen-4.5-testing test] 58896: tolerable FAIL - PUSHED

2015-06-26 Thread osstest service user
flight 58896 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/58896/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-qemut-rhel6hvm-amd 12 guest-start/redhat.repeat fail in 58867 pass in 58896 test-amd64-i386-x

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Luis R. Rodriguez
On Fri, Jun 26, 2015 at 4:54 PM, Benjamin Herrenschmidt wrote: > On Fri, 2015-06-26 at 15:41 -0700, Luis R. Rodriguez wrote: > >> > It wasn't nullified for the main user at the time, the fb. And I >> > mentioned an IB adapter or two for which the code had been hand >> tuned. >> >> This still means

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Benjamin Herrenschmidt
On Fri, 2015-06-26 at 15:41 -0700, Luis R. Rodriguez wrote: > > It wasn't nullified for the main user at the time, the fb. And I > > mentioned an IB adapter or two for which the code had been hand > tuned. > > This still means there could be some affected drivers when used on > powerpc, no? Yes.

[Xen-devel] [xen-4.4-testing test] 58895: regressions - FAIL

2015-06-26 Thread osstest service user
flight 58895 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/58895/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 15 guest-start/debian.repeat fail REGR. vs. 58451 Tests which are

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Luis R. Rodriguez
On Jun 26, 2015 3:05 PM, "Benjamin Herrenschmidt" wrote: > > On Fri, 2015-06-26 at 21:31 +0200, Luis R. Rodriguez wrote: > > > Yeah either, we need to fix our relaxed implementation (patch > > > welcome :-) > > > > Yikes, so although powerpc has useful heuristics to automatically > > enable WC the

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Benjamin Herrenschmidt
On Fri, 2015-06-26 at 21:31 +0200, Luis R. Rodriguez wrote: > > Yeah either, we need to fix our relaxed implementation (patch > > welcome :-) > > Yikes, so although powerpc has useful heuristics to automatically > enable WC the default write ops have been nullifying its effects? The heuristic is

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Benjamin Herrenschmidt
On Fri, 2015-06-26 at 16:24 +, Casey Leedom wrote: > Thanks for looking into this Ben. As it stands now, it seems as > if Write Combined mappings simply aren't supported and/or all > driver writers trying to utilize Write Combined mappings have to > "hand roll" their own solutions which real

Re: [Xen-devel] [PATCH v2 04/12] x86/altp2m: basic data structures and support routines.

2015-06-26 Thread Ed White
On 06/24/2015 03:29 AM, Andrew Cooper wrote: > On 22/06/15 19:56, Ed White wrote: >> diff --git a/xen/include/asm-x86/hvm/vcpu.h b/xen/include/asm-x86/hvm/vcpu.h >> index 3d8f4dc..a1529c0 100644 >> --- a/xen/include/asm-x86/hvm/vcpu.h >> +++ b/xen/include/asm-x86/hvm/vcpu.h >> @@ -118,6 +118,13 @@

[Xen-devel] [xen-4.2-testing test] 58892: tolerable FAIL - PUSHED

2015-06-26 Thread osstest service user
flight 58892 xen-4.2-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/58892/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stop fail like 58833 test-amd64-amd64-xl-win7-a

Re: [Xen-devel] [PATCH V2] libxl: support domainReset

2015-06-26 Thread Jim Fehlig
On 05/27/2015 01:46 PM, Jim Fehlig wrote: Currently, libxl does not provide a reset function, but domainReset can be implemented in the libxl driver by forcibly destroying the domain and starting it again. Signed-off-by: Jim Fehlig --- This is essentially a V2 of a patch submitted quite some t

Re: [Xen-devel] Problems with merlot* AMD Opteron 6376 systems (Was Re: stable trees

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 15:36 -0400, Boris Ostrovsky wrote: > On 06/26/2015 10:52 AM, Jan Beulich wrote: > On 26.06.15 at 16:34, wrote: > >> I did this using rdmsr from mst-tools instead, running on a native > >> kernel gave: > >> > >> # for i in $(seq 0 31) ;do rdmsr -p $i MSR_K8_TOP_MEM2; don

Re: [Xen-devel] Problems with merlot* AMD Opteron 6376 systems (Was Re: stable trees

2015-06-26 Thread Boris Ostrovsky
On 06/26/2015 10:52 AM, Jan Beulich wrote: On 26.06.15 at 16:34, wrote: I did this using rdmsr from mst-tools instead, running on a native kernel gave: # for i in $(seq 0 31) ;do rdmsr -p $i MSR_K8_TOP_MEM2; done 0 [...] 0 Is MSR_K8_TOP_MEM2 defined somewhere in the shell? Just to make sure

Re: [Xen-devel] Tools backports for 4.5 [and 1 more messages]

2015-06-26 Thread Jim Fehlig
On 06/23/2015 02:06 PM, Jim Fehlig wrote: On 06/23/2015 07:12 AM, Ian Jackson wrote: Ian Jackson writes ("Re: [Xen-devel] Tools backports for 4.5 [and 1 more messages]"): Jan Beulich writes ("Re: [Xen-devel] Tools backports for 4.5"): 3380f5b6270e ocaml/xenctrl: Check return values from hyperc

Re: [Xen-devel] [PATCH v2 12/12] x86/altp2m: XSM hooks for altp2m HVM ops

2015-06-26 Thread Ed White
On 06/26/2015 12:24 PM, Daniel De Graaf wrote: > On 06/22/2015 02:56 PM, Ed White wrote: >> From: Ravi Sahita >> >> Signed-off-by: Ravi Sahita > > One comment, below. > > [...] >> diff --git a/tools/flask/policy/policy/modules/xen/xen.if >> b/tools/flask/policy/policy/modules/xen/xen.if >> ind

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Luis R. Rodriguez
On Fri, Jun 26, 2015 at 08:51:58AM +1000, Benjamin Herrenschmidt wrote: > On Thu, 2015-06-25 at 21:40 +, Casey Leedom wrote: > > Hhmmm, so what do PowerPC Drivers do when they want to take > > advantage of Write Combining? Do their own Endian Swizzling > > with the __raw_*() APIs? > > Yeah ei

Re: [Xen-devel] [PATCH v2 12/12] x86/altp2m: XSM hooks for altp2m HVM ops

2015-06-26 Thread Daniel De Graaf
On 06/22/2015 02:56 PM, Ed White wrote: From: Ravi Sahita Signed-off-by: Ravi Sahita One comment, below. [...] diff --git a/tools/flask/policy/policy/modules/xen/xen.if b/tools/flask/policy/policy/modules/xen/xen.if index f4cde11..c95109f 100644 --- a/tools/flask/policy/policy/modules/xen

Re: [Xen-devel] [PATCH] libxl: Add missing #include of

2015-06-26 Thread Roger Pau Monné
El 26/06/15 a les 19.27, Ian Jackson ha escrit: > Ian Jackson writes ("[PATCH] libxl: Add missing #include of "): >> "libxl: ao abort: Handle SIGTERM in save/restore helper" requires >> this; without it we fail to build on BSD. >> >> Reported-by: Ian Campbell >> CC: Ian Campbell >> CC: Roger Pau

Re: [Xen-devel] [PATCH] xen/arm: Remove unused field eoi_cpu in arch_irq_desc

2015-06-26 Thread Julien Grall
On 26/06/2015 18:04, Ian Campbell wrote: On Fri, 2015-06-26 at 17:57 +0200, Julien Grall wrote: Hi Ian, On 26/06/2015 17:54, Ian Campbell wrote: On Fri, 2015-06-26 at 16:43 +0100, Julien Grall wrote: This field have been set but not used since Xen 4.5. Slim down Xen by about 4K by removing

Re: [Xen-devel] [PATCH v4 6/6] xen: arm: Import of_bus PCI entry from Linux (as a dt_bus entry)

2015-06-26 Thread Julien Grall
Hi Ian, On 08/05/2015 13:27, Ian Campbell wrote: This provides specific handlers for the PCI bus relating to matching and translating. It's mostly similar to the defaults but includes some additional error checks and other PCI specific bits. I though the previous patch (#5) was enough to handl

Re: [Xen-devel] [PATCH v4 5/6] xen: arm: map child MMIO and IRQs to dom0 for PCI bus DT nodes.

2015-06-26 Thread Julien Grall
Hi Ian, NIT in the title: The final point is not necessary On 08/05/2015 13:26, Ian Campbell wrote: +static int map_device_children(struct domain *d, + const struct dt_device_node *dev) +{ +bool_t need_mapping = !dt_device_for_passthrough(dev); +int ret; +

Re: [Xen-devel] [PATCH v4 4/6] xen: arm: drop redundant extra call to vgic_reserve_virq

2015-06-26 Thread Julien Grall
Hi Ian, On 08/05/2015 13:26, Ian Campbell wrote: This is only needed if we are giving the IRQ to dom0 (as opposed to setting it up for passthrough due to xen,passthrough property). There is already a call to vgic_reserve_virq inside the if ( need_mapping ), so drop this one. Signed-off-by: Ian

Re: [Xen-devel] [PATCH v4 1/6] xen: dt: add dt_for_each_irq_map helper

2015-06-26 Thread Julien Grall
Hi Ian, On 08/05/2015 13:26, Ian Campbell wrote: This function iterates over a nodes interrupt-map property and calls a callback for each interrupt. For now it only supplies the translated IRQ since my use case has no need of e.g. child unit address. These can be added as needed by any future us

Re: [Xen-devel] [PATCH] libxl: Add missing #include of

2015-06-26 Thread Ian Jackson
Ian Jackson writes ("[PATCH] libxl: Add missing #include of "): > "libxl: ao abort: Handle SIGTERM in save/restore helper" requires > this; without it we fail to build on BSD. > > Reported-by: Ian Campbell > CC: Ian Campbell > CC: Roger Pau Monne > Signed-off-by: Ian Jackson Given that this i

[Xen-devel] [PATCH] libxl: Add missing #include of

2015-06-26 Thread Ian Jackson
"libxl: ao abort: Handle SIGTERM in save/restore helper" requires this; without it we fail to build on BSD. Reported-by: Ian Campbell CC: Ian Campbell CC: Roger Pau Monne Signed-off-by: Ian Jackson --- tools/libxl/libxl_save_helper.c |1 + 1 file changed, 1 insertion(+) diff --git a/tool

Re: [Xen-devel] [PATCH RFC 7/9] libxl: introduce specific error codes in libxl_device_disk_add

2015-06-26 Thread Rob Hoes
On 24 Jun 2015, at 16:28, Ian Jackson wrote: > > Rob Hoes writes ("[PATCH RFC 7/9] libxl: introduce specific error codes in > libxl_device_disk_add"): >> Signed-off-by: Rob Hoes > ... >> diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c >> index f622981..2f56c6e 100644 >> --- a/tools/libxl

Re: [Xen-devel] [Xen-staging] [xen staging] libxl: ao abort: Handle SIGTERM in save/restore helper

2015-06-26 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-staging] [xen staging] libxl: ao abort: Handle SIGTERM in save/restore helper"): > On Fri, 2015-06-26 at 16:03 +, patch...@xen.org wrote: > > commit b6ac9b7fc6bb18057a1aa114296b32f59b40e187 > > Author: Ian Jackson > > AuthorDate: Tue Feb 10 19:05:06 2015 +00

Re: [Xen-devel] [Xen-staging] [xen staging] libxl: ao abort: Handle SIGTERM in save/restore helper

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 16:03 +, patch...@xen.org wrote: > commit b6ac9b7fc6bb18057a1aa114296b32f59b40e187 > Author: Ian Jackson > AuthorDate: Tue Feb 10 19:05:06 2015 + > Commit: Ian Jackson > CommitDate: Fri Jun 26 16:53:51 2015 +0100 > > libxl: ao abort: Handle SIGTERM in sa

Re: [Xen-devel] [libvirt test] 58119: regressions - FAIL

2015-06-26 Thread Ian Jackson
Anthony PERARD writes ("Re: [Xen-devel] [libvirt test] 58119: regressions - FAIL"): > FYI, I have looked at how long it takes for QEMU to start, from libxl point > of view, and from strace point of view. > > For libxl, I have look at the time difference between a call to > libxl__ev_xswatch_regis

Re: [Xen-devel] [PATCH] libxl: Increase device model startup timeout to 1min.

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 12:57 +0100, Anthony PERARD wrote: > On a busy host, QEMU may take more than 10s to start. Please show your working here so that in 2 years when we want to know why this value was chosen we don't have to go scrobbling around in the ML archives looking for the data you gathere

Re: [Xen-devel] [PATCH RFC 9/9] libxl: introduce specific error codes in libxl_device_nic_add

2015-06-26 Thread Rob Hoes
> On 24 Jun 2015, at 16:11, Ian Jackson wrote: > > Rob Hoes writes ("[PATCH RFC 9/9] libxl: introduce specific error codes in > libxl_device_nic_add"): >> Signed-off-by: Rob Hoes > ... >> +# NIC parameters could not be determined >> +(ENUM_PREV, "NIC_SCRIPT_UNDETERMINED"), >> +

Re: [Xen-devel] [PATCH v2 09/12] x86/altp2m: add remaining support routines.

2015-06-26 Thread Ed White
On 06/24/2015 11:19 AM, Andrew Cooper wrote: > On 24/06/15 18:47, Ed White wrote: This looks like some hoop jumping around the assertions in domain_pause() and vcpu_pause(). We should probably have some new helpers where the domain needs to be paused, possibly while in cont

Re: [Xen-devel] Dom0 kernel panic when porting xen to new arm soc

2015-06-26 Thread Julien Grall
On 25/06/2015 16:09, Peng Fan wrote: Hi Julien, Hi, On 6/23/2015 9:56 PM, Peng Fan wrote: Hi, On 6/23/2015 9:36 PM, Julien Grall wrote: Hi, On 23/06/15 14:03, Peng Fan wrote: I did not enable LPAE for DOM0 kernel, use shor page table. Following is the full log from uboot to kernel with

Re: [Xen-devel] [PATCH v2 07/12] x86/altp2m: add control of suppress_ve.

2015-06-26 Thread Ed White
On 06/25/2015 11:04 PM, Jan Beulich wrote: On 25.06.15 at 18:36, wrote: >> On 06/25/2015 01:12 AM, Jan Beulich wrote: >> On 24.06.15 at 19:53, wrote: On 06/24/2015 07:38 AM, Jan Beulich wrote: On 22.06.15 at 20:56, wrote: >> --- a/xen/include/asm-x86/p2m.h >> +++ b

Re: [Xen-devel] [PATCH RFC 8/9] libxl: introduce specific error codes in libxl_device_cdrom_insert

2015-06-26 Thread Rob Hoes
> On 24 Jun 2015, at 16:26, Ian Jackson wrote: > > Rob Hoes writes ("[PATCH RFC 8/9] libxl: introduce specific error codes in > libxl_device_cdrom_insert"): >> Signed-off-by: Rob Hoes > >> if (libxl_get_stubdom_id(ctx, domid) != 0) { >> LOG(ERROR, "cdrom-insert doesn't work for st

Re: [Xen-devel] [PATCH v7 5/9] PCI: Add pci_iomap_wc() variants

2015-06-26 Thread Casey Leedom
Thanks for looking into this Ben. As it stands now, it seems as if Write Combined mappings simply aren't supported and/or all driver writers trying to utilize Write Combined mappings have to "hand roll" their own solutions which really isn't supportable. One thing that might be considered is

Re: [Xen-devel] Problems with merlot* AMD Opteron 6376 systems (Was Re: stable trees (was: [xen-4.2-testing test] 58584: regressions))

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 15:52 +0100, Jan Beulich wrote: > >>> On 26.06.15 at 16:34, wrote: > > I did this using rdmsr from mst-tools instead, running on a native > > kernel gave: > > > > # for i in $(seq 0 31) ;do rdmsr -p $i MSR_K8_TOP_MEM2; done > > 0 > >[...] > > 0 > > Uniformly uncachable for

[Xen-devel] [PATCH 5/7] xen: kill sched_sedf.c

2015-06-26 Thread Dario Faggioli
Signed-off-by: Dario Faggioli --- Cc: George Dunlap --- xen/common/sched_sedf.c | 1556 --- 1 file changed, 1556 deletions(-) delete mode 100644 xen/common/sched_sedf.c diff --git a/xen/common/sched_sedf.c b/xen/common/sched_sedf.c deleted file mode

[Xen-devel] [PATCH 4/7] xen: get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
more specifically, of all the symbols and references to it. Signed-off-by: Dario Faggioli --- Cc: George Dunlap --- docs/misc/xen-command-line.markdown |2 +- xen/common/Makefile |1 - xen/common/schedule.c |1 - xen/include/public/domctl.h |

[Xen-devel] [PATCH 7/7] docs: get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
Signed-off-by: Dario Faggioli --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Cc: George Dunlap --- docs/INDEX |1 - docs/man/xlcpupool.cfg.pod.5|4 --- docs/misc/sedf_scheduler_mini-HOWTO.txt | 44 ---

[Xen-devel] [PATCH 6/7] xl: get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
Signed-off-by: Dario Faggioli --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Cc: George Dunlap --- docs/man/xl.cfg.pod.5 | 24 docs/man/xl.pod.1 | 42 -- tools/libxl/xl.h |1 tools/libxl/xl_cmdimpl.c | 139

[Xen-devel] [PATCH 3/7] libxc: get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
Signed-off-by: Dario Faggioli --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Cc: George Dunlap --- tools/libxc/Makefile |1 - tools/libxc/include/xenctrl.h | 12 -- tools/libxc/xc_sedf.c | 78 - 3 f

[Xen-devel] [PATCH 2/7] tools: python: get rid of the SEDF scheduler bindings

2015-06-26 Thread Dario Faggioli
as it is going away from libxc, so these won't build any longer. Signed-off-by: Dario Faggioli --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Cc: George Dunlap --- tools/python/xen/lowlevel/xc/xc.c | 70 - 1 file changed, 70 del

[Xen-devel] [PATCH 0/7] get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
Hi everyone, This series is the follow up of this proposal and conversation: http://lists.xen.org/archives/html/xen-devel/2015-05/msg02874.html Let me quote this (again), from 2006: git show db51cd09d37ea44b126bb259f9392248afd768e6 ... diff --git a/docs/src/interface.tex b/docs/src/interfac

[Xen-devel] [PATCH 1/7] libxl: get rid of the SEDF scheduler

2015-06-26 Thread Dario Faggioli
only the interface is left in place, for backward compile-time compatibility, but every attempt to use it would throw an error. Signed-off-by: Dario Faggioli --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Cc: George Dunlap --- tools/libxl/libxl.c| 73 ++---

Re: [Xen-devel] Xen 4.6 Development Update (2 WEEKS TO FREEZE, important information in preamble)

2015-06-26 Thread Julien Grall
On 26/06/2015 13:16, wei.l...@citrix.com wrote: (Note, please trim your quotes when replying, and also trim the CC list if necessary. You might also consider changing the subject line of your reply to "Status of FOO (Was: Xen 4.6 Development Update (X months reminder)") Hi all Hi Wei, === H

Re: [Xen-devel] [PATCH] x86/nHVM: generic hook adjustments

2015-06-26 Thread Boris Ostrovsky
On 06/26/2015 11:56 AM, Jan Beulich wrote: On 26.06.15 at 17:50, wrote: On 06/26/2015 11:42 AM, Jan Beulich wrote: On 26.06.15 at 16:58, wrote: On 06/26/2015 05:02 AM, Jan Beulich wrote: -uint64_t nsvm_vcpu_guestcr3(struct vcpu *v) -{ -return vcpu_nestedsvm(v).ns_vmcb_guestcr3; -}

Re: [Xen-devel] Stop testing SEDF almost at all (at least until Xen 4.2) [was: Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS]

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 18:09 +0200, Dario Faggioli wrote: > Ok, I'm fine with this. Looking at make-flight, we seem to want to be > able to run tests on Xen 3.*, 4.0, 4.1. As I said in my email, I can't > speak for these. I honestly think that SEDF was already broken, so I > personally would be mor

Re: [Xen-devel] Stop testing SEDF almost at all (at least until Xen 4.2) [was: Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS]

2015-06-26 Thread Dario Faggioli
On Fri, 2015-06-26 at 16:24 +0100, Ian Jackson wrote: > Dario Faggioli writes ("Stop testing SEDF almost at all (at least until Xen > 4.2) [was: Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS]"): > > However, I honestly think that testing SEDF for earlier versions than > > xen-unsta

Re: [Xen-devel] [PATCH] xen/arm: Remove unused field eoi_cpu in arch_irq_desc

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 17:57 +0200, Julien Grall wrote: > Hi Ian, > > On 26/06/2015 17:54, Ian Campbell wrote: > > On Fri, 2015-06-26 at 16:43 +0100, Julien Grall wrote: > >> This field have been set but not used since Xen 4.5. Slim down Xen by > >> about 4K by removing it. > >> > >> Also fix comme

[Xen-devel] [ovmf test] 58888: tolerable FAIL - PUSHED

2015-06-26 Thread osstest service user
flight 5 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/5/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 58856 test-amd64-i386-xl-qemuu-win7-amd64 1

Re: [Xen-devel] NUMA config handling bug on reuse of libxl domain config

2015-06-26 Thread Dario Faggioli
On Fri, 2015-06-26 at 14:20 +0100, Ian Jackson wrote: > Ian Jackson writes ("Re: [PATCH v4 00/35] libxl ao abort request > (cancellation)"): > > Ian Jackson writes ("[PATCH v4 00/35] libxl ao abort request > > (cancellation)"): > > > This is v4 of my series to provide support for abandoning a > >

Re: [Xen-devel] [PATCH] xen/arm: Remove unused field eoi_cpu in arch_irq_desc

2015-06-26 Thread Julien Grall
Hi Ian, On 26/06/2015 17:54, Ian Campbell wrote: On Fri, 2015-06-26 at 16:43 +0100, Julien Grall wrote: This field have been set but not used since Xen 4.5. Slim down Xen by about 4K by removing it. Also fix comment coding style. Signed-off-by: Julien Grall --- xen/arch/arm/irq.c|

Re: [Xen-devel] [PATCH v4 00/35] libxl ao abort request (cancellation)

2015-06-26 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH v4 00/35] libxl ao abort request (cancellation)"): > On Fri, 2015-06-26 at 16:40 +0100, Ian Jackson wrote: > > Ian Jackson writes ("Re: [PATCH v4 00/35] libxl ao abort request > > (cancellation)"): > > If there are no further objections I think this can go into st

Re: [Xen-devel] [PATCH] x86/nHVM: generic hook adjustments

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:50, wrote: > On 06/26/2015 11:42 AM, Jan Beulich wrote: > On 26.06.15 at 16:58, wrote: >>> On 06/26/2015 05:02 AM, Jan Beulich wrote: -uint64_t nsvm_vcpu_guestcr3(struct vcpu *v) -{ -return vcpu_nestedsvm(v).ns_vmcb_guestcr3; -} - >>

Re: [Xen-devel] [PATCH v3] x86/arm/mm: use gfn instead of pfn in p2m_get_mem_access/p2m_set_mem_access

2015-06-26 Thread Julien Grall
Hi, On 26/06/2015 17:17, Vitaly Kuznetsov wrote: 'pfn' and 'start_pfn' are ambiguous, both these functions expect GFNs as input. On x86 the interface of p2m_set_mem_access() in p2m.c doesn't match the declaration in p2m-common.h as 'pfn' is being used instead of 'start_pfn'. On ARM both p2m_se

Re: [Xen-devel] [PATCH] xen/arm: Remove unused field eoi_cpu in arch_irq_desc

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 16:43 +0100, Julien Grall wrote: > This field have been set but not used since Xen 4.5. Slim down Xen by > about 4K by removing it. > > Also fix comment coding style. > > Signed-off-by: Julien Grall > --- > xen/arch/arm/irq.c| 7 --- > xen/include/asm-arm/irq.h

Re: [Xen-devel] [xen-unstable test] 58886: regressions - FAIL

2015-06-26 Thread Wei Liu
On Fri, Jun 26, 2015 at 04:42:39PM +0100, Ian Jackson wrote: > osstest service user writes ("[xen-unstable test] 58886: regressions - FAIL"): > > flight 58886 xen-unstable real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/58886/ > > > > Regressions :-( > > > > Tests which did not s

Re: [Xen-devel] [PATCH v4 00/35] libxl ao abort request (cancellation)

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 16:40 +0100, Ian Jackson wrote: > Ian Jackson writes ("Re: [PATCH v4 00/35] libxl ao abort request > (cancellation)"): > > Ian Jackson writes ("[PATCH v4 00/35] libxl ao abort request > > (cancellation)"): > > > This is v4 of my series to provide support for abandoning a > >

Re: [Xen-devel] traps.c:3227: GPF (0000): ffff82d080194a4d -> ffff82d080239d85 and other dom0 induced log messages

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:41, wrote: > from 3.16 to 3.19 we gained a lot of these, if i remember correctly > related to > perf being enabled in the kernel: > > + traps.c:2655:d0v0 Domain attempted WRMSR c081 from > 0xe023e008 to 0x00230010. > + traps.c:2655:d0v0 Doma

Re: [Xen-devel] [PATCH] x86/nHVM: generic hook adjustments

2015-06-26 Thread Boris Ostrovsky
On 06/26/2015 11:42 AM, Jan Beulich wrote: On 26.06.15 at 16:58, wrote: On 06/26/2015 05:02 AM, Jan Beulich wrote: -uint64_t nsvm_vcpu_guestcr3(struct vcpu *v) -{ -return vcpu_nestedsvm(v).ns_vmcb_guestcr3; -} - If this is removed then I don't see why we still need ns_vmcb_guestcr3, I

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread linux
On 2015-06-26 17:22, Jan Beulich wrote: On 26.06.15 at 17:04, wrote: On 26.06.15 at 15:38, wrote: On 2015-06-26 14:41, Jan Beulich wrote: On 26.06.15 at 13:02, wrote: Strange, i don't see *any* of your printk's being hit ... (xl dmesg attached). So does the guest (in the working case) us

Re: [Xen-devel] [xen-unstable test] 58886: regressions - FAIL

2015-06-26 Thread Ian Jackson
Ian Campbell writes ("Re: [xen-unstable test] 58886: regressions - FAIL"): > On Fri, 2015-06-26 at 16:42 +0100, Ian Jackson wrote: > > osstest service user writes ("[xen-unstable test] 58886: regressions - > > FAIL"): > > > flight 58886 xen-unstable real [real] > > > http://logs.test-lab.xenprojec

Re: [Xen-devel] [PATCH v3] x86/arm/mm: use gfn instead of pfn in p2m_get_mem_access/p2m_set_mem_access

2015-06-26 Thread Razvan Cojocaru
On 06/26/2015 06:17 PM, Vitaly Kuznetsov wrote: > 'pfn' and 'start_pfn' are ambiguous, both these functions expect GFNs as > input. > > On x86 the interface of p2m_set_mem_access() in p2m.c doesn't match the > declaration in p2m-common.h as 'pfn' is being used instead of 'start_pfn'. > > On ARM

Re: [Xen-devel] [xen-unstable test] 58886: regressions - FAIL

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 16:42 +0100, Ian Jackson wrote: > osstest service user writes ("[xen-unstable test] 58886: regressions - FAIL"): > > flight 58886 xen-unstable real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/58886/ > > > > Regressions :-( > > > > Tests which did not succeed

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:16, wrote: > On Fri, Jun 26, 2015 at 04:04:30PM +0100, Jan Beulich wrote: >> I think this explains it - you're running in PVHVM mode, which I >> never tried with those patches. I'd even have to go dig to see how >> they drive MSI-X in the first place in that case. Nor do I im

[Xen-devel] [PATCH] xen/arm: Remove unused field eoi_cpu in arch_irq_desc

2015-06-26 Thread Julien Grall
This field have been set but not used since Xen 4.5. Slim down Xen by about 4K by removing it. Also fix comment coding style. Signed-off-by: Julien Grall --- xen/arch/arm/irq.c| 7 --- xen/include/asm-arm/irq.h | 1 - 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/xe

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:20, wrote: > On 2015-06-26 17:04, Jan Beulich wrote: >> I think this explains it - you're running in PVHVM mode, which I >> never tried with those patches. I'd even have to go dig to see how >> they drive MSI-X in the first place in that case. Nor do I immediately >> know whe

Re: [Xen-devel] [xen-unstable test] 58886: regressions - FAIL

2015-06-26 Thread Ian Jackson
osstest service user writes ("[xen-unstable test] 58886: regressions - FAIL"): > flight 58886 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/58886/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-

Re: [Xen-devel] [PATCH] x86/nHVM: generic hook adjustments

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 16:58, wrote: > On 06/26/2015 05:02 AM, Jan Beulich wrote: >> >> -uint64_t nsvm_vcpu_guestcr3(struct vcpu *v) >> -{ >> -return vcpu_nestedsvm(v).ns_vmcb_guestcr3; >> -} >> - > > If this is removed then I don't see why we still need ns_vmcb_guestcr3, > I don't see it bei

[Xen-devel] traps.c:3227: GPF (0000): ffff82d080194a4d -> ffff82d080239d85 and other dom0 induced log messages

2015-06-26 Thread linux
Hi Jan / David, On the other thread you asked about some messages in xl-dmesg, over time there are more that come from a kernel booting. (mind you that there can be some changes in Kconfigs between the kernels i have used.) 3.16 was clean this is what xl-dmesg gained on log message that seem

Re: [Xen-devel] [PATCH v4 00/35] libxl ao abort request (cancellation)

2015-06-26 Thread Ian Jackson
Ian Jackson writes ("Re: [PATCH v4 00/35] libxl ao abort request (cancellation)"): > Ian Jackson writes ("[PATCH v4 00/35] libxl ao abort request (cancellation)"): > > This is v4 of my series to provide support for abandoning a > > long-running libxl operation. > > I've pushed it here > > http

Re: [Xen-devel] [PATCH OSSTEST v4 00/21] Initial support for ARM64

2015-06-26 Thread Ian Campbell
On Fri, 2015-06-26 at 16:25 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [Xen-devel] [PATCH OSSTEST v4 00/21] Initial > support for ARM64"): > > On Fri, 2015-06-26 at 13:53 +0100, Ian Jackson wrote: > > > Can we bubble this patch: > > > > A Add arm64 build and test jobs > > > to the

Re: [Xen-devel] gnttab scalability and ticket locks status

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 16:30, wrote: > On Fri, 2015-06-26 at 14:49 +0100, David Vrabel wrote: >> On 26/06/15 12:16, wei.l...@citrix.com wrote: >> > >> > == Blocker == >> > >> > == Hypervisor == >> > >> > * gnttab: improve scalability (good) >> > - David Vrabel >> >> This is done. >> >> > *

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

2015-06-26 Thread osstest service user
flight 58886 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/58886/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-rumpuserxen-amd64 15 rumpuserxen-demo-xenstorels/xenstorels.repeat fail REGR. vs.

Re: [Xen-devel] [PATCH v3] x86/arm/mm: use gfn instead of pfn in p2m_get_mem_access/p2m_set_mem_access

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:17, wrote: > 'pfn' and 'start_pfn' are ambiguous, both these functions expect GFNs as > input. > > On x86 the interface of p2m_set_mem_access() in p2m.c doesn't match the > declaration in p2m-common.h as 'pfn' is being used instead of 'start_pfn'. > > On ARM both p2m_set_m

Re: [Xen-devel] [PATCH 20/35] libxl: ao abort: Provide public ao abort request API

2015-06-26 Thread Wei Liu
On Fri, Jun 26, 2015 at 04:13:59PM +0100, Wei Liu wrote: > On Fri, Jun 26, 2015 at 04:05:07PM +0100, Ian Jackson wrote: > > Wei Liu writes ("Re: [PATCH 20/35] libxl: ao abort: Provide public ao abort > > request API"): > > > On Thu, Jun 25, 2015 at 06:44:20PM +0100, Ian Jackson wrote: > > > [...]

Re: [Xen-devel] [PATCH] xl: Change output from xl -N create to be more useful

2015-06-26 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] xl: Change output from xl -N create to be more useful"): > On Fri, Jun 26, 2015 at 03:29:15PM +0100, Ian Jackson wrote: ... > > Note that this change is NOT BACKWARDS COMPATIBLE. But it would only > > adversely affects anyone who uses `xl -N create' and then saves and

Re: [Xen-devel] [PATCH OSSTEST v4 00/21] Initial support for ARM64

2015-06-26 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [PATCH OSSTEST v4 00/21] Initial support for ARM64"): > On Fri, 2015-06-26 at 13:53 +0100, Ian Jackson wrote: > > Can we bubble this patch: > > > A Add arm64 build and test jobs > > to the end ? > > > > If so we could apply all the rest. > > I did so with re

Re: [Xen-devel] Stop testing SEDF almost at all (at least until Xen 4.2) [was: Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS]

2015-06-26 Thread Ian Jackson
Dario Faggioli writes ("Stop testing SEDF almost at all (at least until Xen 4.2) [was: Re: [PATCH v2] OSSTest: stop testing SEDF, start testing RTDS]"): > However, I honestly think that testing SEDF for earlier versions than > xen-unstable (at least until 4.2) is also just a waste of test > resour

[Xen-devel] Status of VM event patches (Was: Re: Xen 4.6 Development Update (2 WEEKS TO FREEZE, important information in preamble))

2015-06-26 Thread Razvan Cojocaru
On 06/26/2015 02:16 PM, wei.l...@citrix.com wrote: > * VM event patches (none) >Add support for XSETBV vm_events, >Support hybernating guests >Support for VMCALL-based vm_events > - Razvan Cojocaru Since V2, there are now 3 more patches that we'd like to see in Xen mainline at some

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread Jan Beulich
>>> On 26.06.15 at 17:04, wrote: On 26.06.15 at 15:38, wrote: >> On 2015-06-26 14:41, Jan Beulich wrote: >> On 26.06.15 at 13:02, wrote: Strange, i don't see *any* of your printk's being hit ... (xl dmesg attached). >>> >>> So does the guest (in the working case) use MSI-X at

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread linux
On 2015-06-26 17:04, Jan Beulich wrote: (re-adding xen-devel) Sorry mist the reply-all button. On 26.06.15 at 15:38, wrote: On 2015-06-26 14:41, Jan Beulich wrote: On 26.06.15 at 13:02, wrote: Strange, i don't see *any* of your printk's being hit ... (xl dmesg attached). So does the gu

[Xen-devel] [PATCH v3] x86/arm/mm: use gfn instead of pfn in p2m_get_mem_access/p2m_set_mem_access

2015-06-26 Thread Vitaly Kuznetsov
'pfn' and 'start_pfn' are ambiguous, both these functions expect GFNs as input. On x86 the interface of p2m_set_mem_access() in p2m.c doesn't match the declaration in p2m-common.h as 'pfn' is being used instead of 'start_pfn'. On ARM both p2m_set_mem_access and p2m_get_mem_access interfaces don't

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread Konrad Rzeszutek Wilk
On Fri, Jun 26, 2015 at 04:04:30PM +0100, Jan Beulich wrote: > (re-adding xen-devel) > > >>> On 26.06.15 at 15:38, wrote: > > On 2015-06-26 14:41, Jan Beulich wrote: > > On 26.06.15 at 13:02, wrote: > >>> Strange, i don't see *any* of your printk's being hit ... (xl dmesg > >>> attached). >

Re: [Xen-devel] [RFC PATCH v3 07/18] xen/arm: ITS: implement hw_irq_controller for LPIs

2015-06-26 Thread Julien Grall
On 26/06/2015 16:25, Vijay Kilari wrote: Hi Julien, Hi Vijay, +its_send_inv(its_dev, col, virq); +} + +void its_set_affinity(struct irq_desc *desc, int cpu) +{ +struct its_device *its_dev = get_irq_device(desc); +struct its_collection *target_col; + +/* Physical collection

Re: [Xen-devel] [PATCH] xen-netfront: Remove the meaningless code

2015-06-26 Thread David Vrabel
On 27/06/15 00:17, Liang Li wrote: > The function netif_set_real_num_tx_queues() will return -EINVAL if > the second parameter < 1, so call this function with the second > parameter set to 0 is meaningless. Reviewed-by: David Vrabel David ___ Xen-deve

Re: [Xen-devel] [PATCH 20/35] libxl: ao abort: Provide public ao abort request API

2015-06-26 Thread Wei Liu
On Fri, Jun 26, 2015 at 04:05:07PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 20/35] libxl: ao abort: Provide public ao abort > request API"): > > On Thu, Jun 25, 2015 at 06:44:20PM +0100, Ian Jackson wrote: > > [...] > > > + * It is possible to abort even an operation which is being

Re: [Xen-devel] [PATCH] xl: Change output from xl -N create to be more useful

2015-06-26 Thread Wei Liu
On Fri, Jun 26, 2015 at 03:29:15PM +0100, Ian Jackson wrote: > Currently, xl -N create produces: > > { > "domid": null, > "config": { > "c_info": { > "type": "pv", > [etc] > } > > The domid is always NULL (as the domain has not been created

Re: [Xen-devel] [PATCH 20/35] libxl: ao abort: Provide public ao abort request API

2015-06-26 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 20/35] libxl: ao abort: Provide public ao abort request API"): > On Thu, Jun 25, 2015 at 06:44:20PM +0100, Ian Jackson wrote: > [...] > > + * It is possible to abort even an operation which is being performed > > + * synchronously, but since in that case how==NULL you h

Re: [Xen-devel] [RFC PATCH v3 07/18] xen/arm: ITS: implement hw_irq_controller for LPIs

2015-06-26 Thread Julien Grall
Hi Vijay, On 26/06/2015 14:54, Vijay Kilari wrote: On Tue, Jun 23, 2015 at 8:02 PM, Julien Grall wrote: Hi Vijay, On 22/06/15 13:01, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K Implements hw_irq_controller api's required to handle LPI's This patch doesn't hw_irq_controller for LPI

Re: [Xen-devel] Xen-unstable: pci-passthrough of device using MSI-X interrupts not working after commit x86/MSI: track host and guest masking separately

2015-06-26 Thread Jan Beulich
(re-adding xen-devel) >>> On 26.06.15 at 15:38, wrote: > On 2015-06-26 14:41, Jan Beulich wrote: > On 26.06.15 at 13:02, wrote: >>> Strange, i don't see *any* of your printk's being hit ... (xl dmesg >>> attached). >> >> So does the guest (in the working case) use MSI-X at all for the >> de

Re: [Xen-devel] [PATCH 19/35] libxl: ao: Provide manip_refcnt

2015-06-26 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 19/35] libxl: ao: Provide manip_refcnt"): > On Thu, Jun 25, 2015 at 06:44:19PM +0100, Ian Jackson wrote: > > Previously we used in_initiator to stop the ao being freed while we > > were still in the initiator function (which would result in the > > initiator's call to li

Re: [Xen-devel] [PATCH v2] xen: arm: Log a warning message when a deprecated hypercall is used

2015-06-26 Thread Julien Grall
On 26/06/2015 16:55, Ian Campbell wrote: + * Only use this for hypercalls which were deprecated (i.e. replaced + * by something else) before Xen on ARM was created, i.e. *not* for + * hypercalls which are simply not yet used on ARM. + */ +#define HYPERCALL_DEPRECATED(_name, _nr_args)

Re: [Xen-devel] [PATCH 4/4] xen: sched: get rid of cpupool_scheduler_cpumask()

2015-06-26 Thread Joshua Whitehead
On 6/26/2015 10:08 AM, Juergen Gross wrote: On 06/25/2015 02:15 PM, Dario Faggioli wrote: and of (almost every) direct use of cpupool_online_cpumask(). In fact, what we really want for the most of the times, is the set of valid pCPUs of the cpupool a certain domain is part of. Furthermore, in c

Re: [Xen-devel] [PATCH] x86/nHVM: generic hook adjustments

2015-06-26 Thread Boris Ostrovsky
On 06/26/2015 05:02 AM, Jan Beulich wrote: -uint64_t nsvm_vcpu_guestcr3(struct vcpu *v) -{ -return vcpu_nestedsvm(v).ns_vmcb_guestcr3; -} - If this is removed then I don't see why we still need ns_vmcb_guestcr3, I don't see it being used anywhere except for being assigned a value. -bo

Re: [Xen-devel] [PATCH 35/35] libxl: When save/restore helper dies, do not overwrite rc

2015-06-26 Thread Wei Liu
On Thu, Jun 25, 2015 at 06:44:35PM +0100, Ian Jackson wrote: > If we already have an rc (eg from ao abort), keep it. > > Signed-off-by: Ian Jackson Acked-by: Wei Liu > --- > v4: New patch in this version of the series. > --- > tools/libxl/libxl_save_callout.c | 13 - > 1 file ch

Re: [Xen-devel] [PATCH 34/35] libxl: libxl__ev_child pass actual pid to callback

2015-06-26 Thread Wei Liu
On Thu, Jun 25, 2015 at 06:44:34PM +0100, Ian Jackson wrote: > The callbacks actually ignore this except for logging, but we should > log the correct pid. > > Signed-off-by: Ian Jackson Acked-by: Wei Liu > --- > v4: New patch in this version of the series. > --- > tools/libxl/libxl_fork.c |

Re: [Xen-devel] [PATCH 32/35] libxl: Fix libxl__get_domid error reporting

2015-06-26 Thread Wei Liu
On Thu, Jun 25, 2015 at 06:44:32PM +0100, Ian Jackson wrote: > Make it log something if the xenstore path does not exist. > > Signed-off-by: Ian Jackson Acked-by: Wei Liu > --- > v4: New patch in this version of the series. > --- > tools/libxl/libxl.c |6 -- > 1 file changed, 4 insert

Re: [Xen-devel] [PATCH 33/35] libxl: spawn: Always debug log middle child process death

2015-06-26 Thread Wei Liu
On Thu, Jun 25, 2015 at 06:44:33PM +0100, Ian Jackson wrote: > Otherwise the debug log is strangely devoid of an explanation for the > spawn completing. > > We decorate `what', as otherwise the logged message is rather alarming > (especially if the death is due to us sending SIGKILL, which even >

  1   2   3   >