[Xen-devel] [linux-4.1 bisection] complete test-amd64-i386-freebsd10-amd64

2016-06-14 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-i386-freebsd10-amd64 testid guest-start Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/qemu-xen-tradit

Re: [Xen-devel] [Patch v11 1/3] IOMMU: add a timeout parameter for device IOTLB invalidation

2016-06-14 Thread Xu, Quan
On June 02, 2016 6:25 PM, Jan Beulich wrote: > >>> On 01.06.16 at 11:05, wrote: > > From: Quan Xu > > v11: Change the timeout parameter from 'vtd_qi_timeout' to > > 'iommu_dev_iotlb_timeout', which is not only for VT-d device > > IOTLB invalidation, but also for other IOMMU implementatio

Re: [Xen-devel] [PATCH v8 02/11] IOMMU/MMU: enhance the call trees of IOMMU unmapping and mapping

2016-06-14 Thread Xu, Quan
On June 14, 2016 12:37 AM, George Dunlap wrote: > On Mon, Jun 13, 2016 at 4:17 PM, Xu, Quan wrote: > > From: Quan Xu > > > > When IOMMU mapping is failed, we issue a best effort rollback, > > stopping IOMMU mapping, unmapping the previous IOMMU maps and > then > > reporting the error up to the

[Xen-devel] [xen-unstable bisection] complete test-amd64-amd64-qemuu-nested-intel

2016-06-14 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-amd64-qemuu-nested-intel testid xen-boot/l1 Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git Tree: qemuu git://x

Re: [Xen-devel] Question of xl dump-core

2016-06-14 Thread wj zhou
Hi, Thanks a lot for your reply! On Tue, Jun 14, 2016 at 11:02 PM, Konrad Rzeszutek Wilk wrote: > On Tue, Jun 14, 2016 at 08:21:16AM +0800, wj zhou wrote: >> Hello all, > > Hey, > > CC-ing Daniel, and Dave. >> >> Sorry to disturb you, but I really want to figure it out. >> The xen core of redhat

Re: [Xen-devel] [PATCH] xen/arm: map_dev_mmio_region: The iomem permission check should be done on MFN

2016-06-14 Thread Shannon Zhao
Hi Julien, On 2016/6/14 19:50, Julien Grall wrote: > The helper iomem_access_permitted expects MFNs in parameters and not > GNFs. Thankfully only the hardware domain can call this function and > it will always be with GFNS == MFNs for now. > > Also, fix the printf to use the MFN range and not the

Re: [Xen-devel] [PATCH v8 09/11] vt-d: fix the IOMMU flush issue

2016-06-14 Thread Tian, Kevin
> From: Xu, Quan > Sent: Tuesday, June 14, 2016 5:04 PM > > On June 14, 2016 4:27 PM, Jan Beulich wrote: > > >>> On 14.06.16 at 10:10, wrote: > > > On June 13, 2016 11:52 PM, Jan Beulich wrote: > > >> >>> "Xu, Quan" 06/13/16 5:22 PM >>> > > >> >From: Quan Xu > > >> >@@ -546,17 +550,37 @@ stat

[Xen-devel] [linux-3.10 test] 95665: regressions - trouble: blocked/broken/fail/pass

2016-06-14 Thread osstest service owner
flight 95665 linux-3.10 real [real] http://logs.test-lab.xenproject.org/osstest/logs/95665/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-win7-amd64 3 host-install(3) broken REGR. vs. 86412 test-amd64-amd64-xl-mu

