Re: [ovs-dev] [worker v2 14/14] system-stats: Run as part of worker process.

2012-07-18 Thread Ben Pfaff
On Wed, Jul 11, 2012 at 05:34:56PM -0500, Ethan Jackson wrote: > > +/* We can end up in these states is */ > > This sentence doesn't make a lot of sense to me. Yeah, I agree. I don't know what it was meant to say, so I deleted it. > Otherwise looks good, thanks. Thanks. I'll push this ser

Re: [ovs-dev] [worker v2 14/14] system-stats: Run as part of worker process.

2012-07-11 Thread Ethan Jackson
> +/* We can end up in these states is */ This sentence doesn't make a lot of sense to me. Otherwise looks good, thanks. Ethan > +S_REQUEST_SENT, /* Sent a request to worker. */ > +S_REPLY_RECEIVED/* Received a reply from worker. */ > +} state; > + > +/* In

[ovs-dev] [worker v2 14/14] system-stats: Run as part of worker process.

2012-06-27 Thread Ben Pfaff
The stats gathering can be time-consuming in some cases, so it's better to do it in the worker process. Signed-off-by: Ben Pfaff --- vswitchd/bridge.c | 59 - vswitchd/system-stats.c | 160 +-- vswitchd/system-stats.h |6 +-