linux-kernel - Re: [PATCH] pci: irq: Add an early parameter to limit pci irq numbers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAAhV-H64NWn0gk6Y0gAHe+YUUuALPySgKPC1WR5faWihV_s5rQ@mail.gmail.com>
Date:   Thu, 1 Jun 2023 12:19:04 +0800
From:   Huacai Chen <chenhuacai@...il.com>
To:     Jason Gunthorpe <jgg@...pe.ca>
Cc:     Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
        Bjorn Helgaas <helgaas@...nel.org>,
        Huacai Chen <chenhuacai@...ngson.cn>,
        Bjorn Helgaas <bhelgaas@...gle.com>,
        Thomas Gleixner <tglx@...utronix.de>,
        "Ahmed S . Darwish" <darwi@...utronix.de>,
        Kevin Tian <kevin.tian@...el.com>, linux-pci@...r.kernel.org,
        Jianmin Lv <lvjianmin@...ngson.cn>,
        Jiaxun Yang <jiaxun.yang@...goat.com>,
        loongson-kernel@...ts.loongnix.cn,
        Juxin Gao <gaojuxin@...ngson.cn>,
        Marc Zyngier <maz@...nel.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] pci: irq: Add an early parameter to limit pci irq numbers

Hi, Jason,

On Thu, Jun 1, 2023 at 3:14 AM Jason Gunthorpe <jgg@...pe.ca> wrote:
>
> On Mon, May 29, 2023 at 02:52:29PM +0800, Huacai Chen wrote:
>
> > > But IMO what you are proposing seems like usecase driven and may not work all
> > > the time due to architecture limitation. This again proves that the existing
> > > solution is sufficient enough.
>
> > Yes, it's a usecase driven solution, so I provide a cmdline parameter
> > to let the user decide.
>
> The NIC drivers should be consuming interrupts based on the number of
> queues they are using, and that is something you can control from the
> command line, eg ethtool IIRC. Usually it defaults to the number of
> CPUs.
>
> Basically, you want to enable the user to configure the system with a
> user specified reduced number of NIC queues, and we already have way
> to do that.
Yes, ethtool is a possible way, thank you very much.

Huacai
>
> Jason