Re: [Xen-devel] [PATCH 09/17] SVM: use generic instruction decoding

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 19:56, wrote: > On 08/09/16 14:14, Jan Beulich wrote: >> int __get_instruction_length_from_list(struct vcpu *v, >> const enum instruction_index *list, unsigned int list_count) >> { >> struct vmcb_struct *vmcb = v->arch.hvm_svm.vmcb; >> -unsigned int i, j, ins

Re: [Xen-devel] [PATCH v4 1/2] xen: replace complicated tlbflush check with an inline function

2016-09-14 Thread Dongli Zhang
> > I rewrite the inline function in xen/include/xen/mm.h to: > > > > +#include > > + > > +static inline bool accumulate_tlbflush(bool need_tlbflush, > > + const struct page_info *page, > > + uint32_t tlbflush_timestamp)

Re: [Xen-devel] [PATCH 08/17] x86emul: generate and make use of canonical opcode representation

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 19:30, wrote: >> @@ -435,6 +438,51 @@ struct x86_emulate_ctxt >> void *data; >> }; >> >> +/* >> + * This encodes the opcode extension in a "natural" way: > > I am not sure what you mean by natural way here. All you seem to mean > is that you are encoding instructions w

Re: [Xen-devel] [PATCH] x86: fold code in load_segments()

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 19:12, wrote: > On 14/09/16 16:24, Jan Beulich wrote: >> --- a/xen/arch/x86/domain.c >> +++ b/xen/arch/x86/domain.c >> @@ -1745,22 +1745,22 @@ static void load_segments(struct vcpu *n >> (unsigned long *)pv->kernel_sp; >> unsigned long cs_and_mask, rflags;

Re: [Xen-devel] [PATCH 06/17] x86emul: add EVEX decoding

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 19:05, wrote: > On 08/09/16 14:12, Jan Beulich wrote: >> --- a/xen/arch/x86/x86_emulate/x86_emulate.c >> +++ b/xen/arch/x86/x86_emulate/x86_emulate.c >> @@ -336,6 +336,27 @@ union vex { >> ptr[1] = rex | REX_PREFIX; \ >> } while (0) >> >> +union evex { >> +uint8_

Re: [Xen-devel] [PATCH v4 1/2] xen: replace complicated tlbflush check with an inline function

2016-09-14 Thread Jan Beulich
>>> On 15.09.16 at 00:36, wrote: >> I don't think you should pass this into the function ... >> >> > +{ >> > +return page->u.free.need_tlbflush && >> > + page->tlbflush_timestamp <= tlbflush_current_time && >> >> ... and use tlbflush_current_time() here instead. > > I rewrite the

Re: [Xen-devel] [PATCH 2/2] x86/vm_event: Allow returning i-cache for emulation

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 18:20, wrote: > On Wed, Sep 14, 2016 at 9:55 AM, Jan Beulich wrote: > On 13.09.16 at 20:12, wrote: >>> When emulating instructions the emulator maintains a small i-cache fetched >>> from the guest memory. This patch extends the vm_event interface to allow >>> returning thi

Re: [Xen-devel] [PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Kyle Huey
On Wed, Sep 14, 2016 at 6:17 PM, Andy Lutomirski wrote: > On Wed, Sep 14, 2016 at 3:03 PM, Kyle Huey wrote: >> On Wed, Sep 14, 2016 at 2:35 PM, Dave Hansen >> wrote: >>> On 09/14/2016 02:01 PM, Kyle Huey wrote: > >>> Is any of this useful to optimize away at compile-time? We have config >>> opt

Re: [Xen-devel] [PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Kyle Huey
On Wed, Sep 14, 2016 at 2:35 PM, Dave Hansen wrote: > On 09/14/2016 02:01 PM, Kyle Huey wrote: >> Xen advertises the underlying support for CPUID faulting but not does pass >> through writes to the relevant MSR, nor does it virtualize it, so it does >> not actually work. For now mask off the relev

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

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

Re: [Xen-devel] [PATCH v5 00/16] Xen ARM DomU ACPI support

2016-09-14 Thread Boris Ostrovsky
- julien.gr...@arm.com wrote: > Hi Stefano, > > On 14/09/2016 21:48, Stefano Stabellini wrote: > > On Wed, 14 Sep 2016, Julien Grall wrote: > >> On 14/09/2016 02:06, Stefano Stabellini wrote: > >>> On Wed, 14 Sep 2016, Shannon Zhao wrote: > On 2016/9/13 23:17, Julien Grall wrote: >

Re: [Xen-devel] [PATCH v2 2/2] xen_platform: SUSE xenlinux unplug for emulated PCI

2016-09-14 Thread Stefano Stabellini
On Fri, 2 Sep 2016, Olaf Hering wrote: > Implement SUSE specific unplug protocol for emulated PCI devices > in PVonHVM guests. Its a simple 'outl(1, (ioaddr + 4));'. > This protocol was implemented and used since Xen 3.0.4. > It is used in all SUSE/SLES/openSUSE releases up to SLES11SP3 and > openS

[Xen-devel] [qemu-mainline baseline-only test] 67712: tolerable FAIL

2016-09-14 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67712 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67712/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-qemuu-nested-intel 16 debian-hvm-insta

Re: [Xen-devel] [PATCH v2 1/2] xen_platform: unplug also SCSI disks

2016-09-14 Thread Stefano Stabellini
On Fri, 2 Sep 2016, Olaf Hering wrote: > Using 'vdev=sd[a-o]' will create an emulated LSI controller, which can > be used by the emulated BIOS to boot from disk. If the HVM domU has also > PV driver the disk may appear twice in the guest. To avoid this an > unplug of the emulated hardware is needed

[Xen-devel] [xen-4.6-testing test] 100957: tolerable FAIL - PUSHED

2016-09-14 Thread osstest service owner
flight 100957 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100957/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 15 guest-start/debian.repeatfail like 100895 test-amd64-amd64-xl-qemu

[Xen-devel] [ovmf baseline-only test] 67713: all pass

2016-09-14 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67713 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67713/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 062f9fd2cfaad09f6dd0e302094bc030827eb706 baseline v

Re: [Xen-devel] [PATCH v7 0/2] qemu-qdisk: Implementation of grant copy operation.

2016-09-14 Thread Stefano Stabellini
Hi Wei, I am happy to queue up this for QEMU, but I'll wait for the first patch to be committed to Xen before sending a pull request. Is that OK? Cheers, Stefano On Wed, 14 Sep 2016, Paulina Szubarczyk wrote: > Hi, > > It is a proposition for implementation of grant copy operation in qemu-qdis

Re: [Xen-devel] [PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Andy Lutomirski
On Wed, Sep 14, 2016 at 3:03 PM, Kyle Huey wrote: > On Wed, Sep 14, 2016 at 2:35 PM, Dave Hansen > wrote: >> On 09/14/2016 02:01 PM, Kyle Huey wrote: >> Is any of this useful to optimize away at compile-time? We have config >> options for when we're running as a guest, and this seems like a fea

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Peng Fan
Hi Edgar, On Wed, Sep 14, 2016 at 04:16:58PM +0200, Edgar E. Iglesias wrote: >On Wed, Sep 14, 2016 at 08:40:09PM +0800, Peng Fan wrote: >> On Wed, Sep 14, 2016 at 01:34:10PM +0100, Julien Grall wrote: >> > >> > >> >On 14/09/16 13:18, Peng Fan wrote: >> >>Hello Julien, >> >> >> >>On Wed, Sep 14, 201

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

2016-09-14 Thread osstest service owner
flight 100954 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100954/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemut-win7-amd64 16 guest-stop fail like 100933 test-amd64-i386-xl-qemuu-wi

[Xen-devel] [DOC v6] PV Calls protocol design

2016-09-14 Thread Stefano Stabellini
Hi all, This is the design document of the PV Calls protocol. You can find prototypes of the Linux frontend and backend drivers here: git://git.kernel.org/pub/scm/linux/kernel/git/sstabellini/xen.git pvcalls-6 To use them, make sure to enable CONFIG_XEN_PVCALLS in your kernel config and add "pvc

Re: [Xen-devel] [PATCH v4 1/2] xen: replace complicated tlbflush check with an inline function

2016-09-14 Thread Dongli Zhang
> I don't think you should pass this into the function ... > > > +{ > > +return page->u.free.need_tlbflush && > > + page->tlbflush_timestamp <= tlbflush_current_time && > > ... and use tlbflush_current_time() here instead. I rewrite the inline function in xen/include/xen/mm.h to:

[Xen-devel] [PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Kyle Huey
Xen advertises the underlying support for CPUID faulting but not does pass through writes to the relevant MSR, nor does it virtualize it, so it does not actually work. For now mask off the relevant bit on MSR_PLATFORM_INFO. Signed-off-by: Kyle Huey --- arch/x86/include/asm/cpufeatures.h | 1 +

Re: [Xen-devel] [PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Dave Hansen
On 09/14/2016 02:01 PM, Kyle Huey wrote: > Xen advertises the underlying support for CPUID faulting but not does pass > through writes to the relevant MSR, nor does it virtualize it, so it does > not actually work. For now mask off the relevant bit on MSR_PLATFORM_INFO. That needs to make it into

[Xen-devel] [RESEND][PATCH v2 2/3] x86 Test and expose CPUID faulting capabilities in /proc/cpuinfo

2016-09-14 Thread Kyle Huey
Xen advertises the underlying support for CPUID faulting but not does pass through writes to the relevant MSR, nor does it virtualize it, so it does not actually work. For now mask off the relevant bit on MSR_PLATFORM_INFO. Signed-off-by: Kyle Huey --- arch/x86/include/asm/cpufeatures.h | 1 +

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

2016-09-14 Thread osstest service owner
flight 100955 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100955/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 062f9fd2cfaad09f6dd0e302094bc030827eb706 baseline version: ovmf 18b00c8c8773e62b1e2c4

Re: [Xen-devel] [PATCH v5 00/16] Xen ARM DomU ACPI support

2016-09-14 Thread Julien Grall
Hi Stefano, On 14/09/2016 21:48, Stefano Stabellini wrote: On Wed, 14 Sep 2016, Julien Grall wrote: On 14/09/2016 02:06, Stefano Stabellini wrote: On Wed, 14 Sep 2016, Shannon Zhao wrote: On 2016/9/13 23:17, Julien Grall wrote: On 13/09/16 14:06, Shannon Zhao wrote: Hi Julien, Hello Sha

Re: [Xen-devel] [PATCH v5 00/16] Xen ARM DomU ACPI support

2016-09-14 Thread Stefano Stabellini
On Wed, 14 Sep 2016, Julien Grall wrote: > Hello, > > On 14/09/2016 02:06, Stefano Stabellini wrote: > > On Wed, 14 Sep 2016, Shannon Zhao wrote: > > > On 2016/9/13 23:17, Julien Grall wrote: > > > > > > > > > > > > On 13/09/16 14:06, Shannon Zhao wrote: > > > > > Hi Julien, > > > > > > > > He

Re: [Xen-devel] [PATCH] prctl, x86 Add PR_[GET|SET]_CPUID for controlling the CPUID instruction.

2016-09-14 Thread Andrew Cooper
On 14/09/2016 20:36, Andy Lutomirski wrote: > On Wed, Sep 14, 2016 at 12:28 PM, Andrew Cooper > wrote: >> On 14/09/2016 20:23, Boris Ostrovsky wrote: >>> On 09/14/2016 02:52 PM, Andy Lutomirski wrote: On Tue, Sep 13, 2016 at 11:13 PM, Kyle Huey wrote: > On Mon, Sep 12, 2016 at 9:56 AM, A

[Xen-devel] [PATCH v7 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-09-14 Thread Paulina Szubarczyk
Copy data operated on during request from/to local buffers to/from the grant references. Before grant copy operation local buffers must be allocated what is done by calling ioreq_init_copy_buffers. For the 'read' operation, first, the qemu device invokes the read operation on local buffers and on

[Xen-devel] [PATCH v7 1/2] libs/gnttab: introduce grant copy interface

2016-09-14 Thread Paulina Szubarczyk
In a linux part an ioctl(gntdev, IOCTL_GNTDEV_GRANT_COPY, ..) system call is invoked. In mini-os the operation is yet not implemented. For the OSs that does not implement gnttab the call of the grant copy operation causes abort. Signed-off-by: Paulina Szubarczyk Reviewed-by: David Vrabel --- to

[Xen-devel] [PATCH v7 0/2] qemu-qdisk: Implementation of grant copy operation.

2016-09-14 Thread Paulina Szubarczyk
Hi, It is a proposition for implementation of grant copy operation in qemu-qdisk and interface in libxc/libs. Changes since v6: qemu-qdisk: -removed blank lines -renamed functions free_buffers -> ioreq_free_copy_buffers, ioreq_copy -> ioreq_grant_copy -merged the if(ioreq_copy) with the condit

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

2016-09-14 Thread osstest service owner
flight 100951 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100951/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-libvirt-qcow2 9 debian-di-install fail in 100941 pass in 100951 test-armhf-armhf-xl-rtds

Re: [Xen-devel] [PATCH 09/17] SVM: use generic instruction decoding

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:14, Jan Beulich wrote: > @@ -89,141 +54,96 @@ static unsigned long svm_nextrip_insn_le > return vmcb->nextrip - vmcb->rip; > } > > -/* First byte: Length. Following bytes: Opcode bytes. */ > -#define MAKE_INSTR(nm, ...) static const u8 OPCODE_##nm[] = { __VA_ARGS__ } > -MAKE_

