<div dir="ltr"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><pre class="gmail-message">So from what I gather, this means that procps's pidof has the problem
described in this bug report?</pre></div></blockquote><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><pre class="gmail-message">From my point of view the only way to solve this without reopening
#719273 is to add a switch which recognizes D processes or not. Or
adds some kind of timeout.</pre></div></blockquote></div></div><div><br></div><div>I don't think so, because:</div><div><br></div><div>1. Nowadays Fedora uses pidof from procps, it does not use neither stat() not realpath(). Its source code has mention:</div><div><br></div><div>"avoiding stat(2) on NFS volumes doesn't make any sense anymore as this reworked solution does not use stat(2) at all"</div><div><a href="https://gitlab.com/procps-ng/procps/-/blob/master/pidof.c#L347">https://gitlab.com/procps-ng/procps/-/blob/master/pidof.c#L347</a></div><div><br></div><div>2. In Debian the intention was to resolve #719273, but it has next statement:<br><br><a href="https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=719273#42">https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=719273#42</a><br>===========<br>Also, at the current time (and IIRC this wasn't the case when we submitted<br>the original patch), start-stop-daemon is a binary not a script, and it<br>doesn't call pidof or killall.  Instead it uses its own code, and that code<br>is subject to the same issue where it hangs on stuck NFS partitions.<br>Therefore, as it stands, applying this patch to 'pidof' will no longer<br>resolve the issue; similar changes would have to be made to 'killall'.<br>===========</div><div><br></div><div>#719273 is Wheezy times (reported 09 Aug 2013).<br></div><div><br></div><div>I don't know how this applies to recent systems. It probably does not apply because of comment above and the fact that systemd now used (at least by default) in place of sysvinit/start-stop-daemon.</div><div><br></div><div>Nonetheless, I wonder what switch to pidof from procps can break<br></div></div></div>