Re: [PATCH 11/16] IB uverbs: add mthca mmap support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



    Andrew> What's the thinking behind the VM_DONTCOPY there?

As I said before, I don't think the thinking behind VM_DONTCOPY was
correct thinking.  Let's take it out.

I've now answered all your questions on this patchset (or at least
written something in response to all your questions ;).  What's your
feeling on merging?  If more info is required, just let me know.  I'll
also be at the kernel summit in a couple of weeks if you want to go
over it in person.

 - R.


There's no need to set VM_DONTCOPY when mmap()ing the hardware
doorbell page into userspace.

Signed-off-by: Roland Dreier <[email protected]>


--- linux-export.orig/drivers/infiniband/hw/mthca/mthca_provider.c	2005-06-28 15:20:28.000000000 -0700
+++ linux-export/drivers/infiniband/hw/mthca/mthca_provider.c	2005-07-05 12:14:20.838664330 -0700
@@ -347,7 +347,6 @@
 		return -EINVAL;
 
 	vma->vm_page_prot = pgprot_noncached(vma->vm_page_prot);
-	vma->vm_flags    |= VM_DONTCOPY;
 
 	if (remap_pfn_range(vma, vma->vm_start,
 			    to_mucontext(context)->uar.pfn,
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]
  Powered by Linux