[lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-06-30 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin --- example/generator/odp_generator.c | 1 - example/packet/odp_pktio.c | 1 - .../linux-generic/include/odp_packet_io_internal.h | 3 +- platform/linux-generic/odp_packet_io.c | 18 platfor

[lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-08 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin --- example/generator/odp_generator.c | 1 - example/packet/odp_pktio.c | 1 - .../linux-generic/include/odp_packet_io_internal.h | 3 +- platform/linux-generic/odp_packet_io.c | 18 platfor

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-06-30 Thread Stuart Haslam
On Tue, Jun 30, 2015 at 03:33:41PM +0200, Nicolas Morey-Chaisemartin wrote: > Signed-off-by: Nicolas Morey-Chaisemartin Reviewed-by: Stuart Haslam > --- > example/generator/odp_generator.c | 1 - > example/packet/odp_pktio.c | 1 - > .../linux-gener

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-06-30 Thread Maxim Uvarov
Why it's needed to remove basic socket? Some people use very old machines mmap and mmsg might be not supported. Maxim. On 06/30/15 17:45, Stuart Haslam wrote: On Tue, Jun 30, 2015 at 03:33:41PM +0200, Nicolas Morey-Chaisemartin wrote: Signed-off-by: Nicolas Morey-Chaisemartin Reviewed-by: S

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-06-30 Thread Stuart Haslam
On Tue, Jun 30, 2015 at 06:28:10PM +0300, Maxim Uvarov wrote: > Why it's needed to remove basic socket? Some people use very old > machines mmap and mmsg might be not supported. > > Maxim. It simplifies the implementation - this change removed 127 lines and avoided having to add another bunch. T

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-06-30 Thread Nicolas Morey-Chaisemartin
I got a little bit mixed up between the msg and mmsg version. After a quick look, we should be able to fall back to recvmsg/sendmsg for a basic socket implementation But yes, this patch removed a lot of code and a new version would be very quick to implement (mostly replacing mmsg by msg). Nico

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-01 Thread Maxim Uvarov
On 06/30/15 19:06, Nicolas Morey-Chaisemartin wrote: I got a little bit mixed up between the msg and mmsg version. After a quick look, we should be able to fall back to recvmsg/sendmsg for a basic socket implementation But yes, this patch removed a lot of code and a new version would be very qu

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-01 Thread Prashant Upadhyaya
e: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation On 06/30/15 19:06, Nicolas Morey-Chaisemartin wrote: > I got a little bit mixed up between the msg and mmsg version. > After a quick look, we should be able to fall back to recvmsg/sendmsg > for a basic

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-01 Thread Savolainen, Petri (Nokia - FI/Espoo)
uly 01, 2015 3:09 PM > To: Nicolas Morey-Chaisemartin; Stuart Haslam > Cc: lng-odp@lists.linaro.org > Subject: Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic > socket implementation > > On 06/30/15 19:06, Nicolas Morey-Chaisemartin wrote: > > I got a little b

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-01 Thread Stuart Haslam
On Wed, Jul 01, 2015 at 12:15:54PM +, Prashant Upadhyaya wrote: > Hi guys, > > I object too. Why would you want to 'remove' a functionality. > The 'basic' socket is a very good lowest common denominator of implementation > across all the machines/linux > And it was anyway a fallback mechanism

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-01 Thread Nicolas Morey-Chaisemartin
July 01, 2015 3:09 PM >> To: Nicolas Morey-Chaisemartin; Stuart Haslam >> Cc: lng-odp@lists.linaro.org >> Subject: Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic >> socket implementation >> >> On 06/30/15 19:06, Nicolas Morey-Chaisemartin wrote: &g

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-03 Thread Nicolas Morey-Chaisemartin
on't remove > the basic socket. > > Regards > -Prashant > > > -Original Message- > From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Maxim > Uvarov > Sent: Wednesday, July 01, 2015 5:39 PM > To: Nicolas Morey-Chaisemartin; Stuart Haslam &

Re: [lng-odp] [PATCHv2 1/3] linux-generic: pktio: remove basic socket implementation

2015-07-03 Thread Maxim Uvarov
ic socket. Regards -Prashant -Original Message- From: lng-odp [mailto:lng-odp-boun...@lists.linaro.org] On Behalf Of Maxim Uvarov Sent: Wednesday, July 01, 2015 5:39 PM To: Nicolas Morey-Chaisemartin; Stuart Haslam Cc: lng-odp@lists.linaro.org Subject: Re: [lng-odp] [PATCHv2 1/3] linux-generi