[Xen-devel] [rumpuserxen test] 53061: regressions - FAIL

2015-05-05 Thread osstest service user
flight 53061 rumpuserxen real [real] http://logs.test-lab.xenproject.org/osstest/logs/53061/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-rumpuserxen 5 rumpuserxen-build fail REGR. vs. 33866 build-i386-rumpuserxe

Re: [Xen-devel] [PATCH v6 01/14] x86: add socket_to_cpumask

2015-05-05 Thread Chao Peng
On Fri, Apr 24, 2015 at 03:46:11PM +0100, Jan Beulich wrote: > >>> On 23.04.15 at 11:55, wrote: > > @@ -301,6 +304,8 @@ static void set_cpu_sibling_map(int cpu) > > } > > } > > } > > + > > +cpumask_set_cpu(cpu, &socket_to_cpumask[cpu_to_socket(cpu)]); > > } > > The

Re: [Xen-devel] [PATCH] libxl: include a XLU_Config in _libxlDriverConfig

2015-05-05 Thread Wei Liu
On Tue, Apr 28, 2015 at 06:40:05AM +, Olaf Hering wrote: > Upcoming changes for vscsi will use libxlutil.so to prepare the > configuration for libxl. The helpers needs a xlu struct for logging. > Provide one and reuse the existing output as log target. > Do you not need to call xlu_cfg_destro

Re: [Xen-devel] [PATCH] tools: add conigure --with-dumpdir=DIR option

2015-05-05 Thread Wei Liu
On Thu, Apr 30, 2015 at 02:24:06PM +, Olaf Hering wrote: > The current base directory /var/xen for domU dumps will be patched to > /var/lib/xen by most distros. Provide a configure option to avoid > patching the source. > > If the option is not specified the default remains /var/xen/dump. > >

Re: [Xen-devel] [PATCH v6 01/14] x86: add socket_to_cpumask

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 09:44, wrote: > On Fri, Apr 24, 2015 at 03:46:11PM +0100, Jan Beulich wrote: >> >>> On 23.04.15 at 11:55, wrote: >> > @@ -717,6 +722,14 @@ void __init smp_prepare_cpus(unsigned int max_cpus) >> > >> > stack_base[0] = stack_start; >> > >> > +nr_sockets = DIV_ROUND_U

Re: [Xen-devel] Xen 4.6 Development Update (three months reminder)

2015-05-05 Thread Wei Liu
On Mon, May 04, 2015 at 08:51:56PM +0800, Yu, Zhang wrote: > Hi Wei, > > Thanks for your reply. > > On 5/4/2015 5:44 PM, Wei Liu wrote: > >(Thanks for trimming the CC list before hand) > > > >On Mon, May 04, 2015 at 02:05:49PM +0800, Yu, Zhang wrote: > >>Hi Wei, > >> > > > >Hello. > > > >> This

Re: [Xen-devel] [PATCH] libxl: Disallow save or migrate when host devices are assigned to a guest.

2015-05-05 Thread Dario Faggioli
On Fri, 2015-05-01 at 15:32 -0600, Jim Fehlig wrote: > Dario Faggioli wrote: > > I mean what NUMA configuration are we talking about, and when and under > > what circumstances would you consider it "migratable"? > > > > E.g. if a domain is configured to use memory allocated from certain numa >

Re: [Xen-devel] [PATCH 1/3] xen/vt-d: need barriers to workaround CLFLUSH

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 04:45, wrote: > Does this work for everyone? Please first of all explain why the interfaces in asm/flushtlb.h can't be used here (at least when flushing entire pages). Because - as said before - for a complete fix you'd need to deal with the CLFLUSH use(s) elsewhere in the syst

[Xen-devel] [OSSTEST] (Probably benign) bug in create_webfile

2015-05-05 Thread Ian Campbell
In http://www.chiark.greenend.org.uk/~xensrcts/logs/37298/test-amd64-amd64-xl-qemut-debianhvm-amd64/info.html I was incorrectly passing a $ho to preseed_hook_script when I meant $gho, this meant that instead of webspace-lake-frog_debianhvm.guest.osstest_late_command-2 it ended up trying to create

Re: [Xen-devel] [PATCH v2 2/3] x86/hvm: introduce functions for HVMOP_get/set_param allowance checks

2015-05-05 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 01 May 2015 20:28 > To: Paul Durrant > Cc: Andrew Cooper; xen-de...@lists.xenproject.org; Keir (Xen.org) > Subject: Re: [PATCH v2 2/3] x86/hvm: introduce functions for > HVMOP_get/set_param allowance checks > > >>>

Re: [Xen-devel] [RFC][PATCH 10/13] tools: extend XENMEM_set_memory_map

2015-05-05 Thread Wei Liu
On Tue, Apr 14, 2015 at 08:42:39AM +0800, Chen, Tiejun wrote: > On 2015/4/13 19:02, Wei Liu wrote: > >On Mon, Apr 13, 2015 at 10:09:51AM +0800, Chen, Tiejun wrote: > >[...] > >>>Hardcoded value? > >> > >>Yes. Actually, we intend to use this to present that lowmem entry, > >> > >>tools/firmware/hvml

