Re: [Devel] [PATCH 0/3] Fuse KIO: fix fsync/flush loss

2018-12-03 Thread Alexey Kuznetsov
Ack to all пн, 3 дек. 2018 г., 16:03 Pavel Butsykin pbutsy...@virtuozzo.com: #VSTOR-18475 Pavel Butsykin (3): fs/fuse kio: fix loss of FUSE_FSYNC/FUSE_FLUSH reqs fs/fuse kio: fix sreq and msg leak in prepare_map_flush_ireq() fs/fuse kio: retry allocation

[Devel] [PATCH RHEL7 COMMIT] ms/ovl: fix recursive oi->lock in ovl_link()

2018-12-03 Thread Konstantin Khorenko
The commit is pushed to "branch-rh7-3.10.0-862.20.2.vz7.73.x-ovz" and will appear at https://src.openvz.org/scm/ovz/vzkernel.git after rh7-3.10.0-862.20.2.vz7.73.17 --> commit 92f85553310b4c8d70f9388c1c837aabc3f1e9d6 Author: Amir Goldstein Date: Thu Oct 18 09:45:49 2018 +0300 ms/ovl:

Re: [Devel] [PATCH] kio_pcs: Compact the logs and make eyes happy

2018-12-03 Thread Pavel Butsykin
On 30.11.2018 19:44, Pavel Emelianov wrote: > On 11/30/2018 05:57 PM, Pavel Butsykin wrote: >> Hmm, __kfuse_trace() duplicates message to debugfs trace, without '\n' > > Yes, and AFAIS most of the existing messages already come w/o it. More likely logs without '\n' are present mainly for error

Re: [Devel] [PATCH 1/3] fs/fuse kio: fix loss of FUSE_FSYNC/FUSE_FLUSH reqs

2018-12-03 Thread Pavel Butsykin
sorry, Not there XD pls, don't ignore this. On 03.12.2018 16:04, Pavel Butsykin wrote: > pls ignore this. > > On 03.12.2018 16:02, Pavel Butsykin wrote: >> Fast-Path can loss fsync/flush requests, because PCS_REQ_T_SYNC >> request set up >> PCS_MAP_FLUSHING flag, but never cleans it. Although

Re: [Devel] [PATCH 1/3] fs/fuse kio: fix loss of FUSE_FSYNC/FUSE_FLUSH reqs

2018-12-03 Thread Pavel Butsykin
pls ignore this. On 03.12.2018 16:02, Pavel Butsykin wrote: > Fast-Path can loss fsync/flush requests, because PCS_REQ_T_SYNC request set up > PCS_MAP_FLUSHING flag, but never cleans it. Although initially > PCS_MAP_FLUSHING > flag was created for sync timer, so PCS_REQ_T_SYNC request should

[Devel] [PATCH 2/3] fs/fuse kio: fix sreq and msg leak in prepare_map_flush_ireq()

2018-12-03 Thread Pavel Butsykin
Signed-off-by: Pavel Butsykin --- fs/fuse/kio/pcs/pcs_map.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/fuse/kio/pcs/pcs_map.c b/fs/fuse/kio/pcs/pcs_map.c index 4bd18c5224ab..1e700dff2043 100644 --- a/fs/fuse/kio/pcs/pcs_map.c +++ b/fs/fuse/kio/pcs/pcs_map.c @@ -3016,6 +3016,8 @@

[Devel] [PATCH 3/3] fs/fuse kio: retry allocation PCS_IREQ_FLUSH request

2018-12-03 Thread Pavel Butsykin
If cslist has changed during request allocation, that is no reason not to run PCS_IREQ_FLUSH. In the case when cslist has changed let's re-allocate the request with new cslist and try it again to fix possible loss of fsync/flush request. Signed-off-by: Pavel Butsykin ---

[Devel] [PATCH 0/3] Fuse KIO: fix fsync/flush loss

2018-12-03 Thread Pavel Butsykin
#VSTOR-18475 Pavel Butsykin (3): fs/fuse kio: fix loss of FUSE_FSYNC/FUSE_FLUSH reqs fs/fuse kio: fix sreq and msg leak in prepare_map_flush_ireq() fs/fuse kio: retry allocation PCS_IREQ_FLUSH request fs/fuse/kio/pcs/pcs_map.c | 37 + 1 file changed, 25

[Devel] [PATCH 1/3] fs/fuse kio: fix loss of FUSE_FSYNC/FUSE_FLUSH reqs

2018-12-03 Thread Pavel Butsykin
Fast-Path can loss fsync/flush requests, because PCS_REQ_T_SYNC request set up PCS_MAP_FLUSHING flag, but never cleans it. Although initially PCS_MAP_FLUSHING flag was created for sync timer, so PCS_REQ_T_SYNC request should ignore everything related to PCS_REQ_T_SYNC flag. This patch adds