From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from cantor2.suse.de ([195.135.220.15]:58895 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752468AbbDBPDE (ORCPT ); Thu, 2 Apr 2015 11:03:04 -0400 Date: Thu, 2 Apr 2015 17:02:58 +0200 From: Jan Kara To: linux-media@vger.kernel.org Cc: Hans Verkuil , Mauro Carvalho Chehab , linux-mm@kvack.org, dri-devel@lists.freedesktop.org, David Airlie , Jan Kara Subject: Re: [PATCH 0/9 v2] Helper to abstract vma handling in media layer Message-ID: <20150402150258.GA31277@quack.suse.cz> References: <1426593399-6549-1-git-send-email-jack@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1426593399-6549-1-git-send-email-jack@suse.cz> Sender: linux-media-owner@vger.kernel.org List-ID: 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? Honza -- Jan Kara SUSE Labs, CR From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f178.google.com (mail-wi0-f178.google.com [209.85.212.178]) by kanga.kvack.org (Postfix) with ESMTP id A68F56B0038 for ; Thu, 2 Apr 2015 11:03:05 -0400 (EDT) Received: by wibgn9 with SMTP id gn9so108927470wib.1 for ; Thu, 02 Apr 2015 08:03:05 -0700 (PDT) Received: from mx2.suse.de (cantor2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id md11si9914152wic.55.2015.04.02.08.03.03 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 02 Apr 2015 08:03:03 -0700 (PDT) Date: Thu, 2 Apr 2015 17:02:58 +0200 From: Jan Kara Subject: Re: [PATCH 0/9 v2] Helper to abstract vma handling in media layer Message-ID: <20150402150258.GA31277@quack.suse.cz> References: <1426593399-6549-1-git-send-email-jack@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1426593399-6549-1-git-send-email-jack@suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: linux-media@vger.kernel.org Cc: Hans Verkuil , Mauro Carvalho Chehab , linux-mm@kvack.org, dri-devel@lists.freedesktop.org, David Airlie , Jan Kara 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? Honza -- Jan Kara SUSE Labs, CR -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kara Subject: Re: [PATCH 0/9 v2] Helper to abstract vma handling in media layer Date: Thu, 2 Apr 2015 17:02:58 +0200 Message-ID: <20150402150258.GA31277@quack.suse.cz> References: <1426593399-6549-1-git-send-email-jack@suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by gabe.freedesktop.org (Postfix) with ESMTP id 6F72D6E451 for ; Thu, 2 Apr 2015 08:03:04 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1426593399-6549-1-git-send-email-jack@suse.cz> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: linux-media@vger.kernel.org Cc: Jan Kara , Mauro Carvalho Chehab , dri-devel@lists.freedesktop.org, linux-mm@kvack.org, Hans Verkuil List-Id: dri-devel@lists.freedesktop.org ICBIZWxsbywKCk9uIFR1ZSAxNy0wMy0xNSAxMjo1NjozMCwgSmFuIEthcmEgd3JvdGU6Cj4gICBB ZnRlciBhIGxvbmcgcGF1c2UgSSdtIHNlbmRpbmcgc2Vjb25kIHZlcnNpb24gb2YgbXkgcGF0Y2gg c2VyaWVzIHRvIGFic3RyYWN0Cj4gdm1hIGhhbmRsaW5nIGZyb20gdGhlIHZhcmlvdXMgbWVkaWEg ZHJpdmVycy4gQWZ0ZXIgdGhpcyBwYXRjaCBzZXQgZHJpdmVycyBoYXZlCj4gdG8ga25vdyBtdWNo IGxlc3MgZGV0YWlscyBhYm91dCB2bWFzLCB0aGVpciB0eXBlcywgYW5kIGxvY2tpbmcuIE15IG1v dGl2YXRpb24KPiBmb3IgdGhlIHNlcmllcyBpcyB0aGF0IEkgd2FudCB0byBjaGFuZ2UgZ2V0X3Vz ZXJfcGFnZXMoKSBsb2NraW5nIGFuZCBJIHdhbnQgdG8KPiBoYW5kbGUgc3VidGxlIGxvY2tpbmcg ZGV0YWlscyBpbiBhcyBmZXcgcGxhY2VzIGFzIHBvc3NpYmxlLgo+IAo+IFRoZSBjb3JlIG9mIHRo ZSBzZXJpZXMgaXMgdGhlIG5ldyBoZWxwZXIgZ2V0X3ZhZGRyX3BmbnMoKSB3aGljaCBpcyBnaXZl biBhCj4gdmlydHVhbCBhZGRyZXNzIGFuZCBpdCBmaWxscyBpbiBQRk5zIGludG8gcHJvdmlkZWQg YXJyYXkuIElmIFBGTnMgY29ycmVzcG9uZCB0bwo+IG5vcm1hbCBwYWdlcyBpdCBhbHNvIGdyYWJz IHJlZmVyZW5jZXMgdG8gdGhlc2UgcGFnZXMuIFRoZSBkaWZmZXJlbmNlIGZyb20KPiBnZXRfdXNl cl9wYWdlcygpIGlzIHRoYXQgdGhpcyBmdW5jdGlvbiBjYW4gYWxzbyBkZWFsIHdpdGggcGZubWFw LCBtaXhlZCwgYW5kIGlvCj4gbWFwcGluZ3Mgd2hpY2ggaXMgd2hhdCB0aGUgbWVkaWEgZHJpdmVy cyBuZWVkLgo+IAo+IEkgaGF2ZSB0ZXN0ZWQgdGhlIHBhdGNoZXMgd2l0aCB2aXZpZCBkcml2ZXIg c28gYXQgbGVhc3QgdmIyIGNvZGUgZ290IHNvbWUKPiBleHBvc3VyZS4gQ29udmVyc2lvbiBvZiBv dGhlciBkcml2ZXJzIHdhcyBqdXN0IGNvbXBpbGUtdGVzdGVkIHNvIEknZCBsaWtlIHRvCj4gYXNr IHJlc3BlY3RpdmUgbWFpbnRhaW5lcnMgaWYgdGhleSBjb3VsZCBoYXZlIGEgbG9vay4gIEFsc28g SSdkIGxpa2UgdG8gYXNrIG1tCj4gZm9sa3MgdG8gY2hlY2sgcGF0Y2ggMi85IGltcGxlbWVudGlu ZyB0aGUgaGVscGVyLiBUaGFua3MhCiAgUGluZz8gQW55IHJlYWN0aW9ucz8KCgkJCQkJCQkJSG9u emEKCi0tIApKYW4gS2FyYSA8amFja0BzdXNlLmN6PgpTVVNFIExhYnMsIENSCl9fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxp c3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==