: Auxiliary function for smallest-fit option lengths
PATCH 1/11 : Auxiliary function for smallest-fit option lengths
[DCCP]: Auxiliary function for smallest-fit option lengths
This introduces a function to determine the smallest-fit data type to carry options, which is used by the feature-negotiation code to insert option values.
While doing this, I found that the same code is required to set the NDP length, whose length is variable between 1..6 bytes (RFC 4340, 7.7).
There seem to be remains of the old code base when apparently NDP count was only 3 bytes long. I have removed the annotations, as well as an unused constant limiting the maximum NDP value to 3 bytes, and added a FIXME to increase the data type for NDP up to u64. I didn't want to add this to this patch also, since various other parts (e.g. CCID3) implicitly rely on u32. For the moment, u32 seems to more than ample anyway, so the FIXME could even remain until there is a `high-speed' CCID.
--- include/linux/dccp.h | 4 +--- net/dccp/feat.h | 8 ++++++++ net/dccp/options.c | 13 ++++++------- 3 files changed, 15 insertions(+), 10 deletions(-)
net/dccp/feat.h +8 -0
@@ -112,4 +112,12 @@ extern int dccp_feat_clone(struct sockextern int dccp_feat_clone_list (struct list_head const *,struct list_head *);extern int dccp_feat_init (struct sock *sk); +/* Find smallest-fit for @value, but not more than 6 bytes (current maximum) */ +static inline u8 dccp_bytes_per_value (const u64 value) +{ +if (value >0xFFFFFFFFull ) +return 6 ; +return value >0xFFFF ?4 : (value >0xFF ?2 :1 ); +} +#endif /* _DCCP_FEAT_H */
net/dccp/options.c +6 -7
@@ -115,7 +115,7 @@ int dccp_parse_options(struct sock *sk, mandatory =1 ;break ;case DCCPO_NDP_COUNT : -if (len >3 ) +if (len >6 )/* FIXME: make dccpor_ndp u64 */ goto out_invalid_option; opt_recv->dccpor_ndp =dccp_decode_value_var (value, len);
@@ -299,11 +299,6 @@ static void dccp_encode_value_var(const *to++ = (value &0xFF ); } -static inline int dccp_ndp_len (const int ndp) -{ -return likely (ndp <=0xFF ) ?1 : ndp <=0xFFFF ?2 :3 ; -} -int dccp_insert_option (struct sock *sk,struct sk_buff *skb,const unsigned char option,const void *value,const unsigned char len)
@@ -337,7 +332,11 @@ static int dccp_insert_option_ndp(structif (ndp >0 ) {unsigned char *ptr; -const int ndp_len =dccp_ndp_len (ndp); +const int ndp_len =dccp_bytes_per_value (ndp); +/* +* FIXME: increase the data type of dccps_ndp_count to u64 and +* track changes in code - NDP is up to 6 bytes (RFC 4340, 7.7) +*/ const int len = ndp_len +2 ;if (DCCP_SKB_CB (skb)->dccpd_opt_len + len >DCCP_MAX_OPT_LEN )
include/linux/dccp.h +1 -3
@@ -374,8 +374,6 @@ static inline unsigned int dccp_hdr_len(/* FIXME: for now we're default to 1 but it should really be 0 */ #define DCCPF_INITIAL_SEND_NDP_COUNT 1 -#define DCCP_NDP_LIMIT 0xFFFFFF -/** * struct dccp_minisock - Minimal DCCP connection representation *
@@ -456,7 +454,7 @@ extern int dccp_parse_options(struct soc struct sk_buff *skb);struct dccp_options_received { -u32 dccpor_ndp;/* only 24 bits */ +u32 dccpor_ndp;/* FIXME: can be up to 6 bytes (7.7) */ u32 dccpor_timestamp;u32 dccpor_timestamp_echo;u32 dccpor_elapsed_time;
Re: [PATCH 1/11]: Auxiliary function for smallest-fit option
Em Mon, Oct 01, 2007 at 03:18:11PM +0100, Gerrit Renker escreveu:
Quoted 25 lines
[DCCP]: Auxiliary function for smallest-fit option lengths
This introduces a function to determine the smallest-fit data type to carry options, which is used by the feature-negotiation code to insert option values.
While doing this, I found that the same code is required to set the NDP length, whose length is variable between 1..6 bytes (RFC 4340, 7.7).
There seem to be remains of the old code base when apparently NDP count was only 3 bytes long. I have removed the annotations, as well as an unused constant limiting the maximum NDP value to 3 bytes, and added a FIXME to increase the data type for NDP up to u64. I didn't want to add this to this patch also, since various other parts (e.g. CCID3) implicitly rely on u32. For the moment, u32 seems to more than ample anyway, so the FIXME could even remain until there is a `high-speed' CCID.
Signed-off-by Gerrit Renker <[email protected]>--- include/linux/dccp.h | 4 +--- net/dccp/feat.h | 8 ++++++++ net/dccp/options.c | 13 ++++++------- 3 files changed, 15 insertions(+), 10 deletions(-)
net/dccp/feat.h extern int dccp_feat_clone(struct sock · line 120
Quoted 9 lines
net/dccp/feat.h +6
-0@@ -112,4 +112,12 @@ extern int dccp_feat_clone(struct sockextern int dccp_feat_clone_list (struct list_head const *,struct list_head *);extern int dccp_feat_init (struct sock *sk); +/* Find smallest-fit for @value, but not more than 6 bytes (current maximum) */ +static inline u8 dccp_bytes_per_value (const u64 value) +{ +if (value >0xFFFFFFFFull ) +return 6 ; +return value >0xFFFF ?4 : (value >0xFF ?2 :1 );
Gerrit, one more coding style request: please always add an space after the '?'.
- Arnaldo
Quoting Arnaldo Carvalho de Melo:
| > +/* Find smallest-fit for @value, but not more than 6 bytes (current maximum) */
| > +static inline u8 dccp_bytes_per_value(const u64 value)
| > +{
| > + if (value > 0xFFFFFFFFull)
| > + return 6;
| > + return value > 0xFFFF? 4 : (value > 0xFF? 2 : 1);
|
| Gerrit, one more coding style request: please always add an space after
| the '?'.Thank you for spotting - I will take this `home' and do the same for all patches in the test tree.
I wish I could save you these requests, but the question of space or not space was not very clear to me - I could find
Will in any case also re-read Documentation/CodingStyle and update the patches if anything is not conform.
Gerrit
Quoted 8 lines · Gerrit Renker
[DCCP]: Auxiliary function for smallest-fit option lengths
This introduces a function to determine the smallest-fit data type to carry options, which is used by the feature-negotiation code to insert option values.
While doing this, I found that the same code is required to set the NDP length, whose length is variable between 1..6 bytes (RFC 4340, 7.7).
Quoted 8 lines
There seem to be remains of the old code base when apparently NDP count was only 3 bytes long. I have removed the annotations, as well as an unused constant limiting the maximum NDP value to 3 bytes, and added a FIXME to increase the data type for NDP up to u64. I didn't want to add this to this patch also, since various other parts (e.g. CCID3) implicitly rely on u32. For the moment, u32 seems to more than ample anyway, so the FIXME could even remain until there is a `high-speed' CCID.
Signed-off-by Gerrit Renker <[email protected]>