I don't know the intension of setting VALID status in the skb destructor. But I think it need to be set UNUSED status in case of error then release skb, or rx ring might be filled with RESERVED frames.
Signed-off-by: Ken-ichirou MATSUZAWA <cha...@h4.dion.ne.jp> --- net/netlink/af_netlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c index e6134f4..85ccd8b 100644 --- a/net/netlink/af_netlink.c +++ b/net/netlink/af_netlink.c @@ -864,7 +864,7 @@ static void netlink_skb_destructor(struct sk_buff *skb) } else { if (!(NETLINK_CB(skb).flags & NETLINK_SKB_DELIVERED)) { hdr->nm_len = 0; - netlink_set_status(hdr, NL_MMAP_STATUS_VALID); + netlink_set_status(hdr, NL_MMAP_STATUS_UNUSED); } ring = &nlk_sk(sk)->rx_ring; } -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html