[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-14 Thread Thomas Monjalon
Neil, John, I think we all want the same thing: simple and generic APIs. 2014-04-14 10:10, John W. Linville: > On Mon, Apr 14, 2014 at 03:45:31PM +0200, Thomas Monjalon wrote: > > From the user's point of view, it must be possible to create some virtual > > devices instead of using real ones. Tha

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-14 Thread Thomas Monjalon
Hi John, 2014-04-14 09:20, John W. Linville: > On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > > 11/04/2014 20:08, Richardson, Bruce : > > > The ring PMD is probably best treated separately from the other PMDs as > > > it's not really a device poll-mode driver. Instead, it's a g

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-14 Thread John W. Linville
On Mon, Apr 14, 2014 at 03:45:31PM +0200, Thomas Monjalon wrote: > Hi John, > > 2014-04-14 09:20, John W. Linville: > > On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > > > 11/04/2014 20:08, Richardson, Bruce : > > > > The ring PMD is probably best treated separately from the oth

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-14 Thread Neil Horman
On Mon, Apr 14, 2014 at 03:45:31PM +0200, Thomas Monjalon wrote: > Hi John, > > 2014-04-14 09:20, John W. Linville: > > On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > > > 11/04/2014 20:08, Richardson, Bruce : > > > > The ring PMD is probably best treated separately from the oth

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-14 Thread John W. Linville
On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > Hi Bruce, > > 11/04/2014 20:08, Richardson, Bruce : > > From: Neil Horman > > > On Fri, Apr 11, 2014 at 06:18:08PM +0200, Thomas Monjalon wrote: > > > > It seems that your patch is not removing > > > > rte_eth_ring_pair_create/rte_

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-12 Thread Richardson, Bruce
> -Original Message- > From: Neil Horman [mailto:nhorman at tuxdriver.com] > Sent: Saturday, April 12, 2014 12:03 PM > To: Thomas Monjalon > Cc: Richardson, Bruce; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API > > On Sa

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-12 Thread Neil Horman
dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API > > > > On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > > > Hi Bruce, > > > > > > 11/04/2014 20:08, Richardson, Bruce : > > > >

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-12 Thread Thomas Monjalon
Hi Bruce, 11/04/2014 20:08, Richardson, Bruce : > From: Neil Horman > > On Fri, Apr 11, 2014 at 06:18:08PM +0200, Thomas Monjalon wrote: > > > It seems that your patch is not removing > > > rte_eth_ring_pair_create/rte_eth_ring_pair_attach so I'm not sure you > > > can dynamically change the PMD i

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-12 Thread Neil Horman
On Sat, Apr 12, 2014 at 08:05:22AM +0200, Thomas Monjalon wrote: > Hi Bruce, > > 11/04/2014 20:08, Richardson, Bruce : > > From: Neil Horman > > > On Fri, Apr 11, 2014 at 06:18:08PM +0200, Thomas Monjalon wrote: > > > > It seems that your patch is not removing > > > > rte_eth_ring_pair_create/rte_

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Richardson, Bruce
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman > Sent: Friday, April 11, 2014 6:45 PM > To: Thomas Monjalon > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API > > On Fri, Ap

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Thomas Monjalon
2014-04-11 11:50, Neil Horman: > There are still problems with the libraries after Oliviers patch series: > > [nhorman at hmsreliant lib]$ ldd librte_pmd_pcap.so > statically linked > > The DSO's built by dpdk are not actually build as shared libraries but > rather statically linked toget

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Thomas Monjalon
2014-04-11 09:36, Olivier Matz: > Instead of having a list of virtual device drivers in EAL code, add an > API to register drivers. Thanks to this new registration method, we can > remove the references to pmd_ring, pmd_pcap and pmd_xenvirt in EAL code. > This also enables the ability to register a

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Thomas Monjalon
Hi Neil, 2014-04-11 06:49, Neil Horman: > On Fri, Apr 11, 2014 at 09:36:52AM +0200, Olivier Matz wrote: > > Instead of having a list of virtual device drivers in EAL code, add an > > API to register drivers. Thanks to this new registration method, we can > > remove the references to pmd_ring, pmd_

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Neil Horman
On Fri, Apr 11, 2014 at 06:18:08PM +0200, Thomas Monjalon wrote: > 2014-04-11 11:50, Neil Horman: > > There are still problems with the libraries after Oliviers patch series: > > > > [nhorman at hmsreliant lib]$ ldd librte_pmd_pcap.so > > statically linked > > > > The DSO's built by dpdk

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Neil Horman
On Fri, Apr 11, 2014 at 03:11:56PM +0200, Thomas Monjalon wrote: > Hi Neil, > > 2014-04-11 06:49, Neil Horman: > > On Fri, Apr 11, 2014 at 09:36:52AM +0200, Olivier Matz wrote: > > > Instead of having a list of virtual device drivers in EAL code, add an > > > API to register drivers. Thanks to thi

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Olivier Matz
Instead of having a list of virtual device drivers in EAL code, add an API to register drivers. Thanks to this new registration method, we can remove the references to pmd_ring, pmd_pcap and pmd_xenvirt in EAL code. This also enables the ability to register a virtual device driver as a shared libra

[dpdk-dev] [PATCH v2 07/11 1/2] vdev: new registration API

2014-04-11 Thread Neil Horman
On Fri, Apr 11, 2014 at 09:36:52AM +0200, Olivier Matz wrote: > Instead of having a list of virtual device drivers in EAL code, add an > API to register drivers. Thanks to this new registration method, we can > remove the references to pmd_ring, pmd_pcap and pmd_xenvirt in EAL code. > This also ena