[PATCH 0/26] get_user_pages() cleanup

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

 



  Hello,

  In my quest for changing locking around page faults to make things easier for
filesystems I found out get_user_pages() users could use a cleanup.  The
knowledge about necessary locking for get_user_pages() is in tons of places in
drivers and quite a few of them actually get it wrong (don't have mmap_sem when
calling get_user_pages() or hold mmap_sem when calling copy_from_user() in the
surrounding code). Rather often this actually doesn't seem necessary. This
patch series converts lots of places to use either get_user_pages_fast()
or a new simple wrapper get_user_pages_unlocked() to remove the knowledge
of mmap_sem from the drivers. I'm still looking into converting a few remaining
drivers (most notably v4l2) which are more complex.

As I already wrote, in some cases I actually think drivers were buggy (and I
note that in corresponding changelogs). I would really like to ask respective
maintainers to have a look at the patches in their area. Also any other
comments are welcome. Thanks.

								Honza

PS: Sorry for the huge recipient list but I don't really know how to trim it
    down...

CC: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>
CC: Andreas Dilger <andreas.dilger@xxxxxxxxx>
CC: Andy Walls <awalls@xxxxxxxxxxxxxxxx>
CC: Arnd Bergmann <arnd@xxxxxxxx>
CC: Benjamin LaHaise <bcrl@xxxxxxxxx>
CC: ceph-devel@xxxxxxxxxxxxxxx
CC: Dan Williams <dan.j.williams@xxxxxxxxx>
CC: David Airlie <airlied@xxxxxxxx>
CC: dri-devel@xxxxxxxxxxxxxxxxxxxxx
CC: Gleb Natapov <gleb@xxxxxxxxxx>
CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
CC: hpdd-discuss@xxxxxxxxxxxx
CC: Jarod Wilson <jarod@xxxxxxxxxxxx>
CC: Jayant Mangalampalli <jayant.mangalampalli@xxxxxxxxx>
CC: Jean-Christophe Plagniol-Villard <plagnioj@xxxxxxxxxxxx>
CC: Jesper Nilsson <jesper.nilsson@xxxxxxxx>
CC: Kai Makisara <Kai.Makisara@xxxxxxxxxxx>
CC: kvm@xxxxxxxxxxxxxxx
CC: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
CC: linux-aio@xxxxxxxxx
CC: linux-cris-kernel@xxxxxxxx
CC: linux-fbdev@xxxxxxxxxxxxxxx
CC: linux-fsdevel@xxxxxxxxxxxxxxx
CC: linux-ia64@xxxxxxxxxxxxxxx
CC: linux-media@xxxxxxxxxxxxxxx
CC: linux-nfs@xxxxxxxxxxxxxxx
CC: linux-rdma@xxxxxxxxxxxxxxx
CC: linux-scsi@xxxxxxxxxxxxxxx
CC: Manu Abraham <abraham.manu@xxxxxxxxx>
CC: Mark Allyn <mark.a.allyn@xxxxxxxxx>
CC: Mikael Starvik <starvik@xxxxxxxx>
CC: Mike Marciniszyn <infinipath@xxxxxxxxx>
CC: Naren Sankar <nsankar@xxxxxxxxxxxx>
CC: Paolo Bonzini <pbonzini@xxxxxxxxxx>
CC: Peng Tao <tao.peng@xxxxxxx>
CC: Roland Dreier <roland@xxxxxxxxxx>
CC: Sage Weil <sage@xxxxxxxxxxx>
CC: Scott Davilla <davilla@xxxxxxx>
CC: Timur Tabi <timur@xxxxxxxxxxxxx>
CC: Tomi Valkeinen <tomi.valkeinen@xxxxxx>
CC: Tony Luck <tony.luck@xxxxxxxxx>
CC: Trond Myklebust <Trond.Myklebust@xxxxxxxxxx>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux