Re: [Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Marciniszyn, Mike
> > This driver already makes use of ioremap_wc() on PIO buffers, so > > convert it to use arch_phys_wc_add(). > > This is probably OK, but I think you should also remove the qib_wc_pat module > parameter. > > Jason Revise based on Jason's request and I will do some testing. Mike _

Re: [Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Jason Gunthorpe
On Tue, Apr 21, 2015 at 02:50:35PM -0700, Luis R. Rodriguez wrote: > - if (qib_wc_pat) { > - resource_size_t vl15off; > - /* > - * We do not set WC on the VL15 buffers to avoid > - * a rare problem with unaligned writes from > - * i

Re: [Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Jason Gunthorpe
On Tue, Apr 21, 2015 at 01:37:55PM -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > This driver already makes use of ioremap_wc() on PIO buffers, > so convert it to use arch_phys_wc_add(). This is probably OK, but I think you should also remove the qib_wc_pat module parameter. Ja

[Xen-devel] virq injection probelem in Xen ARM 4.5

2015-04-21 Thread 신정섭
HI I might find the VIRQ injection in Xen ARM 4.5 and fix with simple way. Check please. Problem - In Xen ARM 4.5, SPI's pending_irq structure could be accessed simultaneously from each pCPU. Reason - When inject SPI VIRQ to domain0, Xen ARM 4.5 set bit HW(31bit) bit in list register. so do

Re: [Xen-devel] [RFC PATCH] x86/asm/irq: Don't use POPF but STI

2015-04-21 Thread Andy Lutomirski
On Tue, Apr 21, 2015 at 9:12 AM, Linus Torvalds wrote: > On Tue, Apr 21, 2015 at 5:45 AM, Ingo Molnar wrote: >> >> Totally untested and not signed off yet: because we'd first have to >> make sure (via irq flags debugging) that it's not used in reverse, to >> re-disable interrupts: > > Not only mi

[Xen-devel] [PATCH v4 2/2] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This driver already makes use of ioremap_wc() on PIO buffers, so convert it to use arch_phys_wc_add(). The qib driver uses a mmap() special case for when PAT is not used, this behaviour used to be determined with a module parameter but since we have been asked to just r

Re: [Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Luis R. Rodriguez
On Tue, Apr 21, 2015 at 2:12 PM, Andy Lutomirski wrote: > On Tue, Apr 21, 2015 at 2:09 PM, Luis R. Rodriguez > wrote: >> On Tue, Apr 21, 2015 at 1:59 PM, Marciniszyn, Mike >> wrote: > This driver already makes use of ioremap_wc() on PIO buffers, so > convert it to use arch_phys_wc_add(

Re: [Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Andy Lutomirski
On Tue, Apr 21, 2015 at 2:09 PM, Luis R. Rodriguez wrote: > On Tue, Apr 21, 2015 at 1:59 PM, Marciniszyn, Mike > wrote: >>> > This driver already makes use of ioremap_wc() on PIO buffers, so >>> > convert it to use arch_phys_wc_add(). >>> >>> This is probably OK, but I think you should also remov

Re: [Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Luis R. Rodriguez
On Tue, Apr 21, 2015 at 1:59 PM, Marciniszyn, Mike wrote: >> > This driver already makes use of ioremap_wc() on PIO buffers, so >> > convert it to use arch_phys_wc_add(). >> >> This is probably OK, but I think you should also remove the qib_wc_pat module >> parameter. >> >> Jason > > Revise based

[Xen-devel] [PATCH v3] x86: mtrr: generalize run time disabling of MTRR

2015-04-21 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" It is possible to enable CONFIG_MTRR and CONFIG_X86_PAT and end up with a system with MTRR functionality disabled and PAT functionality enabled. This can happen for instance when the Xen hypervisor is used where MTRR is not supported but PAT is. This can happen on Linux

[Xen-devel] [PATCH v3 2/3] IB/qib: use arch_phys_wc_add()

2015-04-21 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This driver already makes use of ioremap_wc() on PIO buffers, so convert it to use arch_phys_wc_add(). Cc: Toshi Kani Cc: Rickard Strandqvist Cc: Mike Marciniszyn Cc: Roland Dreier Cc: Sean Hefty Cc: Hal Rosenstock Cc: Dennis Dalessandro Cc: Andy Lutomirski Cc:

[Xen-devel] [PATCH v3 1/4] pci: add pci_iomap_wc() variants

2015-04-21 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" This allows drivers to take advantage of write-combining when possible. Ideally we'd have pci_read_bases() just peg an IORESOURCE_WC flag for us but where exactly video devices memory lie varies *largely* and at times things are mixed with MMIO registers, sometimes we ca

Re: [Xen-devel] [PATCH v1 05/47] pci: add pci_iomap_wc() variants

2015-04-21 Thread Luis R. Rodriguez
On Tue, Apr 21, 2015 at 12:25 PM, Michael S. Tsirkin wrote: > To summarise, driver needs to know what it's doing, > we can't set WC in the pci core automatically. Thanks, I'll document this and proceed with device driver helpers to aid with this. Luis __

Re: [Xen-devel] [PATCH v1 05/47] pci: add pci_iomap_wc() variants

2015-04-21 Thread Michael S. Tsirkin
On Mon, Mar 23, 2015 at 12:20:47PM -0500, Bjorn Helgaas wrote: > pci_iomap_range() already makes a cacheable mapping if > IORESOURCE_CACHEABLE; I'm guessing that you would like it to > automatically use WC if the BAR if IORESOURCE_PREFETCH, e.g., > > if (flags & IORESOURCE_CACHEABLE) > retur

Re: [Xen-devel] [PATCH v1 05/47] pci: add pci_iomap_wc() variants

2015-04-21 Thread Michael S. Tsirkin
On Tue, Apr 21, 2015 at 07:52:49PM +0200, Luis R. Rodriguez wrote: > On Thu, Mar 26, 2015 at 04:00:54AM +0100, Luis R. Rodriguez wrote: > > On Mon, Mar 23, 2015 at 12:20:47PM -0500, Bjorn Helgaas wrote: > > > Hi Luis, > > > > > > This seems OK to me, > > > > Great. > > > > > but I'm curious abo

Re: [Xen-devel] [PATCH v1 05/47] pci: add pci_iomap_wc() variants

2015-04-21 Thread Luis R. Rodriguez
On Thu, Mar 26, 2015 at 04:00:54AM +0100, Luis R. Rodriguez wrote: > On Mon, Mar 23, 2015 at 12:20:47PM -0500, Bjorn Helgaas wrote: > > Hi Luis, > > > > This seems OK to me, > > Great. > > > but I'm curious about a few things. > > > > On Fri, Mar 20, 2015 at 6:17 PM, Luis R. Rodriguez > > wro

[Xen-devel] [PATCH] raisin: Some git-checkout improvements

2015-04-21 Thread George Dunlap
1. Switch local variables to lower-case and declare them local. 2. Cloning git trees from remote repos is often a very long operation. Allow the user to specify a faster git cache as a prefix. 3. At the moment you can either check out a specific changeset or "master", but you can't check out a di

Re: [Xen-devel] [PATCH v2 0/6] raisin: introduce ovmf and linux

2015-04-21 Thread Stefano Stabellini
On Tue, 21 Apr 2015, George Dunlap wrote: > On 04/21/2015 05:46 PM, Stefano Stabellini wrote: > > Hi all, > > > > this patch series includes a few clean-ups and introduces a component to > > build ovmf and a component to build a linux kernel. The latter is not > > enabled by default, and could pro

[Xen-devel] [PATCH v2 3/6] raisin: rename ARCH to RAISIN_ARCH

2015-04-21 Thread Stefano Stabellini
Rename exported environmental variable ARCH to RAISIN_ARCH not to conflict with any component Makefile variables. Signed-off-by: Stefano Stabellini Acked-by: George Dunlap --- README |2 +- components/grub |6 +++--- components/libvirt |2 +-

[Xen-devel] [PATCH v2 6/6] raisin: build linux

2015-04-21 Thread Stefano Stabellini
Add a component, disabled by default, to build a linux kernel with the Xen kconfig options enabled. Signed-off-by: Stefano Stabellini Acked-by: George Dunlap --- components/linux | 120 + components/series |1 + defconfig |4

Re: [Xen-devel] [OSSTEST PATCH] ts-kernel-build: Enable x86 IOMMU options

2015-04-21 Thread Konrad Rzeszutek Wilk
On Tue, Apr 21, 2015 at 04:44:07PM +0100, Ian Campbell wrote: > On Tue, 2015-04-21 at 16:36 +0100, Ian Jackson wrote: > > This has a variety of beneficial implications: > > > > * The kernel becomes more like the kind of distro kernels that Xen > >users are probably using. > > > > * We are m

[Xen-devel] [PATCH v2 5/6] raisin: introduce ovmf

2015-04-21 Thread Stefano Stabellini
Add a component to build ovmf and pass the output binary to the xen build. Signed-off-by: Stefano Stabellini --- Changes in v2: - only enable ovmf in Xen, if we actually have built ovmf. --- components/series |1 + components/xen|7 ++- defconfig |6 -- 3 files

[Xen-devel] [PATCH v2 0/6] raisin: introduce ovmf and linux

2015-04-21 Thread Stefano Stabellini
Hi all, this patch series includes a few clean-ups and introduces a component to build ovmf and a component to build a linux kernel. The latter is not enabled by default, and could probably benefit from a bit more work on the kconfig options to really be useful, but it is a start. Changes in v2:

Re: [Xen-devel] [PATCH v2 0/6] raisin: introduce ovmf and linux

2015-04-21 Thread George Dunlap
On 04/21/2015 05:46 PM, Stefano Stabellini wrote: > Hi all, > > this patch series includes a few clean-ups and introduces a component to > build ovmf and a component to build a linux kernel. The latter is not > enabled by default, and could probably benefit from a bit more work on > the kconfig op

[Xen-devel] [PATCH v2 1/6] raisin: introduce verbose_echo

2015-04-21 Thread Stefano Stabellini
A new utility function to make the code more readable and compact: prints a message if VERBOSE = 1. Signed-off-by: Stefano Stabellini --- Changes in v2: - rename _verbose_echo to verbose_echo --- lib/common-functions.sh | 46 -- 1 file changed, 16

[Xen-devel] [PATCH v2 4/6] raisin: pass --with-system-seabios with seabios was built

2015-04-21 Thread Stefano Stabellini
Detect whether we have built seabios and only pass the relative command line argument to Xen if we actually did. Signed-off-by: Stefano Stabellini --- components/xen |9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/components/xen b/components/xen index b3426f0..ac9

[Xen-devel] [PATCH v2 2/6] raisin: remove duplicate source config in raise

2015-04-21 Thread Stefano Stabellini
Signed-off-by: Stefano Stabellini Acked-by: George Dunlap --- raise | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/raise b/raise index bce6908..68dbfd8 100755 --- a/raise +++ b/raise @@ -17,9 +17,12 @@ _help() { } # Include your defaults -if [[ -e "./c

[Xen-devel] [OSSTEST PATCH] target_cmd_build: Delete build-ok-stamp before starting

2015-04-21 Thread Ian Jackson
Many of the callers of target_cmd_build use a build-ok-stamp idiom to detect failed builds. This idiom does not work if the stamp file exists already, so delete it. In the future we may move more of the test build-ok-stamp, echo ok, into TestSupport, but this will do for now. Signed-off-by: Ian

Re: [Xen-devel] [RFC PATCH] x86/asm/irq: Don't use POPF but STI

2015-04-21 Thread Linus Torvalds
On Tue, Apr 21, 2015 at 5:45 AM, Ingo Molnar wrote: > > Totally untested and not signed off yet: because we'd first have to > make sure (via irq flags debugging) that it's not used in reverse, to > re-disable interrupts: Not only might that happen in some place, I *really* doubt that a conditiona

Re: [Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-04-21 Thread Olaf Hering
On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: > On Tue, Apr 21, 2015 at 04:35:17PM +0200, Olaf Hering wrote: > > On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: > > > > > On Fri, Apr 17, 2015 at 08:30:56AM +, Olaf Hering wrote: > > > > The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in

Re: [Xen-devel] [PATCH v2 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Andrew Cooper
On 21/04/15 16:47, Andrew Cooper wrote: > For top-level targets which use more than a single program to produce content > (txt already, pdf once pandoc is supported), these current checks are > unsuitable. > > By moving the the install checks to the rules which actually use the programs, > it is no

[Xen-devel] [PATCH v2 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Andrew Cooper
For top-level targets which use more than a single program to produce content (txt already, pdf once pandoc is supported), these current checks are unsuitable. By moving the the install checks to the rules which actually use the programs, it is now possible to build a subset of a top-level target

[Xen-devel] [PATCH v2 7/7] docs/build: Support generation of pandoc documents

2015-04-21 Thread Andrew Cooper
pandoc is a superset of markdown Signed-off-by: Andrew Cooper Acked-by: Ian Campbell --- v2: Extend changes from previous patch --- docs/Makefile | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/docs/Makefile b/docs/Makefile index 4306a95..9183252 100644 --

Re: [Xen-devel] [PATCH 4/5] raisin: introduce ovmf

2015-04-21 Thread George Dunlap
On 04/21/2015 04:45 PM, George Dunlap wrote: > On 04/21/2015 04:43 PM, Stefano Stabellini wrote: >> On Tue, 21 Apr 2015, George Dunlap wrote: >>> On 04/21/2015 03:55 PM, Stefano Stabellini wrote: diff --git a/components/series b/components/series index f0f3cfa..fe9092a 100644 --- a/c

Re: [Xen-devel] [PATCH 4/5] raisin: introduce ovmf

2015-04-21 Thread George Dunlap
On 04/21/2015 04:43 PM, Stefano Stabellini wrote: > On Tue, 21 Apr 2015, George Dunlap wrote: >> On 04/21/2015 03:55 PM, Stefano Stabellini wrote: >>> diff --git a/components/series b/components/series >>> index f0f3cfa..fe9092a 100644 >>> --- a/components/series >>> +++ b/components/series >>> @@

Re: [Xen-devel] [OSSTEST PATCH] ts-kernel-build: Enable x86 IOMMU options

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 16:36 +0100, Ian Jackson wrote: > This has a variety of beneficial implications: > > * The kernel becomes more like the kind of distro kernels that Xen >users are probably using. > > * We are more likely to discover any bugs in Linux where Linux >running under Xen

Re: [Xen-devel] [PATCH 4/5] raisin: introduce ovmf

2015-04-21 Thread Stefano Stabellini
On Tue, 21 Apr 2015, George Dunlap wrote: > On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > > diff --git a/components/series b/components/series > > index f0f3cfa..fe9092a 100644 > > --- a/components/series > > +++ b/components/series > > @@ -1,4 +1,5 @@ > > seabios > > +ovmf > > xen > > qem

Re: [Xen-devel] [PATCH 2/5] raisin: remove duplicate source config in raise

2015-04-21 Thread Stefano Stabellini
On Tue, 21 Apr 2015, George Dunlap wrote: > On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > > Signed-off-by: Stefano Stabellini > > I take it this got here by a bad merge or somethign? yes, I think it happened like that... > Acked-by: George Dunlap > > > --- > > raise | 15 +---

[Xen-devel] [OSSTEST PATCH] ts-kernel-build: Enable x86 IOMMU options

2015-04-21 Thread Ian Jackson
This has a variety of beneficial implications: * The kernel becomes more like the kind of distro kernels that Xen users are probably using. * We are more likely to discover any bugs in Linux where Linux running under Xen (eg as dom0) fights with Xen for control of io mediation resource

Re: [Xen-devel] [PATCH 1/5] raisin: introduce _verbose_echo

2015-04-21 Thread Stefano Stabellini
On Tue, 21 Apr 2015, George Dunlap wrote: > On 04/21/2015 03:54 PM, Stefano Stabellini wrote: > > A new utility function to make the code more readable and compact: > > prints a message if VERBOSE = 1. > > > > Signed-off-by: Stefano Stabellini > > --- > > lib/common-functions.sh | 46 > >

Re: [Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-04-21 Thread Konrad Rzeszutek Wilk
On Tue, Apr 21, 2015 at 04:35:17PM +0200, Olaf Hering wrote: > On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: > > > On Fri, Apr 17, 2015 at 08:30:56AM +, Olaf Hering wrote: > > > The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev > > > property. Add the missing :LUN part to

Re: [Xen-devel] [PATCH v3] xen: sched_rt: print useful affinity info when dumping

2015-04-21 Thread Dario Faggioli
First of all, for the ones that received multiple copies of the patch, sorry... my bad letting my fingers slip while composing the `stg mail' command :-/ On Tue, 2015-04-21 at 17:22 +0200, Dario Faggioli wrote: > In fact, printing the cpupool's CPU online mask > for each vCPU is just redundant, as

Re: [Xen-devel] [PATCH v3] xen: sched_rt: print useful affinity info when dumping

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 17:22, wrote: > @@ -409,6 +435,15 @@ rt_init(struct scheduler *ops) > if ( prv == NULL ) > return -ENOMEM; > > +ASSERT( !_cpumask_scratch || nr_rt_ops > 0 ); > + > +if ( !_cpumask_scratch ) { Coding style. > +_cpumask_scratch = xmalloc_array(cpu

Re: [Xen-devel] [PATCH OSSTEST v2] Arrange for core dumps to be placed in /var/core and collect them

2015-04-21 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v2] Arrange for core dumps to be placed in /var/core and collect them"): > Refactor the $kvp_replace helper in ts-xen-install into a generic > helper (which requires using ::EO and ::EI for namespacing) for use > with target_editfile and use it to edit /etc/sys

Re: [Xen-devel] [PATCH v10 3/6] Support for BIOS interrupt handler

2015-04-21 Thread Kevin O'Connor
On Mon, Mar 30, 2015 at 05:09:47AM +, Xu, Quan wrote: > > From: Stefan Berger [mailto:stef...@linux.vnet.ibm.com] > > On 03/27/2015 03:58 AM, Xu, Quan wrote: > > >> From: Xu, Quan > > >>> From: Stefan Berger [mailto:stef...@linux.vnet.ibm.com] > > >>> On 03/26/2015 07:01 AM, Xu, Quan wrote: > >

Re: [Xen-devel] [RFC PATCH] x86/asm/irq: Don't use POPF but STI

2015-04-21 Thread Ingo Molnar
* Borislav Petkov wrote: > On Tue, Apr 21, 2015 at 02:45:58PM +0200, Ingo Molnar wrote: > > From 6f01f6381e8293c360b7a89f516b8605e357d563 Mon Sep 17 00:00:00 2001 > > From: Ingo Molnar > > Date: Tue, 21 Apr 2015 13:32:13 +0200 > > Subject: [PATCH] x86/asm/irq: Don't use POPF but STI > > > > So

[Xen-devel] [PATCH v3] xen: sched_rt: print useful affinity info when dumping

2015-04-21 Thread Dario Faggioli
In fact, printing the cpupool's CPU online mask for each vCPU is just redundant, as that is the same for all the vCPUs of all the domains in the same cpupool, while hard affinity is already part of the output of dumping domains info. Instead, print the intersection between hard affinity and online

Re: [Xen-devel] [PATCH OSSTEST] ts-host-install: Arrange for ssh logins to have no corefile size limit

2015-04-21 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] ts-host-install: Arrange for ssh logins to have no corefile size limit"): > Collect the output of "cat /proc/self/limits" so we get some clue if > this isn't working for some reason. Acked-by: Ian Jackson ___ Xen-

Re: [Xen-devel] [PATCH OSSTEST] ts-libvirt-build: set coredump ulimit to unlimited for libvirtd

2015-04-21 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] ts-libvirt-build: set coredump ulimit to unlimited for libvirtd"): > Signed-off-by: Ian Campbell Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Tamas K Lengyel
On Tue, Apr 21, 2015 at 5:13 PM, Jan Beulich wrote: > >>> On 21.04.15 at 16:42, wrote: > > On Tue, Apr 21, 2015 at 4:29 PM, Jan Beulich wrote: > > > >> >>> On 21.04.15 at 16:24, wrote: > >> > On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: > >> >> >>> On 21.04.15 at 15:23, wrote: > >> >

Re: [Xen-devel] [PATCH 2/5] raisin: remove duplicate source config in raise

2015-04-21 Thread George Dunlap
On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini I take it this got here by a bad merge or somethign? Acked-by: George Dunlap > --- > raise | 15 +-- > 1 file changed, 5 insertions(+), 10 deletions(-) > > diff --git a/raise b/raise > index b

Re: [Xen-devel] [PATCH v5 12/13] tools: add tools support for Intel CAT

2015-04-21 Thread Dario Faggioli
On Tue, 2015-04-21 at 17:49 +0800, Chao Peng wrote: > On Tue, Apr 21, 2015 at 03:24:37AM +0200, Dario Faggioli wrote: > > On Fri, 2015-04-17 at 22:33 +0800, Chao Peng wrote: > > > This is the xc/xl changes to support Intel Cache Allocation > > > Technology(CAT). Two commands are introduced: > > > -

Re: [Xen-devel] [PATCH OSSTEST] ts-xen-*: Avoid use of uninitialised $r{"enable_xsm"}

2015-04-21 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] ts-xen-*: Avoid use of uninitialised $r{"enable_xsm"}"): > Use of uninitialized value $r{"enable_xsm"} in pattern match (m//) at > ./ts-xen-install line 49. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-d

Re: [Xen-devel] [PATCH 5/5] raisin: build linux

2015-04-21 Thread George Dunlap
On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > Add a component, disabled by default, to build a linux kernel with the > Xen kconfig options enabled. > > Signed-off-by: Stefano Stabellini Acked-by: George Dunlap I'm wondering if at some point we might want to give people the option to pass

Re: [Xen-devel] [PATCH 3/5] raisin: rename ARCH to RAISIN_ARCH

2015-04-21 Thread George Dunlap
On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > Rename exported environmental variable ARCH to RAISIN_ARCH not to > conflict with any component Makefile variables. > > Signed-off-by: Stefano Stabellini Acked-by: George Dunlap > --- > README |2 +- > components/gru

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 16:42, wrote: > On Tue, Apr 21, 2015 at 4:29 PM, Jan Beulich wrote: > >> >>> On 21.04.15 at 16:24, wrote: >> > On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: >> >> >>> On 21.04.15 at 15:23, wrote: >> >> > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: >> >> >

Re: [Xen-devel] [PATCH 1/5] raisin: introduce _verbose_echo

2015-04-21 Thread George Dunlap
On 04/21/2015 03:54 PM, Stefano Stabellini wrote: > A new utility function to make the code more readable and compact: > prints a message if VERBOSE = 1. > > Signed-off-by: Stefano Stabellini > --- > lib/common-functions.sh | 46 -- > 1 file changed,

Re: [Xen-devel] tg3 NIC driver bug in 3.14.x under Xen [and 3 more messages]

2015-04-21 Thread Ian Jackson
Prashant Sreedharan writes ("Re: tg3 NIC driver bug in 3.14.x under Xen [and 3 more messages]"): > On Fri, 2015-04-17 at 15:12 -0400, David Miller wrote: > > From: Konrad Rzeszutek Wilk > > Date: Fri, 17 Apr 2015 15:04:48 -0400 > > > A huge amount of NIC drivers use the DMA API, however if > > >

Re: [Xen-devel] [PATCH 4/7] docs/build: Do not use move-if-changed

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 16:00 +0100, Andrew Cooper wrote: > On 21/04/15 15:12, Ian Campbell wrote: > > On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > >> Nothing expensive depends on these results. > > But are those uses a problem? > > Less fluff in the build logs, and less load moving fil

Re: [Xen-devel] [PATCH 4/5] raisin: introduce ovmf

2015-04-21 Thread George Dunlap
On 04/21/2015 03:55 PM, Stefano Stabellini wrote: > diff --git a/components/series b/components/series > index f0f3cfa..fe9092a 100644 > --- a/components/series > +++ b/components/series > @@ -1,4 +1,5 @@ > seabios > +ovmf > xen > qemu > qemu_traditional > diff --git a/components/xen b/componen

Re: [Xen-devel] [PATCH 4/7] docs/build: Do not use move-if-changed

2015-04-21 Thread Andrew Cooper
On 21/04/15 15:12, Ian Campbell wrote: > On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: >> Nothing expensive depends on these results. > But are those uses a problem? Less fluff in the build logs, and less load moving files around the filesystem. ~Andrew

[Xen-devel] [PATCH 2/5] raisin: remove duplicate source config in raise

2015-04-21 Thread Stefano Stabellini
Signed-off-by: Stefano Stabellini --- raise | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/raise b/raise index bce6908..68dbfd8 100755 --- a/raise +++ b/raise @@ -17,9 +17,12 @@ _help() { } # Include your defaults -if [[ -e "./config" ]] ; then -. .

[Xen-devel] [PATCH 3/5] raisin: rename ARCH to RAISIN_ARCH

2015-04-21 Thread Stefano Stabellini
Rename exported environmental variable ARCH to RAISIN_ARCH not to conflict with any component Makefile variables. Signed-off-by: Stefano Stabellini --- README |2 +- components/grub |6 +++--- components/libvirt |2 +- components/qemu

[Xen-devel] [PATCH 1/5] raisin: introduce _verbose_echo

2015-04-21 Thread Stefano Stabellini
A new utility function to make the code more readable and compact: prints a message if VERBOSE = 1. Signed-off-by: Stefano Stabellini --- lib/common-functions.sh | 46 -- 1 file changed, 16 insertions(+), 30 deletions(-) diff --git a/lib/common-func

[Xen-devel] [PATCH 5/5] raisin: build linux

2015-04-21 Thread Stefano Stabellini
Add a component, disabled by default, to build a linux kernel with the Xen kconfig options enabled. Signed-off-by: Stefano Stabellini --- components/linux | 120 + components/series |1 + defconfig |4 +- 3 files changed, 124

Re: [Xen-devel] [PATCH 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Andrew Cooper
On 21/04/15 15:23, Ian Campbell wrote: > On Tue, 2015-04-21 at 15:18 +0100, Jan Beulich wrote: > On 21.04.15 at 16:15, wrote: >>> On Mon, 2015-04-20 at 13:01 +0100, Andrew Cooper wrote: On 20/04/15 12:57, Jan Beulich wrote: On 20.04.15 at 12:49, wrote: >> @@ -103,12 +88,20 @

[Xen-devel] [PATCH 4/5] raisin: introduce ovmf

2015-04-21 Thread Stefano Stabellini
Add a component to build ovmf and pass the output binary to the xen build. Signed-off-by: Stefano Stabellini --- components/ovmf | 60 + components/series |1 + components/xen|3 ++- defconfig |6 -- 4 files chan

[Xen-devel] [PATCH 0/5] raisin: introduce ovmf and linux

2015-04-21 Thread Stefano Stabellini
Hi all, this patch series includes a few clean-ups and introduces a component to build ovmf and a component to build a linux kernel. The latter is not enabled by default, and could probably benefit from a bit more work on the kconfig options to really be useful, but it is a start. Stefano Stabel

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 15:29 +0100, Jan Beulich wrote: > >>> On 21.04.15 at 16:24, wrote: > > On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: > >> >>> On 21.04.15 at 15:23, wrote: > >> > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: > >> >> On 20/04/15 16:06, Tamas K Lengyel wrote:

Re: [Xen-devel] [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 15:30 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in > TestSupport.pm for nested test"): > > On Tue, 2015-04-21 at 14:28 +0100, Ian Jackson wrote: > > > I'm only objecting to the involvement of the host property machinery

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Tamas K Lengyel
On Tue, Apr 21, 2015 at 4:29 PM, Jan Beulich wrote: > >>> On 21.04.15 at 16:24, wrote: > > On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: > >> >>> On 21.04.15 at 15:23, wrote: > >> > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: > >> >> On 20/04/15 16:06, Tamas K Lengyel wrote:

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 16:33, wrote: > On Tue, Apr 21, 2015 at 4:14 PM, Jan Beulich wrote: > >> >>> On 21.04.15 at 15:23, wrote: >> > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: >> >> On 20/04/15 16:06, Tamas K Lengyel wrote: >> >> > The current implementation of three memops, >> XENMEM

Re: [Xen-devel] [PATCH v5 12/13] tools: add tools support for Intel CAT

2015-04-21 Thread Dario Faggioli
On Tue, 2015-04-21 at 15:01 +0100, Ian Campbell wrote: > On Tue, 2015-04-21 at 17:49 +0800, Chao Peng wrote: > > I do like this suggestion and I have ever considered it actually. The > > only thing prevents me is that we need an extra _get_socket_count in xl > > for TARGET_ALL case. So libxl__coun

Re: [Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-04-21 Thread Olaf Hering
On Tue, Apr 21, Konrad Rzeszutek Wilk wrote: > On Fri, Apr 17, 2015 at 08:30:56AM +, Olaf Hering wrote: > > The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev > > property. Add the missing :LUN part to the comment. > > What about the older kernels that had said driver? W

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Tamas K Lengyel
On Tue, Apr 21, 2015 at 4:14 PM, Jan Beulich wrote: > >>> On 21.04.15 at 15:23, wrote: > > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: > >> On 20/04/15 16:06, Tamas K Lengyel wrote: > >> > The current implementation of three memops, > XENMEM_current_reservation, > >> > XENMEM_maximum

Re: [Xen-devel] [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test

2015-04-21 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST Nested PATCH v8 3/7] Edit some APIs in TestSupport.pm for nested test"): > On Tue, 2015-04-21 at 14:28 +0100, Ian Jackson wrote: > > I'm only objecting to the involvement of the host property machinery. > > I'm afraid I'm not 100% sure I understand, but do you j

Re: [Xen-devel] [PATCH 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 15:18 +0100, Jan Beulich wrote: > >>> On 21.04.15 at 16:15, wrote: > > On Mon, 2015-04-20 at 13:01 +0100, Andrew Cooper wrote: > >> On 20/04/15 12:57, Jan Beulich wrote: > >> On 20.04.15 at 12:49, wrote: > >> >> @@ -103,12 +88,20 @@ install: install-man-pages install-ht

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 16:24, wrote: > On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: >> >>> On 21.04.15 at 15:23, wrote: >> > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: >> >> On 20/04/15 16:06, Tamas K Lengyel wrote: >> >> > The current implementation of three memops, XENMEM_curr

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 15:14 +0100, Jan Beulich wrote: > >>> On 21.04.15 at 15:23, wrote: > > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: > >> On 20/04/15 16:06, Tamas K Lengyel wrote: > >> > The current implementation of three memops, XENMEM_current_reservation, > >> > XENMEM_maximum_r

Re: [Xen-devel] [PATCH V15 2/9] xen/arm: Allow hypervisor access to mem_access protected pages

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 16:10 +0200, Tamas K Lengyel wrote: > > > On Tue, Apr 21, 2015 at 3:14 PM, Ian Campbell > wrote: > On Mon, 2015-04-20 at 17:06 +0200, Tamas K Lengyel wrote: > > -paddr_t p2m_lookup(struct domain *d, paddr_t paddr, > p2m_type_t *t) > > +static

Re: [Xen-devel] [PATCH 4/7] docs/build: Do not use move-if-changed

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > Nothing expensive depends on these results. But are those uses a problem? > > Also prefer $(INSTALL_DATA) over cp to get correct file attributes (see > fb33b2b "docs: make .txt files over-writable when building from r/o sources") > > Sig

Re: [Xen-devel] [PATCH 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 16:15, wrote: > On Mon, 2015-04-20 at 13:01 +0100, Andrew Cooper wrote: >> On 20/04/15 12:57, Jan Beulich wrote: >> On 20.04.15 at 12:49, wrote: >> >> @@ -103,12 +88,20 @@ install: install-man-pages install-html >> >> >> >> # Individual file build targets >> >> man1/%.

Re: [Xen-devel] [PATCH 7/7] docs/build: Support generation of pandoc documents

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > pandoc is a superset of markdown > > Signed-off-by: Andrew Cooper Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 6/7] docs/build: Move install checks into individual build targets

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 13:01 +0100, Andrew Cooper wrote: > On 20/04/15 12:57, Jan Beulich wrote: > On 20.04.15 at 12:49, wrote: > >> @@ -103,12 +88,20 @@ install: install-man-pages install-html > >> > >> # Individual file build targets > >> man1/%.1: man/%.pod.1 Makefile > >> +ifdef POD2MA

Re: [Xen-devel] [PATCH V15 5/9] xen: Make gpfn related memops compatible with wider return values

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 15:23, wrote: > On Mon, 2015-04-20 at 16:22 +0100, Andrew Cooper wrote: >> On 20/04/15 16:06, Tamas K Lengyel wrote: >> > The current implementation of three memops, XENMEM_current_reservation, >> > XENMEM_maximum_reservation and XENMEM_maximum_gpfn return values as an >> > int.

Re: [Xen-devel] [PATCH 5/7] docs/build: Do not create directories if we are not going to use them

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > and be quite about doing so; these are only intermediate directories. > > No practical change, but the build log is roughly halved. > > Signed-off-by: Andrew Cooper Acked-by: Ian Campbell _

Re: [Xen-devel] [PATCH V15 8/9] tools/tests: Enable xen-access on ARM

2015-04-21 Thread Tamas K Lengyel
On Tue, Apr 21, 2015 at 3:28 PM, Ian Campbell wrote: > On Mon, 2015-04-20 at 17:06 +0200, Tamas K Lengyel wrote: > > Did this patch go away for a bit? It wasn't in v14 and the last on I > seem to have was v11? (I don't mind, just surprised not to see a bunch > of versions) > Yes, this one patch

Re: [Xen-devel] [PATCH 3/7] docs/build: Move two rules for consistency, and comment sections

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > No functional change. > > Signed-off-by: Andrew Cooper Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 2/7] docs/build: Do not open-code $*

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > Sometimes there is already a round enough wheel to hand. I love the double use of sed and the unescaped .'s. > > No functional change. > > Signed-off-by: Andrew Cooper Acked-by: Ian Campbell (I agree with the conclusion of the subthre

Re: [Xen-devel] [PATCH 1/7] docs/build: Misc cleanup

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 11:49 +0100, Andrew Cooper wrote: > * Use $(PANDOC) from ./configure > * Swap '-N' for its less-obscure longer form > * Don't explicitly echo about markdown. The call to markdown is emitted > * Whitespace cleanup > > Signed-off-by: Andrew Cooper Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH V15 2/9] xen/arm: Allow hypervisor access to mem_access protected pages

2015-04-21 Thread Tamas K Lengyel
On Tue, Apr 21, 2015 at 3:14 PM, Ian Campbell wrote: > On Mon, 2015-04-20 at 17:06 +0200, Tamas K Lengyel wrote: > > -paddr_t p2m_lookup(struct domain *d, paddr_t paddr, p2m_type_t *t) > > +static paddr_t __p2m_lookup(struct domain *d, paddr_t paddr, p2m_type_t > *t) > > I'd have been inclined to

Re: [Xen-devel] [PATCH v5 13/13] docs: add xl-psr.markdown

2015-04-21 Thread Ian Campbell
On Fri, 2015-04-17 at 22:33 +0800, Chao Peng wrote: > @@ -1534,7 +1540,8 @@ Show CAT hardware information. > > =item B [I] I I > > -Set cache capacity bitmasks(CBM) for a domain. > +Set cache capacity bitmasks(CBM) for a domain. For how to specify I > +please refer to the link above. I think

Re: [Xen-devel] [PATCH v4 3/5] libxl: add support for vscsi

2015-04-21 Thread Konrad Rzeszutek Wilk
On Fri, Apr 17, 2015 at 08:30:58AM +, Olaf Hering wrote: > Port pvscsi support from xend to libxl: > > vscsi=['pdev,vdev{,options}'] > xl scsi-attach > xl scsi-detach > xl scsi-list > > Signed-off-by: Olaf Hering > Cc: Ian Jackson > Cc: Stefano Stabellini > Cc: Ian Campbell > Cc: Wei

Re: [Xen-devel] [PATCH v20 02/13] x86/VPMU: Add public xenpmu.h

2015-04-21 Thread Jan Beulich
>>> On 21.04.15 at 15:38, wrote: > On 04/21/2015 03:53 AM, Jan Beulich wrote: > On 20.04.15 at 18:38, wrote: >>> On 04/20/2015 04:50 AM, Jan Beulich wrote: >>> On 09.04.15 at 17:44, wrote: > --- /dev/null > +++ b/xen/include/public/pmu.h > @@ -0,0 +1,38 @@ > +#ifndef __X

Re: [Xen-devel] [PATCH v5 12/13] tools: add tools support for Intel CAT

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 17:49 +0800, Chao Peng wrote: > On Tue, Apr 21, 2015 at 03:24:37AM +0200, Dario Faggioli wrote: > > On Fri, 2015-04-17 at 22:33 +0800, Chao Peng wrote: > > > This is the xc/xl changes to support Intel Cache Allocation > > > Technology(CAT). Two commands are introduced: > > > -

Re: [Xen-devel] effect of ticks in xen credit scheduler

2015-04-21 Thread George Dunlap
On Tue, Apr 21, 2015 at 10:48 AM, teabe wrote: > Hi, > I am student starting to work with Xen. I am using Xen 4.2.0. Out of curiosity, why are you looking at 4.2.0 instead of 4.5, or unstable? > I am trying to > understand the functioning of the credit scheduler in Xen. I try to modify > the

Re: [Xen-devel] [PATCH v5 11/13] tools/libxl: add command to show CMT hardware info

2015-04-21 Thread Ian Campbell
On Tue, 2015-04-21 at 02:37 +0200, Dario Faggioli wrote: > On Fri, 2015-04-17 at 22:33 +0800, Chao Peng wrote: > > Add dedicated one to show hardware information. > > > > [root@vmm-psr]xl psr-cmt-hwinfo > > Cache Monitoring Technology (CMT): > > Enabled : 1 > > Total RMID : 63 > > Sup

Re: [Xen-devel] [PATCH v5 10/13] tools/libxl: minor name changes for CMT commands

2015-04-21 Thread Ian Campbell
On Mon, 2015-04-20 at 18:07 +0200, Dario Faggioli wrote: > On Fri, 2015-04-17 at 22:33 +0800, Chao Peng wrote: > > Use "-" instead of "_" for monitor types. > > > This easier to type, so I like it. :-) > > > Signed-off-by: Chao Peng > > > Reviewed-by: Dario Faggioli Acked-by: Ian Campbell

Re: [Xen-devel] [PATCH v4 1/5] vscsiif.h: fix WWN notation for p-dev property

2015-04-21 Thread Konrad Rzeszutek Wilk
On Fri, Apr 17, 2015 at 08:30:56AM +, Olaf Hering wrote: > The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev > property. Add the missing :LUN part to the comment. What about the older kernels that had said driver? > > Signed-off-by: Olaf Hering > Cc: Ian Campbell > Cc

Re: [Xen-devel] [PATCH] Config.mk: Fix (and, effectively, update) QEMU_TAG

2015-04-21 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH] Config.mk: Fix (and, effectively, update) QEMU_TAG"): > On 21.04.15 at 12:33, wrote: > > In 952944f7 "QEMU_TAG update" my tag update script mangled the > > machinery which sets QEMU_TRADITIONAL_REVISION, by replacing the first > > assignment to QEMU_TRADITIONAL_RE

  1   2   >