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

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

[Xen-devel] [linux-linus test] 106520: regressions - FAIL

2017-03-07 Thread osstest service owner
flight 106520 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/106520/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 11 guest-start fail REGR. vs. 59254

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Julien Grall
Hi Stefano, On 03/07/2017 07:54 PM, Stefano Stabellini wrote: On Tue, 7 Mar 2017, Julien Grall wrote: Given that upstream GRUB doesn't yet support booting Xen on ARM (without any additional patches), I think that the ability to completely change the command line from the EFI shell would be

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

2017-03-07 Thread osstest service owner
flight 106510 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/106510/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-libvirt 13 saverestore-support-checkfail like 106434 test-armhf-armhf-libvirt-xsm 13

Re: [Xen-devel] [PATCH 13/29] drivers, media: convert vb2_vmarea_handler.refcount from atomic_t to refcount_t

2017-03-07 Thread Sakari Ailus
Hi Elena, On Mon, Mar 06, 2017 at 04:21:00PM +0200, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free >

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

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 06:52, wrote: > flight 106504 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/106504/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > [...] >

Re: [Xen-devel] [PATCH 01/10] x86: assembly, ENTRY for fn, GLOBAL for data

2017-03-07 Thread Ingo Molnar
* h...@zytor.com wrote: > On March 1, 2017 2:27:54 AM PST, Ingo Molnar wrote: > >> > So how about using macro names that actually show the purpose, instead > >> > of > >> > importing all the crappy, historic, essentially randomly chosen debug > >> > symbol

[Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Zhongze Liu
Added 3 new config entries in common/Kconfig: CMDLINE_BOOL, CMDLINE and CMDLINE_OVERRIDE These 3 entries enable an embedded command line to be compiled in the hypervisor. If CMDLINE_BOOL is set to y, both arm and x86 startup routines will append the bootloader command line to the built-in

Re: [Xen-devel] [PATCH] x86: drop unneeded __packed attributes

2017-03-07 Thread Jan Beulich
>>> On 06.03.17 at 18:15, wrote: > On Mon, Mar 06, 2017 at 09:57:23AM -0700, Jan Beulich wrote: >> >>> On 06.03.17 at 17:42, wrote: >> > There where a couple of unneeded packed attributes in several x86-specific >> > structures, that are obviously

Re: [Xen-devel] [PATCH 11/29] drivers, media: convert cx88_core.refcount from atomic_t to refcount_t

2017-03-07 Thread Sakari Ailus
On Mon, Mar 06, 2017 at 04:20:58PM +0200, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. >

Re: [Xen-devel] [PATCH 12/29] drivers, media: convert s2255_dev.num_channels from atomic_t to refcount_t

2017-03-07 Thread Sakari Ailus
Hi Elena, On Mon, Mar 06, 2017 at 04:20:59PM +0200, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free >

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

2017-03-07 Thread osstest service owner
flight 106508 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/106508/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-arndale 15 guest-start/debian.repeat fail REGR. vs. 106491 Regressions

Re: [Xen-devel] [PATCH v5 01/17] x86emul: support most memory accessing MMX/SSE{, 2, 3} insns

2017-03-07 Thread Jan Beulich
>>> On 03.03.17 at 15:58, wrote: > @@ -6183,6 +6579,76 @@ x86_emulate( > goto cannot_emulate; > } > > +if ( state->simd_size ) > +{ > +#ifdef __XEN__ > +uint8_t *buf = stub.ptr; > +#else > +uint8_t *buf = get_stub(stub); > +#endif > + > +

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

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

Re: [Xen-devel] [PATCH 11/29] drivers, media: convert cx88_core.refcount from atomic_t to refcount_t

2017-03-07 Thread Sergei Shtylyov
On 3/7/2017 10:52 AM, Reshetova, Elena wrote: refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena

[Xen-devel] [PATCH v4 2/5] xen: rename xen_modified_memory() to xen_hvm_modified_memory()

2017-03-07 Thread Paul Durrant
This patch is a purely cosmetic change that avoids a name collision in a subsequent patch. Signed-off-by: Paul Durrant Reviewed-by: Anthony Perard --- Cc: Paolo Bonzini Cc: Stefano Stabellini ---

[Xen-devel] [PATCH v4 3/5] xen: create wrappers for all other uses of xc_hvm_XXX() functions

2017-03-07 Thread Paul Durrant
This patch creates inline wrapper functions in xen_common.h for all open coded calls to xc_hvm_XXX() functions outside of xen_common.h so that use of xen_xc can be made implicit. This again is in preparation for the move to using libxendevicemodel. Signed-off-by: Paul Durrant

[Xen-devel] [PATCH v4 5/5] xen: use libxendevicemodel when available

2017-03-07 Thread Paul Durrant
This patch modifies the wrapper functions in xen_common.h to use the new xendevicemodel interface if it is available along with compatibility code to use the old libxenctrl interface if it is not. Signed-off-by: Paul Durrant Reviewed-by: Anthony Perard

[Xen-devel] [PATCH v4 1/5] xen: make use of xen_xc implicit in xen_common.h inlines

2017-03-07 Thread Paul Durrant
Doing this will make the transition to using the new libxendevicemodel interface less intrusive on the callers of these functions, since using the new library will require a change of handle. NOTE: The patch also moves the 'externs' for xen_xc and xen_fmem from xen_backend.h to

[Xen-devel] [PATCH v4 4/5] configure: detect presence of libxendevicemodel

2017-03-07 Thread Paul Durrant
This patch adds code in configure to set CONFIG_XEN_CTRL_INTERFACE_VERSION to a new value of 490 if libxendevicemodel is present in the build environment. Signed-off-by: Paul Durrant Reviewed-by: Anthony Perard --- Cc: Stefano Stabellini

[Xen-devel] [PATCH v4 0/5] xen: use new xendevicemodel library

2017-03-07 Thread Paul Durrant
My recent patches to Xen [1] introduced a new library to support running device models for HVM guests. This series ports QEMU onto the new library if it is available in the build environment. [1] Patches starting with

Re: [Xen-devel] [PATCH v2 5/5] xen: use libxendevicemodel when available

2017-03-07 Thread Paul Durrant
> -Original Message- > From: Stefano Stabellini [mailto:sstabell...@kernel.org] > Sent: 06 March 2017 19:14 > To: Paul Durrant > Cc: 'Stefano Stabellini' ; Anthony Perard > ; xen-de...@lists.xenproject.org; qemu-

Re: [Xen-devel] [PATCH 1/7] x86/hvm: Correctly identify implicit supervisor accesses

2017-03-07 Thread Andrew Cooper
On 27/02/17 14:03, Andrew Cooper wrote: > All actions which refer to the active ldt/gdt/idt or task register > (e.g. loading a new segment selector) are known as implicit supervisor > accesses, even when the access originates from user code. It turns out that this has a bugfix in it which I

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 09:34, wrote: As an initial remark: Am I right in guessing that you manually prefix [Xen-devel] to your message subject? Please don't do so - receiving patches without that prefix serves as an indication to the receiver that (s)he is on the Cc list. >

Re: [Xen-devel] [PATCH v5 01/17] x86emul: MMX/SSEn support

2017-03-07 Thread Andrew Cooper
On 03/03/17 14:56, Jan Beulich wrote: > This aims at covering most MMX/SSEn/AVX instructions in the 0x0f-escape > space with memory operands. Not covered here are irregular moves, > converts, and {,U}COMIS{S,D} (modifying EFLAGS). > > Note that the distinction between simd_*_fp isn't strictly

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Zhongze Liu
Thanks for your time reviewing my code. 2017-03-07 17:36 GMT+08:00 Jan Beulich : On 07.03.17 at 09:34, wrote: > > As an initial remark: Am I right in guessing that you manually prefix > [Xen-devel] to your message subject? Please don't do so -

Re: [Xen-devel] [PATCH 2/7] x86/shadow: Try to correctly identify implicit supervisor accesses

2017-03-07 Thread George Dunlap
On 27/02/17 14:03, Andrew Cooper wrote: > Signed-off-by: Andrew Cooper > --- > CC: Jan Beulich > CC: Tim Deegan > CC: George Dunlap > --- > xen/arch/x86/mm/shadow/multi.c | 60 >

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

2017-03-07 Thread Chao Gao
On Tue, Mar 07, 2017 at 02:16:50AM -0700, Jan Beulich wrote: On 07.03.17 at 06:52, wrote: >> flight 106504 xen-unstable real [real] >> http://logs.test-lab.xenproject.org/osstest/logs/106504/ >> >> Regressions :-( >> >> Tests which did not succeed and are

[Xen-devel] [distros-debian-snapshot test] 68641: tolerable trouble: blocked/broken/fail/pass

2017-03-07 Thread Platform Team regression test user
flight 68641 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68641/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-i386-daily-netboot-pvgrub 10 guest-start fail blocked in 68619

Re: [Xen-devel] [PATCH 1/7] x86/hvm: Correctly identify implicit supervisor accesses

2017-03-07 Thread George Dunlap
On 27/02/17 14:03, Andrew Cooper wrote: > All actions which refer to the active ldt/gdt/idt or task register > (e.g. loading a new segment selector) are known as implicit supervisor > accesses, even when the access originates from user code. > > The distinction is necessary in the pagewalk when

[Xen-devel] [linux-linus test] 106509: regressions - FAIL

2017-03-07 Thread osstest service owner
flight 106509 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/106509/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 11 guest-start fail REGR. vs. 59254

Re: [Xen-devel] [PATCH 2/7] x86/shadow: Try to correctly identify implicit supervisor accesses

2017-03-07 Thread Andrew Cooper
On 07/03/17 11:26, George Dunlap wrote: > On 27/02/17 14:03, Andrew Cooper wrote: >> Signed-off-by: Andrew Cooper >> --- >> CC: Jan Beulich >> CC: Tim Deegan >> CC: George Dunlap >> --- >>

Re: [Xen-devel] [PATCH v5 04/17] x86emul: support {, V}{, U}COMIS{S, D}

2017-03-07 Thread Andrew Cooper
On 03/03/17 14:59, Jan Beulich wrote: > 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 v5 12/17] x86emul: support SSE4.1 insns

2017-03-07 Thread Andrew Cooper
On 03/03/17 15:04, Jan Beulich wrote: > ... and their AVX equivalents. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v5 016/17] x86emul: support AESNI insns

