[parisc-linux] [PATCH] Shared mmap [Take 3]

Matthew Wilcox matthew@wil.cx
Wed, 14 Mar 2001 21:55:46 +0000


On Mon, Mar 12, 2001 at 05:50:42PM +0000, Matthew Wilcox wrote:
> 
> This patch changes shared mmaps of files to always be aligned at the
> same offset within the processor's d-cache.  This fixes PA-RISC & MIPS.
> Sparc & IA-64 need some of their support changing, but I don't want to
> do that until everyone's had the chance to disagree with my changes :-)

What, no comment?  no flames, no insults?  is this an uncontroversial patch?

Anyway, I have some improvements.

 * use inode->i_mapping->i_mmap_shared to determine whether there are
   any existing mappings rather than keeping track of it through i_mmap_offset.
 * take `pgoff' into account.  Oops, that was the whole point of the patch.
 * ensure `offset' is limited to the range [0 .. SHMLBA)

this time it's actually tested on PA-RISC.  Should work on MIPS too.
Sparc & IA-64 will need work.

Index: include/linux/fs.h
===================================================================
RCS file: /home/cvs/parisc/linux/include/linux/fs.h,v
retrieving revision 1.10
diff -u -p -r1.10 fs.h
--- fs.h	2001/02/02 03:37:12	1.10
+++ fs.h	2001/03/14 21:11:54
@@ -424,6 +424,7 @@ struct inode {
 
 	unsigned long		i_state;
 
+	unsigned int		i_mmap_offset; /* Handle L1 d-cache aliasing */
 	unsigned int		i_flags;
 	unsigned char		i_sock;
 
Index: include/linux/mm.h
===================================================================
RCS file: /home/cvs/parisc/linux/include/linux/mm.h,v
retrieving revision 1.16
diff -u -p -r1.16 mm.h
--- mm.h	2001/02/08 22:39:19	1.16
+++ mm.h	2001/03/14 21:11:54
@@ -423,6 +423,7 @@ extern void insert_vm_struct(struct mm_s
 extern void __insert_vm_struct(struct mm_struct *, struct vm_area_struct *);
 extern void build_mmap_avl(struct mm_struct *);
 extern void exit_mmap(struct mm_struct *);
+extern unsigned long get_shared_area(unsigned long, unsigned long, unsigned int);
 extern unsigned long get_unmapped_area(unsigned long, unsigned long);
 
 extern unsigned long do_mmap_pgoff(struct file *file, unsigned long addr,
Index: mm/mmap.c
===================================================================
RCS file: /home/cvs/parisc/linux/mm/mmap.c,v
retrieving revision 1.19
diff -u -p -r1.19 mmap.c
--- mmap.c	2001/02/02 03:37:18	1.19
+++ mmap.c	2001/03/14 21:11:55
@@ -190,6 +190,7 @@ unsigned long do_mmap_pgoff(struct file 
 {
 	struct mm_struct * mm = current->mm;
 	struct vm_area_struct * vma;
+	struct inode *inode = NULL;
 	int correct_wcount = 0;
 	int error;
 
@@ -223,17 +224,18 @@ unsigned long do_mmap_pgoff(struct file 
 	 * of the memory object, so we don't do any here.
 	 */
 	if (file != NULL) {
+		inode = file->f_dentry->d_inode;
 		switch (flags & MAP_TYPE) {
 		case MAP_SHARED:
 			if ((prot & PROT_WRITE) && !(file->f_mode & FMODE_WRITE))
 				return -EACCES;
 
 			/* Make sure we don't allow writing to an append-only file.. */
-			if (IS_APPEND(file->f_dentry->d_inode) && (file->f_mode & FMODE_WRITE))
+			if (IS_APPEND(inode) && (file->f_mode & FMODE_WRITE))
 				return -EACCES;
 
 			/* make sure there are no mandatory locks on the file. */
-			if (locks_verify_locked(file->f_dentry->d_inode))
+			if (locks_verify_locked(inode))
 				return -EAGAIN;
 
 			/* fall through */
@@ -253,11 +255,20 @@ unsigned long do_mmap_pgoff(struct file 
 	if (flags & MAP_FIXED) {
 		if (addr & ~PAGE_MASK)
 			return -EINVAL;
+	} else if (inode && (flags & MAP_SHARED)) {
+		if (inode->i_mapping->i_mmap_shared) {
+			addr = get_shared_area(addr, len, inode->i_mmap_offset + (pgoff << PAGE_SHIFT));
+		} else {
+			addr = get_unmapped_area(addr, len);
+			if (addr) {
+				inode->i_mmap_offset = (addr - (pgoff << PAGE_SHIFT)) & (SHMLBA - 1);
+			}
+		}
 	} else {
 		addr = get_unmapped_area(addr, len);
-		if (!addr)
-			return -ENOMEM;
 	}
+	if (!addr)
+		return -ENOMEM;
 
 	/* Determine the object being mapped and call the appropriate
 	 * specific mapper. the address has already been validated, but
@@ -348,7 +359,7 @@ unsigned long do_mmap_pgoff(struct file 
 
 	insert_vm_struct(mm, vma);
 	if (correct_wcount)
-		atomic_inc(&file->f_dentry->d_inode->i_writecount);
+		atomic_inc(&inode->i_writecount);
 	
 	mm->total_vm += len >> PAGE_SHIFT;
 	if (flags & VM_LOCKED) {
@@ -359,7 +370,7 @@ unsigned long do_mmap_pgoff(struct file 
 
 unmap_and_free_vma:
 	if (correct_wcount)
-		atomic_inc(&file->f_dentry->d_inode->i_writecount);
+		atomic_inc(&inode->i_writecount);
 	vma->vm_file = NULL;
 	fput(file);
 	/* Undo any partial mapping done by a device driver. */
@@ -376,6 +387,30 @@ free_vma:
  * Return value 0 means ENOMEM.
  */
 #ifndef HAVE_ARCH_UNMAPPED_AREA
+
+#define DCACHE_ALIGN(addr) ((addr) &~ (SHMLBA - 1))
+
+unsigned long get_shared_area(unsigned long addr, unsigned long len, unsigned int offset)
+{
+	struct vm_area_struct *vmm;
+	offset &= SHMLBA - 1;
+
+	if (len > TASK_SIZE)
+		return 0;
+	if (!addr)
+		addr = TASK_UNMAPPED_BASE;
+	addr = DCACHE_ALIGN(addr - offset) + offset;
+
+	for (vmm = find_vma(current->mm, addr); ; vmm = vmm->vm_next) {
+		/* At this point:  (!vmm || addr < vmm->vm_end). */
+		if (TASK_SIZE - len < addr)
+			return 0;
+		if (!vmm || addr + len <= vmm->vm_start)
+			return addr;
+		addr = DCACHE_ALIGN(vmm->vm_end - offset) + offset;
+	}
+}
+
 unsigned long get_unmapped_area(unsigned long addr, unsigned long len)
 {
 	struct vm_area_struct * vmm;
Index: mm/mremap.c
===================================================================
RCS file: /home/cvs/parisc/linux/mm/mremap.c,v
retrieving revision 1.7
diff -u -p -r1.7 mremap.c
--- mremap.c	2001/01/25 00:03:36	1.7
+++ mremap.c	2001/03/14 21:11:55
@@ -276,7 +276,11 @@ unsigned long do_mremap(unsigned long ad
 	ret = -ENOMEM;
 	if (flags & MREMAP_MAYMOVE) {
 		if (!(flags & MREMAP_FIXED)) {
-			new_addr = get_unmapped_area(0, new_len);
+			if (vma->vm_file && (vma->vm_flags & VM_SHARED)) {
+				new_addr = get_shared_area(0, new_len, vma->vm_file->f_dentry->d_inode->i_mmap_offset + (vma->vm_pgoff << PAGE_SHIFT));
+			} else {
+				new_addr = get_unmapped_area(0, new_len);
+			}
 			if (!new_addr)
 				goto out;
 		}


_______________________________________________
parisc-linux-cvs mailing list
parisc-linux-cvs@lists.parisc-linux.org
http://lists.parisc-linux.org/cgi-bin/mailman/listinfo/parisc-linux-cvs

-- 
Revolutions do not require corporate support.