On Mon, Sep 23, 2013 at 01:25:25PM -0700, Ben Pfaff wrote:
> On Mon, Sep 23, 2013 at 10:57:22AM -0700, Jarno Rajahalme wrote:
> > The "key" member in struct flow_miss refers to memory held by the "struct 
> > upcall", hence the upcalls should be freed only after the flow misses are 
> > processed by the main thread. How about this instead:
> > 
> >     Free upcalls after they are used.
> > 
> > Signed-off-by: Jarno Rajahalme <jrajaha...@nicira.com>
> 
> Thanks.
> 
> This code is really confusing.  I am working on a cleanup.

I changed my mind.  Talking to Ethan, I'd be stomping on his toes if I
did that, so I just applied your patch.  Thanks!
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to