Re: [Xen-devel] [PATCH] xsm: correct AVC lookups for two sysctls

2017-08-09 Thread Andrew Cooper
On 09/08/17 17:56, Daniel De Graaf wrote: > The current code was incorrectly using SECCLASS_XEN instead of > SECCLASS_XEN2, resulting in the wrong permission being checked. > > GET_CPU_LEVELLING_CAPS was checking MTRR_DEL > GET_CPU_FEATURESET was checking MTRR_READ > > The default XSM policy only

[Xen-devel] [PATCH] common/domain_page: Drop domain_mmap_cache infrastructure

2017-08-09 Thread Andrew Cooper
This infrastructure is used exclusively by the x86 do_mmu_update() hypercall. Mapping and unmapping domain pages is probably not the slow part of that function, but even with an opencoded caching implementation, Bloat-o-meter reports: function old new

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

2017-08-09 Thread Daniel De Graaf
On 08/09/2017 03:06 AM, Juergen Gross wrote: Modify the custom parameter parsing routines in: xen/xsm/flask/flask_op.c to indicate whether the parameter value was parsed successfully. Acked-by: Daniel De Graaf ___ Xen-devel

Re: [Xen-devel] [PATCH v2] x86/xen/64: Rearrange the SYSCALL entries

2017-08-09 Thread Juergen Gross
On 08/08/17 05:59, Andy Lutomirski wrote: > Xen's raw SYSCALL entries are much less weird than native. Rather > than fudging them to look like native entries, use the Xen-provided > stack frame directly. > > This lets us eliminate entry_SYSCALL_64_after_swapgs and two uses of > the

[Xen-devel] [ovmf baseline-only test] 71957: all pass

2017-08-09 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71957 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71957/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 7ef0dae092afcfb6fab7e8372c78097672168c4a baseline

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

2017-08-09 Thread osstest service owner
flight 112536 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/112536/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stop fail REGR. vs. 112526 Tests which did not

Re: [Xen-devel] [PATCH v8 2/2] x86/monitor: Notify monitor if an emulation fails.

2017-08-09 Thread Wei Liu
On Tue, Aug 08, 2017 at 09:06:38PM +0300, Petre Pircalabu wrote: > If case of a vm_event with the emulate_flags set, if the instruction > is not implemented by the emulator, the monitor should be notified instead > of directly injecting a hw exception. > This behavior can be used to re-execute an

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

2017-08-09 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 18/52] xen/arch/x86/psr.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/arch/x86/psr.c | 28

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

2017-08-09 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_credit2_runqueue() can be removed. Cc: George Dunlap Cc: Dario Faggioli

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

2017-08-09 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 --- xen/drivers/passthrough/vtd/dmar.c | 4 +++- 1 file

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

2017-08-09 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 ---

[Xen-devel] [PATCH v1 05/13] x86: implement get hw info flow for MBA

2017-08-09 Thread Yi Sun
This patch implements get HW info flow for MBA including its callback function and sysctl interface. Signed-off-by: Yi Sun --- v1: - sort 'PSR_INFO_IDX_' macros as feature. (suggested by Chao Peng) - rename 'PSR_INFO_IDX_MBA_LINEAR' to

[Xen-devel] [PATCH v1 08/13] tools: create general interfaces to support psr allocation features

2017-08-09 Thread Yi Sun
This patch creates general interfaces in libxl to support all psr allocation features. Add 'LIBXL_HAVE_PSR_MBA' to indicate interface change. Please note, the functionality cannot work until later patches are applied. Signed-off-by: Yi Sun --- v1: - change

[Xen-devel] [PATCH v1 07/13] x86: implement set value flow for MBA

2017-08-09 Thread Yi Sun
This patch implements set value flow for MBA including its callback function and domctl interface. It also changes the memebers in 'cos_write_info' to transfer the feature array, feature properties array and value array. Then, we can write all features values on the cos id into MSRs.

[Xen-devel] [PATCH v1 11/13] tools: implemet new get value interface suitable for all psr allocation features

2017-08-09 Thread Yi Sun
This patch implements a new get value interface in tools suitable for all psr allocation features and the whole flow. It also enables MBA support in tools to get MBA value. Signed-off-by: Yi Sun --- tools/libxc/include/xenctrl.h | 7 +- tools/libxc/xc_psr.c

