Patrick McHardy wrote:
David S. Miller wrote:

From: Patrick McHardy <[EMAIL PROTECTED]>
Date: Tue, 19 Jul 2005 08:22:31 +0200

Here's a first shot. The neighbour part is kind of ugly, but should
work. Suggestions for making it nicer are welcome.


As for the neighbour uglies, I think you can safely stick that info
into the SKB control block... At least I think that would work.


I think it should work. Here is the new patch.

Since you are changing the meaning of skb->stamp, could you please
also rename the skb->stamp to skb->stamp_offset or similar?

That way, any out-of-tree code that uses skb->stamp will no longer
compile (it is much better to fail at compile time than run time).

You could also comment the code around skb->stamp to mention the new
methods that should be used to get/set the skb timestamp.

Ben

--
Ben Greear <[EMAIL PROTECTED]>
Candela Technologies Inc  http://www.candelatech.com

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to