git://git.onelab.eu / sliver-openvswitch.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 81158e4) | patch
datapath: Avoid duplicate test in tnl_free_linked_skbs().
authorBen Pfaff <blp@nicira.com>
Tue, 1 Mar 2011 23:33:40 +0000 (15:33 -0800)
committerBen Pfaff <blp@nicira.com>
Wed, 2 Mar 2011 22:34:08 +0000 (14:34 -0800)
commitcb034511802ff7516ca2a94da1e161965fb80ad0
tree51d50044d57ef1de5e0e20e1b2caf60de01214c9tree | snapshot
parent81158e4f4a7071c689aac7ac4fb1389c8e8d3875commit | diff
datapath: Avoid duplicate test in tnl_free_linked_skbs().

I don't see the value in testing 'skb' for nonnull twice.  This is only
called on error paths, so it seems really doubtful to me that there is any
value in the "unlikely", either.  (It could be put into the "while"
expression if it is valuable.)

Signed-off-by: Ben Pfaff <blp@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>
datapath/tunnel.c diff | blob | history
sliver-openvswitch