Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Sagi Grimberg <sagi@xxxxxxxxxxx>
Subject: Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
From: Daniel Wagner <dwagner@xxxxxxx>
Date: Wed, 21 Jun 2023 13:19:31 +0200
Cc: linux-nvme@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-block@xxxxxxxxxxxxxxx, Chaitanya Kulkarni <kch@xxxxxxxxxx>, "Shin'ichiro Kawasaki" <shinichiro@xxxxxxxxxxxx>, Hannes Reinecke <hare@xxxxxxx>, James Smart <jsmart2021@xxxxxxxxx>, Martin Belanger <Martin.Belanger@xxxxxxxx>
In-reply-to: <083e29ff-4ad0-d05f-0521-56ace13629e6@grimberg.me>
References: <20230620132703.20648-1-dwagner@suse.de> <20230620132703.20648-2-dwagner@suse.de> <380cde65-8794-cfbc-237f-30a8d7e9330c@grimberg.me> <f6rvvhgim4nk34onfgbwyfctgvgdbdigwbsugjvizgtli46p57@s7c4gkqrpjqy> <083e29ff-4ad0-d05f-0521-56ace13629e6@grimberg.me>

On Wed, Jun 21, 2023 at 12:50:29PM +0300, Sagi Grimberg wrote:
> > > Why not simply wait for live? The connecting is obviously racy...
> > 
> > That is what the new version is doing. It's waiting for the live state and then
> > checks the queue count.
> 
> Maybe don't fold waiting for live into waiting for queue_count.

Alright, will do!

References:
- [PATCH blktests v1 0/3] More fixes for FC enabling
  - From: Daniel Wagner
- [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
  - From: Daniel Wagner
- Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
  - From: Sagi Grimberg
- Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
  - From: Daniel Wagner
- Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
  - From: Sagi Grimberg

Prev by Date: Re: [PATCH V2 0/4] nvme: fix two kinds of IO hang from removing NSs
Next by Date: Re: [PATCH] virtio_pmem: do flush synchronously
Previous by thread: Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
Next by thread: Re: [PATCH blktests v1 1/3] nvme/048: Check for queue count check directly
Index(es):
- Date
- Thread

[Index of Archives] [Linux RAID] [Linux SCSI] [Linux ATA RAID] [IDE] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Device Mapper]