Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-30 Thread Dmitry Kozlyuk
On Wed, 30 Sep 2020 09:11:41 +0100, Kinsella, Ray wrote: > On 05/08/2020 12:17, Dmitry Kozlyuk wrote: > > On Wed, 5 Aug 2020 10:31:31 +0100, Kinsella, Ray wrote: > >> On 30/07/2020 22:06, Dmitry Kozlyuk wrote: > >>> struct cmdline exposes platform-specific members it contains, most > >>> notabl

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-30 Thread Kinsella, Ray
On 05/08/2020 12:17, Dmitry Kozlyuk wrote: > On Wed, 5 Aug 2020 10:31:31 +0100, Kinsella, Ray wrote: >> On 30/07/2020 22:06, Dmitry Kozlyuk wrote: >>> struct cmdline exposes platform-specific members it contains, most >>> notably struct termios that is only available on Unix. Make the >>> struct

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-18 Thread Kinsella, Ray
On 17/09/2020 14:34, Olivier Matz wrote: > Hi Dmitry, > > On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: >> struct cmdline exposes platform-specific members it contains, most >> notably struct termios that is only available on Unix. Make the >> structure opaque. >> >> Remove te

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-18 Thread Kinsella, Ray
On 17/09/2020 18:05, Stephen Hemminger wrote: > On Thu, 17 Sep 2020 15:34:43 +0200 > Olivier Matz wrote: > >> Hi Dmitry, >> >> On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: >>> struct cmdline exposes platform-specific members it contains, most >>> notably struct termios that

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-18 Thread Ferruh Yigit
On 9/18/2020 9:33 AM, Bruce Richardson wrote: On Thu, Sep 17, 2020 at 10:05:48AM -0700, Stephen Hemminger wrote: On Thu, 17 Sep 2020 15:34:43 +0200 Olivier Matz wrote: Hi Dmitry, On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: struct cmdline exposes platform-specific members

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-18 Thread Bruce Richardson
On Thu, Sep 17, 2020 at 10:05:48AM -0700, Stephen Hemminger wrote: > On Thu, 17 Sep 2020 15:34:43 +0200 > Olivier Matz wrote: > > > Hi Dmitry, > > > > On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: > > > struct cmdline exposes platform-specific members it contains, most > > > no

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-17 Thread Dmitry Kozlyuk
Hi Olivier, thanks for the review. > In my opinion, we could accept the patch for 20.11, knowing it reduce > the risk of future ABI breakage, and that cmdline is not a core > component of DPDK. However it has to be discussed and accepted by other > people. > > Else, the patch would be delayed for

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-17 Thread Stephen Hemminger
On Thu, 17 Sep 2020 15:34:43 +0200 Olivier Matz wrote: > Hi Dmitry, > > On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: > > struct cmdline exposes platform-specific members it contains, most > > notably struct termios that is only available on Unix. Make the > > structure opaque.

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-09-17 Thread Olivier Matz
Hi Dmitry, On Fri, Jul 31, 2020 at 12:06:45AM +0300, Dmitry Kozlyuk wrote: > struct cmdline exposes platform-specific members it contains, most > notably struct termios that is only available on Unix. Make the > structure opaque. > > Remove tests checking struct cmdline content as meaningless. >

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-08-05 Thread Dmitry Kozlyuk
On Wed, 5 Aug 2020 10:31:31 +0100, Kinsella, Ray wrote: > On 30/07/2020 22:06, Dmitry Kozlyuk wrote: > > struct cmdline exposes platform-specific members it contains, most > > notably struct termios that is only available on Unix. Make the > > structure opaque. > > > > Remove tests checking struct

Re: [dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-08-05 Thread Kinsella, Ray
On 30/07/2020 22:06, Dmitry Kozlyuk wrote: > struct cmdline exposes platform-specific members it contains, most > notably struct termios that is only available on Unix. Make the > structure opaque. > > Remove tests checking struct cmdline content as meaningless. > > Add cmdline_get_rdline() to

[dpdk-dev] [PATCH v2 1/7] cmdline: make implementation opaque

2020-07-30 Thread Dmitry Kozlyuk
struct cmdline exposes platform-specific members it contains, most notably struct termios that is only available on Unix. Make the structure opaque. Remove tests checking struct cmdline content as meaningless. Add cmdline_get_rdline() to access history buffer. The new function is currently used o