Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1641385 > unrolled thread

[RFC] NVMe Configuraiton using sysctl

Started byOza Oza <oza.oza@broadcom.com>
First post2017-05-15 10:40 +0200
Last post2017-05-15 16:40 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [RFC] NVMe Configuraiton using sysctl Oza Oza <oza.oza@broadcom.com> - 2017-05-15 10:40 +0200
    Re: [RFC] NVMe Configuraiton using sysctl Oza Oza <oza.oza@broadcom.com> - 2017-05-15 10:50 +0200
      Re: [RFC] NVMe Configuraiton using sysctl Sagi Grimberg <sagi@grimberg.me> - 2017-05-15 11:20 +0200
        Re: [RFC] NVMe Configuraiton using sysctl Oza Oza <oza.oza@broadcom.com> - 2017-05-15 13:10 +0200
        Re: [RFC] NVMe Configuraiton using sysctl Keith Busch <keith.busch@intel.com> - 2017-05-15 16:40 +0200

#1641385 — [RFC] NVMe Configuraiton using sysctl

FromOza Oza <oza.oza@broadcom.com>
Date2017-05-15 10:40 +0200
Subject[RFC] NVMe Configuraiton using sysctl
Message-ID<tHjHQ-5H0-13@gated-at.bofh.it>
Hi,

we are configuring interrupt coalesce for NVMe, but right now, it uses
module param.
so the same interrupt coalesce settings get applied for all the NVMEs
connected to different RCs.

ideally it should be with sysctl.
for e.g.
sysctl should provide interface to change
Per-CPU IO queue pairs, interrupt coalesce settings etc..

please suggest if we could have/implement sysctl module for NVMe ?

Regards,
Oza.

[toc] | [next] | [standalone]

#1641393

FromOza Oza <oza.oza@broadcom.com>
Date2017-05-15 10:50 +0200
Message-ID<tHjRv-5Kw-3@gated-at.bofh.it>
In reply to#1641385
On Mon, May 15, 2017 at 2:04 PM, Oza Oza <oza.oza@broadcom.com> wrote:
> Hi,
>
> we are configuring interrupt coalesce for NVMe, but right now, it uses
> module param.
> so the same interrupt coalesce settings get applied for all the NVMEs
> connected to different RCs.
>
> ideally it should be with sysctl.
> for e.g.
> sysctl should provide interface to change
> Per-CPU IO queue pairs, interrupt coalesce settings etc..
>
> please suggest if we could have/implement sysctl module for NVMe ?
>
> Regards,
> Oza.

+ linux-nvme@lists.infradead.org

[toc] | [prev] | [next] | [standalone]

#1641450

FromSagi Grimberg <sagi@grimberg.me>
Date2017-05-15 11:20 +0200
Message-ID<tHkky-6aZ-13@gated-at.bofh.it>
In reply to#1641393
>> Hi,

Hi Oza,

>> we are configuring interrupt coalesce for NVMe, but right now, it uses
>> module param.
>> so the same interrupt coalesce settings get applied for all the NVMEs
>> connected to different RCs.
>>
>> ideally it should be with sysctl.

If at all, I would place this in nvme-cli (via ioctl) instead of
sysctl.

>> for e.g.
>> sysctl should provide interface to change
>> Per-CPU IO queue pairs, interrupt coalesce settings etc..

My personal feeling is that percpu granularity is a lot to take in for
the user, and also can yield some unexpected performance
characteristics. But I might be wrong here..

>> please suggest if we could have/implement sysctl module for NVMe ?

I have asked this before, but interrupt coalescing has very little
merit without being able to be adaptive. net drivers maintain online
stats and schedule interrupt coalescing modifications.

Should work in theory, but having said that, interrupt coalescing as a
whole is essentially unusable in nvme since the coalescing time limit
is in units of 100us increments...

[toc] | [prev] | [next] | [standalone]

#1641544

