From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from szxga02-in.huawei.com (szxga02-in.huawei.com [119.145.14.65]) by dpdk.org (Postfix) with ESMTP id 2EA9F5ACA for ; Fri, 23 Jan 2015 04:53:36 +0100 (CET) Received: from 172.24.2.119 (EHLO szxeml427-hub.china.huawei.com) ([172.24.2.119]) by szxrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id CGD99536; Fri, 23 Jan 2015 11:53:33 +0800 (CST) Received: from [127.0.0.1] (10.177.19.115) by szxeml427-hub.china.huawei.com (10.82.67.182) with Microsoft SMTP Server id 14.3.158.1; Fri, 23 Jan 2015 11:53:32 +0800 Message-ID: <54C1C5BA.8040801@huawei.com> Date: Fri, 23 Jan 2015 11:53:30 +0800 From: Linhaifeng User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 MIME-Version: 1.0 To: "Xie, Huawei" , "dev@dpdk.org" References: <1418247477-13920-1-git-send-email-huawei.xie@intel.com> <1418247477-13920-9-git-send-email-huawei.xie@intel.com> <54892D4D.3060708@huawei.com> In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.177.19.115] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH RFC v2 08/12] lib/librte_vhost: vhost-user support X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 23 Jan 2015 03:53:39 -0000 On 2015/1/23 11:40, Xie, Huawei wrote: > > >> -----Original Message----- >> From: Linhaifeng [mailto:haifeng.lin@huawei.com] >> Sent: Thursday, December 11, 2014 1:36 PM >> To: Xie, Huawei; dev@dpdk.org >> Cc: haifeng.lin@intel.com >> Subject: Re: [dpdk-dev] [PATCH RFC v2 08/12] lib/librte_vhost: vhost-user >> support >> >> >> >> On 2014/12/11 5:37, Huawei Xie wrote: >>> vhost-user support >>> >>> >>> Signed-off-by: Huawei Xie >>> --- >>> lib/librte_vhost/Makefile | 5 +- >>> + case VHOST_USER_SET_LOG_FD: >> >> should close fd for fd leak when receive VHOST_USER_SET_LOG_FD msg? > Ok, would at least close the fd though we don't support. > Do you know how to test this? reboot VM many times then watch the number of fd with command 'ls /proc/pidof vhost/fd |wc -l' >> >>> + RTE_LOG(INFO, VHOST_CONFIG, "not implemented.\n"); >>> + >>> + case VHOST_USER_SET_VRING_ERR: >> >> should close fd for fd leak? > Would do. >> >>> + RTE_LOG(INFO, VHOST_CONFIG, "not implemented\n"); >>> + break; >>> +#endif >>> >> >> -- >> Regards, >> Haifeng > > > -- Regards, Haifeng