Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-29 Thread Tian, Kevin
> From: Xuquan (Euler) [mailto:xuqu...@huawei.com] > Sent: Friday, August 19, 2016 8:59 PM > > When Xen apicv is enabled, wall clock time is faster on Windows7-32 guest > with high payload (with 2vCPU, captured from xentrace, in high payload, the > count of IPI interrupt increases rapidly between

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Sanghyun Hong
Yes, this will allow the hypervisor to collect samples from multiple guests. However, the tool (perf) probably won't be able to properly process these samples. But you can try. I understand, thus, I applied the patches and set the pmu_mode to all. However, I’m really curious what you mean by the

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Sanghyun Hong
Hi Boris, [Appreciate for your support, and I have one last question] Since we’re using the Xen hypervisor (not the KVM that is type-II), I think we cannot use the perf kvm command. Here are the outputs: # perf kvm --host --guest record -C 1 sleep 1 [ perf record: Woken up 1 times to

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Sanghyun Hong
I really appreciate for your answers. Last but not the least, I want to make it more clear: > The hypervisor will provide dom0 with a raw sample (guest's, dom0's or > hypervisor's) and then it's the job of dom0 kernel to properly tag and > format it and make it available to the userland (i.e.

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Sanghyun Hong
Hi Boris, I’ve found the documentations(https://github.com/torvalds/linux/blob/master/Documentation/ABI/testing/sysfs-hypervisor-pmu) in the kernel source code, and it seems like if we change the mode from self to all then we can collect counters for all the domains, right? All the best,

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

2016-08-29 Thread osstest service owner
flight 100656 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100656/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf e53f1e253e01026029f5ce7474a9d8421c8a0fbb baseline version: ovmf

Re: [Xen-devel] [PATCH v4 07/16] libxl/arm: Construct ACPI GTDT table

2016-08-29 Thread Shannon Zhao
On 2016/8/30 2:16, Julien Grall wrote: > On 16/08/2016 06:25, Shannon Zhao wrote: >> From: Shannon Zhao >> >> Construct GTDT table with the interrupt information of timers. >> >> Signed-off-by: Shannon Zhao >> --- >>

Re: [Xen-devel] [PATCH v4 14/16] public/hvm/params.h: Add macros for HVM_PARAM_CALLBACK_TYPE_PPI

2016-08-29 Thread Shannon Zhao
On 2016/8/30 3:00, Julien Grall wrote: > Hi Shannon, > > On 16/08/2016 06:25, Shannon Zhao wrote: >> From: Shannon Zhao >> >> Add macros for HVM_PARAM_CALLBACK_TYPE_PPI operation values and update >> them in evtchn_fixup(). >> >> Signed-off-by: Shannon Zhao

Re: [Xen-devel] [PATCH v4 16/16] libxl/arm: Add the size of ACPI tables to maxmem

2016-08-29 Thread Shannon Zhao
On 2016/8/30 3:07, Julien Grall wrote: > Hi Shannon, > > On 16/08/2016 06:25, Shannon Zhao wrote: >> From: Shannon Zhao >> >> While it defines the maximum size of guest ACPI tables in guest >> memory layout, here it adds the size to set the target maxmem >> to avoid

Re: [Xen-devel] [PATCH v4 05/16] libxl/arm: Construct ACPI RSDP table

2016-08-29 Thread Shannon Zhao
On 2016/8/30 2:05, Julien Grall wrote: > Hi Shannon, > > On 25/08/2016 04:05, Shannon Zhao wrote: >> >> >> On 2016/8/24 20:52, Wei Liu wrote: >>> On Tue, Aug 16, 2016 at 06:25:02PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > Construct ACPI RSDP

Re: [Xen-devel] [PATCH v4 14/16] public/hvm/params.h: Add macros for HVM_PARAM_CALLBACK_TYPE_PPI

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Add macros for HVM_PARAM_CALLBACK_TYPE_PPI operation values and update them in evtchn_fixup(). Signed-off-by: Shannon Zhao --- xen/arch/arm/domain_build.c | 8

Re: [Xen-devel] [PATCH v4 12/16] libxl/arm: Factor finalise_one_memory_node as a gerneric function

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Rename finalise_one_memory_node to finalise_one_node and pass the node name via function parameter. This is useful for adding ACPI module which will be added by a later patch. Signed-off-by:

Re: [Xen-devel] [PATCH v4 09/16] libxl/arm: Construct ACPI MADT table

2016-08-29 Thread Julien Grall
On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao According to the GIC version, construct the MADT table. Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 84 1 file

Re: [Xen-devel] [PATCH v4 16/16] libxl/arm: Add the size of ACPI tables to maxmem

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao While it defines the maximum size of guest ACPI tables in guest memory layout, here it adds the size to set the target maxmem to avoid providing less available memory for guest. Signed-off-by:

Re: [Xen-devel] [PATCH v4 00/16] Xen ARM DomU ACPI support

2016-08-29 Thread Julien Grall
Hi Wei, On 24/08/2016 08:58, Wei Liu wrote: On Tue, Aug 16, 2016 at 06:24:57PM +0800, Shannon Zhao wrote: From: Shannon Zhao The design of this feature is described as below. Firstly, the toolstack (libxl) generates the ACPI tables according the number of vcpus and

Re: [Xen-devel] [PATCH v4 05/16] libxl/arm: Construct ACPI RSDP table

2016-08-29 Thread Julien Grall
Hi Shannon, On 25/08/2016 04:05, Shannon Zhao wrote: On 2016/8/24 20:52, Wei Liu wrote: On Tue, Aug 16, 2016 at 06:25:02PM +0800, Shannon Zhao wrote: From: Shannon Zhao Construct ACPI RSDP table and add a helper to calculate the ACPI table checksum.

Re: [Xen-devel] [PATCH v4 13/16] libxl/arm: Add ACPI module

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Add the ARM Multiboot module for ACPI, so UEFI or DomU can get the base address of ACPI tables from it. Signed-off-by: Shannon Zhao Acked-by: Julien Grall

Re: [Xen-devel] [PATCH v4 10/16] libxl/arm: Construct ACPI FADT table

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/tools/libxl/libxl_arm_acpi.c

Re: [Xen-devel] [PATCH v4 08/16] libxl/arm: Factor MPIDR computing codes out as a helper

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Factor MPIDR computing codes out as a helper, so it could be shared between DT and ACPI. Signed-off-by: Shannon Zhao Acked-by: Julien Grall

Re: [Xen-devel] [PATCH v4 06/16] libxl/arm: Construct ACPI XSDT table

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 30 ++ 1 file changed, 30 insertions(+) diff --git

Re: [Xen-devel] [PATCH v4 05/16] libxl/arm: Construct ACPI RSDP table

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Construct ACPI RSDP table and add a helper to calculate the ACPI table checksum. Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 38

Re: [Xen-devel] [PATCH v4 07/16] libxl/arm: Construct ACPI GTDT table

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao Construct GTDT table with the interrupt information of timers. Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 29 + 1 file

[Xen-devel] [xen-unstable test] 100655: tolerable FAIL - PUSHED

2016-08-29 Thread osstest service owner
flight 100655 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100655/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-rtds 6 xen-boot fail like 100627 build-amd64-rumpuserxen

Re: [Xen-devel] [PATCH v4 03/16] libxl/arm: Generate static ACPI DSDT table

2016-08-29 Thread Julien Grall
Hi Shannon, On 16/08/2016 06:25, Shannon Zhao wrote: From: Shannon Zhao It uses static DSDT table like the way x86 uses. Currently the DSDT table only contains processor device objects and it generates the maximal objects which so far is 128. Also only check iasl for

Re: [Xen-devel] [PATCH] Fix a BUG_ON issue

2016-08-29 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Monday, August 29, 2016 7:51 PM > To: Wu, Feng > Cc: xen-devel@lists.xen.org > Subject: Re: [PATCH] Fix a BUG_ON issue > > >>> On 29.08.16 at 11:14, wrote: > > ---

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Boris Ostrovsky
On 08/29/2016 05:48 PM, Sanghyun Hong wrote: > I really appreciate for your answers. Last but not the least, I want to make > it more clear: > >> The hypervisor will provide dom0 with a raw sample (guest's, dom0's or >> hypervisor's) and then it's the job of dom0 kernel to properly tag and >>

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Boris Ostrovsky
On 08/29/2016 05:08 PM, Sanghyun Hong wrote: >> Yes, this will allow the hypervisor to collect samples from multiple >> guests. However, the tool (perf) probably won't be able to properly >> process these samples. But you can try. > > I understand, thus, I applied the patches and set > the

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Boris Ostrovsky
On 08/29/2016 02:42 PM, Sanghyun Hong wrote: > Hi Boris, > > I’ve found the > documentations(https://github.com/torvalds/linux/blob/master/Documentation/ABI/testing/sysfs-hypervisor-pmu) > in the kernel source code, and it seems like if we change the mode > from *self* to *all* then we can

Re: [Xen-devel] [PATCH v2 1/2] mini-os: partially revert "remove using start_info ..."

2016-08-29 Thread Samuel Thibault
Juergen Gross, on Mon 29 Aug 2016 16:18:26 +0200, wrote: > Commit e33452c4f5547ed14defe6382b3b53664ac5bd8a ("remove using > start_info in architecture independent code") removed the start_info > variable completely. grub stubdom needs the start_info structure. > > Readd the start_info structure,

Re: [Xen-devel] [PATCH v2 1/2] mini-os: partially revert "remove using start_info ..."

2016-08-29 Thread Samuel Thibault
Samuel Thibault, on Mon 29 Aug 2016 22:49:07 +0200, wrote: > Juergen Gross, on Mon 29 Aug 2016 16:18:26 +0200, wrote: > > +union start_info_union > > +{ > > +start_info_t start_info; > > +char padding[512]; > > +}; > > Why defining a union? 512 is actually not enough for start_info_t.

Re: [Xen-devel] [PATCH v2 1/2] mini-os: partially revert "remove using start_info ..."

2016-08-29 Thread Samuel Thibault
Hello, Juergen Gross, on Mon 29 Aug 2016 16:18:26 +0200, wrote: > +union start_info_union > +{ > +start_info_t start_info; > +char padding[512]; > +}; Why defining a union? 512 is actually not enough for start_info_t. Samuel ___ Xen-devel

Re: [Xen-devel] [PATCH 2/2] mini-os: don't get xenbus parameters if xenbus is disabled

2016-08-29 Thread Samuel Thibault
Wei Liu, on Mon 29 Aug 2016 12:11:21 +0100, wrote: > On Mon, Aug 29, 2016 at 01:01:09PM +0200, Juergen Gross wrote: > > get_xenbus() should be called only if CONFIG_XENBUS is set. > > > > Signed-off-by: Juergen Gross > > Reviewed-by: Wei Liu Acked-by:

Re: [Xen-devel] [MINIOS PATCH 1/3] Makefile: simplify all_sources macro

2016-08-29 Thread Samuel Thibault
Juergen Gross, on Mon 29 Aug 2016 15:07:20 +0200, wrote: > On 29/08/16 15:01, Wei Liu wrote: > > There is no SCCS file. > > > > Signed-off-by: Wei Liu > > Reviewed-by: Juergen Gross Acked-by: Samuel Thibault

Re: [Xen-devel] [MINIOS PATCH 3/3] gitignore: ignore files generated by various indexing software

2016-08-29 Thread Samuel Thibault
Juergen Gross, on Mon 29 Aug 2016 15:07:55 +0200, wrote: > On 29/08/16 15:01, Wei Liu wrote: > > Signed-off-by: Wei Liu > > Reviewed-by: Juergen Gross Acked-by: Samuel Thibault ___

Re: [Xen-devel] [MINIOS PATCH 2/3] Makefile: add gtags target

2016-08-29 Thread Samuel Thibault
Juergen Gross, on Mon 29 Aug 2016 15:07:39 +0200, wrote: > On 29/08/16 15:01, Wei Liu wrote: > > Use GNU Global to generate source code index. > > > > Signed-off-by: Wei Liu > > Reviewed-by: Juergen Gross Acked-by: Samuel Thibault

[Xen-devel] Xen EFI build system and gcov

2016-08-29 Thread Wei Liu
Hi Jan Today I had some free cycles so I spent some time looking at gcov support in the hypervisor and tried to write a patch to fix the currently broken gcov build. But my patch alone is not enough to fix that. There seems to be a problem with the EFI Makefile. With my patch applied,

Re: [Xen-devel] [PATCH] xen: Remove event channel notification through Xen PCI platform device

2016-08-29 Thread Boris Ostrovsky
On 08/26/2016 05:55 PM, KarimAllah Ahmed wrote: > Ever since commit 254d1a3f02eb ("xen/pv-on-hvm kexec: shutdown watches > from old kernel") using the INTx interrupt from Xen PCI platform device for > event channel notification would just lockup the guest during bootup. > postcore_initcall now

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Razvan Cojocaru
On 08/29/16 19:20, Jan Beulich wrote: On 29.08.16 at 18:02, wrote: >> On 08/29/16 18:42, Jan Beulich wrote: >> On 29.08.16 at 17:29, wrote: On 08/26/2016 11:14 AM, Jan Beulich wrote: On 26.08.16 at 09:40,

[Xen-devel] [xen-unstable-smoke test] 100654: tolerable all pass - PUSHED

2016-08-29 Thread osstest service owner
flight 100654 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100654/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Boris Ostrovsky
On 08/29/2016 09:18 AM, Sanghyun Hong wrote: > Dear Xen-Devel Community: > > I’m a grad student working on measuring performance counters at the > Xen domains. I read this > thread(https://wiki.xenproject.org/wiki/Xen_Profiling:_oprofile_and_perf) in > web, and it says using Linux perf command

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 18:02, wrote: > On 08/29/16 18:42, Jan Beulich wrote: > On 29.08.16 at 17:29, wrote: >>> On 08/26/2016 11:14 AM, Jan Beulich wrote: >>> On 26.08.16 at 09:40, wrote: > On 08/26/16

Re: [Xen-devel] [PATCH] x86/PV: make PMU MSR handling consistent

2016-08-29 Thread Boris Ostrovsky
On 08/29/2016 11:37 AM, Jan Beulich wrote: > So far accesses to Intel MSRs on an AMD system fall through to the > default case, while accesses to AMD MSRs on an Intel system bail (in > the RDMSR case without updating EAX and EDX). Make the "AMD MSRs on > Intel" case match the "Intel MSR on AMD"

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Razvan Cojocaru
On 08/29/16 18:42, Jan Beulich wrote: On 29.08.16 at 17:29, wrote: >> On 08/26/2016 11:14 AM, Jan Beulich wrote: >> On 26.08.16 at 09:40, wrote: On 08/26/16 10:18, Jan Beulich wrote: On 26.08.16 at 08:11,

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 17:29, wrote: > On 08/26/2016 11:14 AM, Jan Beulich wrote: > On 26.08.16 at 09:40, wrote: >>> On 08/26/16 10:18, Jan Beulich wrote: >>> On 26.08.16 at 08:11, wrote: > @@ -76,6 +76,17

[Xen-devel] [PATCH] x86/PV: make PMU MSR handling consistent

2016-08-29 Thread Jan Beulich
So far accesses to Intel MSRs on an AMD system fall through to the default case, while accesses to AMD MSRs on an Intel system bail (in the RDMSR case without updating EAX and EDX). Make the "AMD MSRs on Intel" case match the "Intel MSR on AMD" one. Signed-off-by: Jan Beulich

Re: [Xen-devel] linux-4.7.* under xen-4.6.* gives "unhandled page fault (ec=0000)" at boot. -- Root cause found.

2016-08-29 Thread Håkon Alstadheim
Den 29. aug. 2016 13:04, skrev Jan Beulich: On 29.08.16 at 01:14, wrote: > Den 17. aug. 2016 21:56, I wrote (to xen-users, as I am no developer): >>> I'm on gentoo, running gentoo-sources kernel for dom0. >>> >>> I am unable to run gentoo-sources-4.7.{0,1}. I'm

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Razvan Cojocaru
On 08/26/2016 11:14 AM, Jan Beulich wrote: On 26.08.16 at 09:40, wrote: >> On 08/26/16 10:18, Jan Beulich wrote: >> On 26.08.16 at 08:11, wrote: --- a/xen/common/compat/memory.c +++ b/xen/common/compat/memory.c @@

[Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-29 Thread Seth Forshee
Mounting proc in user namespace containers fails if the xenbus filesystem is mounted on /proc/xen because this directory fails the "permanently empty" test. proc_create_mount_point() exists specifically to create such mountpoints in proc but is currently proc-internal. Export this interface to

Re: [Xen-devel] linux-4.7.* under xen-4.6.* gives "unhandled page fault (ec=0000)" at boot. -- Root cause found.

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 16:26, wrote: > On Mon, Aug 29, 2016 at 05:04:05AM -0600, Jan Beulich wrote: >> >>> On 29.08.16 at 01:14, wrote: >> >> First of all - please don't cross post. >> >> > Den 17. aug. 2016 21:56, I wrote (to xen-users, as I am no

Re: [Xen-devel] [PATCH] xen/HVM: Add guarding logic for VMX specific code

2016-08-29 Thread Konrad Rzeszutek Wilk
On Mon, Aug 29, 2016 at 07:01:23AM -0500, Suravee Suthikulpanit wrote: > The struct hvm_domain.vmx is defined in a union along with the svm. > This can causes issue for SVM since this code is used in the common You scared me with the 'can causes'! Digging in the pahole output - it It can't cause

[Xen-devel] [PATCH v2 0/2] mini-os: repair stubdom build

2016-08-29 Thread Juergen Gross
The patch series adding HVMlite support for Mini-OS unfortunately broke building Xen's stubdoms. This small series repairs the broken bits again. V2: patch 1: remove CONFIG_KEEP_STARTINFO Juergen Gross (2): mini-os: partially revert "remove using start_info ..." mini-os: don't get xenbus

Re: [Xen-devel] linux-4.7.* under xen-4.6.* gives "unhandled page fault (ec=0000)" at boot. -- Root cause found.

2016-08-29 Thread Konrad Rzeszutek Wilk
On Mon, Aug 29, 2016 at 05:04:05AM -0600, Jan Beulich wrote: > >>> On 29.08.16 at 01:14, wrote: > > First of all - please don't cross post. > > > Den 17. aug. 2016 21:56, I wrote (to xen-users, as I am no developer): > >> I'm on gentoo, running gentoo-sources kernel

[Xen-devel] [PATCH v2 2/2] mini-os: don't get xenbus parameters if xenbus is disabled

2016-08-29 Thread Juergen Gross
get_xenbus() should be called only if CONFIG_XENBUS is set. Signed-off-by: Juergen Gross --- include/xenbus.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/xenbus.h b/include/xenbus.h index 5646a25..c254652 100644 --- a/include/xenbus.h +++

[Xen-devel] [PATCH v2 1/2] mini-os: partially revert "remove using start_info ..."

2016-08-29 Thread Juergen Gross
Commit e33452c4f5547ed14defe6382b3b53664ac5bd8a ("remove using start_info in architecture independent code") removed the start_info variable completely. grub stubdom needs the start_info structure. Readd the start_info structure, but make it dependent on CONFIG_PARAVIRT. Signed-off-by: Juergen

Re: [Xen-devel] [PATCH] stubdom: add CONFIG_KEEP_STARTINFO for stubdoms which reference start_info

2016-08-29 Thread Juergen Gross
On 29/08/16 13:01, Juergen Gross wrote: > grub-stubdom and ioemu-stubdom rely on Mini-OS exporting the start_info > structure via a global variable. To use this feature in future we must > specify CONFIG_KEEP_STARTINFO in the Mini-OS config file. > > Signed-off-by: Juergen Gross

[Xen-devel] [distros-debian-sid test] 67603: tolerable FAIL

2016-08-29 Thread Platform Team regression test user
flight 67603 distros-debian-sid real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67603/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-amd64-sid-netboot-pygrub 9 debian-di-install fail like 67575

Re: [Xen-devel] [PATCH v4 14/16] kprobes: move kprobe declarations to asm-generic/kprobes.h

2016-08-29 Thread Masami Hiramatsu
On Tue, 23 Aug 2016 18:31:05 +0200 "Luis R. Rodriguez" wrote: > On Tue, Aug 23, 2016 at 12:11:40AM +0900, Masami Hiramatsu wrote: > > On Fri, 19 Aug 2016 14:34:12 -0700 > > mcg...@kernel.org wrote: > > > > > From: "Luis R. Rodriguez" > > > > > > Often all

[Xen-devel] Ping: [PATCH] replace bogus -ENOSYS uses

2016-08-29 Thread Jan Beulich
>>> On 09.08.16 at 12:40, wrote: > This doesn't cover all of them, just the ones that I think would most > obviously better be -EINVAL or -EOPNOTSUPP. Ping? There was a little bit of feedback, but non that really resulted in a need to change something (so far at least). Jan

[Xen-devel] [PATCH RFC] x86/32on64: don't modify guest descriptors without need

2016-08-29 Thread Jan Beulich
There are two cases: The obvious one is that system gates with type 0 shouldn't have what might be their DPL altered - such descriptors can't be used anyway without incurring a #GP, and hence adjusting its DPL is only risking to confuse the guest. The less obvious (and potentially controversial)

[Xen-devel] [PATCH] x86/32on64: misc adjustments to call gate emulation

2016-08-29 Thread Jan Beulich
- There's no 32-bit displacement in 16-bit addressing mode. - It is wrong to ASSERT() anything on parts of an instruction fetched from guest memory. - The two scaling bits of a SIB byte don't affect whether there is no scaled index register. Signed-off-by: Jan Beulich ---

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

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

[Xen-devel] Questions about Using Perf at Dom0

2016-08-29 Thread Sanghyun Hong
Dear Xen-Devel Community: I’m a grad student working on measuring performance counters at the Xen domains. I read this thread(https://wiki.xenproject.org/wiki/Xen_Profiling:_oprofile_and_perf) in web, and it says using Linux perf command will let us collecting performance counters in both

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Juergen Gross
On 29/08/16 14:32, Andrew Cooper wrote: > On 29/08/2016 13:28, Juergen Gross wrote: >> On 29/08/16 13:47, Andrew Cooper wrote: >>> On 29/08/2016 12:17, Juergen Gross wrote: On 29/08/16 13:09, Wei Liu wrote: > On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: >> grub

Re: [Xen-devel] [MINIOS PATCH 3/3] gitignore: ignore files generated by various indexing software

2016-08-29 Thread Juergen Gross
On 29/08/16 15:01, Wei Liu wrote: > Signed-off-by: Wei Liu Reviewed-by: Juergen Gross Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [MINIOS PATCH 2/3] Makefile: add gtags target

2016-08-29 Thread Juergen Gross
On 29/08/16 15:01, Wei Liu wrote: > Use GNU Global to generate source code index. > > Signed-off-by: Wei Liu Reviewed-by: Juergen Gross Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [MINIOS PATCH 1/3] Makefile: simplify all_sources macro

2016-08-29 Thread Juergen Gross
On 29/08/16 15:01, Wei Liu wrote: > There is no SCCS file. > > Signed-off-by: Wei Liu Reviewed-by: Juergen Gross Juergen ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

[Xen-devel] [MINIOS PATCH 0/3] Some miscellaneous patches

2016-08-29 Thread Wei Liu
Wei Liu (3): Makefile: simplify all_sources macro Makefile: add gtags target gitignore: ignore files generated by various indexing software .gitignore | 7 +++ Makefile | 6 +- 2 files changed, 12 insertions(+), 1 deletion(-) -- 2.1.4

[Xen-devel] [MINIOS PATCH 3/3] gitignore: ignore files generated by various indexing software

2016-08-29 Thread Wei Liu
Signed-off-by: Wei Liu --- .gitignore | 7 +++ 1 file changed, 7 insertions(+) diff --git a/.gitignore b/.gitignore index efc193c..e55133d 100644 --- a/.gitignore +++ b/.gitignore @@ -2,6 +2,13 @@ *.o *.a *.swp +cscope.* +GPATH +GRTAGS +GTAGS +TAGS +tags +

[Xen-devel] [MINIOS PATCH 1/3] Makefile: simplify all_sources macro

2016-08-29 Thread Wei Liu
There is no SCCS file. Signed-off-by: Wei Liu --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index b783684..a8ae67c 100644 --- a/Makefile +++ b/Makefile @@ -175,7 +175,7 @@ clean: arch_clean define all_sources

[Xen-devel] [MINIOS PATCH 2/3] Makefile: add gtags target

2016-08-29 Thread Wei Liu
Use GNU Global to generate source code index. Signed-off-by: Wei Liu --- Makefile | 4 1 file changed, 4 insertions(+) diff --git a/Makefile b/Makefile index a8ae67c..43dcbd6 100644 --- a/Makefile +++ b/Makefile @@ -190,3 +190,7 @@ tags: .PHONY: TAGS TAGS:

[Xen-devel] [linux-3.10 baseline-only test] 67602: regressions - FAIL

2016-08-29 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67602 linux-3.10 real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67602/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qcow210 guest-start

Re: [Xen-devel] IOMMU initialization failure when using linux-as-bootloader

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 14:00, wrote: >> I think this is reasonable, despite it certainly being unexpected for >> the BIOS to turn such on when not putting the system into x2APIC >> mode. > > It seems that linux doesn't use x2APIC because the BIOS explicitly > "opts

Re: [Xen-devel] find_next{,_zero}_bit() inconsistencies

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 14:03, wrote: > On 29/08/2016 12:59, Jan Beulich wrote: >> in the context of >> https://lists.xenproject.org/archives/html/xen-devel/2016-08/msg03068.html >> I once again came across the different behavior our various >> implementations of the

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Andrew Cooper
On 29/08/2016 13:28, Juergen Gross wrote: > On 29/08/16 13:47, Andrew Cooper wrote: >> On 29/08/2016 12:17, Juergen Gross wrote: >>> On 29/08/16 13:09, Wei Liu wrote: On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: > grub stubdom needs the start_info structure. Keep a copy

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Juergen Gross
On 29/08/16 13:47, Andrew Cooper wrote: > On 29/08/2016 12:17, Juergen Gross wrote: >> On 29/08/16 13:09, Wei Liu wrote: >>> On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: grub stubdom needs the start_info structure. Keep a copy of it in pv mini-os if configured via

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Razvan Cojocaru
On 08/29/2016 03:11 PM, Razvan Cojocaru wrote: > On 08/26/2016 10:18 AM, Jan Beulich wrote: > On 26.08.16 at 08:11, wrote: >>> +rc = p2m_set_mem_access(d, _gfn(mao.pfn), arr, mao.nr, start_iter, >>> +MEMOP_CMD_MASK,

Re: [Xen-devel] [PATCH RFC] tools/libxc, xen/x86: Added xc_set_mem_access_sparse()

2016-08-29 Thread Razvan Cojocaru
On 08/26/2016 10:18 AM, Jan Beulich wrote: On 26.08.16 at 08:11, wrote: >> +rc = p2m_set_mem_access(d, _gfn(mao.pfn), arr, mao.nr, start_iter, >> +MEMOP_CMD_MASK, mao.access, 0); > > Please don't pass mao.pfn here when it's

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Wei Liu
On Mon, Aug 29, 2016 at 01:17:56PM +0200, Juergen Gross wrote: > On 29/08/16 13:09, Wei Liu wrote: > > On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: > >> grub stubdom needs the start_info structure. Keep a copy of it in > >> pv mini-os if configured via "CONFIG_KEEP_STARTINFO".

Re: [Xen-devel] IOMMU initialization failure when using linux-as-bootloader

2016-08-29 Thread Sylvain Munaut
Hi Jan, > I think this is reasonable, despite it certainly being unexpected for > the BIOS to turn such on when not putting the system into x2APIC > mode. It seems that linux doesn't use x2APIC because the BIOS explicitly "opts out" of it : "DMAR-IR: x2apic is disabled because BIOS sets x2apic

[Xen-devel] find_next{,_zero}_bit() inconsistencies

2016-08-29 Thread Jan Beulich
Hello, in the context of https://lists.xenproject.org/archives/html/xen-devel/2016-08/msg03068.html I once again came across the different behavior our various implementations of the $subject functions, in particular their varying handling of the offset argument being greater / greater-or-equal

Re: [Xen-devel] [PATCH] Fix a BUG_ON issue

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 11:14, wrote: > --- a/xen/drivers/passthrough/io.c > +++ b/xen/drivers/passthrough/io.c > @@ -243,7 +243,7 @@ static struct vcpu *vector_hashing_dest(const struct > domain *d, > for ( i = 0; i <= mod; i++ ) > { > idx =

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Andrew Cooper
On 29/08/2016 12:17, Juergen Gross wrote: > On 29/08/16 13:09, Wei Liu wrote: >> On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: >>> grub stubdom needs the start_info structure. Keep a copy of it in >>> pv mini-os if configured via "CONFIG_KEEP_STARTINFO". This should >>> default to

Re: [Xen-devel] [Minios-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Juergen Gross
On 29/08/16 13:09, Wei Liu wrote: > On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: >> grub stubdom needs the start_info structure. Keep a copy of it in >> pv mini-os if configured via "CONFIG_KEEP_STARTINFO". This should >> default to "n" in order to have it enabled only when

Re: [Xen-devel] [PATCH 2/2] mini-os: don't get xenbus parameters if xenbus is disabled

2016-08-29 Thread Wei Liu
On Mon, Aug 29, 2016 at 01:01:09PM +0200, Juergen Gross wrote: > get_xenbus() should be called only if CONFIG_XENBUS is set. > > Signed-off-by: Juergen Gross Reviewed-by: Wei Liu ___ Xen-devel mailing list

Re: [Xen-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Wei Liu
On Mon, Aug 29, 2016 at 01:01:08PM +0200, Juergen Gross wrote: > grub stubdom needs the start_info structure. Keep a copy of it in > pv mini-os if configured via "CONFIG_KEEP_STARTINFO". This should > default to "n" in order to have it enabled only when really needed. > I'm not sure I understand

Re: [Xen-devel] linux-4.7.* under xen-4.6.* gives "unhandled page fault (ec=0000)" at boot. -- Root cause found.

2016-08-29 Thread Jan Beulich
>>> On 29.08.16 at 01:14, wrote: First of all - please don't cross post. > Den 17. aug. 2016 21:56, I wrote (to xen-users, as I am no developer): >> I'm on gentoo, running gentoo-sources kernel for dom0. >> >> I am unable to run gentoo-sources-4.7.{0,1}. I'm running

[Xen-devel] [PATCH 1/2] mini-os: support keeping start_info structure conditionally

2016-08-29 Thread Juergen Gross
grub stubdom needs the start_info structure. Keep a copy of it in pv mini-os if configured via "CONFIG_KEEP_STARTINFO". This should default to "n" in order to have it enabled only when really needed. Signed-off-by: Juergen Gross --- Config.mk| 2 ++

[Xen-devel] [PATCH 0/2] mini-os: repair stubdom build

2016-08-29 Thread Juergen Gross
The patch series adding HVMlite support for Mini-OS unfortunately broke building Xen's stubdoms. This small series repairs the broken bits again. Please note that there is still some action required in Xen: ioemu and grub stubdom minios.cfg files need CONFIG_KEEP_STARTINFO=y to be added. I'm

[Xen-devel] [PATCH 2/2] mini-os: don't get xenbus parameters if xenbus is disabled

2016-08-29 Thread Juergen Gross
get_xenbus() should be called only if CONFIG_XENBUS is set. Signed-off-by: Juergen Gross --- include/xenbus.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/include/xenbus.h b/include/xenbus.h index 5646a25..c254652 100644 --- a/include/xenbus.h +++

[Xen-devel] [PATCH] stubdom: add CONFIG_KEEP_STARTINFO for stubdoms which reference start_info

2016-08-29 Thread Juergen Gross
grub-stubdom and ioemu-stubdom rely on Mini-OS exporting the start_info structure via a global variable. To use this feature in future we must specify CONFIG_KEEP_STARTINFO in the Mini-OS config file. Signed-off-by: Juergen Gross --- stubdom/grub/minios.cfg | 1 +

Re: [Xen-devel] IOMMU initialization failure when using linux-as-bootloader

2016-08-29 Thread Jan Beulich
>>> On 27.08.16 at 12:40, wrote: > I'll test this in more detail next week to make sure things really > work as expected (atm I'm just testing if dom0 boots properly). And if > things turn OK and you think this is a proper fix for the issue and > safe to apply, I

Re: [Xen-devel] [PATCH net-next] xen-netfront: avoid packet loss when ethernet header crosses page boundary

2016-08-29 Thread Vitaly Kuznetsov
Vitaly Kuznetsov writes: > David Vrabel writes: > >> On 22/08/16 16:42, Vitaly Kuznetsov wrote: >>> >>> I see two ways to fix the issue: >>> - Change the 'wire' protocol between netfront and netback to start keeping >>> the original SKB

Re: [Xen-devel] Save/Restore is not working properly

2016-08-29 Thread Cendrin Sa
I have tested it both on Xen 4.8 and Debian Stretch : Kernel version 4.6 Xen 4.6 and Debian Jessie : Kernel version 3.19 and also Xen 4.6 and 4.7 on ubuntu Kernel 3.16 On Wed, Aug 24, 2016 at 3:38 PM, Roger Pau Monné wrote: > On Fri, Aug 19, 2016 at 08:45:49PM +0430,

Re: [Xen-devel] [PATCH v2 0/3] mini-os: some cleanups

2016-08-29 Thread Wei Liu
On Mon, Aug 29, 2016 at 08:17:19AM +0200, Juergen Gross wrote: > Do some cleanups in Mini-OS. > > V2: modified patch 2 as suggested by Andrew Cooper > > Juergen Gross (3): > mini-os: cleanup x86_32.S > mini-os: cleanup x86_64.S > mini-os: remove unused functions from sched.c > >

Re: [Xen-devel] [PATCH v3 5/6] x86/time: implement PVCLOCK_TSC_STABLE_BIT

2016-08-29 Thread Jan Beulich
>>> On 26.08.16 at 17:44, wrote: > On 08/25/2016 11:37 AM, Jan Beulich wrote: > On 24.08.16 at 14:43, wrote: >>> This patch proposes relying on host TSC synchronization and >>> passthrough to the guest, when running on a TSC-safe

Re: [Xen-devel] [PATCH v3 3/6] x86/time: streamline platform time init on plt_update()

2016-08-29 Thread Jan Beulich
>>> On 26.08.16 at 17:12, wrote: > On 08/25/2016 11:13 AM, Jan Beulich wrote: > On 24.08.16 at 14:43, wrote: >>> And use to initialize platform time solely for clocksource=tsc, >>> as opposed to initializing platform overflow timer, which

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

2016-08-29 Thread osstest service owner
flight 100653 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100653/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 37136e069dc0a0b8a47098d80d8f6f742db88c04 baseline version: ovmf

[Xen-devel] [PATCH] Fix a BUG_ON issue

2016-08-29 Thread Feng Wu
The 'idx' can equal to the max number of vCPUs, fix it. Signed-off-by: Feng Wu --- xen/drivers/passthrough/io.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/drivers/passthrough/io.c b/xen/drivers/passthrough/io.c index 9e6b46c..66577b6 100644 ---

Re: [Xen-devel] [PATCH v3 4/6] x86/time: refactor read_platform_stime()

2016-08-29 Thread Jan Beulich
>>> On 26.08.16 at 17:13, wrote: > On 08/25/2016 11:17 AM, Jan Beulich wrote: > On 24.08.16 at 14:43, wrote: >>> In the case of clocksource=tsc we will >>> use it to set tsc_timestamp. >> >> I don't see how this relates to the patch

Re: [Xen-devel] [PATCH v3 2/6] x86/time: implement tsc as clocksource

2016-08-29 Thread Jan Beulich
>>> On 26.08.16 at 17:11, wrote: > On 08/25/2016 11:06 AM, Jan Beulich wrote: > On 24.08.16 at 14:43, wrote: >>> This patch introduces support for using TSC as platform time source >>> which is the highest resolution time and most

  1   2   >