On Fri, Aug 23, 2024 at 09:27:26AM -0700, Joanne Koong wrote: > Prior to this change, data->ff is checked and if not initialized then > initialized in the fuse_writepages_fill() callback, which gets called > for every dirty page in the address space mapping. > > This logic is better placed in the main fuse_writepages() caller where > data.ff is initialized before walking the dirty pages. > > No functional changes added. > > Signed-off-by: Joanne Koong <joannelkoong@xxxxxxxxx> You remove the out label in the previous patch, and then add it back here, you can probably merge the previous patch and this patch into one patch and it would look fine, and reduce the churn a bit. Thanks, Josef