[Xen-devel] [PATCH v4 1/5] x86/time: refactor init_platform_time()

2016-09-14 Thread Joao Martins
And accomodate platform time source initialization in try_platform_time(). This is a preparatory patch for deferring TSC clocksource initialization to the stage where all CPUS are up (verify_tsc_reliability init call). Signed-off-by: Joao Martins Reviewed-by: Konrad Rzeszutek Wilk Reviewed-by: J

[Xen-devel] [PATCH v4 5/5] x86/time: extend "tsc" param with "stable:socket"

2016-09-14 Thread Joao Martins
Extend the "tsc" boot parameter is to further relax TSC restrictions and allow it to be used on machines that guarantee reliable TSC across sockets. This is up to board manufacturers and there's no way for the OS to probe this property, therefore user needs to explicitly set this option. Also make

[Xen-devel] [PATCH v4 3/5] x86/time: refactor read_platform_stime()

2016-09-14 Thread Joao Martins
To allow the caller to fetch the last read from the clocksource which was used to calculate system_time. This is a prerequisite for a subsequent patch that will use this last read. Signed-off-by: Joao Martins --- Cc: Jan Beulich Cc: Andrew Cooper Changes since v3: - Add mention of this being

[Xen-devel] [PATCH v4 2/5] x86/time: implement tsc as clocksource

