[ovs-dev] [PATCH 2/2] bridge: Don't pay attention to columns that vswitchd doesn't need.
Ben Pfaff
blp at nicira.com
Wed Aug 11 22:30:08 UTC 2010
On Wed, Aug 11, 2010 at 12:04:19AM -0700, Justin Pettit wrote:
> On Jul 19, 2010, at 4:30 PM, Ben Pfaff wrote:
>
> > Not replicating unneeded columns has some value in avoiding CPU time and
> > bandwidth to the database. In ovs-vswitchd, setting cur_cfg! as write-only
>
> Is the bang significant? Is it factorial? Is it just a misplaced
> sense of excitement like "Yahoo!"?
Typo. Thanks, removed.
> > + /* Didn't really changed but the OVSDB monitor protocol always
> > + * includes every value in a row. */
>
> changed -> change
Thanks, fixed.
> > + ovsdb_idl_omit(idl, &ovsrec_port_col_fake_bridge);
>
> I guess it shouldn't be changing once the bridge has been created, but
> do we not care if fake_bridge has changed?
The vswitch doesn't care about fake_bridge at all, and it never reads or
writes that column. Its only purpose is to tell ovs-vsctl that it
should pretend that that port is a bridge.
> This series will be a great improvement for systems with a lot of
> configuration state. Thanks!
Thank you for the review.
More information about the dev
mailing list