linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: catalin.marinas@arm.com (Catalin Marinas)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 8/8] ARM: Implement phys_mem_access_prot() to avoid attributes aliasing
Date: Tue, 04 May 2010 17:44:59 +0100	[thread overview]
Message-ID: <20100504164459.26355.88200.stgit@e102109-lin.cambridge.arm.com> (raw)
In-Reply-To: <20100504163823.26355.58568.stgit@e102109-lin.cambridge.arm.com>

ARMv7 onwards requires that there are no aliases to the same physical
location using different memory types (i.e. Normal vs Strongly Ordered).
Access to SO mappings when the unaligned accesses are handled in
hardware is also Unpredictable (pgprot_noncached() mappings in user
space).

The /dev/mem driver requires uncached mappings with O_SYNC. The patch
implements the phys_mem_access_prot() function which generates Strongly
Ordered memory attributes if !pfn_valid() (independent of O_SYNC) and
Normal Noncacheable (writecombine) if O_SYNC.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
---
 arch/arm/include/asm/pgtable.h |    3 +++
 arch/arm/mm/mmu.c              |   14 ++++++++++++++
 2 files changed, 17 insertions(+), 0 deletions(-)

diff --git a/arch/arm/include/asm/pgtable.h b/arch/arm/include/asm/pgtable.h
index 1139768..6fae619 100644
--- a/arch/arm/include/asm/pgtable.h
+++ b/arch/arm/include/asm/pgtable.h
@@ -317,6 +317,9 @@ static inline pte_t pte_mkspecial(pte_t pte) { return pte; }
 #if __LINUX_ARM_ARCH__ >= 7
 #define pgprot_dmacoherent(prot) \
 	__pgprot_modify(prot, L_PTE_MT_MASK|L_PTE_EXEC, L_PTE_MT_BUFFERABLE)
+#define __HAVE_PHYS_MEM_ACCESS_PROT
+extern pgprot_t phys_mem_access_prot(struct file *file, unsigned long pfn,
+				     unsigned long size, pgprot_t vma_prot);
 #else
 #define pgprot_dmacoherent(prot) \
 	__pgprot_modify(prot, L_PTE_MT_MASK|L_PTE_EXEC, L_PTE_MT_UNCACHED)
diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c
index 241c24a..deb62cc 100644
--- a/arch/arm/mm/mmu.c
+++ b/arch/arm/mm/mmu.c
@@ -14,6 +14,7 @@
 #include <linux/bootmem.h>
 #include <linux/mman.h>
 #include <linux/nodemask.h>
+#include <linux/fs.h>
 
 #include <asm/cputype.h>
 #include <asm/mach-types.h>
@@ -485,6 +486,19 @@ static void __init build_mem_type_table(void)
 	}
 }
 
