Re: [Xen-devel] [PATCH v15.1 13/23] x86: refactor psr: CDP: implement CPU init flow.

2017-08-04 Thread Yi Sun
On 17-08-03 23:42:01, Jan Beulich wrote: > >>> Yi Sun 08/04/17 5:23 AM >>> > >On 17-08-03 03:37:06, Jan Beulich wrote: > >> >>> On 03.08.17 at 04:10, wrote: > >> > @@ -1283,10 +1344,20 @@ static void psr_cpu_init(void) > >> > feat = feat_l3; > >> > feat_l3 = NULL; > >> > > >>

Re: [Xen-devel] xen-blkfront hang

2017-08-04 Thread Valentin Vidic
On Mon, Jul 31, 2017 at 09:09:19AM +0800, Dongli Zhang wrote: > To verify whether the above patch would help, please check the nr_grant_frames > value in guest domU. If this value is exactly the same of maximum grant frames > (by default, xen mainline uses 32) and the number of free grant reference

[Xen-devel] [ovmf test] 112433: all pass - PUSHED

2017-08-04 Thread osstest service owner
flight 112433 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/112433/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 09cc872d0242304329e6c21e91bef14fa9949744 baseline version: ovmf bb4831c03dd15ff8528dc

[Xen-devel] [xen-unstable-smoke test] 112438: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112438 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112438/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

[Xen-devel] [Question] how to avoid Xen using a memory range?

2017-08-04 Thread Chao Gao
Hi, everyone. I have a machine which has two numa nodes. The NODE0 contains memory range from 0 to 0x18400MB and NODE1 contains memory range from 0x18400MB to 0x1c400MB. The resource available to dom0 is restricted through adding "dom0_mem=10G dom0_nodes=0 dom0_max_vcpus=48" to Xen Command Line. E

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Dario Faggioli
On Thu, 2017-08-03 at 17:39 -0400, Meng Xu wrote: > On Thu, Aug 3, 2017 at 11:53 AM, Dario Faggioli > wrote: > > > > How about, here at libxl level, we use the "extratime" field that > > we > > have as a leftover from SEDF (and which had, in that scheduler, a > > similar meaning)? > > > > If we

Re: [Xen-devel] [PATCH RFC v1 3/3] xl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Dario Faggioli
On Thu, 2017-08-03 at 18:02 -0400, Meng Xu wrote: > On Thu, Aug 3, 2017 at 12:03 PM, Dario Faggioli > wrote: > > > > > @@ -702,14 +705,18 @@ int main_sched_rtds(int argc, char **argv) > > >  int *vcpus = (int *)xmalloc(sizeof(int)); /* IDs of VCPUs > > > that > > > change */ > > >  int *p

Re: [Xen-devel] xen-blkfront hang

2017-08-04 Thread Roger Pau Monné
On Fri, Aug 04, 2017 at 10:00:09AM +0200, Valentin Vidic wrote: > On Mon, Jul 31, 2017 at 09:09:19AM +0800, Dongli Zhang wrote: > > To verify whether the above patch would help, please check the > > nr_grant_frames > > value in guest domU. If this value is exactly the same of maximum grant > > fr

Re: [Xen-devel] [PATCH v7 00/14] arm/mem_access: Walk guest page tables in SW if mem_access is active

2017-08-04 Thread Sergej Proskurin
Hi Julien, Sorry for the late reply. On 07/31/2017 04:38 PM, Julien Grall wrote: > > > On 18/07/17 13:24, Sergej Proskurin wrote: >> Hi all, > > Hi, > >> >> The function p2m_mem_access_check_and_get_page is called from the function >> get_page_from_gva if mem_access is active and the hardware

Re: [Xen-devel] xen-blkfront hang

2017-08-04 Thread Dongli Zhang
On 08/04/2017 05:13 PM, Roger Pau Monné wrote: > On Fri, Aug 04, 2017 at 10:00:09AM +0200, Valentin Vidic wrote: >> On Mon, Jul 31, 2017 at 09:09:19AM +0800, Dongli Zhang wrote: >>> To verify whether the above patch would help, please check the >>> nr_grant_frames >>> value in guest domU. If thi

[Xen-devel] [Patch for staging 1/2] docs: remove a special character to avoid html creation error.

2017-08-04 Thread Yi Sun
The '®' (a special character) may cause html document creation failure. So remove it from the feature document. Signed-off-by: Yi Sun --- docs/features/intel_psr_cat_cdp.pandoc | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/docs/features/intel_psr_cat_cdp.pandoc b/do

[Xen-devel] [Patch for staging 2/2] x86: remove an ASSERT to avoid crash when destroy a domain.

2017-08-04 Thread Yi Sun
In 'psr_free_cos', we should not use 'ASSERT(socket_info)' because the 'socket_info' is allocated only if 'psr' boot parameter is set. So remove it and use 'psr_alloc_feat_enabled' to check if 'socket_info' is valid or not to avoid crash. Signed-off-by: Yi Sun --- xen/arch/x86/psr.c | 4 +--- 1

