[patch 07/13] x86: PAT use reserve free memtype in pci_mmap_page_range

Previous thread: [ANNOUNCE] ddtree: A git kernel tree for storage servers by Daniel Phillips on Wednesday, March 19, 2008 - 4:02 am. (8 messages)

Next thread: [PATCH] give elf_check_arch() a chance for checking endian mismatch by Roy Lee on Wednesday, March 19, 2008 - 3:03 pm. (3 messages)
To: <ak@...>, <ebiederm@...>, <rdreier@...>, <torvalds@...>, <gregkh@...>, <airlied@...>, <davej@...>, <mingo@...>, <tglx@...>, <hpa@...>, <akpm@...>, <arjan@...>, <jesse.barnes@...>
Cc: <linux-kernel@...>, Venkatesh Pallipadi <venkatesh.pallipadi@...>, Suresh Siddha <suresh.b.siddha@...>
Date: Tuesday, March 18, 2008 - 8:00 pm

Add reserve_memtype and free_memtype wrapper for pci_mmap_page_range. Free
is called on unmap, but identity map continues to be mapped as per
pci_mmap_page_range request, until next request for the same region calls
ioremap_change_attr(), which will go through without conflict. This way of
mapping is identical to one used in ioremap/iounmap.

Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>

Index: linux-2.6-x86.git/arch/x86/pci/i386.c
===================================================================
--- linux-2.6-x86.git.orig/arch/x86/pci/i386.c 2008-03-18 02:16:05.000000000 -0700
+++ linux-2.6-x86.git/arch/x86/pci/i386.c 2008-03-18 03:29:39.000000000 -0700
@@ -30,6 +30,9 @@
#include <linux/init.h>
#include <linux/ioport.h>
#include <linux/errno.h>
+#include <linux/bootmem.h>
+
+#include <asm/pat.h>

#include "pci.h"

@@ -297,10 +300,34 @@
pci_write_config_byte(dev, PCI_LATENCY_TIMER, lat);
}

+static void pci_unmap_page_range(struct vm_area_struct *vma)
+{
+ u64 addr = (u64)vma->vm_pgoff << PAGE_SHIFT;
+ free_memtype(addr, addr + vma->vm_end - vma->vm_start);
+}
+
+static void pci_track_mmap_page_range(struct vm_area_struct *vma)
+{
+ u64 addr = (u64)vma->vm_pgoff << PAGE_SHIFT;
+ unsigned long flags = pgprot_val(vma->vm_page_prot)
+ & _PAGE_CACHE_MASK;
+
+ reserve_memtype(addr, addr + vma->vm_end - vma->vm_start, flags, NULL);
+}
+
+static struct vm_operations_struct pci_mmap_ops = {
+ .open = pci_track_mmap_page_range,
+ .close = pci_unmap_page_range,
+};
+
int pci_mmap_page_range(struct pci_dev *dev, struct vm_area_struct *vma,
enum pci_mmap_state mmap_state, int write_combine)
{
unsigned long prot;
+ u64 addr = vma->vm_pgoff << PAGE_SHIFT;
+ unsigned long len = vma->vm_end - vma->vm_start;
+ unsigned long flags;
+ unsigned long new_flags;

/* I/O space canno...

Previous thread: [ANNOUNCE] ddtree: A git kernel tree for storage servers by Daniel Phillips on Wednesday, March 19, 2008 - 4:02 am. (8 messages)

Next thread: [PATCH] give elf_check_arch() a chance for checking endian mismatch by Roy Lee on Wednesday, March 19, 2008 - 3:03 pm. (3 messages)