Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Received-SPF: Pass (protection.outlook.com: domain of arm.com designates
 63.35.35.123 as permitted sender) receiver=protection.outlook.com;
 client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com;
Authentication-Results-Original: linaro.org; dkim=none (message not signed)
 header.d=none;linaro.org; dmarc=none action=none header.from=arm.com;
Date:   Fri, 1 May 2020 11:42:16 +0100
From:   Brian Starkey <brian.starkey@arm.com>
To:     John Stultz <john.stultz@linaro.org>
Cc:     lkml <linux-kernel@vger.kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Sumit Semwal <sumit.semwal@linaro.org>,
        "Andrew F. Davis" <afd@ti.com>,
        Benjamin Gaignard <benjamin.gaignard@linaro.org>,
        Liam Mark <lmark@codeaurora.org>,
        Pratik Patel <pratikp@codeaurora.org>,
        Laura Abbott <labbott@redhat.com>,
        Chenbo Feng <fengc@google.com>,
        Alistair Strachan <astrachan@google.com>,
        Sandeep Patil <sspatil@google.com>,
        Hridya Valsaraju <hridya@google.com>,
        Christoph Hellwig <hch@lst.de>,
        Marek Szyprowski <m.szyprowski@samsung.com>,
        Robin Murphy <robin.murphy@arm.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org,
        linux-mm@kvack.org, nd@arm.com
Subject: Re: [RFC][PATCH 1/4] devicetree: bindings: Add linux,cma-heap tag
 for reserved memory
Message-ID: <20200501104216.4f226c2a7bzval5o@DESKTOP-E1NTVVP.localdomain>
References: <20200501073949.120396-1-john.stultz@linaro.org>
 <20200501073949.120396-2-john.stultz@linaro.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200501073949.120396-2-john.stultz@linaro.org>
User-Agent: NeoMutt/20180716-849-147d51-dirty
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from DESKTOP-E1NTVVP.localdomain (82.1.208.173) by LO2P265CA0147.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:9::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2958.27 via Frontend Transport; Fri, 1 May 2020 10:42:16 +0000
X-Originating-IP: [82.1.208.173]
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: 983f6900-2183-4a1d-35df-08d7edbc5702
X-MS-TrafficTypeDiagnostic: AM6PR08MB3112:|AM6PR08MB3112:|VI1PR08MB2942:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <VI1PR08MB2942BC2C94882EB43D49BADEF0AB0@VI1PR08MB2942.eurprd08.prod.outlook.com>
x-checkrecipientrouted: true
NoDisclaimer: true
X-MS-Oob-TLC-OOBClassifiers: OLM:9508;OLM:9508;
X-Forefront-PRVS: 0390DB4BDA
X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM6PR08MB3829.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(346002)(366004)(376002)(136003)(39860400002)(396003)(44832011)(86362001)(55016002)(6916009)(186003)(2906002)(5660300002)(1076003)(4326008)(54906003)(7416002)(316002)(8936002)(26005)(8676002)(52116002)(66476007)(66556008)(478600001)(7696005)(66946007)(6506007)(9686003)(956004)(16526019)(142933001);DIR:OUT;SFP:1101;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam-Untrusted: BCL:0;
X-Microsoft-Antispam-Message-Info-Original: GeLbvZ5GKnRLQ+DQMiNJ5K6mGWI55ym4qL128swDGro9bWVpHv6RVDr0393cdPy76zODHsWeZmJxy7+3qE1czObKKuEgu5ErbEZyj2wShI1qU2bLKD9hBJb7CUTeRks/vf95iOJeN/iSoYCfKt7ENahJwh0Pn2+zfdBiVwduqdjHGnq063q44ImpwXubSXUT/gfJPLN5/d84ya5I9aJPO/mjq6OoXwvaYcerVASxt9QDmlXlSn4Yk1KycQ0Rg6wKxpWNg9lM6sMBDbr7Lt8sGIXvn5jKVrePtAj15y6a/Z2xYOkow/8MXMalk0ogHoIh2Fv7/i3G9ugYXdhKutPCWG1KiRWszDnUV5mnLYApAUEU2qwiJQs5D0qZcb6v7ZSmr7pjxpG/AOOBl0C136w86FgsKaeKolsockoQKnlJdkIgvu2enbbyg/zFAQD9Ye6Q+KMQnlEz0VEYYf287Q00yhWWkc631oOSLBgXKbxEAsZzVxa1mzxUR/IqFpx6StSW
X-MS-Exchange-AntiSpam-MessageData: v1fGfe4rETbOV2BNMZx4jpkyAd2A9wPBeq6iSEAMF3KS4s379D6vUsec/pqSQ09KNfUHWc9farkqR/SdpsvqBde/UGo3MhA/gP7nYbdgkLkOS1Me1hYNv8eVvIjH9NvXM1vkLWiuCEZ5kJy3SX1lm/92iGA1uIlhuctGrLAfDI3g6ujZMxdIHjpHVw9ZwpF7ucYEgoxI4nGi4PBThRivCCbyGo4IKxrWGhtImiJJ2U7K0o5JZLeCH/aLkRAZt01I8Q0CHMxQiy74Rs/CPXaMRGYiTrez7FqH7N2eGv9aA8qHxrZC/IZDfP3/hOkmMkslZnvnoOy0QMf5N2CYXr2iv9CUhjwqtjEXhmFW9cA5z+lUEAyjiNEchnLW2Ne2iJInuPqarpLso7v+TF9PL9VBJoL9UJCxOJ5uAUa9sUiDisOPmQ4A0HLzRd05SzNwNVAvARq0wNlfQSzjmhr1b6/YCAGajxbJLBtA6UJwgyKt77eftPGKo900gAYyVYLU5wTvMZMMABZOy6nzuWCw3tzoBYmDdD+cjC+zfKe71qLyEqVHTFu2cLUeVn9UVnClXVH67MhOQCys1N63/VODyIogf0bO9GcFIHTQ9BgEKwK2FxReuoqOaNy9lm71KMbUi5KQELGld9gP2rZOIlIC7VGiYK9A5DonVVyPQBnHCmnAt5gvpLXpXtTpeYgMw4asCS8Uxe0CYwgw+ST1GuOa9/g68tEof1vI+RQthH37s4WhLRf9Yzzfb8TpzvXn4Otq9ms8A0I1/mtUFzLhEFImaNq7f5SM1RSfM268e/8Vz6BSxFM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR08MB3112
Original-Authentication-Results: linaro.org; dkim=none (message not signed)
 header.d=none;linaro.org; dmarc=none action=none header.from=arm.com;
