Hi T Ishii, <div><br></div><div>Thanks for patch.</div><div>What is the impact of this on 3.7.11?.Is health is not properly work?</div><div> Could you please share the steps to apply this patch to 3.7.11. </div><div><br></div><div>Thanks, </div><div>Siva.<br><br>On Wednesday, October 16, 2019, Tatsuo Ishii &lt;<a href="mailto:ishii@sraoss.co.jp">ishii@sraoss.co.jp</a>&gt; wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I accidentally found a bug in pool_check_fd in 3.7 and above.<br>
If does following check:<br>
<br>
                        if (processType == PT_MAIN &amp;&amp; processState == PERFORMING_HEALTH_CHECK &amp;&amp; errno == EINTR &amp;&amp; health_check_timer_expired)<br>
<br>
>From 3.7, health check has been given separate processType because it<br>
has become separate process from main. Thus this should be fixed to:<br>
<br>
                        if (processType == PT_HEALTH_CHECK &amp;&amp; errno == EINTR &amp;&amp; health_check_timer_expired)<br>
<br>
In 4.0 or earlier this bug effectively disables health_check_timeout<br>
while reading packets from backend. For 4.1 or later, this does not<br>
affect because pool_read in health check because in the code path<br>
pool_check_fd does not gets called.<br>
<br>
Attached is a patch to fix the bug.<br>
<br>
Best regards,<br>
--<br>
Tatsuo Ishii<br>
SRA OSS, Inc. Japan<br>
English: <a href="http://www.sraoss.co.jp/index_en.php" target="_blank">http://www.sraoss.co.jp/index_<wbr>en.php</a><br>
Japanese:<a href="http://www.sraoss.co.jp" target="_blank">http://www.sraoss.co.<wbr>jp</a><br>
</blockquote></div>