Re: [Xen-devel] [PATCH v2 2/4] x86/MSI-X: access MSI-X table only after having enabled MSI-X

>>> On 16.04.15 at 20:21, wrote: > What kernel are you using? Or better yet - what branch/tree could > one find it at? As always, I'm personally using my own kernel, not in any branch/tree. But for all purposes here, our "Kernel-of-the-Day" should be good enough, i.e. the stuff under http://downl

Re: [Xen-devel] [v2 11/11] p2m/ept: enable PML in p2m-ept for log-dirty

On 04/17/2015 02:28 PM, Jan Beulich wrote: On 17.04.15 at 04:40, wrote: On 04/16/2015 11:54 PM, Jan Beulich wrote: On 15.04.15 at 09:03, wrote: This patch firstly enables EPT A/D bits if PML is used, as PML depends on EPT A/D bits to work. A bit is set for all present leaf p2m types, D b

Re: [Xen-devel] [v2 06/11] vmx: add help functions to support PML

On 04/17/2015 02:58 PM, Jan Beulich wrote: On 17.04.15 at 08:51, wrote: On 04/17/2015 02:23 PM, Jan Beulich wrote: On 17.04.15 at 05:10, wrote: On 04/16/2015 11:42 PM, Jan Beulich wrote: On 15.04.15 at 09:03, wrote: +void vmx_vcpu_flush_pml_buffer(struct vcpu *v) +{ +uint64_t *pml_b

Re: [Xen-devel] [v2 11/11] p2m/ept: enable PML in p2m-ept for log-dirty

>>> On 17.04.15 at 09:10, wrote: > On 04/17/2015 02:28 PM, Jan Beulich wrote: > On 17.04.15 at 04:40, wrote: >>> On 04/16/2015 11:54 PM, Jan Beulich wrote: >>> On 15.04.15 at 09:03, wrote: > This patch firstly enables EPT A/D bits if PML is used, as PML depends on >>> EPT > A/D b

Re: [Xen-devel] [v2 06/11] vmx: add help functions to support PML

>>> On 17.04.15 at 09:23, wrote: > > On 04/17/2015 02:58 PM, Jan Beulich wrote: > On 17.04.15 at 08:51, wrote: >>> On 04/17/2015 02:23 PM, Jan Beulich wrote: >>> On 17.04.15 at 05:10, wrote: > On 04/16/2015 11:42 PM, Jan Beulich wrote: > On 15.04.15 at 09:03, wrote: >>

Re: [Xen-devel] [v1] x86/cpuidle: get accurate C0 value with xenpm tool

-Original Message- From: Jan Beulich [mailto:jbeul...@suse.com] Sent: Thursday, April 16, 2015 4:49 PM To: Han, Huaitong Cc: xen-devel@lists.xen.org Subject: Re: [v1] x86/cpuidle: get accurate C0 value with xenpm tool >>> On 16.04.15 at 08:03, wrote: > When checking the ACPI funciton o

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

flight 50428 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/50428/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 9 windows-install fail REGR. vs. 50391 Regressions which a

Re: [Xen-devel] [v1] x86/cpuidle: get accurate C0 value with xenpm tool

>>> On 17.04.15 at 09:36, wrote: > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Thursday, April 16, 2015 4:49 PM On 16.04.15 at 08:03, wrote: >> When checking the ACPI funciton of C-status, after 100 seconds sleep, >> the sampling value of C0 C-status from the xenpm tool decreases.

Re: [Xen-devel] [v2 06/11] vmx: add help functions to support PML

On 04/17/2015 03:37 PM, Jan Beulich wrote: On 17.04.15 at 09:23, wrote: On 04/17/2015 02:58 PM, Jan Beulich wrote: On 17.04.15 at 08:51, wrote: On 04/17/2015 02:23 PM, Jan Beulich wrote: On 17.04.15 at 05:10, wrote: On 04/16/2015 11:42 PM, Jan Beulich wrote: On 15.04.15 at 09:03, wrot

Re: [Xen-devel] [PATCH 1/1] x86/hvm: prevent hvm_free_ioreq_gmfn() clobber of arbitrary memory

>>> On 16.04.15 at 18:37, wrote: > At 12:32 +0100 on 16 Apr (1429187564), Jan Beulich wrote: >> >>> On 16.04.15 at 12:53, wrote: >> > I would be inclined to use a bigger hammer here. IMO refactoring like >> > this makes it easier to reason about (compile tested only): >> >> This looks like a pr

Re: [Xen-devel] [PATCH] passthrough/amd: avoid reading an uninitialized variable.

>>> On 16.04.15 at 19:19, wrote: > On 16/04/15 17:44, Tim Deegan wrote: >> update_intremap_entry_from_msi() doesn't write to its data pointer on >> some error paths, so we copying that variable into the msg would count >> as undefined behaviour. >> >> Signed-off-by: Tim Deegan >> Cc: Suravee Suth

[Xen-devel] [PATCH v4 4/5] vscsiif.h: add some notes about xenstore layout

Signed-off-by: Olaf Hering Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- xen/include/public/io/vscsiif.h | 68 + 1 file changed, 68 insertions(+) diff --git a/xen/include/public/io/vscsiif.h b/xen/include/public/i

[Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev property. Add the missing :LUN part to the comment. Signed-off-by: Olaf Hering Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- xen/include/public/io/vscsiif.h | 2 +- 1 file changed,

[Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu --- docs/man/xl.cfg.pod.5| 55 +++ docs/man/xl.pod.1

[Xen-devel] [PATCH v4 0/5] libbxl: add support for pvscsi, iteration 4

Port vscsi=[] and scsi-{attach,detach,list} commands from xend to libxl. TODO: - find better name for ->feature_host (perhaps ->sg_io as used in libvirt?) - maybe use events instead of polling for "state" changes in reconfigure (libxl__wait_for_backend vs. libxl__ev_devstate_wait) - add code

[Xen-devel] [PATCH v4 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

Just to make them public, not meant for merging: The scripts used during development to create a bunch of SCSI devices in dom0 using the Linux target framework. targetcli3 and rtslib3 is used. A patch is required for python-rtslib: http://article.gmane.org/gmane.linux.scsi.target.devel/8146 Signe

[Xen-devel] [PATCH v4 2/5] docs: add vscsi to xenstore-paths.markdown

Signed-off-by: Olaf Hering Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- docs/misc/xenstore-paths.markdown | 10 ++ 1 file changed, 10 insertions(+) diff --git a/docs/misc/xenstore-paths.markdown b/docs/misc/xenstore-paths.markdown index d94ea9

Re: [Xen-devel] [v2 06/11] vmx: add help functions to support PML

At 11:32 +0800 on 17 Apr (1429270332), Kai Huang wrote: > > > On 04/17/2015 08:10 AM, Tim Deegan wrote: > > At 22:57 + on 16 Apr (1429225024), Tian, Kevin wrote: > > > >>> From: Kai Huang [mailto:kai.hu...@linux.intel.com] > >>> +if ( !p2m_change_type_one(v->domain, gfn, p2m_ram_logdi

Re: [Xen-devel] [PATCH v4 2/5] docs: add vscsi to xenstore-paths.markdown

On Fri, Apr 17, Olaf Hering wrote: > @@ -302,6 +307,10 @@ A virtual keyboard device backend. Described by > A virtual network device backend. Described by > [xen/include/public/io/netif.h][NETIF] > > + ~/backend/vscsi/$DOMID/$DEVID/* [] > + > +A PV SCSI backend. Described in [pvscsi.txt](p

Re: [Xen-devel] [PATCH 4/9] raisin: add a component to build qemu_traditional

On Thu, 2015-04-16 at 17:03 +0100, Stefano Stabellini wrote: > On Wed, 15 Apr 2015, Andrew Cooper wrote: > > > -./configure --prefix=$PREFIX > > > --with-system-qemu=/usr/bin/qemu-system-i386 > > > +./configure --prefix=$PREFIX > > > --with-system-qemu=/usr/bin/qemu-system-i386 \ > > > +

Re: [Xen-devel] [PATCH 3/8] raisin: Use fakeroot for mkdeb so we can build the package as non-root

On Thu, 2015-04-16 at 17:19 +0100, George Dunlap wrote: > Signed-off-by: George Dunlap AIUI raisin does build dependency handling/tracking. If so then fakeroot ought to be added to it I think? > --- > CC: Stefano Stabellini Does CC below a CC work? Interesting. > --- > lib/common-functions.s

Re: [Xen-devel] Archiving Xen on ARM and PVOPS subprojects

On Thu, 2015-04-16 at 14:25 -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Apr 16, 2015 at 12:37:15PM +0100, Ian Campbell wrote: > > On Wed, 2015-04-15 at 17:23 +0100, Lars Kurth wrote: > > > > > > According to http://xenproject.org/governance.html we would need to > > > perform an archivation revi

Re: [Xen-devel] [PATCH 2/3] libxl: fd events: Break out fd_occurs

On Thu, 2015-04-16 at 19:23 +0100, Ian Jackson wrote: > No functional change, only code motion. > > Currently, contrary to this function's name, there are two sites where > efd->func() is called so one of them doesn't go through here just yet. > That will be dealt with in the next commit. > > Sig

Re: [Xen-devel] [PATCH 0/2] Re: libvirtd live-locking on CTX_LOCK when doing 'virsh save /tmp/blah' with guest corrupting memory (on purpose).

On Thu, 2015-04-16 at 18:18 +0100, Ian Jackson wrote: > Jim Fehlig writes ("Re: [PATCH 0/2] Re: libvirtd live-locking on CTX_LOCK > when doing 'virsh save /tmp/blah' with guest corrupting memory (on > purpose)."): > > On 04/14/2015 11:31 AM, Ian Jackson wrote: > > > I have produced what I think

Re: [Xen-devel] [PATCH 1/3] libxl: fd events: Break out libxl__fd_poll_recheck

On Thu, 2015-04-16 at 19:23 +0100, Ian Jackson wrote: > Replaces two call sites where a rechecking poll() was open-coded. > > No functional change, other than to highly unusual error path > diagnosis, and debug and error message output. > > Signed-off-by: Ian Jackson > CC: Jim Fehlig > CC: Konr

Re: [Xen-devel] [PATCH 3/3] libxl: fd events: Suppress spurious fd events

On Thu, 2015-04-16 at 19:23 +0100, Ian Jackson wrote: > Always recheck with poll() right before making the callback. > > All sorts of things may have happened since poll() originally signaled > the fd. We would like the main functional libxl code not to have to > worry about spurious wakeups. Th

Re: [Xen-devel] [v2 06/11] vmx: add help functions to support PML

On 04/17/2015 04:36 PM, Tim Deegan wrote: At 11:32 +0800 on 17 Apr (1429270332), Kai Huang wrote: On 04/17/2015 08:10 AM, Tim Deegan wrote: At 22:57 + on 16 Apr (1429225024), Tian, Kevin wrote: From: Kai Huang [mailto:kai.hu...@linux.intel.com] +if ( !p2m_change_type_one(v->dom

Re: [Xen-devel] [PATCH OSSTEST v2 0/5] Tweaks to allow running non-master production instances

On Wed, 2015-04-15 at 14:47 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH OSSTEST v2 0/5] Tweaks to allow running > non-master production instances"): > > Since you have now acked everything may I push to the Cambridge branch > > (once the VM is working again)? > > Yes. > > > (mod

Re: [Xen-devel] memaccess: skipping mem_access_send_req

On Fri, Apr 10, 2015 at 17:03:26, Andrew Cooper wrote: > Are you perhaps looking for something similar to Intel #VE support? Yes, in that we want a way to notify a guest that it has made an access that violated a stage-2 / EPT permission. However, for our purposes a trap into the hypervisor foll

Re: [Xen-devel] memaccess: skipping mem_access_send_req

On Wed, Apr 15, 2015 at 10:26:52, Ian Campbell wrote: > > We would like to use memaccess to perform (1) - but rather than > > pausing the VCPU in (2), instead simply directly inject the > > exception into the VCPU. > > That is, into the VCPUs whose permissions have been modified behind > its back a

Re: [Xen-devel] [PATCH 3/8] raisin: Use fakeroot for mkdeb so we can build the package as non-root

On Fri, 17 Apr 2015, Ian Campbell wrote: > On Thu, 2015-04-16 at 17:19 +0100, George Dunlap wrote: > > Signed-off-by: George Dunlap > > AIUI raisin does build dependency handling/tracking. If so then fakeroot > ought to be added to it I think? Good point! Please add fakeroot as Debian specific d

[Xen-devel] [PATCH OSSTEST] cri-args-hostlists: No longer export $OSSTEST_CONFIG

All sites now have a suitable $HOME/.xen-osstest/cri-args-hostslists.settings in place which does this. Signed-off-by: Ian Campbell --- To be applied once the following commit passes Cambridge push gate, is merged into the master instance and passes the push gate there. commit 5926203d0851792e4f

Re: [Xen-devel] [PATCH v5 p2 04/19] xen/arm: Implement hypercall DOMCTL_{, un}bind_pt_pirq

On Fri, 2015-04-17 at 07:02 +0100, Julien Grall wrote: > Hi Ian, > > On 16/04/2015 16:40, Ian Campbell wrote: > > On Thu, 2015-04-16 at 16:20 +0100, Julien Grall wrote: > Concerning XSM, even if ARM is using one hypercall rather than 2, the > resulting check is nearly the same. > >

Re: [Xen-devel] Is it ok to routing periperal irq to any Domain0's vCPU on Xen ARM 4.5.x?

On Fri, 2015-04-17 at 11:36 +0900, 신정섭 wrote: > > > I'm studying periperal irq routing to Domain0's vCPU What do you mean by "peripheral irq routing"? Do you mean supporting the guest writing to GICD_ITARGER to cause an interrupt to be injected to a specific vcpu? I thought that was supposed t

Re: [Xen-devel] [libvirt bisection] complete build-armhf-libvirt

On Thu, 2015-04-16 at 21:46 +, osstest service user wrote: > branch xen-unstable > xen branch xen-unstable > job build-armhf-libvirt > test libvirt-build > > Tree: libvirt git://libvirt.org/libvirt.git > Tree: libvirt_gnulib git://git.sv.gnu.org/gnulib.git > Tree: qemuu git://xenbits.xen.org/s

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

On Wed, 2015-04-15 at 14:16 +0100, Daniel P. Berrange wrote: > Yeah, there is nothing Xen specific about the problem - it is entirely > down to the build toolchain & compiler options. FYI our bisector has now tripped over another related problem, http://lists.xen.org/archives/html/xen-devel/2015-

Re: [Xen-devel] [PATCH 1/8] raisin: Fix non-verbose case

On Thu, 16 Apr 2015, George Dunlap wrote: > Because we use "set -e", we can't use the "a && b" construct, as it will fail > and stop the script. > > Signed-off-by: George Dunlap I am wondering whether we should ban both a && b and a || b from the code and just go with the more verbose but also

Re: [Xen-devel] [PATCH 4/8] raisin: Use PKGTYPE rather than DISTRO to determine how to build a package

On Thu, 16 Apr 2015, George Dunlap wrote: > Signed-off-by: George Dunlap > --- > CC: Stefano Stabellini > --- > lib/common-functions.sh | 18 +- > 1 file changed, 9 insertions(+), 9 deletions(-) > > diff --git a/lib/common-functions.sh b/lib/common-functions.sh > index 373d6fb..

Re: [Xen-devel] [PATCH 2/8] Include actual config in package, not defconfig

On Thu, 16 Apr 2015, George Dunlap wrote: > We want to include the actual config used to build the packages. > > Signed-off-by: George Dunlap Acked and committed > CC: Stefano Stabellini > --- > raise | 1 - > scripts/mkdeb | 2 +- > scripts/mkrpm | 2 +- > 3 files changed, 2 inserti

Re: [Xen-devel] [PATCH 5/8] raisin: Fix CentOS build

On Thu, 16 Apr 2015, George Dunlap wrote: > Add package dependencies for CentOS. Also use PKGTYPE rather than > DISTRO to determine if we need rpm-build. > > I've tested this for xen but not for libvirt or grub. > > Signed-off-by: George Dunlap > --- > CC: Stefano Stabellini > --- > component

[Xen-devel] [PATCH OSSTEST] sg-run-job: Repeat rumpuserxen-demo-xenstorels test 150 times

(Was: Re: [osstest test] 50423: regressions - FAIL) On Fri, 2015-04-17 at 11:10 +0100, Ian Campbell wrote: > On Thu, 2015-04-16 at 21:36 +, osstest service user wrote: > > flight 50423 osstest real [real] > > http://logs.test-lab.xenproject.org/osstest/logs/50423/ > > > > Regressions :-( > >

Re: [Xen-devel] [PATCH 6/8] raisin: Break build into components and allow the sub-steps to be called individually

On Thu, 16 Apr 2015, George Dunlap wrote: > This makes it easier to debug just one aspect of the build. > > Signed-off-by: George Dunlap > --- > CC: Stefano Stabellini > --- > lib/commands.sh | 16 > raise | 2 +- > 2 files changed, 13 insertions(+), 5 deletions(-) >

Re: [Xen-devel] [RFC] Linux save_fl PVOP for Xen

On 16/04/15 19:40, Andrew Cooper wrote: > Having recently got some Broadwell hardware, our automatic test system > discovered that 32bit PV guests would reliably blow up while attempting > to boot. > > It turns out that the save_fl PVOP is at fault. The comment is false, > as setup_smap() uses it

[Xen-devel] kernel BUG at drivers/xen/grant-table.c:923!

(Was Re: [osstest test] 50423: regressions - FAIL) This cropped up in an osstest flight (the results only go to Ian and I). On Fri, 2015-04-17 at 11:10 +0100, Ian Campbell wrote: > On Thu, 2015-04-16 at 21:36 +, osstest service user wrote: > > flight 50423 osstest real [real] > > http://logs.

Re: [Xen-devel] memaccess: skipping mem_access_send_req

On Fri, 2015-04-17 at 10:35 +0100, Gareth Stockwell wrote: > On Wed, Apr 15, 2015 at 10:26:52, Ian Campbell wrote: > > > We would like to use memaccess to perform (1) - but rather than > > Is the guest expected to be aware of this, i.e. to be somewhat > > paravirtualised? I suppose it must have to

Re: [Xen-devel] kernel BUG at drivers/xen/grant-table.c:923!

On 17/04/15 11:28, Ian Campbell wrote: > (Was Re: [osstest test] 50423: regressions - FAIL) > > This cropped up in an osstest flight (the results only go to Ian and I). > > On Fri, 2015-04-17 at 11:10 +0100, Ian Campbell wrote: >> On Thu, 2015-04-16 at 21:36 +, osstest service user wrote: >>>

Re: [Xen-devel] [PATCH 05/19] xen: arm: Add and use r/o+raz and w/o+wi helpers

On Fri, 2015-04-17 at 07:18 +0100, Julien Grall wrote: > >>> +/* Read only + read as zero */ > >> > >> This comment may confuse developer who wants to implement RO register > >> which another value than 0. > >> > >> I got confuse too. It would be nice to expand the comment for the RO case. > > > >

Re: [Xen-devel] [PATCH 7/8] raisin: Rework component specification

On Thu, 16 Apr 2015, George Dunlap wrote: > Allow COMPONENTS to be specified in the config (or on the command-line) > > Now you can keep all components enabled in your config but build only > one like so: > > COMPONENTS="xen" ./raise build > > Signed-off-by: George Dunlap Good idea, I wanted t

Re: [Xen-devel] [PATCH 7/8] raisin: Rework component specification

On Thu, 16 Apr 2015, George Dunlap wrote: > Allow COMPONENTS to be specified in the config (or on the command-line) > > Now you can keep all components enabled in your config but build only > one like so: > > COMPONENTS="xen" ./raise build > > Signed-off-by: George Dunlap > --- > CC: Stefano St

Re: [Xen-devel] kernel BUG at drivers/xen/grant-table.c:923!

On Fri, Apr 17, 2015 at 11:28:39AM +0100, Ian Campbell wrote: > (Was Re: [osstest test] 50423: regressions - FAIL) > > This cropped up in an osstest flight (the results only go to Ian and I). > > On Fri, 2015-04-17 at 11:10 +0100, Ian Campbell wrote: > > On Thu, 2015-04-16 at 21:36 +, osstest

Re: [Xen-devel] kernel BUG at drivers/xen/grant-table.c:923!

On Fri, 2015-04-17 at 11:31 +0100, David Vrabel wrote: > On 17/04/15 11:28, Ian Campbell wrote: > > (Was Re: [osstest test] 50423: regressions - FAIL) > > > > This cropped up in an osstest flight (the results only go to Ian and I). > > > > On Fri, 2015-04-17 at 11:10 +0100, Ian Campbell wrote: >

Re: [Xen-devel] kernel BUG at drivers/xen/grant-table.c:923!

On Fri, 2015-04-17 at 11:45 +0100, Wei Liu wrote: > On Fri, Apr 17, 2015 at 11:28:39AM +0100, Ian Campbell wrote: > > (Was Re: [osstest test] 50423: regressions - FAIL) > > > > This cropped up in an osstest flight (the results only go to Ian and I). > > > > On Fri, 2015-04-17 at 11:10 +0100, Ian

Re: [Xen-devel] [PATCH 8/8] raisin: RFC Add blktap2 as an external tree

On Thu, 16 Apr 2015, George Dunlap wrote: > Until we start hosting the blktap repo on xenbits, use the one from github. > > Also, we need to pass in the directories where to find the > libblktapctl headers in the Xen build. > > Signed-off-by: George Dunlap > --- > Note: For now use the "upstream

Re: [Xen-devel] [PATCH 8/8] raisin: RFC Add blktap2 as an external tree

On Thu, 16 Apr 2015, George Dunlap wrote: > Until we start hosting the blktap repo on xenbits, use the one from github. > > Also, we need to pass in the directories where to find the > libblktapctl headers in the Xen build. > > Signed-off-by: George Dunlap > --- > Note: For now use the "upstream

Re: [Xen-devel] [PATCH 8/8] raisin: RFC Add blktap2 as an external tree

On Fri, 2015-04-17 at 11:50 +0100, Stefano Stabellini wrote: > > +local DEP_CentOS_x86_32="$DEP_Redhat_common" > > +local DEP_CentOS_x86_64="$DEP_Redhat_x86_32" > > Redhat? I don't know no Redhat. Is the use of x86_32 rather than common on the x86_64 case deliberate? Ian.

[Xen-devel] crash in efi_runtime_call

Since booting xen fails on my ProBook unless I specify "maxcpus=1" I tried the EFI firmware today. To my surprise it boots and finds all cpus. But once some efi driver in dom0 is loaded xen crashes. The same happens with xen-4.4 as included in SLE12. ... (XEN) Xen call trace: (XEN)[

[Xen-devel] [xen-4.4-testing test] 50429: tolerable FAIL - PUSHED

flight 50429 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/50429/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-freebsd10-amd64 16 guest-localmigrate/x10 fail in 50392 pass in 50429 test-amd64-amd64-xl-win

[Xen-devel] Question about DMA on 1:1 mapping dom0 of arm64

Hi all, According to my recent experience, there might be some problems of swiotlb dma map on 1:1 mapping arm64 dom0 with large memory. The issue is like below: For those arm64 server with large memory, it is possible to set dom0_mem > 4G (e.g. I have one set with 16G). In this case, according to

[Xen-devel] [PATCH] libxl: use DEBUG log level instead of INFO

Make libxl less noisy when destroying a domain. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- tools/libxl/libxl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 511eef1..c106756 100644 --- a/tools/libxl/libxl.c

Re: [Xen-devel] [PATCH V2] xen/arm64: Use virtual address when setting up early_printk fixmap

On Wed, Apr 15, 2015 at 03:41:44PM +0100, Ian Campbell wrote: > On Wed, 2015-04-08 at 12:33 +0100, Julien Grall wrote: > > Hi Chen, > > > > On 07/04/15 12:24, Chen Baozi wrote: > > > We have already had the boot pagetable when reaching the point > > > > s/had/added/ ? > > I think "switched too"

Re: [Xen-devel] Commit 1aeb1156fa43fe2cd2b5003995b20466cd19a622: "x86 don't change affinity with interrupt unmasked", APCI errors and assorted pci trouble

>>> On 14.04.15 at 14:46, wrote: > I just had a hunch .. could it be related to the kernel apci/irq refactoring > series of Jiang Liu, that already caused a lot of trouble in 3.17, 3.18 and > 3.19 > with Xen. And yes that seems to be the case: > > On Xen without "x86 don't change affinity with

Re: [Xen-devel] [PATCH 13/19] xen: arm: Annotate registers trapped by MDCR_EL2.TDRA

On Mon, 2015-04-06 at 15:24 +0200, Julien Grall wrote: > Hi Ian, > > On 31/03/2015 12:07, Ian Campbell wrote: > > Signed-off-by: Ian Campbell > > --- > > xen/arch/arm/traps.c | 32 > > xen/include/asm-arm/cpregs.h |4 > > xen/include/asm-a

Re: [Xen-devel] crash in efi_runtime_call

On 17/04/15 12:17, Olaf Hering wrote: > Since booting xen fails on my ProBook unless I specify "maxcpus=1" I > tried the EFI firmware today. To my surprise it boots and finds all > cpus. But once some efi driver in dom0 is loaded xen crashes. The same > happens with xen-4.4 as included in SLE12. >

Re: [Xen-devel] [PATCH 15/19] xen: arm: Annotate registers trapped by MDCR_EL2.TDA

On Mon, 2015-04-06 at 15:41 +0200, Julien Grall wrote: > Hi Ian, > > On 31/03/2015 12:07, Ian Campbell wrote: > > Gather the affected handlers in a single place per trap type. > > > > Signed-off-by: Ian Campbell > > --- > > xen/arch/arm/traps.c | 71 > > ++

Re: [Xen-devel] [PATCH 18/19] xen: arm: Annotate registers trapped when CNTHCTL_EL2.EL1PCEN == 0

On Mon, 2015-04-06 at 15:58 +0200, Julien Grall wrote: > Hi Ian, > > On 31/03/2015 12:07, Ian Campbell wrote: > > Signed-off-by: Ian Campbell > > --- > > xen/arch/arm/traps.c | 20 ++-- > > 1 file changed, 18 insertions(+), 2 deletions(-) > > > > diff --git a/xen/arch/arm/tra

Re: [Xen-devel] [PATCHv2 3/6] xen: generic xadd() for ticket locks

On Thu, 2015-04-16 at 16:28 +0100, Jan Beulich wrote: > >>> On 10.04.15 at 16:19, wrote: > > +#define xadd(ptr, v) generic_xaddl((ptr), (v)) > > I think it is at least confusing to call the thing xadd (looking to be > size generic) and then expand to generic_xaddl (only supporting > 32-bit operat

Re: [Xen-devel] [PATCHv2 3/6] xen: generic xadd() for ticket locks

On 17/04/15 13:32, Ian Campbell wrote: > On Thu, 2015-04-16 at 16:28 +0100, Jan Beulich wrote: > On 10.04.15 at 16:19, wrote: >>> +#define xadd(ptr, v) generic_xaddl((ptr), (v)) >> >> I think it is at least confusing to call the thing xadd (looking to be >> size generic) and then expand to gen

Re: [Xen-devel] [PATCHv2 3/6] xen: generic xadd() for ticket locks

On Fri, 2015-04-17 at 13:32 +0100, Ian Campbell wrote: > On Thu, 2015-04-16 at 16:28 +0100, Jan Beulich wrote: > > >>> On 10.04.15 at 16:19, wrote: > > > +#define xadd(ptr, v) generic_xaddl((ptr), (v)) > > > > I think it is at least confusing to call the thing xadd (looking to be > > size generic

Re: [Xen-devel] crash in efi_runtime_call

>>> On 17.04.15 at 13:59, wrote: > On 17/04/15 12:17, Olaf Hering wrote: >> Since booting xen fails on my ProBook unless I specify "maxcpus=1" I >> tried the EFI firmware today. To my surprise it boots and finds all >> cpus. But once some efi driver in dom0 is loaded xen crashes. The same >> happe

Re: [Xen-devel] crash in efi_runtime_call

On 17/04/15 13:39, Jan Beulich wrote: On 17.04.15 at 13:59, wrote: >> On 17/04/15 12:17, Olaf Hering wrote: >>> Since booting xen fails on my ProBook unless I specify "maxcpus=1" I >>> tried the EFI firmware today. To my surprise it boots and finds all >>> cpus. But once some efi driver in do

[Xen-devel] [PATCH 0/2] raisin: build qemu-traditional and seabios

Hi all, This patch series builds qemu-traditional and seabios separately from the Xen tree. It also change the QEMU build to be more Xen specific, installing the QEMU binary under /usr/lib/xen/bin. Changes compared to the previous version of the qemu-traditional patch: - --enable-rombios (otherw

Re: [Xen-devel] [PATCHv2 3/6] xen: generic xadd() for ticket locks

On Fri, 2015-04-17 at 13:34 +0100, David Vrabel wrote: > On 17/04/15 13:32, Ian Campbell wrote: > > On Thu, 2015-04-16 at 16:28 +0100, Jan Beulich wrote: > > On 10.04.15 at 16:19, wrote: > >>> +#define xadd(ptr, v) generic_xaddl((ptr), (v)) > >> > >> I think it is at least confusing to call th

[Xen-devel] [PATCH v2 1/2] raisin: add a component to build qemu_traditional

Introduce a component to build qemu-traditional out of xen-unstable. Do not compile qemu-traditional from xen-unstable by passing the right command line option to configure. Signed-off-by: Stefano Stabellini --- components/qemu_traditional | 49 +++ comp

[Xen-devel] [PATCH 2/2] raisin: introduce seabios component

Build SeaBIOS as a separate component. Pass --with-system-seabios to the xen configure script. Signed-off-by: Stefano Stabellini --- components/seabios | 57 components/series |1 + components/xen |3 ++- defconfig |

Re: [Xen-devel] [PATCHv2 3/6] xen: generic xadd() for ticket locks

On 17/04/15 14:09, Ian Campbell wrote: > On Fri, 2015-04-17 at 13:34 +0100, David Vrabel wrote: >> >> Can you use >> >> git://xenbits.xen.org/people/dvrabel/xen.git ticketlocks-v3 git://xenbits.xen.org/people/dvrabel/xen.git ticketlock-v3 > I tried that and it built and booted just fine on both

Re: [Xen-devel] Question about DMA on 1:1 mapping dom0 of arm64

On Fri, 2015-04-17 at 19:24 +0800, Chen Baozi wrote: > Hi all, > > According to my recent experience, there might be some problems of swiotlb > dma map on 1:1 mapping arm64 dom0 with large memory. The issue is like below: > > For those arm64 server with large memory, it is possible to set dom0_me

Re: [Xen-devel] Archiving Xen on ARM and PVOPS subprojects

On Wed, 15 Apr 2015, Lars Kurth wrote: > Hi all, > I wanted to make the proposal to archive the following two subproject on the > grounds that they completed their goals > > a) http://xenproject.org/developers/teams/pvops.html > b) http://xenproject.org/developers/teams/arm-hypervisor.html > > I

Re: [Xen-devel] crash in efi_runtime_call

On Fri, Apr 17, 2015 at 01:54:28PM +0100, Andrew Cooper wrote: > On 17/04/15 13:39, Jan Beulich wrote: > On 17.04.15 at 13:59, wrote: > >> On 17/04/15 12:17, Olaf Hering wrote: > >>> Since booting xen fails on my ProBook unless I specify "maxcpus=1" I > >>> tried the EFI firmware today. To my

Re: [Xen-devel] [PATCH v2 2/3] xen: arm: propagate gic's #interrupt-cells property to dom0.

On Mon, 2015-03-16 at 16:01 +, Julien Grall wrote: > Hi Ian, > > On 12/03/15 17:17, Ian Campbell wrote: > > This is similar to 816f5bb1f074 "xen: arm: propagate gic's > > should propagate (rather than invent our own value) since this value > > is used to size fields within other properties wit

Re: [Xen-devel] crash in efi_runtime_call

On 17/04/15 14:40, Konrad Rzeszutek Wilk wrote: > On Fri, Apr 17, 2015 at 01:54:28PM +0100, Andrew Cooper wrote: >> On 17/04/15 13:39, Jan Beulich wrote: >> On 17.04.15 at 13:59, wrote: On 17/04/15 12:17, Olaf Hering wrote: > Since booting xen fails on my ProBook unless I specify "max

Re: [Xen-devel] crash in efi_runtime_call

>>> On 17.04.15 at 15:40, wrote: > I actually did cobble a patch like this, but it is based on Daniel's > Multibootv2 > so it won't apply cleany. See attached patchset with various 'work-arounds'. > > Jan if you are OK with them (well the 'idea' behind them) I can refresh > it against staging an

[Xen-devel] [PATCH v2 02/19] xen: arm: Fix handling of ICC_{SGI1R, SGI0R, ASGI1R}_EL1

Having injected an undefined instruction we don't want to also advance pc. So return. The ICC_{SGI0R,ASGI1R}_EL1 case was previously missing a break, so would have fallen through to the default case and injected a second undef, corrupting SPSR_EL1 and ELR_EL1 for the guest. Signed-off-by: Ian Cam

[Xen-devel] [PATCH v2 11/19] xen: arm: Annotate handlers for CPTR_EL2.Tx

Also expand on the comment when writing CPTR_EL2 to mention that most of the bits we are setting are RES1 on arm64 anyway. Signed-off-by: Ian Campbell --- v2: s/PCTR/CPTR/ Expand the comment when writing to CPTR_EL2 --- xen/arch/arm/traps.c | 23 +-- 1 file changed, 21

[Xen-devel] [PATCH v2 04/19] xen: arm: provide and use a handle_raz_wi helper

Reduces the use of goto in the trap handlers to none. Some explicitly 32-bit types become register_t here, but that's OK, on 32-bit they are 32-bit already and on 64-bit it is fine/harmless to set the larger register, a 32-bit guest won't see the top half in any case. Per section B1.2.1 (ARMv8 DD

Re: [Xen-devel] [PATCH v2 2/4] x86/MSI-X: access MSI-X table only after having enabled MSI-X

>>> On 15.04.15 at 19:41, wrote: > On Mon, Apr 13, 2015 at 10:05:14AM +0100, Jan Beulich wrote: >> >>> On 10.04.15 at 22:02, wrote: >> > On Wed, Mar 25, 2015 at 04:39:49PM +, Jan Beulich wrote: >> >> As done in Linux by f598282f51 ("PCI: Fix the NIU MSI-X problem in a >> >> better way") and i

[Xen-devel] [PATCH v2 13/19] xen: arm: Annotate registers trapped by MDCR_EL2.TDRA

DBGDRAR and DBGDSAR are actually two cp or sys registers each, one 32-bit and one 64-bit. The cpregs #define is suffixed "64" and annotations are added to both handlers. MDRAR_EL1 (arm64 version of DBGDRAR) wasn't handled, so add that here. Signed-off-by: Ian Campbell --- v2: Move comment next t

[Xen-devel] [PATCH v2 10/19] xen: arm: implement handling of registers trapped by CPTR_EL2.TTA

Add explicit handler for 64-bit CP14 accesses, with more relevant debug message (as per other handlers) and to provide a place for a comment. Signed-off-by: Ian Campbell --- v2: Changed title from "xen: arm: Annotate registers trapped by CPTR_EL2.TTA" Add "And all other unknown registers"

[Xen-devel] [PATCH v2 19/19] xen: arm: Annotate source of ICC SGI register trapping

I was unable to find an ARMv8 ARM reference to this, so refer to the GIC Architecture Specification instead. ARMv8 ARM does cover other ways of trapping these accesses via ICH_HCR_EL2 but we don't use those and they trap additional registers as well. Signed-off-by: Ian Campbell Reviewed-by: Juli

[Xen-devel] [PATCH v2 01/19] xen: arm: constify union hsr and struct hsr_* where possible.

Signed-off-by: Ian Campbell Reviewed-by: Julien Grall --- xen/arch/arm/traps.c | 41 + 1 file changed, 21 insertions(+), 20 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c index aaa9d93..69b9513 100644 --- a/xen/arch/arm/traps.c ++

[Xen-devel] [PATCH v2 09/19] xen: arm: Annotate registers trapped by HCR_EL1.TIDCP

This traps variety of implementation defined registers, so add a note to the default case of the respective handler. Signed-off-by: Ian Campbell Reviewed-by: Julien Grall --- v2: Typo in subject --- xen/arch/arm/traps.c | 16 1 file changed, 16 insertions(+) diff --git a/xen

[Xen-devel] [PATCH v2 00/19] xen: arm: cleanup traps.c

While working on reenabling 32-bit user space on arm63 I concluded that the trap handling in traps.c had grown into a twisty confusing mess. Lets try and sort that out. This series contains two halves (after a couple of preparatory cleanups). First clean up the goto maze which we've found ourselv

[Xen-devel] [PATCH v2 03/19] xen: arm: call inject_undef_exception directly

Reducing the amount of goto maze considerably. Signed-off-by: Ian Campbell Reviewed-by: Julien Grall --- xen/arch/arm/traps.c | 56 +++--- 1 file changed, 26 insertions(+), 30 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c in

[Xen-devel] [PATCH v2 05/19] xen: arm: Add and use r/o+raz and w/o+wi helpers

Signed-off-by: Ian Campbell --- v2: Move last paramter of a handle_ro_raz call to next patch where it belongs. --- xen/arch/arm/traps.c | 52 -- 1 file changed, 33 insertions(+), 19 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/ar

[Xen-devel] [PATCH v2 08/19] xen: arm: implement handling of ACTLR_EL1 trap

While annotating ACTLR I noticed that we don't appear to handle the 64-bit version of this trap. Do so and annotate everything. Signed-off-by: Ian Campbell --- v2: s/TASC/TACR/ and s/HSR/HCR/ --- xen/arch/arm/traps.c | 20 xen/include/asm-arm/sysregs.h |1 + 2

[Xen-devel] [PATCH v2 06/19] xen: arm: add minimum exception level argument to trap handler helpers

Removes a load of boiler plate. Signed-off-by: Ian Campbell --- v2: Move last parameter of a call to handle_ro_raz here where it belongs. Added asserts for valid min_el values --- xen/arch/arm/traps.c | 73 +++--- 1 file changed, 39 insertion

[Xen-devel] [PATCH v2 17/19] xen: arm: Remove CNTPCT_EL0 trap handling.

We set CNTHCTL_EL2.EL1PCTEN and therefore according to ARMv8 (DDI 0487A.d) D1-1510 Table D1-60 we are not trapping this. Signed-off-by: Ian Campbell Reviewed-by: Julien Grall --- xen/arch/arm/traps.c |1 - xen/arch/arm/vtimer.c | 30 -- 2 files changed, 31 del

[Xen-devel] [PATCH v2 14/19] xen: arm: Annotate registers trapped by MDCR_EL2.TDOSA

Gather the affected handlers in a single place per trap type. Add some HSR_SYSREG and AArch32 defines for those registers (because I'd already typed them in when I realised I didn't need them). Signed-off-by: Ian Campbell --- v2: Move comment block in cp14_dbg handler from incorrect place in

[Xen-devel] [PATCH v2 07/19] xen: arm: Annotate trap handler for HSR_EL2.{TWI, TWE, TSC}

Reference the bit which enables the trap and the section/page which describes what that bit enables. These ones are pretty trivial, included for completeness. Signed-off-by: Ian Campbell --- v2: s/HSR_EL2/HCR_EL2/ --- xen/arch/arm/traps.c | 17 + 1 file changed, 17 insertions(

[Xen-devel] [PATCH v2 16/19] xen: arm: Annotate registers trapped by MDCR_EL2.TPM and TPMCR

Signed-off-by: Ian Campbell --- xen/arch/arm/traps.c | 39 ++- 1 file changed, 34 insertions(+), 5 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c index 97cde45..d4505b5 100644 --- a/xen/arch/arm/traps.c +++ b/xen/arch/arm/traps.c @@ -1

[Xen-devel] [PATCH v2 15/19] xen: arm: Annotate registers trapped by MDCR_EL2.TDA

Gather the affected handlers in a single place per trap type. Signed-off-by: Ian Campbell --- xen/arch/arm/traps.c | 60 +- 1 file changed, 49 insertions(+), 11 deletions(-) diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c index 7606bff.

  1   2   >