2017-03-07 Thread Andrew Cooper
On 03/03/17 15:07, Jan Beulich wrote: > ... and their AVX equivalents. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

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

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 05:24, wrote: > On Tue, Mar 07, 2017 at 02:16:50AM -0700, Jan Beulich wrote: > On 07.03.17 at 06:52, wrote: >>> flight 106504 xen-unstable real [real] >>> http://logs.test-lab.xenproject.org/osstest/logs/106504/ >>> >>>

Re: [Xen-devel] [PATCH 7/7] x86/pagewalk: Re-implement the pagetable walker

2017-03-07 Thread George Dunlap
On 27/02/17 14:03, Andrew Cooper wrote: > The existing pagetable walker has complicated return semantics, which squeeze > multiple pieces of information into single integer. This would be fine if the > information didn't overlap, but it does. > > Specifically, _PAGE_INVALID_BITS for 3-level

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

2017-03-07 Thread osstest service owner
flight 106524 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/106524/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 5 xen-buildfail REGR. vs. 105963 Tests which did not

Re: [Xen-devel] [PATCH 12/29] drivers, media: convert s2255_dev.num_channels from atomic_t to refcount_t

2017-03-07 Thread Reshetova, Elena
> Hi Elena, > > On Mon, Mar 06, 2017 at 04:20:59PM +0200, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to

Re: [Xen-devel] [RFC PATCH] mm, hotplug: get rid of auto_online_blocks

2017-03-07 Thread Igor Mammedov
On Mon, 6 Mar 2017 15:54:17 +0100 Michal Hocko wrote: > On Fri 03-03-17 18:34:22, Igor Mammedov wrote: > > On Fri, 3 Mar 2017 09:27:23 +0100 > > Michal Hocko wrote: > > > > > On Thu 02-03-17 18:03:15, Igor Mammedov wrote: > > > > On Thu, 2 Mar 2017

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 12:21, wrote: > 2017-03-07 17:36 GMT+08:00 Jan Beulich : > On 07.03.17 at 09:34, wrote: >>> +#endif /* CONFIG_CMDLINE_OVERRIDE */ >>> +#endif /* CONFIG_CMDLINE_BOOL */ >> >> I think this #ifdef-ary can be

