From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans Verkuil Subject: Re: [PATCH 0/9 v2] Helper to abstract vma handling in media layer Date: Thu, 02 Apr 2015 17:25:48 +0200 Message-ID: <551D5F7C.4080400@xs4all.nl> References: <1426593399-6549-1-git-send-email-jack@suse.cz> <20150402150258.GA31277@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150402150258.GA31277@quack.suse.cz> Sender: linux-media-owner@vger.kernel.org To: Jan Kara , linux-media@vger.kernel.org Cc: Hans Verkuil , Mauro Carvalho Chehab , linux-mm@kvack.org, dri-devel@lists.freedesktop.org, David Airlie , Marek Szyprowski , Pawel Osciak List-Id: dri-devel@lists.freedesktop.org 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 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wi0-f182.google.com (mail-wi0-f182.google.com [209.85.212.182]) by kanga.kvack.org (Postfix) with ESMTP id 677496B006E for ; Thu, 2 Apr 2015 11:26:22 -0400 (EDT) Received: by wixm2 with SMTP id m2so58160676wix.0 for ; Thu, 02 Apr 2015 08:26:21 -0700 (PDT) Received: from lb2-smtp-cloud3.xs4all.net (lb2-smtp-cloud3.xs4all.net. [194.109.24.26]) by mx.google.com with ESMTPS id u15si9276021wjr.155.2015.04.02.08.26.20 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Thu, 02 Apr 2015 08:26:20 -0700 (PDT) Message-ID: <551D5F7C.4080400@xs4all.nl> Date: Thu, 02 Apr 2015 17:25:48 +0200 From: Hans Verkuil MIME-Version: 1.0 Subject: Re: [PATCH 0/9 v2] Helper to abstract vma handling in media layer References: <1426593399-6549-1-git-send-email-jack@suse.cz> <20150402150258.GA31277@quack.suse.cz> In-Reply-To: <20150402150258.GA31277@quack.suse.cz> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Jan Kara , linux-media@vger.kernel.org Cc: Hans Verkuil , Mauro Carvalho Chehab , linux-mm@kvack.org, dri-devel@lists.freedesktop.org, David Airlie , Marek Szyprowski , Pawel Osciak 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 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 -- 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