[PATCH v7 3/4] Implement -e status=set option

Paul Chaignon paul.chaignon at gmail.com
Sun Jul 7 15:14:35 UTC 2019


On Sun, Jul 07, 2019 at 01:08:06AM +0300, Dmitry V. Levin wrote:
> On Sat, Jul 06, 2019 at 09:19:57AM +0200, Paul Chaignon wrote:
> [...]
> > +.BR select (2)
> > +is called, but a different thread calls
> > +.BR clock_gettime (2)
> > +before
> > +.BR select (2)
> > +finishes:
> > +.CW
> > +[pid 28779] clock_gettime(CLOCK_REALTIME, {1130322148, 939977000}) = 0
> > +[pid 28772] select(4, [3], NULL, NULL, NULL <unfinished ...>
> > +[pid 28772] <... select resumed> )      = 1 (in [3])
> 
> I think in case of staging output this unfinished/resumed diagnostics
> is redundant and could be easily avoided, e.g.

I was worried the users would not notice the change of syscall's order,
but that not a good way to indicate it anyway.  I applied the changes and
fixed the tests accordingly.  Thanks!

Paul


More information about the Strace-devel mailing list