[Xen-devel] [PATCH v2 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "

2016-06-14 Thread Dongli Zhang
Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" is true. Thus, it is not necessary to update "j" when "i == ARRAY_SIZE(main_options)" is false. Signed-off-by: Dongli Zhang --- tools/misc/xen-livepatch.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff -

[Xen-devel] [linux-3.14 test] 95657: regressions - trouble: blocked/broken/fail/pass

2016-06-14 Thread osstest service owner
flight 95657 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/95657/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 13 xen-boot/l1fail REGR. vs. 95164 test-amd64-i386-freebs

[Xen-devel] Xen-unstable 4.8: HVM domain_crash called from emulate.c:144 RIP: c000:[<000000000000336a>]

2016-06-14 Thread linux
Hi, Just tested latest xen-unstable 4.8 (xen_changeset git:d337764), but one of the latest commits seems to have broken boot of HVM guests (using qemu-xen) previous build with xen_changeset git:6e908ee worked fine. -- Sander (XEN) [2016-06-14 22:47:36.827] HVM19 save: CPU (XEN) [2016-06-14 22

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

2016-06-14 Thread osstest service owner
flight 95735 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/95735/ 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 12

[Xen-devel] [xen-unstable test] 95654: regressions - trouble: blocked/broken/fail/pass

2016-06-14 Thread osstest service owner
flight 95654 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/95654/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-qemuu-nested-intel 13 xen-boot/l1fail REGR. vs. 95353 Tests which are fail

Re: [Xen-devel] [PATCH 14/17] ocaml/libxs: generate a paths.ml

2016-06-14 Thread David Scott
> On 14 Jun 2016, at 10:36, Wei Liu wrote: > > On Mon, Jun 13, 2016 at 08:50:02PM +0100, David Scott wrote: >> >>> On 13 Jun 2016, at 16:22, Wei Liu wrote: >>> >>> On Mon, Jun 13, 2016 at 04:19:59PM +0100, Ian Jackson wrote: Wei Liu writes ("[PATCH 14/17] ocaml/libxs: generate a paths.ml

Re: [Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "

2016-06-14 Thread Konrad Rzeszutek Wilk
On Tue, Jun 14, 2016 at 09:22:45PM +0200, Olaf Hering wrote: > On Fri, Jun 10, Dongli Zhang wrote: > > > Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" > > is true. Thus, it is not necessary to update "j" when "i == > > ARRAY_SIZE(main_options)" is false. > > This brea

Re: [Xen-devel] [PATCH 17/17] oxenstored: honour XEN_RUN_STORED in systemd C stub

2016-06-14 Thread David Scott
> On 13 Jun 2016, at 08:49, Wei Liu wrote: > > Generate a _paths.h for that and add proper dependency. > > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > Cc: David Scott > --- > .gitignore| 1 + > tools/ocaml/xenstored/Makefile| 7 +++ > tools/ocaml/xe

Re: [Xen-devel] [PATCH 16/17] oxenstored: honour XEN_RUN_STORED and XEN_CONFIG_DIR

2016-06-14 Thread David Scott
> On 13 Jun 2016, at 08:49, Wei Liu wrote: > > Only contain changes to ocaml source code. C stub files will be handled > separately. > > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > Cc: David Scott > --- > tools/ocaml/xenstored/define.ml| 6 +++--- > tools/ocaml/xenstored/disk.ml

Re: [Xen-devel] [PATCH 15/17] ocaml/libxs: honour XEN_RUN_STORED

2016-06-14 Thread David Scott
> On 13 Jun 2016, at 08:49, Wei Liu wrote: > > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > Cc: David Scot > --- > tools/ocaml/libs/xs/xs.ml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/ocaml/libs/xs/xs.ml b/tools/ocaml/libs/xs/xs.ml > index 7e14487..db99

Re: [Xen-devel] Next 4.6.x stable release, numbering, qemu-tag

2016-06-14 Thread Konrad Rzeszutek Wilk
> And, we ought not to let this issue delay the actual point release and > it is close to being on the critical path. A decision is needed. My personal preferred color is 4.6.2.1 (and I believe we did a similar release in the past: RELEASE-4.1.6.1) ___

Re: [Xen-devel] [PATCH 1/1] tools/livepatch: cleanup unnecessary "j = ARRAY_SIZE(action_options); "

2016-06-14 Thread Olaf Hering
On Fri, Jun 10, Dongli Zhang wrote: > Local variable "j" would be used only when "i == ARRAY_SIZE(main_options)" > is true. Thus, it is not necessary to update "j" when "i == > ARRAY_SIZE(main_options)" is false. This breaks the build with gcc45: [ 153s] cc1: warnings being treated as errors [

Re: [Xen-devel] [PATCH 02/15] flask/policy: split out rules for system_r

2016-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 09, 2016 at 10:47:05AM -0400, Daniel De Graaf wrote: > When the all_system_role module is enabled, any domain type can be > created using the system_r role, which was the default. When it is > disabled, domains not using the default types (dom0_t and domU_t) must > use another role suc

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

2016-06-14 Thread osstest service owner
flight 95732 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/95732/ 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 12

Re: [Xen-devel] [PATCH 01/15] flask/policy: split into modules

2016-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 09, 2016 at 10:47:04AM -0400, Daniel De Graaf wrote: > This makes it easier to enable or disable parts of the XSM policy. > > Signed-off-by: Daniel De Graaf Reviewed-by: Konrad Rzeszutek Wilk ___ Xen-devel mailing list Xen-devel@lists.xen

[Xen-devel] Next 4.6.x stable release, numbering, qemu-tag

2016-06-14 Thread Ian Jackson
We still have an unanswered question about the forthcoming 4.6.x stable release. To summarise: After tagging qemu-xen-4.6.2, a build issue was discovered: qemu-xen wanted a new patch to fix the build on recent Ubuntu. We decided to include this patch in the forthcoming stable point release of Xe

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Anthony PERARD
On Tue, Jun 14, 2016 at 05:57:00PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > reboot?"): > > What Andrew means is that QEMU shouldn't have kept the CPU state > > structures in the first place. My response explains why that is not > > possible

[Xen-devel] [qemu-upstream-4.3-testing test] 95663: trouble: blocked/broken

2016-06-14 Thread osstest service owner
flight 95663 qemu-upstream-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/95663/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i3863 host-install(3) broken REG

Re: [Xen-devel] xenbits "official" repo for XTF (was Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)

2016-06-14 Thread Andrew Cooper
On 13/06/16 15:11, Ian Jackson wrote: > George Dunlap writes ("Re: [Xen-devel] xenbits "official" repo for XTF (was > Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)"): >> On Mon, Jun 13, 2016 at 11:10 AM, Andrew Cooper >>> I am not completely averse to changing it, but I don't see an >>> alternativ

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 06:03:01PM +0100, Wei Liu wrote: > On Tue, Jun 14, 2016 at 05:57:00PM +0100, Ian Jackson wrote: > > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > > reboot?"): > > > What Andrew means is that QEMU shouldn't have kept the CPU state > > > structures in

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 05:59:30PM +0100, Andrew Cooper wrote: > On 14/06/16 17:57, Ian Jackson wrote: > > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > > reboot?"): > >> What Andrew means is that QEMU shouldn't have kept the CPU state > >> structures in the first place. M

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 05:57:00PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > reboot?"): > > What Andrew means is that QEMU shouldn't have kept the CPU state > > structures in the first place. My response explains why that is not > > possible

Re: [Xen-devel] xenbits "official" repo for XTF (was Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)

2016-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 09, 2016 at 04:04:59PM +0100, Ian Jackson wrote: > Roger Pau Monne writes ("[Xen-devel] [PATCH 0/2] xtf: add launcher (+1 > bugfix)"): > > This series contains a bugfix for the build infrastructure and a basic > > launcher for XTF. Patches can also be found in the following git repo:

Re: [Xen-devel] [PATCH 10/17] build: introduce XEN_RUN_STORED

2016-06-14 Thread Ian Jackson
Wei Liu writes ("[PATCH 10/17] build: introduce XEN_RUN_STORED"): > It defaults to /var/run/xenstored. It will be used later to remove some > hard-coded paths in tree. There should be no visible change to default > configuration. Acked-by: Ian Jackson

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Andrew Cooper
On 14/06/16 17:57, Ian Jackson wrote: > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > reboot?"): >> What Andrew means is that QEMU shouldn't have kept the CPU state >> structures in the first place. My response explains why that is not >> possible from a QEMU upstream poin

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Ian Jackson
Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?"): > What Andrew means is that QEMU shouldn't have kept the CPU state > structures in the first place. My response explains why that is not > possible from a QEMU upstream point of view. I don't think it addresses my poin

Re: [Xen-devel] [PATCH] tools: bump some library version numbers to 4.8

2016-06-14 Thread Ian Jackson
Wei Liu writes ("[PATCH] tools: bump some library version numbers to 4.8"): > It is a pretty safe thing to do and would avoid accidentally overwrite > the old libraries when doing development. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@

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

2016-06-14 Thread Tamas K Lengyel
On Jun 14, 2016 10:33, "Konrad Rzeszutek Wilk" wrote: > > > diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c > > index a522423..ba06fb0 100644 > > --- a/xen/arch/x86/mm/mem_sharing.c > > +++ b/xen/arch/x86/mm/mem_sharing.c > > @@ -1294,6 +1294,54 @@ int relinquish_shared_

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 05:34:22PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across > reboot?"): > > On Mon, Jun 06, 2016 at 06:34:44PM +0100, Andrew Cooper wrote: > > > Why does qemu even care? It has nothing to do with vcpu handling. > > > There

Re: [Xen-devel] [PATCH 3/3] libxl: update vcpus bitmap in retrieved guest config

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 05:31:56PM +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH 3/3] libxl: update vcpus bitmap in retrieved guest > config"): > > ... because the available vcpu bitmap can change during domain life time > > due to cpu hotplug and unplug. > > Is this cpu hotplug something t

Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?

2016-06-14 Thread Ian Jackson
Wei Liu writes ("Re: [Xen-devel] "xl vcpu-set" not persistent across reboot?"): > On Mon, Jun 06, 2016 at 06:34:44PM +0100, Andrew Cooper wrote: > > Why does qemu even care? It has nothing to do with vcpu handling. > > There should not be any qemu vcpu records in the first place. > > IIRC upstre

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

2016-06-14 Thread Konrad Rzeszutek Wilk
> diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c > index a522423..ba06fb0 100644 > --- a/xen/arch/x86/mm/mem_sharing.c > +++ b/xen/arch/x86/mm/mem_sharing.c > @@ -1294,6 +1294,54 @@ int relinquish_shared_pages(struct domain *d) > return rc; > } > > +static int bu

Re: [Xen-devel] [PATCH 3/3] libxl: update vcpus bitmap in retrieved guest config

2016-06-14 Thread Ian Jackson
Wei Liu writes ("[PATCH 3/3] libxl: update vcpus bitmap in retrieved guest config"): > ... because the available vcpu bitmap can change during domain life time > due to cpu hotplug and unplug. Is this cpu hotplug something that the guest can cause, or are we just talking about toolstack changes ?

Re: [Xen-devel] [PATCH v2 2/3] libxl: update vcpus bitmap in retrieved guest config

2016-06-14 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 2/3] libxl: update vcpus bitmap in retrieved guest config"): > On Tue, Jun 14, 2016 at 11:58:58AM +0100, Anthony PERARD wrote: > > I did: > > LD_LIBRARY_PATH=`pwd` ./xl -vvv migrate arch localhost > > Never tried this so I'm not sure what might go wrong... That is

[Xen-devel] [xen-unstable-smoke test] 95722: trouble: broken/pass

2016-06-14 Thread osstest service owner
flight 95722 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/95722/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 3 host-install(3) broken REGR. v

Re: [Xen-devel] [PULL 0/2] xen-20160614-tag

2016-06-14 Thread Peter Maydell
; > are available in the git repository at: > > > git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-20160614-tag > > for you to fetch changes up to b1b23e5bbfb66d9401e2c2b0646fb721d94a3f83: > >

Re: [Xen-devel] Question about sharing spinlock_t among VMs in Xen

2016-06-14 Thread Andrew Cooper
On 14/06/16 03:13, Meng Xu wrote: > On Mon, Jun 13, 2016 at 6:54 PM, Andrew Cooper > wrote: >> On 13/06/2016 18:43, Meng Xu wrote: >>> Hi, >>> >>> I have a quick question about using the Linux spin_lock() in Xen >>> environment to protect some host-wide shared (memory) resource among >>> VMs. >>>

[Xen-devel] [xen-4.7-testing test] 95653: regressions - trouble: blocked/broken/fail/pass

2016-06-14 Thread osstest service owner
flight 95653 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/95653/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-xsm 3 host-install(3) broken REGR. vs. 95446 test-amd64-i386-x

Re: [Xen-devel] [PATCH 3/3] Update README.md

2016-06-14 Thread Konrad Rzeszutek Wilk
On Fri, Jun 10, 2016 at 12:02:45PM +0100, Ross Lagerwall wrote: > Update the example and project status. Add Contributing and Maintainers > sections. Reviewed-by: Konrad Rzeszutek Wilk > > Signed-off-by: Ross Lagerwall > --- > README.md | 76 > -

Re: [Xen-devel] [PATCH 2/3] Update to use a .config file

2016-06-14 Thread Konrad Rzeszutek Wilk
On Fri, Jun 10, 2016 at 12:02:44PM +0100, Ross Lagerwall wrote: > Remove the old --xen-debug option, and instead, require the user to pass > a .config file matching the original build's .config. Hm, that throws this off a bit for the older hypervisors (to which I had backported livepatch). Perhaps

Re: [Xen-devel] [PATCH 1/3] Don't accept fuzz when patching

2016-06-14 Thread Konrad Rzeszutek Wilk
On Fri, Jun 10, 2016 at 12:02:43PM +0100, Ross Lagerwall wrote: > When testing and applying patches, set fuzz=0 so that patches must apply > exactly. Also set "-f" to avoid interactive questions, and reorder so > that patches are tested before the output directory is created. > > Signed-off-by: R

Re: [Xen-devel] [PATCH] x86/HVM: rename mmio_gva field to mmio_gla

2016-06-14 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 14 June 2016 15:45 > To: xen-devel > Cc: Andrew Cooper; Paul Durrant > Subject: [PATCH] x86/HVM: rename mmio_gva field to mmio_gla > > ... to correctly reflect its purpose. To make things consistent also > rename h

[Xen-devel] [PULL 2/2] xen: Clean up includes

2016-06-14 Thread Stefano Stabellini
From: Peter Maydell Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell Reviewed-by: Stefano Stabellini Signed-off-by: Stefano Stabellini --- hw/usb/xen-

[Xen-devel] [PULL 1/2] xen/blkif: avoid double access to any shared ring request fields

2016-06-14 Thread Stefano Stabellini
From: Jan Beulich Commit f9e98e5d7a ("xen/blkif: Avoid double access to src->nr_segments") didn't go far enough: src->operation is also being used twice. And nothing was done to prevent the compiler from using the source side of the copy done by blk_get_request() (granted that's very unlikely).

[Xen-devel] [PULL 0/2] xen-20160614-tag

2016-06-14 Thread Stefano Stabellini
llini/qemu-dm.git tags/xen-20160614-tag for you to fetch changes up to b1b23e5bbfb66d9401e2c2b0646fb721d94a3f83: xen: Clean up includes (2016-06-14 15:37:43 +0100) Xen

Re: [Xen-devel] [PATCH] x86: show remote CPU state upon fatal NMI

2016-06-14 Thread Andrew Cooper
On 14/06/16 15:33, Jan Beulich wrote: > Quite frequently the watchdog would hit an innocent CPU, e.g. one > trying to acquire a spin lock a remote CPU holds for extended periods > of time, or a random CPU in TSC calbration rendezvous. In such cases > the register and stack dump for that CPU doesn't

Re: [Xen-devel] Question of xl dump-core

2016-06-14 Thread Konrad Rzeszutek Wilk
On Tue, Jun 14, 2016 at 08:21:16AM +0800, wj zhou wrote: > Hello all, Hey, CC-ing Daniel, and Dave. > > Sorry to disturb you, but I really want to figure it out. > The xen core of redhat 6 with pod is unable to be used with crash. > > I installed a hvm of redhat 6 by xen 4.7.0-rc2. > And the me

Re: [Xen-devel] [PATCH V9 2/3] drivers/pl011: Use combination of UARTRIS and UARTMSC instead of UARTMIS

2016-06-14 Thread Julien Grall
Hello Shanker, On 13/06/16 18:43, Shanker Donthineni wrote: On 06/13/2016 05:30 AM, Stefano Stabellini wrote: On Thu, 9 Jun 2016, Shanker Donthineni wrote: The Masked interrupt status register (UARTMIS) is not described in ARM SBSA 2.x document. Anding of two registers UARTMSC and UARTRIS va

[Xen-devel] [PATCH] x86/HVM: rename mmio_gva field to mmio_gla

2016-06-14 Thread Jan Beulich
... to correctly reflect its purpose. To make things consistent also rename handle_mmio_with_translation()'s respective parameter (but don't touch sh_page_fault(), as renaming its parameter would require quite a few more changes there). Suggested-by: Andrew Cooper Signed-off-by: Jan Beulich ---