[Xen-devel] [PATCH v1 00/13] Enable Memory Bandwidth Allocation in Xen

2017-08-09 Thread Yi Sun
Hi, all, We plan to bring a new PSR (Platform Shared Resource) feature called Intel Memory Bandwidth Allocation (MBA) to Xen. Besides the MBA enabling, we change some interfaces to make them more general but not only for CAT. The implementation bases on below patch set which has been merged

[Xen-devel] [PATCH v1 13/13] docs: add MBA description in docs

2017-08-09 Thread Yi Sun
This patch adds MBA description in related documents. Signed-off-by: Yi Sun --- docs/man/xl.pod.1.in | 33 ++ docs/misc/xl-psr.markdown | 60 +++ 2 files changed, 93 insertions(+) diff --git

[Xen-devel] [PATCH v1 09/13] tools: implement the new get hw info interface suitable to all psr allocation features

2017-08-09 Thread Yi Sun
This patch implements a new get hw info interface suitable for all psr allocation features and the whole flow. It also enables MBA support in tools to get MBA HW info. Signed-off-by: Yi Sun --- v1: - remove the pointless initialization for 'xc_psr_feat_type'.

[Xen-devel] [PATCH v1 12/13] tools: implemet new set value interface suitable for all psr allocation features

2017-08-09 Thread Yi Sun
This patch implements a new set value interface in tools suitable for all psr allocation features and the whole flow. It also enables MBA support to set MBA value in tools. Signed-off-by: Yi Sun --- tools/libxc/include/xenctrl.h | 6 ++--- tools/libxc/xc_psr.c

[Xen-devel] [PATCH v1 03/13] x86: rename 'cbm_type' to 'psr_val_type' to make it general

2017-08-09 Thread Yi Sun
This patch renames 'cbm_type' to 'psr_val_type' to make it be general. Then, we can reuse this for all psr allocation features. Signed-off-by: Yi Sun --- xen/arch/x86/domctl.c | 16 ++--- xen/arch/x86/psr.c| 59

[Xen-devel] [PATCH v1 01/13] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-08-09 Thread Yi Sun
This patch creates MBA feature document in doc/features/. It describes key points to implement MBA which is described in details in Intel SDM "Introduction to Memory Bandwidth Allocation". Signed-off-by: Yi Sun --- v1: - remove a special character to avoid the error

[Xen-devel] [PATCH v1 10/13] tools: rename 'xc_psr_cat_type' to 'xc_psr_val_type'

2017-08-09 Thread Yi Sun
This patch renames 'xc_psr_cat_type' to 'xc_psr_val_type' so that the structure name is common for all allocation features. Signed-off-by: Yi Sun --- v1: - split this patch out from next 'get value' patch. (suggested by Wei Liu) ---

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

2017-08-09 Thread Yi Sun
This patch implements get value domctl interface for MBA. Signed-off-by: Yi Sun --- xen/arch/x86/domctl.c | 7 +++ xen/include/public/domctl.h | 1 + 2 files changed, 8 insertions(+) diff --git a/xen/arch/x86/domctl.c b/xen/arch/x86/domctl.c index

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

2017-08-09 Thread Yi Sun
This patch implements main data structures of MBA. Like CAT features, MBA HW info has cos_max which means the max cos registers number, and thrtl_max which means the max throttle value (delay value). It also has a flag to represent if the throttle value is linear or not. One COS register of MBA

[Xen-devel] [PATCH v1 02/13] Rename PSR sysctl/domctl interfaces and xsm policy to make them be general

2017-08-09 Thread Yi Sun
This patch renames PSR sysctl/domctl interfaces and related xsm policy to make them be general for all resource allocation features but not only for CAT. Then, we can resuse the interfaces for all allocation features. Basically, it changes 'cat' to 'alloc'. E.g.: 1. psr_cat_op -> psr_alloc_op 2.

Re: [Xen-devel] [PATCH v8 1/2] x86emul: New return code for unimplemented instruction

2017-08-09 Thread Paul Durrant
> -Original Message- > From: Petre Pircalabu [mailto:ppircal...@bitdefender.com] > Sent: 08 August 2017 19:07 > To: xen-devel@lists.xen.org > Cc: Ian Jackson ; Wei Liu ; > Andrew Cooper ; George Dunlap >

[Xen-devel] [ovmf baseline-only test] 71953: all pass

