On Sun, Jun 11, 2023 at 10:16 PM Dave Chinner <david@xxxxxxxxxxxxx> wrote:
>
> > + /* vhost workers don't participate in core dups */
> > + if ((current->flags & (PF_IO_WORKER | PF_USER_WORKER)) != PF_USER_WORKER)
> > + goto out;
> > +
>
> That would appear to make things worse. mkfs.xfs hung in Z state on
> exit and never returned to the shell.
Well, duh, that's because I'm a complete nincompoop who just copied
the condition from the other cases, but those other cases were for
testing the "this is *not* a vhost worker".
Here the logic - as per the comment I added - was supposed to be "is
this a vhost worker".
So that "!=" should obviously have been a "==".
Not that I'm at all convinced that that will fix the problem you are
seeing, but at least it shouldn't have made things worse like the
getting the condition completely wrong did.
Linus