2016-09-14 Thread Joao Martins
Recent x86/time changes improved a lot of the monotonicity in xen timekeeping, making it much harder to observe time going backwards. Although platform timer can't be expected to be perfectly in sync with TSC and so get_s_time won't be guaranteed to always return monotonically increasing values acr

[Xen-devel] [PATCH v4 4/5] x86/time: implement PVCLOCK_TSC_STABLE_BIT

2016-09-14 Thread Joao Martins
This patch proposes relying on host TSC synchronization and passthrough to the guest, when running on a TSC-safe platform. On time_calibration we retrieve the platform time in ns and the counter read by the clocksource that was used to compute system time. We introduce a new rendezous function whic

[Xen-devel] [PATCH v4 0/5] x86/time: PVCLOCK_TSC_STABLE_BIT support

2016-09-14 Thread Joao Martins
Hey, This is v4 on the pvclock TSC series addressing comments from previous version, see individual patches for complete changelog. This series is divided as follows: R * Patch 1: Small refactor around init_platform_time to reuse initialization code when switching to TSC

Re: [Xen-devel] [PATCH 08/17] x86emul: generate and make use of canonical opcode representation

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:14, Jan Beulich wrote: "of a canonical opcode representation". You appear to be inventing your own here, but it isn't the only canonical form you could represent x86 opcodes with. > --- a/xen/arch/x86/x86_emulate/x86_emulate.h > +++ b/xen/arch/x86/x86_emulate/x86_emulate.h > @@ -4

