[dpdk-dev] [PATCH] doc: deprecate vhost-cuse

2016-07-21 Thread Loftus, Ciara
> Subject: [dpdk-dev] [PATCH] doc: deprecate vhost-cuse > > Vhost-cuse was invented before vhost-user exist. The both are actually > doing the same thing: a vhost-net implementation in user space. But they > are not exactly the same thing. > > Firstly, vhost-cuse is harder for use; no one seems

[dpdk-dev] [PATCH] vhost: fix missing flag reset on stop

2016-06-29 Thread Loftus, Ciara
> > Commit 550c9d27d143 ("vhost: set/reset device flags internally") moves > the VIRTIO_DEV_RUNNING set/reset to vhost lib. But I missed one reset > on stop; here fixes it. > > Fixes: 550c9d27d143 ("vhost: set/reset device flags internally") >

[dpdk-dev] [RFC] librte_vhost: Add unix domain socket fd registration

2016-06-24 Thread Loftus, Ciara
> > On Tue, Jun 21, 2016 at 09:15:03AM -0400, Aaron Conole wrote: > > Yuanhan Liu writes: > > > > > On Fri, Jun 17, 2016 at 11:32:36AM -0400, Aaron Conole wrote: > > >> Prior to this commit, the only way to add a vhost-user socket to the > > >> system is by relying on librte_vhost to open the

[dpdk-dev] [PATCH 3/6] vhost: add reconnect ability

2016-05-10 Thread Loftus, Ciara
> On Tue, May 10, 2016 at 09:00:45AM +, Xie, Huawei wrote: > > On 5/10/2016 4:42 PM, Michael S. Tsirkin wrote: > > > On Tue, May 10, 2016 at 08:07:00AM +, Xie, Huawei wrote: > > >> On 5/10/2016 3:56 PM, Michael S. Tsirkin wrote: > > >>> On Tue, May 10, 2016 at 07:24:10AM +, Xie, Huawei

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-07 Thread Loftus, Ciara
> On 4/7/2016 8:29 AM, Rich Lane wrote: > > If the vhost PMD were configured with more queues than the guest, the > old > > code would segfault in rte_vhost_enable_guest_notification due to a NULL > > virtqueue pointer. > > > > Fixes: ee584e9710b9 ("vhost: add driver on top of the library") > >

[dpdk-dev] [PATCH] vhost: call rte_vhost_enable_guest_notification only on enabled queues

2016-04-07 Thread Loftus, Ciara
> > If the vhost PMD were configured with more queues than the guest, the old > code would segfault in rte_vhost_enable_guest_notification due to a NULL > virtqueue pointer. > > Fixes: ee584e9710b9 ("vhost: add driver on top of the library") > Signed-off-by: Rich Lane > --- >

[dpdk-dev] [PATCH] vhost: Fix retrieval of numa information in PMD

2016-04-06 Thread Loftus, Ciara
> > On Wed, Apr 06, 2016 at 03:49:25PM +0900, Tetsuya Mukawa wrote: > > On 2016/04/06 1:09, Ciara Loftus wrote: > > > After some testing, it was found that retrieving numa information > > > about a vhost device via a call to get_mempolicy is more > > > accurate when performed during the

[dpdk-dev] [PATCH] vhost PMD: Fix wrong handling of maximum value of rx/tx queues