Re: [Xen-devel] Is data transfer encrypted in live migration?

2015-05-05 Thread Ian Campbell
On Sun, 2015-05-03 at 14:00 +0430, Alireza Vaezi wrote: > Hi > > does xen establish a secure (encrypted) channel via ssh to transfer > data in live migration? This is a function of the toolstack in use. For the xl toolstack the answer is yes, everything (including the guest memory contents) is tr

Re: [Xen-devel] Is data transfer encrypted in live migration?

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 10:33 +0100, Ian Campbell wrote: > On Sun, 2015-05-03 at 14:00 +0430, Alireza Vaezi wrote: > > Hi > > > > does xen establish a secure (encrypted) channel via ssh to transfer > > data in live migration? BTW, this is really a user support question, please you xen-users and not

Re: [Xen-devel] [PATCH v2 2/3] x86/hvm: introduce functions for HVMOP_get/set_param allowance checks

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 11:25, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 01 May 2015 20:28 >> And I continue to think that retaining these comments would be useful for >> documentation purposes (even when switching to white lists - they should >> then remain explicit case labels g

Re: [Xen-devel] [PATCH OSSTEST v5 04/24] Debian: refactor code to add preseed commands to the preseed file

2015-05-05 Thread Ian Campbell
On Mon, 2015-05-04 at 02:08 +, Pang, LongtaoX wrote: > I have verified this squash patch in my environment. > After push in this patch based on your former v5_patch [04,05,06], the > 'late_command' work fine. > So, I could add ' in-target sed -i 's/^deb > *cdrom/#&/g' /etc/apt/sources.list;'

[Xen-devel] [PATCH] xen: Disable RAM-to-RAM copy in hvmemul_rep_movs() when mem_access_emulate_enabled

2015-05-05 Thread Razvan Cojocaru
The mem_access client might want to use hvm_emulate_one_no_write(), in which case the RAM-to-RAM copy code in hvmemul_rep_movs() would lead to an unwanted (and unexpected) write operation. Signed-off-by: Razvan Cojocaru --- xen/arch/x86/hvm/emulate.c |8 1 file changed, 8 insertions

Re: [Xen-devel] [PATCH] xen: Disable RAM-to-RAM copy in hvmemul_rep_movs() when mem_access_emulate_enabled

2015-05-05 Thread Razvan Cojocaru
On 05/05/2015 01:01 PM, Razvan Cojocaru wrote: > The mem_access client might want to use hvm_emulate_one_no_write(), > in which case the RAM-to-RAM copy code in hvmemul_rep_movs() would > lead to an unwanted (and unexpected) write operation. This patch requires Tamas' last series, which is now in

Re: [Xen-devel] [PATCH v4 1/2] iommu VT-d: separate rmrr addition function

2015-05-05 Thread Jan Beulich
>>> On 28.04.15 at 01:50, wrote: > From: Elena Ufimtseva > > In preparation for auxiliary RMRR data provided on Xen > command line, make RMRR adding a separate function. > No functional changes. > > Reviewed-by: Konrad Rzeszutek Wilk > > Acked-by: Kevin Tian Are these the ones from v1? Did

Re: [Xen-devel] [PATCH v2 2/2] Update configure scripts by calling autogen

2015-05-05 Thread Ian Campbell
On Mon, 2015-05-04 at 12:19 +0100, Wei Liu wrote: > On Thu, Apr 16, 2015 at 12:24:59PM +0100, Stefano Stabellini wrote: > > I run ./autogen.sh and committed the result. > > > > This patch needs to be squashed into previous to keep the tree > bisectable. I assumed the intention was for the commit

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-05-05 Thread Wei Liu
On Wed, Apr 22, 2015 at 10:08:52AM +0200, Olaf Hering wrote: > On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: > > > On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: > > > +=item B > > > + > > > +Specifies the PVSCSI devices to be provided to the guest. PVSCSI passes > > > +dom0 SCSI devi

Re: [Xen-devel] [PATCH v4 2/2] iommu: add rmrr Xen command line option for misc rmrrs

2015-05-05 Thread Jan Beulich
>>> On 28.04.15 at 01:50, wrote: > Format for rmrr Xen command line option: > rmrr=start<-end>=[s1]bdf1[,[s1]bdf2[,...]];start<-end>=[s2]bdf1[,[s2]bdf2[,...]] > If grub used and multiple ranges are specified, ';' should be > quoted/escaped, > refer to grub man for more information. I think you m

Re: [Xen-devel] [PATCH] xen: Disable RAM-to-RAM copy in hvmemul_rep_movs() when mem_access_emulate_enabled

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 12:01, wrote: > The mem_access client might want to use hvm_emulate_one_no_write(), > in which case the RAM-to-RAM copy code in hvmemul_rep_movs() would > lead to an unwanted (and unexpected) write operation. I don't follow: hvm_emulate_one_no_write() uses hvm_emulate_ops_no_wr

Re: [Xen-devel] [Qemu-devel] [PATCH v6 3/6] Qemu-Xen-vTPM: Xen frontend driver infrastructure

2015-05-05 Thread Stefan Berger
On 05/04/2015 10:41 PM, Xu, Quan wrote: -Original Message- From: Stefan Berger [mailto:stef...@linux.vnet.ibm.com] Sent: Monday, May 04, 2015 11:36 PM To: Xu, Quan; stefano.stabell...@eu.citrix.com; ebl...@redhat.com Cc: qemu-de...@nongnu.org; wei.l...@citrix.com; dgde...@tycho.nsa.gov;

[Xen-devel] [PATCH v3 0/3] x86/hvm: HVMOP_get/set_param improvements

2015-05-05 Thread Paul Durrant
The following 3 patches re-structure the code implementing HVMOP_set_param and HVMOP_get_param. Patch #1 gives each operation its own function Patch #2 splits out checks for getting/setting non-reflexive params and setting params with change-once semantics, as well as the XSM check into separate

[Xen-devel] [PATCH v3 2/3] x86/hvm: introduce functions for HVMOP_get/set_param allowance checks

2015-05-05 Thread Paul Durrant
Some parameters can only (validly) be set once. Some should not be set by a guest for its own domain, and others must not be set since they require the domain to be paused. Consolidate these checks, along with the XSM check, in a new hvm_allow_set_param() function for clarity. Also, introduce hvm_

[Xen-devel] [PATCH v3 1/3] x86/hvm: give HVMOP_set_param and HVMOP_get_param their own functions

2015-05-05 Thread Paul Durrant
The level of switch nesting in those ops is getting unreadable. Giving them their own functions does introduce some code duplication in the the pre-op checks but the overall result is easier to follow. This patch is code movement (including style fixes). There is no functional change. Signed-off-

[Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Paul Durrant
There are actually very few HVM parameters that a guest needs to read and even fewer that a guest needs to write. Use white-lists to specify those parameters and also ensre that, by default, newly introduced parameters are not accessible. Signed-off-by: Paul Durrant Cc: Keir Fraser Cc: Jan Beuli

Re: [Xen-devel] [PATCH v6 01/14] x86: add socket_to_cpumask

2015-05-05 Thread Chao Peng
On Tue, May 05, 2015 at 10:11:15AM +0100, Jan Beulich wrote: > >>> On 05.05.15 at 09:44, wrote: > > On Fri, Apr 24, 2015 at 03:46:11PM +0100, Jan Beulich wrote: > >> >>> On 23.04.15 at 11:55, wrote: > >> > @@ -717,6 +722,14 @@ void __init smp_prepare_cpus(unsigned int max_cpus) > >> > > >> >

Re: [Xen-devel] Xen-unstable + linux 4.1-mergewindow: problems with PV guest pci passthrough: pcifront pci-0: pciback not responding!!!

2015-05-05 Thread David Vrabel
On 04/05/15 23:18, Sander Eikelenboom wrote: > Hello Sander, > > Monday, April 27, 2015, 5:48:00 PM, you wrote: > >> Hi David / Konrad, > >> Here the other problem i found, which is introduced somewhere in the >> 4.1 mergewindow: > >> on 4.1.0-rc1 (with the one revert to get things booting) i

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Stefano Stabellini
On Mon, 4 May 2015, Julien Grall wrote: > Hi Vijay, > > On 04/05/2015 16:19, Vijay Kilari wrote: > > > > > > How did you implement the interrupt mode? Could it be improve? > > > > > > > > > > > > > > > 1) In physical ITS driver its_device is created with devID > > > > > 00:00.1 > > > > > wi

Re: [Xen-devel] [PATCH v3 2/3] x86/hvm: introduce functions for HVMOP_get/set_param allowance checks

2015-05-05 Thread Andrew Cooper
On 05/05/15 11:25, Paul Durrant wrote: > Some parameters can only (validly) be set once. Some should not be set > by a guest for its own domain, and others must not be set since they > require the domain to be paused. Consolidate these checks, along with > the XSM check, in a new hvm_allow_set_para

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Stefano Stabellini
On Mon, 4 May 2015, Vijay Kilari wrote: > On Thu, Apr 30, 2015 at 7:59 PM, Julien Grall wrote: > > Hi, > > > > On 30/04/15 14:47, Stefano Stabellini wrote: > > > > If the devid is not within this range, the ITS won't recognize the > > value and > > won't be able to send the interr

[Xen-devel] [GIT PULL OSSTEST] Merge up changes from Cambridge instance.

2015-05-05 Thread Ian Campbell
The following changes since commit 6587e4cb8f336d2c9774b3b8d3ca88d6ebbc03e2: Reduce guest ssh check up timeout to 10s (2015-04-02 12:50:29 +0100) are available in the git repository at: git://xenbits.xen.org/people/ianc/osstest.git origin-cam/production for you to fetch changes up to e7773b

Re: [Xen-devel] [PATCHv7 1/3] gnttab: Introduce rwlock to protect updates to grant table state

2015-05-05 Thread Jan Beulich
>>> On 30.04.15 at 15:28, wrote: > --- a/docs/misc/grant-tables.txt > +++ b/docs/misc/grant-tables.txt > @@ -74,7 +74,33 @@ is complete. > matching map track entry is then removed, as if unmap had been invoked. > These are not used by the transfer mechanism. >map->domid : owner of

Re: [Xen-devel] [PATCH v6 01/14] x86: add socket_to_cpumask

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 12:25, wrote: > On Tue, May 05, 2015 at 10:11:15AM +0100, Jan Beulich wrote: >> >>> On 05.05.15 at 09:44, wrote: >> > On Fri, Apr 24, 2015 at 03:46:11PM +0100, Jan Beulich wrote: >> >> >>> On 23.04.15 at 11:55, wrote: >> >> > @@ -717,6 +722,14 @@ void __init smp_prepare_cpus(u

Re: [Xen-devel] [GIT PULL OSSTEST] Merge up changes from Cambridge instance.

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 11:40 +0100, Ian Campbell wrote: > The following changes since commit 6587e4cb8f336d2c9774b3b8d3ca88d6ebbc03e2: > > Reduce guest ssh check up timeout to 10s (2015-04-02 12:50:29 +0100) > > are available in the git repository at: > > git://xenbits.xen.org/people/ianc/oss

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Andrew Cooper
On 05/05/15 11:25, Paul Durrant wrote: > There are actually very few HVM parameters that a guest needs to read > and even fewer that a guest needs to write. Use white-lists to specify > those parameters and also ensre that, by default, newly introduced > parameters are not accessible. > > Signed-of

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 3:58 PM, Stefano Stabellini wrote: > On Mon, 4 May 2015, Julien Grall wrote: >> Hi Vijay, >> >> On 04/05/2015 16:19, Vijay Kilari wrote: >> > > > > > How did you implement the interrupt mode? Could it be improve? >> > > > > >> > > > > >> > > > > 1) In physical ITS drive

[Xen-devel] [PATCH OSSTEST v2 0/2] A pair of XSM fixes

2015-05-05 Thread Ian Campbell
The first of these was posted before and even applied to pretest but it was broken (incorrect operator precedence). The second is new. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Julien Grall
On 05/05/15 11:28, Stefano Stabellini wrote: > On Mon, 4 May 2015, Julien Grall wrote: >> Hi Vijay, >> >> On 04/05/2015 16:19, Vijay Kilari wrote: >>> How did you implement the interrupt mode? Could it be improve? >> >> >> 1) In physical ITS driver its_device is created with de

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Julien Grall
On 05/05/15 11:39, Stefano Stabellini wrote: > On Mon, 4 May 2015, Vijay Kilari wrote: >> On Thu, Apr 30, 2015 at 7:59 PM, Julien Grall >> wrote: >>> Hi, >>> >>> On 30/04/15 14:47, Stefano Stabellini wrote: >>> >>> If the devid is not within this range, the ITS won't recognize the >>

