[ovs-dev] [PATCH] ofproto-dpif: ovs-vswitchd.log showing unexpected flow

Ben Pfaff blp at nicira.com
Sat Aug 3 05:21:37 UTC 2013


On Fri, Aug 02, 2013 at 08:22:17PM -0700, Andy Zhou wrote:
> This patch prevents the same subfacet from trying to install the kernel
> flow more than once.
> 
> This is how a kernel flow can become unexpected to the user space. 
> When a incoming packet did not match any flow in the kernel, it
> would be sent up to the user space. A subfacet would be created and 
> a corresponding kenrel flow installed. So far so good.
> 
> Just before the kernel flow was installed, another packet of the same
> flow were to arrive at the kernel, it would also be queued up
> for user space.  This time, user space would find the subfacet just
> created (subfacet_create()) due of the first packet, and attempt
> to install the same kernel flow again, but could not (kernel returns EEXIST).
> User space would then mark the subfacet as not installed (SF_NOT_INSTALLED).

Can we just ignore EEXIST in the error handling loop at the bottom of
handle_miss_upcalls()?



More information about the dev mailing list