<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Oct 8, 2025 at 7:45 AM 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">Hi Patrick,<br>
<br>
I am assuming the move of send_packets is because we can't send<br>
arbitrary packets with a perf generator. I believe this should be<br>
explained as it's not clear why the move and it feels counterintuitive<br>
at first.<br></blockquote><div><br></div><div>Sort of. We can send arbitrary packets with a perf generator, but there is a particular use case for send_packet (send a small number of arbitrary packets) and this is just "better" done by a capturing packet generator like Scapy since it is concerned with reliability. The performance traffic generators have a different focus (send in high volume), so in my view it makes sense to encourage people to use a capturing traffic generator when they want to send a small amount of arbitrary packets. If you think this idea is okay, yes I can update the commit message. Or, I can reverse the decision altogether if you don't think my justification is sound.</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">
<br>
On Wed, Oct 01, 2025 at 07:16:57PM +0000, Patrick Robb wrote:<br>
> diff --git a/dts/framework/testbed_model/traffic_generator/performance_traffic_generator.py b/dts/framework/testbed_model/traffic_generator/performance_traffic_generator.py<br>
> new file mode 100644<br>
> index 0000000000..6b23faa1a5<br><br>
> +    def setup(self, topology: Topology) -> None:<br>
> +        """Overrides :meth:`.traffic_generator.TrafficGenerator.setup`."""<br>
> +        for port in self._tg_node.ports:<br>
> +            self._tg_node.main_session.configure_port_mtu(2000, port)<br>
<br>
should do:<br>
  port.configure_mtu(2000)<br></blockquote><div><br></div><div>Thanks for starting a review. Do you have the hardware available to test the patch as well? I am wondering whether my docs additions are sufficient (I think they are, but a second opinion is always good). If you can't, I'll ask Andrew to test it.</div></div></div>