[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2016-01-06 Thread Yuanhan Liu
On Wed, Jan 06, 2016 at 12:56:58PM +0900, Tetsuya Mukawa wrote: > On 2015/12/29 6:59, Rich Lane wrote: > > On Wed, Dec 23, 2015 at 11:58 PM, Tetsuya Mukawa > > wrote: > >> Hi Rich and Yuanhan, > >> > >> I guess we have 2 implementations here. > >> > >> 1. rte_eth_vhost_get_queue_event() returns e

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2016-01-06 Thread Tetsuya Mukawa
On 2015/12/29 6:59, Rich Lane wrote: > On Wed, Dec 23, 2015 at 11:58 PM, Tetsuya Mukawa wrote: >> Hi Rich and Yuanhan, >> >> I guess we have 2 implementations here. >> >> 1. rte_eth_vhost_get_queue_event() returns each event. >> 2. rte_eth_vhost_get_queue_status() returns current status of the que

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-28 Thread Rich Lane
On Wed, Dec 23, 2015 at 11:58 PM, Tetsuya Mukawa wrote: > > Hi Rich and Yuanhan, > > I guess we have 2 implementations here. > > 1. rte_eth_vhost_get_queue_event() returns each event. > 2. rte_eth_vhost_get_queue_status() returns current status of the queues. > > I guess option "2" is more generic

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Tetsuya Mukawa
On 2015/12/24 14:37, Rich Lane wrote: > On Wed, Dec 23, 2015 at 7:09 PM, Tetsuya Mukawa wrote: > >> On 2015/12/22 13:47, Rich Lane wrote: >>> On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu < >> yuanhan.liu at linux.intel.com> >>> wrote: >>> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wr

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Tetsuya Mukawa
On 2015/12/24 13:00, Yuanhan Liu wrote: > On Thu, Dec 24, 2015 at 12:09:10PM +0900, Tetsuya Mukawa wrote: >> On 2015/12/22 13:47, Rich Lane wrote: >>> On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu >> linux.intel.com> >>> wrote: >>> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote:

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Tetsuya Mukawa
On 2015/12/24 12:51, Yuanhan Liu wrote: > On Wed, Dec 23, 2015 at 11:00:15PM +0100, Thomas Monjalon wrote: >> 2015-12-23 10:44, Yuanhan Liu: >>> On Tue, Dec 22, 2015 at 01:38:29AM -0800, Rich Lane wrote: On Mon, Dec 21, 2015 at 9:47 PM, Yuanhan Liu >>> linux.intel.com> wrote:

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Tetsuya Mukawa
On 2015/12/24 12:09, Tetsuya Mukawa wrote: > On 2015/12/22 13:47, Rich Lane wrote: >> On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu >> wrote: >> >>> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: I'm using the vhost callbacks and struct virtio_net with the vhost PMD >>> in a few >>

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Tetsuya Mukawa
On 2015/12/22 13:47, Rich Lane wrote: > On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu > wrote: > >> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: >>> I'm using the vhost callbacks and struct virtio_net with the vhost PMD >> in a few >>> ways: >> Rich, thanks for the info! >> >>> 1. new

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Yuanhan Liu
On Thu, Dec 24, 2015 at 12:09:10PM +0900, Tetsuya Mukawa wrote: > On 2015/12/22 13:47, Rich Lane wrote: > > On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu > linux.intel.com> > > wrote: > > > >> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: > >>> I'm using the vhost callbacks and struct

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-24 Thread Yuanhan Liu
On Wed, Dec 23, 2015 at 11:00:15PM +0100, Thomas Monjalon wrote: > 2015-12-23 10:44, Yuanhan Liu: > > On Tue, Dec 22, 2015 at 01:38:29AM -0800, Rich Lane wrote: > > > On Mon, Dec 21, 2015 at 9:47 PM, Yuanhan Liu > > linux.intel.com> > > > wrote: > > > > > > On Mon, Dec 21, 2015 at 08:47:28PM

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-23 Thread Thomas Monjalon
2015-12-23 10:44, Yuanhan Liu: > On Tue, Dec 22, 2015 at 01:38:29AM -0800, Rich Lane wrote: > > On Mon, Dec 21, 2015 at 9:47 PM, Yuanhan Liu > linux.intel.com> > > wrote: > > > > On Mon, Dec 21, 2015 at 08:47:28PM -0800, Rich Lane wrote: > > > The queue state change callback is the one ne

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-23 Thread Rich Lane
On Wed, Dec 23, 2015 at 7:09 PM, Tetsuya Mukawa wrote: > On 2015/12/22 13:47, Rich Lane wrote: > > On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu < > yuanhan.liu at linux.intel.com> > > wrote: > > > >> On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: > >>> I'm using the vhost callbacks an

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-23 Thread Yuanhan Liu
On Tue, Dec 22, 2015 at 01:38:29AM -0800, Rich Lane wrote: > On Mon, Dec 21, 2015 at 9:47 PM, Yuanhan Liu > wrote: > > On Mon, Dec 21, 2015 at 08:47:28PM -0800, Rich Lane wrote: > > The queue state change callback is the one new API that needs to be > > added because > > normal NI

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-22 Thread Yuanhan Liu
On Mon, Dec 21, 2015 at 08:47:28PM -0800, Rich Lane wrote: > On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu > wrote: > > On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: > > I'm using the vhost callbacks and struct virtio_net with the vhost PMD > in > a few > > ways: > >

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-22 Thread Yuanhan Liu
On Mon, Dec 21, 2015 at 11:10:10AM +0900, Tetsuya Mukawa wrote: > nes: 168 > > On 2015/12/19 3:01, Rich Lane wrote: > > I'm using the vhost callbacks and struct virtio_net with the vhost PMD in a > > few ways: > > > > 1. new_device/destroy_device: Link state change (will be covered by the > > link

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-22 Thread Yuanhan Liu
On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: > I'm using the vhost callbacks and struct virtio_net with the vhost PMD in a > few > ways: Rich, thanks for the info! > > 1. new_device/destroy_device: Link state change (will be covered by the link > status interrupt). > 2. new_device

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-22 Thread Rich Lane
On Mon, Dec 21, 2015 at 9:47 PM, Yuanhan Liu wrote: > On Mon, Dec 21, 2015 at 08:47:28PM -0800, Rich Lane wrote: > > The queue state change callback is the one new API that needs to be > > added because > > normal NICs don't have this behavior. > > Again I'd ask, will vring_state_changed() be eno

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-21 Thread Rich Lane
On Mon, Dec 21, 2015 at 7:41 PM, Yuanhan Liu wrote: > On Fri, Dec 18, 2015 at 10:01:25AM -0800, Rich Lane wrote: > > I'm using the vhost callbacks and struct virtio_net with the vhost PMD > in a few > > ways: > > Rich, thanks for the info! > > > > > 1. new_device/destroy_device: Link state change

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-21 Thread Tetsuya Mukawa
On 2015/12/19 3:01, Rich Lane wrote: > I'm using the vhost callbacks and struct virtio_net with the vhost PMD in a > few ways: > > 1. new_device/destroy_device: Link state change (will be covered by the > link status interrupt). > 2. new_device: Add first queue to datapath. > 3. vring_state_changed

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-21 Thread Tetsuya Mukawa
On 2015/12/18 19:03, Xie, Huawei wrote: > On 12/18/2015 12:15 PM, Yuanhan Liu wrote: >> On Fri, Dec 18, 2015 at 12:15:42PM +0900, Tetsuya Mukawa wrote: >>> On 2015/12/17 20:42, Yuanhan Liu wrote: On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: > The vhost PMD will be a wrap

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Tetsuya Mukawa
On 2015/12/18 13:15, Yuanhan Liu wrote: > On Fri, Dec 18, 2015 at 12:15:42PM +0900, Tetsuya Mukawa wrote: >> On 2015/12/17 20:42, Yuanhan Liu wrote: >>> On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: The vhost PMD will be a wrapper of vhost library, but some of vhost libr

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Tetsuya Mukawa
On 2015/12/18 12:15, Tetsuya Mukawa wrote: > On 2015/12/17 20:42, Yuanhan Liu wrote: >> On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: >>> The vhost PMD will be a wrapper of vhost library, but some of vhost >>> library APIs cannot be mapped to ethdev library APIs. >>> Becasue of th

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Tetsuya Mukawa
On 2015/12/17 20:42, Yuanhan Liu wrote: > On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: >> The vhost PMD will be a wrapper of vhost library, but some of vhost >> library APIs cannot be mapped to ethdev library APIs. >> Becasue of this, in some cases, we still need to use vhost lib

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Yuanhan Liu
On Fri, Dec 18, 2015 at 12:15:42PM +0900, Tetsuya Mukawa wrote: > On 2015/12/17 20:42, Yuanhan Liu wrote: > > On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: > >> The vhost PMD will be a wrapper of vhost library, but some of vhost > >> library APIs cannot be mapped to ethdev library

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Xie, Huawei
On 12/18/2015 12:15 PM, Yuanhan Liu wrote: > On Fri, Dec 18, 2015 at 12:15:42PM +0900, Tetsuya Mukawa wrote: >> On 2015/12/17 20:42, Yuanhan Liu wrote: >>> On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: The vhost PMD will be a wrapper of vhost library, but some of vhost l

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-18 Thread Rich Lane
I'm using the vhost callbacks and struct virtio_net with the vhost PMD in a few ways: 1. new_device/destroy_device: Link state change (will be covered by the link status interrupt). 2. new_device: Add first queue to datapath. 3. vring_state_changed: Add/remove queue to datapath. 4. destroy_device:

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-12-17 Thread Yuanhan Liu
On Tue, Nov 24, 2015 at 06:00:01PM +0900, Tetsuya Mukawa wrote: > The vhost PMD will be a wrapper of vhost library, but some of vhost > library APIs cannot be mapped to ethdev library APIs. > Becasue of this, in some cases, we still need to use vhost library APIs > for a port created by the vhost P

[dpdk-dev] [PATCH v5 1/3] vhost: Add callback and private data for vhost PMD

2015-11-24 Thread Tetsuya Mukawa
The vhost PMD will be a wrapper of vhost library, but some of vhost library APIs cannot be mapped to ethdev library APIs. Becasue of this, in some cases, we still need to use vhost library APIs for a port created by the vhost PMD. Currently, when virtio device is created and destroyed, vhost libra