linux-kernel - Re: [PATCH] fs/writeback: convert wbc_account_cgroup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20241025174959.GH31418@suse.cz>
Date: Fri, 25 Oct 2024 19:49:59 +0200
From: David Sterba <dsterba@...e.cz>
To: "Pankaj Raghav (Samsung)" <kernel@...kajraghav.com>
Cc: Theodore Ts'o <tytso@....edu>, Chao Yu <chao@...nel.org>,
	Jonathan Corbet <corbet@....net>, willy@...radead.org,
	Josef Bacik <josef@...icpanda.com>,
	Johannes Weiner <hannes@...xchg.org>,
	Michal Koutný <mkoutny@...e.com>,
	Alexander Viro <viro@...iv.linux.org.uk>,
	"Darrick J . Wong" <djwong@...nel.org>,
	Zefan Li <lizefan.x@...edance.com>, Tejun Heo <tj@...nel.org>,
	akpm@...ux-foundation.org, Christian Brauner <brauner@...nel.org>,
	Andreas Dilger <adilger.kernel@...ger.ca>,
	Jaegeuk Kim <jaegeuk@...nel.org>, Jan Kara <jack@...e.cz>,
	Chris Mason <clm@...com>, David Sterba <dsterba@...e.com>,
	cgroups@...r.kernel.org, linux-btrfs@...r.kernel.org,
	linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-f2fs-devel@...ts.sourceforge.net,
	linux-fsdevel@...r.kernel.org, mcgrof@...nel.org,
	gost.dev@...sung.com, linux-doc@...r.kernel.org,
	linux-xfs@...r.kernel.org, Pankaj Raghav <p.raghav@...sung.com>
Subject: Re: [PATCH] fs/writeback: convert wbc_account_cgroup_owner to take a
 folio

On Thu, Sep 26, 2024 at 04:01:21PM +0200, Pankaj Raghav (Samsung) wrote:
> From: Pankaj Raghav <p.raghav@...sung.com>
> 
> Most of the callers of wbc_account_cgroup_owner() are converting a folio
> to page before calling the function. wbc_account_cgroup_owner() is
> converting the page back to a folio to call mem_cgroup_css_from_folio().
> 
> Convert wbc_account_cgroup_owner() to take a folio instead of a page,
> and convert all callers to pass a folio directly except f2fs.
> 
> Convert the page to folio for all the callers from f2fs as they were the
> only callers calling wbc_account_cgroup_owner() with a page. As f2fs is
> already in the process of converting to folios, these call sites might
> also soon be calling wbc_account_cgroup_owner() with a folio directly in
> the future.
> 
> No functional changes. Only compile tested.
> 
> Signed-off-by: Pankaj Raghav <p.raghav@...sung.com>
> ---
>  fs/btrfs/extent_io.c                    | 7 +++----
>  fs/btrfs/inode.c                        | 2 +-

If you ever want to continue with the conversion then for the btrfs part

Acked-by: David Sterba <dsterba@...e.com>