[Devel] [PATCH rh7 v2 2/3] ve/fs/sync: per containter sync and syncfs

[Andrey Ryabinin]

On 02/03/2016 04:49 PM, Andrey Ryabinin wrote:

>>
>> If sync_ub != NULL, you sync inodes and bdevs twice, here and in
>> sync_filesystems_ve. Better move all the iterate_supers/bdevs from here
>> to sync_filesystems under !ub branch?
>>
> 
> I'm thinking about something like this:
> 
> @@ -255,15 +253,22 @@ SYSCALL_DEFINE0(sync)
>  
>                 if (fsb == FSYNC_FILTERED)
>                         sync_ub = get_io_ub();
> +
> +               if (sync_ub && (sync_ub != get_ub0())) {
> +                       wakeup_flusher_threads_ub(0, sync_ub, WB_REASON_SYNC);
> +                       sync_filesystems_ve(get_exec_env(), ub, nowait);
							s/ub/sync_ub	
> +                       sync_filesystems_ve(get_exec_env(), ub, wait);
> +                       goto skip;
> +               }
>         }
> 
> 
> !sync_ub path left untouched.
> _______________________________________________
> Devel mailing list
> Devel at openvz.org
> https://lists.openvz.org/mailman/listinfo/devel
>