Re: [Xen-devel] [PATCH] xen: Disable RAM-to-RAM copy in hvmemul_rep_movs() when mem_access_emulate_enabled

2015-05-05 Thread Razvan Cojocaru
On 05/05/2015 01:23 PM, Jan Beulich wrote: On 05.05.15 at 12:01, wrote: >> The mem_access client might want to use hvm_emulate_one_no_write(), >> in which case the RAM-to-RAM copy code in hvmemul_rep_movs() would >> lead to an unwanted (and unexpected) write operation. > > I don't follow: hv

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Ian Campbell
On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: > >> diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h > >> index 5330dfe..0149d06 100644 > >> --- a/xen/include/asm-arm/event.h > >> +++ b/xen/include/asm-arm/event.h > >> @@ -39,7 +39,12 @@ static inline int > >> local_e

Re: [Xen-devel] [PATCH] config: generate Paths-$SUBSYSTEM.mk

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 11:21 +0100, Wei Liu wrote: > On Thu, Apr 23, 2015 at 11:15:03AM +0100, Ian Campbell wrote: > > On Thu, 2015-04-23 at 10:10 +0100, Wei Liu wrote: > > > ... so that we can run ./configure in each subsystem. > > > > > > Make inclusion of Paths-*.mk mandatory. > > > > Having do

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 4:38 PM, Julien Grall wrote: > On 05/05/15 11:28, Stefano Stabellini wrote: >> On Mon, 4 May 2015, Julien Grall wrote: >>> Hi Vijay, >>> >>> On 04/05/2015 16:19, Vijay Kilari wrote: How did you implement the interrupt mode? Could it be improve? >>> >>> >

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Julien Grall
On 05/05/15 12:06, Vijay Kilari wrote: >One MSI per domain is always consumed because each domain during > ITS initialization creates Virtual ITS and sends basic initialization > commands (MAPC) AFAICT, MAPC won't be translated to a physical command. So not interrupt completion is necessary.

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Ian Campbell wrote: > On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: > > >> diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h > > >> index 5330dfe..0149d06 100644 > > >> --- a/xen/include/asm-arm/event.h > > >> +++ b/xen/include/asm-arm/event.h > > >

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Julien Grall wrote: > On 05/05/15 11:28, Stefano Stabellini wrote: > > On Mon, 4 May 2015, Julien Grall wrote: > >> Hi Vijay, > >> > >> On 04/05/2015 16:19, Vijay Kilari wrote: > >>> How did you implement the interrupt mode? Could it be improve? > >> > >> > >>

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Julien Grall wrote: > On 05/05/15 11:39, Stefano Stabellini wrote: > > On Mon, 4 May 2015, Vijay Kilari wrote: > >> On Thu, Apr 30, 2015 at 7:59 PM, Julien Grall > >> wrote: > >>> Hi, > >>> > >>> On 30/04/15 14:47, Stefano Stabellini wrote: > >>> > >>> If the devid is

