[PATCH v2 1/1] common/cnxk: fix static assertion failure

[Ferruh Yigit]

On 3/2/2022 1:46 PM, Vamsi Attunuru wrote:
> Use dynamically allocated memory for storing soft expiry
> ring base addresses which fixes the static assertion failure,
> as the size of dynamic allocation depends on RTE_MAX_ETHPORTS
> which varies based on the build config.
> 

Hi Vamsi,

"fix static assertion failure" is not enough descriptive.
assertions already added to verify assumptions, and in this case
it seems it failed, but what was actually wrong?

Is it that allocated memory size for ring wrong? (this is what I got
from commit log but I am not sure)

Can you please describe what actually was wrong and fixed now?

> Bugzilla ID: 940
> Fixes: d26185716d3f ("net/cnxk: support outbound soft expiry notification")
> Cc:stable at dpdk.org
> 
> Reported-by: Wei Ling<weix.ling at intel.com>
> Reported-by: Yu Jiang<yux.jiang at intel.com>
> Signed-off-by: Vamsi Attunuru<vattunuru at marvell.com>
> Signed-off-by: Srikanth Yalavarthi<syalavarthi at marvell.com>
> ---
> V2: Add bugzilla & reportee details, remove unused changes.
> ---