+#if __LINUX_ARM_ARCH__ >= 7
+pgprot_t phys_mem_access_prot(struct file *file, unsigned long pfn,
+			      unsigned long size, pgprot_t vma_prot)
+{
+	if (!pfn_valid(pfn))
+		return pgprot_noncached(vma_prot);
+	else if (file->f_flags & O_SYNC)
+		return pgprot_writecombine(vma_prot);
+	return vma_prot;
+}
+EXPORT_SYMBOL(phys_mem_access_prot);
+#endif
+
 #define vectors_base()	(vectors_high() ? 0xffff0000 : 0)
 
 static void __init alloc_init_pte(pmd_t *pmd, unsigned long addr,

  parent reply	other threads:[~2010-05-04 16:44 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-04 16:44 [PATCH 0/8] Various patches for comments and upstream Catalin Marinas
2010-05-04 16:44 ` [PATCH 1/8] ARM: Improve the L2 cache performance when PL310 is used Catalin Marinas
2010-05-04 17:02   ` Jason McMullan
2010-05-05 16:07     ` Catalin Marinas
2010-05-04 16:44 ` [PATCH 2/8] ARM: Implement read/write for ownership in the ARMv6 DMA cache ops Catalin Marinas
2010-05-04 17:04   ` Jason McMullan
2010-05-05 16:23     ` Catalin Marinas
2010-05-05 13:26   ` George G. Davis
2010-05-06 14:40     ` Catalin Marinas
2010-05-06 15:57       ` George G. Davis
2010-05-12 12:51   ` Ronen Shitrit
2010-05-12 13:55     ` Catalin Marinas
2010-05-12 15:03       ` Ronen Shitrit
2010-05-12 18:48       ` Russell King - ARM Linux
2010-05-12 18:59         ` Russell King - ARM Linux
2010-05-12 20:00           ` Ronen Shitrit
2010-05-12 20:04             ` Russell King - ARM Linux
2010-05-12 20:19               ` Ronen Shitrit
2010-05-12 21:21           ` [PATCH 2/8] ARM: Implement read/write for ownership in theARMv6 " Catalin Marinas
2010-05-13  5:27             ` Ronen Shitrit
2010-05-13  8:26               ` Catalin Marinas
2010-05-13 13:54                 ` George G. Davis
2010-05-13 14:15                   ` Catalin Marinas
2010-05-13 20:34                     ` George G. Davis
2010-05-14 16:29                       ` Catalin Marinas
2010-05-14 16:42                         ` Catalin Marinas
2010-05-15  1:26                           ` George G. Davis
2010-05-16  6:28                             ` Ronen Shitrit
2010-05-16  6:29                 ` Ronen Shitrit
2010-05-16 15:01                   ` Russell King - ARM Linux
2010-05-17  6:29                     ` Ronen Shitrit
2010-05-17  6:57                       ` Russell King - ARM Linux
2010-05-17  7:34                         ` Ronen Shitrit
2010-05-17  7:43                           ` Russell King - ARM Linux
2010-05-17  8:29                             ` Ronen Shitrit
2010-05-17  8:57                               ` Russell King - ARM Linux
2010-05-17  9:50                                 ` Ronen Shitrit
2010-05-17 10:03                                   ` Russell King - ARM Linux
2010-05-17 11:26                                     ` Ronen Shitrit
2010-05-17 11:31                                       ` Russell King - ARM Linux
2010-05-17 11:45                                         ` Catalin Marinas
2010-05-17 10:00                           ` Catalin Marinas
2010-05-17 11:29                             ` Ronen Shitrit
2010-05-17 11:42                               ` Catalin Marinas
2010-05-17 12:04                                 ` Ronen Shitrit
2010-05-17 13:45                                   ` Catalin Marinas
2010-05-17  9:51                   ` Catalin Marinas
2010-05-17  9:57                     ` Catalin Marinas
2010-05-17  9:59                       ` Ronen Shitrit
2010-05-17 11:08                         ` Catalin Marinas
2010-05-17 11:27                           ` Ronen Shitrit
2010-05-17 11:47                             ` Catalin Marinas
2010-05-17 13:46                               ` [PATCH 2/8] ARM: Implement read/write for ownership intheARMv6 " Catalin Marinas
2010-05-04 16:44 ` [PATCH 3/8] ARM: Align machine_desc.phys_io to a 1MB section Catalin Marinas
2010-05-04 16:44 ` [PATCH 4/8] ARM: Remove the domain switching on ARMv6k/v7 CPUs Catalin Marinas
2010-05-04 16:44 ` [PATCH 5/8] ARM: Fix the __arm_ioremap_caller() definition in nommu.c Catalin Marinas
2010-05-04 17:19   ` Russell King - ARM Linux
2010-05-04 16:44 ` [PATCH 6/8] ARM: Implement copy_to_user_page() for noMMU Catalin Marinas
2010-05-04 17:19   ` Russell King - ARM Linux
2010-05-04 16:44 ` [PATCH 7/8] ARM: Use the Inner Shareable I-cache and BTB ops on ARMv7 SMP Catalin Marinas
2010-05-28 18:50   ` Russell King - ARM Linux
2010-05-28 21:37     ` [PATCH 7/8] ARM: Use the Inner Shareable I-cache and BTB opson " Catalin Marinas
2010-05-04 16:44 ` Catalin Marinas [this message]
2010-05-04 16:48 ` [PATCH 0/8] Various patches for comments and upstream Catalin Marinas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100504164459.26355.88200.stgit@e102109-lin.cambridge.arm.com \
    --to=catalin.marinas@arm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).