Re: [PATCH v2] app/procinfo: show all non-owned ports

2022-06-06 Thread Subendu Santra
t; > > On Tue, 24 May 2022 22:46:05 -0700 > > Subendu Santra wrote: > > > > > Show all non-owned ports when no port mask is specified > > > > > > show-port option without the mask option, displays only the last > > > non-owned port. Show all

Re: [PATCH v2] app/procinfo: show all non-owned ports

2022-05-24 Thread Subendu Santra
I have updated the v2 of the patch with the explanation. Kindly advise if it looks ok. Regards, Subendu. On Wed, May 25, 2022 at 11:22 AM Subendu Santra wrote: > > Show all non-owned ports when no port mask is specified > > show-port option without the mask option, displays only th

[PATCH v2] app/procinfo: show all non-owned ports

2022-05-24 Thread Subendu Santra
rg Signed-off-by: Subendu Santra --- app/proc-info/main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/proc-info/main.c b/app/proc-info/main.c index 56070a3317..2be24b584e 100644 --- a/app/proc-info/main.c +++ b/app/proc-info/main.c @@ -1504,10 +1504,10 @@ main(int

[PATCH v2] app/procinfo: show all non-owned ports

2022-05-24 Thread Subendu Santra
rg Signed-off-by: Subendu Santra --- app/proc-info/main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/proc-info/main.c b/app/proc-info/main.c index 56070a3317..2be24b584e 100644 --- a/app/proc-info/main.c +++ b/app/proc-info/main.c @@ -1504,10 +1504,10 @@ main(int

Re: [PATCH] app/procinfo: show all non-owned ports

2022-05-23 Thread Subendu Santra
lon wrote: > > 23/05/2022 12:53, Subendu Santra: > > Hi Reshma, Thomas, > > > > What do you think about this patch? > > It looks to be a fix, right? > Please add a sentence about the previous behaviour, > explaining what is fixed. > > Ideally you should show t

Re: [PATCH] app/procinfo: show all non-owned ports

2022-05-23 Thread Subendu Santra
Hi Reshma, Thomas, What do you think about this patch? Regards, Subendu. On Tue, May 10, 2022 at 6:52 PM Subendu Santra wrote: > > Show all non-owned ports when no port mask is specified > > Fixes: 1dd6cffb6571 ("app/procinfo: provide way to request info on owned >

Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

2022-05-11 Thread Subendu Santra
Agreed. I will change it to uint64_t and send it for review. Thanks for your help. Regards, Subendu. On Wed, May 11, 2022 at 1:32 AM Stephen Hemminger wrote: > > On Tue, 10 May 2022 14:39:05 +0530 > Subendu Santra wrote: > > > Hi Stephen, Thomas, > > > > O

[PATCH] app/procinfo: show all non-owned ports

2022-05-10 Thread Subendu Santra
Show all non-owned ports when no port mask is specified Fixes: 1dd6cffb6571 ("app/procinfo: provide way to request info on owned ports") Cc: step...@networkplumber.org Signed-off-by: Subendu Santra --- app/proc-info/main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) di

Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

2022-05-10 Thread Subendu Santra
rm it runs on. Kindly share your thoughts on this. Regards, Subendu. On Wed, May 4, 2022 at 11:18 PM Stephen Hemminger < step...@networkplumber.org> wrote: > On Tue, 03 May 2022 10:47:58 +0200 > Thomas Monjalon wrote: > > > 24/04/2022 07:34, Subendu Santra: > > &g

Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

2022-05-03 Thread Subendu Santra
Hi Stephen, Could you please help us understand the rationale behind showing just the last non-owned port in case the port mask was not specified? I really appreciate your help in this regard. Regards, Subendu. On Sun, Apr 24, 2022 at 11:04 AM Subendu Santra wrote: > Hi Stephen, > >

Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

2022-04-26 Thread Subendu Santra
Hi Stephen, We were going through the patch set: https://inbox.dpdk.org/dev/20200715212228.28010-7-step...@networkplumber.org/ and hoping to get clarification on the behaviour if post mask is not specified in the input to `dpdk-proc-info` tool. Specifically, In PATCH v3 6/7, we see this: +