On Fri, 11 Apr 2008, Patrick McHardy wrote:

quoted text
> Jesper Dangaard Brouer wrote:
>> @@ -87,6 +111,16 @@ int tc_calc_rtable(struct tc_ratespec *r, __u32
>>  *rtab, int cell_log, unsigned mt
>>           unsigned sz = ((i+1)<<cell_log);
>>           if (sz < mpu)
>>               sz = mpu;
>>  +        switch (linklayer) {
>>  +        case LINKLAYER_ATM:
>>  +            sz = tc_align_to_atm(sz);
>>  +            break;
>>  +        case LINKLAYER_ETHERNET:
>>  +            // No size adjustments on Ethernet
>>  +            break;
>
> Couldn't this use the ARPHRD values?

It would make it more difficult to expand to new linklayer types.  For 
example I cannot find a define suitable for DOCSIS (cabel modems) (Thats 
next om my list to make DaveM happy ;-))

If we need to use the ARPHRD values (ARPHRD_ATM + ARPHDR_ETHER), I'm 
wondering which "if_arp.h" is the correct to include.

#include <linux/if_arp.h>
  or
#include <net/if_arp.h>

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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html