Re: [Xen-devel] [PATCHv7 2/3] gnttab: refactor locking for scalability

2015-05-05 Thread Jan Beulich
>>> On 30.04.15 at 15:28, wrote: >>> On 30.04.15 at 15:28, wrote: > This patch refactors grant table locking. It splits the previous > single spinlock per grant table into multiple locks. The heavily > modified components of the grant table (the maptrack state and the > active entries) are now pr

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Vijay Kilari
On Tue, May 5, 2015 at 5:17 PM, Julien Grall wrote: > On 05/05/15 12:06, Vijay Kilari wrote: >>One MSI per domain is always consumed because each domain during >> ITS initialization creates Virtual ITS and sends basic initialization >> commands (MAPC) > > AFAICT, MAPC won't be translated to a

Re: [Xen-devel] [libvirt] [PATCH] libxl: support soundhw for hvm domains

2015-05-05 Thread Michal Privoznik
On 04.05.2015 22:35, Jim Fehlig wrote: > The xend driver and the parsing/formating code in src/xenconfig > have long supported soundhw. Add support in the libxl driver too. > > Signed-off-by: Jim Fehlig > --- > > This patch continues the tradition of silently ignoring unsupported > config in th

Re: [Xen-devel] [PATCH] OSSTEST: introduce a raisin build test

