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 A7CA5CD4857 for ; Wed, 4 Sep 2024 15:38:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 35C4C6B040F; Wed, 4 Sep 2024 11:38:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 30BD16B0410; Wed, 4 Sep 2024 11:38:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 185766B0411; Wed, 4 Sep 2024 11:38:16 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id EB9046B040F for ; Wed, 4 Sep 2024 11:38:15 -0400 (EDT) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 9EB2040206 for ; Wed, 4 Sep 2024 15:38:15 +0000 (UTC) X-FDA: 82527462150.13.D431A14 Received: from nyc.source.kernel.org (nyc.source.kernel.org [147.75.193.91]) by imf17.hostedemail.com (Postfix) with ESMTP id E7FA040016 for ; Wed, 4 Sep 2024 15:38:13 +0000 (UTC) Authentication-Results: imf17.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=EuHeDTM0; spf=pass (imf17.hostedemail.com: domain of brauner@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=brauner@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1725464246; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=PjRir7jJP9mBjTfcTeUhs10N9N+RWJzvEsaKZ7ca3Hs=; b=2g7DR7iVSYDs+CJLeYu4ylP/2FUrLc7aKb4DizOUEp/NvoHhKj9ulOBNGkyef58vyhNNY/ J2Qi8oksohin4U1DJzr/5t79RsBHuETUn9roNEiCamb0Wj+BpeUqwO1xu9q725Ezmv0h+m hWGtOnMiztSA3hNad7MR4vsvDEWIc6Q= ARC-Authentication-Results: i=1; imf17.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=EuHeDTM0; spf=pass (imf17.hostedemail.com: domain of brauner@kernel.org designates 147.75.193.91 as permitted sender) smtp.mailfrom=brauner@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1725464246; a=rsa-sha256; cv=none; b=vD7EfxvNVuxUrP1syC8hXtmXjFzrgYMM3lxLkTOX67ZpD2SF+GNcCBfv31VH7gXATa0Ju5 4famTps7mpXT12x9IJzOPt+nspEe4yI7MVmoC8/UZ/lZC2PABYvIKeWy2KWep0vZQr79Np SGw43lpTZXctePs42Q+C4lM1xyVNo4c= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id AAFC5A44502; Wed, 4 Sep 2024 15:38:05 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BC531C4CEC2; Wed, 4 Sep 2024 15:38:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1725464292; bh=ZfETaz0B7LpTRKh6FFeIC4dGBSh3vZF5GwyRz7DMWRI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EuHeDTM05btjoP8pFurCaeX1oVCeyJ9ptCWfhdduyvLn6BlIRYhgEjHVLsEF2sgiT 7jZrMDJoiSA16ed6Tc+OPnb3G46fLteMBRPkoWQj/XDhjhhiglgwf04pPfvKv742xE M9BkU/OMUICIIYcBZZyMipSyU/2E2xB/xEtYcicD5uT72SqbOKWHdW74l2nZUj9vKx Z4oEsH0Tot/qjPma8+bOMtUaIWehNEawAQVrWQflbbPE7A9DjDgOPgu+MJ4mDsS7mt wikB5MaVc9SQqoSuIfgInozphr83S+5R2FG5T0HF48kpL0Y+Z/acF1fSpp4wumnNQT dGhllyWdiFyTQ== Date: Wed, 4 Sep 2024 17:38:08 +0200 From: Christian Brauner To: Mike Rapoport Cc: Vlastimil Babka , Jens Axboe , Jann Horn , Linus Torvalds , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v2 12/15] slab: create kmem_cache_create() compatibility layer Message-ID: <20240904-seide-witzfigur-c35c62726d93@brauner> References: <20240903-work-kmem_cache_args-v2-0-76f97e9a4560@kernel.org> <20240903-work-kmem_cache_args-v2-12-76f97e9a4560@kernel.org> <20240904-storch-worin-32db25e60f32@brauner> <23eb55c3-0a8c-404b-b787-9f21c2739c4e@suse.cz> <20240904-absuchen-gockel-8246820867b4@brauner> <24717bab-7d5d-4ed1-a17d-65d4676e22a9@suse.cz> <20240904-gegessen-kalziumreich-2817b07433b7@brauner> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Rspamd-Server: rspam03 X-Rspam-User: X-Rspamd-Queue-Id: E7FA040016 X-Stat-Signature: ofudatktcf31g36a8umuufqygyka6m5m X-HE-Tag: 1725464293-846833 X-HE-Meta: U2FsdGVkX18BPgtFFr0wPvEWGDjY8ZVInj6BDPrL0MvoqYST78p9mU7Ckl+SaZ+VXEP7bY8IItUNG6IqCEUiPzmi2VJRzukuf+5uNxzhCcCUE9vkJGCYbq+yP8b+skGMQ3PIXeNFdpAlREtuRC6CT9nqQwjRZRDrf1YiKKu/QtLiydTTaTMmmifM/mjFw/rKjlaQL/+VPzgCOQR8d67yUiGBgZn/r0jhqR/8fXz931Me2LL+NrL+8Vr9EuN+HGiqf4RBrOCkZ+nDeHuDUd+B/LCCL+bp+PgCuT4QL7ovLuezUiGoRLIuEJXyQwH9iRkQTEn+OwNtw8TK9C/5ntjOYZq6OVpwqmw6CcRcZC1KA7BLTIu018ffqmI9YuZi0syRZsawdj7pF5tE+mD+8aZmNfUULsGJL2aYFPb7QU64t6YgX5i6nNX8/SW+sl0CxA42KaKo6zMTuuRZAPI1UIPM8+SM0avIW4xyGArfnFyrlapOeKfA63LwHbCwlLzZQZ9qYk0Ms5ARn+z9Ea8ljbE8cs5fPIFI4CZmRymaFRNiDdU8139YcAjH57/hNHSNch0PSRfsAFtSc48ZqYIcLnOmu9cwNvUaQeARfSPNob+Z+X+E8CMIic8b4QlXgmd1pwziP6IMqwVSVuBBwF/HQEHxNA3pLSnC0UyKMIB/DXuTA/I70Em5X3HhO4TRAcBFkTD9cpExpWLlCK7M8fBQMABxwxTnVnGYsVov2GEwPpoVh5nCwgjPEB8k1IWDpH69IcSKQ6nPQqB23xtJ3BofrNavh1gAXcBpl06WtGiK5zSRLvq2VhnPIE3MPRhd6cqG3SKbQfGklRFsyuZ2HOs4ZKzsXGNiKH3mG/rKWTSWWJ9eGsdUrmJHo8OuUgl460xKHzeKfmw+28ZSlPnm1SpG+4kj5e5DZS3mNRHIrmqCeK72sER9Gj85cAAiPDpF6orgr1yM07bbEnPQ0SmzS+Di9aj lm62FO+r SbCnzbmg3bU95pUd7YiEQv2q3/bLVzMKQmAawdK+IuRC11+evD/sGUU3iiQDFUT3z5K0Xk2hQkMiFDZw1jlVrDZiwyFbCGw9kKL0jO+MLe7RMFByJp56HxrpYO0LUgmYY0KNQ0Addd2TQobtvjCwxnh7vcvToh9OcELzwCoxU1yEDpUy91RprWBrwvDtjSJy5OTuvENrFqUz2Sjsbe5WfZsWpXKMczGDSCZ2uYCnSMELfLWOlk5u0WAmygLIpbP8iwoiCavOjs11oQF5zv6T1Vr8q20K6+cGzF/LipoUzJYtKV+o2eM4jDSL5gw85G4q43X0dHD7ytAwy/3GT6puZoi/DnfrCNo975L7bpCdfjmnRKHt6x8TnS6ZIAw== 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: List-Subscribe: List-Unsubscribe: On Wed, Sep 04, 2024 at 06:11:10PM GMT, Mike Rapoport wrote: > On Wed, Sep 04, 2024 at 04:44:03PM +0200, Christian Brauner wrote: > > On Wed, Sep 04, 2024 at 03:33:30PM GMT, Vlastimil Babka wrote: > > > On 9/4/24 13:38, Christian Brauner wrote: > > > > On Wed, Sep 04, 2024 at 12:50:28PM GMT, Vlastimil Babka wrote: > > > >> On 9/4/24 11:45, Christian Brauner wrote: > > > >> > On Wed, Sep 04, 2024 at 08:14:24AM GMT, Mike Rapoport wrote: > > > >> >> On Tue, Sep 03, 2024 at 04:20:53PM +0200, Christian Brauner wrote: > > > >> >> > Use _Generic() to create a compatibility layer that type switches on the > > > >> >> > third argument to either call __kmem_cache_create() or > > > >> >> > __kmem_cache_create_args(). This can be kept in place until all callers > > > >> >> > have been ported to struct kmem_cache_args. > > > >> >> > > > > >> >> > Signed-off-by: Christian Brauner > > > >> >> > > > >> >> Reviewed-by: Mike Rapoport (Microsoft) > > > >> >> > > > >> >> > --- > > > >> >> > include/linux/slab.h | 13 ++++++++++--- > > > >> >> > mm/slab_common.c | 10 +++++----- > > > >> >> > 2 files changed, 15 insertions(+), 8 deletions(-) > > > >> >> > > > > >> >> > diff --git a/include/linux/slab.h b/include/linux/slab.h > > > >> >> > index aced16a08700..4292d67094c3 100644 > > > >> >> > --- a/include/linux/slab.h > > > >> >> > +++ b/include/linux/slab.h > > > >> >> > @@ -261,9 +261,10 @@ struct kmem_cache *__kmem_cache_create_args(const char *name, > > > >> >> > unsigned int object_size, > > > >> >> > struct kmem_cache_args *args, > > > >> >> > slab_flags_t flags); > > > >> >> > -struct kmem_cache *kmem_cache_create(const char *name, unsigned int size, > > > >> >> > - unsigned int align, slab_flags_t flags, > > > >> >> > - void (*ctor)(void *)); > > > >> >> > + > > > >> >> > +struct kmem_cache *__kmem_cache_create(const char *name, unsigned int size, > > > >> >> > + unsigned int align, slab_flags_t flags, > > > >> >> > + void (*ctor)(void *)); > > > >> >> > > > >> >> As I said earlier, this can become _kmem_cache_create and > > > >> >> __kmem_cache_create_args can be __kmem_cache_create from the beginning. > > > >> > > > >> I didn't notice an answer to this suggestion? Even if it's just that you > > > >> don't think it's worth the rewrite, or it's not possible because X Y Z. > > > >> Thanks. > > > > > > > > I'm confused. I sent two patches as a reply to the thread plus the > > > > answer below and there's two patches in v3 that you can use or drop. > > > > > > Right, that's the part below. But the suggestion above, and also in Mike's > > > reply to 02/12 was AFAICS to rename __kmem_cache_create_args to > > > __kmem_cache_create (since patch 02) and here __kmem_cache_create to > > > _kmem_cache_create. It just seemed odd to see no reaction to that (did I > > > miss or not receive it?). > > > > Oh, I see. I read it as a expressing taste and so I didn't bother > > replying. And I really dislike single underscore function names so I > > would like to avoid it and it also seems more confusing to me. > > Heh, not quite. I don't like kmem_cache_create_args essentially becoming a > replacement for kmem_cache_create* and I'd prefer __kmem_cache_create > naming. > > As for the single underscore, I don't have strong feelings about it, but I > do think that it should be renamed to something else than > __kmem_cache_create to leave __kmem_cache_create for the core function. I honestly don't care especially because it's not visible outside of the header. If you care then a simple patch on top of the series to rename to whatever is fine by me. But single vs double underscore with fundamentally different parameters seems ripe for visual confusion and the compatibility layer will go away anyway.