Re: [PATCH v11 8/8] vhost: use vhost_tasks for worker threads

From: Nicolas Dichtel
Date: Tue May 09 2023 - 04:17:32 EST


Le 09/05/2023 à 10:09, Nicolas Dichtel a écrit :
> Le 06/05/2023 à 00:37, Mike Christie a écrit :
> [snip]
>> (Nicolas, I confirmed it fixes the 'ps a' case, but couldn't replicate the 'ps'
>> case. If you could test the ps only case or give me info on what /usr/bin/example
>> was doing I can replicate and test here):
> With you patch:
> $ ps a
> PID TTY STAT TIME COMMAND
> 191 ttyS0 Ss 0:00 /bin/sh -li
> 1255 ttyS0 SLl 0:53 /usr/bin/example
> 1742 ttyS0 R+ 0:00 ps a
> $ ps
> PID TTY TIME CMD
> 191 ttyS0 00:00:00 sh
> 1743 ttyS0 00:00:00 ps
Sorry, this is wrong, here is the right screenshot:
$ ps
PID TTY TIME CMD
538 ttyS0 00:00:00 login
573 ttyS0 00:00:00 bash
8282 ttyS0 00:00:04 example:2
8825 ttyS0 00:00:00 ps
$ ps a
PID TTY STAT TIME COMMAND
538 ttyS0 Ss 0:00 /bin/login -p --
540 tty1 Ss+ 0:00 /sbin/agetty -o -p -- \u --noclear tty1 linux
573 ttyS0 S 0:00 /bin/bash -li
8282 ttyS0 RLl 0:05 /usr/bin/example
8829 ttyS0 R+ 0:00 ps a

It fixes the issue.


Thank you,
Nicolas