netdev - Re: [PATCH net-next v2] bnxt: fix bnxt_get_avail

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACKFLikqOuJSpn3uZ8rnscOJvB76-cZDXy5OMzjd-GCqEYhHFg@mail.gmail.com>
Date: Thu, 2 May 2024 13:56:13 -0700
From: Michael Chan <michael.chan@...adcom.com>
To: David Wei <dw@...idwei.uk>
Cc: netdev@...r.kernel.org, Pavan Chebbi <pavan.chebbi@...adcom.com>, 
	Andy Gospodarek <andrew.gospodarek@...adcom.com>, Jakub Kicinski <kuba@...nel.org>, 
	"David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, 
	Paolo Abeni <pabeni@...hat.com>
Subject: Re: [PATCH net-next v2] bnxt: fix bnxt_get_avail_msix() returning
 negative values

On Thu, May 2, 2024 at 1:38 PM David Wei <dw@...idwei.uk> wrote:
>
> Current net-next/main does not boot for older chipsets e.g. Stratus.
>
> Sample dmesg:
> [   11.368315] bnxt_en 0000:02:00.0 (unnamed net_device) (uninitialized): Able to reserve only 0 out of 9 requested RX rings
> [   11.390181] bnxt_en 0000:02:00.0 (unnamed net_device) (uninitialized): Unable to reserve tx rings
> [   11.438780] bnxt_en 0000:02:00.0 (unnamed net_device) (uninitialized): 2nd rings reservation failed.
> [   11.487559] bnxt_en 0000:02:00.0 (unnamed net_device) (uninitialized): Not enough rings available.
> [   11.506012] bnxt_en 0000:02:00.0: probe with driver bnxt_en failed with error -12
>
> This is caused by bnxt_get_avail_msix() returning a negative value for
> these chipsets not using the new resource manager i.e. !BNXT_NEW_RM.
> This in turn causes hwr.cp in __bnxt_reserve_rings() to be set to 0.
>
> In the current call stack, __bnxt_reserve_rings() is called from
> bnxt_set_dflt_rings() before bnxt_init_int_mode(). Therefore,
> bp->total_irqs is always 0 and for !BNXT_NEW_RM bnxt_get_avail_msix()
> always returns a negative number.
>
> Historically, MSIX vectors were requested by the RoCE driver during
> run-time and bnxt_get_avail_msix() was used for this purpose. Today,
> RoCE MSIX vectors are statically allocated. bnxt_get_avail_msix() should
> only be called for the BNXT_NEW_RM() case to reserve the MSIX ahead of
> time for RoCE use.
>
> bnxt_get_avail_msix() is also be simplified to handle the BNXT_NEW_RM()
> case only.
>
> v2:
>  - only call bnxt_get_avail_msix() if BNXT_NEW_RM() is used
>  - remove non BNXT_NEW_RM() logic in bnxt_get_avail_msix()
>  - make bnxt_get_avail_msix() static
>
> Fixes: d630624ebd70 ("bnxt_en: Utilize ulp client resources if RoCE is not registered")
> Signed-off-by: David Wei <dw@...idwei.uk>

Thanks.
Reviewed-by: Michael Chan <michael.chan@...adcom.com>

Download attachment "smime.p7s" of type "application/pkcs7-signature" (4209 bytes)