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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6216BC6FA82 for ; Wed, 14 Sep 2022 17:13:57 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E6F258D0002; Wed, 14 Sep 2022 13:13:56 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DF9578D0001; Wed, 14 Sep 2022 13:13:56 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C71A18D0002; Wed, 14 Sep 2022 13:13:56 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id B6E358D0001 for ; Wed, 14 Sep 2022 13:13:56 -0400 (EDT) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay09.hostedemail.com (Postfix) with ESMTP id 89DA980B5E for ; Wed, 14 Sep 2022 17:13:56 +0000 (UTC) X-FDA: 79911338472.05.B6FDB97 Received: from mail-qt1-f180.google.com (mail-qt1-f180.google.com [209.85.160.180]) by imf19.hostedemail.com (Postfix) with ESMTP id 2A9C31A0088 for ; Wed, 14 Sep 2022 17:13:55 +0000 (UTC) Received: by mail-qt1-f180.google.com with SMTP id j10so8872163qtv.4 for ; Wed, 14 Sep 2022 10:13:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date; bh=IYYxWUkQ4C0EsxoWOU79iFj20ViO8vkpHbae7xo5rcs=; b=Na0/xpW50l0p2LWBaIM/XHkvizf5pdNgEm4MPvDGNs4tfXsVzy3nI2WggDrVrrfbfR 8vmNnBl+TiJloFytXKKx8ij7l7JHUAO5qqumCGMHz6gej+TwY2yY1uOnd6rQDW/onfEy mhfMenDe81IYCFu4mQwRrjZDNi+aIw2ZMXMs+lAyEOjSqv3xEgtOBhmny+WULZS0YZpw ZcZTsS7nWNzFQLBA2LcSPlF+DsEm3gxpzQ/M7fhQy4+5Bt/EF4tS+d1c9J4vprzq2WbL WOk5WKySaKYD+sBXvttDusj4zJ0fsKNQJHk9ppoZkcp4ZXt4nHQMWhY5rTa+LYvW5jK+ LGIA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date; bh=IYYxWUkQ4C0EsxoWOU79iFj20ViO8vkpHbae7xo5rcs=; b=mvzkgkG+SLGPf6cXk+ZT2Pn6OmvNxdf6353BPl7ZZOSZ3RkRD6Ru0+81PEtKLkvT/7 MQmg4sRnVIszVwnDKNwex6m6W+PvqZWMFdChnLz8RL+13QIDQjOJtkxUu5QtMIVosQT5 cmctbXIowTh/tCfiUbiiyP8tupmHAMY79Pm+B9MOh5p0c5vjaZgjLM1+5EicEdoi+5Pv /cmmj+AIrUDUuZwQ3k8KkipWKNp3E8HU2aTMxbF6wXMGIF0Z5GNksR871UFFMz1Lvwnc 6viD5eAl+Yxy0OMt35CTYwnWpxZZpxkDK+N6SQntfm1T2iU7hXXMlKK3BUCLcYSuGCd3 MDLw== X-Gm-Message-State: ACgBeo0z8F0/vsylwA0niqvUWMIeqqRShnV9t0pI5hYM5ofS0J/lAW6C /1nzltpItmiMOXFfhLJWZ4k= X-Google-Smtp-Source: AA6agR6pteFhhNot1KtRms4GIZpWVbQhIx9TFnmz61Z566R8w8B0323kiW2uAkt7jf8sPbn99v0wlg== X-Received: by 2002:ac8:5e51:0:b0:343:483:f9d6 with SMTP id i17-20020ac85e51000000b003430483f9d6mr33904642qtx.244.1663175635331; Wed, 14 Sep 2022 10:13:55 -0700 (PDT) Received: from [10.69.40.226] ([192.19.223.252]) by smtp.gmail.com with ESMTPSA id h16-20020ac85850000000b00342f8d4d0basm1990133qth.43.2022.09.14.10.13.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 14 Sep 2022 10:13:54 -0700 (PDT) Message-ID: <57f19774-39a1-03a6-fe68-83d7e4b16521@gmail.com> Date: Wed, 14 Sep 2022 10:13:51 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.1.2 Subject: Re: [PATCH 16/21] dt-bindings: reserved-memory: introduce designated-movable-block Content-Language: en-US To: Rob Herring Cc: Andrew Morton , Jonathan Corbet , Krzysztof Kozlowski , Frank Rowand , Mike Kravetz , Muchun Song , Mike Rapoport , Christoph Hellwig , Marek Szyprowski , Robin Murphy , Borislav Petkov , "Paul E. McKenney" , Neeraj Upadhyay , Randy Dunlap , Damien Le Moal , Florian Fainelli , David Hildenbrand , Zi Yan , Oscar Salvador , Hari Bathini , Kees Cook , - , KOSAKI Motohiro , Mel Gorman , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-mm@kvack.org, iommu@lists.linux.dev References: <20220913195508.3511038-1-opendmb@gmail.com> <20220913195508.3511038-17-opendmb@gmail.com> <20220914145506.GA2149379-robh@kernel.org> From: Doug Berger In-Reply-To: <20220914145506.GA2149379-robh@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1663175636; a=rsa-sha256; cv=none; b=zXVZ/E9z9mK9502+T6WbTPra92vQK0CYeGDpw53Ye+aMA3oexRdH4RJaFNSYf32abmKMoe mLcDD3gv76bzqsku2GJ5e3ATsywvSsgnIcVd+gqyQj2tqoRjXmadud9dWVdFx5kUGsq/2i TGG2eCJQx+fojvKdKsu/ETR8c4fvO8s= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b="Na0/xpW5"; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf19.hostedemail.com: domain of opendmb@gmail.com designates 209.85.160.180 as permitted sender) smtp.mailfrom=opendmb@gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1663175636; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=IYYxWUkQ4C0EsxoWOU79iFj20ViO8vkpHbae7xo5rcs=; b=LqvBzmulpeC+CiY3NH0qti3+9uM6xtcG0Y4Uk0cjCPbSgrAWH4SKDdhm2VQe4NsSFXbQeO pfXzczDN2I0GcDSXQnlUINkQ5bjE1SCaaJ2d+Y8SBwfdX01Vmti9LwW4q3VRekgPmGkSVI I2I9WbEXNLAksYPrXbwjVzqAFy3/W4w= X-Rspam-User: X-Rspamd-Queue-Id: 2A9C31A0088 X-Rspamd-Server: rspam05 X-Stat-Signature: ut446cpgpin4z8g9om8gcya5dihmo8d1 Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b="Na0/xpW5"; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf19.hostedemail.com: domain of opendmb@gmail.com designates 209.85.160.180 as permitted sender) smtp.mailfrom=opendmb@gmail.com X-HE-Tag: 1663175635-554314 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On 9/14/2022 7:55 AM, Rob Herring wrote: > On Tue, Sep 13, 2022 at 12:55:03PM -0700, Doug Berger wrote: >> Introduce designated-movable-block.yaml to document the >> devicetree binding for Designated Movable Block children of the >> reserved-memory node. > > What is a Designated Movable Block? This patch needs to stand on its > own. As noted in my reply to your [PATCH 00/21] comment, my intention in submitting the entire patch set (and specifically PATCH 00/21]) was to communicate this context. Now that I believe I understand that only this patch should have been submitted to the devicetree-spec mailing list, I will strive harder to make it more self contained. > > Why does this belong or need to be in DT? While my preferred method of declaring Designated Movable Blocks is through the movablecore kernel parameter, I can conceive that others may wish to take advantage of the reserved-memory DT nodes. In particular, it has the advantage that a device can claim ownership of the reserved-memory via device tree, which is something that has yet to be implemented for DMBs defined with movablecore. > >> >> Signed-off-by: Doug Berger >> --- >> .../designated-movable-block.yaml | 51 +++++++++++++++++++ >> 1 file changed, 51 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/reserved-memory/designated-movable-block.yaml >> >> diff --git a/Documentation/devicetree/bindings/reserved-memory/designated-movable-block.yaml b/Documentation/devicetree/bindings/reserved-memory/designated-movable-block.yaml >> new file mode 100644 >> index 000000000000..42f846069a2e >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/reserved-memory/designated-movable-block.yaml >> @@ -0,0 +1,51 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/reserved-memory/designated-movable-block.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: /reserved-memory Designated Movable Block node binding >> + >> +maintainers: >> + - devicetree-spec@vger.kernel.org >> + >> +allOf: >> + - $ref: "reserved-memory.yaml" >> + >> +properties: >> + compatible: >> + const: designated-movable-block >> + description: >> + This indicates a region of memory meant to be placed into >> + ZONE_MOVABLE. > > Don't put Linuxisms into bindings. I will avoid ZONE_MOVABLE if this commit is included in V2 of this patch set. > >> + >> +unevaluatedProperties: false >> + >> +required: >> + - compatible >> + - reusable >> + >> +examples: >> + - | >> + reserved-memory { >> + #address-cells = <0x2>; >> + #size-cells = <0x2>; >> + >> + DMB0@10800000 { >> + compatible = "designated-movable-block"; >> + reusable; >> + reg = <0x0 0x10800000 0x0 0x2d800000>; >> + }; >> + >> + DMB1@40000000 { >> + compatible = "designated-movable-block"; >> + reusable; >> + reg = <0x0 0x40000000 0x0 0x30000000>; >> + }; >> + >> + DMB2@80000000 { >> + compatible = "designated-movable-block"; >> + reusable; >> + reg = <0x0 0x80000000 0x0 0x2fc00000>; >> + }; >> + }; >> -- >> 2.25.1 >> >> Thank you for the review! -Doug