Am Wed, 05 Dec 2007 18:15:51 +1100 schrieb [EMAIL PROTECTED]: > Convert uio from nopage to fault. > > Signed-off-by: Nick Piggin <[EMAIL PROTECTED]> > Cc: [EMAIL PROTECTED]
Hi Nick, could you please add me to Cc: for UIO stuff: Cc: [EMAIL PROTECTED] The patch itself looks alright to me, but I didn't test with a real driver. Acked-by: Hans J Koch <[EMAIL PROTECTED]> Thanks, Hans > Cc: [email protected] > --- > drivers/uio/uio.c | 14 ++++++-------- > 1 file changed, 6 insertions(+), 8 deletions(-) > > Index: linux-2.6/drivers/uio/uio.c > =================================================================== > --- linux-2.6.orig/drivers/uio/uio.c > +++ linux-2.6/drivers/uio/uio.c > @@ -412,30 +412,28 @@ static void uio_vma_close(struct vm_area > idev->vma_count--; > } > > -static struct page *uio_vma_nopage(struct vm_area_struct *vma, > - unsigned long address, int *type) > +static int uio_vma_fault(struct vm_area_struct *vma, struct vm_fault > *vmf) { > struct uio_device *idev = vma->vm_private_data; > - struct page* page = NOPAGE_SIGBUS; > + struct page *page; > > int mi = uio_find_mem_index(vma); > if (mi < 0) > - return page; > + return VM_FAULT_SIGBUS; > > if (idev->info->mem[mi].memtype == UIO_MEM_LOGICAL) > page = virt_to_page(idev->info->mem[mi].addr); > else > page = > vmalloc_to_page((void*)idev->info->mem[mi].addr); get_page(page); > - if (type) > - *type = VM_FAULT_MINOR; > - return page; > + vmf->page = page; > + return 0; > } > > static struct vm_operations_struct uio_vm_ops = { > .open = uio_vma_open, > .close = uio_vma_close, > - .nopage = uio_vma_nopage, > + .fault = uio_vma_fault, > }; > > static int uio_mmap_physical(struct vm_area_struct *vma) > -- 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/

