Bug#926896: sysvinit-utils: pidof is unreliable

[Matthew Vernon]

Hi,

On 20/10/2021 15:29, Jesse Smith wrote:
> Is there a reason for wanting to revert this behaviour instead of using
> the "-z" flag on the command line? If you use pidof a lot and expect to
> see processes that are in the uninterruptable sleep state then making an
> alias of pidof='pidof -z' seems like a straight forward approach.
> 
> Reverting the change entirely means the default behaviour could hang in
> NFS environments and I'm not sure non-functioning is a better situation
> than skipping sleeping processes.

The RedHat bug that was the similar issue to #719273 (i.e. that resulted 
in the behaviour of pidof being changed) took a slightly different 
approach - https://bugzilla.redhat.com/show_bug.cgi?id=138788 (patch is 
https://bugzilla.redhat.com/attachment.cgi?id=113650&action=diff ); did 
that ever make its way to upstream?

My feeling is that I'd like us to follow upstream's approach here (hi 
Jesse :) ); it's not obvious to me that having "do look at D processes" 
be an optional switch is wrong.

Regards,

Matthew