2016-03-22 Thread Loftus, Ciara
> > Currently, the maximum value of rx/tx queueus are kept by EAL. But, > the value are used like below different meanings in vhost PMD. > - The maximum value of current enabled queues. > - The maximum value of current supported queues. > > This wrong double meaning will cause an issue like

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-22 Thread Loftus, Ciara
> > On 2016/03/22 10:55, Tetsuya Mukawa wrote: > > On 2016/03/22 0:40, Loftus, Ciara wrote: > >>> + > >>> +static void > >>> +eth_dev_info(struct rte_eth_dev *dev, > >>> + struct rte_eth_dev_info *dev_info) > >>> +{ >

[dpdk-dev] [PATCH v13 2/2] vhost: Add VHOST PMD

2016-03-21 Thread Loftus, Ciara
Hi Tetsuya, Thanks for the patches. Just one query below re max queue numbers. Thanks, Ciara > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Tetsuya Mukawa > Sent: Monday, March 21, 2016 5:45 AM > To: dev at dpdk.org > Cc: Richardson, Bruce ; >

[dpdk-dev] [PATCH v2 3/3] vhost: fix vq realloc at numa_realloc

2016-03-07 Thread Loftus, Ciara
> > vq is allocated on pairs, hence we should do pair reallocation > at numa_realloc() as well, otherwise an error like following > occurs while do numa reallocation: > > VHOST_CONFIG: reallocate vq from 0 to 1 node > PANIC in rte_free(): > Fatal error: Invalid memory > > The reason

[dpdk-dev] [ovs-dev] OVS with DPDK Meetup notes

2015-12-02 Thread Loftus, Ciara
> > > > On Thu, Nov 26, 2015 at 05:56:08PM +, Traynor, Kevin wrote: > > > Hi All, > > > > > > Just wanted to post some summary notes on the recent OVS with DPDK > Meetup > > we > > > had after the OVS conference. Thanks to everyone for the often lively > > discussion. > > > I've collated and

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

2015-10-30 Thread Loftus, Ciara
> > These variables are needed to be able to manage one of virtio devices > using both vhost library APIs and vhost PMD. > For example, if vhost PMD uses current callback handler and private data > provided by vhost library, A DPDK application that links vhost library > cannot use some of vhost

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-10-20 Thread Loftus, Ciara
> > On 2015/09/24 2:47, Loftus, Ciara wrote: > >> The patch introduces a new PMD. This PMD is implemented as thin > wrapper > >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. > >> The PMD can have 'iface' parameter like

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-10-19 Thread Loftus, Ciara
> On 2015/10/16 21:52, Bruce Richardson wrote: > > On Mon, Aug 31, 2015 at 12:55:26PM +0900, Tetsuya Mukawa wrote: > >> The patch introduces a new PMD. This PMD is implemented as thin > wrapper > >> of librte_vhost. It means librte_vhost is also needed to compile the PMD. > >> The PMD can have

[dpdk-dev] [RFC PATCH v2] vhost: Add VHOST PMD

2015-09-23 Thread Loftus, Ciara
> The patch introduces a new PMD. This PMD is implemented as thin wrapper > of librte_vhost. It means librte_vhost is also needed to compile the PMD. > The PMD can have 'iface' parameter like below to specify a path to connect > to a virtio-net device. > > $ ./testpmd -c f -n 4 --vdev

[dpdk-dev] [RFC PATCH] vhost: Add VHOST PMD

2015-09-17 Thread Loftus, Ciara
> > On 2015/09/16 1:27, Loftus, Ciara wrote: > >> + > >> +static int > >> +rte_pmd_vhost_devinit(const char *name, const char *params) > >> +{ > >> + struct rte_kvargs *kvlist = NULL; > >> + int ret = 0; > >> +

[dpdk-dev] [RFC PATCH] vhost: Add VHOST PMD

2015-09-15 Thread Loftus, Ciara
> + > +static int > +rte_pmd_vhost_devinit(const char *name, const char *params) > +{ > + struct rte_kvargs *kvlist = NULL; > + int ret = 0; > + int index; > + char *iface_name; > + > + RTE_LOG(INFO, PMD, "Initializing pmd_vhost for %s\n", name); > + > + kvlist =

[dpdk-dev] [PATCH v2] vhost: provide vhost API to unregister vhost unix domain socket

2015-06-05 Thread Loftus, Ciara
> -Original Message- > From: Xie, Huawei > Sent: Friday, June 05, 2015 4:26 AM > To: dev at dpdk.org > Cc: Loftus, Ciara; Xie, Huawei; Sun, Peng A > Subject: [PATCH v2] vhost: provide vhost API to unregister vhost unix domain > socket > > rte_vhost_dr

[dpdk-dev] [PATCH] vhost: provide vhost API to unregister vhost unix domain socket

2015-06-03 Thread Loftus, Ciara
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Huawei Xie > Sent: Tuesday, June 02, 2015 2:50 AM > To: dev at dpdk.org > Cc: Sun, Peng A > Subject: [dpdk-dev] [PATCH] vhost: provide vhost API to unregister vhost > unix domain socket > >