linux-rdma.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* iSER multi-queue configuration
@ 2016-04-29 21:11 Ming Lin
  2016-04-30  6:13 ` Ming Lin
  0 siblings, 1 reply; 5+ messages in thread
From: Ming Lin @ 2016-04-29 21:11 UTC (permalink / raw)
  To: linux-rdma; +Cc: Bart Van Assche, target-devel

Hi,

I use mlx5 to setup iSER and have scsi mq enabled.

root@xxx:~# cat /sys/module/scsi_mod/parameters/use_blk_mq
Y

But it seems there is only 1 HW queue that maps to 48 SW queues.

root@xxx:/sys/block/sdb/mq# ls
0

root@xxx:/sys/block/sdb/mq# ls 0/
active  cpu1   cpu11  cpu13  cpu15  cpu17  cpu19  cpu20  cpu22  cpu24
cpu26  cpu28  cpu3   cpu31  cpu33  cpu35  cpu37  cpu39  cpu40  cpu42
cpu44  cpu46  cpu5  cpu7  cpu9      dispatched  pending  run
cpu0    cpu10  cpu12  cpu14  cpu16  cpu18  cpu2   cpu21  cpu23  cpu25
cpu27  cpu29  cpu30  cpu32  cpu34  cpu36  cpu38  cpu4   cpu41  cpu43
cpu45  cpu47  cpu6  cpu8  cpu_list  io_poll     queued   tags

How to configure it that I can have 48 HW queues?
That is 1 HW queue map to 1 SW queue.

Thanks.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: iSER multi-queue configuration
  2016-04-29 21:11 iSER multi-queue configuration Ming Lin
@ 2016-04-30  6:13 ` Ming Lin
       [not found]   ` <1461996836.4373.5.camel-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Ming Lin @ 2016-04-30  6:13 UTC (permalink / raw)
  To: linux-rdma, Sagi Grimberg; +Cc: Bart Van Assche, target-devel

On Fri, 2016-04-29 at 14:11 -0700, Ming Lin wrote:
> Hi,
> 
> I use mlx5 to setup iSER and have scsi mq enabled.
> 
> root@xxx:~# cat /sys/module/scsi_mod/parameters/use_blk_mq
> Y
> 
> But it seems there is only 1 HW queue that maps to 48 SW queues.
> 
> root@xxx:/sys/block/sdb/mq# ls
> 0
> 
> root@xxx:/sys/block/sdb/mq# ls 0/
> active  cpu1   cpu11  cpu13  cpu15  cpu17  cpu19  cpu20  cpu22  cpu24
> cpu26  cpu28  cpu3   cpu31  cpu33  cpu35  cpu37  cpu39  cpu40  cpu42
> cpu44  cpu46  cpu5  cpu7  cpu9      dispatched  pending  run
> cpu0    cpu10  cpu12  cpu14  cpu16  cpu18  cpu2   cpu21  cpu23  cpu25
> cpu27  cpu29  cpu30  cpu32  cpu34  cpu36  cpu38  cpu4   cpu41  cpu43
> cpu45  cpu47  cpu6  cpu8  cpu_list  io_poll     queued   tags
> 
> How to configure it that I can have 48 HW queues?
> That is 1 HW queue map to 1 SW queue.

Hi Sagi,

I'm looking at why there is only 1 HW queue for iser.

2185 int scsi_mq_setup_tags(struct Scsi_Host *shost)
2186 {
...
2199         shost->tag_set.nr_hw_queues = shost->nr_hw_queues ? : 1;


It seems because iser code does not set shost->nr_hw_queues at all.

Why?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: iSER multi-queue configuration
       [not found]   ` <1461996836.4373.5.camel-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
@ 2016-05-01  2:53     ` Sagi Grimberg
       [not found]       ` <57256FB6.7030607-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Sagi Grimberg @ 2016-05-01  2:53 UTC (permalink / raw)
  To: Ming Lin, linux-rdma-u79uwXL29TY76Z2rM5mHXA; +Cc: Bart Van Assche, target-devel


> Hi Sagi,

Hi Ming,

