[CRIU] Re: [PATCH 6/8] crtools: Drop pr_debug from fdset ops

Pavel Emelyanov xemul at parallels.com
Mon Feb 20 05:04:23 EST 2012


On 02/20/2012 01:56 PM, Cyrill Gorcunov wrote:
> They are redundant, and simply overlog the output
> 
> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>

ack

> ---
>  crtools.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/crtools.c b/crtools.c
> index 743a849..fd7c707 100644
> --- a/crtools.c
> +++ b/crtools.c
> @@ -159,7 +159,6 @@ void __close_cr_fdset(struct cr_fdset *cr_fdset)
>  	for (i = 0; i < CR_FD_MAX; i++) {
>  		if (cr_fdset->fds[i] == -1)
>  			continue;
> -		pr_debug("Closed %d/%d\n", i, cr_fdset->fds[i]);
>  		close_safe(&cr_fdset->fds[i]);
>  		cr_fdset->fds[i] = -1;
>  	}
> @@ -224,7 +223,6 @@ static struct cr_fdset *cr_fdset_open(int pid, unsigned long use_mask,
>  		}
>  		fdset->fds[i] = ret;
>  
> -		pr_debug("Opened %s with %d\n", path, ret);
>  		if (flags == O_RDONLY) {
>  			u32 magic;
>  



More information about the CRIU mailing list