Linux-NVME Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Naman Jain <namjain@linux.microsoft.com>
To: changfengnan@bytedance.com
Cc: lrizzo@google.com, hch@infradead.org, tglx@linutronix.de,
	maz@kernel.org, rizzo.unipi@gmail.com, pabeni@redhat.com,
	linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org,
	bhelgaas@google.com, netdev@vger.kernel.org,
	linux-nvme@lists.infradead.org, guzebing@bytedance.com,
	kbusch@kernel.org
Subject: Re: [PATCH v5 0/7] Global Software Interrupt Moderation (GSIM)
Date: Mon, 19 Oct 2026 06:44:29 +0000	[thread overview]
Message-ID: <20261019064429.362555-1-namjain@linux.microsoft.com> (raw)
In-Reply-To: <8e816cdc-a714-49d8-943f-677617682352@bytedance.com>

On Wed, Sep 09, 2026 at 04:52:27PM +0800, Fengnan wrote:
> 在 2026/9/9 16:06, Luigi Rizzo 写道:
> > On Wed, Sep 9, 2026 at 8:25 AM Christoph Hellwig <hch@infradead.org> wrote:
> >> On Sat, Sep 05, 2026 at 10:30:22PM +0200, Thomas Gleixner wrote:
> >>>> it appears that GSIM is only effective in scenarios where multi disks at very high
> >>>> IOPS; in some cases, there was a noticeable performance regression.
> >>>> If there’s something wrong with my configuration, please correct me.
> >>> So we have a NVME specific mechanism to tackle the same problem and a
> >>> more generic version which is subsystem "independent".
> >> I'm not sure they tackle the entirely same problem, although they are
> >> very related.
> > There is overlap but it is accidental.
> > The difference between the two mechanism is the following:
> >
> > - GSIM addresses hardware limitations (SoCs that create huge PCIe
> >    backpressure on each MSIx interrupt, eventually reducing the available
> >    bandwidth too much, regardless of how many CPUs can process interrupts).
> >    The goal here is to throttle the total MSIx rate, blocking them at
> >    the source (PCIe device) rather than later in the interrupt controller.
> >
> > - the NVME mechanism (and NAPI) aim to amortize interrupt processing
> >    overhead. It does so by recreating some fine grained single-source
> >    interrupt moderation (something that NICs do in hardware, but NVME
> >    does not, with 100us being way too coarse for modern SSDs) and an
> >    adaptive scheme to adjust the delay and decide when to kick in.
> That's a great summary.
>
> As for generic solutions, I've tried several approaches before, but none
> of them
> could achieve the same or close results as NVMe adaptive interrupt polling.
> I'm still experimenting with different options.
>
>
> >
> > I think there is a reason for both.
> >
> > GSIM because it is more general and addresses an issue the others cannot handle
> > the subsystem-specific scheme because they can be more efficient
> > (eg they can hook into existing helpers like the threaded irq handler,
> > call directly the handler without going through the full irq chain,
> > use more targeted heuristics to decide when to kick in).
> >
> > cheers
> > luigi

Sharing the test results with GSIM and other NVME specific approaches
here as well. Previously discussed in
https://lore.kernel.org/all/1d12109d-2f47-4ca5-af7b-66c032a379a7@linux.microsoft.com/

There are couple of implementations posted on NVME list, including mine,
to solve nvme issues. Below data also suggests that while GSIM may bring
generic improvements in certain cases, we may still need some NVME
specific changes to address hardirq workload problems.

Background: I was debugging soft lockup issues where sufficiently high FIO
workload can lockup the CPU cores on which high number of NVME PCI MSI
interrupts are affined.

System: 128 ARM64 vCPUs, two NUMA nodes (CPUs 0-63 / 64-127),
There are four data NVMe controllers, each with 14 interrupt-driven I/O 
queues. Corresponding queue vectors from the four controllers are routed
to the same CPU. Therefore:

   vectors per affected CPU:       4
   affected interrupt CPUs:       14
   total data-NVMe I/O vectors:   56

The affected CPUs are:
   0, 10, 19, 28, 37, 46, 55,
   64, 74, 83, 92, 101, 110, 119
The reproduced soft lockups happen on exactly those 14 CPUs.

The production fio command is equivalent to:
   --filename=dev1:dev2:dev3:dev4
   --numjobs=128
   --iodepth=256
   --group_reporting

so the potential total is:
   128 x 256 = 32,768 requests

The NVMe hardware queue depth is 1,024, with 1,023 usable entries.

Default GSIM changes could not prevent soft lockup. Luigi suggested
a timer push change in addition to GSIM changes, which prevents the
lockups, but has an impact on latency and throughput. These are marked as
"push" in below tables.

NVMe fallback: changes I proposed in NVME, here:
https://lore.kernel.org/all/20261009050556.2817978-1-namjain@linux.microsoft.com/
threaded NVMe IRQs: Default kernel with nvme.use_threaded_interrupts=1

High-load results are medians of three runs, you can find complete 
minimum-to-maximum range:

                               IOPS median       complete range
   NVMe fallback                 5.004M          5.002-5.004M
   threaded NVMe IRQs            3.140M          3.100-3.169M
   GSIM fixed 10 us + push       1.962M          1.935-2.025M
   GSIM fixed 30 us + push       1.790M          1.775-1.822M
   GSIM fixed 50 us + push       1.744M          1.736-1.746M

Completion latency:
                               mean        p99          p99.9
   NVMe fallback              6.533 ms    14.352 ms     16.908 ms
   threaded NVMe IRQs        10.421 ms    22.151 ms     24.510 ms
   GSIM fixed 10 us + push   16.570 ms    38.011 ms    270.533 ms
   GSIM fixed 30 us + push   18.283 ms    40.108 ms    492.831 ms
   GSIM fixed 50 us + push   18.743 ms    40.632 ms    541.065 ms

More details and numbers in my RFC series.

Regards,
Naman


      reply	other threads:[~2026-10-19  6:44 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20260819124341.4185621-1-lrizzo@google.com>
2026-08-20  7:09 ` [PATCH v5 0/7] Global Software Interrupt Moderation (GSIM) Christoph Hellwig
2026-08-20  7:34   ` Luigi Rizzo
2026-08-20 11:46     ` changfengnan
2026-09-05 20:30       ` Thomas Gleixner
2026-09-09  6:25         ` Christoph Hellwig
2026-09-09  8:06           ` Luigi Rizzo
2026-09-09  8:52             ` Fengnan
2026-10-19  6:44               ` Naman Jain [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20261019064429.362555-1-namjain@linux.microsoft.com \
    --to=namjain@linux.microsoft.com \
    --cc=bhelgaas@google.com \
    --cc=changfengnan@bytedance.com \
    --cc=guzebing@bytedance.com \
    --cc=hch@infradead.org \
    --cc=kbusch@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nvme@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=lrizzo@google.com \
    --cc=maz@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=rizzo.unipi@gmail.com \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox