> -----Original Message----- > From: Pattan, Reshma > Sent: Monday, February 26, 2018 11:02 AM > To: Varghese, Vipin <vipin.vargh...@intel.com>; dev@dpdk.org > Cc: Mcnamara, John <john.mcnam...@intel.com> > Subject: RE: [PATCH] app/pdump: check for ports > > > > > -----Original Message----- > > From: Varghese, Vipin > > Sent: Monday, February 26, 2018 4:30 PM > > To: dev@dpdk.org; Pattan, Reshma <reshma.pat...@intel.com> > > Cc: Mcnamara, John <john.mcnam...@intel.com>; Varghese, Vipin > > <vipin.vargh...@intel.com> > > Subject: [PATCH] app/pdump: check for ports > >
<snipped> > > + if (rte_eth_dev_count() == 0) > > + rte_exit(EXIT_FAILURE, "Failed to fetch ports for > PDUMP!\n"); > > + > > Good one, can you just change the failure log to rte_exit(EXIT_FAILURE, "No > Ethernet ports - bye\n"); As, I see most of the other examples and apps do > use the similar check and log as above. :-) > Sure, I will send a v2 patch with updated log information. > Thanks, > Reshma