Re: [Xen-devel] [PATCH] x86/HVM: rename mmio_gva field to mmio_gla

2016-06-14 Thread Andrew Cooper
On 14/06/16 15:44, Jan Beulich wrote: > ... to correctly reflect its purpose. To make things consistent also > rename handle_mmio_with_translation()'s respective parameter (but don't > touch sh_page_fault(), as renaming its parameter would require quite a > few more changes there). > > Suggested-by

Re: [Xen-devel] [PATCH] xen: move xen_sysdev to xen_backend.c

2016-06-14 Thread Anthony PERARD
On Mon, Jun 13, 2016 at 11:12:21AM +0200, Juergen Gross wrote: > Commit 9432e53a5bc88681b2d3aec4dac9db07c5476d1b added xen_sysdev as a > system device to serve as an anchor for removable virtual buses. This > introduced a build failure for non-x86 builds with CONFIG_XEN_BACKEND > set, as xen_sysdev

Re: [Xen-devel] [PATCH] xen: Clean up includes

2016-06-14 Thread Peter Maydell
On 14 June 2016 at 15:37, Stefano Stabellini wrote: > I didn't lose it, I thought you had already committed it as > 21cbfe5f37aaa3a13d3af28454e762c05be67429, but I realize now that > although they have the same commit message, they are not the same patch. > > I was wondering how it got upstream gi

