DPDK patches and discussions
 help / color / mirror / Atom feed
From: Patrik Andersson R <patrik.r.andersson@ericsson.com>
To: "Xie, Huawei" <huawei.xie@intel.com>,
	Christian Ehrhardt <christian.ehrhardt@canonical.com>,
	Daniele Di Proietto <diproiettod@vmware.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
	Thomas Monjalon <thomas.monjalon@6wind.com>,
	"Ananyev, Konstantin" <konstantin.ananyev@intel.com>,
	Yuanhan Liu <yuanhan.liu@linux.intel.com>
Subject: Re: [dpdk-dev] [RFC] vhost user: add error handling for fd > 1023
Date: Mon, 11 Apr 2016 08:06:06 +0200	[thread overview]
Message-ID: <570B3ECE.3050905@ericsson.com> (raw)
In-Reply-To: <C37D651A908B024F974696C65296B57B4C6D90F2@SHSMSX101.ccr.corp.intel.com>

I fully agree with this course of action.

Thank you,

Patrik


On 04/08/2016 08:47 AM, Xie, Huawei wrote:
> On 4/7/2016 10:52 PM, Christian Ehrhardt wrote:
>> I totally agree to that there is no deterministic rule what to expect.
>> The only rule is that #fd certainly always is > #vhost_user devices.
>> In various setup variants I've crossed fd 1024 anywhere between 475
>> and 970 vhost_user ports.
>>
>> Once the discussion continues and we have an updates version of the
>> patch with some more agreement I hope I can help to test it.
> Thanks. Let us first temporarily fix this problem for robustness, then
> we consider whether upgrade to (e)poll.
> Will check the patch in detail later. Basically it should work but need
> check whether we need extra fixes elsewhere.

  reply	other threads:[~2016-04-11  6:06 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-18  9:13 Patrik Andersson
2016-03-30  9:05 ` Xie, Huawei
2016-04-05  8:40   ` Patrik Andersson R
2016-04-07 14:49     ` Christian Ehrhardt
2016-04-08  6:47       ` Xie, Huawei
2016-04-11  6:06         ` Patrik Andersson R [this message]
2016-04-11  9:34           ` Christian Ehrhardt
2016-04-11 11:21             ` Patrik Andersson R

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=570B3ECE.3050905@ericsson.com \
    --to=patrik.r.andersson@ericsson.com \
    --cc=christian.ehrhardt@canonical.com \
    --cc=dev@dpdk.org \
    --cc=diproiettod@vmware.com \
    --cc=huawei.xie@intel.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=thomas.monjalon@6wind.com \
    --cc=yuanhan.liu@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).