On Wed, Jul 24, 2019 at 09:32:37AM -0400, Qian Cai wrote:
>       for (f = 0; f < skb_shinfo(skb)->nr_frags; f++)
> -             count += TXD_USE_COUNT(skb_shinfo(skb)->frags[f].size);
> +             count += TXD_USE_COUNT(skb_shinfo(skb)->frags[f].bv_len);
>  #else

No, this is the wrong fix.  Use the fine accessor instead:

+               count += 
TXD_USE_COUNT(skb_frag_size(&skb_shinfo(skb)->frags[f]));

although now there's a line length problem.  Most drivers do:

                skb_frag_t frag = &skb_shinfo(skb)->frags[f];
                count += TXD_USE_COUNT(skb_frag_size(frag));

Reply via email to