From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by aws-us-west-2-korg-lkml-1.web.codeaurora.org (Postfix) with ESMTP id 2B324C433EF for ; Wed, 13 Jun 2018 07:08:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D5800208AF for ; Wed, 13 Jun 2018 07:08:08 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="rqcDYain" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D5800208AF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934638AbeFMHIG (ORCPT ); Wed, 13 Jun 2018 03:08:06 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:43211 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934115AbeFMHIC (ORCPT ); Wed, 13 Jun 2018 03:08:02 -0400 Received: by mail-lf0-f67.google.com with SMTP id n15-v6so2226865lfn.10; Wed, 13 Jun 2018 00:08:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding:content-language; bh=QAWQEoFARIkG4dOz9CjfoMlxDtMdgyCbNtsv05T+MNk=; b=rqcDYain9scA8QH04hgPOzZtlmGuC+RPYChoP6Bp3PlXdx9UO8xMh8ZnDYcuG/BIJn YOsBbJTV8Zb0vJSMAb9YvckLOI/lnaaE7DGOuYf5nWmYmqwdIUK1HR0K0pbx+xKEuUjl TjpWVcdi6JJQX9HAJdEaaJhsmU4t8Zsk0GM8rnwD8gjfcaA2GAdTQCW1LDKDfF7h8urh k9eoWjwFItObLigdpiX+NooAW+gXUxo6Z3MV6KX3sg64FG/mNVRbxrUp9C/UBimVWrk3 WbCM7Hgt1X8PRklvPQTUXhVpht8bSVKt3lP3w+txX/hBKWTaKNLuOVorUZL9n7gcDQfr VWCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding :content-language; bh=QAWQEoFARIkG4dOz9CjfoMlxDtMdgyCbNtsv05T+MNk=; b=Rs8IPHJPK26fuaAspUc2YAUMBknvlBBIa631Sh7FaVe8QA3Y0XJdTQzPsh22/ok4+H sx1JifzHKsO8N5c7dSjGOrCqeFm/TblohSUO7PpLHBFo2QakOF/xJB2jDglykeEmim9M hgfu+ODQgY+ri0ysjOUZJU9VkvYnXDdFA2/xWScpzv32JeQOU2DCSqkQyoCv1a06uBgh xtUQxDXVTFlTtjqH2gdaa6Q2cypgFhLJ+GrMehgSW5eiOpN33ngulP6hEttRcn+aKbkN xe5+mAzyuBF5wDc9VGe3xPbLehPHgow+1dpPNSftRwUA9XYbiij0gzNNVt6jdz0EcViP Kl5A== X-Gm-Message-State: APt69E3orrbUjFavkeIX9USYTxxk87U/T37tNRhfr7IoWi3oftkIrysF RIN0zMJuZx4GMAaDZ7RIpjI= X-Google-Smtp-Source: ADUXVKL34UC9rgQU/FcGLE2lFkO6pMq50kHE837qLRxFcamq39BotrbS+FUIYGdOoNiDR/amRTxwGg== X-Received: by 2002:a2e:9d5:: with SMTP id 204-v6mr2189640ljj.40.1528873680613; Wed, 13 Jun 2018 00:08:00 -0700 (PDT) Received: from [10.17.182.9] (ll-74.141.223.85.sovam.net.ua. [85.223.141.74]) by smtp.gmail.com with ESMTPSA id v25-v6sm407244ljh.57.2018.06.13.00.07.58 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 13 Jun 2018 00:07:59 -0700 (PDT) Subject: Re: [PATCH v3 4/9] xen/grant-table: Allow allocating buffers suitable for DMA To: Boris Ostrovsky , xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, jgross@suse.com, konrad.wilk@oracle.com Cc: daniel.vetter@intel.com, dongwon.kim@intel.com, matthew.d.roper@intel.com, Oleksandr Andrushchenko References: <20180612134200.17456-1-andr2000@gmail.com> <20180612134200.17456-5-andr2000@gmail.com> <4ab26c9a-155a-cd04-fbf6-c38c6429959b@oracle.com> From: Oleksandr Andrushchenko Message-ID: Date: Wed, 13 Jun 2018 10:07:58 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 In-Reply-To: <4ab26c9a-155a-cd04-fbf6-c38c6429959b@oracle.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/13/2018 04:12 AM, Boris Ostrovsky wrote: > > > On 06/12/2018 09:41 AM, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko >> >> Extend grant table module API to allow allocating buffers that can >> be used for DMA operations and mapping foreign grant references >> on top of those. >> The resulting buffer is similar to the one allocated by the balloon >> driver in terms that proper memory reservation is made >> ({increase|decrease}_reservation and VA mappings updated if needed). >> This is useful for sharing foreign buffers with HW drivers which >> cannot work with scattered buffers provided by the balloon driver, >> but require DMAable memory instead. >> >> Signed-off-by: Oleksandr Andrushchenko >> > > Reviewed-by: Boris Ostrovsky > > with a small nit below > > >> --- >>   drivers/xen/Kconfig       | 13 ++++++ >>   drivers/xen/grant-table.c | 97 +++++++++++++++++++++++++++++++++++++++ >>   include/xen/grant_table.h | 18 ++++++++ >>   3 files changed, 128 insertions(+) >> >> diff --git a/drivers/xen/Kconfig b/drivers/xen/Kconfig >> index e5d0c28372ea..39536ddfbce4 100644 >> --- a/drivers/xen/Kconfig >> +++ b/drivers/xen/Kconfig >> @@ -161,6 +161,19 @@ config XEN_GRANT_DEV_ALLOC >>         to other domains. This can be used to implement frontend drivers >>         or as part of an inter-domain shared memory channel. >>   +config XEN_GRANT_DMA_ALLOC >> +    bool "Allow allocating DMA capable buffers with grant reference >> module" >> +    depends on XEN && HAS_DMA >> +    help >> +      Extends grant table module API to allow allocating DMA capable >> +      buffers and mapping foreign grant references on top of it. >> +      The resulting buffer is similar to one allocated by the balloon >> +      driver in terms that proper memory reservation is made >> +      ({increase|decrease}_reservation and VA mappings updated if >> needed). > > I think you should drop the word "terms" and say "is made *by*" and > "VA mappings *are* updated" > > And similar change in the commit message. > Will, change, thank you > -boris > From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleksandr Andrushchenko Subject: Re: [PATCH v3 4/9] xen/grant-table: Allow allocating buffers suitable for DMA Date: Wed, 13 Jun 2018 10:07:58 +0300 Message-ID: References: <20180612134200.17456-1-andr2000@gmail.com> <20180612134200.17456-5-andr2000@gmail.com> <4ab26c9a-155a-cd04-fbf6-c38c6429959b@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-lf0-x242.google.com (mail-lf0-x242.google.com [IPv6:2a00:1450:4010:c07::242]) by gabe.freedesktop.org (Postfix) with ESMTPS id 481646E5DF for ; Wed, 13 Jun 2018 07:08:02 +0000 (UTC) Received: by mail-lf0-x242.google.com with SMTP id 36-v6so2219293lfr.11 for ; Wed, 13 Jun 2018 00:08:02 -0700 (PDT) In-Reply-To: <4ab26c9a-155a-cd04-fbf6-c38c6429959b@oracle.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Boris Ostrovsky , xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, jgross@suse.com, konrad.wilk@oracle.com Cc: daniel.vetter@intel.com, dongwon.kim@intel.com, Oleksandr Andrushchenko List-Id: dri-devel@lists.freedesktop.org T24gMDYvMTMvMjAxOCAwNDoxMiBBTSwgQm9yaXMgT3N0cm92c2t5IHdyb3RlOgo+Cj4KPiBPbiAw Ni8xMi8yMDE4IDA5OjQxIEFNLCBPbGVrc2FuZHIgQW5kcnVzaGNoZW5rbyB3cm90ZToKPj4gRnJv bTogT2xla3NhbmRyIEFuZHJ1c2hjaGVua28gPG9sZWtzYW5kcl9hbmRydXNoY2hlbmtvQGVwYW0u Y29tPgo+Pgo+PiBFeHRlbmQgZ3JhbnQgdGFibGUgbW9kdWxlIEFQSSB0byBhbGxvdyBhbGxvY2F0 aW5nIGJ1ZmZlcnMgdGhhdCBjYW4KPj4gYmUgdXNlZCBmb3IgRE1BIG9wZXJhdGlvbnMgYW5kIG1h cHBpbmcgZm9yZWlnbiBncmFudCByZWZlcmVuY2VzCj4+IG9uIHRvcCBvZiB0aG9zZS4KPj4gVGhl IHJlc3VsdGluZyBidWZmZXIgaXMgc2ltaWxhciB0byB0aGUgb25lIGFsbG9jYXRlZCBieSB0aGUg YmFsbG9vbgo+PiBkcml2ZXIgaW4gdGVybXMgdGhhdCBwcm9wZXIgbWVtb3J5IHJlc2VydmF0aW9u IGlzIG1hZGUKPj4gKHtpbmNyZWFzZXxkZWNyZWFzZX1fcmVzZXJ2YXRpb24gYW5kIFZBIG1hcHBp bmdzIHVwZGF0ZWQgaWYgbmVlZGVkKS4KPj4gVGhpcyBpcyB1c2VmdWwgZm9yIHNoYXJpbmcgZm9y ZWlnbiBidWZmZXJzIHdpdGggSFcgZHJpdmVycyB3aGljaAo+PiBjYW5ub3Qgd29yayB3aXRoIHNj YXR0ZXJlZCBidWZmZXJzIHByb3ZpZGVkIGJ5IHRoZSBiYWxsb29uIGRyaXZlciwKPj4gYnV0IHJl cXVpcmUgRE1BYWJsZSBtZW1vcnkgaW5zdGVhZC4KPj4KPj4gU2lnbmVkLW9mZi1ieTogT2xla3Nh bmRyIEFuZHJ1c2hjaGVua28gCj4+IDxvbGVrc2FuZHJfYW5kcnVzaGNoZW5rb0BlcGFtLmNvbT4K Pgo+IFJldmlld2VkLWJ5OiBCb3JpcyBPc3Ryb3Zza3kgPGJvcmlzLm9zdHJvdnNreUBvcmFjbGUu Y29tPgo+Cj4gd2l0aCBhIHNtYWxsIG5pdCBiZWxvdwo+Cj4KPj4gLS0tCj4+IMKgIGRyaXZlcnMv eGVuL0tjb25maWfCoMKgwqDCoMKgwqAgfCAxMyArKysrKysKPj4gwqAgZHJpdmVycy94ZW4vZ3Jh bnQtdGFibGUuYyB8IDk3ICsrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKwo+ PiDCoCBpbmNsdWRlL3hlbi9ncmFudF90YWJsZS5oIHwgMTggKysrKysrKysKPj4gwqAgMyBmaWxl cyBjaGFuZ2VkLCAxMjggaW5zZXJ0aW9ucygrKQo+Pgo+PiBkaWZmIC0tZ2l0IGEvZHJpdmVycy94 ZW4vS2NvbmZpZyBiL2RyaXZlcnMveGVuL0tjb25maWcKPj4gaW5kZXggZTVkMGMyODM3MmVhLi4z OTUzNmRkZmJjZTQgMTAwNjQ0Cj4+IC0tLSBhL2RyaXZlcnMveGVuL0tjb25maWcKPj4gKysrIGIv ZHJpdmVycy94ZW4vS2NvbmZpZwo+PiBAQCAtMTYxLDYgKzE2MSwxOSBAQCBjb25maWcgWEVOX0dS QU5UX0RFVl9BTExPQwo+PiDCoMKgwqDCoMKgwqDCoCB0byBvdGhlciBkb21haW5zLiBUaGlzIGNh biBiZSB1c2VkIHRvIGltcGxlbWVudCBmcm9udGVuZCBkcml2ZXJzCj4+IMKgwqDCoMKgwqDCoMKg IG9yIGFzIHBhcnQgb2YgYW4gaW50ZXItZG9tYWluIHNoYXJlZCBtZW1vcnkgY2hhbm5lbC4KPj4g wqAgK2NvbmZpZyBYRU5fR1JBTlRfRE1BX0FMTE9DCj4+ICvCoMKgwqAgYm9vbCAiQWxsb3cgYWxs b2NhdGluZyBETUEgY2FwYWJsZSBidWZmZXJzIHdpdGggZ3JhbnQgcmVmZXJlbmNlIAo+PiBtb2R1 bGUiCj4+ICvCoMKgwqAgZGVwZW5kcyBvbiBYRU4gJiYgSEFTX0RNQQo+PiArwqDCoMKgIGhlbHAK Pj4gK8KgwqDCoMKgwqAgRXh0ZW5kcyBncmFudCB0YWJsZSBtb2R1bGUgQVBJIHRvIGFsbG93IGFs bG9jYXRpbmcgRE1BIGNhcGFibGUKPj4gK8KgwqDCoMKgwqAgYnVmZmVycyBhbmQgbWFwcGluZyBm b3JlaWduIGdyYW50IHJlZmVyZW5jZXMgb24gdG9wIG9mIGl0Lgo+PiArwqDCoMKgwqDCoCBUaGUg cmVzdWx0aW5nIGJ1ZmZlciBpcyBzaW1pbGFyIHRvIG9uZSBhbGxvY2F0ZWQgYnkgdGhlIGJhbGxv b24KPj4gK8KgwqDCoMKgwqAgZHJpdmVyIGluIHRlcm1zIHRoYXQgcHJvcGVyIG1lbW9yeSByZXNl cnZhdGlvbiBpcyBtYWRlCj4+ICvCoMKgwqDCoMKgICh7aW5jcmVhc2V8ZGVjcmVhc2V9X3Jlc2Vy dmF0aW9uIGFuZCBWQSBtYXBwaW5ncyB1cGRhdGVkIGlmIAo+PiBuZWVkZWQpLgo+Cj4gSSB0aGlu ayB5b3Ugc2hvdWxkIGRyb3AgdGhlIHdvcmQgInRlcm1zIiBhbmQgc2F5ICJpcyBtYWRlICpieSoi IGFuZCAKPiAiVkEgbWFwcGluZ3MgKmFyZSogdXBkYXRlZCIKPgo+IEFuZCBzaW1pbGFyIGNoYW5n ZSBpbiB0aGUgY29tbWl0IG1lc3NhZ2UuCj4KV2lsbCwgY2hhbmdlLCB0aGFuayB5b3UKPiAtYm9y aXMKPgoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJp LWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBz Oi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==