Re: [Xen-devel] xen-blkfront hang

2017-08-04 Thread Roger Pau Monné
On Fri, Aug 04, 2017 at 05:36:29PM +0800, Dongli Zhang wrote: > > > On 08/04/2017 05:13 PM, Roger Pau Monné wrote: > > On Fri, Aug 04, 2017 at 10:00:09AM +0200, Valentin Vidic wrote: > >> On Mon, Jul 31, 2017 at 09:09:19AM +0800, Dongli Zhang wrote: > >>> To verify whether the above patch would h

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

2017-08-04 Thread osstest service owner
flight 112430 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/112430/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-examine 7 reboot fail REGR. vs. 110515 test-amd64-amd64-xl

[Xen-devel] [libvirt test] 112436: tolerable trouble: blocked/broken/pass - PUSHED

2017-08-04 Thread osstest service owner
flight 112436 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/112436/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-libvirt-xsm 1 build-check(1) blocked n/a build-arm64-libvirt 1 build-check(1)

Re: [Xen-devel] Next Xen ARM community call - Wednesday 2nd August 2017

2017-08-04 Thread Mirela Simonovic
Hi Stefano, Thank you, we're looking into this. Regards, Mirela On Wed, Aug 2, 2017 at 8:25 PM, Stefano Stabellini wrote: > Hi everybody, > > the Xen based suspend notification interface is implemented in Linux by > drivers/xen/manage.c. The guest monitors a node on xenstore named > "control/s

[Xen-devel] [linux-3.18 test] 112431: trouble: blocked/broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112431 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/112431/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-pvops 3 capture-logs broken REGR. vs. 112102

Re: [Xen-devel] [PATCH v2 1/3] x86/vlapic: Introduce vlapic_update_timer

2017-08-04 Thread Anthony PERARD
On Thu, Aug 03, 2017 at 08:59:10AM -0600, Jan Beulich wrote: > >>> Anthony PERARD 07/18/17 7:10 PM >>> > >There should not be any functionality change with this patch. > > > >This function is used when the APIC_TMICT register is updated. > > > >vlapic_update_timer is introduce as it will be use al

Re: [Xen-devel] [PATCH v2] xl: add --clear option to dmesg command

2017-08-04 Thread Wei Liu
On Tue, Aug 01, 2017 at 05:10:27PM +0100, Wei Liu wrote: > On Tue, Aug 01, 2017 at 11:57:50PM +0800, Xiao Liang wrote: > > From: xiliang > > > > The manual of xl says --clear option is supported and that option worked > > for xm. Add that to xl now. > > I will wrap this long line to 72 columns

Re: [Xen-devel] [PATCH v2] xl: add --clear option to dmesg command

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 11:57:53AM +0100, Wei Liu wrote: > On Tue, Aug 01, 2017 at 05:10:27PM +0100, Wei Liu wrote: > > On Tue, Aug 01, 2017 at 11:57:50PM +0800, Xiao Liang wrote: > > > From: xiliang > > > > > > The manual of xl says --clear option is supported and that option worked > > > for x

[Xen-devel] [qemu-upstream-unstable baseline-only test] 71935: regressions - trouble: blocked/broken/fail/pass

2017-08-04 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 71935 qemu-upstream-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71935/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-win7-amd64 1

Re: [Xen-devel] [Patch for staging 1/2] docs: remove a special character to avoid html creation error.

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 05:29:36PM +0800, Yi Sun wrote: > The '®' (a special character) may cause html document creation > failure. So remove it from the feature document. Not sure how you tested it but the original error was found by pandoc. > > Signed-off-by: Yi Sun Change looks good: Teste

[Xen-devel] [distros-debian-jessie test] 71936: regressions - trouble: blocked/broken/fail/pass

2017-08-04 Thread Platform Team regression test user
flight 71936 distros-debian-jessie real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71936/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-pvops 6 kernel-build fail REGR. vs. 71901 Tests w

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

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

[Xen-devel] [PATCH v4] x86/hvm: Allow guest_request vm_events coming from userspace

2017-08-04 Thread Alexandru Isaila
In some introspection usecases, an in-guest agent needs to communicate with the external introspection agent. An existing mechanism is HVMOP_guest_request_vm_event, but this is restricted to kernel usecases like all other hypercalls. Introduce a mechanism whereby the introspection agent can white

[Xen-devel] [PATCH 3/3] xen: remove not used trace functions

2017-08-04 Thread Juergen Gross
There are some Xen specific trace functions defined in include/trace/events/xen.h. Remove them. Signed-off-by: Juergen Gross --- include/trace/events/xen.h | 20 1 file changed, 20 deletions(-) diff --git a/include/trace/events/xen.h b/include/trace/events/xen.h index 677e8

[Xen-devel] [PATCH 0/3] xen: do some cleanups

