Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-26 Thread Gary Lin
On Wed, Apr 27, 2016 at 05:39:40AM +, Ni, Ruiyu wrote: > > > Regards, > Ray > > >-Original Message- > >From: Gary Lin [mailto:g...@suse.com] > >Sent: Wednesday, April 27, 2016 12:29 PM > >To: Ni, Ruiyu > >Cc: edk2-de...@lists.01.org; Xen Devel > >Subject: Re: [edk2] OVMF broken und

Re: [Xen-devel] [PATCH v9 04/27] xen/xsplice: Hypervisor implementation of XEN_XSPLICE_op

2016-04-26 Thread Jan Beulich
>>> On 26.04.16 at 19:50, wrote: > On Tue, Apr 26, 2016 at 04:21:10AM -0600, Jan Beulich wrote: >> >>> On 25.04.16 at 17:34, wrote: >> > The implementation does not actually do any patching. >> > >> > It just adds the framework for doing the hypercalls, >> > keeping track of ELF payloads, and th

Re: [Xen-devel] [PATCH v2 07/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones).

2016-04-26 Thread Jan Beulich
>>> On 27.04.16 at 08:21, wrote: > On April 26, 2016 8:49 PM, Jan Beulich wrote: >> Hmm, the "positive" here has nothing to do with the "positive" in patch 1. >> Please just have a look at xenmem_add_to_physmap() as a whole. >> > > Thanks for reminding me. The 'positive' is ' rc = start + done

Re: [Xen-devel] [for-4.7] x86/emulate: synchronize LOCKed instruction emulation

2016-04-26 Thread Jan Beulich
>>> On 26.04.16 at 19:39, wrote: > On 26/04/16 18:23, Razvan Cojocaru wrote: >> Regarding this version of the patch, Jan has asked for more information >> on the performance impact, but I'm not sure how to obtain it in a >> rigorous manner. If it is decided that a version of this patch is >> desir

Re: [Xen-devel] [for-4.7] x86/emulate: synchronize LOCKed instruction emulation

2016-04-26 Thread Jan Beulich
>>> On 26.04.16 at 19:23, wrote: > On 04/26/16 19:03, George Dunlap wrote: >> On 19/04/16 17:35, Jan Beulich wrote: >> Razvan Cojocaru 04/19/16 1:01 PM >>> I think this might be because the LOCK prefix should guarantee that the instruction that follows it has exclusive use of shared

Re: [Xen-devel] [PATCH v2 07/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones).

2016-04-26 Thread Xu, Quan
On April 26, 2016 8:49 PM, Jan Beulich wrote: > >>> On 26.04.16 at 14:23, wrote: > > On April 25, 2016 6:19 PM, Jan Beulich wrote: > >> >>> On 18.04.16 at 16:00, wrote: > >> > --- a/xen/common/memory.c > >> > +++ b/xen/common/memory.c > >> > @@ -678,8 +678,9 @@ static int xenmem_add_to_physmap

Re: [Xen-devel] [PATCH v2] xen/arm: gicv2: Export GICv2m register frames to domain0 by device tree

2016-04-26 Thread Wei Chen
Hi Julien, On 26 April 2016 at 18:49, Julien Grall wrote: > Hello Wei, > > On 25/04/2016 10:39, Wei Chen wrote: >> >> This patch adds v2m extension support in GIC-v2 driver. The GICv2 driver >> detects the MSI frames from device tree and creates corresponding device >> tree nodes in Domain0's DTB

Re: [Xen-devel] [Xen-users] How to install Xen on Fedora 23?

2016-04-26 Thread Jason Long
Can it work on Workstation version too? Or just server version of fedora needed? On Wednesday, April 27, 2016 9:53 AM, Jason Long wrote: Not matter. I did it too : # grub2-mkconfig -o /boot/grub2/grub.cfg Generating grub configuration file ... Found linux image: /boot/vmlinuz-4.4.7-300.fc23.i6

[Xen-devel] [ovmf test] 92910: regressions - FAIL

2016-04-26 Thread osstest service owner
flight 92910 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/92910/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 65543 test-amd64-i386-xl-qemuu-ovm

Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-26 Thread Ni, Ruiyu
Regards, Ray >-Original Message- >From: Gary Lin [mailto:g...@suse.com] >Sent: Wednesday, April 27, 2016 12:29 PM >To: Ni, Ruiyu >Cc: edk2-de...@lists.01.org; Xen Devel >Subject: Re: [edk2] OVMF broken under Xen (in PCI initialisation) > >On Tue, Apr 26, 2016 at 09:40:42AM +, Ni, R

Re: [Xen-devel] [Xen-users] How to install Xen on Fedora 23?

2016-04-26 Thread Jason Long
Not matter. I did it too : # grub2-mkconfig -o /boot/grub2/grub.cfg Generating grub configuration file ... Found linux image: /boot/vmlinuz-4.4.7-300.fc23.i686 Found initrd image: /boot/initramfs-4.4.7-300.fc23.i686.img Found linux image: /boot/vmlinuz-4.2.3-300.fc23.i686 Found initrd image: /boot