FromOza Oza <oza.oza@broadcom.com>
Date2017-05-15 13:10 +0200
Message-ID<tHm2Z-7kX-13@gated-at.bofh.it>
In reply to#1641450
On Mon, May 15, 2017 at 2:45 PM, Sagi Grimberg <sagi@grimberg.me> wrote:
>
>>> Hi,
>
>
> Hi Oza,
>
>>> we are configuring interrupt coalesce for NVMe, but right now, it uses
>>> module param.
>>> so the same interrupt coalesce settings get applied for all the NVMEs
>>> connected to different RCs.
>>>
>>> ideally it should be with sysctl.
>
>
> If at all, I would place this in nvme-cli (via ioctl) instead of
> sysctl.
>
>>> for e.g.
>>> sysctl should provide interface to change
>>> Per-CPU IO queue pairs, interrupt coalesce settings etc..
>
>
> My personal feeling is that percpu granularity is a lot to take in for
> the user, and also can yield some unexpected performance
> characteristics. But I might be wrong here..
>

I thought of nvme_ioctl, but was not sure whether sysctl or ioctl.
although we are interested only introducing interrupt coalesce,
because that brings improvements.

>>> please suggest if we could have/implement sysctl module for NVMe ?
>
>
> I have asked this before, but interrupt coalescing has very little
> merit without being able to be adaptive. net drivers maintain online
> stats and schedule interrupt coalescing modifications.
>
> Should work in theory, but having said that, interrupt coalescing as a
> whole is essentially unusable in nvme since the coalescing time limit
> is in units of 100us increments...

surprisingly, it brings 20% improvement in CPU utilization for us.
so it saves lot of our CPU cycles there freeing up to do something else.
the value has to be tuned but that's all there it is.
so we are keen on having this to tune in.

so your suggestion is to use IOCTL instead of sysctl right ?.
and as of now we are only interested in interrupt coalesce alone.

Regards,
Oza.

[toc] | [prev] | [next] | [standalone]

#1641756

FromKeith Busch <keith.busch@intel.com>
Date2017-05-15 16:40 +0200
Message-ID<tHpke-Uz-13@gated-at.bofh.it>
In reply to#1641450
On Mon, May 15, 2017 at 12:15:28PM +0300, Sagi Grimberg wrote:
> 
> > > Hi,
> 
> Hi Oza,
> 
> > > we are configuring interrupt coalesce for NVMe, but right now, it uses
> > > module param.
> > > so the same interrupt coalesce settings get applied for all the NVMEs
> > > connected to different RCs.
> > > 
> > > ideally it should be with sysctl.
> 
> If at all, I would place this in nvme-cli (via ioctl) instead of
> sysctl.

That's also how I currently recommend testing this feature out. A problem
with that, though, is the feature isn't persistent across controller
resets, so the setting could be reverted without the user knowing.


> > > for e.g.
> > > sysctl should provide interface to change
> > > Per-CPU IO queue pairs, interrupt coalesce settings etc..
> 
> My personal feeling is that percpu granularity is a lot to take in for
> the user, and also can yield some unexpected performance
> characteristics. But I might be wrong here..

We currently use the IRQ affinity spread to get good default pairings.
It's possible to decouple that, but let's hear what about the default
setting isn't optimal before exposing additional knobs. More user tunables
just means one of us will get to frequently re-explain how to use it!


> > > please suggest if we could have/implement sysctl module for NVMe ?
> 
> I have asked this before, but interrupt coalescing has very little
> merit without being able to be adaptive. net drivers maintain online
> stats and schedule interrupt coalescing modifications.
> 
> Should work in theory, but having said that, interrupt coalescing as a
> whole is essentially unusable in nvme since the coalescing time limit
> is in units of 100us increments...

Yeah, as it is defined, the low depth work-load latency does suffer
quite a bit. If the user only cares about IOPs, though, we find that
coalescing is necessary for some workloads to hit the peak capabilities.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web