Re: [Xen-devel] [PATCH v2 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization.

2016-03-08 Thread Xu, Quan
On March 09, 2016 1:19pm, wrote: > > From: Quan Xu > > Sent: Wednesday, March 09, 2016 11:08 AM > > When iommu_setup() is called in __start_xen(), interrupts have already > > been enabled, and nothing disables (without reenabling) them again in > > the path that leads to calling set_iommu_interrup

Re: [Xen-devel] [PATCH v2 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-08 Thread Xu, Quan
On March 09, 2016 1:21pm, wrote: > > From: Xu, Quan > > Sent: Wednesday, March 09, 2016 11:08 AM > > > > Signed-off-by: Quan Xu > > CC: Keir Fraser > > CC: Jan Beulich > > CC: Andrew Cooper > > CC: Suravee Suthikulpanit > > CC: Feng Wu > > CC: Kevin Tian > > CC: Dario Faggioli > > Acked-b

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread Wu, Feng
> -Original Message- > From: George Dunlap [mailto:george.dun...@citrix.com] > Sent: Wednesday, March 9, 2016 1:06 AM > To: Jan Beulich ; George Dunlap > ; Wu, Feng > Cc: Andrew Cooper ; Dario Faggioli > ; Tian, Kevin ; xen- > de...@lists.xen.org; Konrad Rzeszutek Wilk ; Keir > Fraser >

Re: [Xen-devel] [PATCH v2 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-08 Thread Tian, Kevin
> From: Xu, Quan > Sent: Wednesday, March 09, 2016 11:08 AM > > Signed-off-by: Quan Xu > CC: Keir Fraser > CC: Jan Beulich > CC: Andrew Cooper > CC: Suravee Suthikulpanit > CC: Feng Wu > CC: Kevin Tian > CC: Dario Faggioli Acked-by: Kevin Tian for VT-d part.

Re: [Xen-devel] [PATCH v2 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization.

2016-03-08 Thread Tian, Kevin
> From: Quan Xu > Sent: Wednesday, March 09, 2016 11:08 AM Hi, Quan, sorry that I still didn't quite get below description. > > When iommu_setup() is called in __start_xen(), interrupts have > already been enabled, and nothing disables (without reenabling) > them again in the path that leads to

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread Wu, Feng
> -Original Message- > From: George Dunlap [mailto:george.dun...@citrix.com] > Sent: Wednesday, March 9, 2016 2:39 AM > To: Jan Beulich ; George Dunlap > ; Wu, Feng > Cc: Andrew Cooper ; Dario Faggioli > ; Tian, Kevin ; xen- > de...@lists.xen.org; Konrad Rzeszutek Wilk ; Keir > Fraser >

Re: [Xen-devel] [PATCH v7]xen: sched: convert RTDS from time to event driven model

2016-03-08 Thread Meng Xu
Hi Tianyang, Thank you very much for this version of patch. It is in a really good state, IMHO, although it does have a suspiciuous logic error inside, which can be corrected easily. I didn't mark out all repeated style issues. I think you can correct all of the style issues, such as the spaces i

[Xen-devel] [linux-linus test] 85725: regressions - FAIL

2016-03-08 Thread osstest service owner
flight 85725 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/85725/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 59254 build-amd64-rumpuserx

[Xen-devel] [PATCH v2 2/2] IOMMU/spinlock: Make the pcidevs_lock a recursive one

2016-03-08 Thread Quan Xu
Signed-off-by: Quan Xu CC: Keir Fraser CC: Jan Beulich CC: Andrew Cooper CC: Suravee Suthikulpanit CC: Feng Wu CC: Kevin Tian CC: Dario Faggioli --- xen/arch/x86/domctl.c | 8 +-- xen/arch/x86/hvm/vmsi.c | 4 +- xen/arch/x86/irq.c

[Xen-devel] [PATCH v2 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization.

2016-03-08 Thread Quan Xu
When iommu_setup() is called in __start_xen(), interrupts have already been enabled, and nothing disables (without reenabling) them again in the path that leads to calling set_iommu_interrupt_handler(). There's therefore no risk of them being disabled and needing remaining so, and hence no need for

[Xen-devel] [PATCH v2 0/2] Make the pcidevs_lock a recursive one

2016-03-08 Thread Quan Xu
This patch set makes the pcidevs_lock a recursive one. It is a prereq patch set for Patch:'VT-d Device-TLB flush issue', as the pcidevs_lock may be recursively held for hiding the ATS device, when VT-d Device-TLB flush timed out. In detail: 1. Fix a bug found in AMD IOMMU initialization. When i

Re: [Xen-devel] XSM permissive by default.

2016-03-08 Thread Doug Goldstein
On 3/8/16 7:51 PM, Konrad Rzeszutek Wilk wrote: > Hey, > > I was wondering if it we should change the default flask_bootparam > option from permissive to disabled? > > The reason being is that I was startled to see that my xSplice > code was able to patch the hypervisor from within an PV guest! >

[Xen-devel] [PATCH V17 5/6] domcreate: support pvusb in configuration file

2016-03-08 Thread Chunyan Liu
Add code to support pvusb in domain config file. One could specify usbctrl and usb in domain's configuration file and create domain, then usb controllers will be created and usb device would be attached to guest automatically. One could specify usb controllers and usb devices in config file like t

[Xen-devel] [PATCH V17 6/6] xl: add pvusb commands

2016-03-08 Thread Chunyan Liu
Add pvusb commands: usbctrl-attach, usbctrl-detach, usb-list, usbdev-attach and usbdev-detach. To attach a usb device to guest through pvusb, one could follow following example: #xl usbctrl-attach test_vm version=1 ports=8 #xl usb-list test_vm will show the usb controllers and port usage unde

[Xen-devel] [PATCH V17 0/6] xen pvusb toolstack work

2016-03-08 Thread Chunyan Liu
This patch series is to add pvusb toolstack work, supporting hot add|remove USB device to|from guest and specify USB device in domain configuration file. In V16 4/6: +out: +path = GCSPRINTF(USBBACK_INFO_PATH "/%s", usbdev_encode); +rc = libxl__xs_rm_checked(gc, XBT_NULL, path); This 'rc' s

[Xen-devel] [PATCH V17 3/6] refactor DEFINE_DEVICE_REMOVE to fit for more device types

2016-03-08 Thread Chunyan Liu
For some device type, device removal operation needs to be handled specially, like usbctrl, it needs to remove all usb devices under it first, then remove usbctrl. Extend DEFINE_DEVICE_REMOVE to support generic and custom way For those need to be handled specially, call DEFINE_DEVICE_REMOVE_CUSTOM,

[Xen-devel] [PATCH V17 1/6] libxl: export some functions for pvusb use

2016-03-08 Thread Chunyan Liu
Signed-off-by: Chunyan Liu Signed-off-by: Simon Cao Reviewed-by: Wei Liu Acked-by: Ian Jackson --- tools/libxl/libxl.c | 5 ++--- tools/libxl/libxl_internal.h | 3 +++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 833fd4

[Xen-devel] [PATCH V17 2/6] libxl_utils: add internal function to read sysfs file contents

2016-03-08 Thread Chunyan Liu
Add a new function libxl_read_sysfs_file_contents to handle sysfs file specially. It would be used in later pvusb work. Signed-off-by: Chunyan Liu Acked-by: Ian Jackson --- tools/libxl/libxl_internal.h | 4 +++ tools/libxl/libxl_utils.c| 74 2 f

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

2016-03-08 Thread Chunyan Liu
Add pvusb APIs, including: - attach/detach (create/destroy) virtual usb controller. - attach/detach usb device - list usb controller and usb devices - some other helper functions Signed-off-by: Simon Cao Signed-off-by: George Dunlap Signed-off-by: Chunyan Liu Reviewed-by: George Dunlap ---

Re: [Xen-devel] [PATCH v11 20/27] Support colo mode for qemu disk

2016-03-08 Thread Konrad Rzeszutek Wilk
> Furthermore, I think coming up with a story for PV-aware guests (PVHVM, > PV and PVH) is also non-trivial. For one the disk replication logic is Pls keep in mind that PV guests can use QEMU qdisk if they wish. This is more of 'phy' vs 'file' question. > not implemented in PV block backend, we'

[Xen-devel] XSM permissive by default.

2016-03-08 Thread Konrad Rzeszutek Wilk
Hey, I was wondering if it we should change the default flask_bootparam option from permissive to disabled? The reason being is that I was startled to see that my xSplice code was able to patch the hypervisor from within an PV guest! Further testing showed that I could do 'xl debug-keys R' from

Re: [Xen-devel] [RESEND][PATCH V16 0/6] xen pvusb toolstack work

2016-03-08 Thread Chun Yan Liu
>>> On 3/8/2016 at 07:33 PM, in message <56deb899.7080...@citrix.com>, George Dunlap wrote: > On 08/03/16 01:37, Chunyan Liu wrote: > > This patch series is to add pvusb toolstack work, supporting hot add|remove > > USB device to|from guest and specify USB device in domain configuration > f

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

2016-03-08 Thread osstest service owner
flight 85701 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/85701/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 65543 test-amd64-i386-xl-qemuu-ovm

Re: [Xen-devel] [PATCH v6 for Xen 4.7 3/4] libxl: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Chong Li
On Tue, Mar 8, 2016 at 1:12 PM, Wei Liu wrote: > On Sun, Mar 06, 2016 at 11:55:57AM -0600, Chong Li wrote: > [...] >> tools/libxl/libxl.c | 326 >> >> tools/libxl/libxl.h | 37 + >> tools/libxl/libxl_types.idl | 14 ++ >> 3 files

[Xen-devel] [xen-unstable baseline-only test] 44233: regressions - FAIL

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

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

2016-03-08 Thread Yu-An(Victor) Chen
Sorry for the duplicated email Congyang, I forgot to replied all: Hi Congyang, Thank you for the hint, after building xen, your script works for qemu-xen! so now I am trying to set up the secondary node with the script provided by Changlong in his first reply: ---

[Xen-devel] [qemu-mainline test] 85694: regressions - FAIL

2016-03-08 Thread osstest service owner
flight 85694 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/85694/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 85573 Regressions which a

Re: [Xen-devel] [PATCH v6 for Xen 4.7 4/4] xl: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Chong Li
On Tue, Mar 8, 2016 at 1:12 PM, Wei Liu wrote: > On Sun, Mar 06, 2016 at 11:55:58AM -0600, Chong Li wrote: > [...] >> @@ -6222,77 +6359,166 @@ int main_sched_rtds(int argc, char **argv) >> { >> const char *dom = NULL; >> const char *cpupool = NULL; >> -int period = 0; /* period is i

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread Ben Sanda
George, >FWIW, on my "to-do" list for xenalyze for years has been to have the xentrace >process query something (either Xen or Linux) to find the hz rate, and then >write that at the beginning of the xentrace file, so that xenalyze could just >pick that up and use it. Since you're doing some work

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

2016-03-08 Thread osstest service owner
flight 85686 xen-4.3-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85686/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops 3 host-install(3) broken REGR. vs. 83004 build-armhf

Re: [Xen-devel] [libvirt] [PATCH V2 1/4] conf: add 'state' attribute to feature

2016-03-08 Thread Jim Fehlig
Joao Martins wrote: > > On 03/01/2016 04:00 AM, Jim Fehlig wrote: >> Most hypervisors use Hardware Assisted Paging by default and don't >> require specifying the feature in domain conf. But some hypervisors >> support disabling HAP on a per-domain basis. To enable HAP by default >> yet provide a k

Re: [Xen-devel] [PATCH v6 for Xen 4.7 2/4] libxc: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Wei Liu
On Tue, Mar 08, 2016 at 01:32:58PM -0600, Chong Li wrote: > On Tue, Mar 8, 2016 at 1:09 PM, Wei Liu wrote: > > On Sun, Mar 06, 2016 at 11:55:56AM -0600, Chong Li wrote: > >> Add xc_sched_rtds_vcpu_get/set functions to interact with > >> Xen to get/set a domain's per-VCPU parameters. > >> > >> Sign

Re: [Xen-devel] [PATCH v6 for Xen 4.7 2/4] libxc: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Chong Li
On Tue, Mar 8, 2016 at 1:09 PM, Wei Liu wrote: > On Sun, Mar 06, 2016 at 11:55:56AM -0600, Chong Li wrote: >> Add xc_sched_rtds_vcpu_get/set functions to interact with >> Xen to get/set a domain's per-VCPU parameters. >> >> Signed-off-by: Chong Li >> Signed-off-by: Meng Xu >> Signed-off-by: Sisu

Re: [Xen-devel] [PATCH RFC 0/4] Allow qemu to provide emulated disks for devices with hotplug scripts

2016-03-08 Thread Wei Liu
On Wed, Mar 02, 2016 at 11:14:00AM +, George Dunlap wrote: > Qemu in dom0 can only provide emulated disks when the qemu process has > access to an image of the file. At the moment, this only includes > plain files or physical devices in dom0; it does not include anything > to do with a hotplug

[Xen-devel] [linux-mingo-tip-master test] 85705: regressions - FAIL

2016-03-08 Thread osstest service owner
flight 85705 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/85705/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 60684 build-amd6

Re: [Xen-devel] [PATCH v6 for Xen 4.7 4/4] xl: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Wei Liu
On Sun, Mar 06, 2016 at 11:55:58AM -0600, Chong Li wrote: [...] > @@ -6222,77 +6359,166 @@ int main_sched_rtds(int argc, char **argv) > { > const char *dom = NULL; > const char *cpupool = NULL; > -int period = 0; /* period is in microsecond */ > -int budget = 0; /* budget is in m

[Xen-devel] [linux-4.1 test] 85682: regressions - FAIL

2016-03-08 Thread osstest service owner
flight 85682 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/85682/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 6 xen-build fail REGR. vs. 66399 build-i386-rumpuserxen

Re: [Xen-devel] [PATCH v6 for Xen 4.7 3/4] libxl: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Wei Liu
On Sun, Mar 06, 2016 at 11:55:57AM -0600, Chong Li wrote: [...] > tools/libxl/libxl.c | 326 > > tools/libxl/libxl.h | 37 + > tools/libxl/libxl_types.idl | 14 ++ > 3 files changed, 354 insertions(+), 23 deletions(-) > > diff --

Re: [Xen-devel] [PATCH v6 for Xen 4.7 2/4] libxc: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Wei Liu
On Sun, Mar 06, 2016 at 11:55:56AM -0600, Chong Li wrote: > Add xc_sched_rtds_vcpu_get/set functions to interact with > Xen to get/set a domain's per-VCPU parameters. > > Signed-off-by: Chong Li > Signed-off-by: Meng Xu > Signed-off-by: Sisu Xi > > --- > Changes on PATCH v5: > 1) In xc_sched_r

Re: [Xen-devel] [PATCH v6 for Xen 4.7 1/4] xen: enable per-VCPU parameter settings for RTDS scheduler

2016-03-08 Thread Wei Liu
On Sun, Mar 06, 2016 at 11:55:55AM -0600, Chong Li wrote: [...] > @@ -1163,6 +1173,96 @@ rt_dom_cntl( > } > spin_unlock_irqrestore(&prv->lock, flags); > break; > +case XEN_DOMCTL_SCHEDOP_getvcpuinfo: > +if ( guest_handle_is_null(op->u.v.vcpus) ) > +{ >

[Xen-devel] [PATCH 6/9 v2] xl: Improve return and exit codes of list related functions.

2016-03-08 Thread Harmandeep Kaur
flush_stream(), list_domains(), list_vm(), main_list() and main_vm_list() are fixed. Signed-off-by: Harmandeep Kaur --- v2: No changes. --- tools/libxl/xl_cmdimpl.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/x

[Xen-devel] [PATCH 5/9 v2] xl: Improve return and exit codes of pause, destroy and shutdown related functions.

2016-03-08 Thread Harmandeep Kaur
destroy_domain(), wait_for_domain_deaths(), shutdown_domain(), reboot_domain(), main_pause(), main_unpause(), main_destroy() and main_shutdown_or_reboot() are fixed. Signed-off-by: Harmandeep Kaur --- v2: No changes. --- tools/libxl/xl_cmdimpl.c | 26 +- 1 file changed, 1

[Xen-devel] [PATCH 7/9 v2] xl: Improve return and exit codes of create, config_update and rename related functions.

2016-03-08 Thread Harmandeep Kaur
create_domain(), string_realloc_append(), main_create(), main_config_update() and main_rename() are fixed. Signed-off-by: Harmandeep Kaur --- v2: Add create_domain() Remove main_sharing() --- tools/libxl/xl_cmdimpl.c | 54 1 file changed, 27 i

[Xen-devel] [PATCH 9/9 v2] xl: Improve return and exit codes of domain id and name related functions.

2016-03-08 Thread Harmandeep Kaur
find_domain(), main_domid() and main_domname() are fixed. Signed-off-by: Harmandeep Kaur --- v2: Add find_domain(). --- tools/libxl/xl_cmdimpl.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index 4b2ae68..883

[Xen-devel] [PATCH 8/9 v2] xl : Improve return and exit codes debugging, special keys and triggering related functions.

2016-03-08 Thread Harmandeep Kaur
button_press(), main_trigger(), main_sysrq(), main_debug_keys() and main_dmesg() are fixed. Signed-off-by: Harmandeep Kaur --- v2: Add main_sysrq(), main_debug_keys(), main_dmesg() Remove xvasprintf(), main_remus() --- tools/libxl/xl_cmdimpl.c | 16 1 file changed, 8 inserti

[Xen-devel] [PATCH 0/9 v2] xl: convert exit codes related to domain subcommands to EXIT_[SUCCESS|FAILURE]

2016-03-08 Thread Harmandeep Kaur
*main_foo() is treated somewhat as a regular main(), it is changed to return EXIT_SUCCESS or EXIT_FAILURE. *Functions that are not main_foo(), are changed to do 'return 0' or 'return 1', and then 0/1 is taken care in the main_foo() functions that calls them. *Functions in xl_cmdimpl.c related to

[Xen-devel] [PATCH 3/9 v2] xl: Improve return and exit codes of migrate related functions.

2016-03-08 Thread Harmandeep Kaur
Micros CHK_ERRNOVAL, CHK_SYSCALL, MUST are fixed. xvasprintf(), save_domain_core_begin(), create_migration_child(), migrate_read_fixedmessage(), migrate_do_preamble(), migrate_domain(), migrate_receive(), main_migrate_receive() and main_migrate() in the file xl_cmdimpl are fixed. Signed-off-by: H

[Xen-devel] [PATCH 4/9 v2] xl: Improve return and exit codes of console, vnc and core dump related functions.

2016-03-08 Thread Harmandeep Kaur
autoconnect_vncviewer(), main_console(), main_vncviewer(), core_dump_domain() and main_dump_core() in the file xl_cmdimpl are fixed. Signed-off-by: Harmandeep Kaur --- v2: Add autoconnect_vncviewer(). --- tools/libxl/xl_cmdimpl.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(

[Xen-devel] [PATCH 2/9 v2] xl: Improve return and exit codes of restore and save related functions.

2016-03-08 Thread Harmandeep Kaur
save_domain(), main_restore(), main_save() and main_remus() in the file xl_cmdimpl are fixed. Signed-off-by: Harmandeep Kaur --- v2: Add main_remus(). Remove create_domain(). --- tools/libxl/xl_cmdimpl.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --g

[Xen-devel] [PATCH 1/9 v2] xl: Improve return and exit codes of memory related functions.

2016-03-08 Thread Harmandeep Kaur
Return and exit codes of freemem(), set_memory_max(), main_memmax(), set_memory_target(), main_memset() and main_sharing() in the file xl_cmdimpl are fixed. Add comment to explain return vaule of parse_mem_size_kb(). Signed-off-by: Harmandeep Kaur --- v2: Add comment to explain return vaule of p

[Xen-devel] [libvirt test] 85689: tolerable FAIL - PUSHED

2016-03-08 Thread osstest service owner
flight 85689 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/85689/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 12 migrate-support-checkfail never pass test-armhf-armhf-libvirt-xsm 14 guest-saver

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread George Dunlap
On 08/03/16 18:04, Ben Sanda wrote: > All, > > To update to the current situation. I have been able to get xentrace() and > xenalyze working completely (at least as far as I can tell) on ARM. > > For xentrace there were changes to the memory allocation routines to allow > mapping of the Xen Heap

[Xen-devel] [PATCHv1] CA-201372: x86: don't flush the whole cache when changing cachability

2016-03-08 Thread David Vrabel
Introduce the FLUSH_CACHE_BY_VA flag to flush_area_mask() and friends to say that it is safe to use CLFLUSH (i.e., the virtual address is still valid). Use this when changing the cachability of the Xen direct mappings (in response to the guest changing the cachability of its mappings). This signif

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread George Dunlap
On 08/03/16 17:26, Jan Beulich wrote: On 08.03.16 at 18:05, wrote: >> On 08/03/16 15:42, Jan Beulich wrote: >> On 08.03.16 at 15:42, wrote: On Tue, Mar 8, 2016 at 1:10 PM, Wu, Feng wrote: >> -Original Message- >> From: George Dunlap [mailto:george.dun...@citrix.com]

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread Andrew Cooper
On 08/03/16 18:28, Paul Sujkov wrote: > Hi Ben, > > looks like you've done everything I did. I'm far from my working > laptop today, so I can post my work on this issue only tomorrow so you > can look through the code, compare and estimate solutions. > > Regarding ARM build, I'm using xenalyze as b

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread Paul Sujkov
Hi Ben, looks like you've done everything I did. I'm far from my working laptop today, so I can post my work on this issue only tomorrow so you can look through the code, compare and estimate solutions. Regarding ARM build, I'm using xenalyze as both host tool (x86 build, e.g. for gnuplot scatter

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread Dario Faggioli
On Tue, 2016-03-08 at 18:04 +, Ben Sanda wrote: > All, > > To update to the current situation. I have been able to get > xentrace() and > xenalyze working completely (at least as far as I can tell) on ARM. > Great! :-) > For xentrace there were changes to the memory allocation routines to >

Re: [Xen-devel] Xentrace on Xilinx ARM

2016-03-08 Thread Ben Sanda
All, To update to the current situation. I have been able to get xentrace() and xenalyze working completely (at least as far as I can tell) on ARM. For xentrace there were changes to the memory allocation routines to allow mapping of the Xen Heap by dom0, correcting the MFN->PFN translations, add

Re: [Xen-devel] [PATCH v2 3/3] xl: new "loglvl" command

2016-03-08 Thread Dario Faggioli
On Tue, 2016-03-08 at 14:05 +, George Dunlap wrote: > On Tue, Mar 8, 2016 at 8:08 AM, Jan Beulich > wrote: > >  > > Right, and asking people myself to not follow bad examples when > > adding new code, I did take all of your input to adjust the patch. > > Just that in this case the set of bad e

Re: [Xen-devel] [PATCH 2/2] xsm: move FLASK_AVC_STATS to Kconfig

2016-03-08 Thread Daniel De Graaf
On 03/08/2016 11:51 AM, Jan Beulich wrote: On 08.03.16 at 17:22, wrote: On 03/08/2016 04:46 AM, Jan Beulich wrote: On 07.03.16 at 19:42, wrote: Have Kconfig set CONFIG_FLASK_AVC_STATS and prefix all uses with CONFIG_ to use the Kconfig variable. Same question here: What's the benefit of do

Re: [Xen-devel] [PATCH 1/4] x86/alternatives: correct near branch check

2016-03-08 Thread Andrew Cooper
On 07/03/16 16:21, Jan Beulich wrote: On 07.03.16 at 17:11, wrote: >> On 07/03/16 15:56, Jan Beulich wrote: >> On 07.03.16 at 16:43, wrote: On 04/03/16 11:27, Jan Beulich wrote: > Make sure the near JMP/CALL check doesn't consume uninitialized > data, not even in a benign wa

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread Jan Beulich
>>> On 08.03.16 at 18:05, wrote: > On 08/03/16 15:42, Jan Beulich wrote: > On 08.03.16 at 15:42, wrote: >>> On Tue, Mar 8, 2016 at 1:10 PM, Wu, Feng wrote: > -Original Message- > From: George Dunlap [mailto:george.dun...@citrix.com] > > 2. Try to test engineered situa

Re: [Xen-devel] [PATCH v11 20/27] Support colo mode for qemu disk

2016-03-08 Thread Wei Liu
On Mon, Mar 07, 2016 at 10:10:07AM +0800, Wen Congyang wrote: > On 03/05/2016 04:30 AM, Konrad Rzeszutek Wilk wrote: > > On Fri, Mar 04, 2016 at 05:52:09PM +, Ian Jackson wrote: > >> Changlong Xie writes ("[PATCH v11 20/27] Support colo mode for qemu disk"): > >>> +Enable COLO HA for disk. For

Re: [Xen-devel] [PATCH v3 6/6] libxl: add force option for xl vcpu-pin

2016-03-08 Thread Juergen Gross
On 08/03/16 18:16, Dario Faggioli wrote: > On Thu, 2016-03-03 at 17:48 +0100, Juergen Gross wrote: >> In order to be able to undo a vcpu pin override in case of a kernel >> driver error add a flag "-f" to the "xl vcpu-pin" command forcing the >> hypervisor to undo the override. >> >> Cc: Ian Jackso

Re: [Xen-devel] [PATCH v3 6/6] libxl: add force option for xl vcpu-pin

2016-03-08 Thread Dario Faggioli
On Thu, 2016-03-03 at 17:48 +0100, Juergen Gross wrote: > In order to be able to undo a vcpu pin override in case of a kernel > driver error add a flag "-f" to the "xl vcpu-pin" command forcing the > hypervisor to undo the override. > > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Wei Liu > S

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

2016-03-08 Thread George Dunlap
On 08/03/16 01:37, Chunyan Liu wrote: > Add pvusb APIs, including: > - attach/detach (create/destroy) virtual usb controller. > - attach/detach usb device > - list usb controller and usb devices > - some other helper functions > > Signed-off-by: Simon Cao > Signed-off-by: George Dunlap > Sig

Re: [Xen-devel] [PATCH v3 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi

2016-03-08 Thread Andrei Borzenkov
08.03.2016 19:37, Fu Wei пишет: > Hi Andrei, > > On 8 March 2016 at 14:54, Andrei Borzenkov wrote: >> 07.03.2016 11:22, Fu Wei пишет: >>> Hi Andrei, >>> >>> On 28 February 2016 at 00:44, Fu Wei wrote: Hi Andrei On 28 February 2016 at 01:26, Andrei Borzenkov wrote: > 26.02.201

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread George Dunlap
On 08/03/16 15:42, Jan Beulich wrote: On 08.03.16 at 15:42, wrote: >> On Tue, Mar 8, 2016 at 1:10 PM, Wu, Feng wrote: -Original Message- From: George Dunlap [mailto:george.dun...@citrix.com] >> [snip] It seems like there are a couple of ways we could approach this: >>>

Re: [Xen-devel] [PATCH 2/2] xsm: move FLASK_AVC_STATS to Kconfig

2016-03-08 Thread Jan Beulich
>>> On 08.03.16 at 17:22, wrote: > On 03/08/2016 04:46 AM, Jan Beulich wrote: > On 07.03.16 at 19:42, wrote: >>> Have Kconfig set CONFIG_FLASK_AVC_STATS and prefix all uses with CONFIG_ >>> to use the Kconfig variable. >> >> Same question here: What's the benefit of doing it this way? > > Th

Re: [Xen-devel] [PATCH 2/2] tools: detect appropriate debug optimization level

2016-03-08 Thread Wei Liu
On Tue, Mar 08, 2016 at 10:34:42AM -0600, Doug Goldstein wrote: > On 3/8/16 9:38 AM, Wei Liu wrote: > > On Mon, Mar 07, 2016 at 08:23:40PM -0600, Doug Goldstein wrote: > >> The build should not use -O0 as that results in miscompilations. There > > > > This needs some (concrete) references. Is that

Re: [Xen-devel] [PATCH v11 19/27] COLO: introduce new API to prepare/start/do/get_error/stop replication

2016-03-08 Thread Wei Liu
On Fri, Mar 04, 2016 at 05:26:44PM +, Ian Jackson wrote: > Changlong Xie writes ("[PATCH v11 19/27] COLO: introduce new API to > prepare/start/do/get_error/stop replication"): > > From: Wen Congyang > > > > We will use qemu block replication, and qemu provides some qmp commands > > to prepar

Re: [Xen-devel] [PATCH v11 07/27] docs/libxl: Introduce CHECKPOINT_CONTEXT to support migration v2 colo streams

2016-03-08 Thread Wei Liu
On Fri, Mar 04, 2016 at 04:51:20PM +, Ian Jackson wrote: [...] > > > @@ -212,6 +214,11 @@ class VerifyLibxl(VerifyBase): > > if len(content) != 0: > > raise RecordError("Checkpoint end record with non-zero length") > > > > +def verify_record_checkpoint_state(self, c

Re: [Xen-devel] [PATCH v11 03/27] tools/libxl: Add back channel to allow migration target send data back

2016-03-08 Thread Wei Liu
On Fri, Mar 04, 2016 at 04:38:23PM +, Ian Jackson wrote: > Changlong Xie writes ("[PATCH v11 03/27] tools/libxl: Add back channel to > allow migration target send data back"): > > From: Wen Congyang > > > > In COLO mode, secondary needs to send the following data to primary: > > 1. In libxl

Re: [Xen-devel] [PATCH v3 4/4] arm64: update the introduction of xen boot commands in docs/grub.texi

2016-03-08 Thread Fu Wei
Hi Andrei, On 8 March 2016 at 14:54, Andrei Borzenkov wrote: > 07.03.2016 11:22, Fu Wei пишет: >> Hi Andrei, >> >> On 28 February 2016 at 00:44, Fu Wei wrote: >>> Hi Andrei >>> >>> On 28 February 2016 at 01:26, Andrei Borzenkov wrote: 26.02.2016 14:13, fu@linaro.org пишет: > From:

Re: [Xen-devel] [libvirt] [PATCH V2 3/4] Xen drivers: show hap enabled by default in capabilities

2016-03-08 Thread Joao Martins
On 03/01/2016 04:00 AM, Jim Fehlig wrote: > Hardware Assisted Paging is enabled by default in Xen. Change > the capabilities output to reflect this. > > Signed-off-by: Jim Fehlig > --- > src/libxl/libxl_conf.c | 2 +- > src/xen/xen_hypervisor.c | 2 +- > 2 files changed, 2 insertions(+), 2 d

Re: [Xen-devel] [libvirt] [PATCH V2 4/4] libxl: support enabling and disabling feature

2016-03-08 Thread Joao Martins
On 03/01/2016 04:00 AM, Jim Fehlig wrote: > Until now, the libxl driver ignored any setting in domain XML > and deferred to libxl, which enables hap if not specified. While > this is a good default, it prevents disabling hap if desired. > > This change allows disabling hap with . hap is > expli

Re: [Xen-devel] [libvirt] [PATCH V2 2/4] xenconfig: change 'hap' setting to align with Xen behavior

2016-03-08 Thread Joao Martins
On 03/01/2016 04:00 AM, Jim Fehlig wrote: > hap is enabled by default in xm and xl config and usually only > specified when it is desirable to disable hap (hap = 0). Change > the xm,xl <-> xml converter to behave similarly. I.e. only > produce 'hap = 0' when and vice versa. > > Signed-off-by: J

Re: [Xen-devel] [libvirt] [PATCH V2 1/4] conf: add 'state' attribute to feature

2016-03-08 Thread Joao Martins
On 03/01/2016 04:00 AM, Jim Fehlig wrote: > Most hypervisors use Hardware Assisted Paging by default and don't > require specifying the feature in domain conf. But some hypervisors > support disabling HAP on a per-domain basis. To enable HAP by default > yet provide a knob to disable it, extend t

Re: [Xen-devel] [PATCH 2/2] tools: detect appropriate debug optimization level

2016-03-08 Thread Doug Goldstein
On 3/8/16 9:38 AM, Wei Liu wrote: > On Mon, Mar 07, 2016 at 08:23:40PM -0600, Doug Goldstein wrote: >> The build should not use -O0 as that results in miscompilations. There > > This needs some (concrete) references. Is that a known issue in gcc? If > so can you reference the bug number? So its n

Re: [Xen-devel] [PATCH v2 3/3] xl: new "loglvl" command

2016-03-08 Thread Wei Liu
On Fri, Mar 04, 2016 at 09:48:16AM -0700, Jan Beulich wrote: > This is pretty simplistic for now, but I'd rather have someone better > friends with the tools improve it (if desired). > > Signed-off-by: Jan Beulich > > --- a/tools/libxl/libxl.c > +++ b/tools/libxl/libxl.c > @@ -5958,6 +5958,26 @@

Re: [Xen-devel] [PATCH 1/2] xsm: move the XSM_MAGIC value to Kconfig

2016-03-08 Thread Daniel De Graaf
On 03/07/2016 01:42 PM, Doug Goldstein wrote: Let Kconfig set the XSM_MAGIC value for us. Signed-off-by: Doug Goldstein This is not the best place to define this constant: it doesn't make sense for it to be user-configurable. If you want to move it out of config.h, I think the best solution

Re: [Xen-devel] [PATCH 2/2] xsm: move FLASK_AVC_STATS to Kconfig

2016-03-08 Thread Daniel De Graaf
On 03/08/2016 04:46 AM, Jan Beulich wrote: On 07.03.16 at 19:42, wrote: Have Kconfig set CONFIG_FLASK_AVC_STATS and prefix all uses with CONFIG_ to use the Kconfig variable. Same question here: What's the benefit of doing it this way? This removes the stats tracking, which might (I have not

Re: [Xen-devel] [PATCH v2 2/3] libxc: wrapper for log level sysctl

2016-03-08 Thread Wei Liu
On Fri, Mar 04, 2016 at 09:47:32AM -0700, Jan Beulich wrote: > Signed-off-by: Jan Beulich > Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 4/6] libxc: do some retries in xc_cpupool_removecpu() for EBUSY case

2016-03-08 Thread Juergen Gross
On 08/03/16 14:16, Dario Faggioli wrote: > On Thu, 2016-03-03 at 17:48 +0100, Juergen Gross wrote: >> The hypervisor might return EBUSY when trying to remove a cpu from a >> cpupool when a domain running in this cpupool has pinned a vcpu >> temporarily. Do some retries in this case, perhaps the sit

Re: [Xen-devel] [PATCH v3 6/6] libxl: add force option for xl vcpu-pin

2016-03-08 Thread Juergen Gross
On 08/03/16 16:58, Wei Liu wrote: > On Thu, Mar 03, 2016 at 05:48:50PM +0100, Juergen Gross wrote: > [...] >> int libxl_set_vcpuaffinity_all(libxl_ctx *ctx, uint32_t domid, >> unsigned int max_vcpus, >> const libxl_bitmap *cpumap_hard

Re: [Xen-devel] [PATCH v2 3/3] xl: new "loglvl" command

2016-03-08 Thread Wei Liu
On Tue, Mar 08, 2016 at 02:05:01PM +, George Dunlap wrote: > On Tue, Mar 8, 2016 at 8:08 AM, Jan Beulich wrote: > On 07.03.16 at 19:07, wrote: > >> On Mon, 2016-03-07 at 04:46 -0700, Jan Beulich wrote: > >>> > > > On 04.03.16 at 19:45, wrote: > >>> > On Fri, 2016-03-04 at 09:48 -0700, J

Re: [Xen-devel] [PATCH] xen: Restore p2m_access_t enum order to allow bitmask semantics

2016-03-08 Thread Tamas K Lengyel
On Tue, Mar 8, 2016 at 4:52 PM, Jan Beulich wrote: > >>> On 08.03.16 at 16:30, wrote: > > Nested hap code assumed implict bitmask semantics of the p2m_access_t > > enum prior to C/S 4c63692d7c38c5ac414fe97f8ef37b66e05abe5c > > > > The change to the enum ordering broke this assumption and caused

Re: [Xen-devel] [PATCH v3 5/6] libxl: print message how to recover from xl cpupool-cpu-remove errors

2016-03-08 Thread Wei Liu
On Thu, Mar 03, 2016 at 05:48:49PM +0100, Juergen Gross wrote: > An error occurring when calling "xl cpupool-cpu-remove" might leave > the system in a state where a cpu is neither completely free nor in > a cpupool. This can easily be repaired by adding the cpu via > "xl cpupool-cpu-add" to the cpu

Re: [Xen-devel] [PATCH v3 6/6] libxl: add force option for xl vcpu-pin

2016-03-08 Thread Wei Liu
On Thu, Mar 03, 2016 at 05:48:50PM +0100, Juergen Gross wrote: [...] > int libxl_set_vcpuaffinity_all(libxl_ctx *ctx, uint32_t domid, > unsigned int max_vcpus, > const libxl_bitmap *cpumap_hard, > diff --git a/tools/libxl/libxl.h b/to

Re: [Xen-devel] [PATCH v3 03/16] configure: #define SEABIOS_PATH and OVMF_PATH

2016-03-08 Thread Wei Liu
On Thu, Mar 03, 2016 at 05:03:00PM +, Anthony PERARD wrote: > On Tue, Mar 01, 2016 at 11:51:36AM +, Wei Liu wrote: > > On Thu, Feb 25, 2016 at 02:56:01PM +, Anthony PERARD wrote: > > > Those paths are to be used by libxl, in order to load the firmware in > > > memory. If a system path i

Re: [Xen-devel] [PATCH v3 06/16] libxl: Load guest ACPI table from file

2016-03-08 Thread Wei Liu
On Thu, Mar 03, 2016 at 05:12:07PM +, Anthony PERARD wrote: > On Tue, Mar 01, 2016 at 11:51:43AM +, Wei Liu wrote: > > On Thu, Feb 25, 2016 at 02:56:04PM +, Anthony PERARD wrote: > > > A user can provide a different ACPI tables than the default one by using > > > the existing "acpi_firm

Re: [Xen-devel] [PATCH] xen: Restore p2m_access_t enum order to allow bitmask semantics

2016-03-08 Thread Jan Beulich
>>> On 08.03.16 at 16:30, wrote: > Nested hap code assumed implict bitmask semantics of the p2m_access_t > enum prior to C/S 4c63692d7c38c5ac414fe97f8ef37b66e05abe5c > > The change to the enum ordering broke this assumption and caused functional > problems for the nested hap code. As it may be er

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread Jan Beulich
>>> On 08.03.16 at 15:42, wrote: > On Tue, Mar 8, 2016 at 1:10 PM, Wu, Feng wrote: >>> -Original Message- >>> From: George Dunlap [mailto:george.dun...@citrix.com] > [snip] >>> It seems like there are a couple of ways we could approach this: >>> >>> 1. Try to optimize the reverse look-up

Re: [Xen-devel] [PATCH 2/2] tools: detect appropriate debug optimization level

2016-03-08 Thread Wei Liu
On Mon, Mar 07, 2016 at 08:23:40PM -0600, Doug Goldstein wrote: > The build should not use -O0 as that results in miscompilations. There This needs some (concrete) references. Is that a known issue in gcc? If so can you reference the bug number? > have been a few instances on the ML where users w

Re: [Xen-devel] [PATCH 1/2] libxl: ensure var is inited in libxl__domain_firmware

2016-03-08 Thread Wei Liu
On Mon, Mar 07, 2016 at 08:23:39PM -0600, Doug Goldstein wrote: > Some versions of GCC complain that the 'firmware' variable can be used > uninitialized. It looks like the switch inside of the else case is just > confusing GCC. > > Signed-off-by: Doug Goldstein Acked-by: Wei Liu > --- > CC: Ia

Re: [Xen-devel] [PATCH] xen: Restore p2m_access_t enum order to allow bitmask semantics

2016-03-08 Thread Andrew Cooper
On 08/03/16 15:30, Malcolm Crossley wrote: > Nested hap code assumed implict bitmask semantics of the p2m_access_t > enum prior to C/S 4c63692d7c38c5ac414fe97f8ef37b66e05abe5c > > The change to the enum ordering broke this assumption and caused functional > problems for the nested hap code. As it m

[Xen-devel] [PATCH] xen: Restore p2m_access_t enum order to allow bitmask semantics

2016-03-08 Thread Malcolm Crossley
Nested hap code assumed implict bitmask semantics of the p2m_access_t enum prior to C/S 4c63692d7c38c5ac414fe97f8ef37b66e05abe5c The change to the enum ordering broke this assumption and caused functional problems for the nested hap code. As it may be error prone to audit and find all other p2m_ac

[Xen-devel] I'm a beginner and want become a Xen tester.

2016-03-08 Thread Jason Long
Hello. How can I test Xen and report bug? Can it need programming? Tnx. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] test

2016-03-08 Thread Doug Goldstein
On 3/8/16 8:35 AM, Juergen Gross wrote: > This is just a test of the mailman bounce mechanism: I want to be sure > that I receive bounce messages (they have been filtered by SUSE's mail > server). > > Please don't accept, reject or discard this message. I'll do it myself. > > In case it makes it

Re: [Xen-devel] Ideas Re: [PATCH v14 1/2] vmx: VT-d posted-interrupt core logic handling

2016-03-08 Thread George Dunlap
On Tue, Mar 8, 2016 at 1:10 PM, Wu, Feng wrote: >> -Original Message- >> From: George Dunlap [mailto:george.dun...@citrix.com] [snip] >> It seems like there are a couple of ways we could approach this: >> >> 1. Try to optimize the reverse look-up code so that it's not a linear >> linked li

  1   2   >