2017-08-09 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71953 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71953/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 2913ebb2b550f50a14f105e26995dd095e627ba4 baseline

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

2017-08-09 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 --- xen/drivers/cpufreq/cpufreq.c | 18 +-

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

2017-08-09 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 ---

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

2017-08-09 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 06/52] xen/arch/x86/cpu/vpmu.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/arch/x86/cpu/vpmu.c

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

2017-08-09 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 ---

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

2017-08-09 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 --- xen/arch/x86/irq.c | 9

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

2017-08-09 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 --- xen/arch/x86/io_apic.c |

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

2017-08-09 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 --- docs/man/xl.pod.1.in | 5 +

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

2017-08-09 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 15/52] xen/arch/x86/nmi.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/arch/x86/nmi.c | 20

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

2017-08-09 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 --- xen/arch/arm/traps.c

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

2017-08-09 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 10/52] xen/arch/x86/hvm/vmx/vmcs.c: let custom parameter parsing routines return errno

2017-08-09 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 33/52] xen/drivers/passthrough/iommu.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/drivers/passthrough/iommu.c | 15 ++- 1 file

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

2017-08-09 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 --- xen/arch/x86/apic.c | 11

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

2017-08-09 Thread Paul Durrant
> -Original Message- > From: Juergen Gross [mailto:jgr...@suse.com] > Sent: 09 August 2017 08:07 > To: xen-devel@lists.xen.org > Cc: Juergen Gross ; Paul Durrant > ; Jan Beulich ; Andrew > Cooper >

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

2017-08-09 Thread Dario Faggioli
On Wed, 2017-08-09 at 09:06 +0200, Juergen Gross wrote: > Modify the custom parameter parsing routines in: > > xen/common/sched_credit2.c > > to indicate whether the parameter value was parsed successfully. > > Cc: George Dunlap > Signed-off-by: Juergen Gross

Re: [Xen-devel] [PATCH v4 6/9] xen/vpci: add handlers to map the BARs

2017-08-09 Thread Jan Beulich
>>> On 08.08.17 at 14:35, wrote: > On Sat, Jul 29, 2017 at 10:44:02AM -0600, Jan Beulich wrote: >> >>> Roger Pau Monne 07/24/17 4:58 PM >>> >> >On Fri, Jul 14, 2017 at 09:11:29AM -0600, Jan Beulich wrote: >> >> >>> On 30.06.17 at 17:01,

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

2017-08-09 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 --- xen/common/kexec.c | 30 ++ 1 file

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

2017-08-09 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 --- tools/libxc/include/xenctrl.h | 1 +

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

2017-08-09 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 17/52] xen/arch/x86/oprofile/nmi_int.c: let custom parameter parsing routines return errno

2017-08-09 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 ---

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

2017-08-09 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 ---

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

2017-08-09 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 50/52] libxl: add libxl_set_parameters() function

2017-08-09 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 --- tools/libxl/libxl.c | 14 ++

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

2017-08-09 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 --- xen/common/efi/boot.c | 6 +- 1 file changed, 5 insertions(+), 1

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

2017-08-09 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 ---

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

2017-08-09 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 --- xen/drivers/passthrough/vtd/quirks.c | 7 --- 1

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

2017-08-09 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 --- xen/arch/x86/time.c | 6

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

2017-08-09 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 ---

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

2017-08-09 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 52/52] xen: make some console related parameters settable at runtime

2017-08-09 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 48/52] xen: add hypercall for setting parameters at runtime

2017-08-09 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 47/52] xen: add basic support for runtime parameter changing

2017-08-09 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 39/52] xen: check parameter validity when parsing command line

2017-08-09 Thread Juergen Gross
Where possible check validity of parameters in _cmdline_parse() and issue a warning message in case of an error detected. Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich

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

2017-08-09 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 --- xen/arch/x86/mm.c | 7 ++-

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

2017-08-09 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. Cc: Paul Durrant Cc: Jan Beulich Cc: Andrew Cooper Signed-off-by: Juergen Gross

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

2017-08-09 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 ---

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

2017-08-09 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 --- xen/xsm/flask/flask_op.c | 6 -- 1 file changed, 4

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

2017-08-09 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 --- xen/drivers/passthrough/pci.c | 17 - 1 file

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

2017-08-09 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 ---

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