Re: [Xen-devel] [PATCH] xen/x86: actually allocate legacy interrupts on PV guests

2016-04-26 Thread Juergen Gross
On 21/04/16 11:30, Stefano Stabellini wrote: > On Thu, 21 Apr 2016, Juergen Gross wrote: >> On 20/04/16 15:15, Stefano Stabellini wrote: >>> b4ff8389ed14 is incomplete: relies on nr_legacy_irqs() to get the number >>> of legacy interrupts when actually nr_legacy_irqs() returns 0 after >>> probe_825

Re: [Xen-devel] [OSSTEST PATCH] crontab: Drop linux-mingo-tip-master linux-next linux-linus

2016-04-26 Thread Juergen Gross
On 22/04/16 19:08, Roger Pau Monne wrote: > On Fri, Apr 22, 2016 at 03:54:43PM +0100, Ian Jackson wrote: >> It appears that no-one is looking at the output. These have not had a >> push to the tested output branch for at least 250 days (742 days in >> the case of linux-linus!) and the reports don'

[Xen-devel] [PATCH] libxl: don't add cache mode for empty drives

2016-04-26 Thread Jim Fehlig
qemu commit 91a097e7 forbids specifying the cache mode for empty drives. Attempting to create a domain with an empty qdisk cdrom results in qemu-system-x86_64: -drive if=ide,index=1,readonly=on,media=cdrom, cache=writeback,id=ide-832: Must specify either driver or file Change libxl to only emi

Re: [Xen-devel] How to install Xen on Fedora 23?

2016-04-26 Thread Konrad Rzeszutek Wilk
On Tue, Apr 26, 2016 at 03:00:21PM +, Jason Long wrote: > Hello. > I installed Fedora 23 x64 on my PC and id below command for installing Xen : > > # cd /etc/yum.repos.d/ > # wget http://fedorapeople.org/groups/virt/...t-preview.repo Why? No need for that. > # yum update > # yum -y install xe

Re: [Xen-devel] [edk2] OVMF broken under Xen (in PCI initialisation)

2016-04-26 Thread Gary Lin
On Tue, Apr 26, 2016 at 09:40:42AM +, Ni, Ruiyu wrote: > Gary, > I can reproduce the issue and have debugged to get the reason. > In MdeModulePkg/Bus/Pci/PciBusDxe/PciEnumeratorSupport.c: > PciEnumeratorLight() calls PciRootBridgeIo->Configuration() > while the Configuration returns EFI_UNSUPPO

[Xen-devel] [xen-unstable test] 92842: tolerable trouble: blocked/broken/fail/pass - PUSHED

2016-04-26 Thread osstest service owner
flight 92842 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/92842/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-rtds 3 host-install(3) broken pass in 92791 test-armhf-armhf-libvirt 6 xe

Re: [Xen-devel] [PATCH v9 10/27] xsplice: Add helper elf routines

