Re: [PATCH nft 1/2] src: add dscp support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday 2015-11-25 21:01, Patrick McHardy wrote:
>On 25.11, Pablo Neira Ayuso wrote:
>>  #define TYPE_MAX		(__TYPE_MAX - 1)
>> diff --git a/include/proto.h b/include/proto.h
>> index a43bf98..41af0c1 100644
>> --- a/include/proto.h
>> +++ b/include/proto.h
>> @@ -176,7 +176,7 @@ enum ip_hdr_fields {
>>  	IPHDR_INVALID,
>>  	IPHDR_VERSION,
>>  	IPHDR_HDRLENGTH,
>> -	IPHDR_TOS,
>> +	IPHDR_DSCP,
>
>Shouldn't we keep both?

>From what I read from RFC 2474, DSCP is the 6-bit portion.
The 8-bit field is known as
  (a) TOS per IPv4 RFC
  (b) TC (Traffic Class) per IPv6 RFC 2460
  (c) DS per RFC 2472

I would therefore recommend against IPHDR_DSCP, and instead go
with either of IPHDR_TOS, IPHDR_DS, IPHDR_DIFFSERV, or even
some mnemonic involving traffic class.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux