diff options
| author |   Jason A. Donenfeld <Jason@zx2c4.com> | 2020-01-13 18:42:29 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-01-14 11:48:41 -0800 |
| commit | 2cec4448db38758832c2edad439f99584bb8fa0d (patch) | |
| tree | 8272aaa7a7d7ca33e1c9aa62b165fbc30f5be2ba | |
| parent | net: xfrm: use skb_list_walk_safe helper for gso segments (diff) | |
| download | linux-dev-2cec4448db38758832c2edad439f99584bb8fa0d.tar.xz linux-dev-2cec4448db38758832c2edad439f99584bb8fa0d.zip | |
net: openvswitch: use skb_list_walk_safe helper for gso segments
This is a straight-forward conversion case for the new function, keeping
the flow of the existing code as intact as possible.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/openvswitch/datapath.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index e3a37d22539c..659c2a790fe7 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c @@ -321,8 +321,7 @@ static int queue_gso_packets(struct datapath *dp, struct sk_buff *skb, } /* Queue all of the segments. */ - skb = segs; - do { + skb_list_walk_safe(segs, skb, nskb) { if (gso_type & SKB_GSO_UDP && skb != segs) key = &later_key; @@ -330,17 +329,15 @@ static int queue_gso_packets(struct datapath *dp, struct sk_buff *skb, if (err) break; - } while ((skb = skb->next)); + } /* Free all of the segments. */ - skb = segs; - do { - nskb = skb->next; + skb_list_walk_safe(segs, skb, nskb) { if (err) kfree_skb(skb); else consume_skb(skb); - } while ((skb = nskb)); + } return err; } |