Re: [linux-sunxi] [PATCH 2/2] ARM: sun6i: Add SMP support for the Allwinner A31

2013-11-11 Thread Ian Campbell
On Sun, 2013-11-10 at 11:03 +0100, Maxime Ripard wrote: > Hi Ian, > > On Fri, Nov 08, 2013 at 10:25:55AM +0000, Ian Campbell wrote: > > On Fri, 2013-11-08 at 09:40 +0100, Maxime Ripard wrote: > > > > I'm trying to work out if we can make this work with the requir

Re: [PATCH] xen/arm: pfn_to_mfn and mfn_to_pfn return the argument if nothing is in the p2m

2013-11-11 Thread Ian Campbell
cking a pfn or what is the pfn corresponding to an mfn. > Fix the regression by switching to the old behavior. > > Signed-off-by: Stefano Stabellini Acked-by: Ian Campbell I tripped over this on Friday and my local hack was identical, hence also: Tested-by: Ian Campbell Ian. -- To u

Re: [linux-sunxi] [PATCH 2/2] ARM: sun6i: Add SMP support for the Allwinner A31

2013-11-08 Thread Ian Campbell
On Fri, 2013-11-08 at 09:40 +0100, Maxime Ripard wrote: > > I'm trying to work out if we can make this work with the requirement > > which both Xen and KVM have to enter the kernel in NS-HYP mode. > > > > The way this works on e.g. vexpress is (roughly) that u-boot wakes up > > the secondary CPUs

Re: [PATCH 3.8 23/91] xen-netback: Don't destroy the netdev until the vif is shut down

2013-11-08 Thread Ian Campbell
On Thu, 2013-11-07 at 18:14 -0800, Kamal Mostafa wrote: > 3.8.13.13 -stable review patch. If anyone has any objections, please let me > know. This patch caused a regression in the 3.10 stable tree. Two additional patches were needed, see: http://www.spinics.net/lists/netdev/msg256637.htm

Re: [PATCH v9 03/18] arm/xen,arm64/xen: introduce p2m

2013-11-07 Thread Ian Campbell
On Fri, 2013-10-25 at 11:51 +0100, Stefano Stabellini wrote: > Introduce physical to machine and machine to physical tracking > mechanisms based on rbtrees for arm/xen and arm64/xen. > > We need it because any guests on ARM are an autotranslate guests, > therefore a physical address is potentially

Re: [PATCH net-next RFC 0/5] xen-netback: TX grant mapping instead of copy

2013-11-07 Thread Ian Campbell
On Fri, 2013-11-01 at 19:00 +, Zoltan Kiss wrote: > On 01/11/13 10:50, Ian Campbell wrote: > > Does this always avoid copying when bridging/openvswitching/forwarding > > (e.g. masquerading etc)? For both domU->domU and domU->physical NIC? > I've tested the do

Re: [Xen-devel] [PATCH] grant-table: don't set m2p override if kmap_ops is not set

2013-11-06 Thread Ian Campbell
On Tue, 2013-11-05 at 12:53 -0800, Anthony Liguori wrote: > >> Yes, you are completely right, then I have to figure out why blkback > >> works fine with this patch applied (or at least it seems to work fine). > > > > blkback also works for me when testing a similar patch. I'm still > > confused. On

Re: [Xen-devel] [PATCH] grant-table: don't set m2p override if kmap_ops is not set

2013-11-05 Thread Ian Campbell
On Tue, 2013-11-05 at 16:01 +0100, Roger Pau Monné wrote: > On 05/11/13 15:56, Konrad Rzeszutek Wilk wrote: > > On Tue, Nov 05, 2013 at 03:47:08PM +0100, Roger Pau Monné wrote: > >> On 05/11/13 13:36, David Vrabel wrote: > >>> On 05/11/13 11:24, Roger Pau Monne wrote: > IMHO there's no reason

Re: [linux-sunxi] [PATCH 2/2] ARM: sun6i: Add SMP support for the Allwinner A31

2013-11-04 Thread Ian Campbell
Not a comment on the patch, or even A31, but a question about how the SMP boot process works on sunxi generally... On Sun, 2013-11-03 at 10:30 +0100, Maxime Ripard wrote: > The A31 is a quad Cortex-A7. Add the logic to use the IPs used to > control the CPU configuration and the CPU power so that w

Re: [Xen-devel] [PATCH v3] p2m: use GNTTABOP_unmap_and_duplicate if available

2013-11-04 Thread Ian Campbell
On Mon, 2013-11-04 at 17:09 +0100, Roger Pau Monné wrote: > On 04/11/13 16:49, Ian Campbell wrote: > > On Mon, 2013-11-04 at 16:38 +0100, Roger Pau Monne wrote: > >> The new GNTTABOP_unmap_and_duplicate operation > > > > I don't see this op in mainline Xen

Re: [Xen-devel] [PATCH v3] p2m: use GNTTABOP_unmap_and_duplicate if available

2013-11-04 Thread Ian Campbell
On Mon, 2013-11-04 at 16:38 +0100, Roger Pau Monne wrote: > The new GNTTABOP_unmap_and_duplicate operation I don't see this op in mainline Xen anywhere... Was it part of Stefano's original swiotlb for ARM stuff? If so we've dropped that approach for ARM and the new solution doesn't require hyper

Re: [PATCH net-next RFC 0/5] xen-netback: TX grant mapping instead of copy

