Re: [Xen-devel] [RFC v2 2/6] x86/init: use linker tables to simplify x86 init and annotate dependencies

2016-02-19 Thread Andy Lutomirski
On Fri, Feb 19, 2016 at 4:42 PM, Luis R. Rodriguez wrote: > On Fri, Feb 19, 2016 at 08:40:49AM -0800, Andy Lutomirski wrote: >> On Fri, Feb 19, 2016 at 6:15 AM, Luis R. Rodriguez wrote: >> > Any failure on the x86 init path can be catastrophic. >> > A simple

[Xen-devel] [qemu-upstream-unstable test] 83040: tolerable FAIL - PUSHED

2016-02-19 Thread osstest service owner
flight 83040 qemu-upstream-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/83040/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 15 guest-start/debian.repeatfail like 82198

Re: [Xen-devel] [PATCH] xen-netfront: set real_num_tx_queues to zreo avoid to trigger BUG_ON

2016-02-19 Thread Gonglei (Arei)
Hi, Thanks for rapid feedback :) > From: David Miller [mailto:da...@davemloft.net] > Sent: Saturday, February 20, 2016 12:37 PM > > From: Gonglei > Date: Sat, 20 Feb 2016 09:27:26 +0800 > > > It's possible for a race condition to exist between xennet_open() and > >

[Xen-devel] [qemu-upstream-4.4-testing test] 83041: tolerable FAIL - PUSHED

2016-02-19 Thread osstest service owner
flight 83041 qemu-upstream-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/83041/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stopfail blocked in 82234 Tests which did

Re: [Xen-devel] [PATCH] xen-netfront: set real_num_tx_queues to zreo avoid to trigger BUG_ON

2016-02-19 Thread David Miller
From: Gonglei Date: Sat, 20 Feb 2016 09:27:26 +0800 > It's possible for a race condition to exist between xennet_open() and > talk_to_netback(). After invoking netfront_probe() then other > threads or processes invoke xennet_open (such as NetworkManager) > immediately

[Xen-devel] [linux-3.14 test] 83009: tolerable FAIL - PUSHED

2016-02-19 Thread osstest service owner
flight 83009 linux-3.14 real [real] http://logs.test-lab.xenproject.org/osstest/logs/83009/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 79562 build-amd64-rumpuserxen

[Xen-devel] [PATCH] xen-netfront: set real_num_tx_queues to zreo avoid to trigger BUG_ON

2016-02-19 Thread Gonglei
It's possible for a race condition to exist between xennet_open() and talk_to_netback(). After invoking netfront_probe() then other threads or processes invoke xennet_open (such as NetworkManager) immediately may trigger BUG_ON(). Besides, we also should reset real_num_tx_queues in

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

2016-02-19 Thread osstest service owner
flight 83007 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/83007/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-prev 5 xen-build fail REGR. vs. 82825 Tests which are