> I'm looking at why there is only 1 HW queue for iser.
>
> 2185 int scsi_mq_setup_tags(struct Scsi_Host *shost)
> 2186 {
> ...
> 2199         shost->tag_set.nr_hw_queues = shost->nr_hw_queues ? : 1;
>
>
> It seems because iser code does not set shost->nr_hw_queues at all.
>
> Why?

The iscsi driver has not yet been converted to scsi_mq and iser as
an iscsi transport still does not use multi-queue.

In iscsi, a hw queue maps to an iscsi session. In order to support
multi-queue we need to have iscsi open multiple sessions and map
a session to a queue.

I had some preliminary works on this (joint with Mike Christie) but I
didn't have time thus far to implement a full conversion to scsi-mq.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: iSER multi-queue configuration
       [not found]       ` <57256FB6.7030607-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
@ 2016-05-01  6:07         ` Ming Lin
  2016-05-02 15:12           ` Steve Wise
  0 siblings, 1 reply; 5+ messages in thread
From: Ming Lin @ 2016-05-01  6:07 UTC (permalink / raw)
  To: Sagi Grimberg, linux-rdma-u79uwXL29TY76Z2rM5mHXA
  Cc: Bart Van Assche, target-devel

On Sun, 2016-05-01 at 05:53 +0300, Sagi Grimberg wrote:
> > Hi Sagi,
> 
> Hi Ming,
> 
> > I'm looking at why there is only 1 HW queue for iser.
> > 
> > 2185 int scsi_mq_setup_tags(struct Scsi_Host *shost)
> > 2186 {
> > ...
> > 2199         shost->tag_set.nr_hw_queues = shost->nr_hw_queues ? :
> > 1;
> > 
> > 
> > It seems because iser code does not set shost->nr_hw_queues at all.
> > 
> > Why?
> 
> The iscsi driver has not yet been converted to scsi_mq and iser as
> an iscsi transport still does not use multi-queue.

I'm going to compare performance between NVMe over RDMA and SCSI over
RDMA. SRP supports scsi_mq, so I'd use SRP to do the apples-to-apples
comparison.

> 
> In iscsi, a hw queue maps to an iscsi session. In order to support
> multi-queue we need to have iscsi open multiple sessions and map
> a session to a queue.
> 
> I had some preliminary works on this (joint with Mike Christie) but I
> didn't have time thus far to implement a full conversion to scsi-mq.

I'd happy to test it once you done.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: iSER multi-queue configuration
  2016-05-01  6:07         ` Ming Lin
@ 2016-05-02 15:12           ` Steve Wise
  0 siblings, 0 replies; 5+ messages in thread
From: Steve Wise @ 2016-05-02 15:12 UTC (permalink / raw)
  To: 'Ming Lin', 'Sagi Grimberg', linux-rdma
  Cc: 'Bart Van Assche', 'target-devel'



> -----Original Message-----
> From: linux-rdma-owner@vger.kernel.org [mailto:linux-rdma-
> owner@vger.kernel.org] On Behalf Of Ming Lin
> Sent: Sunday, May 01, 2016 1:07 AM
> To: Sagi Grimberg; linux-rdma@vger.kernel.org
> Cc: Bart Van Assche; target-devel
> Subject: Re: iSER multi-queue configuration
> 
> On Sun, 2016-05-01 at 05:53 +0300, Sagi Grimberg wrote:
> > > Hi Sagi,
> >
> > Hi Ming,
> >
> > > I'm looking at why there is only 1 HW queue for iser.
> > >
> > > 2185 int scsi_mq_setup_tags(struct Scsi_Host *shost)
> > > 2186 {
> > > ...
> > > 2199         shost->tag_set.nr_hw_queues = shost->nr_hw_queues ? :
> > > 1;
> > >
> > >
> > > It seems because iser code does not set shost->nr_hw_queues at all.
> > >
> > > Why?
> >
> > The iscsi driver has not yet been converted to scsi_mq and iser as
> > an iscsi transport still does not use multi-queue.
> 
> I'm going to compare performance between NVMe over RDMA and SCSI over
> RDMA. SRP supports scsi_mq, so I'd use SRP to do the apples-to-apples
> comparison.
> 
> >
> > In iscsi, a hw queue maps to an iscsi session. In order to support
> > multi-queue we need to have iscsi open multiple sessions and map
> > a session to a queue.
> >
> > I had some preliminary works on this (joint with Mike Christie) but I
> > didn't have time thus far to implement a full conversion to scsi-mq.
> 
> I'd happy to test it once you done.

Ditto.  SRP as a comparison is only useful for IB/RoCE transports.  iSER will allow iWARP to play in the game as well...

Steve.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-05-02 15:12 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-29 21:11 iSER multi-queue configuration Ming Lin
2016-04-30  6:13 ` Ming Lin
     [not found]   ` <1461996836.4373.5.camel-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2016-05-01  2:53     ` Sagi Grimberg
     [not found]       ` <57256FB6.7030607-NQWnxTmZq1alnMjI0IkVqw@public.gmane.org>
2016-05-01  6:07         ` Ming Lin
2016-05-02 15:12           ` Steve Wise

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).