Re: [Xen-devel] [PATCH v5 02/17] x86emul: support MMX/SSE{, 2, 3} moves

2017-03-07 Thread Andrew Cooper
On 03/03/17 14:58, Jan Beulich wrote: > Previously supported insns are being converted to the new model, and > several new ones are being added. > > To keep the stub handling reasonably simple, integrate SET_SSE_PREFIX() > into copy_REX_VEX(), at once switching the stubs to use an empty REX >

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Julien Grall
Hi Jan, On 03/07/2017 02:18 PM, Jan Beulich wrote: On 07.03.17 at 14:48, wrote: On 03/07/2017 12:52 PM, Jan Beulich wrote: On 07.03.17 at 12:21, wrote: 2017-03-07 17:36 GMT+08:00 Jan Beulich : On 07.03.17 at 09:34,

Re: [Xen-devel] [PATCHv4 18/33] x86/xen: convert __xen_pgd_walk() and xen_cleanmfnmap() to support p4d

2017-03-07 Thread Kirill A. Shutemov
On Mon, Mar 06, 2017 at 03:48:24PM -0500, Boris Ostrovsky wrote: > > > +static int xen_p4d_walk(struct mm_struct *mm, p4d_t *p4d, > > + int (*func)(struct mm_struct *mm, struct page *, enum pt_level), > > + bool last, unsigned long limit) > > +{ > > + int i, nr, flush = 0; >

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Julien Grall
Hi Jan, On 03/07/2017 12:52 PM, Jan Beulich wrote: On 07.03.17 at 12:21, wrote: 2017-03-07 17:36 GMT+08:00 Jan Beulich : On 07.03.17 at 09:34, wrote: +static inline char* __init extract_dom0_options(char *cmdline) +{ +char

Re: [Xen-devel] [PATCH v5 10/17] x86emul: add tables for 0f38 and 0f3a extension space

2017-03-07 Thread Andrew Cooper
On 03/03/17 15:03, Jan Beulich wrote: > Convert the few existing opcodes so far supported. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 14:48, wrote: > On 03/07/2017 12:52 PM, Jan Beulich wrote: > On 07.03.17 at 12:21, wrote: >>> 2017-03-07 17:36 GMT+08:00 Jan Beulich : >>> On 07.03.17 at 09:34, wrote: > +static

Re: [Xen-devel] [PATCH v5 15/17] x86emul: support PCLMULQDQ

2017-03-07 Thread Andrew Cooper
On 03/03/17 15:07, Jan Beulich wrote: > ... and its AVX equivalent. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org

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

2017-03-07 Thread osstest service owner
flight 106513 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/106513/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stopfail like 106482

[Xen-devel] [OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH

2017-03-07 Thread Andrew Cooper
./xtf-runner wants to distinguish between a clean and unclean exits of the test. OSSTest doesn't care, so map unclean exit to failure. Signed-off-by: Andrew Cooper --- CC: Ian Jackson CC: Wei Liu --- ts-xtf-run | 1 +

Re: [Xen-devel] [PATCH 6/7] xen/9pfs: receive responses

2017-03-07 Thread Boris Ostrovsky
On 03/06/2017 03:01 PM, Stefano Stabellini wrote: > Upon receiving a notification from the backend, schedule the > p9_xen_response work_struct. p9_xen_response checks if any responses are > available, if so, it reads them one by one, calling p9_client_cb to send > them up to the 9p layer

Re: [Xen-devel] [PATCH 13/29] drivers, media: convert vb2_vmarea_handler.refcount from atomic_t to refcount_t

2017-03-07 Thread Reshetova, Elena
> Hi Elena, > > On Mon, Mar 06, 2017 at 04:21:00PM +0200, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to

Re: [Xen-devel] [PATCH 1/7] x86/hvm: Correctly identify implicit supervisor accesses

2017-03-07 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: 27 February 2017 14:03 > To: Xen-devel > Cc: Andrew Cooper ; Jan Beulich > ; Paul Durrant ; George > Dunlap

Re: [Xen-devel] [PATCH 4/7] xen/9pfs: connect to the backend

2017-03-07 Thread Boris Ostrovsky
> > +static int xen_9pfs_front_free(struct xen_9pfs_front_priv *priv) > +{ > + int i, j; > + > + list_del(>list); > + > + for (i = 0; i < priv->num_rings; i++) { > + if (priv->rings[i].intf == NULL) > + break; Are we guaranteed that all subsequent

Re: [Xen-devel] [OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH

2017-03-07 Thread Ian Jackson
Andrew Cooper writes ("[OSSTEST PATCH] ts-xtf-run: Understand ./xtf-runner returning CRASH"): > ./xtf-runner wants to distinguish between a clean and unclean exits of the > test. OSSTest doesn't care, so map unclean exit to failure. Acked-by: Ian Jackson

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 15:41, wrote: > Hi Jan, > > On 03/07/2017 02:18 PM, Jan Beulich wrote: > On 07.03.17 at 14:48, wrote: >>> On 03/07/2017 12:52 PM, Jan Beulich wrote: >>> On 07.03.17 at 12:21, wrote: > 2017-03-07

Re: [Xen-devel] [PATCH 5/7] xen/9pfs: send requests to the backend

2017-03-07 Thread Boris Ostrovsky
On 03/06/2017 03:01 PM, Stefano Stabellini wrote: > Implement struct p9_trans_module create and close functions by looking > at the available Xen 9pfs frontend-backend connections. We don't expect > many frontend-backend connections, thus walking a list is OK. > > Send requests to the backend by

[Xen-devel] [PATCH] vlapic/viridian: abort existing APIC assist if any vector is pending in ISR

2017-03-07 Thread Paul Durrant
The vlapic code already aborts an APIC assist if an interrupt is deferred because a higher priority interrupt has already been delivered (and hence its vector is pending in the ISR). However, it is also necessary to abort an APIC assist in the case where a higher priority is about to be delivered

Re: [Xen-devel] [PATCH v2] x86/emul: Correct the decoding of mov to/from cr/dr

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 17:23, wrote: > The mov to/from cr/dr behave as if they were encoded with Mod = 3. When > encoded with Mod != 3, no displacement or SIB bytes are fetched. > > Add a test with a deliberately malformed ModRM byte. (Also add the >

[Xen-devel] [PATCH 3/3] x86: drop underscore prefixed 32-bit register names

2017-03-07 Thread Jan Beulich
Now that all underscore prefixed instances have been replaced, this concludes the register renaming project. Signed-off-by: Jan Beulich --- a/xen/include/public/arch-x86/xen-x86_32.h +++ b/xen/include/public/arch-x86/xen-x86_32.h @@ -114,7 +114,7 @@ #elif defined(__XEN__) ||

Re: [Xen-devel] [PATCH] vlapic/viridian: abort existing APIC assist if any vector is pending in ISR

2017-03-07 Thread Jan Beulich
>>> On 07.03.17 at 15:58, wrote: > The vlapic code already aborts an APIC assist if an interrupt is deferred > because a higher priority interrupt has already been delivered (and hence > its vector is pending in the ISR). > > However, it is also necessary to abort an

Re: [Xen-devel] [PATCH 0/7] Xen transport for 9pfs frontend driver

2017-03-07 Thread Roger Pau Monné
On Mon, Mar 06, 2017 at 12:00:41PM -0800, Stefano Stabellini wrote: > Hi all, > > This patch series implements a new transport for 9pfs, aimed at Xen > systems. > > The transport is based on a traditional Xen frontend and backend drivers > pair. This patch series implements the frontend, which

Re: [Xen-devel] [PATCH 1/3] x86emul: switch away from temporary 32-bit register names

2017-03-07 Thread Andrew Cooper
On 07/03/17 16:42, Jan Beulich wrote: > @@ -3266,36 +3266,36 @@ x86_emulate( > case 0x27: /* daa */ > case 0x2f: /* das */ { > uint8_t al = _regs.al; > -unsigned int eflags = _regs._eflags; > +unsigned int eflags = _regs.eflags; > > -_regs._eflags &=

[Xen-devel] [PATCH v2] x86/emul: Correct the decoding of mov to/from cr/dr

2017-03-07 Thread Andrew Cooper
The mov to/from cr/dr behave as if they were encoded with Mod = 3. When encoded with Mod != 3, no displacement or SIB bytes are fetched. Add a test with a deliberately malformed ModRM byte. (Also add the automatically-generated simd.h to .gitignore.) Signed-off-by: Andrew Cooper

[Xen-devel] [PATCH 0/3] x86: finish 32-bit register renaming

2017-03-07 Thread Jan Beulich
This is concluding the exercise, perhaps with the exception of seeing whether some of the auxiliary stuff added to the public header can now be removed again (which may require tweaking some of the scripts parsing the headers). 1: x86emul: switch away from temporary 32-bit register names 2:

[Xen-devel] [PATCH 1/3] x86emul: switch away from temporary 32-bit register names

2017-03-07 Thread Jan Beulich
Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -811,7 +811,7 @@ do{ asm volatile ( unsigned long tmp; \ invoke_stub(_PRE_EFLAGS("[efl]", "[msk]",

[Xen-devel] [PATCH 2/3] x86/hypercall: switch away from temporary 32-bit register names

2017-03-07 Thread Jan Beulich
Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hypercall.c +++ b/xen/arch/x86/hvm/hypercall.c @@ -146,7 +146,7 @@ int hvm_hypercall(struct cpu_user_regs * struct vcpu *curr = current; struct domain *currd = curr->domain; int mode = hvm_guest_x86_mode(curr); -

Re: [Xen-devel] [PATCH 2/3] x86/hypercall: switch away from temporary 32-bit register names

2017-03-07 Thread Andrew Cooper
On 07/03/17 16:42, Jan Beulich wrote: > 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 3/3] x86: drop underscore prefixed 32-bit register names

2017-03-07 Thread Andrew Cooper
On 07/03/17 16:43, Jan Beulich wrote: > Now that all underscore prefixed instances have been replaced, this > concludes the register renaming project. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper

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

2017-03-07 Thread osstest service owner
flight 106515 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/106515/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64 16 guest-stop fail REGR. vs. 106491 Regressions

[Xen-devel] [RFC] linkage: new macros for functions and data

2017-03-07 Thread Jiri Slaby
SYM_LOCAL_ALIAS_START -- use where there are two local names for one code SYM_ALIAS_START -- use where there are two global names for one code SYM_LOCAL_FUNC_START -- use for local functions SYM_FUNCTION_START -- use for global functions SYM_WEAK_FUNC_START -- use for weak functions

Re: [Xen-devel] [PATCH 1/7] xen: import new ring macros in ring.h

2017-03-07 Thread Julien Grall
Hi Stefano, On 03/06/2017 08:01 PM, Stefano Stabellini wrote: Sync the ring.h file with upstream Xen, to introduce the new ring macros. They will be used by the Xen transport for 9pfs. Signed-off-by: Stefano Stabellini CC: konrad.w...@oracle.com CC:

Re: [Xen-devel] [PATCH 4/7] xen/9pfs: connect to the backend

2017-03-07 Thread Julien Grall
Hi Stefano, On 03/06/2017 08:01 PM, Stefano Stabellini wrote: +static int xen_9pfs_front_alloc_dataring(struct xenbus_device *dev, + struct xen_9pfs_dataring *ring) +{ + int i; + int ret = -ENOMEM; + + init_waitqueue_head(>wq); + spin_lock_init(>lock); +

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Julien Grall
Hi Jan, On 03/07/2017 02:52 PM, Jan Beulich wrote: On 07.03.17 at 15:41, wrote: Hi Jan, On 03/07/2017 02:18 PM, Jan Beulich wrote: On 07.03.17 at 14:48, wrote: On 03/07/2017 12:52 PM, Jan Beulich wrote: On 07.03.17 at 12:21,

Re: [Xen-devel] [PATCH 06/28] ARM: GICv3 ITS: introduce ITS command handling

2017-03-07 Thread Andre Przywara
Hi Julien, On 06/02/17 19:16, Julien Grall wrote: > Hi Andre, > > On 30/01/17 18:31, Andre Przywara wrote: >> To be able to easily send commands to the ITS, create the respective >> wrapper functions, which take care of the ring buffer. >> The first two commands we implement provide methods to

Re: [Xen-devel] [PATCH] Fix typo in function install

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Géza Gémes wrote: > Signed-off-by: Géza Gémes Reviewed-by: Stefano Stabellini I'll commmit > --- > lib/commands.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/commands.sh b/lib/commands.sh >

[Xen-devel] [xen-4.7-testing test] 106528: regressions - FAIL

2017-03-07 Thread osstest service owner
flight 106528 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/106528/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64 9 debian-hvm-install fail REGR. vs. 106251

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

2017-03-07 Thread Xuquan (Quan Xu)
On March 07, 2017 12:24 PM, Chao Gao wrote: >On Tue, Mar 07, 2017 at 02:16:50AM -0700, Jan Beulich wrote: > On 07.03.17 at 06:52, wrote: >>> flight 106504 xen-unstable real [real] >>> http://logs.test-lab.xenproject.org/osstest/logs/106504/ >>> >>> Regressions

Re: [Xen-devel] [PATCH v16 4/9] x86: add multiboot2 protocol support for EFI platforms

2017-03-07 Thread Konrad Rzeszutek Wilk
On Tue, Mar 07, 2017 at 12:39:04AM +0100, Daniel Kiper wrote: > On Wed, Feb 22, 2017 at 09:04:17AM -0800, Doug Goldstein wrote: > > [...] > > > I'm currently at ELC and then on vacation so I don't have access to any > > of the machines currently myself. However the machine I most use to test > >

Re: [Xen-devel] [PATCH 4/7] xen/9pfs: connect to the backend

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Boris Ostrovsky wrote: > > +static int xen_9pfs_front_free(struct xen_9pfs_front_priv *priv) > > +{ > > + int i, j; > > + > > + list_del(>list); > > + > > + for (i = 0; i < priv->num_rings; i++) { > > + if (priv->rings[i].intf == NULL) > > +

Re: [Xen-devel] [PATCH 1/7] xen: import new ring macros in ring.h

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Julien Grall wrote: > Hi Stefano, > > On 03/06/2017 08:01 PM, Stefano Stabellini wrote: > > Sync the ring.h file with upstream Xen, to introduce the new ring macros. > > They will be used by the Xen transport for 9pfs. > > > > Signed-off-by: Stefano Stabellini

Re: [Xen-devel] [PATCH 4/7] xen/9pfs: connect to the backend

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Julien Grall wrote: > Hi Stefano, > > On 03/06/2017 08:01 PM, Stefano Stabellini wrote: > > +static int xen_9pfs_front_alloc_dataring(struct xenbus_device *dev, > > + struct xen_9pfs_dataring *ring) > > +{ > > + int i; > > + int ret = -ENOMEM; > > + > > +

Re: [Xen-devel] [PATCH 5/7] xen/9pfs: send requests to the backend

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Boris Ostrovsky wrote: > On 03/06/2017 03:01 PM, Stefano Stabellini wrote: > > Implement struct p9_trans_module create and close functions by looking > > at the available Xen 9pfs frontend-backend connections. We don't expect > > many frontend-backend connections, thus walking

[Xen-devel] [xen-unstable baseline-only test] 68642: tolerable trouble: blocked/broken/fail/pass

2017-03-07 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68642 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68642/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-libvirt-xsm 13

Re: [Xen-devel] [PATCH 6/7] xen/9pfs: receive responses

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Boris Ostrovsky wrote: > On 03/06/2017 03:01 PM, Stefano Stabellini wrote: > > Upon receiving a notification from the backend, schedule the > > p9_xen_response work_struct. p9_xen_response checks if any responses are > > available, if so, it reads them one by one, calling

[Xen-devel] [PATCH 2/2] x86/emul: Avoid #UD when emulating v{, u}comis{s, d}

2017-03-07 Thread Andrew Cooper
v{,u}comis{s,d} have two operands, so require vex.reg set to ~0. Spotted by AFL Signed-off-by: Andrew Cooper --- CC: Jan Beulich --- xen/arch/x86/x86_emulate/x86_emulate.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Xen-devel] [PATCH 1/2] break

2017-03-07 Thread Andrew Cooper
--- xen/arch/x86/x86_emulate/x86_emulate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/arch/x86/x86_emulate/x86_emulate.c b/xen/arch/x86/x86_emulate/x86_emulate.c index 1b507f7..e09975c 100644 --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++

[Xen-devel] [xen-4.6-testing test] 106529: regressions - FAIL

2017-03-07 Thread osstest service owner
flight 106529 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/106529/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 15 guest-localmigrate/x10 fail REGR. vs. 105991

Re: [Xen-devel] [PATCH 6/7] xen/9pfs: receive responses

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Stefano Stabellini wrote: > > > + > > > + ring = container_of(work, struct xen_9pfs_dataring, work); > > > + priv = ring->priv; > > > + > > > + while (1) { > > > + cons = ring->intf->in_cons; > > > + prod = ring->intf->in_prod; > > > + rmb(); > > > > >

[Xen-devel] [PATCH v3 6/7] xen/mce: remove ASSERT's about mce_[u|d]handler_num in mce_action()

2017-03-07 Thread Haozhong Zhang
Those assertions as well as mce_[u|d]handlers[], mce_[u|d]handler_num and mce_action() were intel only and lifted to the common code by c/s 3a91769d6e1. However, MCE handling on AMD does not use mce_[u|d]handlers[] before and after that commit, so assertions in mce_action() about their size do not

Re: [Xen-devel] [PATCH 24/29] drivers: convert iblock_req.pending from atomic_t to refcount_t

2017-03-07 Thread Nicholas A. Bellinger
Hi Elena, On Mon, 2017-03-06 at 16:21 +0200, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free >

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

2017-03-07 Thread osstest service owner
flight 106532 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/106532/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-xsm 5 xen-buildfail REGR. vs. 106491 Regressions

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Zhongze Liu
So, taking all of the conversations above into consideration, the following changes should be done to this patch: 1. According to Andrew and Jan's suggestions, I'll remove the CMDLINE_BOOL option, and deal with CMDLINE without the #ifdef-ary's. 2. Make the option CMDLINE_OVERRIDE depends on

Re: [Xen-devel] [PATCH v3] xen: Allow a default compiled-in command line using Kconfig

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Julien Grall wrote: > Hi Stefano, > > On 03/07/2017 07:54 PM, Stefano Stabellini wrote: > > On Tue, 7 Mar 2017, Julien Grall wrote: > > Given that upstream GRUB doesn't yet support booting Xen on ARM (without > > any additional patches), I think that the ability to completely

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

2017-03-07 Thread osstest service owner
flight 106536 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106536/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a build-arm64 5

Re: [Xen-devel] [PATCHv4 18/33] x86/xen: convert __xen_pgd_walk() and xen_cleanmfnmap() to support p4d

2017-03-07 Thread Boris Ostrovsky
On 03/07/2017 01:26 PM, Andrew Cooper wrote: > On 07/03/17 18:18, Boris Ostrovsky wrote: Don't we need to pass vaddr down to all routines so that they select appropriate tables? You seem to always be choosing the first one. >>> IIUC, we clear whole page table subtree covered by one pgd

Re: [Xen-devel] [PATCH 10/29] drivers, md: convert stripe_head.count from atomic_t to refcount_t

2017-03-07 Thread Shaohua Li
On Mon, Mar 06, 2017 at 04:20:57PM +0200, Elena Reshetova wrote: > refcount_t type and corresponding API should be > used instead of atomic_t when the variable is used as > a reference counter. This allows to avoid accidental > refcounter overflows that might lead to use-after-free > situations. >

Re: [Xen-devel] [PATCH v4 5/5] xen: use libxendevicemodel when available

2017-03-07 Thread Stefano Stabellini
On Tue, 7 Mar 2017, Paul Durrant wrote: > This patch modifies the wrapper functions in xen_common.h to use the > new xendevicemodel interface if it is available along with compatibility > code to use the old libxenctrl interface if it is not. > > Signed-off-by: Paul Durrant

Re: [Xen-devel] [PATCHv4 18/33] x86/xen: convert __xen_pgd_walk() and xen_cleanmfnmap() to support p4d

2017-03-07 Thread Boris Ostrovsky
>> Don't we need to pass vaddr down to all routines so that they select >> appropriate tables? You seem to always be choosing the first one. > IIUC, we clear whole page table subtree covered by one pgd entry. > So, no, there's no need to pass vaddr down. Just pointer to page table > entry is

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

2017-03-07 Thread osstest service owner
flight 106530 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106530/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a build-arm64 5

  1   2   >