[Swan-dev] IKEv2 nhelpers == 0 failures

[Paul Wouters]

On Fri, 6 Jun 2014, Antony Antony wrote:

> Here is a proposed patch, add timeout_event EVENT_SO_DISCARD to
> STATE_PARENT_R1 with timeout value of:
>
> MAXIMUM_RETRANSMISSIONS_INITIAL + EVENT_RETRANSMIT_DELAY_0
> The default is 30 seconds.

I am confused. MAXIMUM_RETRANSMISSIONS_INITIAL is used to set
maximum_retransmissions_initial which is a counter, not a time unit.
Also, MAXIMUM_RETRANSMISSIONS_INITIAL is a compile time default which
is used to set maximum_retransmissions_initial, so I believe you would
then want to use maximum_retransmissions_initial.

But still I am not sure as other implementations might use other values.

Our EVENT_RETRANSMIT_DELAY_0 is 10 seconds, so 30 seconds would cover 2
or 3 resends of the initiator. I think it is fine, especially as we
still need to bring our own resend timers down to sub-second levels.

So pehraps just set it to event_retransmit_delay_0 ?

Paul