Makes sense to me!

Reviewed-by: Nicholas Pratte <npra...@iol.unh.edu>

On Thu, Mar 6, 2025 at 4:47 PM Dean Marx <dm...@iol.unh.edu> wrote:
>
> Add a field to the TestPmdVerbosePacket text parser
> that stores destination port values for TCP/UDP packets.
>
> Signed-off-by: Dean Marx <dm...@iol.unh.edu>
> ---
>  dts/framework/remote_session/testpmd_shell.py | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/dts/framework/remote_session/testpmd_shell.py 
> b/dts/framework/remote_session/testpmd_shell.py
> index 1f291fcb68..f99c24bb04 100644
> --- a/dts/framework/remote_session/testpmd_shell.py
> +++ b/dts/framework/remote_session/testpmd_shell.py
> @@ -1278,6 +1278,11 @@ class TestPmdVerbosePacket(TextParser):
>      l3_len: int | None = field(default=None, 
> metadata=TextParser.find_int(r"l3_len=(\d+)"))
>      #:
>      l4_len: int | None = field(default=None, 
> metadata=TextParser.find_int(r"l4_len=(\d+)"))
> +    #:
> +    l4_dport: int | None = field(
> +        default=None,
> +        metadata=TextParser.find_int(r"(?:Destination TCP port=|Destination 
> UDP port=)(\d+)"),
> +    )
>
>
>  class RxOffloadCapability(Flag):
> --
> 2.48.1
>

Reply via email to