From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v5 4/5] vhost: eventfd_link: replace copy-pasted sys_close Date: Thu, 09 Jul 2015 02:59:38 +0200 Message-ID: <2113341.vY7no2aSVm@xps13> References: <1427994080-10163-1-git-send-email-pboldin@mirantis.com> <1665929.SfCu3BinAE@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Pavel Boldin To: "Xie, Huawei" , changchun.ouyang@intel.com Return-path: Received: from mail-wi0-f170.google.com (mail-wi0-f170.google.com [209.85.212.170]) by dpdk.org (Postfix) with ESMTP id 5C60C9E7 for ; Thu, 9 Jul 2015 03:00:44 +0200 (CEST) Received: by wiwl6 with SMTP id l6so4021347wiw.0 for ; Wed, 08 Jul 2015 18:00:44 -0700 (PDT) In-Reply-To: <1665929.SfCu3BinAE@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Ping 2015-06-17 17:24, Thomas Monjalon: > 2015-05-07 06:54, Xie, Huawei: > > On 4/16/2015 7:48 PM, Pavel Boldin wrote: > > > + /* Closing the source_fd */ > > > + ret = sys_close(eventfd_copy.source_fd); > > Pavel: > > Here we close the fd and re-install a new file on this fd later. > > sys_close does all cleanup. > > But, for instance, if we allocate new fd later, normally it will reuse > > the just freed fds by sys_close, is there issue here? > > Pavel, Huawei, > Could we come to a conclusion on this patch series please?