Re: [Bug 269133] bnxt(4): BCM57416 - HWRM_CFA_L2_SET_RX_MASK command returned RESOURCE_ALLOC_ERROR error

From: Santiago Martinez <sm_at_codenetworks.net>
Date: Thu, 09 Feb 2023 22:16:49 UTC
Hi there! just a quick update,  the issue is related the following code 
in bnxt_hwrm.c, line 1480.

This gets always true and then returns. I have commented out this and 
the nics is working with stable/13 (today).

Now not sure whats the correct check that this if should do, any hints??

     if (*filter_id != -1) {
         device_printf(softc->dev, "Attempt to re-allocate l2 ctx "
             "filter (fid: 0x%jx)\n", (uintmax_t)*filter_id);

         return EDOOFUS;
     }


Best regards.

Santiago


On 2/7/23 10:18, Santiago Martinez wrote:
> Hi there, just in case, this is the link to the commit that broke the 
> driver for BMC57416.
>
> https://github.com/freebsd/freebsd-src/commit/91fcacc355971f74aa26fc7861020dc3a2a2d717 
>
>
> I'm worried that we are getting close to 13.2 and if its shipped as it 
> is I'm pretty sure it will break some deployments.
>
> Best regards.
>
> Santiago
>
>
> On 2/3/23 09:05, Santiago Martinez wrote:
>> Hi everyone,
>>
>> Seems that commit "2db35273502b3c35aa653effc5c97618567367ab" brakes 
>> bnxt driver for BCM57416 based cards.
>>
>> I have tried reverting the commit and the servers are now back online.
>>
>> if anyone has some spare time and energy, can please take a look at 
>> the commit, as I´m not sure what part of the commit is braking it.
>>
>> For detail please refer to bugzilla. 
>> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=269133
>>
>> Thanks a lot!
>>
>> Santiago
>>
>>
>>
>