Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-12 Thread Jan Wickbom
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan@linux.intel.com] > Sent: den 8 december 2016 06:51 > To: Jan Wickbom > Cc: dev@dpdk.org; Patrik Andersson R > Subject: Re: [PATCH] vhost: allow for many vhost user ports > > On Wed, Dec 07, 2016 at 01:23:48PM +, Jan Wickb

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-07 Thread Yuanhan Liu
On Wed, Dec 07, 2016 at 01:23:48PM +, Jan Wickbom wrote: > > On Thu, Dec 01, 2016 at 04:26:50PM +0100, Jan Wickbom wrote: > > > static int > > > -fdset_fill(fd_set *rfset, fd_set *wfset, struct fdset *pfdset) > > > +fdset_fill(struct pollfd *rwfds, struct fdset *pfdset) ... > > > + rwf

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-07 Thread Jan Wickbom
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan@linux.intel.com] > Sent: den 7 december 2016 11:13 > To: Jan Wickbom > Cc: dev@dpdk.org; Patrik Andersson R > Subject: Re: [PATCH] vhost: allow for many vhost user ports > > On Thu, Dec 01, 2016 at 04:26:50PM +0100, Jan Wickb

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-07 Thread Yuanhan Liu
On Thu, Dec 01, 2016 at 04:26:50PM +0100, Jan Wickbom wrote: > static int > -fdset_fill(fd_set *rfset, fd_set *wfset, struct fdset *pfdset) > +fdset_fill(struct pollfd *rwfds, struct fdset *pfdset) > { > struct fdentry *pfdentry; > - int i, maxfds = -1; > - int num = MAX_FDS; > - >

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-06 Thread Yuanhan Liu
On Tue, Dec 06, 2016 at 11:42:36AM +, Jan Wickbom wrote: > > > @@ -189,7 +206,7 @@ > > > pfdset->fd[i].fd = -1; > > > pfdset->fd[i].rcb = pfdset- > > >fd[i].wcb = NULL; > > > pfdset->fd[i].dat = NULL; > > > - pfdset->num--; >

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-06 Thread Jan Wickbom
> -Original Message- > From: Yuanhan Liu [mailto:yuanhan@linux.intel.com] > Sent: den 6 december 2016 07:57 > To: Jan Wickbom > Cc: huawei@intel.com; dev@dpdk.org; Patrik Andersson R > > Subject: Re: [PATCH] vhost: allow for many vhost user ports > > On Thu, Dec 01, 2016 at 04:26

Re: [dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-05 Thread Yuanhan Liu
On Thu, Dec 01, 2016 at 04:26:50PM +0100, Jan Wickbom wrote: > Currently select() is used to monitor file descriptors for vhostuser > ports. This limits the number of ports possible to create since the > fd number is used as index in the fd_set and we have seen fds > 1023. Yeah, it's a known issue

[dpdk-dev] [PATCH] vhost: allow for many vhost user ports

2016-12-01 Thread Jan Wickbom
Currently select() is used to monitor file descriptors for vhostuser ports. This limits the number of ports possible to create since the fd number is used as index in the fd_set and we have seen fds > 1023. This patch changes select() to poll(). This way we can keep an packed (pollfd) array for the