[ovs-dev] [PATCH] ofproto-dpif: xlate_actions() more judiciously.
Ben Pfaff
blp at nicira.com
Thu Jun 2 23:06:41 UTC 2011
On Thu, Jun 02, 2011 at 04:03:30PM -0700, Jean Tourrilhes wrote:
> On Thu, Jun 02, 2011 at 03:55:05PM -0700, Ben Pfaff wrote:
> > On Thu, Jun 02, 2011 at 03:50:44PM -0700, Ethan Jackson wrote:
> > > > One idea is to make validate_actions() return a flag indicating
> > > > whether there was a resubmit action. ?Only ofproto-dpif.c really uses
> > > > that function anyway.
> > >
> > > I suppose I could do it this way. It's kinda ugly though. Do you
> > > think the patch is worth it? I'd be inclined to just drop it.
> >
> > Jean, how strongly do you feel about this?
>
> On the fence. I'm still discovering all the new stuff in
> 1.1. But, if you have a patch, I'll probably evaluate it at some
> point.
OK. Ethan, let's drop it for now, with an eye to reviving it later if
need be.
One reason I don't care much is because Nicira's controllers use
resubmit *lots*, so it wouldn't save us anything.
More information about the dev
mailing list