Re: [Xen-devel] [PATCH 5/9] apm32: remove paravirt_enabled() use

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 05:17:27PM -0500, Boris Ostrovsky wrote: > > > On 02/19/2016 03:58 PM, Luis R. Rodriguez wrote: > >On Fri, Feb 19, 2016 at 10:08:43AM -0500, Boris Ostrovsky wrote: > > > >>in xen_start_kernel(). Better yet, clear whole .bss. > >> > >>(This applies to the next patch as

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

2016-02-19 Thread osstest service owner
flight 83008 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/83008/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 17 guest-start/debianhvm.repeat fail REGR. vs. 65543

Re: [Xen-devel] [PATCH 5/9] apm32: remove paravirt_enabled() use

2016-02-19 Thread Boris Ostrovsky
On 02/19/2016 03:58 PM, Luis R. Rodriguez wrote: On Fri, Feb 19, 2016 at 10:08:43AM -0500, Boris Ostrovsky wrote: in xen_start_kernel(). Better yet, clear whole .bss. (This applies to the next patch as well). So clear_bss() -- oh look, another call that xen_start_kernel() could have made

Re: [Xen-devel] [PATCH v2 05/30] xen/public: Export cpu featureset information in the public API

2016-02-19 Thread Joao Martins
On 02/19/2016 05:55 PM, Andrew Cooper wrote: > On 19/02/16 17:29, Joao Martins wrote: >> On 02/05/2016 01:41 PM, Andrew Cooper wrote: >>> For the featureset to be a useful object, it needs a stable interpretation, >>> a >>> property which is missing from the current hw_caps interface. >>> >>>

Re: [Xen-devel] [PATCH v3 06/23] xsplice: Implement payload loading (v4)

2016-02-19 Thread Konrad Rzeszutek Wilk
> > + hdr->e_ident[EI_CLASS] != ELFCLASS64 || > > + hdr->e_ident[EI_DATA] != ELFDATA2LSB || > > + hdr->e_ident[EI_OSABI] != ELFOSABI_SYSV || > > + hdr->e_machine != EM_X86_64 || > > + hdr->e_type != ET_REL || > > + hdr->e_phnum != 0 ) > > +{ > >

Re: [Xen-devel] [RFC v2 1/7] sections.h: add sections header to collect all section info

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 01:25:22PM -0800, Greg KH wrote: > On Fri, Feb 19, 2016 at 09:06:03PM +0100, Luis R. Rodriguez wrote: > > On Fri, Feb 19, 2016 at 08:23:29AM -0800, Greg KH wrote: > > > On Fri, Feb 19, 2016 at 05:45:53AM -0800, Luis R. Rodriguez wrote: > > > > + * Due to this file being

Re: [Xen-devel] [RFC v2 2/7] tables.h: add linker table support

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 12:25:55PM -0800, H. Peter Anvin wrote: > On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > > + > > +/** > > + * DOC: Regular linker linker table constructors > > + * > > + * Regular constructors are expected to be used for valid linker table > > entries. > > + * Valid

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

2016-02-19 Thread osstest service owner
flight 83237 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/83237/ 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] [RFC v2 1/7] sections.h: add sections header to collect all section info

2016-02-19 Thread Greg KH
On Fri, Feb 19, 2016 at 09:06:03PM +0100, Luis R. Rodriguez wrote: > On Fri, Feb 19, 2016 at 08:23:29AM -0800, Greg KH wrote: > > On Fri, Feb 19, 2016 at 05:45:53AM -0800, Luis R. Rodriguez wrote: > > > + * Due to this file being licensed under the GPL there is controversy > > > over > > > + *

Re: [Xen-devel] [RFC v2 0/7] linux: add linker tables

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 12:16:29PM -0800, H. Peter Anvin wrote: > On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > > This is my v2 of the original linker table work [0], now with > > six proof of concepts ports of existing code using custom section > > with custom linker script modifications: >

Re: [Xen-devel] [RFC v2 2/7] tables.h: add linker table support

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 12:33:27PM -0800, H. Peter Anvin wrote: > On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > > +/** > > + * LINKTABLE_RUN_ERR - run each linker table entry func and return error > > if any > > + * > > + * @tbl: linker table > > + * @func: structure name for the function

Re: [Xen-devel] [PATCH v3 04/23] elf: Add relocation types to elfstructs.h

2016-02-19 Thread Konrad Rzeszutek Wilk
On Mon, Feb 15, 2016 at 01:34:42AM -0700, Jan Beulich wrote: > >>> On 12.02.16 at 19:05, wrote: > > --- a/xen/include/xen/elfstructs.h > > +++ b/xen/include/xen/elfstructs.h > > @@ -348,6 +348,14 @@ typedef struct { > > #defineELF64_R_TYPE(info) ((info) &

Re: [Xen-devel] [RFC v2 4/7] asm/sections: add a generic push_section_tbl()

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 12:26:51PM -0800, H. Peter Anvin wrote: > On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > > With a generic linker tables solution in place we > > need a general asm solution for declaring entries > > with asm. The first easy target is to cover the C > > asm declarations,

Re: [Xen-devel] [RFC v2 1/6] x86/boot: add BIT() to boot/bitops.h

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 10:14:42AM -0500, Boris Ostrovsky wrote: > > > On 02/19/2016 09:15 AM, Luis R. Rodriguez wrote: > >diff --git a/arch/x86/boot/bitops.h b/arch/x86/boot/bitops.h > >index 878e4b9940d9..232cff0ff4e3 100644 > >--- a/arch/x86/boot/bitops.h > >+++ b/arch/x86/boot/bitops.h > >@@

Re: [Xen-devel] [PATCH 5/9] apm32: remove paravirt_enabled() use

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 10:08:43AM -0500, Boris Ostrovsky wrote: > > > On 02/19/2016 08:08 AM, Luis R. Rodriguez wrote: > >There is already a check for apm_info.bios == 0, the > >apm_info.bios is set from the boot_params.apm_bios_info. > >Both Xen and lguest, which are also the only ones that

Re: [Xen-devel] [PATCH v3 03/23] xen-xsplice: Tool to manipulate xsplice payloads (v4)

2016-02-19 Thread Konrad Rzeszutek Wilk
On Mon, Feb 15, 2016 at 12:59:02PM +, Wei Liu wrote: > On Fri, Feb 12, 2016 at 01:05:41PM -0500, Konrad Rzeszutek Wilk wrote: > [...] > > diff --git a/tools/misc/xen-xsplice.c b/tools/misc/xen-xsplice.c > > new file mode 100644 > > index 000..13f762f > > --- /dev/null > > +++

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 8:42 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 11:33 AM, Corneliu ZUZU > wrote: On 2/19/2016 8:27 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 11:11 AM, Corneliu ZUZU

Re: [Xen-devel] [RFC v2 2/7] tables.h: add linker table support

2016-02-19 Thread H. Peter Anvin
On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > +/** > + * LINKTABLE_RUN_ERR - run each linker table entry func and return error if > any > + * > + * @tbl: linker table > + * @func: structure name for the function name we want to call. > + * @args...: arguments to pass to func > + * > + *

Re: [Xen-devel] [RFC v2 2/7] tables.h: add linker table support

2016-02-19 Thread H. Peter Anvin
On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > + > +/** > + * DOC: Regular linker linker table constructors > + * > + * Regular constructors are expected to be used for valid linker table > entries. > + * Valid uses of weak entries other than the beginning and is currently > + * untested but

Re: [Xen-devel] [RFC v2 4/7] asm/sections: add a generic push_section_tbl()

2016-02-19 Thread H. Peter Anvin
On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > With a generic linker tables solution in place we > need a general asm solution for declaring entries > with asm. The first easy target is to cover the C > asm declarations, guard the header file for now > and define a first generic entry

Re: [Xen-devel] [RFC v2 0/7] linux: add linker tables

2016-02-19 Thread H. Peter Anvin
On 02/19/2016 05:45 AM, Luis R. Rodriguez wrote: > This is my v2 of the original linker table work [0], now with > six proof of concepts ports of existing code using custom section > with custom linker script modifications: > > * DEFINE_LINKTABLE_TEXT(char, kprobes); > *

Re: [Xen-devel] [RFC v2 1/7] sections.h: add sections header to collect all section info

2016-02-19 Thread Luis R. Rodriguez
On Fri, Feb 19, 2016 at 08:23:29AM -0800, Greg KH wrote: > On Fri, Feb 19, 2016 at 05:45:53AM -0800, Luis R. Rodriguez wrote: > > Linux makes extensive use of custom ELF header sections, > > documentation for these are well scatterred. Unify this > > documentation in a central place. > > Minor

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

2016-02-19 Thread osstest service owner
flight 83006 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/83006/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stopfail REGR. vs. 82859 test-armhf-armhf-xl-rtds

Re: [Xen-devel] [PATCH v3 02/23] libxc: Implementation of XEN_XSPLICE_op in libxc (v5).

2016-02-19 Thread Konrad Rzeszutek Wilk
.snip.. > > +/* > > + * The heart of this function is to get an array of xen_xsplice_status_t. > > + * > > + * However it is complex because it has to deal with the hypervisor > > + * returning -EAGAIN or the data that is being returned becomes stale > > + * (another hypercall might alter the

Re: [Xen-devel] [PATCH] MAINTAINERS: add myself for Travis CI config

2016-02-19 Thread Andrew Cooper
On 19/02/2016 19:55, Doug Goldstein wrote: > Add myself as the maintainer for the Travis CI config. > > Signed-off-by: Doug Goldstein Reviewed-by: Andrew Cooper ___ Xen-devel mailing list

[Xen-devel] [PATCH] MAINTAINERS: add myself for Travis CI config

2016-02-19 Thread Doug Goldstein
Add myself as the maintainer for the Travis CI config. Signed-off-by: Doug Goldstein --- CC: Ian Campbell CC: Ian Jackson CC: Jan Beulich CC: Keir Fraser CC: Tim Deegan ---

Re: [Xen-devel] [PATCH v3 01/23] xen/xsplice: Hypervisor implementation of XEN_XSPLICE_op (v10)

2016-02-19 Thread Andrew Cooper
On 19/02/2016 19:36, Konrad Rzeszutek Wilk wrote: >>> long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) u_sysctl) >>> { >>> @@ -460,6 +461,12 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) >>> u_sysctl) >>> ret = tmem_control(>u.tmem_op); >>> break; >>> >>> +

Re: [Xen-devel] [PATCH v3 01/23] xen/xsplice: Hypervisor implementation of XEN_XSPLICE_op (v10)

2016-02-19 Thread Konrad Rzeszutek Wilk
> > long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) u_sysctl) > > { > > @@ -460,6 +461,12 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) > > u_sysctl) > > ret = tmem_control(>u.tmem_op); > > break; > > > > +case XEN_SYSCTL_xsplice_op: > > +ret =

Re: [Xen-devel] [PATCH 3/3] docs: add more info about target= in disk config

2016-02-19 Thread Jim Fehlig
On 02/19/2016 10:23 AM, Ian Jackson wrote: > Jim Fehlig writes ("[PATCH 3/3] docs: add more info about target= in disk > config"): >> target= in disk config can be used to convey arbitrary >> configuration information to backends. Add a bit more info >> to xl-disk-configuration.txt to clarify

[Xen-devel] [PATCH v2] hvmloader: Use xen/errno.h rather than the host systems errno.h

2016-02-19 Thread Andrew Cooper
hvmloader is unhosted, and shouldn't use the system errno.h. It already has to use Xen's errno.h for other hypercalls. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Ian Campbell CC: Ian Jackson

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Tamas K Lengyel
On Fri, Feb 19, 2016 at 11:33 AM, Corneliu ZUZU wrote: > On 2/19/2016 8:27 PM, Tamas K Lengyel wrote: > > > > On Fri, Feb 19, 2016 at 11:11 AM, Corneliu ZUZU > wrote: > >> On 2/19/2016 7:54 PM, Tamas K Lengyel wrote: >> >> >> >> On Fri, Feb 19, 2016

Re: [Xen-devel] [PATCH v3 MISSING/23] xsplice: Design document (v7).

2016-02-19 Thread Konrad Rzeszutek Wilk
On Thu, Feb 18, 2016 at 09:20:00AM -0700, Jan Beulich wrote: > >>> On 12.02.16 at 22:57, wrote: > > +struct xsplice_patch_func { > > +const char *name; > > +Elf64_Xwordnew_addr; > > Missing space. > > > +Elf64_Xword old_addr; > > +Elf64_Word

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 8:27 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 11:11 AM, Corneliu ZUZU > wrote: On 2/19/2016 7:54 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 10:47 AM, Stefano Stabellini

[Xen-devel] [xen-4.3-testing test] 83004: tolerable FAIL - PUSHED

2016-02-19 Thread osstest service owner
flight 83004 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/83004/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemut-win7-amd64 16 guest-stop fail like 78718

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Tamas K Lengyel
On Fri, Feb 19, 2016 at 11:11 AM, Corneliu ZUZU wrote: > On 2/19/2016 7:54 PM, Tamas K Lengyel wrote: > > > > On Fri, Feb 19, 2016 at 10:47 AM, Stefano Stabellini < > stefano.stabell...@eu.citrix.com> wrote: > >> On Fri, 19 Feb 2016,

[Xen-devel] [ovmf bisection] complete test-amd64-amd64-xl-qemuu-ovmf-amd64

2016-02-19 Thread osstest service owner
branch xen-unstable xenbranch xen-unstable job test-amd64-amd64-xl-qemuu-ovmf-amd64 testid guest-start/debianhvm.repeat Tree: linux git://xenbits.xen.org/linux-pvops.git Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git Tree: ovmf https://github.com/tianocore/edk2.git Tree:

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 7:54 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 10:47 AM, Stefano Stabellini > wrote: On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > On 2/19/2016 6:05 PM, Andrew Cooper wrote: > > On

Re: [Xen-devel] [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-19 Thread Lengyel, Tamas
On Fri, Feb 19, 2016 at 10:26 AM, Lengyel, Tamas wrote: > > > > On Fri, Feb 19, 2016 at 10:18 AM, Andrew Cooper > wrote: > >> On 19/02/16 17:06, Lengyel, Tamas wrote: >> >> >> >> On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich

[Xen-devel] [PATCH v7] libxl: allow 'phy' backend to use empty files

2016-02-19 Thread Roger Pau Monne
This was introduced by 97ee1f (~5 years ago), but was probably never surfaced because most people used regular files as CDROM images, so the PHY backend was actually never selected. A year ago this was changed, and now regular RAW files are also handled by the PHY backend, which has made this bug

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 7:15 PM, Jan Beulich wrote: On 19.02.16 at 17:25, wrote: On 2/19/2016 4:26 PM, Jan Beulich wrote: On 18.02.16 at 20:35, wrote: --- MAINTAINERS | 1 + xen/arch/arm/hvm.c | 8 +++

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

2016-02-19 Thread osstest service owner
flight 83214 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/83214/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl 6 xen-boot fail REGR. vs. 83064 Tests which

Re: [Xen-devel] [PATCH v2 05/30] xen/public: Export cpu featureset information in the public API

2016-02-19 Thread Andrew Cooper
On 19/02/16 17:29, Joao Martins wrote: > On 02/05/2016 01:41 PM, Andrew Cooper wrote: >> For the featureset to be a useful object, it needs a stable interpretation, a >> property which is missing from the current hw_caps interface. >> >> Additionly, introduce TSC_ADJUST, SHA, PREFETCHWT1, ITSC,

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Tamas K Lengyel
On Fri, Feb 19, 2016 at 10:47 AM, Stefano Stabellini < stefano.stabell...@eu.citrix.com> wrote: > On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > > On 2/19/2016 6:05 PM, Andrew Cooper wrote: > > > On 19/02/16 16:00, Stefano Stabellini wrote: > > > > On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > > > > >

[Xen-devel] [OSSTEST PATCH] mg-list-all-branches: avoid mistakenly generating `.' in the output

2016-02-19 Thread Ian Jackson
The regex in mg-list-all-branches assumes that the BRANCHES= will either be a singleton entry separated from the following command by a hard tab or a single quoted list of space separated entries, however the xen-unstable-coverity line is singleton separated from the command by a single space. We

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Stefano Stabellini
On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > On 2/19/2016 6:05 PM, Andrew Cooper wrote: > > On 19/02/16 16:00, Stefano Stabellini wrote: > > > On Fri, 19 Feb 2016, Corneliu ZUZU wrote: > > > > On 2/19/2016 3:49 PM, Stefano Stabellini wrote: > > > > > On Thu, 18 Feb 2016, Corneliu ZUZU wrote: > > >

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-19 Thread Roger Pau Monné
El 19/2/16 a les 18:30, Ian Jackson ha escrit: > Roger Pau Monne writes ("[PATCH v6] libxl: allow 'phy' backend to use empty > files"): >> This was introduced by 97ee1f (~5 years ago), but was probably never >> surfaced because most people used regular files as CDROM images, so the PHY >> backend

Re: [Xen-devel] [PATCH] tools/xenalyze: Fix build with clang

2016-02-19 Thread Ian Jackson
George Dunlap writes ("Re: [PATCH] tools/xenalyze: Fix build with clang"): > On 12/02/16 19:06, Andrew Cooper wrote: > > Signed-off-by: Andrew Cooper > > All looks good, thanks. > > Acked-by: George Dunlap Queued, thanks. Ian.

Re: [Xen-devel] [PATCH v6] libxl: allow 'phy' backend to use empty files

2016-02-19 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v6] libxl: allow 'phy' backend to use empty files"): > This was introduced by 97ee1f (~5 years ago), but was probably never > surfaced because most people used regular files as CDROM images, so the PHY > backend was actually never selected. A year ago this was

Re: [Xen-devel] [PATCH v2 05/30] xen/public: Export cpu featureset information in the public API

2016-02-19 Thread Joao Martins
On 02/05/2016 01:41 PM, Andrew Cooper wrote: > For the featureset to be a useful object, it needs a stable interpretation, a > property which is missing from the current hw_caps interface. > > Additionly, introduce TSC_ADJUST, SHA, PREFETCHWT1, ITSC, EFRO and CLZERO > which will be used by later

Re: [Xen-devel] [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-19 Thread Lengyel, Tamas
On Fri, Feb 19, 2016 at 10:18 AM, Andrew Cooper wrote: > On 19/02/16 17:06, Lengyel, Tamas wrote: > > > > On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich wrote: > >> >>> On 16.02.16 at 07:58, < kevin.t...@intel.com> >> wrote: >>

Re: [Xen-devel] [PATCH V2 3/3] docs: add more info about target= in disk config

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH V2 3/3] docs: add more info about target= in disk config"): > On Wed, 2016-02-17 at 10:20 -0700, Jim Fehlig wrote: > > target= in disk config can be used to convey arbitrary > > configuration information to backends. Add a bit more info > > to

Re: [Xen-devel] [PATCH 3/3] docs: add more info about target= in disk config

2016-02-19 Thread Ian Jackson
Jim Fehlig writes ("[PATCH 3/3] docs: add more info about target= in disk config"): > target= in disk config can be used to convey arbitrary > configuration information to backends. Add a bit more info > to xl-disk-configuration.txt to clarify this, including some > simple nbd and rbd qdisk

Re: [Xen-devel] [PATCH] xen/arm64: Make sure we get all debug output

2016-02-19 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH] xen/arm64: Make sure we get all debug output"): > On 18.02.16 at 11:15, wrote: > > I think this would be worth backporting to 4.6 and earlier (as far back as > > it applies). Jan or Ian -- would one of you mind doing so when the time > >

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 12:16:31PM -0500, Konrad Rzeszutek Wilk wrote: > > > .. forbid what? PVHVM resuming with fast=0? Why? Because the guest may > > > fall on its face? > > > > Yes, forbid resuming PVHVM with fast=0 if we have no clear definition of > > how it works. It's not because guest

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 17:38, wrote: > On 19/02/16 15:43, David Vrabel wrote: >> On 19/02/16 15:14, Jan Beulich wrote: >>> >>> Iirc the issue was with a 64-bit XSAVE having got the selector >>> values stuck in via FNSTENV (forcing word size to 4), and a >>> subsequent

Re: [Xen-devel] [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-19 Thread Andrew Cooper
On 19/02/16 17:06, Lengyel, Tamas wrote: > > > On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich > wrote: > > >>> On 16.02.16 at 07:58, > wrote: > >> --- a/xen/arch/x86/hvm/vmx/vmx.c > >>

Re: [Xen-devel] [PATCH] tools: remove xenpmd

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [PATCH] tools: remove xenpmd"): > On Wed, 2016-02-17 at 12:05 -0500, Konrad Rzeszutek Wilk wrote: > > On Wed, Feb 17, 2016 at 02:41:14PM +, Ian Campbell wrote: > > > It has four instances of the same CID 1198719 (strncpy(..., 32) into a > > > 32-byte

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Konrad Rzeszutek Wilk
> > .. forbid what? PVHVM resuming with fast=0? Why? Because the guest may > > fall on its face? > > Yes, forbid resuming PVHVM with fast=0 if we have no clear definition of > how it works. It's not because guest would fall, it's because we can't > tell which side (the guest or the toolstack) is

Re: [Xen-devel] [PATCH] xen/arm64: Make sure we get all debug output

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 18:13, wrote: > Jan Beulich writes ("Re: [PATCH] xen/arm64: Make sure we get all debug > output"): >> On 18.02.16 at 11:15, wrote: >> > I think this would be worth backporting to 4.6 and earlier (as far back as >> > it

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Jan Beulich
>>> On 19.02.16 at 17:25, wrote: > On 2/19/2016 4:26 PM, Jan Beulich wrote: > On 18.02.16 at 20:35, wrote: >>> --- >>> MAINTAINERS | 1 + >>> xen/arch/arm/hvm.c | 8 +++ >>> xen/arch/x86/hvm/event.c

Re: [Xen-devel] [PATCH] xen/blback: Fit the important information of the thread in 17 characters

2016-02-19 Thread Konrad Rzeszutek Wilk
> > Signed-off-by: Konrad Rzeszutek Wilk > > Acked-by: Roger Pau Monné > > According to: > > http://lists.xenproject.org/archives/html/xen-devel/2015-05/msg03505.html > > Should this have a: > > Reported-by: Jan Beulich Yes

Re: [Xen-devel] Adding Xen to the kbuild bot?

2016-02-19 Thread Ian Jackson
Andy Lutomirski writes ("Re: Adding Xen to the kbuild bot?"): > On Feb 4, 2016 7:11 PM, "Fengguang Wu" wrote: > > Do you mean to run basic Xen testing on the various kernel trees that > > 0day robot covers? That is, to catch kernel regressions when running > > under Xen.

Re: [Xen-devel] [PATCH v3 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-19 Thread Lengyel, Tamas
On Tue, Feb 16, 2016 at 3:47 AM, Jan Beulich wrote: > >>> On 16.02.16 at 07:58, wrote: > >> --- a/xen/arch/x86/hvm/vmx/vmx.c > >> +++ b/xen/arch/x86/hvm/vmx/vmx.c > >> @@ -490,6 +490,7 @@ static void vmx_vmcs_save(struct vcpu *v, struct > hvm_hw_cpu > >>

Re: [Xen-devel] git workflow and http://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches

2016-02-19 Thread Konrad Rzeszutek Wilk
On Mon, Feb 15, 2016 at 09:59:11AM -0700, Jan Beulich wrote: > >>> On 15.02.16 at 17:10, wrote: > > El 15/2/16 a les 17:00, Konrad Rzeszutek Wilk ha escrit: > >> Hey, > >> > >> I wanted to ask what folks would think of expanding the Wiki to include > > (making it a diff >

Re: [Xen-devel] [PATCH] xenpaging: don't try to log via xch if xc_interface_close fails

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("[PATCH] xenpaging: don't try to log via xch if xc_interface_close fails"): > Since xch may not be valid (enough) any longer, xc_interface_close > already logs anything of any use before it tears down the integrated > logger so there is no need to log any further in the

Re: [Xen-devel] [PATCH 2/2] tools: gtracestat: make all functions and global data static

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("[PATCH 2/2] tools: gtracestat: make all functions and global data static"): > After "Drop unused functions do_cstate and single_cstate helper" make > all the remaining functions and global data static and in the process > allow the compiler to notice that cond_rec_init() is

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Wei Liu
On Fri, Feb 19, 2016 at 11:20:08AM -0500, Konrad Rzeszutek Wilk wrote: [...] > > > > > > === > > > > > > > > > > > > Note that I fix one place in this version from "guest state" to > > > > > > "guest > > > > > > return code" in the second paragraph. And that sentence is a big big > > > > > >

Re: [Xen-devel] [RFC v2 2/6] x86/init: use linker tables to simplify x86 init and annotate dependencies

2016-02-19 Thread Andy Lutomirski
On Fri, Feb 19, 2016 at 6:15 AM, Luis R. Rodriguez wrote: > Any failure on the x86 init path can be catastrophic. > A simple shift of a call from one place to another can > easily break things. Likewise adding a new call to > one path without considering all x86 requirements >

Re: [Xen-devel] [PATCHv1 3/5] x86/fpu: Add a per-domain field to set the width of FIP/FDP

2016-02-19 Thread David Vrabel
On 19/02/16 15:43, David Vrabel wrote: > On 19/02/16 15:14, Jan Beulich wrote: >> >> Iirc the issue was with a 64-bit XSAVE having got the selector >> values stuck in via FNSTENV (forcing word size to 4), and a >> subsequent XSAVEOPT not further touching the fields >> (because no change to the FPU

[Xen-devel] [PATCH] xl: Improve return and exit codes of parse_config_data().

2016-02-19 Thread Harmandeep Kaur
Turning parse_config_data() exit codes towards using the EXIT_[SUCCESS|FAILURE] constants, instead of instead of arbitrary numbers or libxl return codes. - for main_*: arbitrary -> EXIT_SUCCESS|EXIT_FAILURE. - for internal fucntion: arbitrary -> 0/1. Signed-off-by: Harmandeep Kaur

[Xen-devel] [PATCH] arm64: xen_boot: Fix xen boot using Grub on AARCH64

2016-02-19 Thread Julien Grall
Xen is currently crashing because of malformed compatible property for the boot module. This is because the property string is not null-terminated as requested by the ePAR spec. --- grub-core/loader/arm64/xen_boot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 6:02 PM, Tamas K Lengyel wrote: On Fri, Feb 19, 2016 at 7:26 AM, Jan Beulich > wrote: >>> On 18.02.16 at 20:35, > wrote: > --- > MAINTAINERS | 1

Re: [Xen-devel] [PATCH v5 2/2] build: convert xenoprof to Kconfig

2016-02-19 Thread Konrad Rzeszutek Wilk
On Thu, Feb 18, 2016 at 10:25:57PM -0600, Doug Goldstein wrote: > Convert the xenoprof x86 build time option to Kconfig. > > Signed-off-by: Doug Goldstein > Reviewed-by: Andrew Cooper > Acked-by: Jan Beulich applied

Re: [Xen-devel] [PATCH v5 1/2] xenoprof: fix up ability to disable it

2016-02-19 Thread Konrad Rzeszutek Wilk
On Fri, Feb 19, 2016 at 08:56:32AM +, Andrew Cooper wrote: > On 19/02/2016 04:25, Doug Goldstein wrote: > > Allow Xenoprof to be fully disabled when toggling the option off. > > > > Signed-off-by: Doug Goldstein > > Reviewed-by: Andrew Cooper

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 4:26 PM, Jan Beulich wrote: On 18.02.16 at 20:35, wrote: --- MAINTAINERS | 1 + xen/arch/arm/hvm.c | 8 +++ xen/arch/x86/hvm/event.c| 116 ++-- xen/arch/x86/hvm/hvm.c

Re: [Xen-devel] [RFC v2 1/7] sections.h: add sections header to collect all section info

2016-02-19 Thread Greg KH
On Fri, Feb 19, 2016 at 05:45:53AM -0800, Luis R. Rodriguez wrote: > Linux makes extensive use of custom ELF header sections, > documentation for these are well scatterred. Unify this > documentation in a central place. Minor questions: > + > +You should have received a copy of the

Re: [Xen-devel] [PATCH] version: Document guest_handle

2016-02-19 Thread Razvan Cojocaru
On 02/19/2016 04:32 PM, Konrad Rzeszutek Wilk wrote: > And what it is usually used for. > > Signed-off-by: Konrad Rzeszutek Wilk > --- > xen/include/public/version.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/xen/include/public/version.h

[Xen-devel] [PATCH OSSTEST] sg-report-flight: move "started" column to the left in Step table

2016-02-19 Thread Ian Campbell
With the current ordering of status => started I frequently (more often than not) read the failing step as "(stepno,testid,script) failed at " (where is actually the start time, not the fail time). Move the "started" column to the left of the "status" column. On the basis that

Re: [Xen-devel] [PATCH v3 02/10] xen: reduce number of global variables in xen loader

2016-02-19 Thread Lennart Sorensen
On Fri, Feb 19, 2016 at 05:59:27AM +0100, Juergen Gross wrote: > On 18/02/16 18:00, Lennart Sorensen wrote: > > On Thu, Feb 18, 2016 at 11:34:49AM +0100, Juergen Gross wrote: > >> On 18/02/16 11:22, Daniel Kiper wrote: > >>> On Wed, Feb 17, 2016 at 06:19:29PM +0100, Juergen Gross wrote: > The

Re: [Xen-devel] [PATCH v8 05/13] tools/libxc: support to resume uncooperative HVM guests

2016-02-19 Thread Konrad Rzeszutek Wilk
On Fri, Feb 19, 2016 at 03:16:27PM +, Wei Liu wrote: > On Fri, Feb 19, 2016 at 02:52:11PM +, Ian Campbell wrote: > > On Fri, 2016-02-19 at 14:43 +, Wei Liu wrote: > > > On Fri, Feb 19, 2016 at 09:15:38AM -0500, Konrad Rzeszutek Wilk wrote: > > > > On Thu, Feb 18, 2016 at 12:13:36PM

Re: [Xen-devel] [PATCH] version: Document guest_handle

2016-02-19 Thread Konrad Rzeszutek Wilk
On Fri, Feb 19, 2016 at 04:36:14PM +0200, Razvan Cojocaru wrote: > On 02/19/2016 04:32 PM, Konrad Rzeszutek Wilk wrote: > > And what it is usually used for. > > > > Signed-off-by: Konrad Rzeszutek Wilk > > --- > > xen/include/public/version.h | 4 > > 1 file

Re: [Xen-devel] [PATCH] xen/memguard: Drop memguard_init() entirely

2016-02-19 Thread Andrew Cooper
On 19/02/16 14:44, Jan Beulich wrote: On 18.02.16 at 19:03, wrote: >> It is not obvious what this code is doing. Most of it dates from 2007/2008, >> and there have been substantial changes in Xen's memory handling since then. > Deleting code which isn't understood

Re: [Xen-devel] [PATCH] docs: document handling of metacharacter escape in xl disk format

2016-02-19 Thread Jim Fehlig
On 02/19/2016 03:14 AM, Ian Campbell wrote: > On Thu, 2016-02-18 at 15:44 -0700, Jim Fehlig wrote: >> Ian Campbell wrote: >>> Signed-off-by: Ian Campbell >>> Cc: Jim Fehlig >>> --- >>> docs/misc/xl-disk-configuration.txt | 18 ++ >>> 1

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Corneliu ZUZU
On 2/19/2016 6:05 PM, Andrew Cooper wrote: On 19/02/16 16:00, Stefano Stabellini wrote: On Fri, 19 Feb 2016, Corneliu ZUZU wrote: On 2/19/2016 3:49 PM, Stefano Stabellini wrote: On Thu, 18 Feb 2016, Corneliu ZUZU wrote: + +if ( sync ) +{ +req->flags |=

Re: [Xen-devel] [PATCH 1/3] xl: free config_data on error in domain_create

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 16:08 +, Ian Campbell wrote: >              /* allocate space for the extra config plus two EOLs > > > plus > > > \0 */ > > > @@ -6525,6 +6526,7 @@ int main_networkattach(int argc, char **argv) > > >  } > > >   > > >  xlu_cfg_destroy(config); > > > +config =

