net: allow multiple dev per napi with GRO
authorStephen Hemminger <shemminger@vyatta.com>
Wed, 1 Apr 2009 11:20:20 +0000 (11:20 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 2 Apr 2009 08:07:37 +0000 (01:07 -0700)
GRO assumes that there is a one-to-one relationship between NAPI
structure and network device. Some devices like sky2 share multiple
devices on a single interrupt so only have one NAPI handler. Rather than
split GRO from NAPI, just have GRO assume if device changes that
it is a different flow.

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index 52fea5b28ca694ede44158117d631e093430c25c..91d792d17e098761c2e725204eb065621bba5d57 100644 (file)
@@ -2472,8 +2472,9 @@ static int __napi_gro_receive(struct napi_struct *napi, struct sk_buff *skb)
                return GRO_NORMAL;
 
        for (p = napi->gro_list; p; p = p->next) {
-               NAPI_GRO_CB(p)->same_flow = !compare_ether_header(
-                       skb_mac_header(p), skb_gro_mac_header(skb));
+               NAPI_GRO_CB(p)->same_flow = (p->dev == skb->dev)
+                       && !compare_ether_header(skb_mac_header(p),
+                                                skb_gro_mac_header(skb));
                NAPI_GRO_CB(p)->flush = 0;
        }