X-EOPAttributedMessage: 0
X-MS-Exchange-Transport-CrossTenantHeadersStripped: VE1EUR03FT041.eop-EUR03.prod.protection.outlook.com
X-Forefront-Antispam-Report: CIP:63.35.35.123;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:64aa7808-outbound-1.mta.getcheckrecipient.com;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com;CAT:NONE;SFTY:;SFS:(4636009)(396003)(136003)(376002)(346002)(39860400002)(46966005)(356005)(6506007)(450100002)(81166007)(70586007)(55016002)(1076003)(4326008)(5660300002)(316002)(8936002)(186003)(8676002)(9686003)(54906003)(26005)(6862004)(7696005)(86362001)(82310400002)(70206006)(36906005)(47076004)(336012)(44832011)(956004)(2906002)(478600001)(16526019)(82740400003)(142933001);DIR:OUT;SFP:1101;
X-MS-Office365-Filtering-Correlation-Id-Prvs: e22157a5-6c38-4ed2-0540-08d7edbc5153
X-Forefront-PRVS: 0390DB4BDA
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: jsKh9tN4Deji4JC6o/nLKzSMexNgKOG/pw8pGECuts380DKnd1JBCS8qV7qk8oOX/MYEPzeYrCJ13gnGuLwKdlfdUmLThE5g642o9daGQkhB7IRF43TtUnxhxn1mq0gOyn5TBkKYeIHfjBNTsh6frWuhV7uSXYihd60sl/B1cvG3F03xb25ZGA0m7HCJwyFwLH+V/5YRscK7DHYg273tfOXT2++rR4JiTSQ7idaOOnozZb07MoXr/r73OqSS58drqEjJobDOH9BBxMHYwegdmgrH7Kx8xZXXGNz3TJ91z3430ykoa5OyrvQFMfolSXhsK8dS3NfWuWVmeUXv3hVy2P+SuLXBrDnLY7X4Nfz7dtFLFPPW0LKxc2qQ49GSO5pkb3uyTvU/JfSZjyx484C9x8j1citMMwHS1aisVIrEcaLHnPT02nUxqUnLNsczZf2HqKVbi8deix1LjljnJ67WPSb7VvPINgWptH6sdlj/st0w2GRseo1tOBMlTwaUsmRJGm2N8JGWYOzd+wMAkJQYdYunirVn8im13LkwglPfqJg=
X-OriginatorOrg: arm.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 May 2020 10:42:26.4192
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 983f6900-2183-4a1d-35df-08d7edbc5702
X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com]
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR08MB2942
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

On Fri, May 01, 2020 at 07:39:46AM +0000, John Stultz wrote:
> This patch adds a linux,cma-heap property for CMA reserved memory
> regions, which will be used to allow the region to be exposed via
> the DMA-BUF Heaps interface
> 
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Sumit Semwal <sumit.semwal@linaro.org>
> Cc: "Andrew F. Davis" <afd@ti.com>
> Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Cc: Liam Mark <lmark@codeaurora.org>
> Cc: Pratik Patel <pratikp@codeaurora.org>
> Cc: Laura Abbott <labbott@redhat.com>
> Cc: Brian Starkey <Brian.Starkey@arm.com>
> Cc: Chenbo Feng <fengc@google.com>
> Cc: Alistair Strachan <astrachan@google.com>
> Cc: Sandeep Patil <sspatil@google.com>
> Cc: Hridya Valsaraju <hridya@google.com>
> Cc: Christoph Hellwig <hch@lst.de>
> Cc: Marek Szyprowski <m.szyprowski@samsung.com>
> Cc: Robin Murphy <robin.murphy@arm.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: devicetree@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-mm@kvack.org
> Signed-off-by: John Stultz <john.stultz@linaro.org>
> ---
>  .../devicetree/bindings/reserved-memory/reserved-memory.txt    | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> index bac4afa3b197..e97b6a4c3bc0 100644
> --- a/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> +++ b/Documentation/devicetree/bindings/reserved-memory/reserved-memory.txt
> @@ -68,6 +68,9 @@ Linux implementation note:
>  - If a "linux,cma-default" property is present, then Linux will use the
>    region for the default pool of the contiguous memory allocator.
>  
> +- If a "linux,cma-heap" property is present, then Linux will expose the
> +  the CMA region via the DMA-BUF Heaps interface.
> +

Would it be useful or even possible to give some indication of what
the heap will end up being called? I'm afraid I don't remember what if
any conclusions came out of previous discussions on UAPI for heap
enumeration.

I suppose CMA names haven't been relevant to userspace before, but
they perhaps would be with this change.

Alternatively, leaving it effectively undefined doesn't tie us down,
and something like links in sysfs can be added as a richer API in the
future.

Cheers,
-Brian

>  - If a "linux,dma-default" property is present, then Linux will use the
>    region for the default pool of the consistent DMA allocator.
>  
> -- 
> 2.17.1
>