Re: [Xen-devel] [PATCH 1/3] xl: free config_data on error in domain_create

2016-02-19 Thread Ian Campbell
On Fri, 2016-02-19 at 16:02 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH 1/3] xl: free config_data on error in > domain_create"): > > CID: 1055898 > ... > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > > index d5a397a..e819ee6 100644 > > ---

[Xen-devel] Domctl and physdevop for passthrough (Was: Re: Stabilising some tools only HVMOPs?)

2016-02-19 Thread Wei Liu
On Wed, Feb 17, 2016 at 05:28:08PM +, Wei Liu wrote: > Hi all > > Tools people are in the process of splitting libxenctrl into a set of > stable libraries. One of the proposed libraries is libxendevicemodel > which has a collection of APIs that can be used by device model. > > Currently we

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Andrew Cooper
On 19/02/16 16:00, Stefano Stabellini wrote: > On Fri, 19 Feb 2016, Corneliu ZUZU wrote: >> On 2/19/2016 3:49 PM, Stefano Stabellini wrote: >>> On Thu, 18 Feb 2016, Corneliu ZUZU wrote: + +if ( sync ) +{ +req->flags |= VM_EVENT_FLAG_VCPU_PAUSED; +

Re: [Xen-devel] [PATCH 3/3] xl: create: close restore_fd_to_close on error

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("[PATCH 3/3] xl: create: close restore_fd_to_close on error"): > Currently the fd is opened and then later closed and > restore_fd_to_close set back to -1, however there are several goto out > and goto error_out paths in the interim. > > Since the code resets

Re: [Xen-devel] [PATCH 2/3] xl: use xrealloc in domain create

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("[PATCH 2/3] xl: use xrealloc in domain create"): > Using bare realloc risks leaking the old pointer if the realloc fails. > > Since xrealloc exits on such failures, drop the error handling. > > Noticed while fixing, but not related to, CID 1055898. > > Signed-off-by: Ian

Re: [Xen-devel] [PATCH 1/3] xl: free config_data on error in domain_create

2016-02-19 Thread Ian Jackson
Ian Campbell writes ("[PATCH 1/3] xl: free config_data on error in domain_create"): > CID: 1055898 ... > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > index d5a397a..e819ee6 100644 > --- a/tools/libxl/xl_cmdimpl.c > +++ b/tools/libxl/xl_cmdimpl.c > @@ -2795,6 +2795,7 @@

Re: [Xen-devel] [PATCH] arm/monitor vm-events: Implement guest-request support

2016-02-19 Thread Tamas K Lengyel
On Fri, Feb 19, 2016 at 7:26 AM, Jan Beulich wrote: > >>> On 18.02.16 at 20:35, wrote: > > --- > > MAINTAINERS | 1 + > > xen/arch/arm/hvm.c | 8 +++ > > xen/arch/x86/hvm/event.c| 116 >

  1   2   3   >