[CRIU] [PATCH 2/2] net: sockets -- Don't spam log with known errors

Cyrill Gorcunov gorcunov at gmail.com
Wed Oct 10 09:54:12 MSK 2018


On Tue, Oct 09, 2018 at 11:26:24PM -0700, Andrey Vagin wrote:
> > diff --git a/criu/include/libnetlink.h b/criu/include/libnetlink.h
> > index bf2eb24740b0..f21a0e750337 100644
> > --- a/criu/include/libnetlink.h
> > +++ b/criu/include/libnetlink.h
> > @@ -6,7 +6,7 @@
> >  struct ns_id;
> >  extern int do_rtnl_req(int nl, void *req, int size,
> >  		int (*receive_callback)(struct nlmsghdr *h, struct ns_id *ns, void *),
> > -		int (*error_callback)(int err, void *), struct ns_id *ns, void *);
> > +		int (*error_callback)(int err, struct ns_id *ns, void *), struct ns_id *ns, void *);
> 
> How we are going to use ns in error_callback()?
> 

Well, it is leftover from previous attempts. I'll cleanup.

> > +static int collect_err(int err, struct ns_id *ns, void *arg)
> > +{
> > +	struct sock_diag_greq *gr = arg;
> > +	char family[32], proto[32];
> > +
> > +	/* Fedora 19 hack */
> 
> Pls write more detailed comment here. Fedora 19 is dead for a long time

Hard to tell, I took this code from existing our code base. Probably
we simply should just rip off this comment completely.


More information about the CRIU mailing list