On Fri, Sep 23, 2011 at 3:32 PM, Ben Pfaff <[email protected]> wrote:
> There's only a small race window on datapath creation when get_dpfindex()
> can return 0, so it's pretty unlikely.
>
> Found by inspection.
>
> Signed-off-by: Ben Pfaff <[email protected]>

In general, I haven't been putting likely/unlikely in the upcall path
because it makes the code a little harder to read and I'm not sure
that it's really necessary.  Upcalls are borderline fast/slow path but
if we decide to annotate the branch here then we probably should do it
for the other error conditions as well.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to