[PATCH] test: eal: update base-virtaddr parameter test
Ajit Khaparde
ajit.khaparde at broadcom.com
Thu Jul 20 16:07:27 CEST 2023
On Thu, Jul 20, 2023 at 6:09 AM Aaron Conole <aconole at redhat.com> wrote:
>
> When 97435d7906 ("net/bnxt: update Truflow core") was applied, it introduced
> a number of static variables. These ended up filling the lower address space
> enough that on some platforms, the min addr for heap address to start was
> not within the range specified by base-virtaddr. This is dependent on both
> build and link time options.
>
> The simplest resolution is to just update the base virtaddr parameter to
> move the rte config location.
>
> Reported-by: Patrick Robb <probb at iol.unh.edu>
> Suggested-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
> Signed-off-by: Aaron Conole <aconole at redhat.com>
ACK.
I tested again and the test passed with this change. Thanks.
> ---
> app/test/test_eal_flags.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c
> index d2b91e2075..148e9098e8 100644
> --- a/app/test/test_eal_flags.c
> +++ b/app/test/test_eal_flags.c
> @@ -1017,7 +1017,7 @@ test_misc_flags(void)
>
> /* try running with base-virtaddr param */
> const char *argv13[] = {prgname, "--file-prefix=virtaddr",
> - "--base-virtaddr=0x12345678"};
> + "--base-virtaddr=0x23456789"};
>
> /* try running with --vfio-intr INTx flag */
> const char *argv14[] = {prgname, "--file-prefix=intr",
> --
> 2.40.1
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4218 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://mails.dpdk.org/archives/dev/attachments/20230720/70609ccb/attachment.bin>
More information about the dev
mailing list