Re: [f2fs-dev] [PATCH v1 04/12] f2fs: cache: introduce writeback thread

Chao Yu via Linux-f2fs-devel <[email protected]>
Newsgroups net.sourceforge.lists.linux-f2fs-devel,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 8/23/26 20:54, Wenjie Qi wrote:
> Hi Chao,
> 
>> cache_thread->cache_wb_task = kthread_run(f2fs_cache_writeback_kthread,
>> 						sbi, "%s", name);
>> if (IS_ERR(cache_thread->cache_wb_task))
>> 	return PTR_ERR(cache_thread->cache_wb_task);
> 
> After `kthread_run()` returns an error pointer, `cache_wb_task` retains that
> value.  The later startup and stop paths distinguish only NULL from non-NULL,
> and the stop path passes a non-NULL value to `kthread_stop()`.
> 
> Is `cache_wb_task` expected to remain an error pointer after
> `f2fs_start_cache_wb_thread()` returns an error?

Will fix.

Thanks,


_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.