Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Jan Beulich
>>> On 28.09.15 at 18:09, wrote: > El 21/09/15 a les 17.44, Jan Beulich ha escrit: > On 04.09.15 at 14:09, wrote: >>> Allow the usage of the VCPUOP_initialise, VCPUOP_up, VCPUOP_down and >>> VCPUOP_is_up hypercalls from HVM guests. >>> >>> This patch introduces a new structure (vcpu_hvm_conte

Re: [Xen-devel] [PATCH for Xen 4.6 3/5] tools/libxl: return socket id from libxl_psr_cat_get_l3_info

2015-09-29 Thread Dario Faggioli
On Tue, 2015-09-29 at 11:05 +0800, Chao Peng wrote: > On Mon, Sep 28, 2015 at 04:46:17PM +0100, Wei Liu wrote: > > On Mon, Sep 28, 2015 at 05:35:56PM +0200, Dario Faggioli wrote: > > > But since now you're building the full bitmap, we can use > > > libxl_bitmap_count_set(), for that. > > > > > >

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 04:53, wrote: Monday, September 28, 2015 2:47 PM, wrote: >> >>> On 28.09.15 at 05:08, wrote: >> Thursday, September 24, 2015 12:27 AM, Tim Deegan wrote: > >> It would be a guest kernel bug, but all _we_ care about is that such a guest > kernel >> bug won't affect th

Re: [Xen-devel] [PATCH 1/3] x86/p2m: tighten conditions of IOMMU mapping updates

2015-09-29 Thread Jan Beulich
>>> On 28.09.15 at 18:32, wrote: > On 21/09/15 15:02, Jan Beulich wrote: >> In the EPT case permission changes should also result in updates or >> TLB flushes. >> >> In the NPT case the old MFN does not depend on the new entry being >> valid (but solely on the old one), and the need to update or

[Xen-devel] [linux-3.18 baseline-only test] 38087: tolerable FAIL

2015-09-29 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38087 linux-3.18 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38087/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: test-armhf-armhf-libvirt-qcow2 9 debian-di-insta

[Xen-devel] [PATCH v2 for Xen 4.6 3/6] tools/libxl: return socket id from libxl_psr_cat_get_l3_info

2015-09-29 Thread Chao Peng
The entries returned from libxl_psr_cat_get_l3_info are assumed to be socket-continuous. But this is not true in the hotplug case. This patch gets the socket bitmap for all the sockets on the system first and stores the socket id in the structure libxl_psr_cat_info in libxl_psr_cat_get_l3_info. Th

[Xen-devel] [PATCH v2 for Xen 4.6 1/6] tools/libxl: introduce libxl_get_online_socketmap

2015-09-29 Thread Chao Peng
It sets the bit on the given bitmap if the corresponding socket is available and clears the bit when the corresponding socket is not available. Signed-off-by: Chao Peng --- v2: * rename libxl_socket_bitmap_fill => libxl_get_online_socketmap. * fix blanklines. NOTE:LIBXL_HAVE_SOCKET_BITMAP_ALLOC

[Xen-devel] [PATCH v2 for Xen 4.6 6/6] docs/man: resort sections

2015-09-29 Thread Chao Peng
Section 'IGNORED FOR COMPATIBILITY WITH XM' separates 'CACHE MONITORING TECHNOLOGY' and 'CACHE ALLOCATION TECHNOLOGY' but they really should be put together. Signed-off-by: Chao Peng --- Current incorrect output can be seen at: http://xenbits.xen.org/docs/unstable/man/xl.1.html --- docs/man/xl.

[Xen-devel] [PATCH v2 for Xen 4.6 0/6] Several PSR fixes in libxl

2015-09-29 Thread Chao Peng
The patch series basically contain several PSR related fixes in libxl. patch1-3: fix the socket display error in certain hotplug case. patch4: fix a minor range check. patch5: improve the PSR document. patch6: improve xl man page. Detailed problem and fix please see commit message. Change h

[Xen-devel] [PATCH v2 for Xen 4.6 5/6] docs: make xl-psr.markdown more precise

2015-09-29 Thread Chao Peng
Drop the chapter number as it can be confusing when it gets changed in the referred document. Signed-off-by: Chao Peng Reviewed-by: Dario Faggioli Acked-by: Wei Liu --- v2: * minor commit message adjustment. --- docs/misc/xl-psr.markdown | 8 1 file changed, 4 insertions(+), 4 deletio

[Xen-devel] [PATCH v2 for Xen 4.6 2/6] tools/libxl: fix socket display error for CMT

2015-09-29 Thread Chao Peng
When displaying the CMT information for all the sockets, we assume socket number is continuous. This is not true in the hotplug case. For instance, when the 3rd socket is plugged out on a 4-socket system, the available sockets numbers are 1,2,4 but current we will display the CMT information for so

Re: [Xen-devel] [PATCH 1/3] x86/p2m: tighten conditions of IOMMU mapping updates

2015-09-29 Thread Jan Beulich
>>> On 21.09.15 at 16:02, wrote: > In the EPT case permission changes should also result in updates or > TLB flushes. > > In the NPT case the old MFN does not depend on the new entry being > valid (but solely on the old one), and the need to update or TLB-flush > again also depends on permission

Re: [Xen-devel] [V5 3/4] x86/xsaves: enable xsaves/xrstors for hvm guest

2015-09-29 Thread Jan Beulich
>>> On 21.09.15 at 13:34, wrote: > --- a/xen/arch/x86/hvm/hvm.c > +++ b/xen/arch/x86/hvm/hvm.c > @@ -4550,6 +4550,23 @@ void hvm_cpuid(unsigned int input, unsigned int *eax, > unsigned int *ebx, > *ebx = _eax + _ebx; > } > } > +if ( count == 1 )

