Re: [Cluster-devel] [PATCH 2/4] gfs2: remove ->writepage

2022-07-11 Thread Christoph Hellwig
On Tue, Jul 12, 2022 at 01:22:48AM +0200, Andreas Grünbacher wrote: > It should be possible to remove the .writepage operation in > gfs2_jdata_aops as well, but I must be overlooking something because > that actually breaks things. We'll need to wire up ->migratepage for it first to not lose any

Re: [Cluster-devel] [PATCH 2/4] gfs2: remove ->writepage

2022-07-11 Thread Andreas Grünbacher
Am Mo., 11. Juli 2022 um 06:16 Uhr schrieb Christoph Hellwig : > ->writepage is only used for single page writeback from memory reclaim, > and not called at all for cgroup writeback. Follow the lead of XFS > and remove ->writepage and rely entirely on ->writepages. > > Signed-off-by: Christoph

Re: [Cluster-devel] [PATCH 1/4] gfs2: stop using generic_writepages in gfs2_ail1_start_one

2022-07-11 Thread Christoph Hellwig
On Mon, Jul 11, 2022 at 12:27:47PM +0200, Andreas Gruenbacher wrote: > Can you add the below follow-up cleanup? > - gfs2_lm(sdp, "gfs2_ail1_start_one (generic_writepages) " > - "returned: %d\n", ret); > + gfs2_lm(sdp, "gfs2_ail1_start_one returned

Re: [Cluster-devel] [PATCH 1/4] gfs2: stop using generic_writepages in gfs2_ail1_start_one

2022-07-11 Thread Andreas Gruenbacher
Hello, On Mon, Jul 11, 2022 at 7:27 AM Christoph Hellwig wrote: > > Use filemap_fdatawrite_wbc instead of generic_writepages in > gfs2_ail1_start_one so that the functin can also cope with address_space > operations that only implement ->writepages and to properly account > for cgroup writeback.