Re: [Devel] [PATCH vz9] ploop: fsync after all pios are sent

2024-10-08 Thread Alexander Atanasov
Hello, On 8.10.24 18:42, Alexey Kuznetsov wrote: [Some people who received this message don't often get email from alexey.n.kuznet...@gmail.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ] Hello! if (is_data) queue_work(ploop->wq, &

Re: [Devel] [PATCH vz9] ploop: fsync after all pios are sent

2024-10-08 Thread Alexey Kuznetsov
Hello! if (is_data) queue_work(ploop->wq, &ploop->worker); - if (is_flush) + else if (is_flush) queue_work(ploop->wq, &ploop->fsync_worker); IMHO this looks scary. bio with FLUSH must execute flush first, and only after completion of flush data o

Re: [Devel] [PATCH vz9] ploop: fsync after all pios are sent

2024-10-08 Thread Andrey Zhadchenko
On 10/4/24 09:58, Alexander Atanasov wrote: Currently there are two workers one to handle pios, one to handle flush (via vfs_fsync). This workers are created unbound which means they are run whenever there is a free CPU. When ploop sends pios (via ploop_dispatch_pios) it checks if there are da

Re: [Devel] [PATCH vz9] ploop: fsync after all pios are sent

2024-10-08 Thread Alexander Atanasov
On 8.10.24 14:35, Andrey Zhadchenko wrote: On 10/4/24 09:58, Alexander Atanasov wrote: Currently there are two workers one to handle pios, one to handle flush (via vfs_fsync). This workers are created unbound which means they are run whenever there is a free CPU. When ploop sends pios (via plo