[ovs-dev] [PATCH 2/2] timeval: Optionally enable cached timing at configure time.

Ben Pfaff blp at nicira.com
Wed Oct 17 20:00:33 UTC 2012


[adding back ovs-dev]

On Wed, Oct 17, 2012 at 12:58:14PM -0700, Ethan Jackson wrote:
> > This looks good, but I wonder whether it would be better to make this a
> > command-line option or a runtime ovs-appctl feature, so that one doesn't
> > need to reconfigure and rebuild everything to test changes.
> 
> I actually disagree on this point.  I'm not sure why a user would want
> to enable/disable cached timing at run time.  I need the build time
> option because I develop on x86_64, but other than this development
> use case, I'm not sure changing the setting is particularly useful.

A user would never want to change it at runtime.  Developers might
want it for testing, that's all, without having to do a rebuild.

Oh well, it's a corner case, we can always change it later if it
proves useful.

Thanks,

Ben.



More information about the dev mailing list