Re: [PATCH net v7] bnxt_en: set backing store type from query type

From: Michael Chan

Date: Sat Mar 28 2026 - 23:28:06 EST


On Sat, Mar 28, 2026 at 4:44 PM Pengpeng Hou <pengpeng@xxxxxxxxxxx> wrote:
>
> bnxt_hwrm_func_backing_store_qcaps_v2() stores resp->type from the
> firmware response in ctxm->type and later uses that value to index
> fixed backing-store metadata arrays such as ctx_arr[] and
> bnxt_bstore_to_trace[].
>
> ctxm->type is fixed by the current backing-store query type and matches
> the array index of ctx->ctx_arr. Set ctxm->type from the current loop
> variable instead of depending on resp->type.
>
> Also update the loop to advance type from next_valid_type in the for
> statement, which keeps the control flow simpler for non-valid and
> unchanged entries.
>
> Fixes: 6a4d0774f02d ("bnxt_en: Add support for new backing store query firmware API")
> Signed-off-by: Pengpeng Hou <pengpeng@xxxxxxxxxxx>
> ---
> v7:
> - rename the patch to match the new fix
> - advance type from next_valid_type in the for loop statement

Next time, please wait the full 24 hours before posting a new version. Thanks.

Reviewed-by: Michael Chan <michael.chan@xxxxxxxxxxxx>
Tested-by: Michael Chan <michael.chan@xxxxxxxxxxxx>

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature