| To: | Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] introduce eth_hdr(skb) |
| From: | Joe Perches <joe@xxxxxxxxxxx> |
| Date: | Sat, 2 Oct 2004 11:40:19 -0700 (PDT) |
| Cc: | "David S. Miller" <davem@xxxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <415EE3E1.7020806@conectiva.com.br> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Sat, 2 Oct 2004, Arnaldo Carvalho de Melo wrote:
> Joe Perches wrote:
> >
> > - } else if (skb->mac.ethernet->h_proto ==
> > __constant_htons(ETH_P_ARP)) {
> > + } else if (eth_hdr(skb)->h_proto == htons(ETH_P_ARP)) {
> >
> > I believe the __constant_htons conversions to htons are incorrect.
>
> No they aren't, look at the generated code for both variations when
> receiving a constant :-)
Perhaps, but if you are going to change some, shouldn't you instead
change all the similar net/bridge uses in a different changset?
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] introduce eth_hdr(skb), Arnaldo Carvalho de Melo |
|---|---|
| Next by Date: | Re: [PATCH] introduce eth_hdr(skb), Arnaldo Carvalho de Melo |
| Previous by Thread: | Re: [PATCH] introduce eth_hdr(skb), Arnaldo Carvalho de Melo |
| Next by Thread: | Re: [PATCH] introduce eth_hdr(skb), Arnaldo Carvalho de Melo |
| Indexes: | [Date] [Thread] [Top] [All Lists] |