On 2023/3/14 21:45, Ferruh Yigit wrote: > On 3/14/2023 11:50 AM, Chengwen Feng wrote: >> This patch adds 'dump_eth_priv [port_id]' which could used to dump >> the specific ethdev port private info. >> > > We have already "show port ..." commands to display various port related > information, what do you think to follow same syntax for port private info. > > Something like "show port private_info <port_id>" > > > Or it is possible to append this to the output of the existing port info > ("show port info <port_id>"), > as a new section of output if port has private info? > > This can be an addition parameter to existing command but not sure if we > need it, like: > "show port info <port_id> private on" > > I think displaying private info as port of port info without additional > parameter is OK, it is simpler.
+1 for this, will send a new patch. Thanks. > > ...