[dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path

2016-06-24 Thread Thomas Monjalon
2016-06-24 14:54, Reshma Pattan: > +#define SOCKET_DIR "/pdump_sockets" I think the default socket directory should contain dpdk as prefix. Like dpdk-pdump-sockets (I think dash is preferred for filenames). I wonder whether it should be a hidden directory: ~/.dpdk-pdump-sockets And a

[dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path

2016-06-24 Thread Pattan, Reshma
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, June 24, 2016 3:55 PM > To: Pattan, Reshma > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path > > 2016-06-24 14:54, Res

[dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path

2016-06-24 Thread Pattan, Reshma
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, June 24, 2016 3:55 PM > To: Pattan, Reshma > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path > > 2016-06-24 14:54, Res

[dpdk-dev] [PATCH v4 1/5] pdump: fix default socket path

2016-06-24 Thread Reshma Pattan
SOCKET_PATH_HOME is to specify environment variable "HOME", so it should not contain "/pdump_sockets" in the macro. So remove "/pdump_sockets" from SOCKET_PATH_HOME and create new macro for "/pdump_sockets". Similary removed "/pdump_sockets" from SOCKET_PATH_VAR_RUN. Changes are done in pdump_get_