[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0804110650390.16547@ask.diku.dk>
Date: Fri, 11 Apr 2008 06:53:46 +0200 (CEST)
From: Jesper Dangaard Brouer <hawk@...u.dk>
To: chas3@...rs.sourceforge.net
Cc: Stephen Hemminger <stephen.hemminger@...tta.com>,
netdev <netdev@...r.kernel.org>,
"David S. Miller" <davem@...emloft.net>,
Patrick McHardy <kaber@...sh.net>
Subject: Re: [PATCH 2/7] ATM cell alignment.
On Thu, 10 Apr 2008, Chas Williams (CONTRACTOR) wrote:
> In message <Pine.LNX.4.64.0804092259460.21022@....diku.dk>,Jesper Dangaard Brou
> er writes:
>> +#define ATM_CELL_SIZE 53 /* ATM cell size incl. header */
>> +#define ATM_CELL_PAYLOAD 48 /* ATM payload size */
>
> these constants are already in linux/atm.h
Yes, I remember. My original patch also contained some #ifndef's to
handle that case. I cannot remember, but there was some reason why I
needed those, something about the ATM part of TC not always being
included.
+#ifndef ATM_CELL_SIZE
+#define ATM_CELL_SIZE 53 /* ATM cell size incl. header */
+#endif
+#ifndef ATM_CELL_PAYLOAD
+#define ATM_CELL_PAYLOAD 48 /* ATM payload size */
+#endif
Hilsen
Jesper Brouer
--
-------------------------------------------------------------------
MSc. Master of Computer Science
Dept. of Computer Science, University of Copenhagen
Author of http://www.adsl-optimizer.dk
-------------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists