On 04/02/2015 05:02 PM, Jan Kara wrote: > Hello, > > On Tue 17-03-15 12:56:30, Jan Kara wrote: >> After a long pause I'm sending second version of my patch series to abstract >> vma handling from the various media drivers. After this patch set drivers have >> to know much less details about vmas, their types, and locking. My motivation >> for the series is that I want to change get_user_pages() locking and I want to >> handle subtle locking details in as few places as possible. >> >> The core of the series is the new helper get_vaddr_pfns() which is given a >> virtual address and it fills in PFNs into provided array. If PFNs correspond to >> normal pages it also grabs references to these pages. The difference from >> get_user_pages() is that this function can also deal with pfnmap, mixed, and io >> mappings which is what the media drivers need. >> >> I have tested the patches with vivid driver so at least vb2 code got some >> exposure. Conversion of other drivers was just compile-tested so I'd like to >> ask respective maintainers if they could have a look. Also I'd like to ask mm >> folks to check patch 2/9 implementing the helper. Thanks! > Ping? Any reactions? For patch 1/9: Acked-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> For the other patches I do not feel qualified to give Acks. I've Cc-ed Pawel and Marek who have a better understanding of the mm internals than I do. Hopefully they can review the code. It definitely looks like a good idea, and if nobody else will comment on the vb2 patches in the next 2 weeks, then I'll try to review it myself (for whatever that's worth). Regards, Hans _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
