[ovs-dev] [PATCH] timeval: Simplify poll interval logging.

Ethan Jackson ethan at nicira.com
Fri Oct 19 05:37:21 UTC 2012


> All of it seems fine.  I'm unsure of the rationale for the new criteria
> for coverage logging.  Off-hand, I would have just logged it whenever we
> print the "unreasonably long poll interval" message.  Any particular
> reason for the 100 ms criteria there?

I thought we might want to see the coverage counters more often as
general information.  The rationale is dubious however, I'll change it
and merge.  Thanks for the review.

Ethan



More information about the dev mailing list