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 212AAC5ACB3 for ; Tue, 21 Nov 2023 06:51:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B944D6B0425; Tue, 21 Nov 2023 01:51:40 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id B44A56B042F; Tue, 21 Nov 2023 01:51:40 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A0D5B6B043A; Tue, 21 Nov 2023 01:51:40 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 920226B0425 for ; Tue, 21 Nov 2023 01:51:40 -0500 (EST) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 73A531CB4EB for ; Tue, 21 Nov 2023 06:51:40 +0000 (UTC) X-FDA: 81481040760.23.2A8AD4C Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf12.hostedemail.com (Postfix) with ESMTP id 8BC504000F for ; Tue, 21 Nov 2023 06:51:38 +0000 (UTC) Authentication-Results: imf12.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="Kr/S+GB4"; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf12.hostedemail.com: domain of chrisl@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=chrisl@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1700549498; 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=rg0vf7snjrv8JObGfk+WLn0zJ5rxbtrKIUbLse5CtcY=; b=SjtX9NWUW8Z+2fS1eSZHg0Ne7qQ0usXJy8S2BmHLVX/s6xvySxHM7nqicIjT7VnrO53LVg 4qePeP0RhytXte2d98hPbJbKMIz23RtSf0uTAOUpEKb/vXB6DWZ7zImXsVFKbjyko3L5ow 157PAk2r1yuSq1kTGUvRGL5rFrskVU0= ARC-Authentication-Results: i=1; imf12.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="Kr/S+GB4"; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf12.hostedemail.com: domain of chrisl@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=chrisl@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1700549498; a=rsa-sha256; cv=none; b=5nBkeCo5yT5UZjmO+JL++RuL2XnpYSipOcXepdsvxRCONOw31rRkqeN/vBSaiYVG3aMFUY PTIPeJXKLZpvFHCvY0BeAlv7PRg7ek7jH3C+aOQb5ab7qP5bOZe5ASFYhcAY2y2cpIdLN4 nCJQ3ncJiC2cePQQP7iocpxmRBgyQuI= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 72864611CC for ; Tue, 21 Nov 2023 06:51:37 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1BEEBC433CA for ; Tue, 21 Nov 2023 06:51:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700549497; bh=vcMj82j5UeYxk5/OU71PGwNMQQLupr6YDQ+K2CdWuTk=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=Kr/S+GB4e3na4m45XZknkUKpEmXhdtsl/hwxQraKXKJ1EkiFUoLjYsitKPSpoJtVm 3DkJ4KsXFcwXDkN9R+YK/wqIKpVQrArNtltULfKSK/A2gbWZqWV4cyBy3r3+TGj7eq OdLqKXF5dV6NkJHj9tD30LKzCILXoESDubrSbKpW0UVw18N69nK19gw5ryYhyYtV6/ HWrjCJrP2wOE9fSUz5Cncs81AW6rpAd8YyxzS0693fGXmx9R5sroJqRE5sv+gFsDXo Z4Pp29Xb3bSYZAffs0rvcjXUSJb5ems8icbSzrSQ6E8Q0t3gih8N+zbvK8ANacwqAC 9FOQxFpiebsHg== Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-2851b271e51so1509103a91.1 for ; Mon, 20 Nov 2023 22:51:37 -0800 (PST) X-Gm-Message-State: AOJu0YyClXcmMlcwTseIdeKZpLyjg0yXq8kNZH/V3yNcXlDPN/qpdTwT tPp7u+rmFI/wzlPPST78hi0lX/DPlSzPUQyW6EQdjA== X-Google-Smtp-Source: AGHT+IG95Iz4OSMxFcGhtEsGxJnWg/TdQGGbdNq59LFOgLURew3bL+FbZAwYhBWbgq3LJ0qvoaKk71Nu0oZr4Z5RqQE= X-Received: by 2002:a17:90a:854c:b0:280:4af4:1a41 with SMTP id a12-20020a17090a854c00b002804af41a41mr2745780pjw.15.1700549496540; Mon, 20 Nov 2023 22:51:36 -0800 (PST) MIME-Version: 1.0 References: <20231119194740.94101-1-ryncsn@gmail.com> <20231119194740.94101-8-ryncsn@gmail.com> In-Reply-To: <20231119194740.94101-8-ryncsn@gmail.com> From: Chris Li Date: Mon, 20 Nov 2023 22:51:25 -0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 07/24] mm/swap: move swap_count to header to be shared To: Kairui Song Cc: linux-mm@kvack.org, Andrew Morton , "Huang, Ying" , David Hildenbrand , Hugh Dickins , Johannes Weiner , Matthew Wilcox , Michal Hocko , linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspam-User: X-Rspamd-Server: rspam12 X-Rspamd-Queue-Id: 8BC504000F X-Stat-Signature: sjyuxzcpbfc95pwd1t3roo5udqhjwrbk X-HE-Tag: 1700549498-185493 X-HE-Meta: U2FsdGVkX1+vSLH8rGLPsuT01eYX67nKG+OlitjMJAW9biYQKGnfWNiB5R0u6/hCl24ku6Jx+XniROvVkZQd5d6LLy5mLkOFMai1vSsebMb7PtUGFKsx8Rrd19D2zkYYTECrksZ74gaF8Ml932R6yZiN3lI3TNVgxYDoUGqVkjirDM+MxxDEJBkNHHJ1Qlej8CIQ/WVxyRRFpfA3H5lEal+AmLilh9YrJUIIfspESHOwj72IB3QRwebcPvCl44lIOOp/QGwPPHHBu8xyAMB35rfRFUb1/LR+QFuPrZeUhAPXgea9MF/nB6hJEBcG/++1J8+Y8lLFTzR4MDmp3WoFKJmclWQkdes0F6ZWcAUAAf8fuLLDz2/YgWgl1O8dxtYVxA7rGMkk0HIb3/x0KwAG0oHiBJzPZNXRTKQwVW1MnDo/VOzlOFahbglxHtfgU/cwpXjoF8GfHWMwULqhr0n6Noyr8M98icD+mhdvluN4FFix91tnN9Ilvo5FU0ylDJMr5SJtUSlkZwwGewAlfkGFvZg8qjSfAJa4nN1lOsugRXKQdjtGUMw3VxLj4PUqjkB+xpbj+mBIhXccNr0gPA48Tq2s48wMVzBkTFHz3u1DZHgc30WA8N67EpDpLopvcnL+IRfwAMGKc1TtH7OrZduGl9VArc+D9oUw/AcdyoLMWXvfDXBI0YSJs5nbB2yMyhoL2+fJSi/b/WVnRd8XFm6pTBD1W8kWdV2JZR1ikn3AyBqmGyiKQracICqs4phQfCQNumq8+2naYznrLdc9aiQz1T3Cilq6kPZcOcHnQuB5RkTMLg8q+vKZJ4yvfatfwOlyPF+CsL+6h/koLm1UNapKYJLeMnMhvoPc8KpkpIbTK4bTxSF051ZfjzQXM3+9gYFdfisT0N3RpyrHmai6CwpHVj4C0NfNd7SMaCgcDCzrII6Dz3BaYHeP0OWqy0qnf6eODePS05NOmEP0T+VLy6P 2iUyx60D fy9bYDEBR1NFYPlyuvlHjKiHJLnVDfFRVfhy9d+9Mv7qs80KQGIPohkLI6CUZj6XyVi2P3Amhp5RRpMA5CgbqpNNvh+bYVj8KOmX5QVbcJNs57HxfEdPD+H7RMk9X+t8KEvTRSNRDQM3f5YdRkeFL5tYDcU7g6v2SNnXbUFy62j8IQ5f0EAoZNlPLiJJTmPRDlV8bOcSwu9ERGQr0fG1/rSQLECgbXd+OYomWkxy33I0KaMUzxNcGUUtL/gbtkYQbCs+/TMZcx0v9Yg2HKi8ATvdP28oXkcpnlonA+E9HNU8Fm85UbDQVozrxbNH0Q6mz206KKgDpsTmrhbSm0l/qSQolzQ== 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: Hi Kairui, On Sun, Nov 19, 2023 at 11:48=E2=80=AFAM Kairui Song wro= te: > > From: Kairui Song > > No feature change, prepare for later commits. Again, I don't see the value of having this as a stand alone patch. If one of the later patches needs to use this function as external rather than static, move it with the patch that uses it. From the reviewing point of view, it is unnecessary overhead to cross reference different patches in order to figure out why it is moved. Chris > > Signed-off-by: Kairui Song > --- > mm/swap.h | 5 +++++ > mm/swapfile.c | 5 ----- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/mm/swap.h b/mm/swap.h > index f82d43d7b52a..a9a654af791e 100644 > --- a/mm/swap.h > +++ b/mm/swap.h > @@ -61,6 +61,11 @@ static inline unsigned int folio_swap_flags(struct fol= io *folio) > { > return page_swap_info(&folio->page)->flags; > } > + > +static inline unsigned char swap_count(unsigned char ent) > +{ > + return ent & ~SWAP_HAS_CACHE; /* may include COUNT_CONTINUED flag= */ > +} > #else /* CONFIG_SWAP */ > struct swap_iocb; > static inline void swap_readpage(struct page *page, bool do_poll, > diff --git a/mm/swapfile.c b/mm/swapfile.c > index 0142bfc71b81..a8ae472ed2b6 100644 > --- a/mm/swapfile.c > +++ b/mm/swapfile.c > @@ -114,11 +114,6 @@ static struct swap_info_struct *swap_type_to_swap_in= fo(int type) > return READ_ONCE(swap_info[type]); /* rcu_dereference() */ > } > > -static inline unsigned char swap_count(unsigned char ent) > -{ > - return ent & ~SWAP_HAS_CACHE; /* may include COUNT_CONTINUED fl= ag */ > -} > - > /* Reclaim the swap entry anyway if possible */ > #define TTRS_ANYWAY 0x1 > /* > -- > 2.42.0 > >