From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ot1-f72.google.com (mail-ot1-f72.google.com [209.85.210.72]) by kanga.kvack.org (Postfix) with ESMTP id 3E0C36B2755 for ; Wed, 21 Nov 2018 15:05:18 -0500 (EST) Received: by mail-ot1-f72.google.com with SMTP id 62so3595884otr.14 for ; Wed, 21 Nov 2018 12:05:18 -0800 (PST) Received: from userp2130.oracle.com (userp2130.oracle.com. [156.151.31.86]) by mx.google.com with ESMTPS id e31si20389668otd.314.2018.11.21.12.05.17 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 21 Nov 2018 12:05:17 -0800 (PST) Subject: Re: [PATCH 0/9] Use vm_insert_range References: <20181115154314.GA27850@jordon-HP-15-Notebook-PC> <0c6f1144-6ee0-29df-5e1f-d35d2264e06e@oracle.com> From: Boris Ostrovsky Message-ID: Date: Wed, 21 Nov 2018 15:04:27 -0500 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: owner-linux-mm@kvack.org List-ID: To: Souptick Joarder Cc: Andrew Morton , Matthew Wilcox , Michal Hocko , "Kirill A. Shutemov" , vbabka@suse.cz, Rik van Riel , Stephen Rothwell , rppt@linux.vnet.ibm.com, Peter Zijlstra , Russell King - ARM Linux , robin.murphy@arm.com, iamjoonsoo.kim@lge.com, treding@nvidia.com, Kees Cook , Marek Szyprowski , stefanr@s5r6.in-berlin.de, hjc@rock-chips.com, Heiko Stuebner , airlied@linux.ie, oleksandr_andrushchenko@epam.com, joro@8bytes.org, pawel@osciak.com, Kyungmin Park , mchehab@kernel.org, Juergen Gross , linux-kernel@vger.kernel.org, Linux-MM , linux-arm-kernel@lists.infradead.org, linux1394-devel@lists.sourceforge.net, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, xen-devel@lists.xen.org, iommu@lists.linux-foundation.org, linux-media@vger.kernel.org On 11/21/18 2:56 PM, Souptick Joarder wrote: > On Thu, Nov 22, 2018 at 1:08 AM Boris Ostrovsky > wrote: >> On 11/21/18 1:24 AM, Souptick Joarder wrote: >>> On Thu, Nov 15, 2018 at 9:09 PM Souptick Joarder wrote: >>>> Previouly drivers have their own way of mapping range of >>>> kernel pages/memory into user vma and this was done by >>>> invoking vm_insert_page() within a loop. >>>> >>>> As this pattern is common across different drivers, it can >>>> be generalized by creating a new function and use it across >>>> the drivers. >>>> >>>> vm_insert_range is the new API which will be used to map a >>>> range of kernel memory/pages to user vma. >>>> >>>> All the applicable places are converted to use new vm_insert_range >>>> in this patch series. >>>> >>>> Souptick Joarder (9): >>>> mm: Introduce new vm_insert_range API >>>> arch/arm/mm/dma-mapping.c: Convert to use vm_insert_range >>>> drivers/firewire/core-iso.c: Convert to use vm_insert_range >>>> drm/rockchip/rockchip_drm_gem.c: Convert to use vm_insert_range >>>> drm/xen/xen_drm_front_gem.c: Convert to use vm_insert_range >>>> iommu/dma-iommu.c: Convert to use vm_insert_range >>>> videobuf2/videobuf2-dma-sg.c: Convert to use vm_insert_range >>>> xen/gntdev.c: Convert to use vm_insert_range >>>> xen/privcmd-buf.c: Convert to use vm_insert_range >>> Any further comment on driver changes ? >> Xen drivers (the last two patches) look fine to me. > Thanks, can I considered this as Reviewed-by ? Reviewed-by: Boris Ostrovsky