netdev
[Top] [All Lists]

Re: [PATCH] Fix checksum bug for multicast/broadcast packets on postrout

To: Mika Penttilä <mika.penttila@xxxxxxxxxxx>
Subject: Re: [PATCH] Fix checksum bug for multicast/broadcast packets on postrouting hook
From: James Morris <jmorris@xxxxxxxxxx>
Date: Mon, 16 Feb 2004 08:45:47 -0500 (EST)
Cc: "David S. Miller" <davem@xxxxxxxxxx>, Harald Welte <laforge@xxxxxxxxxxxxx>, <netdev@xxxxxxxxxxx>, Stephen Smalley <sds@xxxxxxxxxxxxxx>
In-reply-to: <403066A2.30705@kolumbus.fi>
Sender: netdev-bounce@xxxxxxxxxxx
On Mon, 16 Feb 2004, Mika Penttilä wrote:

> James Morris wrote:
> 
> >No, it's not needed by all callers, and is just a does-one-thing helper 
> >function.
> >
> >
> It is needed for cloned skbs, so check for this should go in 
> skb_checksum_help().

No, there is no need to bloat a simple helper function out with this 
(and then need to handle OOM internally etc.), and it is not appropriate 
for all callers.

- James
-- 
James Morris
<jmorris@xxxxxxxxxx>




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