Re: [Xen-devel] [PATCH 07/17] x86emul: move x86_execute() common epilogue code

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:13, Jan Beulich wrote: > Only code movement, no functional change. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] x86: fold code in load_segments()

2016-09-14 Thread Andrew Cooper
On 14/09/16 16:24, Jan Beulich wrote: > No need to have the same logic twice. > > Signed-off-by: Jan Beulich > > --- a/xen/arch/x86/domain.c > +++ b/xen/arch/x86/domain.c > @@ -1745,22 +1745,22 @@ static void load_segments(struct vcpu *n > (unsigned long *)pv->kernel_sp; > un

Re: [Xen-devel] [PATCH 06/17] x86emul: add EVEX decoding

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:12, Jan Beulich wrote: > This way we can at least size (and e.g. skip) them if needed, and we > also won't raise the wrong fault due to not having read all relevant > bytes. > > Signed-off-by: Jan Beulich > --- > TBD: I'm kind of undecided whether to right away propagate evex.R into

Re: [Xen-devel] BUG_ON() vs ASSERT()

2016-09-14 Thread Mihai Donțu
On Tue, 13 Sep 2016 19:25:54 +0100 Andrew Cooper wrote: > On 13/09/16 14:46, Mihai Donțu wrote: > > On Tue, 13 Sep 2016 09:10:32 -0400 Konrad Rzeszutek Wilk wrote: > >> On Mon, Sep 12, 2016 at 09:23:41AM -0600, Jan Beulich wrote: > >>> All, > >>> > >>> in > >>> https://lists.xenproject.org/arch

Re: [Xen-devel] [PATCH v4 2/2] xen: move TLB-flush filtering out into populate_physmap during vm creation

2016-09-14 Thread Dario Faggioli
On Mon, 2016-09-12 at 16:16 +0800, Dongli Zhang wrote: > This patch implemented parts of TODO left in commit id > a902c12ee45fc9389eb8fe54eeddaf267a555c58. > We usually put both the (not necessarily full) hash and the subject line of the commit in here. > Signed-off-by: Dongli Zhang >  > diff --

Re: [Xen-devel] Outreachy Winter Internship

2016-09-14 Thread George Dunlap
On 11/09/16 11:52, Kavya Sharma wrote: > Hello Sir,I am Kavya Sharma, an aspiring Outreachy intern.It would be my > privilege to be an intern with xenproject.org this winter.I have read > about Xen Hypervisor Userspace Tools and I am interested in your project > 'golang bindings for libxl'. > > Si

Re: [Xen-devel] [PATCH 05/17] x86emul: add XOP decoding

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 18:11, wrote: > On 08/09/16 14:11, Jan Beulich wrote: >> @@ -1580,6 +1586,9 @@ struct x86_emulate_state { >> ext_0f = vex_0f, >> ext_0f38 = vex_0f38, >> ext_0f3a = vex_0f3a, >> +ext_8f08 = 8, >> +ext_8f09, >> +ext_8f0a, > > W

Re: [Xen-devel] [PATCH 2/2] x86/vm_event: Allow returning i-cache for emulation

2016-09-14 Thread Tamas K Lengyel
On Wed, Sep 14, 2016 at 9:55 AM, Jan Beulich wrote: On 13.09.16 at 20:12, wrote: >> When emulating instructions the emulator maintains a small i-cache fetched >> from the guest memory. This patch extends the vm_event interface to allow >> returning this i-cache via the vm_event response inst

Re: [Xen-devel] [Help] Trigger Watchdog when adding an IPI in vcpu_wake

2016-09-14 Thread Dario Faggioli
On Wed, 2016-09-14 at 18:44 +0800, Wei Yang wrote: > On Tue, Sep 13, 2016 at 01:30:17PM +0200, Dario Faggioli wrote: > >  > > Do you mind sharing just a bit more, such as: > >  - number of pcpus > >  - number of vcpus of the various VMs > 160 pcpus > 16 vcpus in VM and 8 VMs > So, 16x8=128, which

Re: [Xen-devel] [PATCH v4 1/2] xen: replace complicated tlbflush check with an inline function

2016-09-14 Thread Jan Beulich
>>> On 12.09.16 at 10:16, wrote: > This patch cleaned up the code by replacing complicated tlbflush check with > an inline function. We should use this inline function to avoid the long > and complicated to read tlbflush check when implementing TODOs left in > commit a902c12ee45fc9389eb8fe54eeddaf

Re: [Xen-devel] [PATCH 05/17] x86emul: add XOP decoding

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:11, Jan Beulich wrote: > This way we can at least size (and e.g. skip) them if needed, and we > also won't raise the wrong fault due to not having read all relevant > bytes. > > Signed-off-by: Jan Beulich > > --- a/xen/arch/x86/x86_emulate/x86_emulate.c > +++ b/xen/arch/x86/x86_emul

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

2016-09-14 Thread osstest service owner
flight 100948 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/100948/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops4 host-build-prep fail in 100936 REGR. vs. 100895 Tests which are

Re: [Xen-devel] [PATCH 2/2] x86/vm_event: Allow returning i-cache for emulation

2016-09-14 Thread Jan Beulich
>>> On 13.09.16 at 20:12, wrote: > When emulating instructions the emulator maintains a small i-cache fetched > from the guest memory. This patch extends the vm_event interface to allow > returning this i-cache via the vm_event response instead. I guess I'm sightly confused: Isn't the purpose to

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Peter Milesson
On 2016-09-14 16:17, George Dunlap wrote: On Wed, Sep 14, 2016 at 3:04 PM, Peter Milesson wrote: On 2016-09-14 14:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: [snip] I've been using Xen for 3 years, starting out with 4.2, and I've bee

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Peter Milesson
On 2016-09-14 15:30, George Dunlap wrote: On 14/09/16 14:23, Andrew Cooper wrote: On 14/09/16 13:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: [snip] I've been using Xen for 3 years, starting out with 4.2, and I've been upgrading regularl

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Peter Milesson
On 2016-09-14 16:49, Jan Beulich wrote: On 14.09.16 at 16:04, wrote: On 2016-09-14 14:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: Can you think of anything that's changed between 4.6 and 4.7 that would cause the performance problems he's describing when both a video card and a usb

Re: [Xen-devel] [PATCH v3 19/38] arm/p2m: Add HVMOP_altp2m_switch_p2m

2016-09-14 Thread Sergej Proskurin
Hi Julien, On 09/14/2016 12:57 PM, Julien Grall wrote: > > > On 13/09/16 14:00, Sergej Proskurin wrote: >> Hi Julien, > > Hello Sergej, > >> >> On 09/12/2016 10:47 AM, Julien Grall wrote: >>> Hello Sergej, >>> >>> On 16/08/2016 23:16, Sergej Proskurin wrote: Signed-off-by: Sergej Proskurin

[Xen-devel] [PATCH] x86: fold code in load_segments()

2016-09-14 Thread Jan Beulich
No need to have the same logic twice. Signed-off-by: Jan Beulich --- a/xen/arch/x86/domain.c +++ b/xen/arch/x86/domain.c @@ -1745,22 +1745,22 @@ static void load_segments(struct vcpu *n (unsigned long *)pv->kernel_sp; unsigned long cs_and_mask, rflags; +/* Fold up

Re: [Xen-devel] [PATCH v3 00/19] Make ACPI builder available to components other than hvmloader

2016-09-14 Thread Boris Ostrovsky
On 09/07/2016 02:59 PM, Boris Ostrovsky wrote: > The goal here is to build ACPI tables for PVHv2/HVMlite guests while reusing > existing > hvmloader's ACPI builder code. The builder is provided as a library in > tools/libacpi. > > This is version 3 of the series, see individual patches for change

Re: [Xen-devel] [PATCH v3 29/38] arm/p2m: Add HVMOP_altp2m_set_mem_access

2016-09-14 Thread Sergej Proskurin
Hi Julien, On 09/12/2016 02:08 PM, Julien Grall wrote: > Hi Sergej, > > On 16/08/16 23:17, Sergej Proskurin wrote: >> The HVMOP_altp2m_set_mem_access allows to set gfn permissions of >> (currently one page at a time) of a specific altp2m view. In case the >> view does not hold the requested gfn e

Re: [Xen-devel] [PATCH 1/2] vm_event: Sanitize vm_event response handling

2016-09-14 Thread Tamas K Lengyel
On Wed, Sep 14, 2016 at 9:15 AM, Julien Grall wrote: > > > On 14/09/16 16:14, Tamas K Lengyel wrote: >> >> On Wed, Sep 14, 2016 at 3:33 AM, Julien Grall >> wrote: >>> >>> Hello Tamas, >>> >>> On 13/09/16 19:12, Tamas K Lengyel wrote: diff --git a/xen/include/asm-arm/p2m.h b/xen/inc

Re: [Xen-devel] [PATCH 1/2] vm_event: Sanitize vm_event response handling

2016-09-14 Thread Julien Grall
On 14/09/16 16:14, Tamas K Lengyel wrote: On Wed, Sep 14, 2016 at 3:33 AM, Julien Grall wrote: Hello Tamas, On 13/09/16 19:12, Tamas K Lengyel wrote: diff --git a/xen/include/asm-arm/p2m.h b/xen/include/asm-arm/p2m.h index 53c4d78..5e9bc54 100644 --- a/xen/include/asm-arm/p2m.h +++ b/xen/i

Re: [Xen-devel] [PATCH 1/2] vm_event: Sanitize vm_event response handling

2016-09-14 Thread Tamas K Lengyel
On Wed, Sep 14, 2016 at 3:33 AM, Julien Grall wrote: > Hello Tamas, > > On 13/09/16 19:12, Tamas K Lengyel wrote: >> >> diff --git a/xen/include/asm-arm/p2m.h b/xen/include/asm-arm/p2m.h >> index 53c4d78..5e9bc54 100644 >> --- a/xen/include/asm-arm/p2m.h >> +++ b/xen/include/asm-arm/p2m.h >> @@ -1

Re: [Xen-devel] [PATCH 1/2] vm_event: Sanitize vm_event response handling

2016-09-14 Thread Tamas K Lengyel
On Wed, Sep 14, 2016 at 7:38 AM, George Dunlap wrote: > On 13/09/16 19:12, Tamas K Lengyel wrote: >> Setting response flags in vm_event are only ever safe if the vCPUs are >> paused. >> To reflect this we move all checks within the if block that already checks >> whether this is the case. Checks

Re: [Xen-devel] [PATCH 04/17] x86emul: complete decoding of two-byte instructions

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 16:22, wrote: > On 08/09/16 14:10, Jan Beulich wrote: >> This way we can at least size (and e.g. skip) them if needed, and we >> also won't raise the wrong fault due to not having read all relevant >> bytes. > > What faults are you referring to? #UD vs #GP from hitting the %cs

Re: [Xen-devel] [PATCH v3 4/6] Pause/Unpause the domain before/after assigning PI hooks

2016-09-14 Thread Dario Faggioli
On Wed, 2016-09-14 at 02:23 +, Wu, Feng wrote: > Then I tried to implement the function like the following: > > /* This function is called when pcidevs_lock is held */ > void vmx_pi_hooks_assign(struct domain *d) > { > if ( !iommu_intpost || !has_hvm_container_domain(d) ) > return;

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 16:04, wrote: > On 2016-09-14 14:50, Jan Beulich wrote: > On 14.09.16 at 13:34, wrote: >>> Can you think of anything that's changed between 4.6 and 4.7 that >>> would cause the performance problems he's describing when both a video >>> card and a usb controller are passed t

Re: [Xen-devel] [PATCH 04/17] x86emul: complete decoding of two-byte instructions

2016-09-14 Thread Andrew Cooper
On 08/09/16 14:10, Jan Beulich wrote: > This way we can at least size (and e.g. skip) them if needed, and we > also won't raise the wrong fault due to not having read all relevant > bytes. What faults are you referring to? #UD vs #GP from hitting the %cs limit? > > This at once adds correct rais

[Xen-devel] [ovmf baseline-only test] 67711: all pass

2016-09-14 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67711 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67711/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 18b00c8c8773e62b1e2c4a36f490996325570b67 baseline v

[Xen-devel] per-domain logging

2016-09-14 Thread Cedric Bosdonnat
Hi all, I wanted to get libvirt's libxl driver have per-domain logs like all other drivers. After looking at the libxl and XenToolLogger it seems I'll need to add the feature in either libxl XenToolLogger. Would anyone already have an idea how best to add API to allow this? Thanks for your hel

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Peter Milesson
On 2016-09-14 15:23, Andrew Cooper wrote: On 14/09/16 13:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: [snip] I've been using Xen for 3 years, starting out with 4.2, and I've been upgrading regularly, and installed 4.7.0 today. I'm using

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread George Dunlap
On Wed, Sep 14, 2016 at 3:04 PM, Peter Milesson wrote: > > > On 2016-09-14 14:50, Jan Beulich wrote: > > On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: >>> >>> [snip] > > I've been using Xen for 3 years, starting out with 4.2, and I've been >>>

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Edgar E. Iglesias
On Wed, Sep 14, 2016 at 08:40:09PM +0800, Peng Fan wrote: > On Wed, Sep 14, 2016 at 01:34:10PM +0100, Julien Grall wrote: > > > > > >On 14/09/16 13:18, Peng Fan wrote: > >>Hello Julien, > >> > >>On Wed, Sep 14, 2016 at 01:06:01PM +0100, Julien Grall wrote: > >>> > >>> > >>>On 14/09/16 13:03, Peng F

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Peter Milesson
On 2016-09-14 14:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: [snip] I've been using Xen for 3 years, starting out with 4.2, and I've been upgrading regularly, and installed 4.7.0 today. I'm using Windows 10, 64-bit with PCI pass through

Re: [Xen-devel] [Help] Trigger Watchdog when adding an IPI in vcpu_wake

2016-09-14 Thread Wei Yang
On Tue, Sep 13, 2016 at 01:30:17PM +0200, Dario Faggioli wrote: >[using xendevel correct address] > >On Tue, 2016-09-13 at 16:54 +0800, Wei Yang wrote: >> On Fri, 2016-09-09 at 17:41 +0800, Wei Yang wrote: >> >  >> > I'm not surprised by that. Yet, I'd be interested in hearing more >> > about this

Re: [Xen-devel] [PATCH 03/24] xen: credit1: return the 'time remaining to the limit' as next timeslice.

2016-09-14 Thread Dario Faggioli
On Wed, 2016-09-14 at 10:34 +0100, George Dunlap wrote: > On 12/09/16 18:00, Dario Faggioli wrote: > >  > > I also agree on the fact that most of the times ratelimit_us and > > MIN_TIMER will be close enough (like in the example above) that it > > won't probably matter much... but if someone set ra

[Xen-devel] [GIT PULL] xen: regression fix for 4.8-rc6

2016-09-14 Thread David Vrabel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Linus, Please git pull the following tag: git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-4.8b-rc6-tag xen: regression fix for 4.8-rc6 - - Fix SMP boot in arm guests. Thanks. David arch/arm/xen/enlighten.c | 7 +++ 1

Re: [Xen-devel] [PATCH Altp2m cleanup v4 2/4] altp2m cleanup work

2016-09-14 Thread George Dunlap
On 13/09/16 18:38, Lai, Paul wrote: > On Mon, Sep 12, 2016 at 11:47:35AM +0100, George Dunlap wrote: >> On 08/09/16 17:45, Lai, Paul C wrote: >>> [Paul2] in-line >> >> If you're going to engage in discussions on xen-devel it would really be >> worth your time to find a mail setup that allows you to

Re: [Xen-devel] [PATCH 1/2] vm_event: Sanitize vm_event response handling

2016-09-14 Thread George Dunlap
On 13/09/16 19:12, Tamas K Lengyel wrote: > Setting response flags in vm_event are only ever safe if the vCPUs are paused. > To reflect this we move all checks within the if block that already checks > whether this is the case. Checks that are only supported on one architecture > we relocate the bi

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread George Dunlap
On 14/09/16 14:23, Andrew Cooper wrote: > On 14/09/16 13:50, Jan Beulich wrote: > On 14.09.16 at 13:34, wrote: On 2016-09-09 09:51, Peter Milesson wrote: >>> [snip] > I've been using Xen for 3 years, starting out with 4.2, and I've been > upgrading regularly, and installed 4.7.0

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Andrew Cooper
On 14/09/16 13:50, Jan Beulich wrote: On 14.09.16 at 13:34, wrote: >>> On 2016-09-09 09:51, Peter Milesson wrote: >> [snip] I've been using Xen for 3 years, starting out with 4.2, and I've been upgrading regularly, and installed 4.7.0 today. I'm using Windows 10, 64-bit w

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

2016-09-14 Thread osstest service owner
flight 100944 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100944/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-debianhvm-amd64 15 guest-localmigrate/x10 fail REGR. vs. 100933 Regress

Re: [Xen-devel] [PATCH] x86: add a user configurable Kconfig option for the VGA

2016-09-14 Thread Julien Grall
Hello Derek, On 13/09/16 20:40, Derek Straka wrote: diff --git a/xen/drivers/video/Kconfig b/xen/drivers/video/Kconfig index 0ffbbd9..0f208fe 100644 --- a/xen/drivers/video/Kconfig +++ b/xen/drivers/video/Kconfig @@ -3,7 +3,8 @@ config VIDEO bool config VGA - bool + bool "V

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread Jan Beulich
>>> On 14.09.16 at 13:34, wrote: >> On 2016-09-09 09:51, Peter Milesson wrote: > [snip] >>> I've been using Xen for 3 years, starting out with 4.2, and I've been >>> upgrading regularly, and installed 4.7.0 today. >>> >>> I'm using Windows 10, 64-bit with PCI pass through in a VM with >>> PV-driv

Re: [Xen-devel] [PATCH v3 12/19] libacpi: Build DSDT for PVH guests

2016-09-14 Thread Boris Ostrovsky
On 09/14/2016 12:13 AM, Shannon Zhao wrote: > > On 2016/9/8 2:59, Boris Ostrovsky wrote: >> PVH guests require DSDT with only ACPI INFO (Xen-specific) and Processor >> objects. We separate ASL's ACPI INFO definition into dsdt_acpi_info.asl so >> that it can be included in ASLs for both HVM and PVH2

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Peng Fan
On Wed, Sep 14, 2016 at 01:34:10PM +0100, Julien Grall wrote: > > >On 14/09/16 13:18, Peng Fan wrote: >>Hello Julien, >> >>On Wed, Sep 14, 2016 at 01:06:01PM +0100, Julien Grall wrote: >>> >>> >>>On 14/09/16 13:03, Peng Fan wrote: Hello Julien, >>> >>>Hello Peng, >>> On Wed, Sep 14, 2016 at

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Julien Grall
On 14/09/16 13:18, Peng Fan wrote: Hello Julien, On Wed, Sep 14, 2016 at 01:06:01PM +0100, Julien Grall wrote: On 14/09/16 13:03, Peng Fan wrote: Hello Julien, Hello Peng, On Wed, Sep 14, 2016 at 11:47:10AM +0100, Julien Grall wrote: Hello, On 14/09/16 08:41, Peng Fan wrote: On Wed,

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

2016-09-14 Thread osstest service owner
flight 100947 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100947/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 18b00c8c8773e62b1e2c4a36f490996325570b67 baseline version: ovmf d947fbed72226011961e5

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Peng Fan
Hello Julien, On Wed, Sep 14, 2016 at 01:06:01PM +0100, Julien Grall wrote: > > >On 14/09/16 13:03, Peng Fan wrote: >>Hello Julien, > >Hello Peng, > >>On Wed, Sep 14, 2016 at 11:47:10AM +0100, Julien Grall wrote: >>>Hello, >>> >>>On 14/09/16 08:41, Peng Fan wrote: On Wed, Sep 14, 2016 at 08:23

[Xen-devel] [libvirt test] 100945: tolerable FAIL - PUSHED

2016-09-14 Thread osstest service owner
flight 100945 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/100945/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-raw 11 migrate-support-checkfail never pass test-armhf-armhf-libvirt-raw 13 guest-sav

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Julien Grall
On 14/09/16 13:03, Peng Fan wrote: Hello Julien, Hello Peng, On Wed, Sep 14, 2016 at 11:47:10AM +0100, Julien Grall wrote: Hello, On 14/09/16 08:41, Peng Fan wrote: On Wed, Sep 14, 2016 at 08:23:24AM +0100, Julien Grall wrote: diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domai

Re: [Xen-devel] [PATCH V1] xen/arm: domain_build: introduce dom0_lowmem bootargs

2016-09-14 Thread Peng Fan
Hello Julien, On Wed, Sep 14, 2016 at 11:47:10AM +0100, Julien Grall wrote: >Hello, > >On 14/09/16 08:41, Peng Fan wrote: >>On Wed, Sep 14, 2016 at 08:23:24AM +0100, Julien Grall wrote: >>diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c >>index 35ab08d..cc71e6f 100644 >>--- a/

Re: [Xen-devel] [Xen-users] Xen 4.7.0 graphics pass through problems (Win10, AMD Radeon)

2016-09-14 Thread George Dunlap
> On 2016-09-09 09:51, Peter Milesson wrote: [snip] >> I've been using Xen for 3 years, starting out with 4.2, and I've been >> upgrading regularly, and installed 4.7.0 today. >> >> I'm using Windows 10, 64-bit with PCI pass through in a VM with >> PV-drivers, using a AMD Radeon HD6450 card (AMD gr

Re: [Xen-devel] [Xen-users] pv-grub in xen 4.7, ok with kernel 3.x, but fails with 4.x

2016-09-14 Thread Wei Liu
On Wed, Sep 14, 2016 at 12:13:00PM +0100, George Dunlap wrote: > On Sat, Sep 10, 2016 at 9:04 PM, Sarah Newman wrote: > > On 09/10/2016 07:58 AM, Mark Pryor wrote: > >> Hello, > >> > >> In a xen 4.7 source build, pv-grub works with domU kernel 3.x.x, but not > >> with 4.x.x. So, the recent fc24,

Re: [Xen-devel] [PATCH v3 33/38] arm/p2m: Add altp2m paging mechanism

2016-09-14 Thread Julien Grall
On 14/09/16 08:53, Sergej Proskurin wrote: Hi Julien [...] static void do_trap_instr_abort_guest(struct cpu_user_regs *regs, const union hsr hsr) { @@ -2445,6 +2465,14 @@ static void do_trap_instr_abort_guest(struct cpu_user_regs *regs, brea

Re: [Xen-devel] [Xen-users] pv-grub in xen 4.7, ok with kernel 3.x, but fails with 4.x

2016-09-14 Thread George Dunlap
On Sat, Sep 10, 2016 at 9:04 PM, Sarah Newman wrote: > On 09/10/2016 07:58 AM, Mark Pryor wrote: >> Hello, >> >> In a xen 4.7 source build, pv-grub works with domU kernel 3.x.x, but not >> with 4.x.x. So, the recent fc24, xenial, and debian sid/stretch can't use >> pv-grub. > > While this should

Re: [Xen-devel] [PATCH v3 16/38] arm/p2m: Add HVMOP_altp2m_set_domain_state

2016-09-14 Thread Julien Grall
Hello Sergej, On 16/08/16 23:16, Sergej Proskurin wrote: diff --git a/xen/arch/arm/hvm.c b/xen/arch/arm/hvm.c index 180154e..c69da36 100644 --- a/xen/arch/arm/hvm.c +++ b/xen/arch/arm/hvm.c @@ -83,8 +83,40 @@ static int do_altp2m_op(XEN_GUEST_HANDLE_PARAM(void) arg) break; case HV

[Xen-devel] [xen-unstable-coverity test] 100952: all pass - PUSHED

2016-09-14 Thread osstest service owner
flight 100952 xen-unstable-coverity real [real] http://logs.test-lab.xenproject.org/osstest/logs/100952/ Perfect :-) All tests in this flight passed as required version targeted for testing: xen 773522000cc17f6f4323a4d97423790138ea98f2 baseline version: xen a3fe

Re: [Xen-devel] [PATCH v3 19/38] arm/p2m: Add HVMOP_altp2m_switch_p2m

2016-09-14 Thread Julien Grall
On 13/09/16 14:00, Sergej Proskurin wrote: Hi Julien, Hello Sergej, On 09/12/2016 10:47 AM, Julien Grall wrote: Hello Sergej, On 16/08/2016 23:16, Sergej Proskurin wrote: Signed-off-by: Sergej Proskurin --- Cc: Stefano Stabellini Cc: Julien Grall --- v3: Extended the function "altp2m

  1   2   >