[ovs-dev] [PATCH 1/3] lib: A separate file for rxbuf.

[Ben Pfaff]

On Wed, Jun 12, 2013 at 04:48:28PM -0700, Ben Pfaff wrote:
> On Wed, Jun 12, 2013 at 04:43:18PM -0700, Gurucharan Shetty wrote:
> > On Wed, Jun 12, 2013 at 4:38 PM, Ben Pfaff <blp at nicira.com> wrote:
> > 
> > > On Mon, Jun 10, 2013 at 01:10:28AM -0700, Gurucharan Shetty wrote:
> > > > Till now, rxbuf was only used by the worker. An upcoming
> > > > commit will have another client. So carve out rxbuf from
> > > > the worker files.
> > > >
> > > > Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
> > >
> > > Thanks for doing this.  We need a fix.
> > >
> > > I don't see a problem coding-wise with these patches, but they are
> > > more complex than I had hoped.  Also, I had planned to get rid of
> > > 'worker' and its components, so I don't like to see a new use for
> > > them.
> > >
> > > I think I have a simpler design that will also work.  I'm going to try
> > > it out, and I'll report back on my results.
> > >
> > Okay. I am working on the ability to add and remove ovsdb databases during
> > run time and will need the same syncing ability.
> > So, please keep that in mind when you implement your solution.
> 
> OK, thanks for pointing that out.

I sent out a patch that shows the approach I have in mind.  Does it
make sense?  What do you think?
        http://openvswitch.org/pipermail/dev/2013-June/028428.html