netdev
[Top] [All Lists]

[PATCH: Final ] WAS (Re: [PATCH/RFC] (long) network interface changes

To: davem@xxxxxxxxxx
Subject: [PATCH: Final ] WAS (Re: [PATCH/RFC] (long) network interface changes
From: jamal <hadi@xxxxxxxxxx>
Date: Mon, 25 Sep 2000 22:20:36 -0400 (EDT)
Cc: kuznet@xxxxxxxxxxxxx, Robert.Olsson@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, Henner Eisen <eis@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, becker@xxxxxxxxx
In-reply-to: <Pine.GSO.4.20.0009252137160.18843-100000@xxxxxxxxxxxxxxxx>
Sender: owner-netdev@xxxxxxxxxxx
Dave,

Final patch with feedback from Henner to change the naming convention of
the return codes. Clean it up, polish it, junk it etc.

I'd like also to send you a large patch or a series of patches to use the
NET_RX_* codes by the protocols. eg patch:

--------------------------------------------------------
--- ip_input.c  2000/09/23 12:48:56     1.1
+++ ip_input.c  2000/09/23 12:52:52
@@ -341,7 +341,7 @@
 
                skb = skb_cow(skb, skb_headroom(skb));
                if (skb == NULL)
-                       return 0;
+                       return NET_RX_DROP;
                iph = skb->nh.iph;
 
                skb->ip_summed = 0;
@@ -372,7 +372,7 @@
        IP_INC_STATS_BH(IpInHdrErrors);
 drop:
         kfree_skb(skb);
-        return(0);
+        return NET_RX_DROP;
 }
 
 /*
@@ -429,6 +429,6 @@
 drop:
         kfree_skb(skb);
 out:
-        return(0);
+        return NET_RX_DROP;
 }
 
------------------------------------------------------

I realize nobody is using those codes but they would be useful and will
enforce consistency.

cheers,
jamal

Attachment: FF-patch-test8
Description: Text document

<Prev in Thread] Current Thread [Next in Thread>