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

Ben Pfaff blp at nicira.com
Wed Jun 12 23:48:28 UTC 2013


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.



More information about the dev mailing list