diff mbox series

[blktests] block/002: remove debugfs check while blktests is running

Message ID 20220729060411.162529-1-yi.zhang@redhat.com (mailing list archive)
State New, archived
Headers show
Series [blktests] block/002: remove debugfs check while blktests is running | expand

Commit Message

Yi Zhang July 29, 2022, 6:04 a.m. UTC
Seem commit: 99d055b4fd4b ("block: remove per-disk debugfs files in blk_unregister_queue")

Signed-off-by: Yi Zhang <yi.zhang@redhat.com>
---
 tests/block/002 | 3 ---
 1 file changed, 3 deletions(-)

Comments

Christoph Hellwig July 29, 2022, 1:22 p.m. UTC | #1
On Fri, Jul 29, 2022 at 02:04:11PM +0800, Yi Zhang wrote:
> Seem commit: 99d055b4fd4b ("block: remove per-disk debugfs files in blk_unregister_queue")

s/Seem/see/ ?

But yes, this should go in.  I thought I already submitted that, but
it seems like I didn't.
Yi Zhang July 30, 2022, 8:01 a.m. UTC | #2
On Fri, Jul 29, 2022 at 9:22 PM Christoph Hellwig <hch@lst.de> wrote:
>
> On Fri, Jul 29, 2022 at 02:04:11PM +0800, Yi Zhang wrote:
> > Seem commit: 99d055b4fd4b ("block: remove per-disk debugfs files in blk_unregister_queue")
>
> s/Seem/see/ ?
Yes, just send v2 for this patch, thanks for the review.

>
> But yes, this should go in.  I thought I already submitted that, but
> it seems like I didn't.
>
diff mbox series

Patch

diff --git a/tests/block/002 b/tests/block/002
index 9b183e7..15b47a6 100755
--- a/tests/block/002
+++ b/tests/block/002
@@ -25,9 +25,6 @@  test() {
 	blktrace -D "$TMP_DIR" "/dev/${SCSI_DEBUG_DEVICES[0]}" >"$FULL" 2>&1 &
 	sleep 0.5
 	echo 1 > "/sys/block/${SCSI_DEBUG_DEVICES[0]}/device/delete"
-	if [[ ! -d /sys/kernel/debug/block/${SCSI_DEBUG_DEVICES[0]} ]]; then
-		echo "debugfs directory deleted with blktrace active"
-	fi
 	{ kill $!; wait; } >/dev/null 2>/dev/null
 	if [[ -d /sys/kernel/debug/block/${SCSI_DEBUG_DEVICES[0]} ]]; then
 		echo "debugfs directory leaked"