[CRIU] [PATCH 8/9] mounts: handle shared and slave mounts

Cyrill Gorcunov gorcunov at gmail.com
Wed Jul 24 05:18:55 EDT 2013


On Wed, Jul 24, 2013 at 12:10:22PM +0400, Andrey Vagin wrote:
> +	if (mi->parent->flags & MS_SHARED) {
> +		list_for_each_entry(t, &mi->parent->mnt_share, mnt_share)
> +			if (!t->mounted) {
> +				pr_debug("Postpone %s due to %s\n", mi->mountpoint, t->mountpoint);
> +				return 1;
> +			}
> +		list_for_each_entry(t, &mi->parent->mnt_slave_list, mnt_slave)
> +			if (!t->mounted) {
> +				pr_debug("Postpone %s\n", mi->mountpoint);
> +				return 1;
> +			}
> +	}

Can we _please_ start using braces on top level here, ie list_for_each_entry() { ... }
it's not one-liner below. It could be fixed on top of course.


More information about the CRIU mailing list