[PATCH v2 5/7] dts: add support for externally compiled DPDK

[Patrick Robb]

On Mon, Oct 21, 2024 at 9:50 AM Luca Vizzarro <luca.vizzarro at arm.com> wrote:

> From: Tomáš Ďurovec <tomas.durovec at pantheon.tech>
>
> Enable the user to use either a DPDK source tree directory or a
> tarball, with and without a pre-built build directory. These can be
> stored on either SUT node or the DTS host. The DPDK build setup or the
> pre-built binaries can be specified through the configuration file,
> the command line arguments or environment variables.
>
> Signed-off-by: Tomáš Ďurovec <tomas.durovec at pantheon.tech>
> Signed-off-by: Luca Vizzarro <luca.vizzarro at arm.com>
> ---
>  dts/conf.yaml                                |  24 +-
>  dts/framework/config/__init__.py             | 123 ++++++-
>  dts/framework/config/conf_yaml_schema.json   |  62 +++-
>  dts/framework/config/types.py                |  17 +-
>  dts/framework/exception.py                   |   4 +-
>  dts/framework/remote_session/dpdk_shell.py   |   2 +-
>  dts/framework/runner.py                      |   8 +-
>  dts/framework/settings.py                    | 200 +++++++++--
>  dts/framework/test_result.py                 |  23 +-
>  dts/framework/testbed_model/node.py          |  22 +-
>  dts/framework/testbed_model/os_session.py    |  63 +++-
>  dts/framework/testbed_model/posix_session.py |  39 ++-
>  dts/framework/testbed_model/sut_node.py      | 351 +++++++++++++------
>  13 files changed, 732 insertions(+), 206 deletions(-)
>
> diff --git a/dts/conf.yaml b/dts/conf.yaml
> index 814744a1fc..8a65a481d6 100644
> --- a/dts/conf.yaml
> +++ b/dts/conf.yaml
> @@ -5,12 +5,24 @@
>  test_runs:
>    # define one test run environment
>    - dpdk_build:
> -      arch: x86_64
> -      os: linux
> -      cpu: native
> -      # the combination of the following two makes CC="ccache gcc"
> -      compiler: gcc
> -      compiler_wrapper: ccache
> +      # dpdk_tree: Commented out because `tarball` is defined.
>

Should this key name be extended to clarify the desired value is a path,
and whether it is absolute or relative?

+1 to Dean's concerns about using the tarball without --tarball flag if
dpdk_tree is commented out and tarball is defined.

Reviewed-by: Patrick Robb <probb at iol.unh.edu>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mails.dpdk.org/archives/dev/attachments/20241028/1b0d2bfe/attachment.htm>