2016-04-26 Thread Konrad Rzeszutek Wilk
> > +static int elf_resolve_sections(struct xsplice_elf *elf, const void *data) > > +{ ..snip.. > > +/* e_shoff and e_shnum overflow checks are done in > > xsplice_header_check. */ > > +delta = elf->hdr->e_shoff + elf->hdr->e_shnum * elf->hdr->e_shentsize; > > The added comment just helps

[Xen-devel] [xen-4.3-testing test] 92882: trouble: blocked/broken/fail/pass

2016-04-26 Thread osstest service owner
flight 92882 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/92882/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 3 host-install(3) broken REGR. vs. 878

Re: [Xen-devel] [PATCH v9 12/27] xsplice: Implement support for applying/reverting/replacing patches.

2016-04-26 Thread Konrad Rzeszutek Wilk
. snip.. > > Thinking about it again, even more stack conserving would be a > bitmap... Heheh > > > +static int apply_payload(struct payload *data) > > +{ > > +unsigned int i; > > + > > +printk(XENLOG_INFO XSPLICE "%s: Applying %u functions\n", > > +data->name, data->nfuncs);

Re: [Xen-devel] [PATCH v9 16/27] x86, xsplice: Print payload's symbol name and payload name in backtraces

2016-04-26 Thread Konrad Rzeszutek Wilk
> >> When text_addr is void *, how is this calculation wrong then? > > > > I'm sorry, ignore that. I temporarily forgot how void* arithmetic is > > defined for GCC. > > > > The other two points are still valid and may result in incorrect > > backtraces with > 1 payload loaded. > > Of course.

Re: [Xen-devel] [PATCH v9 11/27] xsplice: Implement payload loading

2016-04-26 Thread Konrad Rzeszutek Wilk
> > +static int move_payload(struct payload *payload, struct xsplice_elf *elf) > > +{ ..snip.. > > +for ( i = 1; i < elf->hdr->e_shnum; i++ ) > > +{ > > +if ( elf->sec[i].sec->sh_flags & SHF_ALLOC ) > > +{ > > +uint8_t *buf; > > Perhaps void * again? And missing

[Xen-devel] [ovmf test] 92876: regressions - FAIL

2016-04-26 Thread osstest service owner
flight 92876 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/92876/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 65543 test-amd64-i386-xl-qemuu-ovm

Re: [Xen-devel] [PATCH v9 08/27] arm/x86/vmap: Add v[z|m]alloc_xen and vm_init_type

2016-04-26 Thread Konrad Rzeszutek Wilk
> With vm_alloc() getting removed, vm_free() should get removed > here too. And with that, vm_alloc_type() and vm_free_type() can > then just become vm_alloc() and vm_free() respectively (as static > internal functions). Please take a look at this inline one: From 1c133365d98a02c8f5131cdcde119606

Re: [Xen-devel] [PATCH v9 10/27] xsplice: Add helper elf routines

2016-04-26 Thread Konrad Rzeszutek Wilk
> > +static int xsplice_header_check(const struct xsplice_elf *elf) > > +{ > > +const Elf_Ehdr *hdr = elf->hdr; > > + > > +if ( sizeof(*elf->hdr) > elf->len ) > > +{ > > +dprintk(XENLOG_ERR, XSPLICE "%s: Section header is bigger than > > payload!\n", > > +elf->n

Re: [Xen-devel] [PATCH v9 11/27] xsplice: Implement payload loading

2016-04-26 Thread Konrad Rzeszutek Wilk
> > +static int move_payload(struct payload *payload, struct xsplice_elf *elf) > > +{ .. snip.. > > +/* Compute size of different regions. */ > > +for ( i = 1; i < elf->hdr->e_shnum; i++ ) > > +{ > > +if ( (elf->sec[i].sec->sh_flags & (SHF_ALLOC|SHF_EXECINSTR)) == > > +

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Meng Xu
> >> > 4. A work-conserving mode >> I think we need to consider the item 4 carefully. Work-conserving >> mode >> is not a must for real-time schedulers and it is not the main >> purpose/goal of the RTDS scheduler. >> > It's indeed not a must for real-time schedulers. In fact, it's only > important

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Meng Xu
On Tue, Apr 26, 2016 at 6:49 PM, Dario Faggioli wrote: > On Tue, 2016-04-26 at 14:38 -0400, Meng Xu wrote: >> > So, yes, the scheduler is now feature complete (with the per-vcpu >> > parameters) and adheres to a much more sensible and scalable design >> > (event driven). Yet, these features have b

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Dario Faggioli
On Tue, 2016-04-26 at 16:00 -0400, Meng Xu wrote: > > > > > > > > The feature document template has been put together: > > > http://lists.xenproject.org/archives/html/xen-devel/2015-08/msg01 > > > 929.html > > > > > > And there are feature documents in tree already. > > > > > > Actually, writin

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Dario Faggioli
On Tue, 2016-04-26 at 14:38 -0400, Meng Xu wrote: > > So, yes, the scheduler is now feature complete (with the per-vcpu > > parameters) and adheres to a much more sensible and scalable design > > (event driven). Yet, these features have been merged very recently, > > therefore, when you say "tested

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Dario Faggioli
On Tue, 2016-04-26 at 16:35 +0100, George Dunlap wrote: > On 26/04/16 08:56, Dario Faggioli wrote: > > > > On Mon, 2016-04-25 at 21:44 -0400, Meng Xu wrote: > > >  > > Actually, writing one for RTDS would be a rather interesting and > > useful > > thing to do, IMO! :-) > I think it would be helpfu

[Xen-devel] [xen-4.3-testing test] 92825: trouble: blocked/broken/fail/pass

2016-04-26 Thread osstest service owner
flight 92825 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/92825/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 3 host-install(3) broken REGR. vs. 878

[Xen-devel] [PATCH 3.19.y-ckt 53/66] x86/mm/xen: Suppress hugetlbfs in PV guests

2016-04-26 Thread Kamal Mostafa
3.19.8-ckt20 -stable review patch. If anyone has any objections, please let me know. ---8< From: Jan Beulich commit 103f6112f253017d7062cd74d17f4a514ed4485c upstream. Huge pages are not normally available to PV guests. Not suppressi

[Xen-devel] [ovmf test] 92813: regressions - FAIL

2016-04-26 Thread osstest service owner
flight 92813 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/92813/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 65543 test-amd64-i386-xl-qemuu-ovm

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Meng Xu
>> The feature document template has been put together: >> http://lists.xenproject.org/archives/html/xen-devel/2015-08/msg01929.html >> >> And there are feature documents in tree already. >> >> Actually, writing one for RTDS would be a rather interesting and useful >> thing to do, IMO! :-) > > I th

[Xen-devel] [3.19.y-ckt stable] Patch "x86/mm/xen: Suppress hugetlbfs in PV guests" has been added to the 3.19.y-ckt tree

2016-04-26 Thread Kamal Mostafa
This is a note to let you know that I have just added a patch titled x86/mm/xen: Suppress hugetlbfs in PV guests to the linux-3.19.y-queue branch of the 3.19.y-ckt extended stable tree which can be found at: http://kernel.ubuntu.com/git/ubuntu/linux.git/log/?h=linux-3.19.y-queue This p

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

2016-04-26 Thread osstest service owner
flight 92860 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/92860/ 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] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Meng Xu
On Tue, Apr 26, 2016 at 4:56 AM, Andrew Cooper wrote: > >>> However, inside MAINTAINERS file, the status of RTDS scheduler is >>> marked as Supported (refer to commit point 28041371 by Dario Faggioli >>> on 2015-06-25). >>> >> There's indeed a discrepancy between the way one can read that bit of >

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread Meng Xu
>> When RTDS scheduler is initialized, it will print out that the >> scheduler is an experimental feature with the following lines: >> >> printk("Initializing RTDS scheduler\n" >> >>"WARNING: This is experimental software in development.\n" >> >>"Use at your own risk.\n"

[Xen-devel] [PATCH 4.2.y-ckt 77/93] x86/mm/xen: Suppress hugetlbfs in PV guests

2016-04-26 Thread Kamal Mostafa
4.2.8-ckt9 -stable review patch. If anyone has any objections, please let me know. ---8< From: Jan Beulich commit 103f6112f253017d7062cd74d17f4a514ed4485c upstream. Huge pages are not normally available to PV guests. Not suppressing

Re: [Xen-devel] xen/arm: Assertion 'timer->status >= TIMER_STATUS_inactive' failed at timer.c:279

2016-04-26 Thread Julien Grall
On 26/04/16 18:49, Dario Faggioli wrote: On Tue, 2016-04-26 at 15:25 +0100, Julien Grall wrote: Hi, Hi Dario, [...] commit eca4d65fb67a71c0f6563aafbfdd68e566c53c32 Author: Dario Faggioli Date: Tue Apr 26 17:42:36 2016 +0200 xen: sched: fix killing an uninitialized timer in free_pd

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

2016-04-26 Thread osstest service owner
flight 92800 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/92800/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 60684 build-amd6

Re: [Xen-devel] [PATCH v9 04/27] xen/xsplice: Hypervisor implementation of XEN_XSPLICE_op

2016-04-26 Thread Konrad Rzeszutek Wilk
On Tue, Apr 26, 2016 at 04:21:10AM -0600, Jan Beulich wrote: > >>> On 25.04.16 at 17:34, wrote: > > The implementation does not actually do any patching. > > > > It just adds the framework for doing the hypercalls, > > keeping track of ELF payloads, and the basic operations: > > - query which pa

Re: [Xen-devel] xen/arm: Assertion 'timer->status >= TIMER_STATUS_inactive' failed at timer.c:279

2016-04-26 Thread Dario Faggioli
On Tue, 2016-04-26 at 15:25 +0100, Julien Grall wrote: > Hi Dario, > Hi, > A couple of people have been reported Xen crash on the ARM64 > Foundation Model [1] with recent unstable. > Ok, thanks for reporting. > The crash seems to happen when Xen fails to bring up secondary CPUs > (see stack tra

Re: [Xen-devel] [for-4.7] x86/emulate: synchronize LOCKed instruction emulation

2016-04-26 Thread Andrew Cooper
On 26/04/16 18:23, Razvan Cojocaru wrote: > On 04/26/16 19:03, George Dunlap wrote: >> On 19/04/16 17:35, Jan Beulich wrote: >> Razvan Cojocaru 04/19/16 1:01 PM >>> I think this might be because the LOCK prefix should guarantee that the instruction that follows it has exclusive use o

Re: [Xen-devel] [for-4.7] x86/emulate: synchronize LOCKed instruction emulation

2016-04-26 Thread Razvan Cojocaru
On 04/26/16 19:03, George Dunlap wrote: > On 19/04/16 17:35, Jan Beulich wrote: > Razvan Cojocaru 04/19/16 1:01 PM >>> >>> I think this might be because the LOCK prefix should guarantee that the >>> instruction that follows it has exclusive use of shared memory (for both >>> reads and writes)

Re: [Xen-devel] [PATCH v2 for-4.7 00/14] Fixes for compiling with clang

2016-04-26 Thread Doug Goldstein
On 4/26/16 11:12 AM, Wei Liu wrote: > On Tue, Apr 26, 2016 at 04:52:09PM +0200, Roger Pau Monne wrote: >> Hello, >> >> This is a set of bug fixes for compiling both the hypervisor and the >> toolstack with clang. I've only tested it with clang 3.8.0 from base >> FreeBSD, so I'm not sure if it's g

[Xen-devel] MiniOS on ARM64

2016-04-26 Thread Julien Grall
Hi Chen, IIRC, you mentioned during the last Linaro Connect that you are working on an ARM64 port of MiniOS. Anastassios, in CC, is interested get MiniOS running on ARM64 as well. Do you know what is missing to get MiniOS booting? I have found a tree on your github for the port [1], is it th

Re: [Xen-devel] [PATCH] MAINTAINERS: Clarify the meaning of nested maintainership

2016-04-26 Thread Lars Kurth
On 26/04/2016 17:19, "George Dunlap" wrote: >On 21/04/16 17:59, Lars Kurth wrote: >> >> >> On 21/04/2016 17:03, "George Dunlap" wrote: >> >>> Clarify the meaning of nested maintainership. >>> >>> Signed-off-by: George Dunlap >>> --- >>> >>> MAINTAINERS | 34

[Xen-devel] [xen-unstable-smoke test] 92847: regressions - FAIL

2016-04-26 Thread osstest service owner
flight 92847 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/92847/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 9 debian-hvm-install fail REGR. vs. 92731 Tests whi

Re: [Xen-devel] [PATCH] MAINTAINERS: Clarify the meaning of nested maintainership

2016-04-26 Thread George Dunlap
On 21/04/16 17:59, Lars Kurth wrote: > > > On 21/04/2016 17:03, "George Dunlap" wrote: > >> Clarify the meaning of nested maintainership. >> >> Signed-off-by: George Dunlap >> --- >> >> MAINTAINERS | 34 ++ >> 1 file changed, 34 insertions(+) >> >> diff --git a/M

Re: [Xen-devel] [PATCH v2 for-4.7 00/14] Fixes for compiling with clang

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:09PM +0200, Roger Pau Monne wrote: > Hello, > > This is a set of bug fixes for compiling both the hypervisor and the > toolstack with clang. I've only tested it with clang 3.8.0 from base > FreeBSD, so I'm not sure if it's going to work with _all_ clang versions, >

Re: [Xen-devel] [PATCH v9 18/27] xsplice: Add support for exception tables.

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:35, wrote: > From: Ross Lagerwall > > Add support for exception tables contained within xSplice payloads. If an > exception occurs search either the main exception table or a particular > active payload's exception table depending on the instruction pointer. > > Also we ad

Re: [Xen-devel] [for-4.7] x86/emulate: synchronize LOCKed instruction emulation

2016-04-26 Thread George Dunlap
On 19/04/16 17:35, Jan Beulich wrote: Razvan Cojocaru 04/19/16 1:01 PM >>> >> I think this might be because the LOCK prefix should guarantee that the >> instruction that follows it has exclusive use of shared memory (for both >> reads and writes) but I might be misreading the docs: > > LOCK

Re: [Xen-devel] [PATCH v2 for-4.7 10/14] libxl: add the printf-like attributes to a couple of functions

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:30:36PM +0100, Andrew Cooper wrote: > On 26/04/16 16:29, Wei Liu wrote: > > On Tue, Apr 26, 2016 at 04:52:19PM +0200, Roger Pau Monne wrote: > > [...] > >> @@ -1995,9 +1995,10 @@ _hidden libxl__json_object > >> *libxl__json_parse(libxl__gc *gc_opt, const char *s); > >>

Re: [Xen-devel] [PATCH v9 17/27] xsplice: Add support for bug frames.

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:35, wrote: > From: Ross Lagerwall > > Add support for handling bug frames contained with xsplice modules. If a > trap occurs search either the kernel bug table or an applied payload's > bug table depending on the instruction pointer. > > Signed-off-by: Ross Lagerwall > Si

Re: [Xen-devel] [PATCH v9 15/27] xsplice, symbols: Implement fast symbol names -> virtual addresses lookup

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:35, wrote: > The current mechanism is geared towards fast virtual address -> > symbol names lookup. This is fine for the normal use cases > (BUG_ON, WARN_ON, etc), but for xSplice - where we need to find > hypervisor symbols - it is slow. > > To understand this patch, a desc

Re: [Xen-devel] [PATCH v2 for-4.7 12/14] libxl: fix passing the type argument to xc_psr_*

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:21PM +0200, Roger Pau Monne wrote: > The xc_psr_* functions expect the type to be xc_psr_cat_type instead of > libxl_psr_cbm_type, so do the conversion. > > Signed-off-by: Roger Pau Monné Acked-by: Wei Liu > --- > Cc: Ian Jackson > Cc: Wei Liu > --- > tools/lib

Re: [Xen-devel] [PATCH v9 14/27] xsplice, symbols: Implement symbol name resolution on address.

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:35, wrote: > --- a/xen/arch/x86/platform_hypercall.c > +++ b/xen/arch/x86/platform_hypercall.c > @@ -798,12 +798,13 @@ ret_t > do_platform_op(XEN_GUEST_HANDLE_PARAM(xen_platform_op_t) u_xenpf_op) > static char name[KSYM_NAME_LEN + 1]; /* protected by xenpf_lock */ >

Re: [Xen-devel] [PATCH v2 for-4.7 13/14] oxenstored: fix error when shifting negative value

2016-04-26 Thread Andrew Cooper
On 26/04/16 16:43, Wei Liu wrote: > On Tue, Apr 26, 2016 at 04:37:49PM +0100, Andrew Cooper wrote: >> On 26/04/16 16:35, Wei Liu wrote: >>> The title is a bit too cryptic to me. Where do that shift happen? >> Ocaml stores integers shifted left by one, and with the bottom bit set. >> >> Values with

Re: [Xen-devel] [PATCH v2 for-4.7 13/14] oxenstored: fix error when shifting negative value

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:37:49PM +0100, Andrew Cooper wrote: > On 26/04/16 16:35, Wei Liu wrote: > > The title is a bit too cryptic to me. Where do that shift happen? > > Ocaml stores integers shifted left by one, and with the bottom bit set. > > Values with the bottom bit clear are pointers in

Re: [Xen-devel] [PATCH v2 for-4.7 13/14] oxenstored: fix error when shifting negative value

2016-04-26 Thread Andrew Cooper
On 26/04/16 16:35, Wei Liu wrote: > The title is a bit too cryptic to me. Where do that shift happen? Ocaml stores integers shifted left by one, and with the bottom bit set. Values with the bottom bit clear are pointers into the GC'd heap. Values with the bottom bit set are integers, and need to

Re: [Xen-devel] Should we mark RTDS as supported feature from experimental feature?

2016-04-26 Thread George Dunlap
On 26/04/16 08:56, Dario Faggioli wrote: > On Mon, 2016-04-25 at 21:44 -0400, Meng Xu wrote: >> Hi Dario and all, >> > Hi, > >> When RTDS scheduler is initialized, it will print out that the >> scheduler is an experimental feature with the following lines: >> >> printk("Initializing RTDS sched

Re: [Xen-devel] [PATCH v2 for-4.7 14/14] tools/python: corrently use LDFLAGS and CFLAGS

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:23PM +0200, Roger Pau Monne wrote: > It is incorrect to add the LDFLAGS to the CFLAGS, and some compilers will > error out if linker flags are passed when creating object files. Fix this by > properly passing CFLAGS and LDFLAGS, instead of putting everything in > CFLAG

Re: [Xen-devel] [PATCH v2 for-4.7 13/14] oxenstored: fix error when shifting negative value

2016-04-26 Thread Wei Liu
The title is a bit too cryptic to me. Where do that shift happen? Wei. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v9 13/27] x86/xen_hello_world.xsplice: Test payload for patching 'xen_extra_version'.

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:35, wrote: > v9: old_code and new_code are void, so drop the unsigned long cast > and add void* - in both test-cases and document. > Make tests target on ARM phony > Add build dependencies on x86 build > Include public/sysctl.h as CONFIG_XSPLICE may not be exp

Re: [Xen-devel] [PATCH v2 for-4.7 10/14] libxl: add the printf-like attributes to a couple of functions

2016-04-26 Thread Andrew Cooper
On 26/04/16 16:29, Wei Liu wrote: > On Tue, Apr 26, 2016 at 04:52:19PM +0200, Roger Pau Monne wrote: > [...] >> @@ -1995,9 +1995,10 @@ _hidden libxl__json_object >> *libxl__json_parse(libxl__gc *gc_opt, const char *s); >> _hidden int libxl__device_model_version_running(libxl__gc *gc, uint32_t >>

Re: [Xen-devel] [PATCH V5] vm_event: Allow subscribing to write events for specific MSR-s

2016-04-26 Thread Tamas K Lengyel
> > @@ -55,6 +61,9 @@ void vm_event_cleanup_domain(struct domain *d) > > v->arch.vm_event = NULL; > > } > > > > +xfree(d->arch.monitor_msr_bitmap); > > +d->arch.monitor_msr_bitmap = NULL; > > + > > d->arch.mem_access_emulate_each_rep = 0; > > memset(&d->arch.monitor,

Re: [Xen-devel] [PATCH v2 for-4.7 10/14] libxl: add the printf-like attributes to a couple of functions

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:19PM +0200, Roger Pau Monne wrote: [...] > @@ -1995,9 +1995,10 @@ _hidden libxl__json_object > *libxl__json_parse(libxl__gc *gc_opt, const char *s); > _hidden int libxl__device_model_version_running(libxl__gc *gc, uint32_t > domid); >/* Return the system-wide de

Re: [Xen-devel] [PATCH v2 for-4.7 09/14] xl: fix usage of libxl_get_scheduler

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:18PM +0200, Roger Pau Monne wrote: > It returns an int, not a libxl_scheduler. > > Signed-off-by: Roger Pau Monné > --- > Cc: Ian Jackson > Cc: Wei Liu At the risk of nitpicking too much, I think we should enforce more consistent style, especially in xl which is a

Re: [Xen-devel] [PATCH v2 for-4.7 09/14] xl: fix usage of libxl_get_scheduler

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > It returns an int, not a libxl_scheduler. > > Signed-off-by: Roger Pau Monné Reviewed-by: Doug Goldstein -- Doug Goldstein signature.asc Description: OpenPGP digital signature ___ Xen-devel mailing lis

Re: [Xen-devel] [PATCH v9 12/27] xsplice: Implement support for applying/reverting/replacing patches.

2016-04-26 Thread Jan Beulich
>>> On 25.04.16 at 17:34, wrote: > +static int check_special_sections(const struct xsplice_elf *elf) > +{ > +unsigned int i; > +static const char *const names[] = { ELF_XSPLICE_FUNC }; > +bool_t count[ARRAY_SIZE(names)] = { 0 }; > + > +for ( i = 0; i < ARRAY_SIZE(names); i++ ) > +

Re: [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > It should be an enum, not an unsigned. > > Signed-off-by: Roger Pau Monné Reviewed-by: Doug Goldstein I had to drop the shutdown_reason >= 0 check a few lines below in my version of this patch. But maybe its not necessary after all. -- Doug Goldst

Re: [Xen-devel] [PATCH v2 for-4.7 08/14] libxl: fix shutdown_reason type in list_domains

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:17PM +0200, Roger Pau Monne wrote: > It should be an enum, not an unsigned. > > Signed-off-by: Roger Pau Monné Acked-by: Wei Liu > --- > Cc: Ian Jackson > Cc: Wei Liu > --- > tools/libxl/xl_cmdimpl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >

Re: [Xen-devel] [PATCH v2 for-4.7 06/14] xen/tools: fix substitution of __align8__ uint64_t inside of headers

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > The current seedery doesn't work with BSD sed, so remove the try to match > int64_t also (since there's none at the moment). Also, apply the same > treatment to all arch headers, currently this is only done to x86_64 > headers. > > Signed-off-by: Roger

Re: [Xen-devel] [PATCH v2 for-4.7 05/14] tools/headers: prevent adding two __align8__ to uint64_t in ARM headers

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > Due to the fact that on ARM headers types are substituted to uint64_t and > then uint64_t is also substituted to contain the aligment, this would lead > to some types containing two __align8__ directives. Fix this by first > expanding Xen specific types t

Re: [Xen-devel] [PATCH v2 for-4.7 06/14] xen/tools: fix substitution of __align8__ uint64_t inside of headers

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:15PM +0200, Roger Pau Monne wrote: > The current seedery doesn't work with BSD sed, so remove the try to match > int64_t also (since there's none at the moment). Also, apply the same > treatment to all arch headers, currently this is only done to x86_64 > headers. > >

Re: [Xen-devel] [PATCH v2 for-4.7 07/14] libxc: fix uninitialized variable

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:16PM +0200, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > --- > Cc: Ian Jackson > Cc: Wei Liu > --- > tools/libxc/xc_dom_bzimageloader.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/libxc/xc_dom_bzimageloader.c > b/to

Re: [Xen-devel] [PATCH v2 for-4.7 05/14] tools/headers: prevent adding two __align8__ to uint64_t in ARM headers

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > Due to the fact that on ARM headers types are substituted to uint64_t and > then uint64_t is also substituted to contain the aligment, this would lead > to some types containing two __align8__ directives. Fix this by first > expanding Xen specific types

Re: [Xen-devel] [PATCH v2 for-4.7 05/14] tools/headers: prevent adding two __align8__ to uint64_t in ARM headers

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:14PM +0200, Roger Pau Monne wrote: > Due to the fact that on ARM headers types are substituted to uint64_t and > then uint64_t is also substituted to contain the aligment, this would lead > to some types containing two __align8__ directives. Fix this by first > expandi

Re: [Xen-devel] [PATCH v2 for-4.7 03/14] build: pass HOST{CC/CXX} value down to Kconfig

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > Acked-by: Ian Jackson Reviewed-by: Doug Goldstein -- Doug Goldstein signature.asc Description: OpenPGP digital signature ___ Xen-devel mailing list Xen-devel@lists.xen.

Re: [Xen-devel] [PATCH v2 for-4.7 11/14] libxl: add explicit casts from yajl_gen_status to yajl_status

2016-04-26 Thread Wei Liu
On Tue, Apr 26, 2016 at 04:52:20PM +0200, Roger Pau Monne wrote: > Or else clang complains with: > > implicit conversion from enumeration type 'yajl_gen_status' to different > enumeration type 'yajl_status' [-Werror,-Wenum-conversion] > > Signed-off-by: Roger Pau Monné Acked-by: Wei Liu > ---

Re: [Xen-devel] [PATCH v2 for-4.7 06/14] xen/tools: fix substitution of __align8__ uint64_t inside of headers

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > The current seedery doesn't work with BSD sed, so remove the try to match > int64_t also (since there's none at the moment). Also, apply the same > treatment to all arch headers, currently this is only done to x86_64 > headers. > > Signed-off-by: Roger Pa

Re: [Xen-devel] [PATCH v2 for-4.7 02/14] build: set HOSTCXX based on clang value for Kconfig xconfig target

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > The xconfig Kconfig target requires a C++ compiler because it uses Qt. > > Signed-off-by: Roger Pau Monné Reviewed-by: Doug Goldstein -- Doug Goldstein signature.asc Description: OpenPGP digital signature

Re: [Xen-devel] [PATCH v2 for-4.7 01/14] build: make HOSTCC conditional on the value of clang

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > Previously HOSTCC was always hardcoded to gcc > > Signed-off-by: Roger Pau Monné Reviewed-by: Doug Goldstein -- Doug Goldstein signature.asc Description: OpenPGP digital signature ___ Xen-devel mailing

Re: [Xen-devel] [PATCH v2 for-4.7 03/14] build: pass HOST{CC/CXX} value down to Kconfig

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > Acked-by: Ian Jackson Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 for-4.7 04/14] build: remove Kconfig forced gcc selection

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 for-4.7 14/14] tools/python: corrently use LDFLAGS and CFLAGS

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > It is incorrect to add the LDFLAGS to the CFLAGS, and some compilers will > error out if linker flags are passed when creating object files. Fix this by > properly passing CFLAGS and LDFLAGS, instead of putting everything in > CFLAGS. > > Signed-off-by:

Re: [Xen-devel] [PATCH v2 for-4.7 01/14] build: make HOSTCC conditional on the value of clang

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > Previously HOSTCC was always hardcoded to gcc > > Signed-off-by: Roger Pau Monné Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v2 for-4.7 02/14] build: set HOSTCXX based on clang value for Kconfig xconfig target

2016-04-26 Thread Andrew Cooper
On 26/04/16 15:52, Roger Pau Monne wrote: > The xconfig Kconfig target requires a C++ compiler because it uses Qt. > > Signed-off-by: Roger Pau Monné Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xe

[Xen-devel] How to install Xen on Fedora 23?

2016-04-26 Thread Jason Long
Hello. I installed Fedora 23 x64 on my PC and id below command for installing Xen : # cd /etc/yum.repos.d/ # wget http://fedorapeople.org/groups/virt/...t-preview.repo # yum update # yum -y install xen xen-hypervisor xen-libs xen-runtime # systemctl enable xend.service # systemctl enable xendomain

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

2016-04-26 Thread Jan Beulich
>>> On 26.04.16 at 16:37, wrote: >> -Original Message- > [snip] >> >> Apr 26 10:56:02.583602 (XEN)[] >> >> hvm_mmio_internal+0x37/0x61 >> > >> > Ah. Crap. I forgot about this path >> >> So did I. And my testing didn't catch it because I have a post-4.7 >> patch in place avoiding

Re: [Xen-devel] [PATCH v2 for-4.7 04/14] build: remove Kconfig forced gcc selection

2016-04-26 Thread Doug Goldstein
On 4/26/16 9:52 AM, Roger Pau Monne wrote: > Signed-off-by: Roger Pau Monné > --- > Cc: Doug Goldstein > --- > xen/tools/kconfig/Makefile.kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xen/tools/kconfig/Makefile.kconfig > b/xen/tools/kconfig/Makefile.kconf

Re: [Xen-devel] [PATCH] x86/vMSI-X: write snoops should ignore hvm_mmio_internal() requests

2016-04-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 April 2016 15:39 > To: xen-devel > Cc: Andrew Cooper; Paul Durrant; Wei Liu > Subject: [PATCH] x86/vMSI-X: write snoops should ignore > hvm_mmio_internal() requests > > Those aren't actual I/O requests (and henc

[Xen-devel] [PATCH v2 for-4.7 11/14] libxl: add explicit casts from yajl_gen_status to yajl_status

2016-04-26 Thread Roger Pau Monne
Or else clang complains with: implicit conversion from enumeration type 'yajl_gen_status' to different enumeration type 'yajl_status' [-Werror,-Wenum-conversion] Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Wei Liu --- tools/libxl/libxl_json.c | 29 ++--- 1 f

[Xen-devel] [PATCH v2 for-4.7 09/14] xl: fix usage of libxl_get_scheduler

2016-04-26 Thread Roger Pau Monne
It returns an int, not a libxl_scheduler. Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Wei Liu --- tools/libxl/xl_cmdimpl.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 8ff54e1..2ebca0a 100644

[Xen-devel] [PATCH v2 for-4.7 05/14] tools/headers: prevent adding two __align8__ to uint64_t in ARM headers

2016-04-26 Thread Roger Pau Monne
Due to the fact that on ARM headers types are substituted to uint64_t and then uint64_t is also substituted to contain the aligment, this would lead to some types containing two __align8__ directives. Fix this by first expanding Xen specific types to uint64_t only, and then replacing all the uint64

[Xen-devel] [PATCH v2 for-4.7 07/14] libxc: fix uninitialized variable

2016-04-26 Thread Roger Pau Monne
Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Wei Liu --- tools/libxc/xc_dom_bzimageloader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxc/xc_dom_bzimageloader.c b/tools/libxc/xc_dom_bzimageloader.c index 7fde42a..0a4041c 100644 --- a/tools/libxc/xc_d

[Xen-devel] [PATCH v2 for-4.7 04/14] build: remove Kconfig forced gcc selection

2016-04-26 Thread Roger Pau Monne
Signed-off-by: Roger Pau Monné --- Cc: Doug Goldstein --- xen/tools/kconfig/Makefile.kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/xen/tools/kconfig/Makefile.kconfig b/xen/tools/kconfig/Makefile.kconfig index 815f306..dbd8912 100644 --- a/xen/tools/kconfig/Make

[Xen-devel] [PATCH v2 for-4.7 14/14] tools/python: corrently use LDFLAGS and CFLAGS

2016-04-26 Thread Roger Pau Monne
It is incorrect to add the LDFLAGS to the CFLAGS, and some compilers will error out if linker flags are passed when creating object files. Fix this by properly passing CFLAGS and LDFLAGS, instead of putting everything in CFLAGS. Signed-off-by: Roger Pau Monné --- Cc: Ian Jackson Cc: Wei Liu ---

  1   2   3   >