On Wed, Jun 20, 2012 at 06:38:05PM -0700, Justin Pettit wrote:
> On Jun 14, 2012, at 2:17 PM, Ben Pfaff wrote:
> 
> > +    if (time_msec() >= p->next_op_report) {
> > +        long long int ago = (time_msec() - p->first_op) / 1000;
> 
> Is it worth caching time_msec() in a "now" variable?

I guess I don't think so in this case.  Let me know if you disagree.

> Otherwise, looks good to me.

Thanks.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to