Re: [PATCH] btrfs: move async csum generation out of experimental features

[David Sterba <dsterba@suse.cz>]

On Thu, Sep 03, 2026 at 06:09:41PM +0930, Qu Wenruo wrote:
> @@ -4437,19 +4432,6 @@ void __cold close_ctree(struct btrfs_fs_info *fs_info)
>  	 */
>  	btrfs_flush_workqueue(fs_info->delalloc_workers);
>  
> -	/*
> -	 * We can have ordered extents getting their last reference dropped from
> -	 * the fs_info->workers queue because for async writes for data bios we
> -	 * queue a work for that queue, at btrfs_wq_submit_bio(), that runs
> -	 * run_one_async_done() which calls btrfs_bio_end_io() in case the bio
> -	 * has an error, and that later function can do the final
> -	 * btrfs_put_ordered_extent() on the ordered extent attached to the bio,
> -	 * which adds a delayed iput for the inode. So we must flush the queue
> -	 * so that we don't have delayed iputs after committing the current
> -	 * transaction below and stopping the cleaner and transaction kthreads.
> -	 */
> -	btrfs_flush_workqueue(fs_info->workers);

In the final fs closing sequence there's always some possible
interaction between the workers, but I don't think we need an explicit
flush anymore. The work offloaded to the system workers will be done
before we get here.