2013-11-01 Thread Ian Campbell
On Wed, 2013-10-30 at 00:50 +, Zoltan Kiss wrote: > This patch series use SKBTX_DEV_ZEROCOPY flags to tell the stack it needs to > know when the skb is freed up. Does this always avoid copying when bridging/openvswitching/forwarding (e.g. masquerading etc)? For both domU->domU and domU->physic

Re: [Xen-devel] [PATCH] xen/hvc-console: Make it work with HVM guests.

2013-10-26 Thread Ian Campbell
On Fri, 2013-10-25 at 15:38 -0400, Konrad Rzeszutek Wilk wrote: > On Thu, Oct 24, 2013 at 05:43:54PM +0100, Ian Campbell wrote: > > But stepping back how/why is this function called on x86 when not > > running under Xen? Wouldn't the use have to be using console=hvc or &g

Re: [Xen-devel] [PATCH] xen/hvc-console: Make it work with HVM guests.

2013-10-24 Thread Ian Campbell
On Thu, 2013-10-24 at 17:30 +0100, Julien Grall wrote: > I'm sure the solution "move xen call earlier" will come up in few mails, > so there was a discussion for swiotlb few weeks ago > (https://lkml.org/lkml/2013/8/29/609). And the final decision was to > avoid specific Xen call in arch_setup.

Re: [Xen-devel] EFI and multiboot2 devlopment work for Xen

2013-10-23 Thread Ian Campbell
On Tue, 2013-10-22 at 12:26 -0400, Konrad Rzeszutek Wilk wrote: > It can (at least in Linux). There are two entry points in the Linux kernel > and - one when it is launched from 'linuxefi' (See efi_stub_entry in > arch/x86/boot/compressed/head_64.S), the other when it is launched > from an EFI she

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 18:25 +0200, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 22.10.2013 18:14, Daniel Kiper wrote: > >> > Are you (going to be) in Edinburgh? Matthew was just explaining a bunch > >> > of this stuff to me, it might be useful for you to get it from the > >> > horses mouth ins

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 12:24 -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Oct 22, 2013 at 04:21:47PM +0100, Ian Campbell wrote: > > On Tue, 2013-10-22 at 10:57 -0400, Konrad Rzeszutek Wilk wrote: > > > That 'that' is a standard PE/COFF image? Could you please point me

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 18:01 +0200, Daniel Kiper wrote: > On Tue, Oct 22, 2013 at 03:42:42PM +, Woodhouse, David wrote: > > On Tue, 2013-10-22 at 16:32 +0100, Matthew Garrett wrote: > > > > > > There are two problems with this: > > > > > > 1) The kernel will only boot if it's signed with a key i

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 14:18 +, Woodhouse, David wrote: > > I wonder why Linux can't make the EFI calls to fetch them itself? > > It can. It does. It prefers to. This is what the "EFI boot stub" is all about. Good, this is what I thought, glad to see I'm not talking out my behind for once! >

