Re: hangs after merge window changes

[Dominique Martinet <asmadeus@codewreck.org>]

Eric Van Hensbergen wrote on Wed, Jul 17, 2024 at 01:01:26PM -0500:
> I held way back on adding anything new over the past cycle, so this
> sounds likely some external change maybe perturbing an older issue.
> What guest side mount options, qemu options and qemu version are you
> using?

Mount option if any would be much appreciated, yes.
Also, what version you upgraded from to limit the limit the search a
bit; the previous 6.9 tree?

Also couldn't find much infos about the CI in general, given the traces
I don't think the workload matters all that much but it can give
something to start with e.g. running from initrd and mounting 9p or
using 9p as rootfs; that shouldn't make a difference but might as well
try to reproduce from something close to the actual netdev CI given the
low reproducibility.

> Dominique, you aware of anything coming in that might have messed with
> this?  Maybe the new common code refactoring underpinning
> readahead/etc.?

Given the trace it's probably stuck in __wait_on_freeing_inode (the only
schedule() call in find_inode_fast), so we could be looking at
fallbacks from the iget changes but that made it in 6.9 already so
that's unlikely...

I have a feeling that adding cache will make the error go away if you
just want to fix tests for now, so would be great if you could try
adding -o cache=loose to the 9p mount options; hopefully the inodes
won't be freed as often and that'll make the race harder to hit.


I had a quick look at other globals fs changes and nothing stood out,
it's going to be fun to track down.

> On Wed, Jul 17, 2024 at 12:24 PM Jakub Kicinski <kuba@kernel.org> wrote:
> > Apologies for possible duplicate, I tried to search lore and haven't
> > found anything, but also I can't find the exact 9p archive there :S

The new list is here: https://lore.kernel.org/v9fs/
You probably didn't miss anything :)

-- 
Dominique