[Devel] [PATCH RH8] ve/msync: fix wrong behaviour of fs.fsync-enable

[Alexander Mikhalitsyn]

On Tue, 13 Jul 2021 10:55:47 +0300
Pavel Tikhomirov <ptikhomirov at virtuozzo.com> wrote:

> When FSYNC_NEVER is set in container (in fs.fsync-enable sysctl) syncs
> should be ignored instead of failing with ENOMEM as we have now.
> 
> https://jira.sw.ru/browse/PSBM-131652
> Fixes: a3d65258a0a1 ("ve/fs/sync: Per containter sync and syncfs and
> fs.fsync-enable sysctl")
> Signed-off-by: Pavel Tikhomirov <ptikhomirov at virtuozzo.com>
> ---
>  mm/msync.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mm/msync.c b/mm/msync.c
> index 3a79d8f2be4d..c329fee0fd6c 100644
> --- a/mm/msync.c
> +++ b/mm/msync.c
> @@ -48,9 +48,9 @@ SYSCALL_DEFINE3(msync, unsigned long, start, size_t, len, int, flags)
>  	end = start + len;
>  	if (end < start)
>  		goto out;
> +	error = 0;
>  	if (ve_fsync_behavior() == FSYNC_NEVER)
>  		goto out;
> -	error = 0;
>  	if (end == start)
>  		goto out;
>  	/*
> -- 
> 2.31.1
> 

Thanks, my bad.

Acked-by: Alexander Mikhalitsyn <alexander.mikhalitsyn at virtuozzo.com>