Re: [net Patch] octeontx2-af: Fix issue with loading coalesced KPU profiles

[Kalesh Anakkur Purayil]

On Tue, Mar 26, 2024 at 5:53 PM Hariprasad Kelam <hkelam@xxxxxxxxxxx> wrote:
>
> The current implementation for loading coalesced KPU profiles has
> a limitation.  The "offset" field, which is used to locate profiles
> within the profile is restricted to a u16.
>
> This restricts the number of profiles that can be loaded. This patch
> addresses this limitation by increasing the size of the "offset" field.
>
> Fixes: 11c730bfbf5b ("octeontx2-af: support for coalescing KPU profiles")
> Signed-off-by: Hariprasad Kelam <hkelam@xxxxxxxxxxx>
LGTM
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@xxxxxxxxxxxx>
> ---
>  drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c b/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c
> index e350242bbafb..be709f83f331 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_npc.c
> @@ -1657,7 +1657,7 @@ static int npc_fwdb_detect_load_prfl_img(struct rvu *rvu, uint64_t prfl_sz,
>         struct npc_coalesced_kpu_prfl *img_data = NULL;
>         int i = 0, rc = -EINVAL;
>         void __iomem *kpu_prfl_addr;
> -       u16 offset;
> +       u32 offset;
>
>         img_data = (struct npc_coalesced_kpu_prfl __force *)rvu->kpu_prfl_addr;
>         if (le64_to_cpu(img_data->signature) == KPU_SIGN &&
> --
> 2.17.1
>
>


-- 
Regards,
Kalesh A P

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