net: Add and use skb_list_del_init().
authorDavid S. Miller <davem@davemloft.net>
Tue, 31 Jul 2018 22:27:56 +0000 (15:27 -0700)
committerDavid S. Miller <davem@davemloft.net>
Mon, 10 Sep 2018 17:06:54 +0000 (10:06 -0700)
It documents what is happening, and eliminates the spurious list
pointer poisoning.

In the long term, in order to get proper list head debugging, we
might want to use the list poison value as the indicator that
an SKB is a singleton and not on a list.

Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/skbuff.h
net/core/dev.c
net/ipv4/ip_input.c

index c4c9e3f5cd9ac280a2c49719da202633d98d568b..e3a53ca4a9b51b84b7d75ce87485d4d9109a4cf2 100644 (file)
@@ -1344,6 +1344,12 @@ static inline void skb_mark_not_on_list(struct sk_buff *skb)
        skb->next = NULL;
 }
 
+static inline void skb_list_del_init(struct sk_buff *skb)
+{
+       __list_del_entry(&skb->list);
+       skb_mark_not_on_list(skb);
+}
+
 /**
  *     skb_queue_empty - check if a queue is empty
  *     @list: queue head
index f76dd7e14dd616de4fe65304b477eae5b94fa89f..0b2d777e5b9e85ec95f5204b0d81fa006f5a3a27 100644 (file)
@@ -5295,8 +5295,7 @@ static void __napi_gro_flush_chain(struct napi_struct *napi, u32 index,
        list_for_each_entry_safe_reverse(skb, p, head, list) {
                if (flush_old && NAPI_GRO_CB(skb)->age == jiffies)
                        return;
-               list_del(&skb->list);
-               skb_mark_not_on_list(skb);
+               skb_list_del_init(skb);
                napi_gro_complete(skb);
                napi->gro_hash[index].count--;
        }
@@ -5481,8 +5480,7 @@ static enum gro_result dev_gro_receive(struct napi_struct *napi, struct sk_buff
        ret = NAPI_GRO_CB(skb)->free ? GRO_MERGED_FREE : GRO_MERGED;
 
        if (pp) {
-               list_del(&pp->list);
-               skb_mark_not_on_list(pp);
+               skb_list_del_init(pp);
                napi_gro_complete(pp);
                napi->gro_hash[hash].count--;
        }
index eba7f3883230a0cab5da9a789c9f20228d95b0c1..35a786c0aaa064888540774863cd08e1f21f12a8 100644 (file)
@@ -531,11 +531,7 @@ static void ip_sublist_rcv_finish(struct list_head *head)
        struct sk_buff *skb, *next;
 
        list_for_each_entry_safe(skb, next, head, list) {
-               list_del(&skb->list);
-               /* Handle ip{6}_forward case, as sch_direct_xmit have
-                * another kind of SKB-list usage (see validate_xmit_skb_list)
-                */
-               skb_mark_not_on_list(skb);
+               skb_list_del_init(skb);
                dst_input(skb);
        }
 }