Re: [Xen-devel] [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes

2016-06-14 Thread George Dunlap
On 14/06/16 15:23, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 03:00:12PM +0100, George Dunlap wrote: >> On 14/06/16 14:53, Anthony PERARD wrote: >>> On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: That code might be in qemu-upstream, but it's not in the qemu-xen trees;

[Xen-devel] [PATCH] x86: show remote CPU state upon fatal NMI

2016-06-14 Thread Jan Beulich
Quite frequently the watchdog would hit an innocent CPU, e.g. one trying to acquire a spin lock a remote CPU holds for extended periods of time, or a random CPU in TSC calbration rendezvous. In such cases the register and stack dump for that CPU doesn't really help in the analysis of the problem.

Re: [Xen-devel] [PATCH] xen: Clean up includes

2016-06-14 Thread Stefano Stabellini
On Tue, 14 Jun 2016, Peter Maydell wrote: > On 30 May 2016 at 16:54, Stefano Stabellini wrote: > > On Tue, 24 May 2016, Peter Maydell wrote: > >> Clean up includes so that osdep.h is included first and headers > >> which it implies are not included manually. > >> > >> This commit was created with

Re: [Xen-devel] [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes

2016-06-14 Thread Anthony PERARD
On Tue, Jun 14, 2016 at 03:00:12PM +0100, George Dunlap wrote: > On 14/06/16 14:53, Anthony PERARD wrote: > > On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: > >> That code might be in qemu-upstream, but it's not in the qemu-xen trees; > >> if you revert this bit of the patch and try

Re: [Xen-devel] [PATCH] libxl: correct xl cpupool-numa-split with vcpu limited dom0

2016-06-14 Thread Ian Jackson
Juergen Gross writes ("Re: [Xen-devel] [PATCH] libxl: correct xl cpupool-numa-split with vcpu limited dom0"): > On 14/06/16 12:07, Ian Jackson wrote: > > I looked at the code for a minute or two, and perhaps I'm being dense > > this morning, but I wasn't able to see (from the code and the commit >

Re: [Xen-devel] [PATCH 1/6] xl: remus/colo: only initialise ha variable when necessary

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 03:15:19PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 1/6] xl: remus/colo: only initialise ha variable > when necessary"): > > On Tue, Jun 14, 2016 at 11:18:16AM +0100, Ian Jackson wrote: > > > Wei Liu writes ("[PATCH 1/6] xl: remus/colo: only initialise ha var

Re: [Xen-devel] [PATCH 1/6] xl: remus/colo: only initialise ha variable when necessary

2016-06-14 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 1/6] xl: remus/colo: only initialise ha variable when necessary"): > On Tue, Jun 14, 2016 at 11:18:16AM +0100, Ian Jackson wrote: > > Wei Liu writes ("[PATCH 1/6] xl: remus/colo: only initialise ha variable > > when necessary"): > > > The original code is bogus because

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

2016-06-14 Thread osstest service owner
flight 95649 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/95649/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-libvirt-qemuu-debianhvm-amd64-xsm 9 debian-hvm-install fail REGR. vs. 94856 test-

Re: [Xen-devel] [PATCH] xen: Clean up includes

2016-06-14 Thread Peter Maydell
On 30 May 2016 at 16:54, Stefano Stabellini wrote: > On Tue, 24 May 2016, Peter Maydell wrote: >> Clean up includes so that osdep.h is included first and headers >> which it implies are not included manually. >> >> This commit was created with scripts/clean-includes. >> >> Signed-off-by: Peter May

Re: [Xen-devel] [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes

2016-06-14 Thread George Dunlap
On 14/06/16 14:53, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: >> On 14/06/16 11:31, Stefano Stabellini wrote: >>> On Tue, 14 Jun 2016, George Dunlap wrote: On 14/06/16 11:08, Stefano Stabellini wrote: > On Tue, 14 Jun 2016, George Dunlap wrote: >

Re: [Xen-devel] [PATCH raisin 4/4] Update to 4.7, update qemu and qemu_traditional recipes

2016-06-14 Thread Anthony PERARD
On Tue, Jun 14, 2016 at 11:34:43AM +0100, George Dunlap wrote: > On 14/06/16 11:31, Stefano Stabellini wrote: > > On Tue, 14 Jun 2016, George Dunlap wrote: > >> On 14/06/16 11:08, Stefano Stabellini wrote: > >>> On Tue, 14 Jun 2016, George Dunlap wrote: > On 14/06/16 10:46, Stefano Stabellini

Re: [Xen-devel] Xen 4.7 crash

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 09:38:22AM -0400, Aaron Cornelius wrote: > On 6/14/2016 9:26 AM, Aaron Cornelius wrote: > >On 6/14/2016 9:15 AM, Wei Liu wrote: > >>On Tue, Jun 14, 2016 at 09:11:47AM -0400, Aaron Cornelius wrote: > >>>On 6/9/2016 7:14 AM, Ian Jackson wrote: > Aaron Cornelius writes ("Re

Re: [Xen-devel] Xen 4.7 crash

2016-06-14 Thread Aaron Cornelius
On 6/14/2016 9:26 AM, Aaron Cornelius wrote: On 6/14/2016 9:15 AM, Wei Liu wrote: On Tue, Jun 14, 2016 at 09:11:47AM -0400, Aaron Cornelius wrote: On 6/9/2016 7:14 AM, Ian Jackson wrote: Aaron Cornelius writes ("Re: [Xen-devel] Xen 4.7 crash"): I am not that familiar with the xenstored code,

Re: [Xen-devel] [PATCH v4 3/3] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 15:13, wrote: > On 14/06/16 11:45, Jan Beulich wrote: >>> + struct hvm_ioreq_server *s) >>> +{ >>> +struct p2m_domain *p2m = p2m_get_hostp2m(d); >>> +int rc; >>> + >>> +spin_lock(&p2m->ioreq.lock); >>> + >>> +if ( flags == 0 ) >>> +{ >

Re: [Xen-devel] Xen 4.7 crash

2016-06-14 Thread Aaron Cornelius
On 6/14/2016 9:15 AM, Wei Liu wrote: On Tue, Jun 14, 2016 at 09:11:47AM -0400, Aaron Cornelius wrote: On 6/9/2016 7:14 AM, Ian Jackson wrote: Aaron Cornelius writes ("Re: [Xen-devel] Xen 4.7 crash"): I am not that familiar with the xenstored code, but as far as I can tell the grant mapping wil

Re: [Xen-devel] [PATCH v2 2/3] libxl: update vcpus bitmap in retrieved guest config

2016-06-14 Thread Anthony PERARD
On Tue, Jun 14, 2016 at 11:58:58AM +0100, Anthony PERARD wrote: > On Tue, Jun 14, 2016 at 11:50:12AM +0100, Wei Liu wrote: > > On Tue, Jun 14, 2016 at 11:47:57AM +0100, Anthony PERARD wrote: > > [...] > > > > > > + > > > > > > int libxl_retrieve_domain_configuration(libxl_ctx *ctx, uint32_t > > >

Re: [Xen-devel] [PATCH 00/17] Honour more configure variables in various places (iteration 2)

2016-06-14 Thread Wei Liu
On Mon, Jun 13, 2016 at 08:48:58AM +0100, Wei Liu wrote: > The first three patches are carried over from previous series. > > The rest introduce some new variables and replace various hard-coded paths in > code. > > There is one more path /var/run, with which I'm not sure what to do for the > mom

Re: [Xen-devel] [PATCH v4 1/3] x86/ioreq server: Rename p2m_mmio_write_dm to p2m_ioreq_server.

2016-06-14 Thread George Dunlap
On 14/06/16 11:04, Jan Beulich wrote: On 19.05.16 at 11:05, wrote: >> @@ -5507,6 +5507,8 @@ long do_hvm_op(unsigned long op, >> XEN_GUEST_HANDLE_PARAM(void) arg) >> get_gfn_query_unlocked(d, a.pfn, &t); >> if ( p2m_is_mmio(t) ) >> a.mem_type = HVMM

Re: [Xen-devel] [PATCH v4 3/3] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

2016-06-14 Thread George Dunlap
On 19/05/16 10:05, Yu Zhang wrote: > A new HVMOP - HVMOP_map_mem_type_to_ioreq_server, is added to > let one ioreq server claim/disclaim its responsibility for the > handling of guest pages with p2m type p2m_ioreq_server. Users > of this HVMOP can specify which kind of operation is supposed > to be

Re: [Xen-devel] Xen 4.7 crash

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 09:11:47AM -0400, Aaron Cornelius wrote: > On 6/9/2016 7:14 AM, Ian Jackson wrote: > >Aaron Cornelius writes ("Re: [Xen-devel] Xen 4.7 crash"): > >>I am not that familiar with the xenstored code, but as far as I can tell > >>the grant mapping will be held by the xenstore unt

Re: [Xen-devel] [PATCH v4 3/3] x86/ioreq server: Add HVMOP to map guest ram with p2m_ioreq_server to an ioreq server.

2016-06-14 Thread George Dunlap
On 14/06/16 11:45, Jan Beulich wrote: >> + struct hvm_ioreq_server *s) >> +{ >> +struct p2m_domain *p2m = p2m_get_hostp2m(d); >> +int rc; >> + >> +spin_lock(&p2m->ioreq.lock); >> + >> +if ( flags == 0 ) >> +{ >> +rc = -EINVAL; >> +if ( p2m

Re: [Xen-devel] [PATCH 8/8] xen/arm: p2m_cache_flush: Use the correct terminology and typesafe gfn

2016-06-14 Thread Andrew Cooper
On 14/06/16 13:07, Julien Grall wrote: > p2m_cache_flush is expecting GFNs in parameter and not MFNs. Rename > the variable to *gfn* and use typesafe to avoid possible misusage. > > Signed-off-by: Julien Grall > --- > xen/arch/arm/domctl.c | 2 +- > xen/arch/arm/p2m.c| 10 +-

Re: [Xen-devel] Xen 4.7 crash

2016-06-14 Thread Aaron Cornelius
On 6/9/2016 7:14 AM, Ian Jackson wrote: Aaron Cornelius writes ("Re: [Xen-devel] Xen 4.7 crash"): I am not that familiar with the xenstored code, but as far as I can tell the grant mapping will be held by the xenstore until the xs_release() function is called (which is not called by libxl, and I

Re: [Xen-devel] [PATCH 7/8] xen/mm: Introduce max_gfn and min_gfn

2016-06-14 Thread Andrew Cooper
On 14/06/16 14:05, Jan Beulich wrote: On 14.06.16 at 14:07, wrote: >> --- a/xen/include/xen/mm.h >> +++ b/xen/include/xen/mm.h >> @@ -70,6 +70,9 @@ TYPE_SAFE(unsigned long, gfn); >> #undef gfn_t >> #endif >> >> +#define max_gfn(x, y) _gfn(max(gfn_x(x), gfn_x(y))) >> +#define min_gfn(x, y)

Re: [Xen-devel] [PATCH 0/6] xl/libxl: some cleanup / debugging aid patches

2016-06-14 Thread Wei Liu
On Mon, Jun 06, 2016 at 11:52:06AM +0100, Wei Liu wrote: > A handful of patches I accumulated during freeze when I was debugging various > issues in xl / libxl. > > Wei. > > Wei Liu (6): > xl: remus/colo: only initialise ha variable when necessary > libxl: add emacs block to libxl_linux.c >

Re: [Xen-devel] [PATCH 7/8] xen/mm: Introduce max_gfn and min_gfn

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 14:07, wrote: > --- a/xen/include/xen/mm.h > +++ b/xen/include/xen/mm.h > @@ -70,6 +70,9 @@ TYPE_SAFE(unsigned long, gfn); > #undef gfn_t > #endif > > +#define max_gfn(x, y) _gfn(max(gfn_x(x), gfn_x(y))) > +#define min_gfn(x, y) _gfn(min(gfn_x(x), gfn_x(y))) With my reply t

Re: [Xen-devel] [PATCH 5/8] xen: Use the typesafe mfn and gfn in map_mmio_regions...

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 14:07, wrote: > to avoid mixing machine frame with guest frame. > > Signed-off-by: Julien Grall Non-ARM bits: Acked-by: Jan Beulich > @@ -2217,10 +2217,11 @@ int map_mmio_regions(struct domain *d, >i += 1UL << order, ++iter ) > { > /* OR'ing gfn an

Re: [Xen-devel] [PATCH 4/8] xen/arm: Rename grant_table_gfpn into grant_table_gfn and use the typesafe gfn

2016-06-14 Thread Andrew Cooper
On 14/06/16 13:07, Julien Grall wrote: > diff --git a/xen/include/asm-arm/grant_table.h > b/xen/include/asm-arm/grant_table.h > index 5e076cc..46cfe24 100644 > --- a/xen/include/asm-arm/grant_table.h > +++ b/xen/include/asm-arm/grant_table.h > @@ -30,7 +30,7 @@ static inline int replace_grant_supp

Re: [Xen-devel] [PATCH 3/8] xen: Use typesafe gfn in xenmem_add_to_physmap_one

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 14:07, wrote: > The x86 version of the function xenmem_add_to_physmap_one contains > variable name gpfn and gfn which make the code very confusing. > I have left unchanged for now. > > Also, rename gpfn to gfn in the ARM version as the latter is the correct > acronym for a gues

Re: [Xen-devel] [PATCH 2/8] xen: Use typesafe gfn/mfn in guest_physmap_* helpers

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 14:07, wrote: > --- a/xen/arch/x86/mm/p2m.c > +++ b/xen/arch/x86/mm/p2m.c > @@ -665,21 +665,21 @@ p2m_remove_page(struct p2m_domain *p2m, unsigned long > gfn, unsigned long mfn, > } > > int > -guest_physmap_remove_page(struct domain *d, unsigned long gfn, > -

Re: [Xen-devel] [PATCH 1/8] xen/arm: Rename gmfn_to_mfn to gfn_to_mfn and use gfn/mfn typesafe

2016-06-14 Thread Julien Grall
On 14/06/16 13:44, Jan Beulich wrote: On 14.06.16 at 14:07, wrote: The correct acronym for a guest physical frame is gfn. Also use the recently introduced typesafe gfn/mfn to avoid mixing the two different kind of frame. Signed-off-by: Julien Grall --- xen/arch/arm/p2m.c| 6 +++---

Re: [Xen-devel] [PATCH 4/6] libxl: debug output for args and env when invoking hotplug script

2016-06-14 Thread Wei Liu
On Tue, Jun 14, 2016 at 11:26:56AM +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH 4/6] libxl: debug output for args and env when > invoking hotplug script"): > > Signed-off-by: Wei Liu > ... > > +const char *arg; > > +unsigned int x = 2; > > + > > +arg = args[x]; > >

Re: [Xen-devel] [PATCH 1/8] xen/arm: Rename gmfn_to_mfn to gfn_to_mfn and use gfn/mfn typesafe

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 14:07, wrote: > The correct acronym for a guest physical frame is gfn. Also use > the recently introduced typesafe gfn/mfn to avoid mixing the two > different kind of frame. > > Signed-off-by: Julien Grall > --- > xen/arch/arm/p2m.c| 6 +++--- > xen/common/grant_table

Re: [Xen-devel] xen/x86: efi: warning value truncated

2016-06-14 Thread Jan Beulich
>>> On 14.06.16 at 13:19, wrote: > On 06/14/2016 11:50 AM, Jan Beulich wrote: > On 14.06.16 at 12:15, wrote: >>> On 06/14/2016 10:24 AM, Jan Beulich wrote: >>> On 13.06.16 at 19:13, wrote: > I noticed the warnings below when building Xen x86 > with Livepatch enabled. I

Re: [Xen-devel] [PATCH v3 16/16] xen/arm: arm64: Document Cortex-A57 erratum 834220

2016-06-14 Thread Julien Grall
Hello, On 07/06/16 17:06, Julien Grall wrote: The ARM erratum applies to certain revisions of Cortex-A57. The processor may report a Stage 2 translation fault as the result of Stage 1 fault for load crossing a page boundary when there is a permission fault or device memory fault at stage 1 and a

[Xen-devel] [PATCH 7/8] xen/mm: Introduce max_gfn and min_gfn

2016-06-14 Thread Julien Grall
Those helpers will be useful to find the maximum/minimum between two GFNs without having to unbox/box manually. Signed-off-by: Julien Grall --- Cc: Stefano Stabellini Cc: Jan Beulich Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Konrad Rzeszutek Wilk Cc: Tim Deegan Cc: Wei Liu

[Xen-devel] [PATCH 8/8] xen/arm: p2m_cache_flush: Use the correct terminology and typesafe gfn

2016-06-14 Thread Julien Grall
p2m_cache_flush is expecting GFNs in parameter and not MFNs. Rename the variable to *gfn* and use typesafe to avoid possible misusage. Signed-off-by: Julien Grall --- xen/arch/arm/domctl.c | 2 +- xen/arch/arm/p2m.c| 10 +- xen/include/asm-arm/p2m.h | 2 +- 3 files changed,

[Xen-devel] [PATCH 0/8] xen/arm: Use the typesafes gfn and mfn

2016-06-14 Thread Julien Grall
Hello all, Some of the ARM functions are mixing gfn vs mfn and even physical vs frame. To avoid more confusion, this patch series makes use of the terminology described in xen/include/xen/mm.h and the associated typesafe. This series is based on staging + the branch next-4.8 from Stefano merge.

[Xen-devel] [PATCH 2/8] xen: Use typesafe gfn/mfn in guest_physmap_* helpers

2016-06-14 Thread Julien Grall
Also rename some variables to gfn or mfn when it does not require much rework. Signed-off-by: Julien Grall --- Cc: Stefano Stabellini Cc: Jan Beulich Cc: Andrew Cooper Cc: Paul Durrant Cc: George Dunlap Cc: Ian Jackson Cc: Konrad Rzeszutek Wilk Cc: Tim Deegan Cc: Wei Liu --- xen/arch/a

[Xen-devel] [PATCH 6/8] xen/arm: Rework the interface of p2m_lookup and use typesafe gfn and mfn

2016-06-14 Thread Julien Grall
The prototype and the declaration of p2m_lookup disagree on how the function should be used. One expect a frame number whilst the other an address. Thankfully, everyone is using with an address today. However, most of the callers have to convert a guest frame to an address. Modify the interface to

  1   2   >