2015-05-05 Thread Ian Campbell
On Fri, 2015-04-24 at 16:46 +0100, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini This looks like a good start, a few comments below. > diff --git a/ap-common b/ap-common > index 64749e3..985eeec 100644 > --- a/ap-common > +++ b/ap-common > @@ -47,13 +47,18 @@ > # rumpsrc-related

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Julien Grall
On 05/05/15 12:57, Stefano Stabellini wrote: > On Tue, 5 May 2015, Julien Grall wrote: >>> I think we should pause the guest vcpu until we receive >>> the interrupt. >>> You can do that by calling vcpu_block. >> >> Blocking the vcpu is not the right thing to do. The processing of the >> ITS command

Re: [Xen-devel] [PATCH] libxl: initialize vfb defbools in libxlMakeVfb

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 09:10 -0600, Jim Fehlig wrote: > Ian Campbell wrote: > > On Fri, 2015-04-17 at 13:57 -0600, Jim Fehlig wrote: > > > >> On 04/17/2015 11:59 AM, Olaf Hering wrote: > >> > >>> On Fri, Apr 17, Olaf Hering wrote: > >>> > >>> > If the domU configu has sdl enabled

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Julien Grall
Hi Ian, On 05/05/15 12:40, Ian Campbell wrote: > On Mon, 2015-04-27 at 17:32 +0100, Julien Grall wrote: diff --git a/xen/include/asm-arm/event.h b/xen/include/asm-arm/event.h index 5330dfe..0149d06 100644 --- a/xen/include/asm-arm/event.h +++ b/xen/include/asm-arm/event.h

Re: [Xen-devel] [PATCH v2 0/4] raisin: introduce tests

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 16:48 +0100, Stefano Stabellini wrote: > Hi all, > > this patch series introduces a framework to execute simple unit and > functional tests in raisin. It can be used by developers to validate > their changes before submitting a patch series to xen-devel. It can also > be used

Re: [Xen-devel] [RFC PATCH v2 13/22] xen/arm: its: Add virtual ITS command support

