Kostya,

The patch is OK per-se, please commit it with:

Acked-by: Maxim Patlasov <mpatla...@virtuozzo.com>

Thanks,
Maxim

On 06/21/2016 06:55 AM, Dmitry Monakhov wrote:
(rw & REQ_FUA) branch is impossible because REQ_FUA was cleared line above.
Logic was moved to ploop_req_delay_fua_possible() long time ago.

Signed-off-by: Dmitry Monakhov <dmonak...@openvz.org>
---
  drivers/block/ploop/io_direct.c | 9 ---------
  1 file changed, 9 deletions(-)

diff --git a/drivers/block/ploop/io_direct.c b/drivers/block/ploop/io_direct.c
index 58d7580..a6d83fe 100644
--- a/drivers/block/ploop/io_direct.c
+++ b/drivers/block/ploop/io_direct.c
@@ -108,15 +108,6 @@ dio_submit(struct ploop_io *io, struct ploop_request * 
preq,
        rw &= ~(REQ_FLUSH | REQ_FUA);
- /* In case of eng_state != COMPLETE, we'll do FUA in
-        * ploop_index_update(). Otherwise, we should mark
-        * last bio as FUA here. */
-       if (rw & REQ_FUA) {
-               rw &= ~REQ_FUA;
-               if (preq->eng_state == PLOOP_E_COMPLETE)
-                       postfua = 1;
-       }
-
        bio_list_init(&bl);
if (iblk == PLOOP_ZERO_INDEX)

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to