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

2013-06-04 Thread Catalin Marinas
On Tue, Jun 04, 2013 at 02:58:57PM +0100, Russell King - ARM Linux wrote: > On Tue, Jun 04, 2013 at 12:28:34PM +0100, Catalin Marinas wrote: > > PROT_PTE_DEVICE and PROT_SECT_DEVICE above don't contain any memory type > > information, just attributes/permission - present, young, dirty and XN: > >

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

2013-06-04 Thread Stefano Stabellini
On Tue, 4 Jun 2013, Russell King - ARM Linux wrote: > On Tue, Jun 04, 2013 at 12:28:34PM +0100, Catalin Marinas wrote: > > PROT_PTE_DEVICE and PROT_SECT_DEVICE above don't contain any memory type > > information, just attributes/permission - present, young, dirty and XN: > > > > #define PROT_PTE_D

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

2013-06-04 Thread Russell King - ARM Linux
On Tue, Jun 04, 2013 at 12:28:34PM +0100, Catalin Marinas wrote: > PROT_PTE_DEVICE and PROT_SECT_DEVICE above don't contain any memory type > information, just attributes/permission - present, young, dirty and XN: > > #define PROT_PTE_DEVICE > L_PTE_PRESENT|L_PTE_YOUNG|L_PTE_DIRTY|L

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

2013-06-04 Thread Stefano Stabellini
On Tue, 4 Jun 2013, Catalin Marinas wrote: > On Tue, Jun 04, 2013 at 10:20:50AM +0100, Ian Campbell wrote: > > 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). > > >

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

2013-06-04 Thread Catalin Marinas
On Tue, Jun 04, 2013 at 10:20:50AM +0100, Ian Campbell wrote: > 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[] lo

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

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

2013-06-03 Thread Stefano Stabellini
Define xen_remap as ioremap_cache (MT_MEMORY and MT_DEVICE_CACHED end up having the same AttrIndx encoding). Remove include asm/mach/map.h, not unneeded. Signed-off-by: Stefano Stabellini --- arch/arm/include/asm/xen/page.h |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --gi