2017-08-09 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 --- xen/arch/x86/shutdown.c

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

2017-08-09 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 19/52] xen/arch/x86/setup.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/arch/x86/setup.c | 32

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

2017-08-09 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 27/52] xen/common/memory.c: let custom parameter parsing routines return errno

2017-08-09 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 16/52] xen/arch/x86/numa.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/arch/x86/numa.c | 10

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

2017-08-09 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 01/52] xen/arch/arm/acpi/boot.c: let custom parameter parsing routines return errno

2017-08-09 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 ---

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

2017-08-09 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 29/52] xen/drivers/acpi/tables.c: let custom parameter parsing routines return errno

2017-08-09 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 --- xen/drivers/acpi/tables.c | 5 +++-- 1 file changed, 3

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

2017-08-09 Thread Dario Faggioli
On Wed, 2017-08-09 at 09:06 +0200, Juergen Gross wrote: > 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_credit2_runqueue() can be removed. > > Cc: George Dunlap

Re: [Xen-devel] Is:livepatch-build-tools.git declare it supported? Was:Re: [PATCH for-4.9] livepatch: Declare live patching as a supported feature

2017-08-09 Thread Jan Beulich
>>> On 08.08.17 at 13:16, wrote: > On 08/07/2017 04:59 PM, Jan Beulich wrote: > George Dunlap 08/07/17 12:27 PM >>> >>> So it seems that people are still not quite clear about what I'm proposing. >> >> And indeed your examples helped me

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

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

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

2017-08-09 Thread Paul Durrant
> -Original Message- > From: Juergen Gross [mailto:jgr...@suse.com] > Sent: 09 August 2017 08:06 > To: xen-devel@lists.xen.org > Cc: Juergen Gross ; Paul Durrant > ; Jan Beulich ; Andrew > Cooper >

Re: [Xen-devel] [PATCH v7 13/14] arm/mem_access: Add short-descriptor based gpt