Re: [Xen-devel] EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 15:24 +0100, Ian Campbell wrote: > On Tue, 2013-10-22 at 10:09 -0400, Konrad Rzeszutek Wilk wrote: > > > So it can be booted the same way as xen.efi. But my understanding is > > that folks prefer a bootloader instead of loading the bzImage in an > >

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 10:57 -0400, Konrad Rzeszutek Wilk wrote: > That 'that' is a standard PE/COFF image? Could you please point me > to the code that does that in GRUB2? As I said earlier in the thread, it's a patch which is being carried by all the distros. It is not in upstream grub. For ins

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 10:09 -0400, Konrad Rzeszutek Wilk wrote: > So it can be booted the same way as xen.efi. But my understanding is > that folks prefer a bootloader instead of loading the bzImage in an > NVRAM of a platform with pre-set parameters. Hence that mechanism > is not used by the majo

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 09:42 -0400, Konrad Rzeszutek Wilk wrote: > Looking at the Fedora GRUB2 source, the 'struct linux_kernel_header' is > defined > in the linux/Documentation/x86/boot.txt and hpa is pretty strict > about making it backwards compatible. It also seems to support Xen! > > (Intere

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Tue, 2013-10-22 at 10:31 +0100, Jan Beulich wrote: > >>> On 22.10.13 at 11:26, Ian Campbell wrote: > > AIUI "efilinux" is somewhat badly named and does not use the Linux Boot > > Protocol (i.e. the (b)zImage stuff with real mode entry point) either. >

Re: EFI and multiboot2 devlopment work for Xen

2013-10-22 Thread Ian Campbell
On Mon, 2013-10-21 at 20:57 +0200, Daniel Kiper wrote: > On Mon, Oct 21, 2013 at 09:54:38AM -0400, Peter Jones wrote: > > On Mon, Oct 21, 2013 at 02:57:56PM +0200, Daniel Kiper wrote: > > > Hi, > > > > > > During work on multiboot2 protocol support for Xen it was discovered > > > that memory map pa

Re: [PATCH] powerpc/dts/virtex440: declare address/size-cells for phy device

2013-09-24 Thread Ian Campbell
On Mon, 2013-06-03 at 12:00 +0100, Ian Campbell wrote: > This fixes a warning: > > DTC arch/powerpc/boot/virtex440-ml507.dtb > Warning (reg_format): "reg" property in > /plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7 has invalid length (4 > bytes) (#add

Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-09-24 Thread Ian Campbell
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote: > On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote: > >Currently I see: > > DTC arch/powerpc/boot/kilauea.dtb > >Warning (reg_format): "reg" property in /plb/ppc4xx-msi@C1000 has > >inva

Re: [PATCH 1/3] Add smp support for Allwinner A20(sunxi 7i).

2013-09-23 Thread Ian Campbell
On Sun, 2013-09-22 at 23:31 +0800, cinifr wrote: > > > On Sunday, 22 September 2013, Ian Campbell wrote: > > On Sun, 2013-09-22 at 20:21 +0800, Fan Rong wrote: > > > >> + /* Set boot addr */ > >> + paddr = virt_to_phys(sun7i_secondary_startup); &

Re: [linux-sunxi] [PATCH V3: Add Smp support for Allwinner A20. 1/3] Add smp support for Allwinner A20(sunxi 7i).

2013-09-22 Thread Ian Campbell
On Sun, 2013-09-22 at 20:21 +0800, Fan Rong wrote: > + /* Set boot addr */ > + paddr = virt_to_phys(sun7i_secondary_startup); > + writel(paddr, sunxi7i_cc_base + SUN7I_CPUCFG_BOOTADDR); This means that the secondary cores will miss out on any setup which the bootloader might have done

Re: [linux-sunxi] Re: [PATCH 3/4] Add physical count arch timer support for clocksource in ARMv7.

2013-09-12 Thread Ian Campbell
On Thu, 2013-09-12 at 12:24 +0100, Mark Rutland wrote: > > Why can the bootloader not be fixed to set CNTVOFF (or to boot the > kernel in Hyp mode)? It can: http://lists.xen.org/archives/html/xen-devel/2013-09/msg00880.html I was waiting for Andre's hyp mode u-boot patches to land before postin

Re: [linux-sunxi] Re: [PATCH 4/4] Add arch count timer node in dts for Allwinner A20(sunxi 7i).

2013-09-12 Thread Ian Campbell
On Thu, 2013-09-12 at 15:57 +0100, Mark Rutland wrote: > > + interrupts = <1 13 0x308>, > > +<1 14 0x308>, > > +<1 11 0x308>, > > +<1 10 0x308>; > > + clock-frequency

Re: [PATCH] xen-netback: fix possible format string flaw

2013-09-11 Thread Ian Campbell
On Tue, 2013-09-10 at 21:39 -0700, Kees Cook wrote: > This makes sure a format string cannot accidentally leak into the > kthread_run() call. > > Signed-off-by: Kees Cook Acked-by: Ian Campbell Thanks. Ian. -- To unsubscribe from this list: send the line "unsubscribe lin

Re: [PATCH v5 05/13] xen/arm,arm64: move Xen initialization earlier

2013-09-06 Thread Ian Campbell
On Thu, 2013-09-05 at 17:59 +0100, Stefano Stabellini wrote: > On Thu, 5 Sep 2013, Catalin Marinas wrote: > > On Thu, Aug 29, 2013 at 07:32:26PM +0100, Stefano Stabellini wrote: > > > Move Xen initialization earlier, before any DMA requests can be made. > > > > > > Signed-off-by: Stefano Stabellin

Re: [Xen-devel] [PATCH] net/core: Order-3 frag allocator causes SWIOTLB bouncing under Xen

2013-09-05 Thread Ian Campbell
On Wed, 2013-09-04 at 17:11 -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Sep 04, 2013 at 02:00:40PM -0700, Eric Dumazet wrote: > > Maybe you could add proper infrastructure to deal with Xen limitations. > > I think Ian posted at some point an sysctl patch for that (more for > debugging that anyt

[PATCH] MAINTAINERS: change my DT related maintainer address

2013-08-30 Thread Ian Campbell
Filtering capabilities on my work email are pretty much non-existent and this has turned out to be something of a firehose... Cc: Pawel Moll Cc: Mark Rutland Cc: Stephen Warren Cc: Rob Herring Cc: Olof Johansson Cc: Linus Walleij Signed-off-by: Ian Campbell --- MAINTAINERS |2 +- 1

Re: [PATCH v4 10/10] xen/arm,arm64: enable SWIOTLB_XEN

2013-08-29 Thread Ian Campbell
On Wed, 2013-08-28 at 21:07 +0100, Stefano Stabellini wrote: > On Thu, 15 Aug 2013, Ian Campbell wrote: > > On Thu, 2013-08-15 at 12:10 +0100, Stefano Stabellini wrote: > > > At the moment always rely on swiotlb-xen, but when Xen starts supporting > > > hardware IOMMU

[PATCH] MAINTAINERS: change my DT related maintainer address

2013-08-27 Thread Ian Campbell
Filtering capabilities on my work email are pretty much non-existent and this has turned out to be something of a firehose... Cc: Pawel Moll Cc: Mark Rutland Cc: Stephen Warren Cc: Rob Herring Cc: Olof Johansson Cc: Linus Walleij Signed-off-by: Ian Campbell --- MAINTAINERS |2 +- 1

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-22 Thread Ian Campbell
On Wed, 2013-08-21 at 06:44 -0400, Konrad Rzeszutek Wilk wrote: > Ian Campbell wrote: > >On Tue, 2013-08-20 at 15:35 -0400, Konrad Rzeszutek Wilk wrote: > >> void xen_raw_console_write(const char *str) > >> { > >> - dom0_write_console(0, str, strlen(str

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-21 Thread Ian Campbell
On Tue, 2013-08-20 at 15:35 -0400, Konrad Rzeszutek Wilk wrote: > void xen_raw_console_write(const char *str) > { > - dom0_write_console(0, str, strlen(str)); > + ssize_t len = strlen(str); > + int rc = 0; > + > + if (xen_domain()) { > + rc = dom0_write_console(0, str,

Re: [PATCH v4 10/10] xen/arm,arm64: enable SWIOTLB_XEN

2013-08-15 Thread Ian Campbell
On Thu, 2013-08-15 at 12:10 +0100, Stefano Stabellini wrote: > At the moment always rely on swiotlb-xen, but when Xen starts supporting > hardware IOMMUs we'll be able to avoid it conditionally on the presence > of an IOMMU on the platform. Do we have any idea how we are going to do this? It's ex

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-15 Thread Ian Campbell
On Wed, 2013-08-14 at 08:46 -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Aug 14, 2013 at 08:15:56AM +0100, Ian Campbell wrote: > > On Tue, 2013-08-13 at 20:10 -0400, Konrad Rzeszutek Wilk wrote: > > > On Tue, Aug 13, 2013 at 11:20:05PM +0100, Ian Campbell wrote: > > >

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-14 Thread Ian Campbell
On Tue, 2013-08-13 at 20:10 -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Aug 13, 2013 at 11:20:05PM +0100, Ian Campbell wrote: > > On Tue, 2013-08-13 at 21:59 +0100, Ian Campbell wrote: > > > > > > What it should be is: > > > > > > > >

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-13 Thread Ian Campbell
On Tue, 2013-08-13 at 21:59 +0100, Ian Campbell wrote: > > What it should be is: > > > > > > > > void xen_raw_console_write(const char *str) > > > > { > > > > - dom0_write_console(0, str, strlen(str)); > > &

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-13 Thread Ian Campbell
On Tue, 2013-08-13 at 16:53 -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Aug 13, 2013 at 09:31:44PM +0100, Ian Campbell wrote: > > On Fri, 2013-08-09 at 10:55 -0400, Konrad Rzeszutek Wilk wrote: > > > The xen_raw_printk works great for debugging purposes and for > > >

Re: [Xen-devel] [PATCH] xen/hvc: If we use xen_raw_printk let it also work on HVM guests.

2013-08-13 Thread Ian Campbell
On Fri, 2013-08-09 at 10:55 -0400, Konrad Rzeszutek Wilk wrote: > The xen_raw_printk works great for debugging purposes and for > it print anything the Xen hypervisor has to be built with 'debug=y'. > > As such there is no difference between a PV or an PVHVM guest > using the hypercall, so lets us

Re: [PATCH 3/3] MAINTAINERS: Refactor device tree maintainership

2013-08-07 Thread Ian Campbell
On Tue, 2013-07-23 at 13:44 -0500, Rob Herring wrote: > On 07/23/2013 01:09 PM, Ian Campbell wrote: > > On Tue, 2013-07-23 at 10:59 -0700, Stephen Warren wrote: > > > >> I think the solution is to introduce some new shared/common location for > >> shared/common *

Re: [Xen-devel] [PATCH 2/3] MAINTAINERS: Update the Xen subsystem's with proper mailing list.

2013-08-05 Thread Ian Campbell
On Mon, 2013-08-05 at 14:05 -0400, Konrad Rzeszutek Wilk wrote: > And also drop the virtualization one since we don't really use it. > > CC: stefano.stabell...@eu.citrix.com > Signed-off-by: Konrad Rzeszutek Wilk Acked-by: Ian Campbell The whole thing but especially: >

Re: [PATCH] powerpc/fsl-booke: Rename b4qds.dts -> b4qds.dtsi.

2013-07-31 Thread Ian Campbell
ping? On Fri, 2013-05-31 at 11:14 +0100, Ian Campbell wrote: > This file is a common include for B4860 and B4420 but is not a valid DTS > itself: > DTC arch/powerpc/boot/b4qds.dtb > Error: arch/powerpc/boot/dts/b4qds.dts:35.1-2 syntax error > FATAL ERROR:

Re: DT bindings as ABI [was: Do we have people interested in device tree janitoring / cleanup?]

2013-07-31 Thread Ian Campbell
On Thu, 2013-07-25 at 18:57 +0100, Mark Rutland wrote: > > A possible way to handle this is to have exactly that: A group of > > people that essentially constitute the "standards committee" that meet > > on a regular basis to review and approve new bindings. They should be > > people not just doing

Re: [linux-sunxi] [PATCH 04/10] pinctrl: sunxi: Wrap long compatible declaration lines

2013-07-30 Thread Ian Campbell
On Tue, 2013-07-30 at 16:59 +0200, Maxime Ripard wrote: > The compatible declaration and their associated data are exceeding the > 80-characters width limit. > > Split these lines to make them more readable and fit in the kernel > coding guidelines. > > Signed-off-by: Maxime Ripard > Cc: Linus W

Re: [PATCH v3 1/2] xen/balloon: set a mapping for ballooned out pages

2013-07-24 Thread Ian Campbell
On Wed, 2013-07-24 at 10:58 -0400, Konrad Rzeszutek Wilk wrote: > On Wed, Jul 24, 2013 at 12:05:05PM +0100, Stefano Stabellini wrote: > > On Tue, 23 Jul 2013, Konrad Rzeszutek Wilk wrote: > > > On Tue, Jul 23, 2013 at 07:00:09PM +0100, Ian Campbell wrote: > > > > On

Re: [PATCH 3/3] MAINTAINERS: Refactor device tree maintainership

2013-07-23 Thread Ian Campbell
On Tue, 2013-07-23 at 13:44 -0500, Rob Herring wrote: > On 07/23/2013 01:09 PM, Ian Campbell wrote: > > On Tue, 2013-07-23 at 10:59 -0700, Stephen Warren wrote: > > > >> I think the solution is to introduce some new shared/common location for > >> shared/common *

Re: [PATCH 3/3] MAINTAINERS: Refactor device tree maintainership

2013-07-23 Thread Ian Campbell
On Tue, 2013-07-23 at 10:59 -0700, Stephen Warren wrote: > I think the solution is to introduce some new shared/common location for > shared/common *.dtsi files, into the kernel tree, in the interim. > > When *.dts move out of the kernel, this common location can simply be > consumed as part of t

Re: [PATCH v3 1/2] xen/balloon: set a mapping for ballooned out pages

2013-07-23 Thread Ian Campbell
On Tue, 2013-07-23 at 18:27 +0100, Stefano Stabellini wrote: > +static int __cpuinit balloon_cpu_notify(struct notifier_block *self, > + unsigned long action, void *hcpu) > +{ > + int cpu = (long)hcpu; > + switch (action) { > + case CPU_UP_PREPARE: > +

Re: [PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-19 Thread Ian Campbell
On Fri, 2013-07-19 at 11:06 +0100, Stefano Stabellini wrote: > On Fri, 19 Jul 2013, Ian Campbell wrote: > > On Fri, 2013-07-19 at 10:52 +0100, Stefano Stabellini wrote: > > > On Fri, 19 Jul 2013, Ian Campbell wrote: > > > > On Thu, 2013-07-18 at 17:15 +0100, Julie

Re: [PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-19 Thread Ian Campbell
On Fri, 2013-07-19 at 10:52 +0100, Stefano Stabellini wrote: > On Fri, 19 Jul 2013, Ian Campbell wrote: > > On Thu, 2013-07-18 at 17:15 +0100, Julien Grall wrote: > > > On 17 July 2013 14:25, Stefano Stabellini > > > wrote: > > > > On Mon, 15 Jul 201

Re: [PATCH] arm: choose debug/uncompress.h include when uncompress debug is disabled

2013-07-19 Thread Ian Campbell
On Thu, 2013-07-18 at 17:15 +0100, Julien Grall wrote: > On 17 July 2013 14:25, Stefano Stabellini > wrote: > > On Mon, 15 Jul 2013, Julien Grall wrote: > >> Even if uncompress debug is disabled, some board will continue to print > >> information during uncompress step. > > > > Are you talking abo

Re: kernel panic in skb_copy_bits

2013-07-04 Thread Ian Campbell
On Thu, 2013-07-04 at 02:34 -0700, Eric Dumazet wrote: > On Thu, 2013-07-04 at 09:59 +0100, Ian Campbell wrote: > > On Thu, 2013-07-04 at 16:55 +0800, Joe Jin wrote: > > > > > > Another way is add new page flag like PG_send, when sendpage() be called, > > >

Re: kernel panic in skb_copy_bits

2013-07-04 Thread Ian Campbell
On Thu, 2013-07-04 at 16:55 +0800, Joe Jin wrote: > On 07/01/13 16:11, Ian Campbell wrote: > > On Mon, 2013-07-01 at 11:18 +0800, Joe Jin wrote: > >>> A workaround is to turn off O_DIRECT use by Xen as that ensures > >>> the pages are copied. Xen 4.3 does this

Re: [Xen-devel] [PATCH RFC] xen/arm: domain kernel: Small fixes for making suspendable for arm

2013-07-03 Thread Ian Campbell
On Wed, 2013-07-03 at 16:59 +0100, Stefano Stabellini wrote: > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > > index 2c3bdce..77309f7 100644 > > --- a/arch/arm/Kconfig > > +++ b/arch/arm/Kconfig > > @@ -1469,6 +1469,9 @@ config ARCH_NO_VIRT_TO_BUS > > config ISA_DMA_API > > bool > >

[PATCH] pinctrl: am33xx dt binding: correct include path

2013-07-03 Thread Ian Campbell
Using #include is a bit odd. It happens to work because the DTC flags include -Iarch/FOO/boot/dts as well as arch/FOO/boot/dts/include and arch/FOO/boot/dts/include/dt-bindings is a symlink to include/dt-bindings. Signed-off-by: Ian Campbell Cc: Grant Likely Cc: Linus Walleij Cc: linux-kernel

Re: kernel panic in skb_copy_bits

2013-07-01 Thread Ian Campbell
On Mon, 2013-07-01 at 11:18 +0800, Joe Jin wrote: > > A workaround is to turn off O_DIRECT use by Xen as that ensures > > the pages are copied. Xen 4.3 does this by default. > > > > I believe fixes for this are in 4.3 and 4.2.2 if using the > > qemu upstream DM. Note these aren't real fixes, just

Re: [PATCH next] xen: Convert printks to pr_

2013-06-28 Thread Ian Campbell
On Fri, 2013-06-28 at 03:21 -0700, Joe Perches wrote: > Convert printks to pr_ (excludes printk(KERN_DEBUG...) > to be more consistent throughout the xen subsystem. > > Add pr_fmt with KBUILD_MODNAME or "xen:" KBUILD_MODNAME > Coalesce formats and add missing word spaces > Add missing newlines > A

Re: [PATCH next] xen: Use more current logging styles

2013-06-28 Thread Ian Campbell
s. > > Signed-off-by: Joe Perches Acked-by: Ian Campbell > --- > drivers/net/xen-netback/netback.c | 7 +++ > drivers/net/xen-netfront.c| 28 +--- > 2 files changed, 16 insertions(+), 19 deletions(-) > > diff --git a/drivers/n

Re: [Xen-devel] [PATCH v4 5/6] arm64/xen: introduce CONFIG_XEN on ARM64

2013-06-06 Thread Ian Campbell
On Thu, 2013-06-06 at 10:27 +0100, Catalin Marinas wrote: > On Thu, Jun 06, 2013 at 10:03:39AM +0100, Ian Campbell wrote: > > On Wed, 2013-06-05 at 18:02 +0100, Stefano Stabellini wrote: > > > + * the second in r1/x1, the third in r2/x2 and the fourth in r3/x3. > > &g

Re: [Xen-devel] [PATCH v4 5/6] arm64/xen: introduce CONFIG_XEN on ARM64

2013-06-06 Thread Ian Campbell
ere. If you fix the typo and clarify the fifth argument or not as you wish: Acked-by: Ian Campbell Ian. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.ke

Re: [PATCH v3 6/6] MAINTAINERS: add myself as arm64/xen maintainer

2013-06-05 Thread Ian Campbell
On Wed, 2013-06-05 at 13:15 +0100, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini Acked-by: Ian Campbell > --- > MAINTAINERS |7 +++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index fd3a495..9

Re: [PATCH v3 5/6] arm64/xen: introduce CONFIG_XEN and hypercall.S on ARM64

2013-06-05 Thread Ian Campbell
bellini Acked-by: Ian Campbell > [] > +#define HYPERCALL_SIMPLE(hypercall) \ > +ENTRY(HYPERVISOR_##hypercall)\ > + mov x16, #__HYPERVISOR_##hypercall; \ > + hvc XEN_IMM;

Re: [PATCH v2 2/5] arm64/xen: introduce asm/xen header files on arm64

2013-06-04 Thread Ian Campbell
/xen/events.h is slightly different, so introduce a different file > here (use xchg to implement xchg_xen_ulong and pass regs->pstate to > raw_irqs_disabled_flags). > > Also introduce asm/hypervisor.h and asm/sync_bitops.h. > > Signed-off-by: Stefano Stabellini Acked-by: Ia

Re: [PATCH v2 4/5] arm64/xen: use XEN_IO_PROTO_ABI_ARM on ARM64

2013-06-04 Thread Ian Campbell
On Mon, 2013-06-03 at 16:33 +0100, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini Acked-by: Ian Campbell > --- > include/xen/interface/io/protocols.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/include/xen/interface/i

Re: [PATCH v2 1/5] arm/xen: define xen_remap as ioremap_cached

2013-06-04 Thread Ian Campbell
On Mon, 2013-06-03 at 16:33 +0100, Stefano Stabellini wrote: > Define xen_remap as ioremap_cache (MT_MEMORY and MT_DEVICE_CACHED end up > having the same AttrIndx encoding). The entries in static struct mem_type mem_types[] look entirely different to me. What am I missing? [MT_DEVICE_CACH

Re: [Xen-devel] [PATCH v2 5/5] arm64/xen: introduce CONFIG_XEN and hypercall.S on ARM64

2013-06-03 Thread Ian Campbell
On Mon, 2013-06-03 at 17:25 +0100, Catalin Marinas wrote: > On Mon, Jun 03, 2013 at 04:33:44PM +0100, Stefano Stabellini wrote: > > --- /dev/null > > +++ b/arch/arm64/xen/hypercall.S > ... > > +/* > > + * The Xen hypercall calling convention is very similar to the ARM AEBI > > + * procedure calling

Re: [PATCH] dtc: ensure #line directives don't consume data from the next line

2013-06-03 Thread Ian Campbell
hat is considered sufficient then it is OK by me to apply Tested-by: Ian Campbell to this change too. > > Reported-by: Ian Campbell > Signed-off-by: Stephen Warren > --- > This is a fix for 3.10. > > scripts/dtc/dtc-lexer.l |2 +- > scripts/dtc

Re: [PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-06-03 Thread Ian Campbell
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote: > On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote: > >Currently I see: > > DTC arch/powerpc/boot/kilauea.dtb > >Warning (reg_format): "reg" property in /plb/ppc4xx-msi@C1000 has > >inva

[PATCH] powerpc/dts/virtex440: declare address/size-cells for phy device

2013-06-03 Thread Ian Campbell
t #address-cells value for /plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7 Warning (avoid_default_addr_size): Relying on default #size-cells value for /plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7 Signed-off-by: Ian Campbell Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Gernot V

[PATCH] powerpc/boot: Ignore .dtb files.

2013-06-03 Thread Ian Campbell
Signed-off-by: Ian Campbell Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: linuxppc-...@lists.ozlabs.org Cc: linux-kernel@vger.kernel.org --- arch/powerpc/boot/.gitignore |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/boot/.gitignore b/arch/powerpc/boot/.gitignore index

[PATCH] powerpc/4xx: Fix warning in kilauea.dtb

2013-06-03 Thread Ian Campbell
Adding PCIe MSI support" this platform does not use address-cells=2. Signed-off-by: Ian Campbell Cc: Rupjyoti Sarmah Cc: Tirumala R Marri Cc: Josh Boyer Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: linuxppc-...@lists.ozlabs.org Cc: linux-kernel@vger.kernel.org --- arch/powerpc/boot/dt

Re: DTB build failure due to preproccessing

2013-06-03 Thread Ian Campbell
On Fri, 2013-05-31 at 17:31 +0100, Grant Likely wrote: > On Fri, May 31, 2013 at 5:04 PM, Stephen Warren wrote: > > On 05/31/2013 05:48 AM, Grant Likely wrote: > >> --- > >> commit d01dccdcb3ea8233b09efb9c24db9f057fbd3b37 > >> Author: Grant Likely > >> Date: Fri May 31 12:45:18 2013 +0100 > >>

Re: DTB build failure due to preproccessing

2013-05-31 Thread Ian Campbell
On Fri, 2013-05-31 at 08:01 -0500, Jon Loeliger wrote: > Hrm. Is this a "that's not in the kernel's copy yet" problem? BTW I'm using dtc.git 4e76ec796c90d44d417f82d9db2d67cfe575f8ed and not the kernel copy. dtc-lexer.l in my HEAD is identical to the current master (2e3fc7e9b3a4722a5500afaa9faf

Re: DTB build failure due to preproccessing

2013-05-31 Thread Ian Campbell
On Fri, 2013-05-31 at 08:01 -0500, Jon Loeliger wrote: > > > > > > Line 374 is the "IDSEL 0x16..." line here: > > > interrupt-map = < > > > /* IRQ mapping for pci slots and ALI M1533 > > >... > > >

Re: DTB build failure due to preproccessing

2013-05-31 Thread Ian Campbell
On Fri, 2013-05-31 at 12:48 +0100, Grant Likely wrote: > On Fri, 31 May 2013 11:29:30 +0100, Ian Campbell > wrote: > > This affects arch/powerpc/boot/dts/virtex440-ml510.dts but I think it is > > actually a more general issue: > > > > $ make ARCH=power

DTB build failure due to preproccessing

2013-05-31 Thread Ian Campbell
This affects arch/powerpc/boot/dts/virtex440-ml510.dts but I think it is actually a more general issue: $ make ARCH=powerpc CROSS_COMPILE=powerpc-linux- virtex440-ml510.dtb CC scripts/mod/devicetable-offsets.s GEN scripts/mod/devicetable-offsets.h HO

[PATCH] powerpc/fsl-booke: Rename b4qds.dts -> b4qds.dtsi.

2013-05-31 Thread Ian Campbell
1 make: *** [b4qds.dtb] Error 2 I spotted in build tests of device-tree.git, announcement https://lkml.org/lkml/2013/4/24/209, which builds *.dts. Probably no one would do this this in real life on linux.git but it still seems worth fixing. Signed-off-by: Ian Campbell Cc: Shaveta Leekha

[PATCH] kbuild: make sure we clean up DTB temporary files

2013-05-31 Thread Ian Campbell
Various temporary files used when building DTB files were not suffixed with .tmp and therefore were not cleaned up by "make clean". Signed-off-by: Ian Campbell Cc: Michal Marek Cc: Stephen Warren Cc: Rob Herring Cc: Grant Likely Cc: linux-kbu...@vger.kernel.org Cc: li

Re: [Xen-devel] [PATCH-v2] xen: Don't call arch_trigger_all_cpu_backtrace in dom0(pvm)

2013-05-15 Thread Ian Campbell
On Wed, 2013-05-15 at 16:40 +0800, Zhenzhong Duan wrote: > On 2013-04-10 00:36, Ian Campbell wrote: > > On Mon, 2013-04-08 at 08:42 +0100, Jan Beulich wrote: > >>>>> On 07.04.13 at 07:54, Zhenzhong Duan wrote: > >>> nmi isn't supported in dom0

Re: [RFC] device-tree.git automatic sync from linux.git

2013-05-13 Thread Ian Campbell
On Mon, 2013-05-13 at 08:02 +0100, Michal Simek wrote: > Just small overview it is a Xilinx soft core cpu where you can even setup > some parameters for core itself - multiplier, divider, BS, fpu, cache sizes, > etc. > You have to also compose the whole system and every platform/configuration is

Re: [Xen-devel] [PATCH v3 1/4] xen: move xen_setup_runstate_info and get_runstate_snapshot to drivers/xen/time.c

2013-05-09 Thread Ian Campbell
> > > - use the new common functions in arch/ia64/xen/time.c. > > > > > > Signed-off-by: Stefano Stabellini > > > Acked-by: Ian Campbell > > > CC: konrad.w...@oracle.com > > > > On the generic and x86 side it looks OK to me. I presume you di

Re: [Xen-devel] [PATCH v3 4/4] xen/arm: account for stolen ticks

2013-05-09 Thread Ian Campbell
On Wed, 2013-05-08 at 18:03 +0100, Stefano Stabellini wrote: > > What if we have Xen support enabled but don't run as a Xen guest? > > That's fine as long as the Xen runstate_memory_area interface is > available. > Usually I am a great fun of feature flags, so that every feature can be > dynamical

Re: [PATCH 2/3] xen/arm: do not handle VCPUOP_register_vcpu_info failures

2013-05-08 Thread Ian Campbell
ng the registered vcpu info, even if > -later ones fail to. */ > - per_cpu(xen_vcpu, cpu) = vcpup; > - } > + if (err) > + BUG(); AKA BUG_ON(err); Acked-by: Ian Campbell > + per_cpu(xen_vcpu, cpu) = vcpup; > + > return 0; > } >

Re: [Xen-devel] [PATCH RFC v2 4/4] xen/arm: account for stolen ticks

2013-05-07 Thread Ian Campbell
> > > @@ -301,6 +320,10 @@ static int __init xen_init_events(void) > > > > > > on_each_cpu(xen_percpu_init, NULL, 0); > > > > > > + pv_time_ops.steal_clock = xen_stolen_accounting; > > > + static_key_slow_inc(¶virt_steal_enabled); > > > + static_key_slow_inc(¶virt_steal_rq_enabled); > > > >

Re: [Xen-devel] [PATCH RFC v2 2/4] arm: introduce CONFIG_PARAVIRT and pv_time_ops

2013-05-07 Thread Ian Campbell
On Tue, 2013-05-07 at 13:15 +0100, Stefano Stabellini wrote: > On Tue, 7 May 2013, Ian Campbell wrote: > > On Mon, 2013-05-06 at 15:51 +0100, Stefano Stabellini wrote: > > > Introduce CONFIG_PARAVIRT on ARM. > > > > What about PARAVIRT_TIME_ACCOUNTING? I'm n

Re: [Xen-devel] [PATCH 2/4] xen/vcpu: Document the xen_vcpu_info and xen_vcpu

2013-05-07 Thread Ian Campbell
On Mon, 2013-05-06 at 14:04 +0100, Konrad Rzeszutek Wilk wrote: > They are important structures and it is not clear at first > look what they are for. > > The xen_vcpu is a pointer. By default it points to the shared_info > structure (at the CPU offset location). However if the > VCPUOP_register_v

Re: [Xen-devel] [PATCH RFC v2 2/4] arm: introduce CONFIG_PARAVIRT and pv_time_ops

2013-05-07 Thread Ian Campbell
On Mon, 2013-05-06 at 15:51 +0100, Stefano Stabellini wrote: > Introduce CONFIG_PARAVIRT on ARM. What about PARAVIRT_TIME_ACCOUNTING? I'm not sure what it is but it looks like a more lightweight version of pv stolen time? > The only paravirt interface supported is pv_time_ops.steal_clock. > No ru

Re: [Xen-devel] [PATCH RFC v2 4/4] xen/arm: account for stolen ticks

2013-05-07 Thread Ian Campbell
On Mon, 2013-05-06 at 15:51 +0100, Stefano Stabellini wrote: > Register the runstate_memory_area with the hypervisor. > Use pv_time_ops.steal_clock to account for stolen ticks. > > Signed-off-by: Stefano Stabellini > --- > arch/arm/xen/enlighten.c | 23 +++ > 1 files change

Re: [Xen-devel] [PATCH v2 2/2] xen/balloon: Enforce various limits on target

2013-05-03 Thread Ian Campbell
On Fri, 2013-05-03 at 16:47 +0100, Daniel Kiper wrote: > > > > > Do not forget that guest may change target itself. > > > > > > > > Yes it can, and that can fail either due to maxmem or due to ENOMEM, and > > > > the kernel needs prepared to deal with that when it happens. > > > > > > Sure but why

Re: [Xen-devel] [PATCH v2 2/2] xen/balloon: Enforce various limits on target

2013-05-03 Thread Ian Campbell
On Fri, 2013-05-03 at 14:47 +0100, Daniel Kiper wrote: > On Fri, May 03, 2013 at 02:21:24PM +0100, Ian Campbell wrote: > > On Fri, 2013-05-03 at 14:00 +0100, Daniel Kiper wrote: > > > On Fri, May 03, 2013 at 09:15:32AM +0100, Ian Campbell wrote: > > > > On Thu, 201

Re: [Xen-devel] [PATCH v2 2/2] xen/balloon: Enforce various limits on target

2013-05-03 Thread Ian Campbell
On Fri, 2013-05-03 at 14:00 +0100, Daniel Kiper wrote: > On Fri, May 03, 2013 at 09:15:32AM +0100, Ian Campbell wrote: > > On Thu, 2013-05-02 at 19:04 +0100, Konrad Rzeszutek Wilk wrote: > > > On Thu, May 02, 2013 at 12:34:32PM +0100, Stefano Stabellini wrote: > > [...]

Re: [Xen-devel] [PATCH 2/3] xen: move do_stolen_accounting to drivers/xen/time.c

2013-05-03 Thread Ian Campbell
On Thu, 2013-05-02 at 19:49 +0100, Christopher Covington wrote: > Hi Stefano, > > On 05/01/2013 03:27 PM, Stefano Stabellini wrote: > > Move do_stolen_accounting, xen_vcpu_stolen and related functions and > > static variables to common code (drivers/xen/time.c). > > > > Signed-off-by: Stefano Sta

Re: [Xen-devel] [PATCH v2 2/2] xen/balloon: Enforce various limits on target

2013-05-03 Thread Ian Campbell
On Thu, 2013-05-02 at 19:04 +0100, Konrad Rzeszutek Wilk wrote: > On Thu, May 02, 2013 at 12:34:32PM +0100, Stefano Stabellini wrote: > > On Tue, 30 Apr 2013, Daniel Kiper wrote: > > > > > > > +/* > > > > > > > + * Extra internal memory reserved by libxl. > > > > > > > + * Check tools/libxl/libxl_m

Re: [Xen-devel] [PATCH v2 2/2] xen/balloon: Enforce various limits on target

2013-05-03 Thread Ian Campbell
On Tue, 2013-04-30 at 19:58 +0100, Daniel Kiper wrote: > On Tue, Apr 30, 2013 at 02:44:18PM +0100, Ian Campbell wrote: > > On Tue, 2013-04-30 at 13:59 +0100, Daniel Kiper wrote: > > > On Mon, Apr 29, 2013 at 03:44:09PM +0100, Ian Campbell wrote: > > > > On Mon, 201

<    1   2   3   4   5   6   7   >