Changeset View
Changeset View
Standalone View
Standalone View
sys/netinet/tcp_input.c
Context not available. | |||||
(V_tcp_delack_enabled || (tp->t_flags & TF_NEEDSYN))) | (V_tcp_delack_enabled || (tp->t_flags & TF_NEEDSYN))) | ||||
void inline | void inline | ||||
rscheff: make this uint8_t flags a uint16_t flags, to cater for the ACE bits? | |||||
Done Inline ActionsSure we can do that. rrs: Sure we can do that. | |||||
Done Inline ActionsRemember if you play with the flags, since they are coming from rack_bbr_common.c you rrs: Remember if you play with the flags, since they are coming from rack_bbr_common.c you
may find… | |||||
cc_ecnpkt_handler(struct tcpcb *tp, struct tcphdr *th, uint8_t iptos) | cc_ecnpkt_handler_flags(struct tcpcb *tp, uint16_t flags, uint8_t iptos) | ||||
{ | { | ||||
INP_WLOCK_ASSERT(tp->t_inpcb); | INP_WLOCK_ASSERT(tp->t_inpcb); | ||||
Context not available. | |||||
break; | break; | ||||
} | } | ||||
if (th->th_flags & TH_CWR) | if (flags & TH_CWR) | ||||
tp->ccv->flags |= CCF_TCPHDR_CWR; | tp->ccv->flags |= CCF_TCPHDR_CWR; | ||||
else | else | ||||
tp->ccv->flags &= ~CCF_TCPHDR_CWR; | tp->ccv->flags &= ~CCF_TCPHDR_CWR; | ||||
Context not available. | |||||
} | } | ||||
} | } | ||||
void inline | |||||
cc_ecnpkt_handler(struct tcpcb *tp, struct tcphdr *th, uint8_t iptos) | |||||
{ | |||||
Not Done Inline ActionsAdd the th_x2 << 8 || th_flags to cover all 12 potential TCP flags? Or shall I with the AccECN patch eventually? rscheff: Add the th_x2 << 8 || th_flags to cover all 12 potential TCP flags? Or shall I with the AccECN… | |||||
Done Inline ActionsLets do that in your patch assuming I get uint16_t in place for iptos rrs: Lets do that in your patch assuming I get uint16_t in place for iptos | |||||
cc_ecnpkt_handler_flags(tp, th->th_flags, iptos); | |||||
} | |||||
/* | /* | ||||
* TCP input handling is split into multiple parts: | * TCP input handling is split into multiple parts: | ||||
* tcp6_input is a thin wrapper around tcp_input for the extended | * tcp6_input is a thin wrapper around tcp_input for the extended | ||||
Context not available. |
make this uint8_t flags a uint16_t flags, to cater for the ACE bits?