2015-05-05 Thread Julien Grall
On 05/05/15 13:00, Vijay Kilari wrote: > On Tue, May 5, 2015 at 5:17 PM, Julien Grall wrote: >> On 05/05/15 12:06, Vijay Kilari wrote: >>>One MSI per domain is always consumed because each domain during >>> ITS initialization creates Virtual ITS and sends basic initialization >>> commands (MAP

[Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Vijay Kilari
Hi, As discussed, here is the design doc/txt. ARM GICv3 provides ITS (Interrupt Translation Service) feature to handle MSIx interrupts. Below are various mechanisms to handle ITS commands. ITS command completion detection mechanism: ---

Re: [Xen-devel] [PATCH] libxl: convert strings and ints to xenbus_state

2015-05-05 Thread Ian Campbell
On Fri, 2015-04-24 at 10:17 +0100, Wei Liu wrote: > On Fri, Apr 24, 2015 at 09:07:14AM +, Olaf Hering wrote: > > Convert all plain ints and strings which are used for xenbus "state" > > files to xenbus_state. This makes it easier to find code which deals > > with backend/frontend state changes.

Re: [Xen-devel] [PATCH] xen/arm: p2m: Add an ASSERT to check that p2m lock is taken in __p2m_lookup

2015-05-05 Thread Ian Campbell
On Mon, 2015-04-27 at 15:58 +0100, Julien Grall wrote: > __p2m_lookup should be called with the p2m lock taken. Add an ASSERT in > order to catch wrong caller in debug build. > > Signed-off-by: Julien Grall Acked + applied, thanks. ___ Xen-devel mai

Re: [Xen-devel] [PATCH] tools/libxc: Set HVM_PARAM_CONSOLE_EVTCHN during restore

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 10:02 -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Apr 23, 2015 at 10:24:04AM +0100, Andrew Cooper wrote: > > On 23/04/15 03:49, Boris Ostrovsky wrote: > > > When resuming, the guest needs to check whether the port has changed. HVM > > > guests use this parameter to get the p

Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk

2015-05-05 Thread Ian Campbell
On Wed, 2015-04-29 at 15:08 +0530, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enab

Re: [Xen-devel] [PATCH] xen/arm: Make local_events_need_delivery working with idle VPCU

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 13:00 +0100, Julien Grall wrote: > > I'm inclined to just go with this patch for now, unless Stefano is > > nacking it. > > This patch seem to turn into a workaround, would it be better to move > check idle_check in apply_p2m_check? > > I will prepare a follow-up to avoid pr

Re: [Xen-devel] [PATCHv7 3/3] gnttab: use per-VCPU maptrack free lists

2015-05-05 Thread Jan Beulich
>>> On 30.04.15 at 15:28, wrote: > From: Malcolm Crossley > > Performance analysis of aggregate network throughput with many VMs > shows that performance is signficantly limited by contention on the > maptrack lock when obtaining/releasing maptrack handles from the free > list. > > Instead of a

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

2015-05-05 Thread osstest service user
flight 53015 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/53015/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-amd64 13 guest-localmigrate fail REGR. vs. 52776 test-amd64-amd64-xl-qemut-de

Re: [Xen-devel] [PATCH v6 12/14] tools/libxl: introduce some socket helpers

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: > Add libxl_socket_bitmap_alloc() to allow allocating a socket specific > libxl_bitmap (as it is for cpu/node bitmap). > > Internal function libxl__count_physical_sockets() is introduced together > to get the socket count when the size of bitmap

Re: [Xen-devel] [PATCH v6 12/14] tools/libxl: introduce some socket helpers

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 13:43 +0100, Ian Campbell wrote: > On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: > > Add libxl_socket_bitmap_alloc() to allow allocating a socket specific > > libxl_bitmap (as it is for cpu/node bitmap). > > > > Internal function libxl__count_physical_sockets() is intro

Re: [Xen-devel] [PATCH v6 13/14] tools: add tools support for Intel CAT

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: > This is the xc/xl changes to support Intel Cache Allocation > Technology(CAT). > > 'xl psr-hwinfo' is updated to show CAT info and two new commands > for CAT are introduced: > - xl psr-cat-cbm-set [-s socket] > Set cache capacity bitmasks(C

Re: [Xen-devel] [PATCH v6 14/14] docs: add xl-psr.markdown

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 17:55 +0800, Chao Peng wrote: > +Maximum RMID and supported monitor types in the system can be obtained by: > + > +`xl psr-cmt-hwinfo` I think this is now `xl psr-hwinfo --cmt` ? > +For example, assuming a system with 8 portions and 3 domains: > + > + * A CBM of 0xff for ev

Re: [Xen-devel] [PATCH v10 02/15] libxc/progress: Extend the progress interface

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: > Progress information is logged via a different logger to regular libxc log > messages, and currently can only express a range. However, not everything > which needs reporting as progress comes with a range. Extend the interface to > allow

Re: [Xen-devel] [PATCH v10 09/15] tools/libxc: x86 PV restore code

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: > + * Set a pfn as populated, expanding the tracking structures if needed. To > + * avoid realloc()ing too excessivly, the size increased to the nearest power "excessively". Otherwise: Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: > From: David Vrabel > > Add the specification for a new migration stream format. The document > includes all the details but to summarize: > > The existing (legacy) format is dependant on the word size of the > toolstack. This prevents d

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Andrew Cooper
On 05/05/15 14:03, Ian Campbell wrote: > On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: >> From: David Vrabel >> >> Add the specification for a new migration stream format. The document >> includes all the details but to summarize: >> >> The existing (legacy) format is dependant on the w

Re: [Xen-devel] Xen 4.6 Development Update (three months reminder)

2015-05-05 Thread Yu, Zhang
Thank you, Wei. On 5/5/2015 5:12 PM, Wei Liu wrote: On Mon, May 04, 2015 at 08:51:56PM +0800, Yu, Zhang wrote: Hi Wei, Thanks for your reply. On 5/4/2015 5:44 PM, Wei Liu wrote: (Thanks for trimming the CC list before hand) On Mon, May 04, 2015 at 02:05:49PM +0800, Yu, Zhang wrote: Hi Wei,

Re: [Xen-devel] [PATCH v4 0/3] Set SMMU s2 input-size based on p2m tables

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Hi, > > This is a fix for the issue I'm seeing on ZynqMP with missmatched > setup of the SMMU and the shared p2m page-tables with the CPU. Looking back at previous conversations it seems like your SMMU h

Re: [Xen-devel] [PATCH v4 1/3] xen/arm: Re-order iommu_setup to after setup_virt_paging

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > This is needed to allow the paging setup to probe for > IPA bit sizes to be used in p2m tables prior to iommu setup. > > Reviewed-by: Julien Grall > Signed-off-by: Edgar E. Iglesias This seems like a m

Re: [Xen-devel] [PATCH v4 2/3] xen/arm: Add p2m_ipa_bits

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > Export p2m_ipa_bits holding the bit size of IPAs used in p2m tables. > > Signed-off-by: Edgar E. Iglesias > --- > xen/arch/arm/p2m.c| 5 + > xen/include/asm-arm/p2m.h | 3 +++ > 2 files chan

Re: [Xen-devel] [PATCH v10 14/15] docs: libxc migration stream specification

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 14:04 +0100, Andrew Cooper wrote: > On 05/05/15 14:03, Ian Campbell wrote: > > On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: > >> From: David Vrabel > >> > >> Add the specification for a new migration stream format. The document > >> includes all the details but to

Re: [Xen-devel] [PATCH v4 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-05-05 Thread Ian Campbell
On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: > From: "Edgar E. Iglesias" > > The Stage2 input-size must match what the CPU uses because > the SMMU and the CPU share page-tables. > > Test that the SMMU supports the P2M IPA bit size, use it if > supported or bail out if not. > > Si

Re: [Xen-devel] [PATCH v4 0/3] Set SMMU s2 input-size based on p2m tables

2015-05-05 Thread Julien Grall
Hi Ian, On 05/05/15 14:17, Ian Campbell wrote: > On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: >> From: "Edgar E. Iglesias" >> >> Hi, >> >> This is a fix for the issue I'm seeing on ZynqMP with missmatched >> setup of the SMMU and the shared p2m page-tables with the CPU. > > Lookin

Re: [Xen-devel] [PATCHv4 4/8] arm: provide add_sized()

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: > add_sized(ptr, inc) adds inc to the value at ptr using only the correct > size of loads and stores for the type of *ptr. The add is /not/ atomic. atomic.h is an odd place for it then ;-) But given you use the infrastructure I can't suggest

Re: [Xen-devel] [PATCH v4 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-05-05 Thread Julien Grall
Hi Ian, On 05/05/15 14:24, Ian Campbell wrote: > On Fri, 2015-05-01 at 11:40 +1000, Edgar E. Iglesias wrote: >> From: "Edgar E. Iglesias" >> >> The Stage2 input-size must match what the CPU uses because >> the SMMU and the CPU share page-tables. >> >> Test that the SMMU supports the P2M IPA bit s

Re: [Xen-devel] [PATCHv4 6/8] x86, arm: remove asm/spinlock.h from all architectures

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: > Now that all architecture use a common ticket lock implementation for > spinlocks, remove the architecture specific byte lock implementations. > > Signed-off-by: David Vrabel > Reviewed-by: Tim Deegan > Acked-by: Jan Beulich Acked-by: Ia

Re: [Xen-devel] [PATCH v2] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-05-05 Thread Andy Lutomirski
On Mon, May 4, 2015 at 8:02 AM, Boris Ostrovsky wrote: > Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor > attribute issue") makes AMD processors set SS to __KERNEL_DS in > __switch_to() to deal with cases when SS is NULL. > > This breaks Xen PV guests who do not want to lo

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Vijay Kilari wrote: > Proposal 2: > > Here when guest writes command to vITS queue and updates CWRITER registers, > it is trapped in XEN and below steps are followed to process ITS command > > - Dom0 creates a ITS completion device with device id (00:00.1) and

Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk

2015-05-05 Thread Julien Grall
Hi Pranav, On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable E

Re: [Xen-devel] [PATCHv4 4/8] arm: provide add_sized()

2015-05-05 Thread Tim Deegan
At 14:49 +0100 on 05 May (1430837355), Ian Campbell wrote: > On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: > > add_sized(ptr, inc) adds inc to the value at ptr using only the correct > > size of loads and stores for the type of *ptr. The add is /not/ atomic. > > atomic.h is an odd place

Re: [Xen-devel] Xen/arm: Virtual ITS command queue handling

2015-05-05 Thread Julien Grall
On 05/05/15 14:51, Stefano Stabellini wrote: >> - With this irq/LPI descriptor we can identify the domain/vITS. >> - Info of all the ongoing ITS requests(put in pITS Queue) of this domain is >> stored in ITS command status array (called its_requests). This is >> managed per vITS. >> >> 1) Trap of

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-05-05 Thread Wei Liu
On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: > Port pvscsi support from xend to libxl: > > vscsi=['pdev,vdev{,options}'] > xl scsi-attach > xl scsi-detach > xl scsi-list > > Signed-off-by: Olaf Hering > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei

Re: [Xen-devel] [PATCH v2 0/4] raisin: introduce tests

2015-05-05 Thread Stefano Stabellini
On Tue, 5 May 2015, Ian Campbell wrote: > On Fri, 2015-05-01 at 16:48 +0100, Stefano Stabellini wrote: > > Hi all, > > > > this patch series introduces a framework to execute simple unit and > > functional tests in raisin. It can be used by developers to validate > > their changes before submittin

Re: [Xen-devel] [PATCHv4 5/8] xen: use ticket locks for spin locks

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: > > void _spin_lock_irq(spinlock_t *lock) > { > -LOCK_PROFILE_VAR; > - > ASSERT(local_irq_is_enabled()); > local_irq_disable(); > -check_lock(&lock->debug); > -while ( unlikely(!_raw_spin_trylock(&lock->raw)) ) > -

Re: [Xen-devel] [PATCHv4 5/8] xen: use ticket locks for spin locks

2015-05-05 Thread David Vrabel
On 05/05/15 14:56, Ian Campbell wrote: > On Thu, 2015-04-30 at 16:33 +0100, David Vrabel wrote: >> >> void _spin_lock_irq(spinlock_t *lock) >> { >> -LOCK_PROFILE_VAR; >> - >> ASSERT(local_irq_is_enabled()); >> local_irq_disable(); >> -check_lock(&lock->debug); >> -while (

Re: [Xen-devel] [PATCH v4 3/3] xen/iommu: arm: Use p2m_ipa_bits as stage2 input size

2015-05-05 Thread Ian Campbell
On Tue, 2015-05-05 at 14:48 +0100, Julien Grall wrote: > > Do we need to also check that we are configuring the same number of > > levels of PT etc, or is that already handled? > > The SMMU only care about the number of IPA bits. What ensures that the starting level of the SMMU matches the starti

Re: [Xen-devel] [PATCH v4 4/5] vscsiif.h: add some notes about xenstore layout

2015-05-05 Thread Wei Liu
On Fri, Apr 17, 2015 at 08:30:59AM +, Olaf Hering wrote: > Signed-off-by: Olaf Hering > Cc: Ian Campbell > Cc: Ian Jackson > Cc: Jan Beulich > Cc: Keir Fraser > Cc: Tim Deegan > --- > xen/include/public/io/vscsiif.h | 68 > + > 1 file changed, 68

Re: [Xen-devel] [PATCH v10 00/15] Migration v2 (libxc)

2015-05-05 Thread Ian Campbell
On Thu, 2015-04-23 at 12:48 +0100, Andrew Cooper wrote: > Presented here is v10 of the Migration v2 series (libxc subset), which is able > to function when transparently inserted under an unmodified xl/libxl. I've applied the v11 branch with my minor comments addressed which you provided privately

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: 05 May 2015 11:54 > To: Paul Durrant; xen-de...@lists.xenproject.org > Cc: Keir (Xen.org); Jan Beulich > Subject: Re: [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest > accessibility checks > >

Re: [Xen-devel] [PATCH v2] hypervisor/x86/xen: Unset X86_BUG_SYSRET_SS_ATTRS on Xen PV guests

2015-05-05 Thread Boris Ostrovsky
On 05/05/2015 09:51 AM, Andy Lutomirski wrote: On Mon, May 4, 2015 at 8:02 AM, Boris Ostrovsky wrote: Commit 61f01dd941ba ("x86_64, asm: Work around AMD SYSRET SS descriptor attribute issue") makes AMD processors set SS to __KERNEL_DS in __switch_to() to deal with cases when SS is NULL. This b

Re: [Xen-devel] Next Document Day Moved to Wednesday, May 6

2015-05-05 Thread Russ Pavlicek
Reminder: Our Document Day is tomorrow! Theme for this month: "No-Can-Do Without a HowTo" We've got a lot of good information in the documentation, but it isn't always in a form that will help someone use a particular feature for the first time. Or, the information is present, but it might need t

Re: [Xen-devel] [PATCH v3 3/3] x86/hvm: Use white-lists for HVM param guest accessibility checks

2015-05-05 Thread Jan Beulich
>>> On 05.05.15 at 16:09, wrote: >> From: Andrew Cooper [mailto:andrew.coop...@citrix.com] >> Sent: 05 May 2015 11:54 >> On 05/05/15 11:25, Paul Durrant wrote: >> > @@ -5657,21 +5664,11 @@ static int hvm_allow_set_param(struct domain >> *d, >> > case HVM_PARAM_IDENT_PT: >> > case HVM_PAR

  1   2   >