<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Jul 2, 2025 at 12:43 PM Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com">luca.vizzarro@arm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Make a distinction between example apps and app/ apps by renaming the<br>
build app function. Moreover, provide a dedicated helper function to<br>
retrieve the path to a DPDK app.<br>
<br>
Signed-off-by: Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com" target="_blank">luca.vizzarro@arm.com</a>><br>
Reviewed-by: Paul Szczepanek <<a href="mailto:paul.szczepanek@arm.com" target="_blank">paul.szczepanek@arm.com</a>><br>
---<br>
dts/framework/remote_session/dpdk.py | 8 ++++++--<br>
1 file changed, 6 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/dts/framework/remote_session/dpdk.py b/dts/framework/remote_session/dpdk.py<br>
index e00c4a9bf9..ed1f808bdd 100644<br>
--- a/dts/framework/remote_session/dpdk.py<br>
+++ b/dts/framework/remote_session/dpdk.py<br>
@@ -267,8 +267,8 @@ def _build_dpdk(self) -> None:<br>
self.remote_dpdk_build_dir,<br>
)<br>
<br>
- def build_dpdk_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePath:<br>
- """Build one or all DPDK apps.<br>
+ def build_dpdk_example_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePath:<br>
+ """Build one or all DPDK example apps.<br>
<br></blockquote><div><br></div><div>We have previously been encouraged to avoid using sample apps unless absolutely necessary, so it probably doesn't make sense for us to add this in until we have a specific reason and justification for supporting example apps (and this may never happen).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Requires DPDK to be already built on the SUT node.<br>
<br>
@@ -298,6 +298,10 @@ def build_dpdk_app(self, app_name: str, **meson_dpdk_args: str | bool) -> PurePa<br>
self.remote_dpdk_build_dir, "examples", f"dpdk-{app_name}"<br>
)<br>
<br>
+ def get_app(self, app_name: str) -> PurePath:<br>
+ """Retrieve path for a DPDK app."""<br>
+ return self._session.join_remote_path(self.remote_dpdk_build_dir, "app", f"dpdk-{app_name}")<br>
+<br>
@cached_property<br>
def remote_dpdk_tree_path(self) -> PurePath:<br>
"""The remote DPDK tree path."""<br>
-- <br>
2.43.0<br>
<br></blockquote><div><br></div><div>Otherwise this looks fine for dumpcap and the others /app apps - thanks. </div><div><br></div><div>Reviewed-by: Patrick Robb <<a href="mailto:probb@iol.unh.edu">probb@iol.unh.edu</a>></div><div> </div></div></div>