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

2017-08-16 Thread osstest service owner
flight 112648 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/112648/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-4 47 xtf/test-hvm64-lbr-tsx-vmentry fail REGR. vs. 111514 Tests which

Re: [Xen-devel] [PATCH v1 06/13] x86: implement get value interface for MBA

2017-08-16 Thread Yi Sun
On 17-08-16 14:38:28, Chao Peng wrote: > On Wed, 2017-08-09 at 15:41 +0800, Yi Sun wrote: > > This patch implements get value domctl interface for MBA. > > > > Signed-off-by: Yi Sun > > --- > > ... > > > --- a/xen/include/public/domctl.h > > +++

Re: [Xen-devel] [PATCH 6/8] gnttab: move GNTPIN_* out of header file

2017-08-16 Thread Jan Beulich
>>> On 15.08.17 at 19:14, wrote: > On 15/08/17 15:41, Jan Beulich wrote: >> --- a/xen/common/grant_table.c >> +++ b/xen/common/grant_table.c >> @@ -158,7 +158,24 @@ shared_entry_header(struct grant_table * >> >> /* Active grant entry - used for shadowing

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

2017-08-16 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71978 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71978/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stop

Re: [Xen-devel] [PATCH v1 04/13] x86: implement data structure and CPU init flow for MBA

2017-08-16 Thread Yi Sun
On 17-08-15 11:50:15, Wei Liu wrote: > On Wed, Aug 09, 2017 at 03:41:43PM +0800, Yi Sun wrote: > > -#define PSR_CAT(1<<1) > > -#define PSR_CDP(1<<2) > > +#define PSR_CMT(1u << 0) > > +#define PSR_CAT(1u << 1) > > +#define PSR_CDP(1u << 2) > > +#define

Re: [Xen-devel] [PATCH v5] x86/hvm: Allow guest_request vm_events coming from userspace

2017-08-16 Thread Razvan Cojocaru
On 08/16/2017 02:16 AM, Tamas K Lengyel wrote: > On Tue, Aug 15, 2017 at 2:06 AM, Jan Beulich wrote: > On 14.08.17 at 17:53, wrote: >>> On Tue, Aug 8, 2017 at 2:27 AM, Alexandru Isaila >>> wrote: ---

[Xen-devel] [PATCH] x86: Skip check apic_id_limit for Xen

2017-08-16 Thread Lan Tianyu
Xen vIOMMU device model will be in Xen hypervisor. Skip vIOMMU check for Xen here when vcpu number is more than 255. Signed-off-by: Lan Tianyu --- hw/i386/pc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/i386/pc.c b/hw/i386/pc.c index

Re: [Xen-devel] [PATCH v1 06/13] x86: implement get value interface for MBA

2017-08-16 Thread Chao Peng
On Wed, 2017-08-09 at 15:41 +0800, Yi Sun wrote: > This patch implements get value domctl interface for MBA. > > Signed-off-by: Yi Sun > --- ... > --- a/xen/include/public/domctl.h > +++ b/xen/include/public/domctl.h > @@ -1144,6 +1144,7 @@ struct

Re: [Xen-devel] [PATCH 03/11] xen/arm: traps: Re-order the includes alphabetically

2017-08-16 Thread Bhupinder Thakur
Hi Julien, On 11 August 2017 at 23:32, Julien Grall wrote: > Signed-off-by: Julien Grall > --- > xen/arch/arm/traps.c | 42 ++ > 1 file changed, 22 insertions(+), 20 deletions(-) > > diff --git

Re: [Xen-devel] [XenSummit 2017] Shared coprocessor framework followup

2017-08-16 Thread Andrii Anisov
Hello Edgar, I'm just wondering if you have had a chance to play with SCF? Please do not hesitate to come up with questions and comments. We are extremely interested in them to make the thing better. -- *Andrii Anisov* ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH v3 1/2] gnttab: don't use possibly unbounded tail calls

2017-08-16 Thread Andrew Cooper
On 16/08/17 10:52, Jan Beulich wrote: On 15.08.17 at 17:04, wrote: >> On 15/08/17 14:49, Jan Beulich wrote: >>> @@ -2608,7 +2610,7 @@ static long gnttab_copy( >>> { >>> if ( i && hypercall_preempt_check() ) >>> { >>> -rc = i; >>>

[Xen-devel] [PATCH v4 2/2] gnttab: fix transitive grant handling

2017-08-16 Thread Jan Beulich
Processing of transitive grants must not use the fast path, or else reference counting breaks due to the skipped recursive call to __acquire_grant_for_copy() (its __release_grant_for_copy() counterpart occurs independent of original pin count). Furthermore after re-acquiring temporarily dropped

[Xen-devel] [PATCH v4 1/2] gnttab: don't use possibly unbounded tail calls

2017-08-16 Thread Jan Beulich
There is no guarantee that the compiler would actually translate them to branches instead of calls, so only ones with a known recursion limit are okay: - __release_grant_for_copy() can call itself only once, as __acquire_grant_for_copy() won't permit use of multi-level transitive grants, -

Re: [Xen-devel] [patch v6] vt-d: fix vf of rc integrated pf matched to wrong vt-d unit

2017-08-16 Thread Roger Pau Monné
on wed, aug 16, 2017 at 01:12:24pm +0800, chao gao wrote: > the problem is for a vf of rc integrated pf (e.g. pf's bdf is > 00:02.0), we would wrongly use 00:00.0 to search vt-d unit. > > if a pf is an extended function, the bdf of a traditional function > within the same device should be used to

Re: [Xen-devel] [patch v6] vt-d: fix vf of rc integrated pf matched to wrong vt-d unit

2017-08-16 Thread Roger Pau Monné
On Wed, Aug 16, 2017 at 04:50:51PM +0800, Chao Gao wrote: > On Wed, Aug 16, 2017 at 09:17:46AM +0100, Roger Pau Monné wrote: > >on wed, aug 16, 2017 at 01:12:24pm +0800, chao gao wrote: > >> the problem is for a vf of rc integrated pf (e.g. pf's bdf is > >> 00:02.0), we would wrongly use 00:00.0

Re: [Xen-devel] [PATCH for-2.10 v2 1/2] hw/acpi: Call acpi_set_pci_info when no ACPI tables needed

2017-08-16 Thread Igor Mammedov
On Tue, 15 Aug 2017 22:24:08 +0300 "Michael S. Tsirkin" wrote: > On Tue, Aug 15, 2017 at 02:07:51PM +0200, Igor Mammedov wrote: > > On Tue, 15 Aug 2017 12:15:48 +0100 > > Anthony PERARD wrote: > > > > > To do PCI passthrough with Xen, the property

Re: [Xen-devel] [PATCH 07/11] xen/arm: traps: Export a bunch of helpers to handle emulation

2017-08-16 Thread Bhupinder Thakur
Hi Julien, On 11 August 2017 at 23:32, Julien Grall wrote: > A follow-up patch will move some parts of traps.c in separate files. > The will require to use helpers that are currently statically defined. > Export the following helpers: > - inject_undef64_exception >

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

2017-08-16 Thread osstest service owner
flight 112649 xen-4.8-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/112649/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-vhd 15 guest-start/debian.repeat fail REGR. vs. 111538 Regressions

Re: [Xen-devel] [PATCH v3 1/2] gnttab: don't use possibly unbounded tail calls

2017-08-16 Thread Jan Beulich
>>> On 15.08.17 at 17:04, wrote: > On 15/08/17 14:49, Jan Beulich wrote: >> @@ -2608,7 +2610,7 @@ static long gnttab_copy( >> { >> if ( i && hypercall_preempt_check() ) >> { >> -rc = i; >> +rc = count - i; > > Somewhere,

Re: [Xen-devel] [PATCH v8 08/13] arm/guest_access: Move vgic_access_guest_memory to guest_access.h

2017-08-16 Thread Sergej Proskurin
Hi Julien, On 08/09/2017 10:20 AM, Sergej Proskurin wrote: > This commit moves the function vgic_access_guest_memory to guestcopy.c > and the header asm/guest_access.h. No functional changes are made. > Please note that the function will be renamed in the following commit. > > Signed-off-by:

Re: [Xen-devel] [PATCH v8 08/13] arm/guest_access: Move vgic_access_guest_memory to guest_access.h

2017-08-16 Thread Julien Grall
On 16/08/17 10:58, Sergej Proskurin wrote: Hi Julien, On 08/09/2017 10:20 AM, Sergej Proskurin wrote: This commit moves the function vgic_access_guest_memory to guestcopy.c and the header asm/guest_access.h. No functional changes are made. Please note that the function will be renamed in

Re: [Xen-devel] [PATCH v8 11/13] arm/mem_access: Add long-descriptor based gpt

2017-08-16 Thread Sergej Proskurin
Hi all, On 08/16/2017 12:28 AM, Andrew Cooper wrote: > On 15/08/2017 23:25, Stefano Stabellini wrote: >> On Tue, 15 Aug 2017, Julien Grall wrote: >>> On 14/08/17 22:03, Sergej Proskurin wrote: Hi Julien, On 08/14/2017 07:37 PM, Julien Grall wrote: > Hi Sergej, > > On

[Xen-devel] [PATCH v4 0/2] XSA-226

2017-08-16 Thread Jan Beulich
XSA-226 went out with just a workaround patch. The pair of patches here became ready too late to be reasonably included in the XSA. Nevertheless they aim at fixing the underlying issues, ideally making the workaround unnecessary. 1: gnttab: don't use possibly unbounded tail calls 2: gnttab: fix

Re: [Xen-devel] [patch v6] vt-d: fix vf of rc integrated pf matched to wrong vt-d unit

2017-08-16 Thread Chao Gao
On Wed, Aug 16, 2017 at 09:17:46AM +0100, Roger Pau Monné wrote: >on wed, aug 16, 2017 at 01:12:24pm +0800, chao gao wrote: >> the problem is for a vf of rc integrated pf (e.g. pf's bdf is >> 00:02.0), we would wrongly use 00:00.0 to search vt-d unit. >> >> if a pf is an extended function, the

Re: [Xen-devel] [PATCH v3 1/2] gnttab: don't use possibly unbounded tail calls

2017-08-16 Thread Jan Beulich
>>> On 16.08.17 at 12:01, wrote: > On 16/08/17 10:52, Jan Beulich wrote: > On 15.08.17 at 17:04, wrote: >>> On 15/08/17 14:49, Jan Beulich wrote: @@ -2608,7 +2610,7 @@ static long gnttab_copy( { if ( i &&

Re: [Xen-devel] [xen-unstable test] 112618: regressions - trouble: blocked/broken/fail/pass

2017-08-16 Thread Julien Grall
Hi Wei, On 15/08/17 16:08, Wei Liu wrote: On Mon, Aug 14, 2017 at 06:58:13PM +0100, Julien Grall wrote: Hi, On 14/08/17 00:20, osstest service owner wrote: flight 112618 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/112618/ Regressions :-( Tests which did not

Re: [Xen-devel] [PATCH v4 1/2] gnttab: don't use possibly unbounded tail calls

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:20, Jan Beulich wrote: > There is no guarantee that the compiler would actually translate them > to branches instead of calls, so only ones with a known recursion limit > are okay: > - __release_grant_for_copy() can call itself only once, as > __acquire_grant_for_copy() won't

Re: [Xen-devel] [PATCH 07/11] xen/arm: traps: Export a bunch of helpers to handle emulation

2017-08-16 Thread Julien Grall
On 16/08/17 10:30, Bhupinder Thakur wrote: Hi Julien, Hi Bhupinder, On 11 August 2017 at 23:32, Julien Grall wrote: diff --git a/xen/include/asm-arm/arm64/traps.h b/xen/include/asm-arm/arm64/traps.h new file mode 100644 index 00..5cb45df6bd --- /dev/null

[Xen-devel] [distros-debian-squeeze test] 71980: tolerable trouble: broken/fail/pass

2017-08-16 Thread Platform Team regression test user
flight 71980 distros-debian-squeeze real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71980/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: build-arm64 2 hosts-allocate broken like 71954 build-arm64-pvops

Re: [Xen-devel] [PATCH] x86/asm: add .file directives

2017-08-16 Thread Andrew Cooper
On 16/08/17 13:13, Jan Beulich wrote: > Make sure local symbols are correctly associated with their source > files: I've just run across a cpufreq.c#create_bounce_frame stack trace > entry. Since we have multiple entry.S, don't use __FILE__ there to > fully disambiguate things. > > Signed-off-by:

[Xen-devel] [PATCH] x86/mm: Drop more PV superpage leftovers

2017-08-16 Thread Andrew Cooper
Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Wei Liu --- xen/include/asm-x86/x86_64/page.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/xen/include/asm-x86/x86_64/page.h

[Xen-devel] [PATCH] x86/mm: Drop __PAGE_OFFSET

2017-08-16 Thread Andrew Cooper
It is a vestigial leftover of Xen having inherited Linux's memory management code in the early days. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Wei Liu --- xen/include/asm-x86/x86_64/page.h | 1 - 1 file changed, 1

[Xen-devel] [PATCH v3 39/52] xen: check parameter validity when parsing command line

2017-08-16 Thread Juergen Gross
Where possible check validity of parameters in _cmdline_parse() and issue a warning message in case of an error detected. In order to make sure a custom parameter parsing function really returns a value (error or success), don't use a void pointer for storing the function address, but a proper

[Xen-devel] [PATCH v3 41/52] xen/arch/x86/cpu/mcheck/mce.c: remove custom_param() error messages

2017-08-16 Thread Juergen Gross
With _cmdline_parse() now issuing error messages in case of illegal parameters signalled by parsing functions specified in custom_param() the message issued by mce_set_verbosity() can be removed. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by:

[Xen-devel] [PATCH v3 33/52] xen/drivers/passthrough/iommu.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/passthrough/iommu.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Signed-off-by: Juergen Gross --- V3: - dont modify option value in parsing function ---

[Xen-devel] [PATCH v3 50/52] libxl: add libxl_set_parameters() function

2017-08-16 Thread Juergen Gross
Add a new libxl function to set hypervisor parameters at runtime similar to boot time parameters via command line. Cc: Ian Jackson Cc: Wei Liu Signed-off-by: Juergen Gross --- V2: - corrected coding style (Wei Liu) - removed

[Xen-devel] [PATCH v3 07/52] xen/arch/x86/dom0_build.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/dom0_build.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross --- V3: - cosmetic changes

[Xen-devel] [PATCH v3 26/52] xen/common/kexec.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/kexec.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu --- xen/common/kexec.c | 30

[Xen-devel] [PATCH v3 49/52] libxc: add function to set hypervisor parameters

2017-08-16 Thread Juergen Gross
Add a new libxc function to set hypervisor parameters at runtime similar to boot time parameters via command line. Cc: Ian Jackson Cc: Wei Liu Signed-off-by: Juergen Gross --- V3: - zero padding fields in sysctl.u.set_parameter

[Xen-devel] [PATCH v3 20/52] xen/arch/x86/shutdown.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/shutdown.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross --- V3: - dont stop loop at

[Xen-devel] [PATCH v3 03/52] xen/arch/arm/traps.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/arm/traps.c to indicate whether the parameter value was parsed successfully. Cc: Stefano Stabellini Cc: Julien Grall Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 52/52] xen: make some console related parameters settable at runtime

2017-08-16 Thread Juergen Gross
Support modifying conswitch, console_timestamps, loglvl and guest_loglvl at runtime. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Konrad Rzeszutek Wilk

[Xen-devel] [PATCH v3 09/52] xen/arch/x86/hvm/viridian.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/hvm/viridian.c to indicate whether the parameter value was parsed successfully. Fix an error in the parsing function: up to now it would overwrite the stack in case more than 3 values are specified. Cc: Paul Durrant

[Xen-devel] [PATCH v3 51/52] xl: add new xl command set-parameters

2017-08-16 Thread Juergen Gross
Add a new xl command "set-parameters" to set hypervisor parameters at runtime similar to boot time parameters via command line. Cc: Ian Jackson Cc: Wei Liu Signed-off-by: Juergen Gross Acked-by: Wei Liu ---

[Xen-devel] [PATCH v3 06/52] xen/arch/x86/cpu/vpmu.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/cpu/vpmu.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 05/52] xen/arch/x86/cpu/mcheck/mce.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/cpu/mcheck/mce.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 43/52] xen/arch/x86/io_apic.c: remove custom_param() error messages

2017-08-16 Thread Juergen Gross
With _cmdline_parse() now issuing error messages in case of illegal parameters signalled by parsing functions specified in custom_param() the message issued by setup_ioapic_ack() can be removed. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by:

[Xen-devel] [PATCH v3 48/52] xen: add hypercall for setting parameters at runtime

2017-08-16 Thread Juergen Gross
Add a sysctl hypercall to support setting parameters similar to command line parameters, but at runtime. The parameters to set are specified as a string, just like the boot parameters. Cc: Daniel De Graaf Cc: Ian Jackson Cc: Wei Liu

[Xen-devel] [PATCH v3 31/52] xen/drivers/cpufreq/cpufreq.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/cpufreq/cpufreq.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Signed-off-by: Juergen Gross --- V3: - dont modify option value in handling function - remove

[Xen-devel] [PATCH v3 23/52] xen/common/core_parking.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/core_parking.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

[Xen-devel] [PATCH v3 27/52] xen/common/memory.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/memory.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

[Xen-devel] [PATCH v3 08/52] xen/arch/x86/genapic/probe.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/genapic/probe.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross --- V3: - dont return

[Xen-devel] [PATCH v3 37/52] xen/drivers/video/vesa.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/video/vesa.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

[Xen-devel] [PATCH v2 5/7] gnttab: drop struct active_grant_entry's gfn field for release builds

2017-08-16 Thread Jan Beulich
This shrinks the size from 48 to 40 bytes bytes on 64-bit builds. Switch to gfn_t at once. Signed-off-by: Jan Beulich --- v2: Re-base. Use inline function instead of macro. Switch to gfn_t. --- a/xen/common/grant_table.c +++ b/xen/common/grant_table.c @@ -185,7 +185,9 @@

[Xen-devel] [PATCH v2 3/7] gnttab: drop pointless leading double underscores

2017-08-16 Thread Jan Beulich
They're violating name space rules, and we don't really need them. When followed by "gnttab_", also drop that. Signed-by: Jan Beulich Reviewed-by: Andrew Cooper --- v2: Re-base. Minor formatting adjustment. --- a/xen/common/grant_table.c +++

[Xen-devel] [PATCH v2 2/7] gnttab: type adjustments

2017-08-16 Thread Jan Beulich
In particular use grant_ref_t and grant_handle_t where appropriate. Also switch other nearby type uses to their canonical variants where appropriate and introduce INVALID_MAPTRACK_HANDLE. Signed-by: Jan Beulich Reviewed-by: Andrew Cooper --- v2:

[Xen-devel] [PATCH v2 1/4] x86/mcheck: Minor cleanup to amd_nonfatal

2017-08-16 Thread Andrew Cooper
* Drop trailing whitespace. * Move amd_nonfatal_mcheck_init() into .init.text and drop a trailing return. * Drop unnecessary wmb()'s. Because of Xen's implementation, they are only compiler barriers anyway, and each wrmsr() is already fully serialising. Signed-off-by: Andrew Cooper

Re: [Xen-devel] [RFC PATCH v2 17/22] ARM: vGIC: introduce vgic_lock_vcpu_irq()

2017-08-16 Thread Julien Grall
Hi Andre, On 21/07/17 21:00, Andre Przywara wrote: Since a VCPU can own multiple IRQs, the natural locking order is to take a VCPU lock first, then the individual per-IRQ locks. However there are situations where the target VCPU is not known without looking into the struct pending_irq first,

Re: [Xen-devel] [PATCH v2 1/7] gnttab: avoid spurious maptrack handle allocation failures

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:48, Jan Beulich wrote: > When no memory is available in the hypervisor, rather than immediately > failing the request, try to steal a handle from another vCPU. > > Reported-by: George Dunlap > Signed-off-by: Jan Beulich Reviewed-by:

Re: [Xen-devel] [PATCH v2 4/7] gnttab: re-arrange struct active_grant_entry

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:49, Jan Beulich wrote: > While benign to 32-bit arches, this shrinks the size from 56 to 48 > bytes on 64-bit ones (while still leaving a 16-bit hole). > > Take the opportunity and consistently use bool/true/false for all > is_sub_page uses. > > Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH v2 5/7] gnttab: drop struct active_grant_entry's gfn field for release builds

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:50, Jan Beulich wrote: > This shrinks the size from 48 to 40 bytes bytes on 64-bit builds. > Switch to gfn_t at once. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel

Re: [Xen-devel] [PATCH v2 7/7] gnttab: properly handle transfer slots when changing version

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:51, Jan Beulich wrote: > Reported-by: Andrew Cooper > Signed-off-by: Jan Beulich > --- > v2: New. > --- > Or should we put ourselves on the position that reserved entries aren#t > meant to be used for transfers? Transfers are only

[Xen-devel] [PATCH] x86/asm: add .file directives

2017-08-16 Thread Jan Beulich
Make sure local symbols are correctly associated with their source files: I've just run across a cpufreq.c#create_bounce_frame stack trace entry. Since we have multiple entry.S, don't use __FILE__ there to fully disambiguate things. Signed-off-by: Jan Beulich ---

[Xen-devel] [PATCH v3] passthrough: give XEN_DOMCTL_test_assign_device more sane semantics

2017-08-16 Thread Jan Beulich
So far callers of the libxc interface passed in a domain ID which was then ignored in the hypervisor. Instead, make the hypervisor honor it (accepting DOMID_INVALID to obtain original behavior), allowing to query whether a device can be assigned to a particular domain. Drop XSM's

Re: [Xen-devel] [PATCH v8 08/13] arm/guest_access: Move vgic_access_guest_memory to guest_access.h

2017-08-16 Thread Sergej Proskurin
On 08/16/2017 12:11 PM, Julien Grall wrote: > > > On 16/08/17 10:58, Sergej Proskurin wrote: >> Hi Julien, >> >> >> On 08/09/2017 10:20 AM, Sergej Proskurin wrote: >>> This commit moves the function vgic_access_guest_memory to guestcopy.c >>> and the header asm/guest_access.h. No functional

[Xen-devel] [PATCH v3 00/52] Support for modifying parameters at runtime

2017-08-16 Thread Juergen Gross
Currently parameters of the hypervisor (e.g. console log level) can be set via boot command line. Instead of having to reboot the system in case another setting is desired, being able to modify many of those parameters at runtime would be the better option. This patch series addresses this by

[Xen-devel] [PATCH v3 42/52] xen/arch/x86/hvm/viridian.c: remove custom_param() error messages

2017-08-16 Thread Juergen Gross
With _cmdline_parse() now issuing error messages in case of illegal parameters signalled by parsing functions specified in custom_param() the message issued by parse_viridian_version() can be removed. Cc: Paul Durrant Cc: Jan Beulich Cc: Andrew Cooper

[Xen-devel] [PATCH v3 34/52] xen/drivers/passthrough/pci.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/passthrough/pci.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Signed-off-by: Juergen Gross --- V3: - cosmetic changes (Jan Beulich) - dont modify option value

[Xen-devel] [PATCH v3 19/52] xen/arch/x86/setup.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/setup.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 36/52] xen/drivers/passthrough/vtd/quirks.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/passthrough/vtd/quirks.c to indicate whether the parameter value was parsed successfully. Cc: Kevin Tian Signed-off-by: Juergen Gross Acked-by: Wei Liu ---

[Xen-devel] [PATCH v3 16/52] xen/arch/x86/numa.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/numa.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 40/52] xen/arch/x86/apic.c: remove custom_param() error messages

2017-08-16 Thread Juergen Gross
With _cmdline_parse() now issuing error messages in case of illegal parameters signalled by parsing functions specified in custom_param() the message issued by apic_set_verbosity() can be removed. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by:

[Xen-devel] [PATCH v3 13/52] xen/arch/x86/microcode.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/microcode.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 46/52] xen: carve out a generic parsing function from _cmdline_parse()

2017-08-16 Thread Juergen Gross
In order to support generic parameter parsing carve out the parser from _cmdline_parse(). As this generic function might be called after boot remove the __init annotations from all called sub-functions. Cc: Andrew Cooper Cc: George Dunlap

[Xen-devel] [PATCH v3 32/52] xen/drivers/passthrough/amd/iommu_acpi.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/passthrough/amd/iommu_acpi.c to indicate whether the parameter value was parsed successfully. Cc: Suravee Suthikulpanit Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 02/52] xen/arch/arm/domain_build.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/arm/domain_build.c to indicate whether the parameter value was parsed successfully. Cc: Stefano Stabellini Cc: Julien Grall Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 11/52] xen/arch/x86/io_apic.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/io_apic.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 17/52] xen/arch/x86/oprofile/nmi_int.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/oprofile/nmi_int.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 15/52] xen/arch/x86/nmi.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/nmi.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 14/52] xen/arch/x86/mm.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/mm.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 38/52] xen/xsm/flask/flask_op.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/xsm/flask/flask_op.c to indicate whether the parameter value was parsed successfully. Cc: Daniel De Graaf Signed-off-by: Juergen Gross Acked-by: Daniel De Graaf Acked-by: Wei

[Xen-devel] [PATCH v3 30/52] xen/drivers/char/console.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/char/console.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

[Xen-devel] [PATCH v3 25/52] xen/common/efi/boot.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/efi/boot.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Signed-off-by: Juergen Gross --- V3: - dont return out of loops (Jan Beulich) - dont modify option value

[Xen-devel] [PATCH v3 10/52] xen/arch/x86/hvm/vmx/vmcs.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/hvm/vmx/vmcs.c to indicate whether the parameter value was parsed successfully. Cc: Jun Nakajima Cc: Kevin Tian Cc: Jan Beulich Cc: Andrew Cooper

[Xen-devel] [PATCH v3 01/52] xen/arch/arm/acpi/boot.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/arm/acpi/boot.c to indicate whether the parameter value was parsed successfully. Cc: Stefano Stabellini Cc: Julien Grall Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 47/52] xen: add basic support for runtime parameter changing

2017-08-16 Thread Juergen Gross
Add the needed infrastructure for runtime parameter changing similar to that used at boot time via cmdline. We are using the same parsing functions as for cmdline parsing, but with a different array of parameter definitions. Cc: Andrew Cooper Cc: George Dunlap

[Xen-devel] [PATCH v3 44/52] xen/common/kexec.c: remove custom_param() error messages

2017-08-16 Thread Juergen Gross
With _cmdline_parse() now issuing error messages in case of illegal parameters signalled by parsing functions specified in custom_param() some messages issued by parse_low_crashinfo() and parse_crashinfo_maxaddr() can be removed. Cc: Andrew Cooper Signed-off-by:

[Xen-devel] [PATCH v3 29/52] xen/drivers/acpi/tables.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/acpi/tables.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Signed-off-by: Juergen Gross Acked-by: Wei Liu Acked-by: Jan Beulich

[Xen-devel] [PATCH v3 24/52] xen/common/domain.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/domain.c to indicate whether the parameter value was parsed successfully. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

[Xen-devel] [PATCH v3 04/52] xen/arch/x86/apic.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/apic.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 18/52] xen/arch/x86/psr.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/psr.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross --- V3: - let parse_psr_bool()

[Xen-devel] [PATCH v3 22/52] xen/arch/x86/x86_64/mmconfig-shared.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/x86_64/mmconfig-shared.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross --- V3: - dont

[Xen-devel] [PATCH v3 28/52] xen/common/sched_credit2.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/common/sched_credit2.c to indicate whether the parameter value was parsed successfully. Cc: George Dunlap Cc: Dario Faggioli Signed-off-by: Juergen Gross Acked-by:

[Xen-devel] [PATCH v3 21/52] xen/arch/x86/time.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/time.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v3 35/52] xen/drivers/passthrough/vtd/dmar.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/drivers/passthrough/vtd/dmar.c to indicate whether the parameter value was parsed successfully. Cc: Kevin Tian Signed-off-by: Juergen Gross Acked-by: Wei Liu ---

[Xen-devel] [PATCH v3 12/52] xen/arch/x86/irq.c: let custom parameter parsing routines return errno

2017-08-16 Thread Juergen Gross
Modify the custom parameter parsing routines in: xen/arch/x86/irq.c to indicate whether the parameter value was parsed successfully. Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross Acked-by: Wei Liu

[Xen-devel] [PATCH v2 0/7] gnttab: recent XSA follow-up

2017-08-16 Thread Jan Beulich
1: avoid spurious maptrack handle allocation failures 2: type adjustments 3: drop pointless leading double underscores 4: re-arrange struct active_grant_entry 5: drop struct active_grant_entry's gfn field for release builds 6: clean up main switch in do_grant_table_op() 7: properly handle transfer

Re: [Xen-devel] [PATCH 03/11] xen/arm: traps: Re-order the includes alphabetically

2017-08-16 Thread Julien Grall
Hi Bhupinder, On 16/08/17 07:44, Bhupinder Thakur wrote: Hi Julien, On 11 August 2017 at 23:32, Julien Grall wrote: Signed-off-by: Julien Grall --- xen/arch/arm/traps.c | 42 ++ 1 file changed, 22

[Xen-devel] [xen-4.9-testing baseline-only test] 71979: regressions - trouble: blocked/broken/fail/pass

2017-08-16 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71979 xen-4.9-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71979/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run:

Re: [Xen-devel] [PATCH v2 6/7] gnttab: clean up main switch in do_grant_table_op()

2017-08-16 Thread Andrew Cooper
On 16/08/17 11:50, Jan Beulich wrote: > Add blank lines as necessary and drop unnecessary braces. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list

  1   2   3   >