<div dir="ltr"><div>Adding to the list of functions that revive ...<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 27 Apr 2020 at 12:06, Andrew Cagney <<a href="mailto:andrew.cagney@gmail.com">andrew.cagney@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I just pushed code to implement liveness probes using the retransmit timer.  When retransmits time-out:</div><div><br></div><div>- if the IKE SA hasn't established, it does a 'retry' using ipsecdoi_replace(st, try)<br></div><div><br></div><div>- else, presumably the IKE SA is established, and it calls liveness_action(); I suspect this doesn't handle multiple children, and know it won't handle an IKE exchange timing out<br></div><div><br></div><div>(there's also add_revival(), but I'm not sure if that applies here?  And there's pending ...)</div><div><br></div><div>So my question is what should happen?</div><div><br></div><div>- are the established and not established paths really that different (for instance an established IKE SA may have an incomplete CHILD SA)</div><div><br></div><div>- do established CHILD SAs linger so that the IPsec connection is 'up' (even though evidence suggests it is dead)</div><div><br></div><div>- and I have to wonder what the difference between replace and pending is</div></div></blockquote><div><br></div><div>- a rekey (the obvious next candidate for doing proper retransmits) calls  v2_event_sa_replace()</div><div><br></div></div></div>