From: Alexey Kuznetsov <kuz...@ms2.inr.ac.ru> Date: Wed, 23 Jan 2013 20:20:25 +0400
> Hello! > > On Wed, Jan 23, 2013 at 07:01:52PM +0400, Andrey Vagin wrote: >> -#define tcp_time_stamp ((__u32)(jiffies)) >> +#define tcp_time_stamp(tp) ((__u32)(jiffies) + tp->tsoffset) > > This implies that you always have some tp in hands. AFAIK this is not true, > so that I am puzzled how you were able to make something compilable with this > definition. > > At least in tcp_v4_send_ack() you obviously use some invalid tp, there is > no socket in hands there. The thing which is called "sk" is dummy. > > With container migration I used per-container offset. When you have no > container, > you have to do something a little bit smarter. I'm starting to have almost no confidence in this change, and even the basic conceptual idea behind it. Please go back to the drawing board and really analyze this thing completely rather than just bouncing broken patches to the list every day. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/