| 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@shell.cyberus.ca> |
| 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
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH/RFC] (long) network interface changes, jamal |
|---|---|
| Next by Date: | Re: [PATCH: Final ] WAS (Re: [PATCH/RFC] (long) network interface changes, jamal |
| Previous by Thread: | Re: [PATCH/RFC] (long) network interface changes, jamal |
| Next by Thread: | Re: [PATCH: Final ] WAS (Re: [PATCH/RFC] (long) network interface changes, jamal |
| Indexes: | [Date] [Thread] [Top] [All Lists] |