Re: [Xen-devel] [Patch RFC 01/13] vt-d: Redefine iommu_set_interrupt() for registering MSI interrupt

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:23, wrote: > Signed-off-by: Quan Xu The title isn't really meaningful, and there's no description. > @@ -1084,8 +1086,8 @@ static int __init iommu_set_interrupt(struct > acpi_drhd_unit *drhd) > } > > desc = irq_to_desc(irq); > -desc->handler = &dma_msi_typ

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

2015-09-29 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38088 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38088/ Perfect :-) All tests in this flight passed version targeted for testing: ovmf 28f27af6f007c3794fcc9d098ef91713160f4e5b baseline version: ovm

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Tim Deegan
At 01:09 -0600 on 29 Sep (1443488980), Jan Beulich wrote: > >>> On 28.09.15 at 18:09, wrote: > > El 21/09/15 a les 17.44, Jan Beulich ha escrit: > > On 04.09.15 at 14:09, wrote: > >>> +if ( hvm_paging_enabled(v) && !paging_mode_hap(v->domain) ) > >>> +{ > >>> +/* Shadow-mode C

Re: [Xen-devel] [PATCH v2 for Xen 4.6 6/6] docs/man: resort sections

2015-09-29 Thread Dario Faggioli
On Tue, 2015-09-29 at 15:49 +0800, Chao Peng wrote: > Section 'IGNORED FOR COMPATIBILITY WITH XM' separates 'CACHE > MONITORING > TECHNOLOGY' and 'CACHE ALLOCATION TECHNOLOGY' but they really should > be > put together. > Indeed. > Signed-off-by: Chao Peng > Reviewed-by: Dario Faggioli Thanks

Re: [Xen-devel] [Patch RFC 02/13] vt-d: Register MSI for async invalidation completion interrupt.

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:23, wrote: > +/* IOMMU Queued Invalidation(QI). */ > +static void _qi_msi_unmask(struct iommu *iommu) > +{ > +u32 sts; > +unsigned long flags; > + > +/* Clear IM bit of DMAR_IECTL_REG. */ > +spin_lock_irqsave(&iommu->register_lock, flags); > +sts = dmar_re

Re: [Xen-devel] [PATCH v2 for Xen 4.6 6/6] docs/man: resort sections

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 03:49:55PM +0800, Chao Peng wrote: > Section 'IGNORED FOR COMPATIBILITY WITH XM' separates 'CACHE MONITORING > TECHNOLOGY' and 'CACHE ALLOCATION TECHNOLOGY' but they really should be > put together. > > Signed-off-by: Chao Peng Acked-by: Wei Liu

Re: [Xen-devel] [PATCH v2 for Xen 4.6 2/6] tools/libxl: fix socket display error for CMT

2015-09-29 Thread Dario Faggioli
On Tue, 2015-09-29 at 15:49 +0800, Chao Peng wrote: > When displaying the CMT information for all the sockets, we assume > socket > number is continuous. This is not true in the hotplug case. For > instance, > when the 3rd socket is plugged out on a 4-socket system, the > available > sockets number

[Xen-devel] [linux-3.18 test] 62450: regressions - FAIL

2015-09-29 Thread osstest service owner
flight 62450 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62450/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 11 guest-start fail REGR. vs. 58581 Tests which are failin

Re: [Xen-devel] [PATCH v2 for Xen 4.6 3/6] tools/libxl: return socket id from libxl_psr_cat_get_l3_info

2015-09-29 Thread Dario Faggioli
On Tue, 2015-09-29 at 15:49 +0800, Chao Peng wrote: > The entries returned from libxl_psr_cat_get_l3_info are assumed > to be socket-continuous. But this is not true in the hotplug case. > > This patch gets the socket bitmap for all the sockets on the system > first and stores the socket id in the

Re: [Xen-devel] [PATCH v2 for Xen 4.6 1/6] tools/libxl: introduce libxl_get_online_socketmap

2015-09-29 Thread Dario Faggioli
On Tue, 2015-09-29 at 15:49 +0800, Chao Peng wrote: > It sets the bit on the given bitmap if the corresponding socket is > available and clears the bit when the corresponding socket is not > available. > > Signed-off-by: Chao Peng > --- > v2: > * rename libxl_socket_bitmap_fill => libxl_get_onlin

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-09-29 Thread Tim Deegan
Hi, At 03:08 + on 28 Sep (1443409723), Xu, Quan wrote: > >>> Thursday, September 24, 2015 12:27 AM, Tim Deegan wrote: > > 7/13: I'm not convinced that making the vcpu spin calling > > sched_yield() is a very good plan. Better to explicitly pause the domain > > if you > > need its vcpus not t

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 08:40:23AM +0800, Haozhong Zhang wrote: > On Mon, Sep 28, 2015 at 03:19:25PM +0100, Wei Liu wrote: > > On Mon, Sep 28, 2015 at 03:13:58PM +0800, Haozhong Zhang wrote: > > > This patch adds an option 'vtsc_khz' to allow users to set vcpu's TSC > > > rate in KHz. In the case t

Re: [Xen-devel] [PATCH v2 for Xen 4.6 1/6] tools/libxl: introduce libxl_get_online_socketmap

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 03:49:50PM +0800, Chao Peng wrote: > It sets the bit on the given bitmap if the corresponding socket is > available and clears the bit when the corresponding socket is not > available. > > Signed-off-by: Chao Peng Acked-by: Wei Liu __

Re: [Xen-devel] [PATCH v2 for Xen 4.6 3/6] tools/libxl: return socket id from libxl_psr_cat_get_l3_info

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 03:49:52PM +0800, Chao Peng wrote: > The entries returned from libxl_psr_cat_get_l3_info are assumed > to be socket-continuous. But this is not true in the hotplug case. > > This patch gets the socket bitmap for all the sockets on the system > first and stores the socket id

Re: [Xen-devel] [Patch RFC 03/13] vt-d: Track the Device-TLB invalidation status in an invalidation table.

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:23, wrote: > @@ -139,6 +140,7 @@ static int queue_invalidate_wait(struct iommu *iommu, > unsigned long flags; > u64 entry_base; > struct qinval_entry *qinval_entry, *qinval_entries; > +struct domain *d; > > spin_lock_irqsave(&iommu->register_lock, fl

Re: [Xen-devel] Shared image files and block script performance

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 17:14 -0600, Mike Latimer wrote: > Any better options or ideas? Is part of the problem that shell is a terrible choice for this kind of check? Would shelling out to a helper utility allow this to be written in something better? I think we'd still be constrained in the langu

Re: [Xen-devel] [PATCH v2 for Xen 4.6 5/6] docs: make xl-psr.markdown more precise

2015-09-29 Thread Andrew Cooper
On 29/09/15 08:49, Chao Peng wrote: > Drop the chapter number as it can be confusing when it gets changed in > the referred document. > > Signed-off-by: Chao Peng > Reviewed-by: Dario Faggioli > Acked-by: Wei Liu > --- > v2: > * minor commit message adjustment. > --- > docs/misc/xl-psr.markdown

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/6] Several PSR fixes in libxl

2015-09-29 Thread Wei Liu
Now the reasoning bits. Yes, I'm arguing with myself, :-) We can of course fix it post-4.6, but the released APIs need to be maintained forever (even if it is in fact broken). That would definitely involve lots of compatibility cruft if we fix it post 4.6. This patch series is simple enough to r

Re: [Xen-devel] [Patch RFC 04/13] vt-d: Clear invalidation table in invaidation interrupt handler

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:23, wrote: > --- a/xen/drivers/passthrough/vtd/iommu.c > +++ b/xen/drivers/passthrough/vtd/iommu.c > @@ -1098,6 +1098,28 @@ static void _qi_msi_mask(struct iommu *iommu) > > static void _do_iommu_qi(struct iommu *iommu) > { > +unsigned long nr_dom, i; > +struct do

Re: [Xen-devel] [PATCH 12/13] vmx: Add a call-back to apply TSC scaling ratio to hardware

2015-09-29 Thread Andrew Cooper
On 29/09/15 02:07, Haozhong Zhang wrote: > On Mon, Sep 28, 2015 at 12:02:08PM -0400, Boris Ostrovsky wrote: >> On 09/28/2015 03:13 AM, Haozhong Zhang wrote: >>> This patch adds a new call-back setup_tsc_scaling in struct >>> hvm_function_table to apply the TSC scaling ratio to hardware. For VMX, >>

Re: [Xen-devel] [PATCH v7 21/28] xen/arm: ITS: Add GICR register emulation

2015-09-29 Thread Vijay Kilari
On Mon, Sep 28, 2015 at 4:33 PM, Julien Grall wrote: > On 28/09/15 11:37, Vijay Kilari wrote: >> On Mon, Sep 28, 2015 at 3:23 PM, Ian Campbell >> wrote: >>> On Sat, 2015-09-26 at 21:38 +0530, Vijay Kilari wrote: >>> LPI property table is accessed in interrupt context. So interrupts ar

[Xen-devel] [PATCH OSSTEST 0/3] Misc standalone wrapper improvements

2015-09-29 Thread Ian Campbell
I found these useful for debugging a recent issue. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH OSSTEST 2/3] standalone: Check job status at end of run-job.

2015-09-29 Thread Ian Campbell
Check if the job passed and if not (so status is fail, broken, running etc) then return an error. This is convenient for scripting. Signed-off-by: Ian Campbell --- standalone | 5 + 1 file changed, 5 insertions(+) diff --git a/standalone b/standalone index 0a5d96f..e85457d 100755 --- a/sta

[Xen-devel] [PATCH OSSTEST 3/3] standalone: Rotate logs rather than clobbering them

2015-09-29 Thread Ian Campbell
Keep 300, for no better reason than cr-for-branches does. Signed-off-by: Ian Campbell --- standalone | 1 + 1 file changed, 1 insertion(+) diff --git a/standalone b/standalone index e85457d..c3ff9e2 100755 --- a/standalone +++ b/standalone @@ -196,6 +196,7 @@ ensure_logs() { with_logging() {

[Xen-devel] [PATCH OSSTEST 1/3] standalone: Add get-job-status to pick status out of standalone.db

2015-09-29 Thread Ian Campbell
The return code of sg-run-job does not reflect the state of the job, which is instead written to the database. For the benefit of running tests in a loop until failure add a command to retrieve the status to stdout. Signed-off-by: Ian Campbell --- standalone | 32 ++--

[Xen-devel] [PATCH OSSTEST 4/6] ts-freebsd-install: Use $gho->{Lvdev} instead of target_guest_lv_name

2015-09-29 Thread Ian Campbell
prepareguest has already assigned this so we should use it instead of replicating (perhaps wrongly since target_guest_lv_name and target_choose_vg can behave differently if multiple vgs are present). Signed-off-by: Ian Campbell Cc: Roger Pau Monné --- ts-freebsd-install | 4 ++-- 1 file changed

[Xen-devel] [PATCH OSSTEST 6/6] Switch to Debian 8.0 (jessie) as OS for test hosts

2015-09-29 Thread Ian Campbell
mg-debian-installer-update-all has been run on the production instance and TftpDiVersion is also updated to match. The resulting binaries have also been copied to the Cambridge instance, so update Cambridge config too. Signed-off-by: Ian Campbell --- --- Osstest.pm | 2 +- prod

[Xen-devel] [PATCH OSSTEST 1/6] Debian: Uninstall flash-kernel when creating our own boot.scr

2015-09-29 Thread Ian Campbell
flash-kernel will run from various kernel postinst hooks and overwrite our own boot scripts. While this might be tollerable for the initial installation we don't want to risk it occuring after we have created our own boot.scr to boot xen. dpkg --purge succeeds if the package wasn't installed. Thi

[Xen-devel] [PATCH OSSTEST 0/6] Fixes for switching to Jessie as base OS for test hosts

2015-09-29 Thread Ian Campbell
5 of these patches are fixes for issues which I noticed when trying to do an adhoc xen-unstable flight with Jessie but which are safe to go in whenever (i.e. they work with Wheezy too). The 6th patch actually switches to Jessie which we will want to schedule a bit more carefully. In particular thi

Re: [Xen-devel] [Patch RFC 05/13] vt-d: Clear the IWC field of Invalidation Event Control Register in

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:23, wrote: > --- a/xen/drivers/passthrough/vtd/iommu.c > +++ b/xen/drivers/passthrough/vtd/iommu.c > @@ -1070,6 +1070,27 @@ static hw_irq_controller dma_msi_type = { > }; > > /* IOMMU Queued Invalidation(QI). */ > +static void qi_clear_iwc(struct iommu *iommu) > +{ > +

[Xen-devel] [PATCH OSSTEST 3/6] Debian: uboot: Use di_vg_name() and lv_dev_mapper() for root=

2015-09-29 Thread Ian Campbell
root is not a "guest lv", so using target_guest_lv_name is misleading. target_guest_lv_name also fails to properly handle the guest vg name correctly, which di_vg_name does. Signed-off-by: Ian Campbell --- Osstest/Debian.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Oss

[Xen-devel] [PATCH OSSTEST 5/6] TestSupport: Remove now unused target_guest_lv_name.

2015-09-29 Thread Ian Campbell
Signed-off-by: Ian Campbell --- Osstest/TestSupport.pm | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 2b67e32..c55b847 100644 --- a/Osstest/TestSupport.pm +++ b/Osstest/TestSupport.pm @@ -62,7 +62,7 @@ BEGIN {

[Xen-devel] [PATCH OSSTEST 2/6] Debian: Search for kernel in /boot as well as / when making u-boot script

2015-09-29 Thread Ian Campbell
The vmlinuz and initrd.img symlinks appear to have moved to /boot when installing Jessie on armhf systems compared to Wheezy. Signed-off-by: Ian Campbell --- Osstest/Debian.pm | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debia

Re: [Xen-devel] [Patch RFC 13/13] vt-d: Set the IF bit in Invalidation Wait Descriptor When submit Device-TLB

2015-09-29 Thread Jan Beulich
>>> On 16.09.15 at 15:24, wrote: > @@ -322,6 +330,15 @@ static int flush_context_qi( > return ret; > } > > +static int invalidate_async(struct iommu *iommu, u16 device_id) > +{ > +struct qi_ctrl *qi_ctrl = iommu_qi_ctrl(iommu); > + > +if ( qi_ctrl->qinval_maddr ) > +return

Re: [Xen-devel] [PATCH 2/3] xen/arm: vgic-v3: Correctly retrieve the vCPU associated to a re-distributor

2015-09-29 Thread Wei Liu
On Mon, Sep 28, 2015 at 09:31:35PM +0100, Julien Grall wrote: > When the guest is accessing the re-distributor, Xen retrieves the base > of the re-distributor using a mask based on the stride. > > Although, when the stride contains multiple set, the corresponding mask > will be computed incorrectl

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Haozhong Zhang
On Tue, Sep 29, 2015 at 10:20:21AM +0100, Wei Liu wrote: > On Tue, Sep 29, 2015 at 08:40:23AM +0800, Haozhong Zhang wrote: > > On Mon, Sep 28, 2015 at 03:19:25PM +0100, Wei Liu wrote: > > > On Mon, Sep 28, 2015 at 03:13:58PM +0800, Haozhong Zhang wrote: > > > > This patch adds an option 'vtsc_khz'

Re: [Xen-devel] [PATCH v2 for Xen 4.6 5/6] docs: make xl-psr.markdown more precise

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 10:27 +0100, Andrew Cooper wrote: > On 29/09/15 08:49, Chao Peng wrote: > > Drop the chapter number as it can be confusing when it gets changed in > > the referred document. > > > > Signed-off-by: Chao Peng > > Reviewed-by: Dario Faggioli > > Acked-by: Wei Liu > > --- > >

Re: [Xen-devel] [Qemu-devel] [PULL 0/19] xen-2015-09-08-tag

2015-09-29 Thread Stefano Stabellini
On Mon, 28 Sep 2015, Pasi Kärkkäinen wrote: > Hi, > > On Mon, Sep 28, 2015 at 11:01:11AM +0100, Stefano Stabellini wrote: > > No, unfortunately it is not possible at this stage of the release cycle. > > But users can still use QEMU 2.5 (as soon as it is released, which > > should be in a couple of

Re: [Xen-devel] [Patch RFC 00/13] VT-d Asynchronous Device-TLB Flush for ATS Device

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 11:11, wrote: > With the flush taking longer than Xen can wait for, you'll need to > do something more complex, e.g.: > - keep a log of all relevant pending derefs, to be processed when the >flush completes; or > - have some other method of preventing changes of ownership/

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Andrew Cooper
On 29/09/15 08:09, Jan Beulich wrote: > +uint32_t cs_base; +uint32_t ds_base; +uint32_t ss_base; >>> I continue to question why we have DS here, but not ES (and maybe >>> FS and GS too). I.e. either just CS and SS (which are architecturally >>> required) or at least all f

[Xen-devel] [COVERITY ACCESS] for Lars Kurth

2015-09-29 Thread Lars Kurth
Hi, I agree to the conditions in the XenProject Coverity contribution guidelines [1]. I have been community manager for the Xen Project since 2011 and chairman of the Xen Project Advisory Board since 2013. I would like access primarily to be able to write and speak intelligently about Xen and C

Re: [Xen-devel] [PATCH 12/13] vmx: Add a call-back to apply TSC scaling ratio to hardware

2015-09-29 Thread Haozhong Zhang
On Tue, Sep 29, 2015 at 10:33:14AM +0100, Andrew Cooper wrote: > On 29/09/15 02:07, Haozhong Zhang wrote: > > On Mon, Sep 28, 2015 at 12:02:08PM -0400, Boris Ostrovsky wrote: > >> On 09/28/2015 03:13 AM, Haozhong Zhang wrote: > >>> This patch adds a new call-back setup_tsc_scaling in struct > >>> h

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 15:13 +0800, Haozhong Zhang wrote: > This patch adds an option 'vtsc_khz' to allow users to set vcpu's TSC > rate in KHz. In the case that tsc_mode = 'default', the default value of > 'vtsc_khz' option is the host TSC rate which is used when 'vtsc_khz' > option is set to 0 or

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 08:40 +0800, Haozhong Zhang wrote: > > > @@ -1462,6 +1462,28 @@ static void parse_config_data(const char > > > *config_source, > > > } > > > } > > > > > > +/* "vtsc_khz" option works only if "tsc_mode" option is > > > + * "default". In this case, if "v

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 12:00, wrote: > On 29/09/15 08:09, Jan Beulich wrote: >> > +uint32_t cs_base; > +uint32_t ds_base; > +uint32_t ss_base; I continue to question why we have DS here, but not ES (and maybe FS and GS too). I.e. either just CS and SS (which are archi

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Haozhong Zhang
On Tue, Sep 29, 2015 at 11:04:14AM +0100, Ian Campbell wrote: > On Mon, 2015-09-28 at 15:13 +0800, Haozhong Zhang wrote: > > This patch adds an option 'vtsc_khz' to allow users to set vcpu's TSC > > rate in KHz. In the case that tsc_mode = 'default', the default value of > > 'vtsc_khz' option is th

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Andrew Cooper
On 29/09/15 11:13, Haozhong Zhang wrote: > On Tue, Sep 29, 2015 at 11:04:14AM +0100, Ian Campbell wrote: >> On Mon, 2015-09-28 at 15:13 +0800, Haozhong Zhang wrote: >>> This patch adds an option 'vtsc_khz' to allow users to set vcpu's TSC >>> rate in KHz. In the case that tsc_mode = 'default', the

Re: [Xen-devel] [PATCH 12/13] vmx: Add a call-back to apply TSC scaling ratio to hardware

2015-09-29 Thread Andrew Cooper
On 29/09/15 11:02, Haozhong Zhang wrote: > On Tue, Sep 29, 2015 at 10:33:14AM +0100, Andrew Cooper wrote: >> On 29/09/15 02:07, Haozhong Zhang wrote: >>> On Mon, Sep 28, 2015 at 12:02:08PM -0400, Boris Ostrovsky wrote: On 09/28/2015 03:13 AM, Haozhong Zhang wrote: > This patch adds a new c

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Julien Grall
On 29/09/15 10:50, Haozhong Zhang wrote: > On Tue, Sep 29, 2015 at 10:20:21AM +0100, Wei Liu wrote: >> On Tue, Sep 29, 2015 at 08:40:23AM +0800, Haozhong Zhang wrote: >>> On Mon, Sep 28, 2015 at 03:19:25PM +0100, Wei Liu wrote: On Mon, Sep 28, 2015 at 03:13:58PM +0800, Haozhong Zhang wrote: >>

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Andrew Cooper
On 29/09/15 11:07, Jan Beulich wrote: On 29.09.15 at 12:00, wrote: >> On 29/09/15 08:09, Jan Beulich wrote: >> +uint32_t cs_base; >> +uint32_t ds_base; >> +uint32_t ss_base; > I continue to question why we have DS here, but not ES (and maybe > FS and GS too). I

Re: [Xen-devel] [PATCH] x86/boot: Move/copy sections more efficiently

2015-09-29 Thread Andrew Cooper
On 24/09/15 10:52, Jan Beulich wrote: On 24.09.15 at 10:14, wrote: >> The ALIGN(STACK_SIZE) actually belongs with .bss.stack_aligned, but >> __init_end still needs page alignment because of the init sections being >> freed and returned to the domheap after boot. > Logically that change makes

Re: [Xen-devel] [PATCH OSSTEST 0/6] Fixes for switching to Jessie as base OS for test hosts

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 10:44 +0100, Ian Campbell wrote: > * Switch to apt-cacher-ng on the cache host, since a bug in Jessie's apt >apparently interacts badly with apt-cacher's handling of Ranges in http >requests[0]. I've deployed apt-cacher-ng on an alternative port on the >productio

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 11:24 +0100, Andrew Cooper wrote: > On 29/09/15 11:13, Haozhong Zhang wrote: > > On Tue, Sep 29, 2015 at 11:04:14AM +0100, Ian Campbell wrote: > > > On Mon, 2015-09-28 at 15:13 +0800, Haozhong Zhang wrote: > > > > This patch adds an option 'vtsc_khz' to allow users to set vcpu

Re: [Xen-devel] [PATCH v2 for Xen 4.6 0/6] Several PSR fixes in libxl

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 10:33 +0100, Wei Liu wrote: > Now the reasoning bits. Yes, I'm arguing with myself, :-) > > We can of course fix it post-4.6, but the released APIs need to be > maintained forever (even if it is in fact broken). That would definitely > involve lots of compatibility cruft if w

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Andrew Cooper
On 29/09/15 11:28, Ian Campbell wrote: > On Tue, 2015-09-29 at 11:24 +0100, Andrew Cooper wrote: >> On 29/09/15 11:13, Haozhong Zhang wrote: >>> On Tue, Sep 29, 2015 at 11:04:14AM +0100, Ian Campbell wrote: On Mon, 2015-09-28 at 15:13 +0800, Haozhong Zhang wrote: > This patch adds an optio

Re: [Xen-devel] [PATCH 13/13] tools/libxl: Add 'vtsc_khz' option to set guest TSC rate

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 11:07:17AM +0100, Ian Campbell wrote: > On Tue, 2015-09-29 at 08:40 +0800, Haozhong Zhang wrote: > > > > @@ -1462,6 +1462,28 @@ static void parse_config_data(const char > > > > *config_source, > > > > } > > > > } > > > > > > > > +/* "vtsc_khz" option wor

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 12:25, wrote: > On 29/09/15 11:07, Jan Beulich wrote: > On 29.09.15 at 12:00, wrote: >>> Therefore, we are back to the question of whether to provide all segment >>> registers, or specify a flat layout without specific selector values. I >>> would prefer the former to the

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Andrew Cooper
On 29/09/15 11:33, Jan Beulich wrote: On 29.09.15 at 12:25, wrote: >> On 29/09/15 11:07, Jan Beulich wrote: >> On 29.09.15 at 12:00, wrote: Therefore, we are back to the question of whether to provide all segment registers, or specify a flat layout without specific selector val

Re: [Xen-devel] [PATCH] x86/EPT: adjust types in ept_split_super_page()

2015-09-29 Thread George Dunlap
On 28/09/15 15:35, Jan Beulich wrote: > The function returns a boolean and its current and target level inputs > are unsigned (which in turn allows simplifying the early-out check). > Also convert a non-standard loop variable to an ordinary function scope > one, at once making it unsigned too. > >

Re: [Xen-devel] [PATCH 1/3] x86/p2m: tighten conditions of IOMMU mapping updates

2015-09-29 Thread Jan Beulich
>>> On 23.09.15 at 16:00, wrote: > On Tue, Sep 22, 2015 at 08:15:39AM -0600, Jan Beulich wrote: >> >>> On 21.09.15 at 16:02, wrote: >> > In the EPT case permission changes should also result in updates or >> > TLB flushes. >> > >> > In the NPT case the old MFN does not depend on the new entry be

Re: [Xen-devel] [PATCH v6 24/29] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 12:37, wrote: > On 29/09/15 11:33, Jan Beulich wrote: > On 29.09.15 at 12:25, wrote: >>> On 29/09/15 11:07, Jan Beulich wrote: >>> On 29.09.15 at 12:00, wrote: > Therefore, we are back to the question of whether to provide all segment > registers, or specify a

[Xen-devel] [PATCH] QEMU_UPSTREAM_REVISION update

2015-09-29 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Config.mk |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Config.mk b/Config.mk index 77f83a2..5a3774e 100644 --- a/Config.mk +++ b/Config.mk @@ -254,7 +254,7 @@ SEABIOS_UPSTREAM_URL ?= git://xenbits.xen.org/seabios.git MINIOS_UPSTREAM_URL ?=

Re: [Xen-devel] [PATCH 7/8] xen/arm: gic-v2: Detect automatically aliased GIC400

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 19:07 +0100, Julien Grall wrote: > On 25/09/15 17:26, Ian Campbell wrote: > > On Tue, 2015-09-22 at 18:47 +0100, Julien Grall wrote: > > > +gicv2.map_cbase = ioremap_nocache(cbase, csize); > > > +if ( !gicv2.map_cbase ) > > > panic("GICv2: Failed to ioremap fo

[Xen-devel] [PATCH] x86/p2m-pt: delay freeing of intermediate page tables

2015-09-29 Thread Jan Beulich
Old intermediate page tables must be freed only after IOMMU side updates/flushes have got carried out. Signed-off-by: Jan Beulich --- This was split out from a previously submitted patch. --- a/xen/arch/x86/mm/p2m-pt.c +++ b/xen/arch/x86/mm/p2m-pt.c @@ -487,8 +487,9 @@ p2m_pt_set_entry(struct p2

Re: [Xen-devel] [PATCH v1 2/8] xen/arm: io: Extend write/read handler to pass the register in parameter

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 17:35 +0100, Julien Grall wrote: > Hi Ian, > > On 25/09/15 17:36, Ian Campbell wrote: > > On Fri, 2015-09-25 at 15:51 +0100, Julien Grall wrote: > > > From: Julien Grall > > > > > > Rather than letting each handler to retrieve the register used by the > > > I/O access, add

[Xen-devel] [PATCH] x86/p2m-pt: ignore pt-share flag for shadow mode guests

2015-09-29 Thread Jan Beulich
There is no page table sharing in shadow mode. Signed-off-by: Jan Beulich --- This was split out from a previously submitted patch. --- a/xen/arch/x86/mm/p2m-pt.c +++ b/xen/arch/x86/mm/p2m-pt.c @@ -645,7 +646,7 @@ p2m_pt_set_entry(struct p2m_domain *p2m, if ( iommu_enabled && need_iommu(p

Re: [Xen-devel] [PATCH] x86/p2m-pt: delay freeing of intermediate page tables

2015-09-29 Thread George Dunlap
On 29/09/15 11:51, Jan Beulich wrote: > Old intermediate page tables must be freed only after IOMMU side > updates/flushes have got carried out. > > Signed-off-by: Jan Beulich Reviewed-by: George Dunlap > --- > This was split out from a previously submitted patch. > > --- a/xen/arch/x86/mm/p2

Re: [Xen-devel] [PATCH] x86/p2m-pt: ignore pt-share flag for shadow mode guests

2015-09-29 Thread George Dunlap
On 29/09/15 11:52, Jan Beulich wrote: > There is no page table sharing in shadow mode. > > Signed-off-by: Jan Beulich Reviewed-by: George Dunlap > --- > This was split out from a previously submitted patch. > > --- a/xen/arch/x86/mm/p2m-pt.c > +++ b/xen/arch/x86/mm/p2m-pt.c > @@ -645,7 +646,7

Re: [Xen-devel] [PATCH v1 5/8] xen/arm: vgic: Optimize the way to store GICD_IPRIORITYR in the rank

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 18:10 +0100, Julien Grall wrote: > Hi Ian, > > On 28/09/15 11:50, Ian Campbell wrote: > > > diff --git a/xen/arch/arm/vgic-v3.c b/xen/arch/arm/vgic-v3.c > > > index c013200..2787507 100644 > > > --- a/xen/arch/arm/vgic-v3.c > > > +++ b/xen/arch/arm/vgic-v3.c > > > @@ -430,18

[Xen-devel] minios and xen release branching and tagging

2015-09-29 Thread Ian Jackson
Notes of an irc discussion, for the record: 11:51 liuw: We agreed about minios: we are going to defer branching until it turns out to be necessary. Config.mk will continue to refer to a specific commit id until just before release. 11:51 When we commit to particu

Re: [Xen-devel] [PATCH v1 3/8] xen/arm: Support sign-extension for every read access

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 17:42 +0100, Julien Grall wrote: > > > + */ > > > +if ( info->dabt.sign && (*r & (1UL << (size - 1)) )) > > > +{ > > > +/* > > > + * We are relying on register_t as the same size as > > > + * an unsigned long or order to keep the 32bit some

Re: [Xen-devel] [PATCH v1 2/8] xen/arm: io: Extend write/read handler to pass the register in parameter

2015-09-29 Thread Julien Grall
Hi Ian, On 29/09/15 11:51, Ian Campbell wrote: > On Mon, 2015-09-28 at 17:35 +0100, Julien Grall wrote: >> Hi Ian, >> >> On 25/09/15 17:36, Ian Campbell wrote: >>> On Fri, 2015-09-25 at 15:51 +0100, Julien Grall wrote: From: Julien Grall Rather than letting each handler to retrieve

Re: [Xen-devel] [PATCH v1 3/8] xen/arm: Support sign-extension for every read access

2015-09-29 Thread Ian Campbell
On Mon, 2015-09-28 at 19:22 +0100, Julien Grall wrote: > On 25/09/15 17:44, Ian Campbell wrote: > > On Fri, 2015-09-25 at 15:51 +0100, Julien Grall wrote: > > I think you meant s/bit sign/sign bit/ but more correct would be "Sign > > extend if required". > > > > > + * Note that we expect the r

Re: [Xen-devel] [PATCH v1 3/8] xen/arm: Support sign-extension for every read access

2015-09-29 Thread Julien Grall
On 29/09/15 12:01, Ian Campbell wrote: > On Mon, 2015-09-28 at 17:42 +0100, Julien Grall wrote: > + */ +if ( info->dabt.sign && (*r & (1UL << (size - 1)) )) +{ +/* + * We are relying on register_t as the same size as + * an unsigned

Re: [Xen-devel] [PATCH v1 2/8] xen/arm: io: Extend write/read handler to pass the register in parameter

2015-09-29 Thread Ian Campbell
On Tue, 2015-09-29 at 12:00 +0100, Julien Grall wrote: > Hi Ian, > > On 29/09/15 11:51, Ian Campbell wrote: > > On Mon, 2015-09-28 at 17:35 +0100, Julien Grall wrote: > > > Hi Ian, > > > > > > On 25/09/15 17:36, Ian Campbell wrote: > > > > On Fri, 2015-09-25 at 15:51 +0100, Julien Grall wrote: >

Re: [Xen-devel] [PATCH v1 3/8] xen/arm: Support sign-extension for every read access

2015-09-29 Thread Julien Grall
On 29/09/15 12:03, Ian Campbell wrote: > On Mon, 2015-09-28 at 19:22 +0100, Julien Grall wrote: >> On 25/09/15 17:44, Ian Campbell wrote: >>> On Fri, 2015-09-25 at 15:51 +0100, Julien Grall wrote: >>> I think you meant s/bit sign/sign bit/ but more correct would be "Sign >>> extend if required". >>

Re: [Xen-devel] [PATCH 1/3] x86/p2m: tighten conditions of IOMMU mapping updates

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 04:47:00AM -0600, Jan Beulich wrote: > >>> On 23.09.15 at 16:00, wrote: > > On Tue, Sep 22, 2015 at 08:15:39AM -0600, Jan Beulich wrote: > >> >>> On 21.09.15 at 16:02, wrote: > >> > In the EPT case permission changes should also result in updates or > >> > TLB flushes. > >

Re: [Xen-devel] [PATCH] x86/p2m-pt: ignore pt-share flag for shadow mode guests

2015-09-29 Thread Wei Liu
On Tue, Sep 29, 2015 at 04:52:23AM -0600, Jan Beulich wrote: > There is no page table sharing in shadow mode. > > Signed-off-by: Jan Beulich Release-acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH 0/2] VT-d: x2APIC + IR adjustments

2015-09-29 Thread Jan Beulich
1: use proper error codes in iommu_enable_x2apic_IR() 2: section placement and type adjustments Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 3/4 RFC] x86/p2m: use large pages for MMIO mappings

2015-09-29 Thread Julien Grall
Hi Jan, Sorry I hadn't much time to look at it until now. On 15/09/15 08:34, Jan Beulich wrote: > When mapping large BARs (e.g. the frame buffer of a graphics card) the > overhead or establishing such mappings using onle 4k pages has, > particularly after the XSA-125 fix, become unacceptable. Alt

Re: [Xen-devel] [PATCH] xen/pvhvm: add soft reset on kexec/kdump support

2015-09-29 Thread Vitaly Kuznetsov
Boris Ostrovsky writes: > On 09/25/2015 03:35 PM, Konrad Rzeszutek Wilk wrote: >> On Fri, Sep 25, 2015 at 03:19:57PM -0400, Boris Ostrovsky wrote: >>> On 09/25/2015 03:01 PM, Konrad Rzeszutek Wilk wrote: On Fri, Sep 25, 2015 at 01:17:40PM -0400, Boris Ostrovsky wrote: > On 09/25/2015 12:

Re: [Xen-devel] [PATCH 3/4 RFC] x86/p2m: use large pages for MMIO mappings

2015-09-29 Thread Jan Beulich
>>> On 29.09.15 at 13:33, wrote: > On 15/09/15 08:34, Jan Beulich wrote: >> RFC reasons: >> - ARM side unimplemented (and hence libxc for now made cope with both >> models), the main issue (besides my inability to test any change >> there) being the many internal uses of map_mmio_regions()) >

[Xen-devel] [PATCH 1/2] VT-d: use proper error codes in iommu_enable_x2apic_IR()

2015-09-29 Thread Jan Beulich
... allowing to suppress a confusing messeage combination: When ACPI_DMAR_X2APIC_OPT_OUT is set, so far we first logged a message that IR could not be enabled (hence not using x2APIC), followed by one indicating successful initialization of IR (if no other problems prevented that). Also adjust the

[Xen-devel] [PATCH 2/2] VT-d: section placement and type adjustments

2015-09-29 Thread Jan Beulich
With x2APIC requiring iommu_supports_eim() to return true, we can adjust a few conditonals such that both it and platform_supports_x2apic() can be marked __init. For the latter as well as for platform_supports_intremap() also change the return types to bool_t. Signed-off-by: Jan Beulich --- a/xe

[Xen-devel] write to file from kernel

2015-09-29 Thread Hamed Rs
Is it possible to write to file from xen kernel? Best Regards Hamed Rostamzadeh, MSc Student, Distributed Systems Lab, School of Computer Engineering, Iran University of Science and Technology, Tehran, Iran. https://sites.google.com/site/hamedrostamzade/ ___

Re: [Xen-devel] [PATCH v2] PVH Dom0 RMRR IOMMU mapping regression fix

2015-09-29 Thread Jan Beulich
>>> On 25.09.15 at 22:59, wrote: > From: Elena Ufimtseva > > This patch addresses a regression introduced by commit > 5ae03990c120a7b3067a52d9784c9aa72c0705a6 in new set_identity_p2m_entry. > RMRRs are not being mapped in IOMMU for PVH Dom0. This causes pages faults > and > some long 'hang-lik

Re: [Xen-devel] [PATCH] Add missing license and copyright statements to public interface headers.

2015-09-29 Thread Mike Belopuhov
On Mon, Sep 28, 2015 at 11:24 +0200, Mike Belopuhov wrote: > On Fri, Sep 25, 2015 at 01:12 -0600, Jan Beulich wrote: > > >>> On 22.09.15 at 16:02, wrote: > > > --- xen/include/public/arch-x86/pmu.h > > > +++ xen/include/public/arch-x86/pmu.h > > > > I fixed this up for you this time, but in the f

  1   2   3   >