On Tue, Jul 30, 2013 at 12:24:43AM -0700, Andy Zhou wrote: > Remove the DPIF_FP_MODIFY flag when creating a new flow. When flows arrive in > a batch, userspace may push down multiple unique flow definitions that > overlap when wildcards are applied. Kernels support flow wildcarding > will reject these flow as duplicates (EEXIST), which will be logged > at a lower logging level. > > Signed-off-by: Andy Zhou <[email protected]>
This seems reasonable. There are two other places that specify DPIF_FP_* flags. Patch 2/2 addresses one of them. But I wonder whether subfacet_install(), the other place, should also change. It always uses DPIF_FP_CREATE | DPIF_FP_MODIFY. Should it use only DPIF_FP_CREATE if subfacet->path == SF_NOT_INSTALLED and only DPIF_FP_MODIFY otherwise? Acked-by: Ben Pfaff <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
