Re: [PATCH v3 3/4] block: drop shared-tag fairness throttling
Keith Busch <[email protected]> Wed, 10 Jun 2026 10:35:59 -0600
| Newsgroups | org.kernel.vger.target-devel,dev.linux.lists.virtualization,org.kernel.vger.linux-block,org.kernel.vger.linux-hyperv,org.kernel.vger.linux-scsi,org.ozlabs.lists.linuxppc-dev,org.xenproject.lists.xen-devel |
|---|---|
| Message-ID | <aimSb9I0Vl-68hy9@kbusch-mbp> |
On Wed, Jun 10, 2026 at 09:16:11PM +0530, Sumit Saxena wrote: > The motivation for this change stems from performance issue we > encountered due to false sharing of the 'nr_active_requests_shared_tags' > counter > on certain CPU architectures. I initially submitted a patch to move that > counter to > its own cache line to avoid conflicts with 'nr_requests' and other hot > fields > (see: > https://patchwork.kernel.org/project/linux-scsi/patch/[email protected]/ > ). > > During the review, Bart shared his work, which eliminates the > counter entirely by removing the fairness throttling. My testing confirmed > that > this approach resolved the performance issues and improved IOPS. > This patch is part of a larger set, and I have reported the cumulative > performance > improvements in the cover letter. So the problem is just the atomic operation accounting overhead? I previously thought the device just really needed to consume all the tags to hit performance.