Re: [Xen-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 04:02, wrote: > @@ -491,8 +474,9 @@ static int xen_pt_bar_reg_write(XenPCIPassthroughState > *s, XenPTReg *cfg_entry, > bar_ro_mask = XEN_PT_BAR_IO_RO_MASK | (r_size - 1); > break; > case XEN_PT_BAR_FLAG_UPPER: > +r = &d->io_regions[index-1]; Per

Re: [Xen-devel] [Qemu-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Wu, Feng
> -Original Message- > From: qemu-devel-bounces+feng.wu=intel@nongnu.org > [mailto:qemu-devel-bounces+feng.wu=intel@nongnu.org] On Behalf Of > Jan Beulich > Sent: Wednesday, August 12, 2015 2:59 PM > To: Wu, Feng > Cc: xen-de...@lists.xensource.com; qemu-de...@nongnu.org; > stefan

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 15:18, wrote: > On 05/08/15 13:46, Andrew Cooper wrote: >> On 05/08/15 13:36, Julien Grall wrote: >>> So we need to introduce the concept of in each definition. This patch >>> makes clear that MFN and GFN is always 4KB and PFN may vary. >> >> Is (or rather will) a 4K dom0 able

Re: [Xen-devel] [RFC 2/2] xen/public: grant-table: Specificy the size of the grant

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 13:28, wrote: > --- a/xen/include/public/grant_table.h > +++ b/xen/include/public/grant_table.h > @@ -51,6 +51,11 @@ > * know the real machine address of a page it is sharing. This makes > * it possible to share memory correctly with domains running in > * fully virtualise

Re: [Xen-devel] [xen 4.6 retrospective] [urgent] rename "freeze" window and make release branch as soon as possible after RC1

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 11:22, wrote: > Release Process improvements: > * Reopen the tree development tree as soon as possible after RC1 (I will let > you guys figure out the best RC - let's call it RCx) > * In other words, create the release branch at RCx Assuming this is what the majority wants (I

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Manish Jaggi
Below are the comments. I will also send a Draft 4 taking account of the comments. On Wednesday 12 August 2015 02:04 AM, Konrad Rzeszutek Wilk wrote: On Tue, Aug 04, 2015 at 05:57:24PM +0530, Manish Jaggi wrote: - | PCI Pass-through in Xe

Re: [Xen-devel] [xen 4.6 retrospective] [urgent] rename "freeze" window and make release branch as soon as possible after RC1

2015-08-12 Thread Jan Beulich
>>> On 06.08.15 at 12:52, wrote: > I would even go as far as recommending maintainers to take patches in > their personal trees while xen-unstable is "frozen". Personally I > already do that for QEMU. For small patches that might work. For large series perhaps repeatedly needing non-trivial re-ba

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Shannon Zhao
Hi, I'm working on re-spinning this patchset while encountering a werid problem about xzalloc_bytes. Since I need to copy some ACPI tables, I need to allocate some memory for it. So there are a few places calling xzalloc_bytes. And it fails at the fifth one. The log is shown as following: (XEN)

[Xen-devel] [xen 4.6 retrospective] [bad] review load near freeze point

2015-08-12 Thread Jan Beulich
>>> On 04.08.15 at 14:52, wrote: = Issue / Observation = Maybe my memory regarding the 4.5 release has faded, but I'm having the impression that 4.6 was quite a bit worse. This was at parts due to the sheer number of patch series and their sizes, but also because frequently they took quite a b

Re: [Xen-devel] Hotpatch construction and __LINE__

2015-08-12 Thread Jan Beulich
>>> On 05.08.15 at 15:25, wrote: > Jan had a plan to make Xen read its own DWARF symbol table rather than > using the current cludge we have where we partially link Xen, extract > the public symbol table, rewrite symbol-offsets.c and relink it onto the > end. Either I mis-expressed this, or you m

Re: [Xen-devel] [PATCH v4 26/31] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-08-12 Thread Jan Beulich
>>> On 07.08.15 at 21:03, wrote: > On 07/08/15 11:18, Roger Pau Monne wrote: >> @@ -1135,6 +1136,161 @@ int arch_set_info_guest( >> #undef c >> } >> >> +/* Called by VCPUOP_initialise for HVM guests. */ >> +static int arch_set_info_hvm_guest(struct vcpu *v, vcpu_hvm_context_t *ctx) >> +{ >> +

Re: [Xen-devel] [PATCH v10 06/11] xen: Introduce XEN_DOMCTL_soft_reset

2015-08-12 Thread Jan Beulich
>>> On 11.08.15 at 21:57, wrote: > On Tue, Jul 28, 2015 at 03:28:11PM +0200, Vitaly Kuznetsov wrote: >> @@ -1038,6 +1045,32 @@ void domain_unpause_except_self(struct domain *d) >> domain_unpause(d); >> } >> >> +int domain_soft_reset(struct domain *d) >> +{ >> +struct vcpu *v; >> +

Re: [Xen-devel] [Xen-users] "xl restore" leaks a file descriptor?

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 18:07 +0100, Wei Liu wrote: > On Tue, Aug 11, 2015 at 04:48:13PM +0100, Ian Campbell wrote: > > On Tue, 2015-08-11 at 11:13 -0400, Andrew Armenia wrote: > > > It's the checkpoint file - i.e. the command line argument to xl > > > restore - that is being leaked. > > > > Thanks.

Re: [Xen-devel] [Qemu-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 09:10, wrote: > >> -Original Message- >> From: qemu-devel-bounces+feng.wu=intel@nongnu.org >> [mailto:qemu-devel-bounces+feng.wu=intel@nongnu.org] On Behalf Of >> Jan Beulich >> Sent: Wednesday, August 12, 2015 2:59 PM >> To: Wu, Feng >> Cc: xen-de...@lists.xe

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 17:01 +0100, Julien Grall wrote: > On 11/08/15 16:19, Ian Campbell wrote: > > > IIRC we talked about it few months ago and you said that using > > > balloon > > > page will split in 4K the 1G/2M mapping done in the stage-2 p2m. > > > > Did I? Odd because I'm also of the opin

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 10:42 +0800, Shannon Zhao wrote: > > On 2015/8/12 0:01, Julien Grall wrote: > > On 11/08/15 16:19, Ian Campbell wrote: > > > > IIRC we talked about it few months ago and you said that using > > > > balloon > > > > page will split in 4K the 1G/2M mapping done in the stage-2 p

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 10:47 +0800, Shannon Zhao wrote: > > On 2015/8/11 23:52, Boris Ostrovsky wrote: > > On 08/11/2015 11:35 AM, Ian Campbell wrote: > > > On Tue, 2015-08-11 at 11:29 -0400, Boris Ostrovsky wrote: > > > > On 08/11/2015 10:21 AM, Ian Campbell wrote: > > > > > On Tue, 2015-08-11 at

Re: [Xen-devel] [PATCH 1/2] ACPI/table: Always count matched and successfully parsed entries

2015-08-12 Thread Jan Beulich
>>> On 04.08.15 at 06:19, wrote: > From: Shannon Zhao This is not true. All you did is port a Linux change. > acpi_parse_entries() allows to traverse all available table entries (aka > subtables) by passing max_entries parameter equal to 0, but since its count > variable is only incremented if

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 11:04 +0800, Shannon Zhao wrote: > Hi Julien, > > On 2015/8/12 0:19, Julien Grall wrote: > > Hi Shannon, > > > > On 07/08/15 03:11, Shannon Zhao wrote: > > > 2. Create minimal DT to pass required information to Dom0 > > > -

Re: [Xen-devel] [Qemu-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Wednesday, August 12, 2015 4:43 PM > To: Wu, Feng > Cc: stefano.stabell...@eu.citrix.com; xen-de...@lists.xensource.com; > qemu-de...@nongnu.org > Subject: RE: [Qemu-devel] [Xen-devel] [PATCH v1] xenpt: Properly h

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 16:34 -0400, Konrad Rzeszutek Wilk wrote: > > > 2.2PHYSDEVOP_pci_host_bridge_add hypercall > > -- > > Xen code accesses PCI configuration space based on the sbdf received from > > the > > guest. The order in which the pci device

Re: [Xen-devel] [PATCH 2/2] ACPI/table: Make acpi_table_init return meaningful value

2015-08-12 Thread Jan Beulich
>>> On 04.08.15 at 06:19, wrote: > From: Shannon Zhao > > If ACPI fails to initialize tables, it should return one meaningful > value to tell the caller to diable acpi. > > Signed-off-by: Shannon Zhao > --- > xen/drivers/acpi/tables.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(

Re: [Xen-devel] [PATCH 1/2] ACPI/table: Always count matched and successfully parsed entries

2015-08-12 Thread Shannon Zhao
On 2015/8/12 16:51, Jan Beulich wrote: On 04.08.15 at 06:19, wrote: >> From: Shannon Zhao > > This is not true. All you did is port a Linux change. > Will change this. >> acpi_parse_entries() allows to traverse all available table entries (aka >> subtables) by passing max_entries parame

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Shannon Zhao
On 2015/8/12 16:47, Ian Campbell wrote: > On Wed, 2015-08-12 at 10:47 +0800, Shannon Zhao wrote: >> >> On 2015/8/11 23:52, Boris Ostrovsky wrote: >>> On 08/11/2015 11:35 AM, Ian Campbell wrote: On Tue, 2015-08-11 at 11:29 -0400, Boris Ostrovsky wrote: > On 08/11/2015 10:21 AM, Ian Campbe

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Julien Grall
On 12/08/2015 09:46, Ian Campbell wrote: On Tue, 2015-08-11 at 17:01 +0100, Julien Grall wrote: On 11/08/15 16:19, Ian Campbell wrote: IIRC we talked about it few months ago and you said that using balloon page will split in 4K the 1G/2M mapping done in the stage-2 p2m. Did I? Odd because I

[Xen-devel] Failure to boot HVM guest with more than 32 VCPUS

2015-08-12 Thread Hao, Xudong
Hi, In X86_64 platform, we noticed an issue that Xen boot a RHEL6u6 or Fedora22 guest, when configure the VCPU more than 32, the guest will fail to boot up. Guest config: memory = 2173 vcpus=33 vif = [ 'type=ioemu, mac=00:16:3e:7c:87:72, brid

Re: [Xen-devel] [PATCH v2 1/6] gitignore: Ignore *.orig, *.rej and *.swp files

2015-08-12 Thread Ian Campbell
(Having written the below I see too late that this is a grub patch not a Xen one, a tag in the subject for such cross posted patches would be useful please. Anyway, my opinion counts for very little in this context but I leave it below since already I wrote it. I notice that xen.git#.gitignore _do

[Xen-devel] [PATCH] gitignore: Don't ignore *.rej

2015-08-12 Thread Ian Campbell
These indicate a patch application went wrong, I want to see them in "git status". This appears to have been imported from .hgignore where it has been since 2005. Signed-off-by: Ian Campbell Cc: Daniel Kiper --- Spotted while accidentally reviewing Daniel's patch against grub's .gitignore. ---

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Julien Grall
On 12/08/2015 08:22, Shannon Zhao wrote: Hi, Hi Shannon, It's not part of the design discussion and we are avoiding to mix discussion. Can you please create another thread (or at least renaming the subject)? I'm working on re-spinning this patchset while encountering a werid problem about

Re: [Xen-devel] [PATCH for-4.6] xen/mm: populate_physmap: validate correctly the gfn for direct mapped domain

2015-08-12 Thread Jan Beulich
>>> On 11.08.15 at 19:41, wrote: > --- a/xen/common/memory.c > +++ b/xen/common/memory.c > @@ -126,22 +126,28 @@ static void populate_physmap(struct memop_args *a) > if ( is_domain_direct_mapped(d) ) > { > mfn = gpfn; > -if ( !mfn_valid(mf

[Xen-devel] [PATCH] gitignore: Don't ignore *.rej

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 11:07, wrote: > These indicate a patch application went wrong, I want to see them in > "git status". This appears to have been imported from .hgignore where > it has been since 2005. > > Signed-off-by: Ian Campbell Acked-by: Jan Beulich

Re: [Xen-devel] [PATCH for-4.6] xen/mm: populate_physmap: validate correctly the gfn for direct mapped domain

2015-08-12 Thread Wei Liu
On Tue, Aug 11, 2015 at 06:41:28PM +0100, Julien Grall wrote: > Direct mapped domain has already the memory allocated 1:1, so we are > directly using the gfn as mfn to map the RAM in the guest. > > While we are validating that the page associated to the first mfn belongs to > the domain, the subse

Re: [Xen-devel] [PATCH v10 06/11] xen: Introduce XEN_DOMCTL_soft_reset

2015-08-12 Thread Vitaly Kuznetsov
"Jan Beulich" writes: On 11.08.15 at 21:57, wrote: >> On Tue, Jul 28, 2015 at 03:28:11PM +0200, Vitaly Kuznetsov wrote: >>> @@ -1038,6 +1045,32 @@ void domain_unpause_except_self(struct domain *d) >>> domain_unpause(d); >>> } >>> >>> +int domain_soft_reset(struct domain *d) >>>

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Julien Grall
Hi, (Cc Andrew Turner who worked on the ACPI port for FreeBSD ARM64) On 12/08/2015 09:52, Ian Campbell wrote: On Wed, 2015-08-12 at 11:04 +0800, Shannon Zhao wrote: Hi Julien, On 2015/8/12 0:19, Julien Grall wrote: Hi Shannon, On 07/08/15 03:11, Shannon Zhao wrote: 2. Create minimal DT to

Re: [Xen-devel] [PATCH for-4.6] xen/mm: populate_physmap: validate correctly the gfn for direct mapped domain

2015-08-12 Thread Julien Grall
Hi Jan, On 12/08/2015 10:16, Jan Beulich wrote: On 11.08.15 at 19:41, wrote: --- a/xen/common/memory.c +++ b/xen/common/memory.c @@ -126,22 +126,28 @@ static void populate_physmap(struct memop_args *a) if ( is_domain_direct_mapped(d) ) { mfn = gpfn

Re: [Xen-devel] [PATCH for-4.6] xen/mm: populate_physmap: validate correctly the gfn for direct mapped domain

2015-08-12 Thread Wei Liu
On Wed, Aug 12, 2015 at 10:19:04AM +0100, Wei Liu wrote: > On Tue, Aug 11, 2015 at 06:41:28PM +0100, Julien Grall wrote: > > Direct mapped domain has already the memory allocated 1:1, so we are > > directly using the gfn as mfn to map the RAM in the guest. > > > > While we are validating that the

Re: [Xen-devel] [Xen-users] "xl restore" leaks a file descriptor?

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 09:41 +0100, Ian Campbell wrote: > On Tue, 2015-08-11 at 18:07 +0100, Wei Liu wrote: > > > > I don't see how CLOEXEC would be of any use if xl doesn't actually exec > > anything. > > Duh, for some reason I thought daemonize would activate the CLOEXEC, but > it's just fork wi

Re: [Xen-devel] [PATCH V4 2/2] x86, amd_ucode: Skip microcode updates for final levels

2015-08-12 Thread Jan Beulich
>>> On 11.08.15 at 21:11, wrote: > Some of older[Fam10h] systems require that certain number of > applied microcode patch levels should not be overwritten by > the microcode loader. Otherwise, system hangs are known to occur. > > The 'final_levels' of patch ids have been obtained empirically. > R

Re: [Xen-devel] Failure to boot HVM guest with more than 32 VCPUS

2015-08-12 Thread Vitaly Kuznetsov
"Hao, Xudong" writes: > Hi, > > In X86_64 platform, we noticed an issue that Xen boot a RHEL6u6 or > Fedora22 guest, when configure the VCPU more than 32, the guest will > fail to boot up. The issue is well-known for RHEL6.6 (and is fixed in 6.7 and in 6.6.z) but Fedora22 should boot. The log b

Re: [Xen-devel] [PATCH v10 06/11] xen: Introduce XEN_DOMCTL_soft_reset

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 11:19, wrote: > "Jan Beulich" writes: > > On 11.08.15 at 21:57, wrote: >>> On Tue, Jul 28, 2015 at 03:28:11PM +0200, Vitaly Kuznetsov wrote: @@ -1038,6 +1045,32 @@ void domain_unpause_except_self(struct domain *d) domain_unpause(d); }

Re: [Xen-devel] [Xen-users] "xl restore" leaks a file descriptor?

2015-08-12 Thread Wei Liu
On Wed, Aug 12, 2015 at 09:41:13AM +0100, Ian Campbell wrote: > On Tue, 2015-08-11 at 18:07 +0100, Wei Liu wrote: > > On Tue, Aug 11, 2015 at 04:48:13PM +0100, Ian Campbell wrote: > > > On Tue, 2015-08-11 at 11:13 -0400, Andrew Armenia wrote: > > > > It's the checkpoint file - i.e. the command line

Re: [Xen-devel] [RFC 0/4] HVM x86 enhancements to run Xen deprivileged mode operations

2015-08-12 Thread Jan Beulich
>>> On 06.08.15 at 18:45, wrote: > Performance testing > --- > Performance testing indicates that the overhead for this deprivileged mode is > approximately 25%. This overhead is the cost of moving into deprivileged mode > and then fully back out of deprivileged mode. > > I perfor

Re: [Xen-devel] Hotpatch construction and __LINE__

2015-08-12 Thread Andrew Cooper
On 12/08/15 09:09, Jan Beulich wrote: On 05.08.15 at 15:25, wrote: >> Jan had a plan to make Xen read its own DWARF symbol table rather than >> using the current cludge we have where we partially link Xen, extract >> the public symbol table, rewrite symbol-offsets.c and relink it onto the >>

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Julien Grall
Hi Jan, On 12/08/2015 08:16, Jan Beulich wrote: On 05.08.15 at 15:18, wrote: On 05/08/15 13:46, Andrew Cooper wrote: On 05/08/15 13:36, Julien Grall wrote: So we need to introduce the concept of in each definition. This patch makes clear that MFN and GFN is always 4KB and PFN may vary. Is

Re: [Xen-devel] [RFC 2/2] xen/public: grant-table: Specificy the size of the grant

2015-08-12 Thread Julien Grall
On 12/08/2015 08:21, Jan Beulich wrote: This paragraph should get "on ARM" added somewhere. There is no inherent requirement for such a restriction on another architecture Xen might get ported to; the only restriction of the current interface is that it can't support page sizes from 64k up. On

Re: [Xen-devel] [Xen-users] "xl restore" leaks a file descriptor?

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 10:49 +0100, Wei Liu wrote: > On Wed, Aug 12, 2015 at 09:41:13AM +0100, Ian Campbell wrote: > > On Tue, 2015-08-11 at 18:07 +0100, Wei Liu wrote: > > > On Tue, Aug 11, 2015 at 04:48:13PM +0100, Ian Campbell wrote: > > > > On Tue, 2015-08-11 at 11:13 -0400, Andrew Armenia wrote

Re: [Xen-devel] Commit moratorium for 4.6rc1

2015-08-12 Thread Ian Jackson
Ian Jackson writes ("Commit moratorium for 4.6rc1"): > Please avoid committing anything just now. We need the push gate > clear for a patch to update the tools library sonames, which is needed > for rc1. staging is now open for committing again. But, note that Wei (with RM hat on) has asked that

Re: [Xen-devel] [PATCH OSSTEST] Toolstack/libvirt: use URI in migration command

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 16:07 -0600, Jim Fehlig wrote: > Wei Liu wrote: > > Virsh migrate expects an URI, not a host. We don't actually care what > > kind of transport it uses, the main objective is to test migration, so > > use xen+ssh for the time being. > > > > Signed-off-by: Wei Liu > > Cc: Ian

Re: [Xen-devel] [PATCH for-4.6] tools/libxc: linux: Don't use getpagesize() when unmapping the grants

2015-08-12 Thread Julien Grall
Hi Ian, On 11/08/2015 13:48, Ian Campbell wrote: On Fri, 2015-08-07 at 22:45 +0100, Wei Liu wrote: On Fri, Aug 07, 2015 at 07:53:55PM +0100, Julien Grall wrote: The grants are based on the Xen granularity (i.e 4KB). While the function to map grants for Linux (linux_gnttab_grant_map) is using t

Re: [Xen-devel] [RFC 3/4] HVM x86 deprivileged mode: Code for switching into/out of deprivileged mode

2015-08-12 Thread Jan Beulich
>>> On 11.08.15 at 19:19, wrote: > KVM unilaterally reloads the host task register on vmexit, and I suspect > this is probably the way to go, but have not had time to investigate > whether there is any performance impact from doing so. Given how little > of a TSS is actually used in long mode, I

Re: [Xen-devel] [PATCH OSSTEST] Toolstack/libvirt: use URI in migration command

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 11:05 +0100, Ian Campbell wrote: > On Tue, 2015-08-11 at 16:07 -0600, Jim Fehlig wrote: > > Wei Liu wrote: > > > Virsh migrate expects an URI, not a host. We don't actually care what > > > kind of transport it uses, the main objective is to test migration, > > > so > > > use

Re: [Xen-devel] [PATCH RFC v2 0/5] Multi-queue support for xen-blkfront and xen-blkback

2015-08-12 Thread Bob Liu
On 08/12/2015 01:32 AM, Jens Axboe wrote: > On 08/11/2015 03:45 AM, Rafal Mielniczuk wrote: >> On 11/08/15 07:08, Bob Liu wrote: >>> On 08/10/2015 11:52 PM, Jens Axboe wrote: On 08/10/2015 05:03 AM, Rafal Mielniczuk wrote: ... > Hello, > > We rerun the tests for sequential reads w

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Stefano Stabellini
On Wed, 12 Aug 2015, Ian Campbell wrote: > On Wed, 2015-08-12 at 11:04 +0800, Shannon Zhao wrote: > > Hi Julien, > > > > On 2015/8/12 0:19, Julien Grall wrote: > > > Hi Shannon, > > > > > > On 07/08/15 03:11, Shannon Zhao wrote: > > > > 2. Create minimal DT to pass required information to Dom0 >

[Xen-devel] ANNOUNCEMENT: Xen 4.6 RC1

2015-08-12 Thread Wei Liu
Hi all Xen 4.6 RC1 has been tagged. You can check out the tag 4.6.0-rc1 in xen.git or download tarball from: http://bits.xensource.com/oss-xen/release/4.6.0-rc1/xen-4.6.0-rc1.tar.gz Signature for tarball: http://bits.xensource.com/oss-xen/release/4.6.0-rc1/xen-4.6.0-rc1.tar.gz.sig When reportin

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 11:57, wrote: > On 12/08/2015 08:16, Jan Beulich wrote: > On 05.08.15 at 15:18, wrote: >>> On 05/08/15 13:46, Andrew Cooper wrote: On 05/08/15 13:36, Julien Grall wrote: > So we need to introduce the concept of in each definition. This patch > makes clear that

Re: [Xen-devel] [RFC 2/2] xen/public: grant-table: Specificy the size of the grant

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 12:00, wrote: > > On 12/08/2015 08:21, Jan Beulich wrote: >> This paragraph should get "on ARM" added somewhere. There is no >> inherent requirement for such a restriction on another architecture >> Xen might get ported to; the only restriction of the current interface >> is t

[Xen-devel] [distros-debian-wheezy test] 37821: tolerable trouble: broken/pass

2015-08-12 Thread Platform Team regression test user
flight 37821 distros-debian-wheezy real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/37821/ Failures :-/ but no regressions. Tests which did not succeed, including tests which could not be run: build-armhf 3 host-install(3) broken never pass build-a

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Stefano Stabellini
On Wed, 12 Aug 2015, Andrew Turner wrote: > On Wed, 12 Aug 2015 10:21:55 +0100 > Julien Grall wrote: > > > Hi, > > > > (Cc Andrew Turner who worked on the ACPI port for FreeBSD ARM64) > > > > On 12/08/2015 09:52, Ian Campbell wrote: > > > On Wed, 2015-08-12 at 11:04 +0800, Shannon Zhao wrote: >

[Xen-devel] [BUG][4.5.1]xl cpupool-create segfault (with config file parameter)

2015-08-12 Thread Benjamin Beier
Since xen version 4.5.1 there is a bug with the xl cpupool-create command. It always throws a segmentation fault when used with a config file as parameter. The content of the config file does not really matter (valid or invalid). It also fails with the example config file in /etc/xen/cpupool or

Re: [Xen-devel] [RFC 2/2] xen/public: grant-table: Specificy the size of the grant

2015-08-12 Thread Julien Grall
On 12/08/15 11:35, Jan Beulich wrote: On 12.08.15 at 12:00, wrote: > >> >> On 12/08/2015 08:21, Jan Beulich wrote: >>> This paragraph should get "on ARM" added somewhere. There is no >>> inherent requirement for such a restriction on another architecture >>> Xen might get ported to; the only

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Julien Grall
On 12/08/15 11:33, Jan Beulich wrote: On 12.08.15 at 11:57, wrote: >> On 12/08/2015 08:16, Jan Beulich wrote: >> On 05.08.15 at 15:18, wrote: On 05/08/15 13:46, Andrew Cooper wrote: > On 05/08/15 13:36, Julien Grall wrote: >> So we need to introduce the concept of in each de

Re: [Xen-devel] [PATCH V3 3/6] x86/xsaves: enable xsaves/xrstors for hvm guest

2015-08-12 Thread Shuai Ruan
On Tue, Aug 11, 2015 at 10:37:56AM +0100, Andrew Cooper wrote: > On 11/08/15 08:59, Shuai Ruan wrote: > > On Fri, Aug 07, 2015 at 02:04:51PM +0100, Andrew Cooper wrote: > >> On 07/08/15 09:22, Shuai Ruan wrote: > > void hvm_cpuid(unsigned int input, unsigned int *eax, unsigned int > > *eb

Re: [Xen-devel] [PATCH V3 5/6] x86/xsaves: support compact format for hvm save/restore

2015-08-12 Thread Shuai Ruan
On Tue, Aug 11, 2015 at 10:27:20AM +0100, Andrew Cooper wrote: > On 11/08/15 09:01, Shuai Ruan wrote: > > > >>> + > >>> +/* > >>> + * The FP xstates and SSE xstates are legacy states. They are always > >>> + * in the fixed offsets in the xsave area in either compacted form > >>> + *

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 11:48 +0100, Stefano Stabellini wrote: > On Wed, 12 Aug 2015, Andrew Turner wrote: > > On Wed, 12 Aug 2015 10:21:55 +0100 > > Julien Grall wrote: > > > > > Hi, > > > > > > (Cc Andrew Turner who worked on the ACPI port for FreeBSD ARM64) > > > > > > On 12/08/2015 09:52, Ian

Re: [Xen-devel] [RFC 0/4] HVM x86 enhancements to run Xen deprivileged mode operations

2015-08-12 Thread Ben Catterall
On 12/08/15 10:50, Jan Beulich wrote: On 06.08.15 at 18:45, wrote: Performance testing --- Performance testing indicates that the overhead for this deprivileged mode is approximately 25%. This overhead is the cost of moving into deprivileged mode and then fully back out of depr

Re: [Xen-devel] [BUG][4.5.1]xl cpupool-create segfault (with config file parameter)

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 13:03 +0200, Benjamin Beier wrote: > Since xen version 4.5.1 there is a bug with the xl cpupool-create > command. > It always throws a segmentation fault when used with a config file as > parameter. > The content of the config file does not really matter (valid or invalid).

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 13:13, wrote: > On 12/08/15 11:33, Jan Beulich wrote: > On 12.08.15 at 11:57, wrote: >>> On 12/08/2015 08:16, Jan Beulich wrote: >>> On 05.08.15 at 15:18, wrote: > When the backend is 64K, it will map the foreign 4K at the top of a 64K > page. It's a waste of m

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Julien Grall
On 12/08/15 12:23, Ian Campbell wrote: > Strictly it is considered a separate thing, much like loader.efi, despite > where it lives e.g. it is self contained and not allowed to call into the > kernel proper except via the formal interface provided for the hand-off. > > That might seem like semanti

Re: [Xen-devel] Design doc of adding ACPI support for arm64 on Xen - version 2

2015-08-12 Thread Julien Grall
On 12/08/15 11:36, Andrew Turner wrote: > Would it be possible to add a stdout property and node for the hvc0 > device? It would help FreeBSD as we use this to find the kernel > console. We check for the stdout-path, linux,stdout-path, stdout, > stdin-path, and stdin properties, in that order, with

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Julien Grall
On 12/08/15 12:58, Jan Beulich wrote: On 12.08.15 at 13:13, wrote: >> On 12/08/15 11:33, Jan Beulich wrote: >> On 12.08.15 at 11:57, wrote: On 12/08/2015 08:16, Jan Beulich wrote: On 05.08.15 at 15:18, wrote: >> When the backend is 64K, it will map the foreign 4K at th

Re: [Xen-devel] Enormous size of libvirt libxl-driver.log with Xen 4.2 and 4.3

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 11:19 +0100, Ian Campbell wrote: > On Mon, 2015-08-03 at 11:47 +0100, Ian Campbell wrote: > > After the initial expected logging the file is simply full of: > > > > 2015-08-02 19:12:12 UTC libxl: debug: > > libxl.c:1004:domain_death_xswatch_callback: [evg=0x7f3cc44fa3f0:3] f

Re: [Xen-devel] [PATCH OSSTEST] Toolstack/libvirt: use URI in migration command

2015-08-12 Thread Ian Campbell
On Tue, 2015-08-11 at 16:07 -0600, Jim Fehlig wrote: > Wei Liu wrote: > > Virsh migrate expects an URI, not a host. We don't actually care what > > kind of transport it uses, the main objective is to test migration, so > > use xen+ssh for the time being. > > > > Signed-off-by: Wei Liu > > Cc: Ian

Re: [Xen-devel] [RFC 3/4] HVM x86 deprivileged mode: Code for switching into/out of deprivileged mode

2015-08-12 Thread Ben Catterall
On 11/08/15 18:05, Tim Deegan wrote: Hi, At 17:51 +0100 on 11 Aug (1439315508), Ben Catterall wrote: On 11/08/15 10:55, Tim Deegan wrote: At 11:14 +0100 on 10 Aug (1439205273), Andrew Cooper wrote: On 10/08/15 10:49, Tim Deegan wrote: Hi, At 17:45 +0100 on 06 Aug (1438883118), Ben Cattera

Re: [Xen-devel] [RFC 1/2] xen/mm: Clarify the granularity for each Frame Number

2015-08-12 Thread Jan Beulich
>>> On 12.08.15 at 14:57, wrote: > On 12/08/15 12:58, Jan Beulich wrote: > On 12.08.15 at 13:13, wrote: >>> On 12/08/15 11:33, Jan Beulich wrote: >>> On 12.08.15 at 11:57, wrote: > On 12/08/2015 08:16, Jan Beulich wrote: > On 05.08.15 at 15:18, wrote: >>> When the backen

Re: [Xen-devel] [RFC 3/4] HVM x86 deprivileged mode: Code for switching into/out of deprivileged mode

2015-08-12 Thread Tim Deegan
At 14:22 +0100 on 12 Aug (1439389325), Ben Catterall wrote: > On 11/08/15 18:05, Tim Deegan wrote: > > BTW, I think there need to be three stacks anyway, since the depriv > > code shouldn't be allowed to write to the priv code's stack frames. > > Or maybe I've misunderstood how much access the depr

Re: [Xen-devel] linux-next: manual merge of the xen-tip tree with the tip tree

2015-08-12 Thread Boris Ostrovsky
On 08/12/2015 01:09 AM, Stephen Rothwell wrote: Hi all, Today's linux-next merge of the xen-tip tree got a conflict in: arch/x86/xen/enlighten.c between commit: 9261e050b686 ("x86/asm/tsc, x86/paravirt: Remove read_tsc() and read_tscp() paravirt hooks") from the tip tree and commit:

Re: [Xen-devel] [RFC 3/4] HVM x86 deprivileged mode: Code for switching into/out of deprivileged mode

2015-08-12 Thread Andrew Cooper
On 11/08/15 19:29, Boris Ostrovsky wrote: > On 08/11/2015 01:19 PM, Andrew Cooper wrote: >> On 11/08/15 18:05, Tim Deegan wrote: >> * Under this model, PV exception handlers should copy themselves >> onto >> the privileged execution stack. >> * Currently, the IST handlers copy them

Re: [Xen-devel] [RFC 3/4] HVM x86 deprivileged mode: Code for switching into/out of deprivileged mode

2015-08-12 Thread Andrew Cooper
On 12/08/15 14:29, Andrew Cooper wrote: > On 11/08/15 19:29, Boris Ostrovsky wrote: >> On 08/11/2015 01:19 PM, Andrew Cooper wrote: >>> On 11/08/15 18:05, Tim Deegan wrote: >>> * Under this model, PV exception handlers should copy themselves >>> onto >>> the privileged execution stack.

Re: [Xen-devel] xen/pcifront: Use monotonic clock

2015-08-12 Thread Abhilash Jindal
Oops. Thank you. On Sat, Aug 8, 2015 at 10:16 PM, Boris Ostrovsky wrote: > On 08/08/2015 09:36 PM, Abhilash Jindal wrote: > >> Wall time obtained from do_gettimeofday is susceptible to sudden jumps >> due to user setting the time or due to NTP. >> >> Monotonic time is constantly increasing time

[Xen-devel] [PATCH 2/2] tools: libxl: fixup long line introduced by previous semantic patch

2015-08-12 Thread Ian Campbell
Signed-off-by: Ian Campbell --- tools/libxl/libxl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 865e162..4f2eb24 100644 --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -4866,8 +4866,8 @@ static int libxl__get_memor

[Xen-devel] [PATCH 1/2] tools: libxl: Remove unnecessary trailing \n from log messages.

2015-08-12 Thread Ian Campbell
Both xl's LOG and the various libxl logging mechanisms automatically include a trailing \n. Remove all unnecessary \n's from the logs messages with the following semantic patch. spatch also reindents (I couldn't see how to make it stop). In general it has improved matters but in 1 case it has int

Re: [Xen-devel] [PATCH v2 2/2] cambridge: arrange to test each new baseline

2015-08-12 Thread Ian Campbell
On Mon, 2015-07-27 at 11:55 +0100, Ian Campbell wrote: > Provide a new cr-daily-branch setting OSSTEST_BASELINES_ONLY which > causes it to only attempt to test the current baseline (if it is > untested) There is a problem with this "if it is untested". This is implemented below by setting skipiden

Re: [Xen-devel] Second regression due to libxl: Remove linux udev rules (2ba368d13893402b2f1fb3c283ddcc714659dd9b)

2015-08-12 Thread Konrad Rzeszutek Wilk
On Fri, Aug 07, 2015 at 04:58:57PM +0200, Roger Pau Monné wrote: > El 07/08/15 a les 16.54, Konrad Rzeszutek Wilk ha escrit: > > Ok. I hadn't run your patch yet. Do you want me to run the latest staging > > instead once more with my test-case? > > Yes please, 40s in my test case seemed to be fine.

[Xen-devel] [PATCH v2 0/2] x86/HVM: hvm_map_guest_frame_rw() adjustments

2015-08-12 Thread Jan Beulich
1: honor p2m_ram_ro in hvm_map_guest_frame_rw() 2: correct page dirty marking in hvm_map_guest_frame_rw() Signed-off-by: Jan Beulich --- v2: Re-base patch one on top of altp2m series. New patch 2. ___ Xen-devel mailing list Xen-devel@lists.xen.org htt

[Xen-devel] [PATCH v2 1/2] x86/HVM: hvm_map_guest_frame_rw() adjustments

2015-08-12 Thread Jan Beulich
... and its callers. While all non-nested users are made fully honor the semantics of that type, doing so in the nested case seemed insane (if doable at all, considering VMCS shadowing), and hence there the respective operations are simply made fail. One case not (yet) taken care of is that of a

[Xen-devel] [PATCH v2 2/2] x86/HVM: correct page dirty marking in hvm_map_guest_frame_rw()

2015-08-12 Thread Jan Beulich
Rather than dirtying a page when establishing a (permanent) mapping, dirty it when the page gets unmapped, or - if still mapped - on the final iteration of a save operation. (Transient mappings continue to get dirtied upon getting mapped, to avoid the overhead of tracking.) Signed-off-by: Jan Beul

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Konrad Rzeszutek Wilk
On Wed, Aug 12, 2015 at 01:03:07PM +0530, Manish Jaggi wrote: > Below are the comments. I will also send a Draft 4 taking account of the > comments. > > > On Wednesday 12 August 2015 02:04 AM, Konrad Rzeszutek Wilk wrote: > >On Tue, Aug 04, 2015 at 05:57:24PM +0530, Manish Jaggi wrote: > >>

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Konrad Rzeszutek Wilk
On Wed, Aug 12, 2015 at 09:56:41AM +0100, Ian Campbell wrote: > On Tue, 2015-08-11 at 16:34 -0400, Konrad Rzeszutek Wilk wrote: > > > > > 2.2PHYSDEVOP_pci_host_bridge_add hypercall > > > -- > > > Xen code accesses PCI configuration space based on the

Re: [Xen-devel] [Qemu-devel] [PATCH v1] xenpt: Properly handle 64-bit bar with more than 4G size

2015-08-12 Thread Konrad Rzeszutek Wilk
On Wed, Aug 12, 2015 at 08:53:44AM +, Wu, Feng wrote: > > > > -Original Message- > > From: Jan Beulich [mailto:jbeul...@suse.com] > > Sent: Wednesday, August 12, 2015 4:43 PM > > To: Wu, Feng > > Cc: stefano.stabell...@eu.citrix.com; xen-de...@lists.xensource.com; > > qemu-de...@nongn

Re: [Xen-devel] [PATCH] gitignore: Don't ignore *.rej

2015-08-12 Thread Konrad Rzeszutek Wilk
On Wed, Aug 12, 2015 at 10:07:37AM +0100, Ian Campbell wrote: > These indicate a patch application went wrong, I want to see them in > "git status". This appears to have been imported from .hgignore where > it has been since 2005. > > Signed-off-by: Ian Campbell > Cc: Daniel Kiper Reviewed-by:

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 10:25 -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 12, 2015 at 09:56:41AM +0100, Ian Campbell wrote: > > On Tue, 2015-08-11 at 16:34 -0400, Konrad Rzeszutek Wilk wrote: > > > > > > > 2.2PHYSDEVOP_pci_host_bridge_add hypercall > > > > -

[Xen-devel] [PATCH 0/2] support compressed kernels on ARM64

2015-08-12 Thread Stefano Stabellini
Hi all, this patch series introduces support for compressed kernels, such as the standard Image.gz format used by Linux on arm64 by default, in Xen. Without it, Xen cannot load the default kernel shipped by distros, such as CentOS 7. Stefano Stabellini (2): xen: move perform_gunzip to comm

[Xen-devel] [PATCH 1/2] xen: move perform_gunzip to common

2015-08-12 Thread Stefano Stabellini
The current gunzip code to uncompress the Dom0 kernel is implemented in inflate.c which is included by bzimage.c. I am looking to doing the same on ARM64 but there is quite a bit of boilerplate definitions that I would need to import in order for inflate.c to work correctly. Instead of copying/pa

[Xen-devel] [PATCH 2/2] xen/arm: support compressed kernels

2015-08-12 Thread Stefano Stabellini
Signed-off-by: Stefano Stabellini CC: julien.gr...@citrix.com CC: ian.campb...@citrix.com --- xen/arch/arm/kernel.c | 36 xen/common/Makefile |2 +- xen/include/asm-arm/byteorder.h |2 ++ 3 files changed, 39 insertions(+), 1 del

Re: [Xen-devel] PCI Passthrough Design - Draft 3

2015-08-12 Thread Konrad Rzeszutek Wilk
On Wed, Aug 12, 2015 at 03:42:10PM +0100, Ian Campbell wrote: > On Wed, 2015-08-12 at 10:25 -0400, Konrad Rzeszutek Wilk wrote: > > On Wed, Aug 12, 2015 at 09:56:41AM +0100, Ian Campbell wrote: > > > On Tue, 2015-08-11 at 16:34 -0400, Konrad Rzeszutek Wilk wrote: > > > > > > > > > 2.2PHYSDEVOP

Re: [Xen-devel] [PATCH v3 2/4] x86/compat: Test both PV and PVH guests for compat mode

2015-08-12 Thread Boris Ostrovsky
On 08/12/2015 02:23 AM, Jan Beulich wrote: On 11.08.15 at 19:21, wrote: On 08/11/2015 05:19 AM, Jan Beulich wrote: On 24.07.15 at 19:54, wrote: On 07/23/2015 10:07 AM, Jan Beulich wrote: Plus - is this in line with what the tools are doing? Aren't they assuming !PV <=> native format context

[Xen-devel] [PATCH] x86/p2m: clear_identity_p2m_entry() must cope with 'relaxed' RDM mode

2015-08-12 Thread Jan Beulich
Tearing down a 1:1 mapping that was never established isn't really nice (and in fact hits an ASSERT() in p2m_remove_page()). Convert from a wrapper macro to a proper function which then can take care of the situation. Also take the opportunity to remove the 'page_order' parameter of clear_identity

Re: [Xen-devel] [PATCH 2/2] xen/arm: support compressed kernels

2015-08-12 Thread Ian Campbell
On Wed, 2015-08-12 at 15:47 +0100, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini > CC: julien.gr...@citrix.com > CC: ian.campb...@citrix.com > --- > xen/arch/arm/kernel.c | 36 > > xen/common/Makefile |2 +- > xen/i

Re: [Xen-devel] [PATCH OSSTEST] Toolstack/libvirt: use URI in migration command

2015-08-12 Thread Jim Fehlig
Ian Campbell wrote: > On Tue, 2015-08-11 at 16:07 -0600, Jim Fehlig wrote: > >> Wei Liu wrote: >> >>> Virsh migrate expects an URI, not a host. We don't actually care what >>> kind of transport it uses, the main objective is to test migration, so >>> use xen+ssh for the time being. >>> >>>

Re: [Xen-devel] [PATCH 2/2] xen/arm: support compressed kernels

2015-08-12 Thread Julien Grall
Hi Stefano, I'm sure you don't support all kind of compressed kernels. Can you example in the commit message which one you are supporting? On 12/08/15 15:47, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini > CC: julien.gr...@citrix.com > CC: ian.campb...@citrix.com > --- > xen/arc

  1   2   3   >