Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 14 Jul 2022 08:44:00 -0700
From:   Dan Williams <dan.j.williams@intel.com>
To:     Ira Weiny <ira.weiny@intel.com>,
        Matthew Wilcox <willy@infradead.org>
CC:     Dan Williams <dan.j.williams@intel.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        "Alison Schofield" <alison.schofield@intel.com>,
        Vishal Verma <vishal.l.verma@intel.com>,
        <linux-kernel@vger.kernel.org>, <linux-cxl@vger.kernel.org>,
        <linux-pci@vger.kernel.org>, <linux-fsdevel@vger.kernel.org>
Subject: Re: [RFC PATCH 1/3] xarray: Introduce devm_xa_init()
Message-ID: <62d039c0cfc13_16fb972943f@dwillia2-xfh.jf.intel.com.notmuch>
References: <20220705232159.2218958-1-ira.weiny@intel.com>
 <20220705232159.2218958-2-ira.weiny@intel.com>
 <YshE/pwSUBPAeybU@casper.infradead.org>
 <YshGSgHiAiu9QwiZ@iweiny-desk3>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <YshGSgHiAiu9QwiZ@iweiny-desk3>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?F1IKuNaiAXiJElu17FwdK8KJ7rTV08GwJluboTi6EqSHlDZn5EECuEdpUc2j?=
 =?us-ascii?Q?o09qr6+MR9ij2+MapqsXHu+0xHh3K18p+tq7OcGD1pcaP5tDZivIfc0YOBCf?=
 =?us-ascii?Q?KgN3WXGnYZ23lVT5TmTQLtmu/J0ARWdj5we8ov66ljoU4jB87/ES9RmTO/Q2?=
 =?us-ascii?Q?z46LZBwKoGKSs0NKWmIHaPc5VEr7BHI+rQ05qUBm8QvuQp46B58RB1wuonqm?=
 =?us-ascii?Q?n3xlfn0wdhHQMTK7R9Yqrr+YskU+vvpulaP+0xCevMWu0+AsUKS5I6MhROK3?=
 =?us-ascii?Q?365gzJnCXsxPNMo49+TMTK19xOx3mmEMQLAyEZgB6Jm/BbaCJMkzCUmyiWCs?=
 =?us-ascii?Q?FKq41u9/ImtwPIzNJReII+145ZZLlrAVBQU3csW/ESOOaNvkzk0UDuKp4TGi?=
 =?us-ascii?Q?OQN4d1DF/bmixH1PLNqS76tCtcOk7pEcwJUqXnMdcsqArw86OSGHrCDaLET+?=
 =?us-ascii?Q?ninWqGvPYykd/+anXCcKGDjUyPm2Q0SoVn7Zk2tvqCgnqJKcX/ZOOFOMsGnf?=
 =?us-ascii?Q?tmOm6KlGrutYQEthYqk7NwT4Eb7pwtMKjHIni3i9BTOFOE2fGVD0asCROVQN?=
 =?us-ascii?Q?ju6gYbqVHhGJpDQLZQLKlOgqJMf/gS/Cqg3IPtFk7Daqw8wJFq+/KE6vydF3?=
 =?us-ascii?Q?4i01+JC7Lm5TyO5jstwuQCUVNpgQfxja6VH0HDoD4eSpjgzBV/yWPKWwxI6s?=
 =?us-ascii?Q?nPw8NK9en63SFXfs0KBGFNaV90R+epCw9GqU9EfL7ElJ01yEGc/vLEpCCxxg?=
 =?us-ascii?Q?QlOkcBkfnHSSu/hOyn8+CEw1Fao42Ameuw0eMNXNbM5LUxTiALVqkb0CB+Lu?=
 =?us-ascii?Q?FlHWPKyAdAiCFq40AysD9yc7H5BK9xvXZNsltR59VMSvv5wUP3xHZvHiIUNq?=
 =?us-ascii?Q?jF4VXvAQil+jQXZiOG9vcc7oSYdn4QO6AI7ON73QakQhPuoTpjmOyfIb7jUH?=
 =?us-ascii?Q?Pr+eq7MQbX09EUlOJJi45/AHS4EIjEknJArPtSWHSmZnFnRO8bdCWXmoFtT+?=
 =?us-ascii?Q?9bzFHBUJmPbzGpLpqR6mDfKVBQ9Q4K9gE2IRDq+ZaiTenndulNdHljX0rhPh?=
 =?us-ascii?Q?NokQDyT+LxibZY62jU2DmiWt9PP9PCcjIc+q8oem8S4si9Fomgee3+VIpg2L?=
 =?us-ascii?Q?nH+YDkMQKyiKxdXpdeuI2/BiNs/M0PoTDdkWPXgB+oHha90cxOdhb+hZYIjA?=
 =?us-ascii?Q?nagDGQacNqOVQhcTPMmdjAOw7ovPLA3eJFVq8ycnh8/9NXiSJf33o/jivOq5?=
 =?us-ascii?Q?9X1lkCrDCiwbqsopgUM24J2CmoKdM1hvXLn/thyvv2obrzNXdg23kaRbWZTp?=
 =?us-ascii?Q?tipIDURBWM3IglGGDe8BU/gek0oBTWaLuLfRrVNgUABJFMrqM7QiyI4H/see?=
 =?us-ascii?Q?9CJzPmAhnCOsm+YmDyZ3ydwAZhiNfv/91WMWk3OmuktI+euPVzb/VHjrzr76?=
 =?us-ascii?Q?4KVhprk81VwdYf7Vrn3OjfoZp7tN1pJjJbiL+4+GCh7GNlaiKKDXGKsR0ETY?=
 =?us-ascii?Q?TQmh/L0qrcbdmzLt5Vda1eZZZ0khpCmCgZNK+g7+AFTFpWANlSKfuODK02Zr?=
 =?us-ascii?Q?T0TlAYNQQmhR0fXSda4571juGBF7RjTVaXrFgserqXX/fWhRSSdQlAdkEL7/?=
 =?us-ascii?Q?pg=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 7a5268bf-14a3-40b4-ca9b-08da65afad93
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1101MB2126.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jul 2022 15:44:03.4360
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 5gLpPnIbMgOILlGk5jibJXLBrjZgb/QOUo5aTHiLh5LUc9w3Tqp2UTiV8K2jo8ezIDw1PqQSz7Oj/n8c6F0L/FpUHsQqbWawJZoxp9aHmWk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2937
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Ira Weiny wrote:
> On Fri, Jul 08, 2022 at 03:53:50PM +0100, Matthew Wilcox wrote:
> > On Tue, Jul 05, 2022 at 04:21:57PM -0700, ira.weiny@intel.com wrote:
> > > The main issue I see with this is defining devm_xa_init() in device.h.
> > > This makes sense because a device is required to use the call.  However,
> > > I'm worried about if users will find the call there vs including it in
> > > xarray.h?
> > 
> > Honestly, I don't want users to find it.  This only makes sense if you're
> > already bought in to the devm cult.  I worry people will think that
> > they don't need to do anything else; that everything will be magically
> > freed for them, and we'll leak the objects pointed to from the xarray.
> > I don't even like having xa_destroy() in the API, because of exactly this.
> > 
> 
> Fair enough.  Are you ok with the concept though?

I came here to same the same thing as Matthew. devm_xa_init() does not
lessen review burden like other devm helpers. A reviewer still needs to
go verfy that the patch that uses this makes sure to free all objects in
the xarray before it gets destroyed.

If there still needs to be an open-coded "empty the xarray" step, then
that can just do the xa_destroy() there. So for me, no, the concept of
this just not quite jive.