RE: [EXTERNAL] Re: [net-next PATCH v5 1/6] octeontx2: Set appropriate PF, VF masks and shifts based on silicon
From: Subbaraya Sundeep Bhatta
Date: Mon Dec 09 2024 - 04:10:12 EST
Hi Jakub,
>From: Jakub Kicinski <kuba@xxxxxxxxxx>
>Sent: Sunday, December 8, 2024 8:08 AM
>To: Sai Krishna Gajula <saikrishnag@xxxxxxxxxxx>
>Cc: davem@xxxxxxxxxxxxx; edumazet@xxxxxxxxxx; pabeni@xxxxxxxxxx;
>netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Sunil Kovvuri Goutham
><sgoutham@xxxxxxxxxxx>; Geethasowjanya Akula <gakula@xxxxxxxxxxx>;
>Linu Cherian <lcherian@xxxxxxxxxxx>; Jerin Jacob <jerinj@xxxxxxxxxxx>;
>Hariprasad Kelam <hkelam@xxxxxxxxxxx>; Subbaraya Sundeep Bhatta
><sbhatta@xxxxxxxxxxx>; andrew+netdev@xxxxxxx; kalesh-
>anakkur.purayil@xxxxxxxxxxxx
>Subject: Re: [net-next PATCH v5 1/6] octeontx2: Set appropriate PF, VF masks
>and shifts based on silicon
>
>
>On Wed, 4 Dec 2024 19:38:16 +0530 Sai Krishna wrote:
>> -#define RVU_PFVF_PF_SHIFT 10
>> -#define RVU_PFVF_PF_MASK 0x3F
>> -#define RVU_PFVF_FUNC_SHIFT 0
>> -#define RVU_PFVF_FUNC_MASK 0x3FF
>> +#define RVU_PFVF_PF_SHIFT rvu_pcifunc_pf_shift
>> +#define RVU_PFVF_PF_MASK rvu_pcifunc_pf_mask
>> +#define RVU_PFVF_FUNC_SHIFT rvu_pcifunc_func_shift
>> +#define RVU_PFVF_FUNC_MASK rvu_pcifunc_func_mask
>
>Why do you maintain these defines? Looks like an unnecessary
>indirection.
>
>Given these are simple mask and shift values they probably have trivial
>users. Start by adding helpers which perform the conversions using
>those, then you can more easily update constants.
>
There are too many places these masks are used hence added this
indirection.
# grep RVU_PFVF_ drivers/* -inr | wc -l
135
Thanks,
Sundeep