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

2017-08-23 Thread Juergen Gross
; >> Cc: Jan Beulich <jbeul...@suse.com> >> Cc: Andrew Cooper <andrew.coop...@citrix.com> >> Signed-off-by: Juergen Gross <jgr...@suse.com> > > Reviewed-by: Jan Beulich <jbeul...@suse.com> This patch suffered from the parse_bool() problem, too. Juergen

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 23/08/17 11:41, Jan Beulich wrote: On 23.08.17 at 11:19, wrote: >> Just some thoughts on that topic from my side: >> >> This approach should be fine IMO for memory allocated while creating a >> domain. This is basically the same as a bare metal system where the BIOS >>

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 23/08/17 11:28, Paul Durrant wrote: >> -Original Message- >> From: Juergen Gross [mailto:jgr...@suse.com] >> Sent: 23 August 2017 10:23 >> To: Paul Durrant <paul.durr...@citrix.com>; 'Jan Beulich' >> <jbeul...@suse.com> >> Cc: Tim (Xen.

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 23/08/17 11:43, Jan Beulich wrote: On 23.08.17 at 11:23, wrote: >> On 23/08/17 10:47, Paul Durrant wrote: >>> Making a the number of grant frames a per-vm-configurable quantity would >> seem like a reasonable first step. I'm not convinced of the need for >> separate >>

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

2017-08-23 Thread Juergen Gross
On 22/08/17 13:31, Jan Beulich wrote: On 16.08.17 at 14:52, wrote: >> --- a/xen/common/sysctl.c >> +++ b/xen/common/sysctl.c >> @@ -467,6 +467,42 @@ long do_sysctl(XEN_GUEST_HANDLE_PARAM(xen_sysctl_t) >> u_sysctl) >> copyback = 1; >> break; >> >> +

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

2017-08-23 Thread Juergen Gross
On 23/08/17 11:37, Jan Beulich wrote: On 23.08.17 at 11:27, wrote: >> On 22/08/17 12:04, Jan Beulich wrote: >> On 16.08.17 at 14:52, wrote: @@ -89,44 +89,50 @@ static void __init parse_iommu_param(char *s) s += 3;

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

2017-08-23 Thread Juergen Gross
On 22/08/17 13:24, Jan Beulich wrote: On 16.08.17 at 14:52, wrote: >> static void __init _cmdline_parse(const char *cmdline) >> { >> char opt[128], *optval, *optkey, *q; >> -const char *p = cmdline; >> +const char *p = cmdline, *s, *key; >> const struct

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

2017-08-23 Thread Juergen Gross
On 22/08/17 12:07, Jan Beulich wrote: On 16.08.17 at 14:52, wrote: >> --- a/xen/drivers/passthrough/pci.c >> +++ b/xen/drivers/passthrough/pci.c >> @@ -149,17 +149,18 @@ static struct phantom_dev { >> } phantom_devs[8]; >> static unsigned int nr_phantom_devs; >> >>

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

2017-08-23 Thread Juergen Gross
On 22/08/17 12:04, Jan Beulich wrote: On 16.08.17 at 14:52, wrote: >> @@ -89,44 +89,50 @@ static void __init parse_iommu_param(char *s) >> s += 3; >> >> ss = strchr(s, ','); >> -if ( ss ) >> -*ss = '\0'; >> - >> -if (

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 23/08/17 10:47, Paul Durrant wrote: >> -Original Message- >> From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of Jan >> Beulich >> Sent: 23 August 2017 09:36 >> To: Juergen Gross <jgr...@suse.com> >> Cc: Tim (Xen.org) <t

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 23/08/17 10:36, Jan Beulich wrote: On 23.08.17 at 09:49, wrote: >> On 22/08/17 14:48, Jan Beulich wrote: >> On 21.08.17 at 20:05, wrote: Currently Linux has no support for grant v2 as this would reduce the maximum number of active grants by

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

2017-08-23 Thread Juergen Gross
On 22/08/17 11:53, Jan Beulich wrote: On 16.08.17 at 14:51, wrote: >> --- a/xen/arch/x86/shutdown.c >> +++ b/xen/arch/x86/shutdown.c >> @@ -51,8 +51,11 @@ static int reboot_mode; >> * efiUse the EFI reboot (if running under EFI) >> */ >> static enum reboot_type

Re: [Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-23 Thread Juergen Gross
On 22/08/17 14:48, Jan Beulich wrote: On 21.08.17 at 20:05, wrote: >> Currently Linux has no support for grant v2 as this would reduce the >> maximum number of active grants by a factor of 2 compared to v1, >> because the number of possible grants are limited by the allowed

[Xen-devel] [PATCH 3/5] xen: clean up grant_table.h

2017-08-21 Thread Juergen Gross
Many definitions can be moved from xen/grant_table.h to common/grant_table.c now, as they are no longer used in other sources. Signed-off-by: Juergen Gross <jgr...@suse.com> --- xen/common/grant_table.c | 81 +++- xen/include/xen/grant_table.

[Xen-devel] [PATCH 5/5] xen: add new hypercall to get grant table limits

2017-08-21 Thread Juergen Gross
, GNTTABOP_get_v1_and_v2_max), let the new hypercall return all the needed information. Signed-off-by: Juergen Gross <jgr...@suse.com> --- xen/common/grant_table.c | 26 ++ xen/include/public/grant_table.h | 14 ++ 2 files changed, 40 insertions(+) diff

[Xen-devel] [PATCH 1/5] xen: remove deprecated boot parameter gnttab_max_nr_frames

2017-08-21 Thread Juergen Gross
The boot parameter gnttab_max_nr_frames has been deprecated in Xen 4.5. Remove it now. Signed-off-by: Juergen Gross <jgr...@suse.com> --- xen/common/grant_table.c | 19 +-- 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/xen/common/grant_table.c b/xen/

[Xen-devel] [PATCH 0/5] xen: better grant v2 support

2017-08-21 Thread Juergen Gross
frame numbers for v1 and v2. In order to let the guest know about the actual numbers an interface for probing the v1 and v2 limits is added. Juergen Gross (5): xen: remove deprecated boot parameter gnttab_max_nr_frames xen: move XENMAPSPACE_grant_table code into grant_table.c xen: clean up

[Xen-devel] [PATCH 4/5] xen: support different gnttab_max_frames for grant v1 and v2

2017-08-21 Thread Juergen Gross
values separated by a comma to set both limits to dedicated values. Add some sanity checks to make sure the maximum number of frames isn't lower than the initial number, as this leads to rather strange crashes. Signed-off-by: Juergen Gross <jgr...@suse.com> --- xen/arch/arm/do

[Xen-devel] [PATCH 2/5] xen: move XENMAPSPACE_grant_table code into grant_table.c

2017-08-21 Thread Juergen Gross
-off-by: Juergen Gross <jgr...@suse.com> --- xen/arch/arm/mm.c | 34 xen/arch/x86/mm.c | 41 ++- xen/common/grant_table.c | 38 xen/include/a

Re: [Xen-devel] [PATCH v06 27/36] uapi xen/gntdev.h: include xen/privcmd.h and define grant_ref_t

2017-08-21 Thread Juergen Gross
On 21/08/17 16:31, Boris Ostrovsky wrote: > On 08/21/2017 09:33 AM, Juergen Gross wrote: >> On 06/08/17 18:44, Mikko Rapeli wrote: >>> Both are needed to compile in userspace. Fixes these >>> userspace compile errors: >>> >>> xen/gntdev.h

Re: [Xen-devel] [PATCH v06 27/36] uapi xen/gntdev.h: include xen/privcmd.h and define grant_ref_t

2017-08-21 Thread Juergen Gross
’ > domid_t domid; > ^ > > Signed-off-by: Mikko Rapeli <mikko.rap...@iki.fi> > Cc: Boris Ostrovsky <boris.ostrov...@oracle.com> > Cc: Juergen Gross <jgr...@suse.com> > Cc: xen-de...@lists.xenproject.org > --- > include/uapi/xen/gntdev.h

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

2017-08-21 Thread Juergen Gross
On 21/08/17 10:33, Paul Durrant wrote: >> -Original Message- >> From: Juergen Gross [mailto:jgr...@suse.com] >> Sent: 16 August 2017 13:52 >> To: xen-de...@lists.xenproject.org >> Cc: Juergen Gross <jgr...@suse.com>; Paul Durrant >> <paul.durr..

Re: [Xen-devel] Xen 4.10 Development Update

2017-08-21 Thread Juergen Gross
Per-cpu tasklet > - XEN-28 > - Konrad Rzeszutek Wilk > > * Add support of rcu_idle_{enter,exit} > - XEN-27 > - Dario Faggioli * Add support for modifying hyperviro parameters at runtime - Juergen Gross Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH RFC] x86: enable RCU based table free when PARAVIRT

2017-08-18 Thread Juergen Gross
Context Switches 187514 (175324)Context Switches 187358 (175060) > Sleeps 112633 (24535.5) Sleeps 111743 (23297.6) > > Suggested-by: Peter Zijlstra <pet...@infradead.org> > Signed-off-by: Vitaly Kuznetsov <vkuzn...@redhat.com> Acked-by: Juergen Gross <jgr...@suse.com> Thanks, Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [PATCH] xen: fix build failure related to removing adjust_exception_frame

2017-08-17 Thread Juergen Gross
A kernel configured with XEN_PV but without KVM_GUEST will fail to build since the patch removing the adjust_exception_frame paravirt op. Fix this failure. Reported-by: Sander Eikelenboom <li...@eikelenboom.it> Signed-off-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/xen-

Re: [Xen-devel] Linux kernel build fails with xen branch for-linus-4.14 commit "xen: get rid of paravirt op adjust_exception_frame"

2017-08-17 Thread Juergen Gross
On 17/08/17 09:12, Sander Eikelenboom wrote: > Hi Juergen, > > Today i pulled > git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.gitfor-linus-4.14 > on top of current 4.13-rc5 linux tree. > > Unfortunately the kernel build fails with: > GEN .version > CHK

[Xen-devel] [PATCH v2 1/2] paravirt,xen: remove xen_patch()

2017-08-16 Thread Juergen Gross
-by: Juergen Gross <jgr...@suse.com> --- arch/x86/xen/enlighten_pv.c | 59 + arch/x86/xen/xen-asm.S | 24 -- arch/x86/xen/xen-asm.h | 12 - arch/x86/xen/xen-asm_32.S | 27 - arch/x86/xen/xen-asm_64.S

[Xen-devel] [PATCH v2 0/2] x86: paravirt related cleanup

2017-08-16 Thread Juergen Gross
() (Peter Zijlstra) [1]: https://lkml.org/lkml/2017/5/15/502 Juergen Gross (2): paravirt,xen: remove xen_patch() x86/lguest: remove lguest support MAINTAINERS | 11 - arch/x86/Kbuild |3 - arch/x86/Kconfig |2

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

2017-08-16 Thread Juergen Gross
itrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Dario Faggioli <dario.faggi...@citrix.com> --- xen/common/sched_credit2.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/common/sched_credit2.c b/xen/common/sched_credit2.c index 9b1db1351f..2da9cc2ffb

[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 <ian.jack...@eu.citrix.com> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V2: - corrected coding style (

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3:

[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 <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei Liu <wei.l...@citrix.com> --- xen/c

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

2017-08-16 Thread Juergen Gross
ni <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V2: - replaced literal 8 by BITS_PER_BYTE (Wei Liu) - added test for empty string to parse_bool() V3: - use function pointer in

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

2017-08-16 Thread Juergen Gross
; Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Jan Beulich <jbeul...@suse.com> --- xen/arch/x86/cpu/mcheck/mce.c | 4 1 file changed, 4 deletions(-) diff --git a/xen/arch/x86/cpu/mcheck/mce.c b/xen/arch/x86/cpu/mcheck/mce.c index 8015dff8c4..7affe2591e 100644 --- a/xen/arch/x8

[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 <jbeul...@suse.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - dont modify option value in parsing funct

[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 <ian.jack...@eu.citrix.com> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - zero padding fields in sysc

[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
urr...@citrix.com> Cc: Jan Beulich <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - dont modify option value in parsing function - fix error in parsing routine --- xen/arch/x86/hvm/viridian.c | 28

[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 <sstabell...@kernel.org> Cc: Julien Grall <julien.gr...@arm.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3

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

2017-08-16 Thread Juergen Gross
Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - *_runtime_param() now includes boot parameter setting (Jan

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

2017-08-16 Thread Juergen Gross
t; Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei

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

2017-08-16 Thread Juergen Gross
Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei

[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 <jbeul...@suse.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - dont modify option value in handling funct

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> ---

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

2017-08-16 Thread Juergen Gross
t; Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com>

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

2017-08-16 Thread Juergen Gross
; Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Jan Beulich <jbeul...@suse.com> --- xen/arch/x86/io_apic.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/arch/x86/io_apic.c b/xen/arch/x86/io_apic.c index f767c4560c..f959090ca0 100644 --- a/xen/arch/x86/io_apic.c +++

[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 <ian.jack...@eu.citrix.com> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by:

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

2017-08-16 Thread Juergen Gross
i Liu <wei.l...@citrix.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Cc: George Dunlap <george.dun...@eu.citrix.com> Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim De

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: -

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

2017-08-16 Thread Juergen Gross
parameters() function): corrected coding style (Wei Liu) Juergen Gross (52): xen/arch/arm/acpi/boot.c: let custom parameter parsing routines return errno xen/arch/arm/domain_build.c: let custom parameter parsing routines return errno xen/arch/arm/traps.c: let custom paramet

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

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

2017-08-16 Thread Juergen Gross
; Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Jan Beulich <jbeul...@suse.com> --- xen/arch/x86/apic.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/xen/arch/x86/apic.c b/xen/arch/x86/apic.c index 325b48074a..2638414e08 100644 --- a/xen/arch/x86/apic.c +

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <kevin.t...@intel.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei Liu <wei.l...@citrix.com>

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

2017-08-16 Thread Juergen Gross
ndrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Reviewed-by: Paul Durrant <paul.durr...@citrix.com> --- xen/arch/x86/hvm/viridian.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c

[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 <jbeul...@suse.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - cosmetic changes (Jan Beulich) - dont modify

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <kevin.t...@intel.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei Liu <wei.l...@citrix.com>

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

2017-08-16 Thread Juergen Gross
itrix.com> Cc: Ian Jackson <ian.jack...@eu.citrix.com> Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com>

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

[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 <dgde...@tycho.nsa.gov> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Daniel De Graaf <dgde...@tycho.nsa.gov

[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 <sstabell...@kernel.org> Cc: Julien Grall <julien.gr...@arm.com> Signed-off-by: Juergen Gross <jgr...@suse.com>

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

2017-08-16 Thread Juergen Gross
Dunlap <george.dun...@eu.citrix.com> Cc: Ian Jackson <ian.jack...@eu.citrix.com> Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@

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

2017-08-16 Thread Juergen Gross
ed-off-by: Juergen Gross <jgr...@suse.com> --- xen/common/kexec.c | 5 - 1 file changed, 5 deletions(-) diff --git a/xen/common/kexec.c b/xen/common/kexec.c index fcc68bd4d8..e0f1e3eafa 100644 --- a/xen/common/kexec.c +++ b/xen/common/kexec.c @@ -213,7 +213,6 @@ static int __init parse_low_crashinf

[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 <jbeul...@suse.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei Liu <wei.l...@citrix.com> Acked-by:

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

2017-08-16 Thread Juergen Gross
Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei

[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 <suravee.suthikulpa...@amd.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: We

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

2017-08-16 Thread Juergen Gross
t; Cc: Jan Beulich <jbeul...@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> Cc: Stefano Stabellini <sstabell...@kernel.org> Cc: Tim Deegan <t...@xen.org> Cc: Wei Liu <wei.l...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-by: Wei

[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 <jbeul...@suse.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - dont return out of loops (Jan Beulich) - dont modify

[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
ooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> --- V3: - dont modify option value in parsing function --- xen/arch/x86/hvm/vmx/vmcs.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/xen/arch/x86/hvm/vmx/vmcs.c b/xen

[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 <sstabell...@kernel.org> Cc: Julien Grall <julien.gr...@arm.com> Signed-off-by: Juergen Gross <jgr...@suse.com>

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com&

[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 <george.dun...@eu.citrix.com> Cc: Dario Faggioli <dario.faggi...@citrix.com> Signed-off-by: Juergen Gross <jgr...@

[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 <jbeul...@suse.com> Cc: Andrew Cooper <andrew.coop...@citrix.com> Signed-off-by: Juergen Gross <jgr...@suse.com> Acked-

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

2017-08-15 Thread Juergen Gross
On 15/08/17 17:59, Jan Beulich wrote: On 15.08.17 at 17:52, wrote: >> On 15/08/17 17:45, Jan Beulich wrote: >> On 14.08.17 at 09:08, wrote: --- a/xen/drivers/char/console.c +++ b/xen/drivers/char/console.c @@ -41,6 +41,7 @@

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

2017-08-15 Thread Juergen Gross
On 15/08/17 17:31, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/arch/x86/xen.lds.S >> +++ b/xen/arch/x86/xen.lds.S >> @@ -226,6 +226,10 @@ SECTIONS >> __start_schedulers_array = .; >> *(.data.schedulers) >> __end_schedulers_array = .;

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

2017-08-15 Thread Juergen Gross
On 15/08/17 17:39, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/include/public/sysctl.h >> +++ b/xen/include/public/sysctl.h >> @@ -1096,6 +1096,23 @@ struct xen_sysctl_livepatch_op { >> typedef struct xen_sysctl_livepatch_op xen_sysctl_livepatch_op_t; >>

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

2017-08-15 Thread Juergen Gross
On 15/08/17 17:45, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/drivers/char/console.c >> +++ b/xen/drivers/char/console.c >> @@ -41,6 +41,7 @@ string_param("console", opt_console); >> /* boots. Any other value, or omitting the char, enables >>

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

2017-08-15 Thread Juergen Gross
On 14/08/17 14:46, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/common/kernel.c >> +++ b/xen/common/kernel.c >> optval[-1] = '\0'; >> +break; > > Why? Applies to further break-s you add: At least in the past we > had

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:40, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> @@ -37,13 +37,24 @@ static void __init parse_mmcfg(char *s) >> if ( ss ) >> *ss = '\0'; >> >> -if ( !parse_bool(s) ) >> +switch ( parse_bool(s) ) { >> +case

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:39, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/arch/x86/shutdown.c >> +++ b/xen/arch/x86/shutdown.c >> @@ -51,7 +51,7 @@ static int reboot_mode; >> * efiUse the EFI reboot (if running under EFI) >> */ >> static enum reboot_type

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:37, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> bool __read_mostly acpi_disabled; >> bool __initdata acpi_force; >> static char __initdata acpi_param[10] = ""; >> -static void __init parse_acpi_param(char *s) >> +static int __init

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:35, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/arch/x86/psr.c >> +++ b/xen/arch/x86/psr.c >> @@ -420,7 +420,7 @@ static const struct feat_props l2_cat_props = { >> }; >> >> static void __init parse_psr_bool(char *s, char *value, char

Re: [Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
On 14/08/17 15:18, Jan Beulich wrote: On 14.08.17 at 14:21, wrote: >> On 14/08/17 13:40, Jan Beulich wrote: >> On 14.08.17 at 13:05, wrote: On 14/08/17 12:48, Jan Beulich wrote: On 14.08.17 at 12:35, wrote: >> On

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:31, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> /* opt_watchdog_timeout: Number of seconds to wait before panic. */ >> static unsigned int opt_watchdog_timeout = 5; >> -static void parse_watchdog_timeout(char * s) >> +static int

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

2017-08-14 Thread Juergen Gross
On 14/08/17 15:24, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/arch/x86/dom0_build.c >> +++ b/xen/arch/x86/dom0_build.c >> @@ -47,7 +47,7 @@ static long __init parse_amt(const char *s, const char >> **ps) >> long pages = parse_size_and_unit((*s == '-')

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

2017-08-14 Thread Juergen Gross
On 14/08/17 14:46, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> --- a/xen/common/kernel.c >> +++ b/xen/common/kernel.c >> @@ -23,9 +23,11 @@ enum system_state system_state = SYS_STATE_early_boot; >> xen_commandline_t saved_cmdline; >> static const char __initconst

Re: [Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
On 14/08/17 14:36, Andrew Cooper wrote: > On 14/08/17 11:21, Juergen Gross wrote: >> Current pv guests will only see physical addresses up to 46 bits wide. >> In order to be able to run on a host supporting 5 level paging and to >> make use of any possible memory page ther

Re: [Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
On 14/08/17 13:40, Jan Beulich wrote: On 14.08.17 at 13:05, wrote: >> On 14/08/17 12:48, Jan Beulich wrote: >> On 14.08.17 at 12:35, wrote: On 14/08/17 12:29, Jan Beulich wrote: On 14.08.17 at 12:21, wrote: >> Current

Re: [Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
On 14/08/17 12:48, Jan Beulich wrote: On 14.08.17 at 12:35, wrote: >> On 14/08/17 12:29, Jan Beulich wrote: >> On 14.08.17 at 12:21, wrote: Current pv guests will only see physical addresses up to 46 bits wide. In order to be able to run on a

Re: [Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
On 14/08/17 12:29, Jan Beulich wrote: On 14.08.17 at 12:21, wrote: >> Current pv guests will only see physical addresses up to 46 bits wide. >> In order to be able to run on a host supporting 5 level paging and to >> make use of any possible memory page there, physical

[Xen-devel] [PATCH] include/public: add new elf note for support of huge physical addresses

2017-08-14 Thread Juergen Gross
-by: Juergen Gross <jgr...@suse.com> --- As I'd like to add support for large physical addresses in pv guests rather sooner than later to the Linux kernel, I'm suggesting this public interface change way before any 5 level paging support is added to Xen. --- xen/include/public/elfnote.h | 11

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

2017-08-14 Thread Juergen Gross
On 14/08/17 10:39, Jan Beulich wrote: On 14.08.17 at 09:08, wrote: >> 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 >>

<    1   2   3   4   5   6   7   8   9   10   >