From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 0E123F616 for ; Thu, 12 Jan 2017 10:40:12 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga104.jf.intel.com with ESMTP; 12 Jan 2017 01:40:11 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,349,1477983600"; d="scan'208";a="921663819" Received: from shwdeisgchi083.ccr.corp.intel.com (HELO [10.239.67.193]) ([10.239.67.193]) by orsmga003.jf.intel.com with ESMTP; 12 Jan 2017 01:40:10 -0800 To: Jason Wang , dev@dpdk.org References: <1480689075-66977-1-git-send-email-jianfeng.tan@intel.com> <1482477266-39199-1-git-send-email-jianfeng.tan@intel.com> <1482477266-39199-6-git-send-email-jianfeng.tan@intel.com> <46af618f-c01b-3571-78fc-12d10859a4a1@redhat.com> <4fa4d06c-d359-df12-a073-7c2c2540b634@intel.com> <83c1bc3a-7b16-6f73-f0ba-6ed5c863eaa5@redhat.com> Cc: yuanhan.liu@linux.intel.com, ferruh.yigit@intel.com, cunming.liang@intel.com From: "Tan, Jianfeng" Message-ID: <5129c726-30fa-d4ed-a0c0-5b811772a6fb@intel.com> Date: Thu, 12 Jan 2017 17:40:10 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <83c1bc3a-7b16-6f73-f0ba-6ed5c863eaa5@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v2 5/7] net/virtio_user: add vhost kernel support X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 Jan 2017 09:40:13 -0000 Hi Jason, > Ok, just want to show its advantages. It can be added on top. > > And two more suggestions: > - better to split tap support out of vhost file Good suggestion! Will do that in next version. > - kernel support more than 8 queues on recent kernel, so there's no > need to limit it to 8. When running on old kernel, TUNSETIFF will fail > which can give user a hint to reduce #queues. > After a try, even we remove such restriction in vhost_kernel backend, we are still restricted by VIRTIO_MAX_VIRTQUEUES (8) defined in virtio PMD. Thanks for the suggestions! Jianfeng