[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

Since commit b86dd81
"block: get rid of blk-mq default scheduler choice Kconfig entries",
when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
This flag is useful for admin queues that aren't used for normal IO.
We already use this flag at nvme-pci.

Israel Rukshin (3):
  nvme-rdma: Add BLK_MQ_F_NO_SCHED flag to admin tag set
  nvme-fc: Add BLK_MQ_F_NO_SCHED flag to admin tag set
  nvme-loop: Add BLK_MQ_F_NO_SCHED flag to admin tag set

 drivers/nvme/host/fc.c     | 1 +
 drivers/nvme/host/rdma.c   | 1 +
 drivers/nvme/target/loop.c | 1 +
 3 files changed, 3 insertions(+)

-- 
1.8.3.1

[PATCH 1/3] nvme-rdma: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

Since commit b86dd81
"block: get rid of blk-mq default scheduler choice Kconfig entries",
when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
This flag is useful for admin queues that aren't used for normal IO.

Signed-off-by: Israel Rukshin <israelr at mellanox.com>
---
 drivers/nvme/host/rdma.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/nvme/host/rdma.c b/drivers/nvme/host/rdma.c
index 92a03ff..ee61b98f 100644
--- a/drivers/nvme/host/rdma.c
+++ b/drivers/nvme/host/rdma.c
@@ -699,6 +699,7 @@ static struct blk_mq_tag_set *nvme_rdma_alloc_tagset(struct nvme_ctrl *nctrl,
 		set->driver_data = ctrl;
 		set->nr_hw_queues = 1;
 		set->timeout = ADMIN_TIMEOUT;
+		set->flags = BLK_MQ_F_NO_SCHED;
 	} else {
 		set = &ctrl->tag_set;
 		memset(set, 0, sizeof(*set));
-- 
1.8.3.1

[PATCH 2/3] nvme-fc: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

Since commit b86dd81
"block: get rid of blk-mq default scheduler choice Kconfig entries",
when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
This flag is useful for admin queues that aren't used for normal IO.

Signed-off-by: Israel Rukshin <israelr at mellanox.com>
---
 drivers/nvme/host/fc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c
index af075e9..b383735 100644
--- a/drivers/nvme/host/fc.c
+++ b/drivers/nvme/host/fc.c
@@ -2796,6 +2796,7 @@ enum {
 	ctrl->admin_tag_set.driver_data = ctrl;
 	ctrl->admin_tag_set.nr_hw_queues = 1;
 	ctrl->admin_tag_set.timeout = ADMIN_TIMEOUT;
+	ctrl->admin_tag_set.flags = BLK_MQ_F_NO_SCHED;
 
 	ret = blk_mq_alloc_tag_set(&ctrl->admin_tag_set);
 	if (ret)
-- 
1.8.3.1

[PATCH 3/3] nvme-loop: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

This flag is useful for admin queues that aren't used for normal IO.

Signed-off-by: Israel Rukshin <israelr at mellanox.com>
---
 drivers/nvme/target/loop.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/nvme/target/loop.c b/drivers/nvme/target/loop.c
index 92628c4..c56354e 100644
--- a/drivers/nvme/target/loop.c
+++ b/drivers/nvme/target/loop.c
@@ -365,6 +365,7 @@ static int nvme_loop_configure_admin_queue(struct nvme_loop_ctrl *ctrl)
 	ctrl->admin_tag_set.driver_data = ctrl;
 	ctrl->admin_tag_set.nr_hw_queues = 1;
 	ctrl->admin_tag_set.timeout = ADMIN_TIMEOUT;
+	ctrl->admin_tag_set.flags = BLK_MQ_F_NO_SCHED;
 
 	ctrl->queues[0].ctrl = ctrl;
 	error = nvmet_sq_init(&ctrl->queues[0].nvme_sq);
-- 
1.8.3.1

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Sagi Grimberg]

> Since commit b86dd81
> "block: get rid of blk-mq default scheduler choice Kconfig entries",
> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
> This flag is useful for admin queues that aren't used for normal IO.
> We already use this flag at nvme-pci.

How is the fact that we set a single hw queue has anything to do with
the admin tagset?

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

On 10/18/2017 3:50 PM, Sagi Grimberg wrote:
>
>> Since commit b86dd81
>> "block: get rid of blk-mq default scheduler choice Kconfig entries",
>> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline 
>> scheduler.
>> This flag is useful for admin queues that aren't used for normal IO.
>> We already use this flag at nvme-pci.
>
> How is the fact that we set a single hw queue has anything to do with
> the admin tagset?

Setting nr_hw_queues attribute of admin tag set to 1 influence the value
of nr_hw_queues attribute of the admin queue.
At /block/elevator.c
elevator_init() selects the scheduler according to the value of 
nr_hw_queues
attribute of the admin queue.

if (q->mq_ops) {
     if (q->nr_hw_queues == 1)
         e = elevator_get("mq-deadline", false);
     if (!e)
         return 0;
...

Israel

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Sagi Grimberg]

>>> Since commit b86dd81
>>> "block: get rid of blk-mq default scheduler choice Kconfig entries",
>>> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline 
>>> scheduler.
>>> This flag is useful for admin queues that aren't used for normal IO.
>>> We already use this flag at nvme-pci.
>>
>> How is the fact that we set a single hw queue has anything to do with
>> the admin tagset?
> 
> Setting nr_hw_queues attribute of admin tag set to 1 influence the value
> of nr_hw_queues attribute of the admin queue.
> At /block/elevator.c
> elevator_init() selects the scheduler according to the value of 
> nr_hw_queues
> attribute of the admin queue.

Oh, I confused your meaning with setting nr_hw_queues to 1 in the IO
tagset makes a difference for the admin tagset.

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Israel Rukshin]

On 10/18/2017 5:31 PM, Sagi Grimberg wrote:
>
>>>> Since commit b86dd81
>>>> "block: get rid of blk-mq default scheduler choice Kconfig entries",
>>>> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline 
>>>> scheduler.
>>>> This flag is useful for admin queues that aren't used for normal IO.
>>>> We already use this flag at nvme-pci.
>>>
>>> How is the fact that we set a single hw queue has anything to do with
>>> the admin tagset?
>>
>> Setting nr_hw_queues attribute of admin tag set to 1 influence the value
>> of nr_hw_queues attribute of the admin queue.
>> At /block/elevator.c
>> elevator_init() selects the scheduler according to the value of 
>> nr_hw_queues
>> attribute of the admin queue.
>
> Oh, I confused your meaning with setting nr_hw_queues to 1 in the IO
> tagset makes a difference for the admin tagset.

No,
I just wanted to say that before commit b86dd8 there was no scheduler 
for the admin q.
After commit b86dd8 I saw that mq-deadline scheduler was selected for 
the admin q.

[PATCH 1/3] nvme-rdma: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Sagi Grimberg]

Reviewed-by: Sagi Grimberg <sagi at grimberg.me>

[PATCH 2/3] nvme-fc: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Sagi Grimberg]

Reviewed-by: Sagi Grimberg <sagi at grimberg.me>

[PATCH 3/3] nvme-loop: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Sagi Grimberg]

Reviewed-by: Sagi Grimberg <sagi at grimberg.me>

[Suspected-Phishing][PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Max Gurtovoy]

On 10/18/2017 3:38 PM, Israel Rukshin wrote:
> Since commit b86dd81
> "block: get rid of blk-mq default scheduler choice Kconfig entries",
> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
> This flag is useful for admin queues that aren't used for normal IO.
> We already use this flag at nvme-pci.
> 
> Israel Rukshin (3):
>    nvme-rdma: Add BLK_MQ_F_NO_SCHED flag to admin tag set
>    nvme-fc: Add BLK_MQ_F_NO_SCHED flag to admin tag set
>    nvme-loop: Add BLK_MQ_F_NO_SCHED flag to admin tag set

thanks Israel.
all 3 patches looks good to me,

Reviewed-by: Max Gurtovoy <maxg at mellanox.com>

[PATCH 2/3] nvme-fc: Add BLK_MQ_F_NO_SCHED flag to admin tag set

[James Smart]

On 10/18/2017 5:38 AM, Israel Rukshin wrote:
> Since commit b86dd81
> "block: get rid of blk-mq default scheduler choice Kconfig entries",
> when setting nr_hw_queues to 1 the admin tag set uses mq-deadline scheduler.
> This flag is useful for admin queues that aren't used for normal IO.
>
> Signed-off-by: Israel Rukshin <israelr at mellanox.com>
> ---
>   drivers/nvme/host/fc.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c
> index af075e9..b383735 100644
> --- a/drivers/nvme/host/fc.c
> +++ b/drivers/nvme/host/fc.c
> @@ -2796,6 +2796,7 @@ enum {
>   	ctrl->admin_tag_set.driver_data = ctrl;
>   	ctrl->admin_tag_set.nr_hw_queues = 1;
>   	ctrl->admin_tag_set.timeout = ADMIN_TIMEOUT;
> +	ctrl->admin_tag_set.flags = BLK_MQ_F_NO_SCHED;
>   
>   	ret = blk_mq_alloc_tag_set(&ctrl->admin_tag_set);
>   	if (ret)

Looks fine

Signed-off-by: James Smart? <james.smart at broadcom.com>

[PATCH 0/3] Add BLK_MQ_F_NO_SCHED flag to admin tag set

[Christoph Hellwig]

Thanks Israel,

applied the whole series to nvme-4.15