public inbox for linux-fsdevel@vger.kernel.org
 help / color / mirror / Atom feed
From: Eric Biggers <ebiggers@kernel.org>
To: David Howells <dhowells@redhat.com>
Cc: Alex Markuze <amarkuze@redhat.com>,
	fstests@vger.kernel.org, ceph-devel@vger.kernel.org,
	linux-fsdevel@vger.kernel.org
Subject: Re: Error in generic/397 test script?
Date: Mon, 20 Jan 2025 09:19:18 -0800	[thread overview]
Message-ID: <20250120171918.GB1159@sol.localdomain> (raw)
In-Reply-To: <1201003.1737382806@warthog.procyon.org.uk>

On Mon, Jan 20, 2025 at 02:20:06PM +0000, David Howells wrote:
> generic/429 can also hang:
> 
> 
> 	show_file_contents()
> 	{
> 		echo "--- Contents of files using plaintext names:"
> 		cat $SCRATCH_MNT/edir/@@@ |& _filter_scratch
> 		cat $SCRATCH_MNT/edir/abcd |& _filter_scratch
> 		echo "--- Contents of files using no-key names:"
> 		cat ${nokey_names[@]} |& _filter_scratch | _filter_nokey_filenames edir
> 	}
> 	...
> 	nokey_names=( $(find $SCRATCH_MNT/edir -mindepth 1 | sort) )
> 	printf '%s\n' "${nokey_names[@]}" | \
> 		_filter_scratch | _filter_nokey_filenames edir
> 	show_file_contents
> 
> on the 'cat ...' at the end of show_file_contents().  A check that
> ${nokey_names[0]} is not nothing might be in order.
> 
> However, in this case (in which I'm running these against ceph), I don't think
> that the find should return nothing, so it's not a bug in the test script per
> se.

Similar to what I explained with generic/397, this can only happen if there is a
kernel bug.  Feel free to send a patch that updates the test to not hang in this
scenario, but the kernel bug (which is presumably in unmerged patches and not
yet upstream) will need to be fixed too.

- Eric

  parent reply	other threads:[~2025-01-20 17:19 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-20 12:32 Error in generic/397 test script? David Howells
2025-01-20 14:20 ` David Howells
2025-01-20 15:43   ` David Howells
2025-01-20 17:25     ` Eric Biggers
2025-01-20 17:41       ` David Howells
2025-01-20 17:46         ` Eric Biggers
2025-01-20 17:53           ` Eric Biggers
2025-01-20 17:19   ` Eric Biggers [this message]
2025-01-20 17:17 ` Eric Biggers
2025-01-20 17:26   ` David Howells

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250120171918.GB1159@sol.localdomain \
    --to=ebiggers@kernel.org \
    --cc=amarkuze@redhat.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=dhowells@redhat.com \
    --cc=fstests@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox