[PATCH v2 3/4] dts: add methods for modifying MTU to testpmd shell

Jeremy Spewock jspewock at iol.unh.edu
Fri May 31 23:08:22 CEST 2024


On Fri, May 31, 2024 at 12:34 PM Luca Vizzarro <Luca.Vizzarro at arm.com> wrote:
>
> Due to the nature of this patch the console is spammed with a lot of
> commands. Would it be better to log these in debug and instead log:
>    Setting port X to MTU XXXX
> as INFO?

Potentially, but this would require a global change for how logging
works when sending commands to interactive shells in general. They are
each logged as their own message since they are each individual
commands being sent into the shell which (in general) we do want to
log. I could maybe add an optional flag to the send command function
that logs its output to debug rather than info however which might be
a nicer solution. I agree that it does get cluttered.


More information about the dev mailing list