[Xen-devel] [qemu-mainline baseline-only test] 44210: tolerable trouble: broken/fail/pass

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

Re: [Xen-devel] [PATCH 3/4] xen/errno: Reduce complexity of inclusion

2016-03-03 Thread Jan Beulich
>>> On 01.03.16 at 19:57, wrote: > The inclusion rules conditions for errno.h were unnecesserily complicated, and > required the includer to jump through hoops if they wished to avoid getting > multiple namespaces worth of constants. > > Vastly simply the logic, and document what is going on. Ni

Re: [Xen-devel] [V2] x86/xsaves: calculate the xstate_comp_offsets base on xcomp_bv

2016-03-03 Thread Jan Beulich
>>> On 02.03.16 at 10:46, wrote: > Previous patch using all available features calculate xstate_comp_offsets. > This is wrong.This patch fix this bug by calculating the xstate_comp_offset > based on xcomp_bv of current guest. > Also, the xstate_comp_offset should take alignment into consideration.

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

2016-03-03 Thread osstest service owner
flight 85031 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85031/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 15 guest-localmigrate/x10 fail in 84925 pass in 85031 test-armhf-armhf-xl

Re: [Xen-devel] [PATCH V15 4/6] libxl: add pvusb API

2016-03-03 Thread George Dunlap
On Thu, Mar 3, 2016 at 3:11 AM, Chun Yan Liu wrote: On 3/3/2016 at 02:32 AM, in message <56d731b1.60...@citrix.com>, George Dunlap > wrote: >> On 01/03/16 08:09, Chunyan Liu wrote: >> > +static int usbdev_rebind(libxl__gc *gc, const char *busid) >> > +{ >> > +char **intfs = NULL; >

Re: [Xen-devel] [PATCH V15 4/6] libxl: add pvusb API

2016-03-03 Thread George Dunlap
On Thu, Mar 3, 2016 at 2:59 AM, Chun Yan Liu wrote: On 3/3/2016 at 02:46 AM, in message <56d7350f.7010...@citrix.com>, George > Dunlap wrote: >> Sorry, just looked through the rest of the series, and there's one more >> thing. >> >> Neither here nor in the man page do we explain what to do i

[Xen-devel] [PATCH v7 00/11] grub-xen: support booting huge pv-domains

2016-03-03 Thread Juergen Gross
The Xen hypervisor supports starting a dom0 with large memory (up to the TB range) by not including the initrd and p2m list in the initial kernel mapping. Especially the p2m list can grow larger than the available virtual space in the initial mapping. The started kernel is indicating the support o

[Xen-devel] [PATCH v7 11/11] xen: add capability to load p2m list outside of kernel mapping

2016-03-03 Thread Juergen Gross
Modern pvops linux kernels support a p2m list not covered by the kernel mapping. This capability is flagged by an elf-note specifying the virtual address the kernel is expecting the p2m list to be mapped to. In case the elf-note is set by the kernel don't place the p2m list into the kernel mapping

[Xen-devel] [PATCH v7 07/11] xen: factor out allocation of special pages into separate function

2016-03-03 Thread Juergen Gross
Do the allocation of special pages (start info, console and xenbus ring buffers) in a separate function. This will allow to do the allocation at different times of the boot preparations depending on the features the kernel is supporting. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper ---

[Xen-devel] [PATCH v7 09/11] xen: add capability to load initrd outside of initial mapping

2016-03-03 Thread Juergen Gross
Modern pvops linux kernels support an initrd not covered by the initial mapping. This capability is flagged by an elf-note. In case the elf-note is set by the kernel don't place the initrd into the initial mapping. This will allow to load larger initrds and/or support domains with larger memory, a

[Xen-devel] [PATCH v7 04/11] xen: add elfnote.h to avoid using numbers instead of constants

2016-03-03 Thread Juergen Gross
Various features and parameters of a pv-kernel are specified via elf notes in the kernel image. Those notes are part of the interface between the Xen hypervisor and the kernel. Instead of using num,bers in the code when interpreting the elf notes make use of the header supplied by Xen for that pur

[Xen-devel] [PATCH v7 01/11] xen: make xen loader callable multiple times

2016-03-03 Thread Juergen Gross
The loader for xen paravirtualized environment isn't callable multiple times as it won't free any memory in case of failure. Call grub_relocator_unload() as other modules do it before allocating a new relocator or when unloading the module. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper

[Xen-devel] [PATCH v7 03/11] xen: reduce number of global variables in xen loader

2016-03-03 Thread Juergen Gross
The loader for xen paravirtualized environment is using lots of global variables. Reduce the number by making them either local or by putting them into a single state structure. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper --- grub-core/loader/i386/xen.c | 259 +++-

[Xen-devel] [PATCH v7 05/11] xen: synchronize xen header

2016-03-03 Thread Juergen Gross
Get actual version of include/xen/xen.h from the Xen repository in order to be able to use constants defined there. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper --- include/xen/arch-x86/xen-x86_32.h | 22 +++ include/xen/arch-x86/xen-x86_64.h | 8 +-- include/xen/xen.h

[Xen-devel] [PATCH v7 02/11] xen: avoid memleaks on error

2016-03-03 Thread Juergen Gross
When loading a Xen pv-kernel avoid memory leaks in case of errors. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper --- V5: set grub_errno to GRUB_ERR_NONE to avoid false error reports as requested by Daniel Kiper --- grub-core/loader/i386/xen.c| 2 +- grub-core/loader/i386/x

[Xen-devel] [PATCH v7 06/11] xen: factor out p2m list allocation into separate function

2016-03-03 Thread Juergen Gross
Do the p2m list allocation of the to be loaded kernel in a separate function. This will allow doing the p2m list allocation at different times of the boot preparations depending on the features the kernel is supporting. While at this remove superfluous setting of first_p2m_pfn and nr_p2m_frames as

[Xen-devel] [PATCH v7 08/11] xen: factor out allocation of page tables into separate function

2016-03-03 Thread Juergen Gross
Do the allocation of page tables in a separate function. This will allow to do the allocation at different times of the boot preparations depending on the features the kernel is supporting. Signed-off-by: Juergen Gross Reviewed-by: Daniel Kiper --- grub-core/loader/i386/xen.c | 91 +

[Xen-devel] [PATCH v7 10/11] xen: modify page table construction

2016-03-03 Thread Juergen Gross
Modify the page table construction to allow multiple virtual regions to be mapped. This is done as preparation for removing the p2m list from the initial kernel mapping in order to support huge pv domains. This allows a cleaner approach for mapping the relocator page by using this capability. The

Re: [Xen-devel] Patching error while setting up COLO

2016-03-03 Thread Yu-An(Victor) Chen
Hi Changlong, Thanks for the reply, Again when I am trying to do the following: 5. build qemu-colo 1) cd ~/qemu-colo/; *git checkout colo-xen-v2* I got this error message *"error: pathspec 'colo-xen-v2' did not match any file(s) known to git."* Even if I do git fetch, I still get the same error

