[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