Re: [Xen-devel] [PATCH v3 06/13] xen/arm: Introduce a generic way to describe device

2015-02-02 Thread Jan Beulich
>>> On 30.01.15 at 19:49, wrote: > Currently, Xen is supporting PCI and Platform device (based on Device Tree). > > While Xen only supports Platform device on ARM, Xen will gain support of > PCI soon. > > Some drivers, such as IOMMU drivers, may handle PCI and platform device in > the same way.

[Xen-devel] [PATCH v9 2/3] tools: code refactoring for MBM

2015-02-02 Thread Chao Peng
Make some internal routines common so that total/local memory bandwidth monitoring in the next patch can make use of them. Signed-off-by: Chao Peng Acked-by: Wei Liu --- tools/libxl/libxl_psr.c | 44 +++ tools/libxl/xl_cmdimpl.c | 54

[Xen-devel] [PATCH v9 3/3] tools, docs: add total/local memory bandwith monitoring

2015-02-02 Thread Chao Peng
Add Memory Bandwidth Monitoring(MBM) for VMs. Two types of monitoring are supported: total and local memory bandwidth monitoring. To use it, CMT should be enabled in hypervisor. Signed-off-by: Chao Peng --- Changes in v9: 1. Refactor code in xc_psr_cmt_get_data. 2. Move bandwidth calculation(slee

[Xen-devel] [PATCH v9 1/3] tools: correct coding style for psr

2015-02-02 Thread Chao Peng
- space: remove space after '(' or before ')' in 'if' condition; - indention: align function definition/call arguments; Signed-off-by: Chao Peng Acked-by: Wei Liu --- tools/libxc/include/xenctrl.h | 10 +- tools/libxc/xc_psr.c | 10 +- tools/libxl/libxl.h | 11

[Xen-devel] [PATCH v9 0/3] enable Memory Bandwidth Monitoring (MBM) for VMs

2015-02-02 Thread Chao Peng
Changes from v8: * Merge event mask patch to MBM enabling patch; * Address comments from Ian Campbell(Detail in patch itself). Changes from v7: * Make obfuscating more complex as Jan suggested. * Minor adjustment for commit message. Changes from v6: * Obfuscate the read value of MSR_IA32_TSC by a

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Jan Beulich
>>> On 30.01.15 at 20:19, wrote: > Soon after sending this, I came up with a 2nd way for fix. > Change hvm_has_dm() to use hvm_select_ioreq_server(). Then > the correct answer will be found, and so will not retry. > > To avoid 2 calls to hvm_select_ioreq_server(), it makes > snes to me to return

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-02-02 Thread Jan Beulich
>>> On 30.01.15 at 18:54, wrote: > - xen.efi build will not so strongly depend > on a given GCC and binutils version. While I can see the possibility of making the binutils version dependency go away (by manually creating the PE header), I can't see how you'd overcome the gcc one: The MS ca

Re: [Xen-devel] PCI-Passthrough Error: transmit queue timed out

2015-02-02 Thread Jan Beulich
>>> On 31.01.15 at 09:49, wrote: > 2015-01-31T04:53:45.005161-05:00 linux-gbze kernel: [63331.820079] > [ cut here ] > 2015-01-31T04:53:45.005187-05:00 linux-gbze kernel: [63331.820163] WARNING: > CPU: > 0 PID: 0 at net/sched/sch_generic.c:264 dev_watchdog+0x266/0x270()

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Paul Durrant
> -Original Message- > From: Don Slutz [mailto:dsl...@verizon.com] > Sent: 30 January 2015 19:19 > To: Don Slutz; Jan Beulich; Paul Durrant > Cc: Andrew Cooper; Ian Campbell; Wei Liu; George Dunlap; Ian Jackson; > Stefano Stabellini; xen-devel@lists.xen.org; Keir (Xen.org) > Subject: Re: [P

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 10:51, wrote: >> From: Don Slutz [mailto:dsl...@verizon.com] >> --- a/xen/arch/x86/hvm/hvm.c >> +++ b/xen/arch/x86/hvm/hvm.c >> @@ -2507,9 +2507,45 @@ int hvm_buffered_io_send(ioreq_t *p) >> return 1; >> } >> >> -bool_t hvm_has_dm(struct domain *d) >> +static bool_t hvm_c

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 02 February 2015 10:04 > To: Paul Durrant; Don Slutz > Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Stefano > Stabellini; Wei Liu; xen-devel@lists.xen.org; Keir (Xen.org) > Subject: RE: [PATCH 3/5] h

Re: [Xen-devel] apic-v reduce network performance in my test case

2015-02-02 Thread Jan Beulich
>>> On 31.01.15 at 11:29, wrote: > Recently I met an odd performance problem: when I turn on APIC > Virtualization feature (apicv=1), the network performance of a windows > guest become worse. > > My test case like this: host only have one windows 2008 R2 HVM > guest running,and this

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 11:06, wrote: >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 02 February 2015 10:04 >> To: Paul Durrant; Don Slutz >> Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Stefano >> Stabellini; Wei Liu; xen-devel@lists.xen.org; Ke

Re: [Xen-devel] [PATCH linux-2.6.18] xen: mark pvscsi frontend request consumed only after last read

2015-02-02 Thread Juergen Gross
On 02/02/2015 08:52 AM, Jan Beulich wrote: On 30.01.15 at 14:51, <"jgr...@suse.com".non-mime.internet> wrote: A request in the ring buffer mustn't be read after it has been marked as consumed. Otherwise it might already have been reused by the frontend without violating the ring protocol. This

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: install nasm

2015-02-02 Thread Ian Campbell
On Sat, 2015-01-31 at 22:51 +, Wei Liu wrote: > OVMF requires nasm to build. I suppose this is a new requirement added by the mainline branch we just added? > > Signed-off-by: Wei Liu Acked-by: Ian Campbell > Cc: Ian Jackson > --- > ts-xen-build-prep | 2 +- > 1 file changed, 1 insertio

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: install nasm

2015-02-02 Thread Wei Liu
Ian Campbell writes: > On Sat, 2015-01-31 at 22:51 +, Wei Liu wrote: >> OVMF requires nasm to build. > > I suppose this is a new requirement added by the mainline branch we just > added? >> Yes. The log in the latest two failed runs suggests so. I also tested building latest OVMF HEAD it do

[Xen-devel] support for more than 32 VCPUs when migrating PVHVM guest

2015-02-02 Thread Vitaly Kuznetsov
Hi Konrad, I just hit an issue with PVHVM guests after save/restore (or migration), if a PVHVM guest has > 32 VCPUs it hangs. Turns out, you saw it almost a year ago and even wrote patches to call VCPUOP_register_vcpu_info after resume. Unfortunately these patches never made it to xen/kernel. Do y

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Denys Drozdov
Hi Ian, The issue observed on credit2 scheduler is similar to the rt scheduler on arm platform. The root cause is that interrupts are disabled in the beginning of arm context_switch, thus spin_lock_irq is failing in ASSERT(local_irq_is_enabled()). I propose to change both credit2 and rt scheduler

Re: [Xen-devel] [PATCH 2/2] drivers: xen/block: add multi-page ring support

2015-02-02 Thread Roger Pau Monné
El 23/01/15 a les 10.14, Bob Liu ha escrit: > Extend xen/block to support multi-page ring. > * xen-blkback notify blkfront with feature-multi-ring-pages > * xen-blkfront write to xenstore about how many pages are used as the ring > > If using 4 pages as the ring, inflight requests inscreased fro

Re: [Xen-devel] support for more than 32 VCPUs when migrating PVHVM guest

2015-02-02 Thread Andrew Cooper
On 02/02/15 10:47, Vitaly Kuznetsov wrote: > Hi Konrad, > > I just hit an issue with PVHVM guests after save/restore (or migration), > if a PVHVM guest has > 32 VCPUs it hangs. Turns out, you saw it almost a > year ago and even wrote patches to call VCPUOP_register_vcpu_info after > resume. Unfortu

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: > Hi Ian, > > On 30/01/15 11:30, Ian Campbell wrote: > > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > >> Hello, > >> > >> Ping? Any more review for this version of this series? > > > > I was awaiting a version with the declaration

Re: [Xen-devel] support for more than 32 VCPUs when migrating PVHVM guest

2015-02-02 Thread Vitaly Kuznetsov
Andrew Cooper writes: > On 02/02/15 10:47, Vitaly Kuznetsov wrote: >> Hi Konrad, >> >> I just hit an issue with PVHVM guests after save/restore (or migration), >> if a PVHVM guest has > 32 VCPUs it hangs. Turns out, you saw it almost a >> year ago and even wrote patches to call VCPUOP_register_vc

[Xen-devel] [PATCH v2 0/5] x86: shadow adjustments / allow for more memory to be used

2015-02-02 Thread Jan Beulich
1: shadow: tidy up fragmentary page lists in multi-page shadows 2: shadow: don't needlessly expose internal functions 3: mm: allow for building without shadow mode support 4: IOMMU: correct page_list_first() use 5: provide build time option to support up to 123Tb of memory Note that thanks to Tim'

Re: [Xen-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 09:17 +0800, Tiejun Chen wrote: > When we're working to support IGD GFX passthrough with qemu > upstream, instead of "-gfx_passthru" we'd like to make that > a machine option, "-machine xxx,-igd-passthru=on". This need > to bring a change on tool side. >From which Qemu versio

Re: [Xen-devel] [RFC V10 0/4] domain snapshot document

2015-02-02 Thread Ian Campbell
On Sun, 2015-02-01 at 20:55 +, Dave Scott wrote: > > > On 29 Jan 2015, at 16:36, Ian Campbell wrote: > > > > On Mon, 2015-01-26 at 11:25 +0800, Chunyan Liu wrote: > >> Changes to V9: > > > > This looks good to me, thanks. > > > > Ian/Wei do you have any comments? xapi folks? > > The API l

Re: [Xen-devel] [PATCH 1/2] sched: credit2: respect per-vcpu hard affinity

2015-02-02 Thread Dario Faggioli
On Sat, 2015-01-31 at 20:51 -1000, Justin Weaver wrote: > On Mon, Jan 19, 2015 at 9:21 PM, Justin Weaver wrote: > > Like I said above, I will look at this again. My VMs were getting > > stuck after certain hard affinity changes. I'll roll back some of > > these changes and test it out again. > >

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:49 +0200, Denys Drozdov wrote: Please don't top post. > The issue observed on credit2 scheduler is similar to the rt scheduler > on arm platform. The root cause is that interrupts are disabled in the > beginning of arm context_switch, thus spin_lock_irq is failing in > AS

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 11:58, wrote: > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >> On 30/01/15 11:30, Ian Campbell wrote: >> > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: >> >> Hello, >> >> >> >> Ping? Any more review for this version of this series? >> > >> > I was awaiting

[Xen-devel] [PATCH v2 1/5] x86/shadow: tidy up fragmentary page lists in multi‑page shadows

2015-02-02 Thread Jan Beulich
From: Tim Deegan Multi-page shadows are linked together using the 'list' field. When those shadows are in the pinned list, the list fragments are spliced into the pinned list; otherwise they have no associated list head. Rework the code that handles these fragments to use the page_list interfac

[Xen-devel] [PATCH v2 2/5] x86/shadow: don't needlessly expose internal functions

2015-02-02 Thread Jan Beulich
... and drop an unused one. Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -2383,7 +2383,7 @@ int sh_remove_write_access_from_sl1p(str /* Remove all mappings of a guest frame from the shadow tables. * Returns non-zero if we need to fl

[Xen-devel] [PATCH v2 3/5] x86/mm: allow for building without shadow mode support

2015-02-02 Thread Jan Beulich
Considering the complexity of the code, it seems to be a reasonable thing to allow people to disable that code entirely even outside the immediate need for this by the next patch. Signed-off-by: Jan Beulich --- v2: Set mode table in shadow_vcpu_init() stub. Convert BUG()/BUG_ON() to ASSERT()/

[Xen-devel] [PATCH v2 4/5] IOMMU/x86: correct page_list_first() use

2015-02-02 Thread Jan Beulich
Comparing its result against NULL is unsafe when page lists use normal list entries for linking together - page_list_empty() needs to be used instead. Signed-off-by: Jan Beulich --- a/xen/drivers/passthrough/x86/iommu.c +++ b/xen/drivers/passthrough/x86/iommu.c @@ -85,8 +85,9 @@ int arch_iommu_p

[Xen-devel] [PATCH v2 5/5] x86: provide build time option to support up to 123Tb of memory

2015-02-02 Thread Jan Beulich
As this requires growing struct page_info from 32 to 48 bytes as well as shrinking the always accessible direct mapped memory range from 5Tb to 3.5Tb, this isn't being introduced as a general or default enabled feature. For now setting "bigmem=y" implies "shadow-paging=n", as the shadow paging cod

[Xen-devel] [PATCH] xen-blkfront: fix accounting of reqs when migrating

2015-02-02 Thread Roger Pau Monne
Current migration code uses blk_put_request in order to finish a request before requeuing it. This function doesn't update the statistics of the queue, which completely screws accounting. Use blk_end_request_all instead which properly updates the statistics of the queue. Signed-off-by: Roger Pau M

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 12:14, wrote: > On Mon, 2015-02-02 at 12:49 +0200, Denys Drozdov wrote: >> The issue observed on credit2 scheduler is similar to the rt scheduler >> on arm platform. The root cause is that interrupts are disabled in the >> beginning of arm context_switch, thus spin_lock_irq is f

Re: [Xen-devel] [PATCH v2 1/5] x86/shadow: tidy up fragmentary page lists in multi‑page shadows

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:18, Jan Beulich wrote: > From: Tim Deegan > > Multi-page shadows are linked together using the 'list' field. When > those shadows are in the pinned list, the list fragments are spliced > into the pinned list; otherwise they have no associated list head. > > Rework the code that han

Re: [Xen-devel] [PATCH v2 2/5] x86/shadow: don't needlessly expose internal functions

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:19, Jan Beulich wrote: > ... and drop an unused one. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper > > --- a/xen/arch/x86/mm/shadow/common.c > +++ b/xen/arch/x86/mm/shadow/common.c > @@ -2383,7 +2383,7 @@ int sh_remove_write_access_from_sl1p(str > /* Remove all mappi

Re: [Xen-devel] [PATCH v2 1/5] x86/shadow: tidy up fragmentary page lists in multi‑page shadows

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 12:43, wrote: > On 02/02/15 11:18, Jan Beulich wrote: >> --- a/xen/arch/x86/mm/shadow/common.c >> +++ b/xen/arch/x86/mm/shadow/common.c >> @@ -1196,6 +1196,26 @@ int shadow_cmpxchg_guest_entry(struct vc >> * the free pool. >> */ >> >> +const u32 sh_type_to_size[] = { > >

Re: [Xen-devel] [PATCH v2 4/5] IOMMU/x86: correct page_list_first() use

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:20, Jan Beulich wrote: > Comparing its result against NULL is unsafe when page lists use normal > list entries for linking together - page_list_empty() needs to be used > instead. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper > > --- a/xen/drivers/passthrough/x86/iomm

Re: [Xen-devel] [PATCH v2 0/5] x86: shadow adjustments / allow for more memory to be used

2015-02-02 Thread Tim Deegan
At 11:09 + on 02 Feb (1422871743), Jan Beulich wrote: > 1: shadow: tidy up fragmentary page lists in multi-page shadows > 2: shadow: don't needlessly expose internal functions > 3: mm: allow for building without shadow mode support > 4: IOMMU: correct page_list_first() use > 5: provide build ti

Re: [Xen-devel] [PATCH v2 1/5] x86/shadow: tidy up fragmentary page lists in multi?page shadows

2015-02-02 Thread Tim Deegan
At 11:47 + on 02 Feb (1422874047), Jan Beulich wrote: > >>> On 02.02.15 at 12:43, wrote: > > On 02/02/15 11:18, Jan Beulich wrote: > >> --- a/xen/arch/x86/mm/shadow/common.c > >> +++ b/xen/arch/x86/mm/shadow/common.c > >> @@ -1196,6 +1196,26 @@ int shadow_cmpxchg_guest_entry(struct vc > >> *

Re: [Xen-devel] [PATCH v2 3/5] x86/mm: allow for building without shadow mode support

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:20, Jan Beulich wrote: > Considering the complexity of the code, it seems to be a reasonable > thing to allow people to disable that code entirely even outside the > immediate need for this by the next patch. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper > --- > v2: S

Re: [Xen-devel] [PATCH v2 5/5] x86: provide build time option to support up to 123Tb of memory

2015-02-02 Thread Andrew Cooper
On 02/02/15 11:21, Jan Beulich wrote: > As this requires growing struct page_info from 32 to 48 bytes as well > as shrinking the always accessible direct mapped memory range from 5Tb > to 3.5Tb, this isn't being introduced as a general or default enabled > feature. > > For now setting "bigmem=y" im

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 11:40 +, Jan Beulich wrote: > >>> On 02.02.15 at 12:14, wrote: > > On Mon, 2015-02-02 at 12:49 +0200, Denys Drozdov wrote: > >> The issue observed on credit2 scheduler is similar to the rt scheduler > >> on arm platform. The root cause is that interrupts are disabled in t

Re: [Xen-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 09:17:23AM +0800, Tiejun Chen wrote: > When we're working to support IGD GFX passthrough with qemu > upstream, instead of "-gfx_passthru" we'd like to make that > a machine option, "-machine xxx,-igd-passthru=on". This need > to bring a change on tool side. > > Signed-off-b

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 10:58, Ian Campbell wrote: > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >> Hi Ian, >> >> On 30/01/15 11:30, Ian Campbell wrote: >>> On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: Hello, Ping? Any more review for this version of this series? >>> >>> I w

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 11:15, Jan Beulich wrote: On 02.02.15 at 11:58, wrote: >> On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: >>> On 30/01/15 11:30, Ian Campbell wrote: On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > Hello, > > Ping? Any more review for this version o

Re: [Xen-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough

2015-02-02 Thread Ian Jackson
Wei Liu writes ("Re: [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough"): > On Mon, Feb 02, 2015 at 09:17:23AM +0800, Tiejun Chen wrote: > > When we're working to support IGD GFX passthrough with qemu > > upstream, instead of "-gfx_passthru" we'd like to make that > > a ma

[Xen-devel] [PATCH 1/7] stubdom: fix "make build"

2015-02-02 Thread Wei Liu
Cross compiling libxc requires some symlinks to exist. Note that make -C tools/include requires running tools/configure. But at least now the error message is much better than just a "file not found" error. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jakcson --- stubdom/Makefile | 2 ++ 1

[Xen-devel] [PATCH 7/7] Remove in-tree mini-os directory

2015-02-02 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson [ output trimmed ] ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH 2/7] Makefile: refactor build/clean/distclean targets

2015-02-02 Thread Wei Liu
Factor out per-subsystem build/clean/distclean-% targets, so that we can build subsystems independently in top level directory. The motive behind this is after splitting out mini-os from Xen tree, stubdom is in effect a downstream of mini-os. I would like to have the ability to build it independe

[Xen-devel] [PATCH 3/7] stubdom: don't look for mini-os source file during configure

2015-02-02 Thread Wei Liu
Mini-os source code will be fetched during build. Please rerun autogen.sh after applying this patch. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- stubdom/configure.ac | 1 - 1 file changed, 1 deletion(-) diff --git a/stubdom/configure.ac b/stubdom/configure.ac index 6468203..4

[Xen-devel] [PATCH 5/7] Mini-OS: standalone build

2015-02-02 Thread Wei Liu
In order to keep the tree bisectable all the changes are done in one single commit. Things done in this commit: 1. Import necessary .mk files from Xen. 2. Move all XEN_ related variables to MINIOS_ namespace. 3. Import Xen public header files. 4. Import BSD's list.h and helper script. Mini-OS's

[Xen-devel] [PATCH 4/7] git-checkout.sh: use "mkdir -p"

2015-02-02 Thread Wei Liu
Otherwise mkdir extras/mini-os fails because extras doesn't exist. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- scripts/git-checkout.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/git-checkout.sh b/scripts/git-checkout.sh index 15b3ce9..20ae31f 10

[Xen-devel] [PATCH 6/7] build system: stubdom targets now depends on mini-os target

2015-02-02 Thread Wei Liu
Provide mini-os url and revision in Config.mk Introduce Makefile.mini-os which contains mini-os specific targets. Target mini-os-dir clones mini-os tree from upstream. Make stubdom targets depend on mini-os-dir target. Make subtree-force-update{,-all} depend on mini-os-dir-force-update. Also mak

[Xen-devel] [PATCH 0/7] Split off mini-os to a separate tree

2015-02-02 Thread Wei Liu
As far as I can tell there is no incoming mini-os patches at the moment. So I suggest we commence once we get that last mini-os patch in staging pushed to master. I use following runes to split off mini-os: git filter-branch --tag-name-filter cat \ --subdirectory-filter extras/mini-os/ -- -

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Julien Grall
On 02/02/15 12:16, Ian Campbell wrote: > On Mon, 2015-02-02 at 11:40 +, Jan Beulich wrote: > On 02.02.15 at 12:14, wrote: >>> On Mon, 2015-02-02 at 12:49 +0200, Denys Drozdov wrote: The issue observed on credit2 scheduler is similar to the rt scheduler on arm platform. The root c

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:48 +, Julien Grall wrote: > On 02/02/15 10:58, Ian Campbell wrote: > > On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: > >> Hi Ian, > >> > >> On 30/01/15 11:30, Ian Campbell wrote: > >>> On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: > Hello, >

Re: [Xen-devel] [PATCH v2 5/5] x86: provide build time option to support up to 123Tb of memory

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 12:58, wrote: > On 02/02/15 11:21, Jan Beulich wrote: >> As this requires growing struct page_info from 32 to 48 bytes as well >> as shrinking the always accessible direct mapped memory range from 5Tb >> to 3.5Tb, this isn't being introduced as a general or default enabled >> fe

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:59 +, Julien Grall wrote: > On 02/02/15 12:16, Ian Campbell wrote: > > On Mon, 2015-02-02 at 11:40 +, Jan Beulich wrote: > > On 02.02.15 at 12:14, wrote: > >>> On Mon, 2015-02-02 at 12:49 +0200, Denys Drozdov wrote: > The issue observed on credit2 schedule

Re: [Xen-devel] [PATCH] xen/arm: Fix rtds scheduler for arm

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 13:16, wrote: > On Mon, 2015-02-02 at 11:40 +, Jan Beulich wrote: >> X86 calls context_saved() before >> schedule_tail() and has no need for IRQs to be disabled after >> __context_switch() returned. > > In fact from the comment ("...which may fault...") I think it requires

[Xen-devel] [PATCH v2] xen/evtchn: Alter the alloc/free xen event channel functions to take a domain

2015-02-02 Thread Andrew Cooper
The resource behind an event channel is domain centric rather than vcpu centric. This change allows mem_event_disable() to avoid arbitrarily referencing d->vcpu[0] just to pass the domain. No functional change. Signed-off-by: Andrew Cooper CC: Keir Fraser CC: Jan Beulich CC: Tim Deegan ---

Re: [Xen-devel] [OSSTEST PATCH v4 1/9] overlay: update overlay/etc/grub.d/20_linux_xen

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > This file was originally created to work around Debian bug #633127 > ("/etc/grub/20_linux does not recognise some old Xen kernels"). > > According to Debian bug tracker [0], #633127 bug is fixed in Wheezy. As > we're now using Wheezy in OSSTest w

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Jan Beulich
>>> On 02.02.15 at 13:52, wrote: > On 02/02/15 11:15, Jan Beulich wrote: > On 02.02.15 at 11:58, wrote: >>> On Fri, 2015-01-30 at 11:33 +, Julien Grall wrote: On 30/01/15 11:30, Ian Campbell wrote: > On Thu, 2015-01-29 at 18:32 +, Julien Grall wrote: >> Hello, >>

Re: [Xen-devel] [OSSTEST PATCH v4 3/9] ts-xen-build: build with XSM support if requested

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Signed-off-by: Wei Liu Inconsistent whitespace in the final hunk, but: Acked-by: Ian Campbell This will override the default XSM_ENABLE always, even if no runvar is present, I guess we think that is correct?

Re: [Xen-devel] [OSSTEST PATCH v4 4/9] mfi-common: create build-$arch-xsm job

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by:Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v4 7/9] ts-xen-install: install Xen with XSM support if requested

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Signed-off-by: Wei Liu Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v4 8/9] make-flight: factor out do_pv_debian_tests

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > make-flight | 21 ++--- > 1 file changed, 14 insertions(+), 7 deletions(-) > > diff --git a/make-flight b/make-flight > index 9963a46..35904be 100755 > --- a/make-flight > +++ b/make-flight > @@

Re: [Xen-devel] [OSSTEST PATCH v4 9/9] mfi-common, make-flight: create XSM test jobs

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Duplicate Debian PV and HVM test jobs for XSM testing. > > Signed-off-by: Wei Liu Acked-by: Ian Campbell Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH v4 0/9] XSM test case for OSSTest

2015-02-02 Thread Ian Campbell
On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > Hi all > > This patch series attempts to duplicate some Debian test cases for XSM. This > is version 4 of this series. Apart from any rebasing issues relating to its age I think this series looks good.

[Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Olaf Hering
The Makefile uses wildcard to collect a list of files. The resulting list of files is in directory order, which is random. As a result the generated html files will differ when build on different hosts. Use the built-in sort function to get a stable list of files. Signed-off-by: Olaf Hering Cc:

Re: [Xen-devel] [OSSTEST PATCH v4 8/9] make-flight: factor out do_pv_debian_tests

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 13:18 +, Ian Campbell wrote: > On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > > Signed-off-by: Wei Liu > > --- > > make-flight | 21 ++--- > > 1 file changed, 14 insertions(+), 7 deletions(-) > > > > diff --git a/make-flight b/make-flight > > inde

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Ian Campbell
On Mon, 2015-01-26 at 16:14 -0700, Jim Fehlig wrote: Cc-ing the other toolstack maintainers, both of whom have more familiarity with this part of libxl than I. > The attached patch is a hack I cooked up to fix one of the libvirt-TCK > Xen failures. The test (200-disk-hotplug.t) attempts to hot a

Re: [Xen-devel] [PATCH v2 17/29] Ovmf/Xen: refactor XenBusDxe hypercall implementation

2015-02-02 Thread Laszlo Ersek
On 01/27/15 14:10, Ard Biesheuvel wrote: > On 27 January 2015 at 12:46, Ard Biesheuvel wrote: >> On 27 January 2015 at 12:43, Stefano Stabellini >> wrote: >>> On Mon, 26 Jan 2015, Ard Biesheuvel wrote: This refactors the Xen hypercall implementation that is part of the XenBusDxe driver,

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Jackson
Olaf Hering writes ("[PATCH] docs: create reproducible html"): > The Makefile uses wildcard to collect a list of files. The resulting > list of files is in directory order, which is random. As a result the > generated html files will differ when build on different hosts. I'm in favour of reproduci

Re: [Xen-devel] [PATCH 0/6] xen/arm: Move in/out code to/from init section

2015-02-02 Thread Julien Grall
On 02/02/15 13:12, Jan Beulich wrote: > This example in particular makes clear that looking at just the > declaration is insufficient in certain cases.Whoever is looking > for specific properties of a functions needs to know which ones > (s)he cares about, and then look in the appropriate place. _

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 13:27 +, Ian Campbell wrote: > On Mon, 2015-01-26 at 16:14 -0700, Jim Fehlig wrote: > > Cc-ing the other toolstack maintainers, both of whom have more > familiarity with this part of libxl than I. > > > The attached patch is a hack I cooked up to fix one of the libvirt-T

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Olaf Hering
On Mon, Feb 02, Ian Jackson wrote: > What do you think ? Thats overkill. The maintainers of make removed the internal sort with 3.82, likely for performance reasons. Sorted lists are required only in a few places. The Xen source has only two or three places where the sort is actually required. I

Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions

2015-02-02 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [PATCH RFC] libxl: set disk defaults in remove/destroy functions"): > Perhaps better would be to add a new parameter to0 DEFINE_DEVICE_REMOVE > like extra_setdefault_args which is pasted in the appropriate place? I don't think this is right because... > I su

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:40 +0100, Olaf Hering wrote: > On Mon, Feb 02, Ian Jackson wrote: > > > What do you think ? > > Thats overkill. The maintainers of make removed the internal sort with > 3.82, likely for performance reasons. Aren't you adding uses of it here? > Sorted lists are required

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Jackson
Olaf Hering writes ("Re: [PATCH] docs: create reproducible html"): > On Mon, Feb 02, Ian Jackson wrote: > > What do you think ? > > Thats overkill. The maintainers of make removed the internal sort with > 3.82, likely for performance reasons. Sorted lists are required only in > a few places. The X

Re: [Xen-devel] [PATCH] docs: create reproducible html

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 13:43 +, Ian Campbell wrote: > On Mon, 2015-02-02 at 14:40 +0100, Olaf Hering wrote: > > On Mon, Feb 02, Ian Jackson wrote: > > > > > What do you think ? > > > > Thats overkill. The maintainers of make removed the internal sort with > > 3.82, likely for performance reaso

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Don Slutz
On 02/02/15 03:36, Jan Beulich wrote: On 30.01.15 at 20:19, wrote: >> Soon after sending this, I came up with a 2nd way for fix. >> Change hvm_has_dm() to use hvm_select_ioreq_server(). Then >> the correct answer will be found, and so will not retry. >> >> To avoid 2 calls to hvm_select_iore

Re: [Xen-devel] [PATCH 3/5] hvmemul_do_io: If the send to the ioreq server failed do not retry.

2015-02-02 Thread Don Slutz
On 02/02/15 05:14, Jan Beulich wrote: On 02.02.15 at 11:06, wrote: >>> -Original Message- >>> From: Jan Beulich [mailto:jbeul...@suse.com] >>> Sent: 02 February 2015 10:04 >>> To: Paul Durrant; Don Slutz >>> Cc: Andrew Cooper; George Dunlap; Ian Campbell; Ian Jackson; Stefano >>> S

Re: [Xen-devel] rcu_sched self-detect stall when disable vif device

2015-02-02 Thread Julien Grall
Hi David, On 30/01/15 16:04, David Vrabel wrote: > How about this? This is working for me. Thanks! > 8<-- > xen-netback: stop the guest rx thread after a fatal error > > After commit e9d8b2c2968499c1f96563e6522c56958d5a1d0d (xen-netback: > disable rogue v

Re: [Xen-devel] apic-v reduce network performance in my test case

2015-02-02 Thread Liuqiming (John)
Hi Jan, Thanks for the reply. On 2015/2/2 18:12, Jan Beulich wrote: >>> On 31.01.15 at 11:29, wrote: > Recently I met an odd performance problem: when I turn on APIC > Virtualization feature (apicv=1), the network performance of a windows > guest become worse. > > My test case like

[Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Campbell
- mkpxedir requires the resource to be allocated - mention the need to bless the hosts for comissioning run, and to add proper blessings afterwards. - run mg-execute-flight in screen Signed-off-by: Ian Campbell --- README.dev | 17 - mg-hosts |3 ++- 2 files changed, 18

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-build-prep: install nasm

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 10:38 +, Wei Liu wrote: > Ian Campbell writes: > > > On Sat, 2015-01-31 at 22:51 +, Wei Liu wrote: > >> OVMF requires nasm to build. > > > > I suppose this is a new requirement added by the mainline branch we just > > added? > >> > > Yes. The log in the latest two f

Re: [Xen-devel] support for more than 32 VCPUs when migrating PVHVM guest

2015-02-02 Thread Konrad Rzeszutek Wilk
On Mon, Feb 02, 2015 at 12:03:28PM +0100, Vitaly Kuznetsov wrote: > Andrew Cooper writes: > > > On 02/02/15 10:47, Vitaly Kuznetsov wrote: > >> Hi Konrad, > >> > >> I just hit an issue with PVHVM guests after save/restore (or migration), > >> if a PVHVM guest has > 32 VCPUs it hangs. Turns out, y

Re: [Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] README.dev: Updates to commissioning process"): > - mkpxedir requires the resource to be allocated > - mention the need to bless the hosts for comissioning run, and to add > proper blessings afterwards. > - run mg-execute-flight in screen ... > +$ mf-hosts s

Re: [Xen-devel] [PATCH 1/2] sched: credit2: respect per-vcpu hard affinity

2015-02-02 Thread Dario Faggioli
On Sat, 2015-01-31 at 20:51 -1000, Justin Weaver wrote: > On Mon, Jan 19, 2015 at 9:21 PM, Justin Weaver wrote: > > On Mon, Jan 12, 2015 at 8:05 AM, Dario Faggioli > For example... > I start a guest with one vcpu with hard affinity 8 - 15 and xl > vcpu-list says it's running on pcpu 15 > I run xl

Re: [Xen-devel] [PATCH OSSTEST] README.dev: Updates to commissioning process

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:22 +, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] README.dev: Updates to commissioning > process"): > > - mkpxedir requires the resource to be allocated > > - mention the need to bless the hosts for comissioning run, and to add > > proper blessings aft

Re: [Xen-devel] [RFC PATCH] dpci: Put the dpci back on the list if running on another CPU.

2015-02-02 Thread Konrad Rzeszutek Wilk
On Tue, Jan 13, 2015 at 10:20:00AM +, Jan Beulich wrote: > >>> On 12.01.15 at 17:45, wrote: > > There is race when we clear the STATE_SCHED in the softirq > > - which allows the 'raise_softirq_for' to progress and > > schedule another dpci. During that time the other CPU could > > receive an i

Re: [Xen-devel] [OSSTEST PATCH v4 3/9] ts-xen-build: build with XSM support if requested

2015-02-02 Thread Wei Liu
On Mon, Feb 02, 2015 at 01:14:34PM +, Ian Campbell wrote: > On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > > Signed-off-by: Wei Liu > > Inconsistent whitespace in the final hunk, but: > Acked-by: Ian Campbell > > This will override the default XSM_ENABLE always, even if no runvar is >

Re: [Xen-devel] [PATCH v3] libxl_set_memory_target: retain the same maxmem offset on top of the current target

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:08 +, Stefano Stabellini wrote: > @@ -4775,6 +4781,14 @@ retry_transaction: > new_target_memkb = current_target_memkb + target_memkb; > } else > new_target_memkb = target_memkb - videoram; > + > +if (new_target_memkb <= 0) { > +LIB

Re: [Xen-devel] [PATCH v3] libxl: Wait for ballooning if free memory is increasing

2015-02-02 Thread Ian Campbell
On Fri, 2015-01-30 at 14:01 -0700, Mike Latimer wrote: > During domain startup, all required memory ballooning must complete > within a maximum window of 33 seconds (3 retries, 11 seconds of delay). > If not, domain creation is aborted with a 'failed to free memory' error. > > In order to accommod

Re: [Xen-devel] [PATCH] time: widen wallclock seconds to 64 bits

2015-02-02 Thread Ian Campbell
On Thu, 2015-01-29 at 15:39 +, Jan Beulich wrote: > Linux is in the process of converting their seconds representation to > 64 bits, so in order to support it consistently we should follow suit > (which at some point in quite a few years we'd have to do anyway). To > represent this in struct sh

Re: [Xen-devel] [PATCH 0/7] Split off mini-os to a separate tree

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:58 +, Wei Liu wrote: > As far as I can tell there is no incoming mini-os patches at the moment. So I > suggest we commence once we get that last mini-os patch in staging pushed to > master. Is the new xenbits tree (the proper one, not your people one) for mini-os all s

Re: [Xen-devel] [OSSTEST PATCH v4 3/9] ts-xen-build: build with XSM support if requested

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 14:30 +, Wei Liu wrote: > On Mon, Feb 02, 2015 at 01:14:34PM +, Ian Campbell wrote: > > On Wed, 2014-12-10 at 22:21 +, Wei Liu wrote: > > > Signed-off-by: Wei Liu > > > > Inconsistent whitespace in the final hunk, but: > > Acked-by: Ian Campbell > > > > This wi

Re: [Xen-devel] [PATCH 1/7] stubdom: fix "make build"

2015-02-02 Thread Ian Campbell
On Mon, 2015-02-02 at 12:58 +, Wei Liu wrote: > Cross compiling libxc requires some symlinks to exist. > > Note that make -C tools/include requires running tools/configure. But at > least now the error message is much better than just a "file not found" > error. > > Signed-off-by: Wei Liu A

  1   2   3   >