Re: [Xen-devel] [V2] x86/xsaves: calculate the xstate_comp_offsets base on xcomp_bv

2016-03-03 Thread Shuai Ruan
On Thu, Mar 03, 2016 at 01:45:03AM -0700, Jan Beulich wrote: > > @@ -134,16 +143,19 @@ static void __init setup_xstate_comp(void) > > */ > > xstate_comp_offsets[1] = XSAVE_SSE_OFFSET; > > > > xstate_comp_offsets[2] = FXSAVE_SIZE + XSAVE_HDR_SIZE; > > > > for ( i = 3; i < xs

Re: [Xen-devel] [PATCH RFC 0/8] x86/hvm, libxl: HVM SMT topology support

2016-03-03 Thread Joao Martins
On 03/02/2016 08:03 PM, Andrew Cooper wrote: > On 02/03/16 19:18, Joao Martins wrote: >> >> On 02/25/2016 05:21 PM, Andrew Cooper wrote: >>> On 22/02/16 21:02, Joao Martins wrote: Hey! This series are a follow-up on the thread about the performance of hard-pinned HVM guests. H

[Xen-devel] [PATCH 0/2] x86: more XSA-154 follow-ups

2016-03-03 Thread Jan Beulich
1: use "unsigned int" for cache attribute values 2: HVM: cache attribute pinning adjustments Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [V2] x86/xsaves: calculate the xstate_comp_offsets base on xcomp_bv

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 10:40, wrote: > On Thu, Mar 03, 2016 at 01:45:03AM -0700, Jan Beulich wrote: >> > @@ -134,16 +143,19 @@ static void __init setup_xstate_comp(void) >> > */ >> > xstate_comp_offsets[1] = XSAVE_SSE_OFFSET; >> > >> > xstate_comp_offsets[2] = FXSAVE_SIZE + XSAVE_HDR

Re: [Xen-devel] [PATCH v2 0/2] x86/entry/32: Get rid of paravirt_enabled in ESPFIX

2016-03-03 Thread Borislav Petkov
On Wed, Mar 02, 2016 at 04:47:38PM -0800, Andy Lutomirski wrote: > Because I'm mixing paravirt and cpufeatures a bit oddly. That's fine. All X86_BUG_* are synthetic and exactly for stuff like that. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. _

Re: [Xen-devel] [PATCH RFC 0/8] x86/hvm, libxl: HVM SMT topology support

2016-03-03 Thread Andrew Cooper
On 03/03/16 09:52, Joao Martins wrote: > In particular, I am concerned about giving the toolstack the ability to blindly control the APIC IDs. Their layout is very closely linked to topology, and in particular to the HTT flag. Overall, I want to avoid any possibility of ge

Re: [Xen-devel] [PATCH 1/2] x86: use "unsigned int" for cache attribute values

2016-03-03 Thread Jan Beulich
... where suitable. But note that the type of "cacheattr" in get_page_from_l1e() specifically needs to remain "unsigned long". Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -802,7 +802,7 @@ int is_iomem_page(unsigned long mfn) return (page_get_owner(page) ==

[Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Jan Beulich
- call hvm_get_mem_pinned_cacheattr() for RAM ranges only (requires some re-ordering in epte_get_entry_emt(), to fully handle all MMIO aspects first) - it's documented to be intended for RAM only - remove unnecessary indirection for hvm_get_mem_pinned_cacheattr()'s return of the type - make h

Re: [Xen-devel] [PATCH 1/2] x86: use "unsigned int" for cache attribute values

2016-03-03 Thread Andrew Cooper
On 03/03/16 10:31, Jan Beulich wrote: > ... where suitable. But note that the type of "cacheattr" in > get_page_from_l1e() specifically needs to remain "unsigned long". > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing li

Re: [Xen-devel] Patching error while setting up COLO

2016-03-03 Thread Wen Congyang
On 03/03/2016 05:39 PM, Yu-An(Victor) Chen wrote: > Hi Changlong, > > Thanks for the reply, > > Again when I am trying to do the following: > > 5. build qemu-colo > 1) cd ~/qemu-colo/; *git checkout colo-xen-v2* > * > * > I got this error message *"error: pathspec 'colo-xen-v2' did not match any

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Andrew Cooper
On 03/03/16 10:31, Jan Beulich wrote: > - call hvm_get_mem_pinned_cacheattr() for RAM ranges only (requires > some re-ordering in epte_get_entry_emt(), to fully handle all MMIO > aspects first) - it's documented to be intended for RAM only > - remove unnecessary indirection for hvm_get_mem_pinn

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

2016-03-03 Thread osstest service owner
flight 85178 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/85178/ 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. 85080 Tests which di

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

2016-03-03 Thread Wei Liu
On Tue, Mar 01, 2016 at 06:57:21PM +, Andrew Cooper wrote: > hvmloader is unhosted, and shouldn't use the system errno.h. It already has > to use Xen's errno.h for other hypercalls. The use of public/io/xs_wire.h > requires the use of un-prefixed errno values. > > This fixes the build on str

Re: [Xen-devel] [PATCH v3 16/16] hvmloader: do not depend on SEABIOS_PATH or OVMF_PATH ...

2016-03-03 Thread Wei Liu
On Thu, Feb 25, 2016 at 02:56:14PM +, Anthony PERARD wrote: > ... to compile SeaBIOS and OVMF. Only depends on CONFIG_*. > > If --with-system-* configure option is used, then set *_CONFIG=n to not > compile SEABIOS and OVMF. > > Signed-off-by: Anthony PERARD > Just notice this patch belong

Re: [Xen-devel] [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands

2016-03-03 Thread Paul Durrant
> -Original Message- > From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] > Sent: 02 March 2016 17:23 > To: Ian Jackson > Cc: Bob Liu; Juergen Gross; xen-devel@lists.xen.org; jbeul...@suse.com; > Roger Pau Monne; Paul Durrant > Subject: Re: [RFC PATCH] xen-block: introduces extra r

Re: [Xen-devel] [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands

2016-03-03 Thread Ian Jackson
Paul Durrant writes ("RE: [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands"): > Actually, that's quite desirable. The existing driver is actually a > SCSI HBA driver (because that's all you can write is you use thr > Windows STORPORT wrapper... which you need to if you

Re: [Xen-devel] [PATCH] libxl: introduce LIBXL_VGA_INTERFACE_TYPE_UNKNOWN

2016-03-03 Thread Wei Liu
On Wed, Mar 02, 2016 at 04:46:43PM +0100, Roger Pau Monne wrote: > And use it as the default value for the VGA kind. This allows libxl to set > it to the default value later on when the domain type is known. For HVM > guests the default value is LIBXL_VGA_INTERFACE_TYPE_CIRRUS while for > HVMlite t

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Wei Liu
On Thu, Mar 03, 2016 at 11:03:43AM +, Andrew Cooper wrote: [...] > > @@ -587,20 +578,21 @@ static void free_pinned_cacheattr_entry( > > xfree(container_of(rcu, struct hvm_mem_pinned_cacheattr_range, rcu)); > > } > > > > -int32_t hvm_set_mem_pinned_cacheattr( > > -struct domain *d, >

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Andrew Cooper
On 03/03/16 12:10, Wei Liu wrote: > On Thu, Mar 03, 2016 at 11:03:43AM +, Andrew Cooper wrote: > [...] >>> @@ -587,20 +578,21 @@ static void free_pinned_cacheattr_entry( >>> xfree(container_of(rcu, struct hvm_mem_pinned_cacheattr_range, rcu)); >>> } >>> >>> -int32_t hvm_set_mem_pinned_c

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Wei Liu
On Thu, Mar 03, 2016 at 12:12:40PM +, Andrew Cooper wrote: > On 03/03/16 12:10, Wei Liu wrote: > > On Thu, Mar 03, 2016 at 11:03:43AM +, Andrew Cooper wrote: > > [...] > >>> @@ -587,20 +578,21 @@ static void free_pinned_cacheattr_entry( > >>> xfree(container_of(rcu, struct hvm_mem_pinn

Re: [Xen-devel] [PATCH RFC 0/8] x86/hvm, libxl: HVM SMT topology support

2016-03-03 Thread Joao Martins
On 03/03/2016 10:24 AM, Andrew Cooper wrote: > On 03/03/16 09:52, Joao Martins wrote: >> > In particular, I am concerned about giving the toolstack the ability to > blindly control the APIC IDs. Their layout is very closely linked to > topology, and in particular to the HTT flag. >>>

Re: [Xen-devel] [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands

2016-03-03 Thread Juergen Gross
On 03/03/16 13:03, Ian Jackson wrote: > Paul Durrant writes ("RE: [RFC PATCH] xen-block: introduces extra request to > pass-through SCSI commands"): >> Actually, that's quite desirable. The existing driver is actually a >> SCSI HBA driver (because that's all you can write is you use thr >> Windows

Re: [Xen-devel] [PATCH 3/8] tools/xenalyze: Handle fstat errors properly

2016-03-03 Thread George Dunlap
On 26/02/16 12:25, Ian Jackson wrote: > George Dunlap writes ("[PATCH 3/8] tools/xenalyze: Handle fstat errors > properly"): >> They're pretty unlikely to fail, but doesn't hurt to check. > ... >> -fstat(fd, &s); >> +if ( fstat(fd, &s) ) { >> +perror("fstat"); >> +free(h);

Re: [Xen-devel] [PATCH v4 17/24] arm/acpi: Create min DT stub for Dom0

2016-03-03 Thread Shannon Zhao
On 2016/2/29 23:05, Stefano Stabellini wrote: >> +static int acpi_make_hypervisor_node(const struct kernel_info *kinfo, >> > + struct membank tbl_add[]) >> > +{ >> > +const char compat[] = >> > + >> > "xen,xen-"__stringify(XEN_VERSION)"."__stringify

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 12:03, wrote: > On 03/03/16 10:31, Jan Beulich wrote: >> - call hvm_get_mem_pinned_cacheattr() for RAM ranges only (requires >> some re-ordering in epte_get_entry_emt(), to fully handle all MMIO >> aspects first) - it's documented to be intended for RAM only >> - remove unne

Re: [Xen-devel] [PATCH 6/8] tools/xenalyze: Fix off-by-one in MAX_CPUS range checks

2016-03-03 Thread George Dunlap
On 29/02/16 16:58, George Dunlap wrote: > On 26/02/16 12:30, Ian Jackson wrote: >> George Dunlap writes ("[PATCH 6/8] tools/xenalyze: Fix off-by-one in >> MAX_CPUS range checks"): >>> Skip action / throw error if cpu/vcpu >= MAX_CPUS rather than >. >>> >>> Also add an assertion to vcpu_find, to m

Re: [Xen-devel] [PATCH 2/2] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 13:12, wrote: > On 03/03/16 12:10, Wei Liu wrote: >> On Thu, Mar 03, 2016 at 11:03:43AM +, Andrew Cooper wrote: >> [...] @@ -587,20 +578,21 @@ static void free_pinned_cacheattr_entry( xfree(container_of(rcu, struct hvm_mem_pinned_cacheattr_range, rcu));

Re: [Xen-devel] [PATCH] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Wei Liu
On Wed, Mar 02, 2016 at 08:51:51AM +0200, Razvan Cojocaru wrote: > Currently it's possible for xc_translate_foreign_address() to fail > and errno still be set to success. This patch fixes the issue. > Based on the first half of Don Slutz' patch: > http://lists.xen.org/archives/html/xen-devel/2014-0

Re: [Xen-devel] [PATCH RFC 0/8] x86/hvm, libxl: HVM SMT topology support

2016-03-03 Thread Andrew Cooper
On 03/03/16 12:23, Joao Martins wrote: > > On 03/03/2016 10:24 AM, Andrew Cooper wrote: >> On 03/03/16 09:52, Joao Martins wrote: >> In particular, I am concerned about giving the toolstack the ability to >> blindly control the APIC IDs. Their layout is very closely linked to >> topolo

Re: [Xen-devel] [PATCH] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Razvan Cojocaru
On 03/03/2016 02:47 PM, Wei Liu wrote: > On Wed, Mar 02, 2016 at 08:51:51AM +0200, Razvan Cojocaru wrote: >> Currently it's possible for xc_translate_foreign_address() to fail >> and errno still be set to success. This patch fixes the issue. >> Based on the first half of Don Slutz' patch: >> http:/

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

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 12:18, wrote: > flight 85178 xen-unstable-smoke real [real] > http://logs.test-lab.xenproject.org/osstest/logs/85178/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-armhf-armhf-xl 6 xen-boot

Re: [Xen-devel] [PATCH] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Wei Liu
On Thu, Mar 03, 2016 at 02:53:25PM +0200, Razvan Cojocaru wrote: > On 03/03/2016 02:47 PM, Wei Liu wrote: > > On Wed, Mar 02, 2016 at 08:51:51AM +0200, Razvan Cojocaru wrote: > >> Currently it's possible for xc_translate_foreign_address() to fail > >> and errno still be set to success. This patch f

[Xen-devel] [PATCH] xentrace: simplify map_page function in xenctx

2016-03-03 Thread Wei Liu
There were several problems: 1. Variable mapped was set to NULL so the following two "if"s were useless. 2. Variable previous_mfn was set but never used. Simplify the function by removing two "if"s and previous_mfn. Check if mfn == 0 instead. Finally remove the now unused out label. Wrapped

[Xen-devel] [PATCH V2] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Razvan Cojocaru
Fixes https://jira.bitdefender.biz/browse/HEIMDALL-71 Roughly based on Don Slutz' patch: http://lists.xen.org/archives/html/xen-devel/2014-03/msg03720.html Signed-off-by: Razvan Cojocaru --- Changes since V1: - Turns out that the only case that has occured in my tests where errno is not set

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

2016-03-03 Thread Shannon Zhao
On 2016/3/3 20:53, Jan Beulich wrote: On 03.03.16 at 12:18, wrote: >> > flight 85178 xen-unstable-smoke real [real] >> > http://logs.test-lab.xenproject.org/osstest/logs/85178/ >> > >> > Regressions :-( >> > >> > Tests which did not succeed and are blocking, >> > including tests which co

Re: [Xen-devel] [PATCH V2] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Razvan Cojocaru
On 03/03/2016 03:38 PM, Razvan Cojocaru wrote: > Fixes https://jira.bitdefender.biz/browse/HEIMDALL-71 > Roughly based on Don Slutz' patch: > http://lists.xen.org/archives/html/xen-devel/2014-03/msg03720.html > > Signed-off-by: Razvan Cojocaru > > --- > Changes since V1: > - Turns out that the

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

2016-03-03 Thread osstest service owner
flight 85190 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/85190/ 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. 85080 Tests which di

Re: [Xen-devel] [PATCH V2] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Wei Liu
On Thu, Mar 03, 2016 at 03:42:49PM +0200, Razvan Cojocaru wrote: > On 03/03/2016 03:38 PM, Razvan Cojocaru wrote: > > Fixes https://jira.bitdefender.biz/browse/HEIMDALL-71 > > Roughly based on Don Slutz' patch: > > http://lists.xen.org/archives/html/xen-devel/2014-03/msg03720.html > > > > Signed-o

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

2016-03-03 Thread Andrew Cooper
On 03/03/16 13:38, Shannon Zhao wrote: > > On 2016/3/3 20:53, Jan Beulich wrote: > On 03.03.16 at 12:18, wrote: flight 85178 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/85178/ Regressions :-( Tests which did not succeed and are

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

2016-03-03 Thread Shannon Zhao
On 2016/3/3 21:38, Shannon Zhao wrote: > > > On 2016/3/3 20:53, Jan Beulich wrote: > On 03.03.16 at 12:18, wrote: flight 85178 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/85178/ Regressions :-( Tests which did not succeed a

[Xen-devel] [PATCH V3] libxc: Have xc_translate_foreign_address() set errno properly

2016-03-03 Thread Razvan Cojocaru
Currently it's possible for xc_translate_foreign_address() to fail and errno still be set to success. This patch fixes the issue. Based on the first half of Don Slutz' patch: http://lists.xen.org/archives/html/xen-devel/2014-03/msg03720.html Signed-off-by: Razvan Cojocaru Acked-by: Wei Liu ---

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

2016-03-03 Thread Shannon Zhao
On 2016/3/3 21:54, Andrew Cooper wrote: > On 03/03/16 13:38, Shannon Zhao wrote: >> > >> > On 2016/3/3 20:53, Jan Beulich wrote: >> > On 03.03.16 at 12:18, wrote: > flight 85178 xen-unstable-smoke real [real] > http://logs.test-lab.xenproject.org/osstest/logs/85178/ >>

Re: [Xen-devel] [PATCH v7 00/11] grub-xen: support booting huge pv-domains

2016-03-03 Thread Daniel Kiper
On Thu, Mar 03, 2016 at 10:38:05AM +0100, Juergen Gross wrote: > The Xen hypervisor supports starting a dom0 with large memory (up to > the TB range) by not including the initrd and p2m list in the initial > kernel mapping. Especially the p2m list can grow larger than the > available virtual space

Re: [Xen-devel] [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands

2016-03-03 Thread Konrad Rzeszutek Wilk
On March 3, 2016 7:25:11 AM EST, Juergen Gross wrote: >On 03/03/16 13:03, Ian Jackson wrote: >> Paul Durrant writes ("RE: [RFC PATCH] xen-block: introduces extra >request to pass-through SCSI commands"): >>> Actually, that's quite desirable. The existing driver is actually a >>> SCSI HBA driver (b

[Xen-devel] [PATCH 0/1] ARM: Implement support for write-ctrlreg vm-events

2016-03-03 Thread Corneliu ZUZU
I've decided to include a cover-letter for this patch to ask for some feedback on a few matters (and also to detail the changes that were done here instead of writing that in the commit message). First, DETAILS OF CHANGES: == Moved to common-side: 1) monitor_ctrlreg_bitmask macro 2) hvm_even

[Xen-devel] [PATCH 1/1] arm/monitor vm-events: implement write-ctrlreg support

2016-03-03 Thread Corneliu ZUZU
This patch adds ARM support for write-ctrlreg monitor vm-events. The ARM control-registers that can be monitored are: - VM_EVENT_ARM_SCTLR: AArch32 SCTLR, AArch64 SCTLR_EL1 - VM_EVENT_ARM_TTBR{0,1}: AArch32 TTBR{0,1}, AArch64 TTBR{0,1}_EL1 - VM_EVENT_ARM_TTBCR: AArch32 TTBCR, AArch64

[Xen-devel] [PATCH] arm/timer: fix panic when booting with DT

2016-03-03 Thread Shannon Zhao
From: Shannon Zhao While to support ACPI, patch "arm/acpi: Parse GTDT to initialize timer" refactors the functions preinit_xen_time and init_xen_time. But it wrongly moves the platform_get_irq from init_xen_time to preinit_dt_xen_time and this will cause booting failure. So move platform_get_irq

[Xen-devel] [PATCH v3] xen/errno: Reduce complexity of inclusion

2016-03-03 Thread Andrew Cooper
The inclusion rules conditions for errno.h were unnecesserily complicated, and required the includer to jump through hoops if they wished to avoid getting multiple namespaces worth of constants. Simply the logic, and document what is going on. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC

Re: [Xen-devel] Maintainer Nomination of Julien Grall

2016-03-03 Thread Julien Grall
On 01/03/16 14:29, Stefano Stabellini wrote: Hi all, Hi Stefano, I would like to propose Julien Grall (julien.gr...@arm.com) as co-maintainer for Xen on ARM. His track record of contributions to the project is outstanding and I think speaks for itself. Julien made multiple public presentatio

Re: [Xen-devel] [RFC PATCH] xen-block: introduces extra request to pass-through SCSI commands

2016-03-03 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Konrad Rzeszutek Wilk > Sent: 03 March 2016 14:07 > To: Juergen Gross; Ian Jackson; Paul Durrant > Cc: xen-devel@lists.xen.org; jbeul...@suse.com; Roger Pau Monne > Subject: Re: [Xen-devel] [RFC P

[Xen-devel] [xen-4.6-testing baseline-only test] 44211: regressions - FAIL

2016-03-03 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44211 xen-4.6-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44211/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 15 guest-start

[Xen-devel] [linux-3.16 test] 85048: tolerable FAIL - PUSHED

2016-03-03 Thread osstest service owner
flight 85048 linux-3.16 real [real] http://logs.test-lab.xenproject.org/osstest/logs/85048/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemut-win7-amd64 16 guest-stop fail like 81206 build-i386-rumpuserxen6

Re: [Xen-devel] Maintainer Nomination of Julien Grall

2016-03-03 Thread Ian Jackson
Julien Grall writes ("Re: Maintainer Nomination of Julien Grall"): > On 01/03/16 14:29, Stefano Stabellini wrote: > > I would like to propose Julien Grall (julien.gr...@arm.com) as > > co-maintainer for Xen on ARM. His track record of contributions to the > > project is outstanding and I think spea

[Xen-devel] [PATCH] libxc: initialise rc to -1 at the beginning of meminit_hvm

2016-03-03 Thread Wei Liu
Variable rc is only set either inside a loop or inside some if statements. To avoid confuse gcc with stricter setting we set rc to -1 at the beginning. Reported-by: Doug Goldstein Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Roger Pau Monne --- tools/libxc/xc_dom_x86

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

2016-03-03 Thread Ian Jackson
Roger Pau Monne writes ("[PATCH v7] 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 chang

Re: [Xen-devel] [PATCH v3 04/16] firmware/makefile: install BIOS and ACPI blob ...

2016-03-03 Thread Anthony PERARD
On Mon, Feb 29, 2016 at 09:31:11AM -0700, Jan Beulich wrote: > >>> On 25.02.16 at 15:56, wrote: > > @@ -42,12 +42,19 @@ dsdt_%cpu.asl: dsdt.asl mk_dsdt > > awk 'NR > 1 {print s} {s=$$0}' $< > $@ > > ./mk_dsdt --debug=$(debug) --maxcpu $* >> $@ > > > > -$(filter dsdt_%.c,$(C_SRC)): %.c:

Re: [Xen-devel] [PATCH] arm/timer: fix panic when booting with DT

2016-03-03 Thread Stefano Stabellini
On Thu, 3 Mar 2016, Shannon Zhao wrote: > From: Shannon Zhao > > While to support ACPI, patch "arm/acpi: Parse GTDT to initialize timer" > refactors the functions preinit_xen_time and init_xen_time. But it > wrongly moves the platform_get_irq from init_xen_time to > preinit_dt_xen_time and this w

Re: [Xen-devel] [PATCH] libxc: initialise rc to -1 at the beginning of meminit_hvm

2016-03-03 Thread Ian Jackson
Wei Liu writes ("[PATCH] libxc: initialise rc to -1 at the beginning of meminit_hvm"): > Variable rc is only set either inside a loop or inside some if > statements. To avoid confuse gcc with stricter setting we set rc to -1 > at the beginning. Is this really the best way to fix this ? AFAICT th

Re: [Xen-devel] [PATCH] libxc: initialise rc to -1 at the beginning of meminit_hvm

2016-03-03 Thread Wei Liu
On Thu, Mar 03, 2016 at 03:46:23PM +, Ian Jackson wrote: > Wei Liu writes ("[PATCH] libxc: initialise rc to -1 at the beginning of > meminit_hvm"): > > Variable rc is only set either inside a loop or inside some if > > statements. To avoid confuse gcc with stricter setting we set rc to -1 > >

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Anthony PERARD
On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: > >>> On 25.02.16 at 15:56, wrote: > > --- a/tools/firmware/hvmloader/hvmloader.c > > +++ b/tools/firmware/hvmloader/hvmloader.c > > @@ -303,6 +303,15 @@ int main(void) > > > > smp_initialise(); > > > > +/* Check that tests

Re: [Xen-devel] [PATCH 0/1] ARM: Implement support for write-ctrlreg vm-events

2016-03-03 Thread Corneliu ZUZU
On 3/3/2016 4:10 PM, Corneliu ZUZU wrote: The patch was currently tested on an ARMv8 (CONFIG_ARM_64) machine (after applying a fix I'll send soon). Shannon beat me to it with the fix ;) See "[PATCH] arm/timer: fix panic when booting with DT". Corneliu. __

[Xen-devel] [PATCH] libxc: simplify error handling in meminit_hvm

2016-03-03 Thread Wei Liu
The hunk that prints out error message can only be reached from the loop that allocates memory. Move that hunk into the loop to avoid confusing gcc. Reported-by: Doug Goldstein Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Stefano Stabellini Cc: Roger Pau Monne --- tools/libxc/xc_dom_x86.c

Re: [Xen-devel] [PATCH 0/1] ARM: Implement support for write-ctrlreg vm-events

2016-03-03 Thread Razvan Cojocaru
On 03/03/2016 04:10 PM, Corneliu ZUZU wrote: > Q2) About VM_EVENT_FLAG_DENY > > Q2.1) > Doesn't it require sync = 1 (i.e. the vcpu to be paused) to work? > If so, shouldn't we call vm_event_register_write_resume only after > checking > that VM_EVENT_FLAG_VCPU_PAUSED flag is set (vm_

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Jan Beulich
>>> On 03.03.16 at 17:00, wrote: > On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: >> >>> On 25.02.16 at 15:56, wrote: >> > --- a/tools/firmware/hvmloader/hvmloader.c >> > +++ b/tools/firmware/hvmloader/hvmloader.c >> > @@ -303,6 +303,15 @@ int main(void) >> > >> > smp_initial

[Xen-devel] [xen-4.5-testing test] 85045: regressions - FAIL

2016-03-03 Thread osstest service owner
flight 85045 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85045/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 15 guest-localmigrate/x10 fail REGR. vs. 83135 Regressions

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

2016-03-03 Thread osstest service owner
flight 85200 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/85200/ 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. 85080 Tests which di

Re: [Xen-devel] [PATCH v3 08/16] hvmloader: Locate the BIOS blob

2016-03-03 Thread Anthony PERARD
On Mon, Feb 29, 2016 at 09:56:48AM -0700, Jan Beulich wrote: > >>> On 25.02.16 at 15:56, wrote: > > --- a/tools/firmware/hvmloader/hvmloader.c > > +++ b/tools/firmware/hvmloader/hvmloader.c > > @@ -254,10 +254,32 @@ static void acpi_enable_sci(void) > > BUG_ON(!(pm1a_cnt_val & ACPI_PM1C_SCI_E

Re: [Xen-devel] [PATCH v4 0/2] Clear .bss for PV guests

2016-03-03 Thread David Vrabel
On 26/02/16 19:02, Boris Ostrovsky wrote: > PV guests need to have their .bss zeroed out since it is not guaranteed > to be cleared by Xen's domain builder (even if it is done so currently) Applied to for-linus-4.6, thanks. David ___ Xen-devel mailing

Re: [Xen-devel] [PATCH v3 10/16] hvmloader: Load SeaBIOS from hvm_start_info modules

2016-03-03 Thread Anthony PERARD
On Mon, Feb 29, 2016 at 10:02:10AM -0700, Jan Beulich wrote: > >>> On 25.02.16 at 15:56, wrote: > > @@ -127,22 +124,29 @@ static void seabios_setup_e820(void) > > struct e820entry *e820 = scratch_alloc(sizeof(struct e820entry)*16, 0); > > info->e820 = (uint32_t)e820; > > > > +BUG_O

Re: [Xen-devel] [PATCH v4 RESEND] public/io/netif.h: make control ring hash protocol more general

2016-03-03 Thread David Vrabel
On 26/02/16 16:52, Paul Durrant wrote: > This patch modified the control ring protocol (of which there is > not yet an implementation) to make it more general. Most of the > concepts are not limited to toeplitz hashing so it's best not to > make them unnecessarily specific. > > Apart from changing

Re: [Xen-devel] [PATCH v3 01/16] libxc: Rework extra module initialisation

2016-03-03 Thread Anthony PERARD
On Tue, Mar 01, 2016 at 11:51:26AM +, Wei Liu wrote: > On Thu, Feb 25, 2016 at 02:55:59PM +, Anthony PERARD wrote: > > This patch use xc_dom_alloc_segment() to allocate the memory space for the > > ACPI modules and the SMBIOS modules. This is to replace the arbitrary > > placement of 1MB af

[Xen-devel] [PATCH 0/6] x86/HVM: cache attribute pinning adjustments

2016-03-03 Thread Jan Beulich
1: honor cache attribute pinning for RAM only 2: remove unnecessary indirection from hvm_get_mem_pinned_cacheattr() 3: adjust hvm_set_mem_pinned_cacheattr() error indications 4: limit flushing on cache attribute pinning adjustments 5: adjust hvm_get_mem_pinned_cacheattr() GFN parameter 6: re-format

[Xen-devel] [PATCH 1/6] x86/HVM: honor cache attribute pinning for RAM only

2016-03-03 Thread Jan Beulich
Call hvm_get_mem_pinned_cacheattr() for RAM ranges only, and only when the guest has a physical device assigned: XEN_DOMCTL_pin_mem_cacheattr is documented to be intended for RAM only. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/hvm/mtrr.c @@ -782,32 +782,29 @@ in

Re: [Xen-devel] [PATCH v3 09/16] hvmloader: Check modules whereabouts

2016-03-03 Thread Andrew Cooper
On 03/03/16 16:00, Anthony PERARD wrote: > On Mon, Feb 29, 2016 at 09:58:29AM -0700, Jan Beulich wrote: > On 25.02.16 at 15:56, wrote: >>> --- a/tools/firmware/hvmloader/hvmloader.c >>> +++ b/tools/firmware/hvmloader/hvmloader.c >>> @@ -303,6 +303,15 @@ int main(void) >>> >>> smp_initia

Re: [Xen-devel] [PATCH v4 RESEND] public/io/netif.h: make control ring hash protocol more general

2016-03-03 Thread Paul Durrant
> -Original Message- > From: David Vrabel [mailto:david.vra...@citrix.com] > Sent: 03 March 2016 16:26 > To: Paul Durrant; xen-de...@lists.xenproject.org > Cc: Wei Liu; Ian Jackson; David Vrabel; Jan Beulich > Subject: Re: [Xen-devel] [PATCH v4 RESEND] public/io/netif.h: make control > ring

[Xen-devel] [PATCH 2/6] x86/HVM: remove unnecessary indirection from hvm_get_mem_pinned_cacheattr()

2016-03-03 Thread Jan Beulich
Its return value can easily serve the purpose. We cannot, however, return unspecific "success" anymore for a domain of the wrong type - since no caller exists that would call this for PV domains, simply add an ASSERT(). Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/

[Xen-devel] [PATCH 3/6] x86/HVM: adjust hvm_set_mem_pinned_cacheattr() error indications

2016-03-03 Thread Jan Beulich
Make it return an error on bad domain kind or obviously bad GFN range. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/hvm/mtrr.c @@ -591,8 +591,11 @@ int32_t hvm_set_mem_pinned_cacheattr( struct hvm_mem_pinned_cacheattr_range *range; int rc = 1; -if (

[Xen-devel] [PATCH 4/6] x86/HVM: limit flushing on cache attribute pinning adjustments

2016-03-03 Thread Jan Beulich
Avoid cache flush on EPT when removing a UC- range, since when used this type gets converted to UC anyway (there's no UC- among the types valid in MTRRs and hence EPT's emt field). We might further wwant to consider only forcing write buffer flushes when removing WC ranges. Signed-off-by: Jan Beu

[Xen-devel] [PATCH 5/6] x86/HVM: adjust hvm_get_mem_pinned_cacheattr() GFN parameter

2016-03-03 Thread Jan Beulich
Make it gfn_t and rename it accordingly. Suggested-by: Andrew Cooper Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/hvm/mtrr.c @@ -545,7 +545,7 @@ void hvm_destroy_cacheattr_region_list( int hvm_get_mem_pinned_cacheattr( struct domain *d, -uint64_t guest

[Xen-devel] [PATCH 6/6] x86/HVM: re-format cache attribute pinning code

2016-03-03 Thread Jan Beulich
No intended functional change, albeit it includes ditching a redundant is_hvm_domain(). Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/hvm/mtrr.c @@ -521,14 +521,12 @@ struct hvm_mem_pinned_cacheattr_range { static DEFINE_RCU_READ_LOCK(pinned_cacheattr_rcu_lock);

Re: [Xen-devel] [BUG] xs.watch and xs.unwatch are unreliable

2016-03-03 Thread David Vrabel
On 01/03/16 20:17, Sergei Lebedev wrote: > Hi list, > > I’ve initially wanted to report another inconsistency in > ``xen.lowlevel.xs`` documentation, but this time the issue is more > subtle. [...] > Here’s another example with a string token > > >>> token1 = str(100

  1   2   3   >