subject:"Re\: \[ovs\-dev\] \[PATCH\] timeval\: Remove CACHE_TIME scheme."

Re: [ovs-dev] [PATCH] timeval: Remove CACHE_TIME scheme.

2013-09-04 Thread Alex Wang

Thanks Ethan, I'll adjust accordingly. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] timeval: Remove CACHE_TIME scheme.

2013-09-04 Thread Ethan Jackson

LGTM, just some minor comments. I don't think we need the postfork() function anymore. It's just calling time_init() which will get called later anyway. In the comment to time_alarm we say it refreshes the current time as a side effect. I'd just remove that entirely, doesn't make sense anymore.