2017-08-04 Thread Juergen Gross
Remove stuff no longer needed. Juergen Gross (3): xen: remove tests for pvh mode in pure pv paths xen: remove unused function xen_set_domain_pte() xen: remove not used trace functions arch/x86/include/asm/xen/page.h | 5 - arch/x86/xen/mmu_pv.c | 20 arc

[Xen-devel] [PATCH 1/3] xen: remove tests for pvh mode in pure pv paths

2017-08-04 Thread Juergen Gross
Remove the last tests for XENFEAT_auto_translated_physmap in pure PV-domain specific paths. PVH V1 is gone and the feature will always be "false" in PV guests. Signed-off-by: Juergen Gross --- arch/x86/include/asm/xen/page.h | 3 --- arch/x86/xen/p2m.c | 25 +---

[Xen-devel] [PATCH 2/3] xen: remove unused function xen_set_domain_pte()

2017-08-04 Thread Juergen Gross
The function xen_set_domain_pte() is used nowhere in the kernel. Remove it. Signed-off-by: Juergen Gross --- arch/x86/include/asm/xen/page.h | 2 -- arch/x86/xen/mmu_pv.c | 20 include/trace/events/xen.h | 18 -- 3 files changed, 40 deletions(

Re: [Xen-devel] [PATCH v2 2/3] x86/vlapic: Keep timer running when switching between one-shot and periodic mode

2017-08-04 Thread Anthony PERARD
On Thu, Aug 03, 2017 at 09:21:57AM -0600, Jan Beulich wrote: > >>> Anthony PERARD 07/18/17 7:12 PM >>> > >@@ -678,18 +679,29 @@ static void vlapic_tdt_pt_cb(struct vcpu *v, void > >*data) > >static void vlapic_update_timer(struct vlapic *vlapic, uint32_t lvtt); > >{ > >uint64_t period; > >-ui

Re: [Xen-devel] [PATCH v2 3/3] x86/vlapic: Apply change to TDCR right away to the timer

2017-08-04 Thread Anthony PERARD
On Thu, Aug 03, 2017 at 09:29:02AM -0600, Jan Beulich wrote: > >>> Anthony PERARD 07/18/17 7:10 PM >>> > >@@ -701,6 +702,13 @@ static void vlapic_update_timer(struct vlapic *vlapic, > >uint32_t lvtt); > >delta = period - time_passed; > >} > > > >+if ( vlapic->hw.timer_divisor != old_divisor

Re: [Xen-devel] [PATCH v4] x86/hvm: Allow guest_request vm_events coming from userspace

2017-08-04 Thread Andrew Cooper
On 04/08/17 12:32, Alexandru Isaila wrote: > In some introspection usecases, an in-guest agent needs to communicate > with the external introspection agent. An existing mechanism is > HVMOP_guest_request_vm_event, but this is restricted to kernel usecases > like all other hypercalls. > > Introduce

[Xen-devel] [xen-unstable-smoke test] 112440: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112440 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112440/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

Re: [Xen-devel] [PATCH v4 06/13] libxl: change p9 to use generec add function

2017-08-04 Thread Wei Liu
On Wed, Aug 02, 2017 at 02:37:10PM +0300, Oleksandr Grytsov wrote: [...] > >> >> From other side this rename touches only internals changes: no changes > >> >> in config file > >> >> or CLI interface. > >> >> > >> > > >> > As said, the framework need to be ready to deal with PCI anyway. > >> > > >>

Re: [Xen-devel] [PATCH v15 01/23] docs: create Cache Allocation Technology (CAT) and Code and Data Prioritization (CDP) feature document

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 12:40:25PM +0800, Yi Sun wrote: > On 17-08-03 13:26:37, Wei Liu wrote: > > On Tue, Aug 01, 2017 at 04:48:32PM +0800, Yi Sun wrote: > > > This patch creates CAT and CDP feature document in doc/features/. It > > > describes > > > key points to implement L3 CAT/CDP and L2 CAT

[Xen-devel] [ovmf test] 112439: all pass - PUSHED

2017-08-04 Thread osstest service owner
flight 112439 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/112439/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf ef3d1df77bbd5227c76306e5c64c66d82805bbd9 baseline version: ovmf 09cc872d0242304329e6c

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 10:13:18AM +0200, Dario Faggioli wrote: > On Thu, 2017-08-03 at 17:39 -0400, Meng Xu wrote: > > On Thu, Aug 3, 2017 at 11:53 AM, Dario Faggioli > > wrote: > > > > > > How about, here at libxl level, we use the "extratime" field that > > > we > > > have as a leftover from S

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Wei Liu
On Tue, Aug 01, 2017 at 02:33:53PM -0400, Meng Xu wrote: > Modify libxl_vcpu_sched_params_get/set and sched_rtds_vcpu_get/set > functions to support per-VCPU work conserving flag > > Signed-off-by: Meng Xu > --- > tools/libxl/libxl.h | 1 + Missing a LIBXL_HAVE _

Re: [Xen-devel] [PATCH 4/5] tools/libxenforeignmemory: add support for resource mapping

2017-08-04 Thread Wei Liu
On Wed, Aug 02, 2017 at 10:59:48AM +0100, Paul Durrant wrote: > A previous patch introduced a new HYPERVISOR_memory_op to acquire guest > resources for direct priv-mapping. > > This patch adds new functionality into libxenforeignmemory to make use > of a new privcmd ioctl [1] that uses the new mem

Re: [Xen-devel] [PATCH 5/5] tools/libxenctrl: use new xenforeignmemory API to seed grant table

2017-08-04 Thread Wei Liu
On Wed, Aug 02, 2017 at 10:59:49AM +0100, Paul Durrant wrote: > A previous patch added support for priv-mapping guest resources directly > (rather than having to foreign-map, which requires P2M modification for > HVM guests). > > This patch makes use of the new API to seed the guest grant table un

Re: [Xen-devel] [PATCH v1] tools/libxc: use superpages during restore of HVM guest

2017-08-04 Thread Wei Liu
On Wed, Aug 02, 2017 at 03:45:25PM +0200, Olaf Hering wrote: > During creating of a HVM domU meminit_hvm() tries to map superpages. > After save/restore or migration this mapping is lost, everything is > allocated in single pages. This causes a performance degradition after > migration. > > Add ne

Re: [Xen-devel] [PATCH v1] tools/libxc: use superpages during restore of HVM guest

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 07:43:47AM +0200, Olaf Hering wrote: > On Wed, Aug 02, Olaf Hering wrote: > > > +++ b/tools/libxc/xc_sr_restore_x86_hvm.c > > > +#define SUPERPAGE_2MB_SHIFT 9 > > +#define SUPERPAGE_2MB_NR_PFNS (1UL << SUPERPAGE_2MB_SHIFT) > > +#define SUPERPAGE_1GB_SHIFT 18 > > +#defi

Re: [Xen-devel] [PATCH v1] tools/libxc: use superpages during restore of HVM guest

2017-08-04 Thread Olaf Hering
On Fri, Aug 04, Wei Liu wrote: > Can you split this patch into several: > 1. code movement > 2. refactoring / introduction of new hooks > 3. implementing the new algorithm I tried that, it did not work well. But, I can try again if required. Olaf signature.asc Description: PGP signature __

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Dario Faggioli
On Fri, 2017-08-04 at 13:10 +0100, Wei Liu wrote: > On Fri, Aug 04, 2017 at 10:13:18AM +0200, Dario Faggioli wrote: > > On Thu, 2017-08-03 at 17:39 -0400, Meng Xu wrote: > > > > > *HOWEVER*, in this case, we do have that 'extratime' field already, > > as > > a leftover from SEDF, which is there ta

Re: [Xen-devel] [PATCH 5/5] tools/libxenctrl: use new xenforeignmemory API to seed grant table

2017-08-04 Thread Marek Marczykowski-Górecki
On Fri, Aug 04, 2017 at 01:26:21PM +0100, Wei Liu wrote: > On Wed, Aug 02, 2017 at 10:59:49AM +0100, Paul Durrant wrote: > > A previous patch added support for priv-mapping guest resources directly > > (rather than having to foreign-map, which requires P2M modification for > > HVM guests). > > > >

[Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Juergen Gross
Last year Jan posted a patch series to change hypervisor log level thresholds via xl command [1]. This approach was later modified by Wei resulting in patch series [2]. I'd like to follow up with another approach being able to do the same, but being much more flexible: Instead of controlling only

Re: [Xen-devel] [PATCH 5/5] tools/libxenctrl: use new xenforeignmemory API to seed grant table

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 03:02:03PM +0200, Marek Marczykowski-Górecki wrote: > On Fri, Aug 04, 2017 at 01:26:21PM +0100, Wei Liu wrote: > > On Wed, Aug 02, 2017 at 10:59:49AM +0100, Paul Durrant wrote: > > > A previous patch added support for priv-mapping guest resources directly > > > (rather than

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Andrew Cooper
On 04/08/17 14:20, Juergen Gross wrote: > Last year Jan posted a patch series to change hypervisor log level > thresholds via xl command [1]. This approach was later modified by Wei > resulting in patch series [2]. > > I'd like to follow up with another approach being able to do the same, > but bei

Re: [Xen-devel] [RFC PATCH 3/4] x86/p2m : remove checks that forbid adding foreign pages between HVM guests

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 10:20:24AM +0800, Zhongze Liu wrote: > Two checks in the p2m code forbids sharing physical pages among DomU's by > using > xc_add_to_physmap_batch with XENMAPSPACE_gmfn_foreign. Just simply remove them > in this RFC patch to ask for suggestions on how to properly handle thi

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Juergen Gross
On 04/08/17 15:23, Andrew Cooper wrote: > On 04/08/17 14:20, Juergen Gross wrote: >> Last year Jan posted a patch series to change hypervisor log level >> thresholds via xl command [1]. This approach was later modified by Wei >> resulting in patch series [2]. >> >> I'd like to follow up with anothe

Re: [Xen-devel] [PATCH 5/5] tools/libxenctrl: use new xenforeignmemory API to seed grant table

2017-08-04 Thread Juergen Gross
On 04/08/17 15:21, Wei Liu wrote: > On Fri, Aug 04, 2017 at 03:02:03PM +0200, Marek Marczykowski-Górecki wrote: >> On Fri, Aug 04, 2017 at 01:26:21PM +0100, Wei Liu wrote: >>> On Wed, Aug 02, 2017 at 10:59:49AM +0100, Paul Durrant wrote: A previous patch added support for priv-mapping guest re

[Xen-devel] [xen-unstable-smoke test] 112445: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112445 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112445/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 03:20:09PM +0200, Juergen Gross wrote: > Last year Jan posted a patch series to change hypervisor log level > thresholds via xl command [1]. This approach was later modified by Wei > resulting in patch series [2]. > > I'd like to follow up with another approach being able t

Re: [Xen-devel] [RFC PATCH 3/4] x86/p2m : remove checks that forbid adding foreign pages between HVM guests

2017-08-04 Thread Zhongze Liu
Hi Wei, Yes, indeed. Before the teardown is implemented, even if I simply remove the first check, when I try to destroy the master domain, the domain will become a zombie, because the refcount of the shared pages won't be correctly decreased on slave destruction. I don't know why this hasn't been

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Andrew Cooper
On 04/08/17 14:36, Juergen Gross wrote: > On 04/08/17 15:23, Andrew Cooper wrote: >> On 04/08/17 14:20, Juergen Gross wrote: >>> Last year Jan posted a patch series to change hypervisor log level >>> thresholds via xl command [1]. This approach was later modified by Wei >>> resulting in patch serie

Re: [Xen-devel] [PATCH v15 00/23] Enable L2 Cache Allocation Technology & Refactor psr.c

2017-08-04 Thread Yi Sun
On 17-08-04 12:36:46, Yi Sun wrote: > On 17-08-04 10:21:51, Yi Sun wrote: > > On 17-08-03 18:50:02, Boris Ostrovsky wrote: > > > > > > > > > On 08/03/2017 11:37 AM, Andrew Cooper wrote: > > > >(XEN) [ 1071.542500] Xen call trace: > > > >(XEN) [ 1071.542505][] psr_domain_free+0x23/0xcc > > > >

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

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

[Xen-devel] [xen-unstable test] 112437: regressions - trouble: blocked/broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112437 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/112437/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 112286 Regressions

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 02:53:51PM +0200, Dario Faggioli wrote: > On Fri, 2017-08-04 at 13:10 +0100, Wei Liu wrote: > > On Fri, Aug 04, 2017 at 10:13:18AM +0200, Dario Faggioli wrote: > > > On Thu, 2017-08-03 at 17:39 -0400, Meng Xu wrote: > > > > > > > *HOWEVER*, in this case, we do have that 'ex

Re: [Xen-devel] [PATCH v2] xl: add --clear option to dmesg command

2017-08-04 Thread Xiao Liang
Yes, the full name is "Xiao Liang". Thanks :) On 08/04/2017 07:05 PM, Wei Liu wrote: On Fri, Aug 04, 2017 at 11:57:53AM +0100, Wei Liu wrote: On Tue, Aug 01, 2017 at 05:10:27PM +0100, Wei Liu wrote: On Tue, Aug 01, 2017 at 11:57:50PM +0800, Xiao Liang wrote: From: xiliang The manual of xl s

Re: [Xen-devel] [PATCH v2] xl: add --clear option to dmesg command

2017-08-04 Thread Wei Liu
On Fri, Aug 04, 2017 at 10:34:18PM +0800, Xiao Liang wrote: > Yes, the full name is "Xiao Liang". Thanks :) > Cool, thanks for confirming. I will push your patch shortly. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-deve

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Juergen Gross
On 04/08/17 15:47, Wei Liu wrote: > On Fri, Aug 04, 2017 at 03:20:09PM +0200, Juergen Gross wrote: >> Last year Jan posted a patch series to change hypervisor log level >> thresholds via xl command [1]. This approach was later modified by Wei >> resulting in patch series [2]. >> >> I'd like to foll

Re: [Xen-devel] Runtime adjustment of hypervisor parameters

2017-08-04 Thread Juergen Gross
On 04/08/17 15:57, Andrew Cooper wrote: > On 04/08/17 14:36, Juergen Gross wrote: >> On 04/08/17 15:23, Andrew Cooper wrote: >>> On 04/08/17 14:20, Juergen Gross wrote: Last year Jan posted a patch series to change hypervisor log level thresholds via xl command [1]. This approach was late

[Xen-devel] [xen-unstable-smoke test] 112446: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112446 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112446/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

Re: [Xen-devel] [RFC PATCH 4/4] libxl: support creation and destruction of static shared memory areas

2017-08-04 Thread Wei Liu
I skim through this patch and have some questions. On Fri, Aug 04, 2017 at 10:20:25AM +0800, Zhongze Liu wrote: > + > +static int libxl__sshm_add_master(libxl__gc *gc, uint32_t domid, > + libxl_static_shm *sshm) > +{ > +int rc, aborting; > +char *sshm_path,

Re: [Xen-devel] [PATCH v2 1/3] x86/vlapic: Introduce vlapic_update_timer

2017-08-04 Thread Jan Beulich
>>> Anthony PERARD 08/04/17 12:52 PM >>> >On Thu, Aug 03, 2017 at 08:59:10AM -0600, Jan Beulich wrote: >> >>> Anthony PERARD 07/18/17 7:10 PM >>> >> >+static void vlapic_update_timer(struct vlapic *vlapic, uint32_t lvtt); >> >+{ >> >+uint64_t period; >> >+uint64_t delta; >> >> Why two li

Re: [Xen-devel] [PATCH v2 2/3] x86/vlapic: Keep timer running when switching between one-shot and periodic mode

2017-08-04 Thread Jan Beulich
>> Anthony PERARD 08/04/17 1:38 PM >>> >On Thu, Aug 03, 2017 at 09:21:57AM -0600, Jan Beulich wrote: >> >>> Anthony PERARD 07/18/17 7:12 PM >>> >> >@@ -818,6 +840,7 @@ static void vlapic_reg_write(struct vcpu *v, >> >if ( !vlapic_lvtt_oneshot(vlapic) && !vlapic_lvtt_period(vlapic) ) >> >break; >>

Re: [Xen-devel] [PATCH] tools/libxl: Fix a segment fault when mmio_hole is set in hvm.cfg

2017-08-04 Thread Wei Liu
On Wed, Aug 02, 2017 at 12:32:58PM -0700, Christopher Clark wrote: > On Wed, Jul 12, 2017 at 3:15 AM, Wei Liu wrote: > > > > On Thu, Jul 13, 2017 at 10:03:39AM +0800, Xiong Zhang wrote: > > > When valid mmio_hole is set in hvm.cfg, segment fault happens at accessing > > > localents pointer. > > >

Re: [Xen-devel] [PATCH v2 1/3] x86/vlapic: Introduce vlapic_update_timer

2017-08-04 Thread Anthony PERARD
On Fri, Aug 04, 2017 at 09:40:32AM -0600, Jan Beulich wrote: > >>> Anthony PERARD 08/04/17 12:52 PM >>> > >On Thu, Aug 03, 2017 at 08:59:10AM -0600, Jan Beulich wrote: > >> >>> Anthony PERARD 07/18/17 7:10 PM >>> > >> >+static void vlapic_update_timer(struct vlapic *vlapic, uint32_t lvtt); > >> >

Re: [Xen-devel] [PATCH v2 2/3] x86/vlapic: Keep timer running when switching between one-shot and periodic mode

2017-08-04 Thread Anthony PERARD
On Fri, Aug 04, 2017 at 09:48:59AM -0600, Jan Beulich wrote: > >> Anthony PERARD 08/04/17 1:38 PM >>> > >On Thu, Aug 03, 2017 at 09:21:57AM -0600, Jan Beulich wrote: > >> >>> Anthony PERARD 07/18/17 7:12 PM >>> > >> >@@ -818,6 +840,7 @@ static void vlapic_reg_write(struct vcpu *v, > >> >if ( !vla

Re: [Xen-devel] [Patch for staging 2/2] x86: remove an ASSERT to avoid crash when destroy a domain.

2017-08-04 Thread Jan Beulich
>>> Yi Sun 08/04/17 11:45 AM >>> >--- a/xen/arch/x86/psr.c >+++ b/xen/arch/x86/psr.c >@@ -1294,9 +1294,7 @@ static void psr_free_cos(struct domain *d) >{ >unsigned int socket, cos; > >- ASSERT(socket_info); >- >-if ( !d->arch.psr_cos_ids ) >+if ( !d->arch.psr_cos_ids || !psr_alloc_feat_en

Re: [Xen-devel] [PATCH v5 01/17] rbtree: changes to align the coding conventions with Linux tree

2017-08-04 Thread Praveen Kumar
Hi Dario, On Thu, Aug 3, 2017 at 4:07 PM, Dario Faggioli wrote: > On Fri, 2017-07-14 at 07:05 -0600, Jan Beulich wrote: >> > > > On 14.07.17 at 14:51, wrote: >> > >> > Agreed, I shouldn't have added. >> > rbtree.h file does include incline functions which are actually >> > commented, and in orde

[Xen-devel] [PATCH v6 2/8] mm: Extract allocation loop from alloc_heap_pages()

2017-08-04 Thread Boris Ostrovsky
This will make code a bit more readable, especially with changes that will be introduced in subsequent patches. Signed-off-by: Boris Ostrovsky --- Changes in v6: * Rebased due to changes in the first patch (thus dropped Jan's ACK) xen/common/page_alloc.c | 139 +++---

[Xen-devel] [PATCH v6 7/8] mm: Print number of unscrubbed pages in 'H' debug handler

2017-08-04 Thread Boris Ostrovsky
Signed-off-by: Boris Ostrovsky Reviewed-by: Wei Liu --- xen/common/page_alloc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/xen/common/page_alloc.c b/xen/common/page_alloc.c index e0bbc27..7cd736c 100644 --- a/xen/common/page_alloc.c +++ b/xen/common/page_alloc.c @@ -2323,6 +2323,

[Xen-devel] [PATCH v6 8/8] mm: Make sure pages are scrubbed

2017-08-04 Thread Boris Ostrovsky
Add a debug Kconfig option that will make page allocator verify that pages that were supposed to be scrubbed are, in fact, clean. Signed-off-by: Boris Ostrovsky Reviewed-by: Jan Beulich --- xen/Kconfig.debug | 7 ++ xen/common/page_alloc.c | 63 +++

[Xen-devel] [PATCH v6 1/8] mm: Place unscrubbed pages at the end of pagelist

2017-08-04 Thread Boris Ostrovsky
. so that it's easy to find pages that need to be scrubbed (those pages are now marked with _PGC_need_scrub bit). We keep track of the first unscrubbed page in a page buddy using first_dirty field. For now it can have two values, 0 (whole buddy needs scrubbing) or INVALID_DIRTY_IDX (the buddy does

[Xen-devel] [PATCH v6 4/8] mm: Scrub memory from idle loop

2017-08-04 Thread Boris Ostrovsky
Instead of scrubbing pages during guest destruction (from free_heap_pages()) do this opportunistically, from the idle loop. We might come to scrub_free_pages()from idle loop while another CPU uses mapcache override, resulting in a fault while trying to do __map_domain_page() in scrub_one_page(). T

[Xen-devel] [PATCH v6 6/8] mm: Keep heap accessible to others while scrubbing

2017-08-04 Thread Boris Ostrovsky
Instead of scrubbing pages while holding heap lock we can mark buddy's head as being scrubbed and drop the lock temporarily. If someone (most likely alloc_heap_pages()) tries to access this chunk it will signal the scrubber to abort scrub by setting head's BUDDY_SCRUB_ABORT bit. The scrubber checks

[Xen-devel] [PATCH v6 5/8] spinlock: Introduce spin_lock_cb()

2017-08-04 Thread Boris Ostrovsky
While waiting for a lock we may want to periodically run some code. This code may, for example, allow the caller to release resources held by it that are no longer needed in the critical section protected by the lock. Specifically, this feature will be needed by scrubbing code where the scrubber,

[Xen-devel] [PATCH v6 3/8] mm: Scrub pages in alloc_heap_pages() if needed

2017-08-04 Thread Boris Ostrovsky
When allocating pages in alloc_heap_pages() first look for clean pages. If none is found then retry, take pages marked as unscrubbed and scrub them. Note that we shouldn't find unscrubbed pages in alloc_heap_pages() yet. However, this will become possible when we stop scrubbing from free_heap_page

[Xen-devel] [PATCH v6 0/8] Memory scrubbing from idle loop

2017-08-04 Thread Boris Ostrovsky
Changes in v6: * Changed first_dirty tracking from pointer-based to index-based (patch 1) * Added/modified a few ASSERT()s * Moved/modifed a couple of comments * Adjusted width of INVALID_DIRTY_IDX (see per-patch changes) When a domain is destroyed the hypervisor must scrub domain's pages before

Re: [Xen-devel] [PATCH v5 01/17] rbtree: changes to align the coding conventions with Linux tree

2017-08-04 Thread Jan Beulich
>>> Praveen Kumar 08/04/17 7:00 PM >>> >There will be issue while directly applying the patch from Linux tree >( having changed the file name ) as the line number changes. How do line numbers matter? Jan ___ Xen-devel mailing list Xen-devel@lists.xen

Re: [Xen-devel] [PATCH v5 01/17] rbtree: changes to align the coding conventions with Linux tree

2017-08-04 Thread Praveen Kumar
I tried applying the patches generated from Linux tree and updating the file location ( as per the Xen tree location ); some of the places, I was facing issues. Adding comment to the file, resolved some of the patching issues. So, this is my understanding ( could be completely wrong ) that, with th

Re: [Xen-devel] [RFC PATCH 4/4] libxl: support creation and destruction of static shared memory areas

2017-08-04 Thread Zhongze Liu
Hi Wei, Thank you for reviewing my patch. 2017-08-04 23:20 GMT+08:00 Wei Liu : > I skim through this patch and have some questions. > > On Fri, Aug 04, 2017 at 10:20:25AM +0800, Zhongze Liu wrote: >> + >> +static int libxl__sshm_add_master(libxl__gc *gc, uint32_t domid, >> +

[Xen-devel] [xen-unstable-smoke test] 112448: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112448 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112448/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

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

2017-08-04 Thread Petre Pircalabu
Enforce the distinction between an instruction not implemented by the emulator and the failure to emulate that instruction. Signed-off-by: Petre Pircalabu --- xen/arch/x86/hvm/emulate.c | 1 + xen/arch/x86/x86_emulate/x86_emulate.c | 2 +- xen/arch/x86/x86_emulate/x86_emulate.h | 2 +

[Xen-devel] [PATCH v7 0/2] Singlestep unimplemented x86emul instructions

2017-08-04 Thread Petre Pircalabu
This patchset implements a new way of handling the instructions unimplemented in x86emul. Instead of inserting a hardware exception the monitor will be notified and it will to try to single-step the faulty instruction using the real processor and then resume execution of the normal instruction flow

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

2017-08-04 Thread Petre Pircalabu
If case of a vm_event with the emulate_flags set, if the instruction is not implemented by the emulator, the monitor should be notified instead of directly injecting a hw exception. This behavior can be used to re-execute an instruction not supported by the emulator using the real processor (e.g. a

Re: [Xen-devel] [PATCH 1/3] xen: remove tests for pvh mode in pure pv paths

2017-08-04 Thread Boris Ostrovsky
On 08/04/2017 07:36 AM, Juergen Gross wrote: > Remove the last tests for XENFEAT_auto_translated_physmap in pure > PV-domain specific paths. PVH V1 is gone and the feature will always > be "false" in PV guests. > > Signed-off-by: Juergen Gross Reviewed-by: Boris Ostrovsky I wonder whether the r

Re: [Xen-devel] [PATCH 2/3] xen: remove unused function xen_set_domain_pte()

2017-08-04 Thread Boris Ostrovsky
On 08/04/2017 07:36 AM, Juergen Gross wrote: > The function xen_set_domain_pte() is used nowhere in the kernel. > Remove it. > > Signed-off-by: Juergen Gross Reviewed-by: Boris Ostrovsky (+ Ingo and Steven who are maintainers of include/trace/events/xen.h) > --- > arch/x86/include/asm/xen/pag

Re: [Xen-devel] [PATCH 3/3] xen: remove not used trace functions

2017-08-04 Thread Boris Ostrovsky
On 08/04/2017 07:36 AM, Juergen Gross wrote: > There are some Xen specific trace functions defined in > include/trace/events/xen.h. Remove them. > > Signed-off-by: Juergen Gross (Again, adding Ingo and Steven) Reviewed-by: Boris Ostrovsky although I think "s/some Xen/some unused Xen/" in the c

[Xen-devel] [linux-next test] 112441: regressions - trouble: blocked/broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112441 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/112441/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ws16-amd64 7 xen-boot fail REGR. vs. 112430 test-amd64-i386-xl-q

[Xen-devel] [xen-unstable-smoke test] 112450: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112450 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112450/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Dario Faggioli
On Fri, 2017-08-04 at 15:34 +0100, Wei Liu wrote: > On Fri, Aug 04, 2017 at 02:53:51PM +0200, Dario Faggioli wrote: > > > > Well, I guess yes. But how does this matter? Each scheduler can, if > > it > > wants, use all these parameters in the way it actuallly prefers. > > So, > > the fact that RTDS

Re: [Xen-devel] [PATCH RFC v1 3/3] xl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Meng Xu
On Fri, Aug 4, 2017 at 5:01 AM, Dario Faggioli wrote: > On Thu, 2017-08-03 at 18:02 -0400, Meng Xu wrote: >> On Thu, Aug 3, 2017 at 12:03 PM, Dario Faggioli >> wrote: >> > >> > > @@ -702,14 +705,18 @@ int main_sched_rtds(int argc, char **argv) >> > > int *vcpus = (int *)xmalloc(sizeof(int));

Re: [Xen-devel] [PATCH RFC v1 2/3] libxl: enable per-VCPU work conserving flag for RTDS

2017-08-04 Thread Meng Xu
On Fri, Aug 4, 2017 at 10:34 AM, Wei Liu wrote: > On Fri, Aug 04, 2017 at 02:53:51PM +0200, Dario Faggioli wrote: >> On Fri, 2017-08-04 at 13:10 +0100, Wei Liu wrote: >> > On Fri, Aug 04, 2017 at 10:13:18AM +0200, Dario Faggioli wrote: >> > > On Thu, 2017-08-03 at 17:39 -0400, Meng Xu wrote: >> >

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

2017-08-04 Thread osstest service owner
flight 112442 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/112442/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-examine 7 reboot fail REGR. vs. 110515 test-amd64-amd64-xl

[Xen-devel] [xen-unstable-smoke test] 112451: regressions - trouble: broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112451 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/112451/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-i386 10 debian-hvm-install fail REGR. vs. 112402 test-

[Xen-devel] [linux-3.18 test] 112443: trouble: blocked/broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112443 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/112443/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-pvops 3 capture-logs broken REGR. vs. 112102

[Xen-devel] [qemu-mainline test] 112444: regressions - trouble: blocked/broken/fail/pass

2017-08-04 Thread osstest service owner
flight 112444 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/112444/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 112422 test-armhf

  1   2   >