2017-08-09 Thread Sergej Proskurin
Hi Andrew, >>> diff --git a/xen/arch/arm/guest_walk.c b/xen/arch/arm/guest_walk.c >>> index b258248322..7f34a2b1d3 100644 >>> --- a/xen/arch/arm/guest_walk.c >>> +++ b/xen/arch/arm/guest_walk.c >>> @@ -112,7 +112,12 @@ static int guest_walk_sd(const struct vcpu *v, >>> * level

[Xen-devel] [PATCH v8 03/13] arm/lpae: Introduce lpae_is_page helper

2017-08-09 Thread Sergej Proskurin
This commit introduces a new helper that checks whether the target PTE holds a page mapping or not. This helper will be used as part of the following commits. Signed-off-by: Sergej Proskurin Reviewed-by: Julien Grall --- Cc: Stefano Stabellini

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

2017-08-09 Thread Sergej Proskurin
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: Sergej Proskurin Acked-by: Julien Grall

[Xen-devel] [PATCH v8 01/13] arm/mem_access: Add and cleanup (TCR_|TTBCR_)* defines

2017-08-09 Thread Sergej Proskurin
This commit adds (TCR_|TTBCR_)* defines to simplify access to the respective register contents. At the same time, we adjust the macros TCR_T0SZ and TCR_TG0_* by using the newly introduced TCR_T0SZ_SHIFT and TCR_TG0_SHIFT instead of the hardcoded values. Signed-off-by: Sergej Proskurin

[Xen-devel] [PATCH v8 09/13] arm/guest_access: Rename vgic_access_guest_memory

2017-08-09 Thread Sergej Proskurin
This commit renames the function vgic_access_guest_memory to access_guest_memory_by_ipa. As the function name suggests, the functions expects an IPA as argument. All invocations of this function have been adapted accordingly. Apart from that, we have adjusted all printk messages for cleanup and to

[Xen-devel] [PATCH v8 02/13] arm/mem_access: Add defines supporting PTs with varying page sizes

2017-08-09 Thread Sergej Proskurin
AArch64 supports pages with different (4K, 16K, and 64K) sizes. To enable guest page table walks for various configurations, this commit extends the defines and helpers of the current implementation. Signed-off-by: Sergej Proskurin Reviewed-by: Julien Grall

[Xen-devel] [PATCH v8 04/13] arm/mem_access: Add short-descriptor pte typedefs and macros

2017-08-09 Thread Sergej Proskurin
The current implementation does not provide appropriate types for short-descriptor translation table entries. As such, this commit adds new types, which simplify managing the respective translation table entries. Signed-off-by: Sergej Proskurin Acked-by: Julien Grall

[Xen-devel] [PATCH v8 12/13] arm/mem_access: Add short-descriptor based gpt

2017-08-09 Thread Sergej Proskurin
This commit adds functionality to walk the guest's page tables using the short-descriptor translation table format for both ARMv7 and ARMv8. The implementation is based on ARM DDI 0487B-a J1-6002 and ARM DDI 0406C-b B3-1506. Signed-off-by: Sergej Proskurin Acked-by:

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

2017-08-09 Thread Sergej Proskurin
This commit adds functionality to walk the guest's page tables using the long-descriptor translation table format for both ARMv7 and ARMv8. Similar to the hardware architecture, the implementation supports different page granularities (4K, 16K, and 64K). The implementation is based on ARM DDI

Re: [Xen-devel] [PATCH v4 6/9] xen/vpci: add handlers to map the BARs

2017-08-09 Thread Roger Pau Monné
On Wed, Aug 09, 2017 at 02:17:57AM -0600, Jan Beulich wrote: > >>> On 08.08.17 at 14:35, wrote: > > On Sat, Jul 29, 2017 at 10:44:02AM -0600, Jan Beulich wrote: > >> >>> Roger Pau Monne 07/24/17 4:58 PM >>> > >> >On Fri, Jul 14, 2017 at 09:11:29AM

[Xen-devel] [PATCH v8 13/13] arm/mem_access: Walk the guest's pt in software

2017-08-09 Thread Sergej Proskurin
In this commit, we make use of the gpt walk functionality introduced in the previous commits. If mem_access is active, hardware-based gva to ipa translation might fail, as gva_to_ipa uses the guest's translation tables, access to which might be restricted by the active VTTBR. To side-step

[Xen-devel] [PATCH v8 05/13] arm/mem_access: Introduce GV2M_EXEC permission

2017-08-09 Thread Sergej Proskurin
We extend the current implementation by an additional permission, GV2M_EXEC, which will be used to describe execute permissions of PTE's as part of our guest translation table walk implementation. Signed-off-by: Sergej Proskurin Acked-by: Julien Grall

[Xen-devel] [PATCH v8 06/13] arm/mem_access: Introduce BIT_ULL bit operation

2017-08-09 Thread Sergej Proskurin
We introduce the BIT_ULL macro to using values of unsigned long long as to enable setting bits of 64-bit registers on AArch32. In addition, this commit adds a define holding the register width of 64 bit double-word registers. This define simplifies using the associated constants in the following

[Xen-devel] [linux-linus test] 112523: regressions - trouble: blocked/broken/fail/pass

2017-08-09 Thread osstest service owner
flight 112523 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/112523/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-examine 4 host-install broken REGR. vs. 110515

[Xen-devel] [PATCH v8 07/13] arm/mem_access: Introduce GENMASK_ULL bit operation

2017-08-09 Thread Sergej Proskurin
The current implementation of GENMASK is capable of creating bitmasks of 32-bit values on AArch32 and 64-bit values on AArch64. As we need to create masks for 64-bit values on AArch32 as well, in this commit we introduce the GENMASK_ULL bit operation. Please note that the GENMASK_ULL

Re: [Xen-devel] [PATCH v4 7/9] vpci/msi: add MSI handlers

2017-08-09 Thread Jan Beulich
>>> On 08.08.17 at 17:44, wrote: > On Wed, Aug 02, 2017 at 07:34:28AM -0600, Jan Beulich wrote: >> >>> Roger Pau Monne 06/30/17 5:01 PM >>> >> >+/* Get a PIRQ. */ >> >+rc = allocate_and_map_msi_pirq(pdev->domain, -1, >pirq, >> >+

[Xen-devel] [PATCH v8 10/13] arm/mem_access: Add software guest-page-table walk

2017-08-09 Thread Sergej Proskurin
The function p2m_mem_access_check_and_get_page in mem_access.c translates a gva to an ipa by means of the hardware functionality of the ARM architecture. This is implemented in the function gva_to_ipa. If mem_access is active, hardware-based gva to ipa translation might fail, as gva_to_ipa uses

  1   2   3   >