On Mon, Feb 27, 2017 at 02:43:47PM -0800, Andrei Vagin wrote:
...
> > +
> > +           if (filp1 && filp_epoll) {
> > +                   filp_tgt = get_epoll_tfile_raw_ptr(filp_epoll, 
> > slot.tfd, slot.toff);
> > +                   if (IS_ERR(filp_tgt))
> > +                           ret = PTR_ERR(filp_tgt);
> > +                   else
> > +                           ret = kcmp_ptr(filp1, filp_tgt, KCMP_EPOLL_TFD);
> 
> I think KCMP_FILE has to be used here ^^^^, because we compare files. We
> have to use the same function for all files to be able to sort them, don't we?

Yes. I already have v2 in my queue. Strictly speaking we can use separate
salt here but it will be waste of memory from userspace POV. I didn't send
v2 yet since I'm modifying criu still to test overall picture. Hopefully
will send today.

Reply via email to