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 11565C48BEB for ; Wed, 14 Feb 2024 22:22:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 532D98D0017; Wed, 14 Feb 2024 17:22:41 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4E3898D000E; Wed, 14 Feb 2024 17:22:41 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3837A8D0017; Wed, 14 Feb 2024 17:22:41 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 239588D000E for ; Wed, 14 Feb 2024 17:22:41 -0500 (EST) Received: from smtpin25.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id E9314A03E9 for ; Wed, 14 Feb 2024 22:22:40 +0000 (UTC) X-FDA: 81791834880.25.66E846B Received: from mail-pf1-f178.google.com (mail-pf1-f178.google.com [209.85.210.178]) by imf25.hostedemail.com (Postfix) with ESMTP id 163E2A0009 for ; Wed, 14 Feb 2024 22:22:38 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=fromorbit-com.20230601.gappssmtp.com header.s=20230601 header.b=PPdQZYYN; dmarc=pass (policy=quarantine) header.from=fromorbit.com; spf=pass (imf25.hostedemail.com: domain of david@fromorbit.com designates 209.85.210.178 as permitted sender) smtp.mailfrom=david@fromorbit.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1707949359; 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=ln+OOvGmpFuiRMz6w7NK4oosvSzfGSRKkgDvCaci/QI=; b=fYSqwwLw3Uomd1n+lJxwntRkhXkfshuRJp8gOy62ogoQEBQP+aP6OYkUtGA41661+gEGrQ c3raH84+0kxIkFGu7K0eG0gM/Nyh/Eck7mlXGpldiQbRXELF1ATGiD6ZB4eN2ZZ8bOI8OD qSAPz4TZR44w0hCy9GA2bYKo25J64uw= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=pass header.d=fromorbit-com.20230601.gappssmtp.com header.s=20230601 header.b=PPdQZYYN; dmarc=pass (policy=quarantine) header.from=fromorbit.com; spf=pass (imf25.hostedemail.com: domain of david@fromorbit.com designates 209.85.210.178 as permitted sender) smtp.mailfrom=david@fromorbit.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1707949359; a=rsa-sha256; cv=none; b=oKbz1sYXXIzxmaMf9u4ftZ/2OoYEVWFzKT4D7Q7IOgpQiZ/5W2aAZyH4FAeyto0vPEAfju +w2Sg7iMFuctw093QnQVFCmxiLBonic6ljJ4/aaxN9qfWCXMD0mrv5D+gfKvThjOw1ATuQ WtUQE1UxF5H47SYDNkV/BIvu829wjCg= Received: by mail-pf1-f178.google.com with SMTP id d2e1a72fcca58-6e10ac45684so218882b3a.3 for ; Wed, 14 Feb 2024 14:22:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fromorbit-com.20230601.gappssmtp.com; s=20230601; t=1707949358; x=1708554158; darn=kvack.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ln+OOvGmpFuiRMz6w7NK4oosvSzfGSRKkgDvCaci/QI=; b=PPdQZYYNZH5QHmZ1K163+9JbdvBWUmWY731cHMChW79poFC1mI4OF7yzwuVm+2wYml okY8TaemMgwr6DKZ9C0yOEaEWp33FUHM+ltE726EwywX4T8vn38adi5n3qTldzMwvmOm sxcVGNgjz7HKvOU6A8Y9sTP//HYQMajKM9y2GSrCo8HSmekVxh/N12u32p8VxTt5I7s9 x1eLd15JnCpoCEf3d3MIhus+7wJxlv73t7qxONCoTSdksalsssTNRQpzV0Pu4W7dJ6Ef TPzlLifVI/BZlw4HgNjRF7qht7W3vp7bR2gzLmJ0veItNNaeMtchnCRUCo1M8aj60OVj 0Rbw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1707949358; x=1708554158; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ln+OOvGmpFuiRMz6w7NK4oosvSzfGSRKkgDvCaci/QI=; b=lyvFE3duLumcrfLPQuByswV7Y9Cg15Z3+3n9pJz6s1qnPweCr9/V1I/A/WELsp5xKa qAt1O3oz4qfSTeCQ7Y+Y23Njc4BhVmolKBPaSVTyIUqcsWdeRpfYbiyGgE9XZAKThDn9 GNamnrCZtq5Q5sDqldCtQgY7efoju9lW8ItIwLysuN7RbHQrukY+DEYoeRZ82e2Ai78k heAxNZ1HjmgJ0D/zZcqZNrBY/2cGktLP5yRix0zalG2oxcmH6UmbLv7fQJlTa/xXEZFJ +i0giy0x8BN37YHET4dJYf5RTYPfwwZbBEW4+YnKNBK2RTf1hSMPb3IF32GEX9JY1Dai DW1A== X-Forwarded-Encrypted: i=1; AJvYcCUZcwIKYutTxd6mRNBoCBj3G/6nPaemhyQH+JlPiRm70dPoyY+b0kLkin5dGUBbezeozijsPbZvm+yMcRJRQJOvqC8= X-Gm-Message-State: AOJu0YzSo5Pv/tvgKe4SXN47eiHufuSaEpqSQPOf1MzVnLCL9EEVcJkg ms8pzW0L9Bvv/zo1NEpltwHzcU862XyfKZ++/3RvqR3a7GDvAhqR9Z8757SSAlw= X-Google-Smtp-Source: AGHT+IHTGeIRIsITdPStGM5mFftKvPz626lejWEKKG4hpUncJNP4ImSALCuOSyfVaorY4mjRZpEnMA== X-Received: by 2002:a05:6a20:d703:b0:19e:9b19:96c0 with SMTP id iz3-20020a056a20d70300b0019e9b1996c0mr191489pzb.7.1707949357817; Wed, 14 Feb 2024 14:22:37 -0800 (PST) X-Forwarded-Encrypted: i=1; AJvYcCU7PtM/n32FxjYla27l1NTc4FZPWZTeWl2aTWMjeX5bZ6UFXr/4eDYMTwA+dNcGgpW00xnfUPPiLTf4so7M7gh7Eh2Cnzl3MrbKCbjgmXgriMsUnStZ0w3MHtMyGaF6TMUvNjuL5qs4RsqgqEYsxr/rtvWbsqGm9d4w4f+T4nNuWzlMd5SPIOegSJpg5a02l68a0kBT8OTAg1sf/SB3IL/fhywPuVyeZQ3IuPv98aY4nlj4CFPTzxX5YaELgjNNVF7JzhUtJySgTq1vW6jAthGUidWG9nm2WYjSLtNlSbdp0PC9LCZRUlPUxtsR0CP9wp51IAcW2WpiXZOynSE1/NVAZCcYCU8k4qLJ3paJtJwaUUshyz4k6iMmsniLa3/0yheM7W+Xxg1tpJhJ1wPQdY+fE1LEJf0iCE+Qn5Bx6aY4kF9SGme6eqblzJ0T4I1Q8SGnxNGN3EZ75zjkmROZbDXrt1gVY/YNJy9LhiezSIHej4KoQ9JuFnNXHyIUm+KYEONk6F10gTwXNBAj4WQ4SyYq3T2hus1pfj59Qb92An1kQrtEFCSqPy6COWhJz6MDCLVYjfK4Pu+yrj5elsNGQ5ZPCfKgq55yWqSUAq0aQteHdQ5ZtdJ2rmoi43DEW373YPjHz+pi0YAXhgCfFyiFMmp2cm4sLgvc8Xwsek3ZAD7R1I3d0kOZDl2R1H3pKcWJOOcBZoTLd/nZTl2lTWkofs5ITwOKqK0m56d/I0X9x/VsxDKDNCjRBH7PbqcfyecG61M+vNdzXOz6i90oKUtr9yMdM1yjVQ35OFxHiC76Uqc0oZL3KkZ4QHbEKf/9FVCD62SvL7YIWeEbMmx7NtzJL1EVzqJFOjSP8qPjySwzUbx7WTho2nStqDPnBZIDXsYwqE46h6VqOiueAIla+KDcLuV14hoHrizDieZhTtatDIhVl51SDcl5OIH4g2kMj3 89449MfW 4orQpgBzZgRyIRFWt3NbBLkqH3VBLOYY2f14TB7lDHABw4uHoI+3V5GCNO5DFM/TI0LlvAQzaQ6+7dscmk+xlocuhOuCdJ77E2Yn6JgvP+vBhFAiocrZW8IkCJhIR2C/WR/DlR+js0rV2IAAjUu+sI8uhTp8ew99X7r3wXQ8vdEYJmobUiJZEA8WiQ8utTmiW4o83PJfr9LCn8YMpla7CyCqSTuB3GRlyI5W+GnMCwk21D2RXFIg6vdyrDHtRnk1+gaDUu+mJ0M6kgYF0dNENb0JZrlZaBcw4flrbDfARm9cFRuTvB7Mrxf88PnPPd/086BqjxN1whyMVb/pA2yAftFNMf8AqEZDPi2BqzME8vc+9zMKnY/7FJzmSS/HLQP+eAvYUx9XiRHVTwEuv+JWhejm3I+/NU+Dm2hxFGvhMIqdVIb9awwgWoFiUyk8dMPnLN7ws6TZq0LmnMDu66Lfzk1t8aUYgflyHh9LHbINfxqbVfadUPRa9vFjK5JPWvS0j1mzWVdtTjLEHw987CqDSdKmpa+pC140UfTEVPVSo9kv5BzFcTw5oZBj4yhVmTfAnQEGMVKSP4sHF2VD7RIAT7M5LwiFFIPl2ayZC5BrLsXebz+XZYlNQjeUDwRwGrX/kXCVu5PdHK1RxHAbYlueA/WLdkC7EfBWl1jb/zuE6UdzvvDMLfbewNTgObMXmGiz3X4uE/LpJ5a4WZ5D6GlLspqZGWg+r7JirsgF9XSsAPv8UxXpR0mUH/hwy+sDIY8SuXqGK55iuK6frHqJlK9fH/eSOFO8cAI9AIP0ezd2eFqHqaAVsBGGDOS9XyYjXdVYFptESY5drimhMhRSdh5qAbMOh6B791Qz5lv9j2bglzVrIrAF8gL2DP5Gn12wnVe7aikeZ2A6IfOi6IrFofTXPsiRDJZDF3UGwN/IctxgvMfw4fXtRdvCGzeT7jgIda5jk8711ALIxojzKZIAjCR+Ezz7PIhFu rpSx8B2N Eh3RGFcjmar9lB/m77YBriDv0a/z5MK4hJega5ePb6bGqYBd9A0AiqBrTIMpK/vW4vkvkwopuelxZdMwBZaXEiRPQmksqDUVZ68ehSC4oRDZp20gJ3DKFtsh5VxlFt2+C2WzXXJ/CvY0hDylYj1ZbdJlRFo+Cbf8HEenNohsQTfo= Received: from dread.disaster.area (pa49-181-38-249.pa.nsw.optusnet.com.au. [49.181.38.249]) by smtp.gmail.com with ESMTPSA id t29-20020a62d15d000000b006e0a55790easm9168222pfl.216.2024.02.14.14.22.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 14 Feb 2024 14:22:36 -0800 (PST) Received: from dave by dread.disaster.area with local (Exim 4.96) (envelope-from ) id 1raNeD-006ZVB-2T; Thu, 15 Feb 2024 09:22:33 +1100 Date: Thu, 15 Feb 2024 09:22:33 +1100 From: Dave Chinner To: Suren Baghdasaryan Cc: akpm@linux-foundation.org, kent.overstreet@linux.dev, mhocko@suse.com, vbabka@suse.cz, hannes@cmpxchg.org, roman.gushchin@linux.dev, mgorman@suse.de, dave@stgolabs.net, willy@infradead.org, liam.howlett@oracle.com, corbet@lwn.net, void@manifault.com, peterz@infradead.org, juri.lelli@redhat.com, catalin.marinas@arm.com, will@kernel.org, arnd@arndb.de, tglx@linutronix.de, mingo@redhat.com, dave.hansen@linux.intel.com, x86@kernel.org, peterx@redhat.com, david@redhat.com, axboe@kernel.dk, mcgrof@kernel.org, masahiroy@kernel.org, nathan@kernel.org, dennis@kernel.org, tj@kernel.org, muchun.song@linux.dev, rppt@kernel.org, paulmck@kernel.org, pasha.tatashin@soleen.com, yosryahmed@google.com, yuzhao@google.com, dhowells@redhat.com, hughd@google.com, andreyknvl@gmail.com, keescook@chromium.org, ndesaulniers@google.com, vvvvvv@google.com, gregkh@linuxfoundation.org, ebiggers@google.com, ytcoode@gmail.com, vincent.guittot@linaro.org, dietmar.eggemann@arm.com, rostedt@goodmis.org, bsegall@google.com, bristot@redhat.com, vschneid@redhat.com, cl@linux.com, penberg@kernel.org, iamjoonsoo.kim@lge.com, 42.hyeyoo@gmail.com, glider@google.com, elver@google.com, dvyukov@google.com, shakeelb@google.com, songmuchun@bytedance.com, jbaron@akamai.com, rientjes@google.com, minchan@google.com, kaleshsingh@google.com, kernel-team@android.com, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, iommu@lists.linux.dev, linux-arch@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-modules@vger.kernel.org, kasan-dev@googlegroups.com, cgroups@vger.kernel.org Subject: Re: [PATCH v3 25/35] xfs: Memory allocation profiling fixups Message-ID: References: <20240212213922.783301-1-surenb@google.com> <20240212213922.783301-26-surenb@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240212213922.783301-26-surenb@google.com> X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 163E2A0009 X-Stat-Signature: ucamy6xj7jcm79qw8f7f7eegexxwm6xa X-Rspam-User: X-HE-Tag: 1707949358-37507 X-HE-Meta: U2FsdGVkX1/QsfelrMqUf76VTlw479b0k2Hk1WqNsskYEJ/otzo7hQQ2mCE8QtNkrfYFy2dK5Be6Nsemgi/fUvt3Q0QxGM2FTblhYLM78st3nBOI78D9dADPCngFT3SZywOpCl2+fBGhcXGMljjTI/zcks4UCvvvqF9uvgZTiKBPzQ1gzjwgjSJg2ZElYFh+yklPhBWCkA+OSr7MLJFXy7ynhQwa/D/wZhedmAy3CIhQjRpVpOM8yx8jM4UgIDw1sC4I5uwsSpYJNVuciJqqofPhL5yDivwSFG9BXKETiZGYjBspRm9Kmggz+QnZmYU8QoM9sKwT/qWPq5yFd0p78UGQsUNaO3WvJoDqxalDGK6kkQrIfvk8Ybr6GEWYMQ4xQ8hsbAYM44xlxzyT4PMh0I/fi7RtuPiHOiXfxj5h8JDZB4EjsvxwSMaEQxZbiooNcVBlfUI0l+DIgLJ2mMFLi6TmQmcsmaaB6HvyOI+iaWASrBl4J/AiQ9R33PFr2ilViznvpeyZ8iScN+j9y6G8kBihG+W/Cm5TD9VC+5J+sgyEoW8uGEpPwm6QbZCOSs5/ZKKt06JLWCwQXycmQTNKMlMApEgoVjpnlNUpcmWBarKBV2sFUu5J7xvIu8B7fT/nNlPBHZoPM4nj66Lxx+4ApLTAw6FbTJHXlYG/0tERw/RBMIvpQEmkxRVRfrp5PTeFmkzHM+tpWe/BTvtwNNQ2xJ7jFAO206kNwnGhJnNCAT92xP55Fpja61VDLbiz2kHoBWp4ZN4nfCzPTXetVOJMb26RDIq4DWjMLILSEsenn/R/2qKQcfkZYJnvASdkXulvnlO+m3tfgJOi646Tm5KhFfxrgsot6kX2UA55EYEX7yT3wwWw6w6hk4rosnf+3IHujRC1yAT2Q4owF+l67+Fsp3Sp/ZTHV6TAF+sg3G2WQN5Vv6dTXEaVoyQrk/f3irGWmq7Q8sMa0kVNl8LgQtN 5NVzdQjI r5f7yqqO1WIVsOTVdPy/GrZjyNJXp+WmnpNM96V8bIju2b1ITZaWrMc1nwPq2EH6Tzo1z7v7hIK077/rdpEZObEW+vsv5BWqKyTQi2sLei5oFpfmEFzhA/TYVDJ4y4UM3GGnXYDekRcVgNcwKJT1NJRaRfFd5xTXr/DBlvXAsirR6xxF8lf4RmfhNXS5EQzipUnQd 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 Mon, Feb 12, 2024 at 01:39:11PM -0800, Suren Baghdasaryan wrote: > From: Kent Overstreet > > This adds an alloc_hooks() wrapper around kmem_alloc(), so that we can > have allocations accounted to the proper callsite. > > Signed-off-by: Kent Overstreet > Signed-off-by: Suren Baghdasaryan > --- > fs/xfs/kmem.c | 4 ++-- > fs/xfs/kmem.h | 10 ++++------ > 2 files changed, 6 insertions(+), 8 deletions(-) > > diff --git a/fs/xfs/kmem.c b/fs/xfs/kmem.c > index c557a030acfe..9aa57a4e2478 100644 > --- a/fs/xfs/kmem.c > +++ b/fs/xfs/kmem.c > @@ -8,7 +8,7 @@ > #include "xfs_trace.h" > > void * > -kmem_alloc(size_t size, xfs_km_flags_t flags) > +kmem_alloc_noprof(size_t size, xfs_km_flags_t flags) > { > int retries = 0; > gfp_t lflags = kmem_flags_convert(flags); > @@ -17,7 +17,7 @@ kmem_alloc(size_t size, xfs_km_flags_t flags) > trace_kmem_alloc(size, flags, _RET_IP_); > > do { > - ptr = kmalloc(size, lflags); > + ptr = kmalloc_noprof(size, lflags); > if (ptr || (flags & KM_MAYFAIL)) > return ptr; > if (!(++retries % 100)) > diff --git a/fs/xfs/kmem.h b/fs/xfs/kmem.h > index b987dc2c6851..c4cf1dc2a7af 100644 > --- a/fs/xfs/kmem.h > +++ b/fs/xfs/kmem.h > @@ -6,6 +6,7 @@ > #ifndef __XFS_SUPPORT_KMEM_H__ > #define __XFS_SUPPORT_KMEM_H__ > > +#include > #include > #include > #include > @@ -56,18 +57,15 @@ kmem_flags_convert(xfs_km_flags_t flags) > return lflags; > } > > -extern void *kmem_alloc(size_t, xfs_km_flags_t); > static inline void kmem_free(const void *ptr) > { > kvfree(ptr); > } > > +extern void *kmem_alloc_noprof(size_t, xfs_km_flags_t); > +#define kmem_alloc(...) alloc_hooks(kmem_alloc_noprof(__VA_ARGS__)) > > -static inline void * > -kmem_zalloc(size_t size, xfs_km_flags_t flags) > -{ > - return kmem_alloc(size, flags | KM_ZERO); > -} > +#define kmem_zalloc(_size, _flags) kmem_alloc((_size), (_flags) | KM_ZERO) > > /* > * Zone interfaces > -- > 2.43.0.687.g38aa6559b0-goog These changes can be dropped - the fs/xfs/kmem.[ch] stuff is now gone in linux-xfs/for-next. -Dave. -- Dave Chinner david@fromorbit.com