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 op in this bio is executed.
This used to be an absolute requirement since 2010. Lines
above look like they are desynchronized whch will defintely
lead to corruptions.

F.e. fs commit is submitte
_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to