> > - cl = cmdline_stdin_new(main_ctx, "testpmd> ");
> > - if (cl == NULL) {
> > + testpmd_cl = cmdline_stdin_new(main_ctx, "testpmd> ");
> > + if (testpmd_cl == NULL) {
> > return;
> > }
>
> Style nit: don't need {} around single statement.
>
> > +static void
> > +sigint_handler(__rte_unused int signum) {
> > + if (signum == SIGINT || signum == SIGTERM) {
>
> signmum is used, so don't want __rte_unused
> Thanks :) Will fix these in the next version.

