From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3] Restore support for virtio on FreeBSD Date: Mon, 13 Apr 2015 14:54:42 +0200 Message-ID: <2279951.2s8DnS74SK@xps13> References: <1428450303-97954-1-git-send-email-razamir22@gmail.com> <1428927569-98070-1-git-send-email-razamir22@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Raz Amir Return-path: In-Reply-To: <1428927569-98070-1-git-send-email-razamir22-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Please provide more information in the commit message. We need to know what was the problem (crash) in the git history. Then when doing git blame, we'll have the full explanation. 2015-04-13 15:19, Raz Amir: > Fixes: 8a312224bcde ("eal/bsd: fix fd leak") > > Signed-off-by: Raz Amir > --- > lib/librte_eal/bsdapp/eal/eal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c > index 871d5f4..e20f915 100644 > --- a/lib/librte_eal/bsdapp/eal/eal.c > +++ b/lib/librte_eal/bsdapp/eal/eal.c > @@ -426,7 +426,7 @@ rte_eal_iopl_init(void) > fd = open("/dev/io", O_RDWR); > if (fd < 0) > return -1; > - close(fd); > + /* keep fd open for iopl */ Yes we need a comment but "for iopl" is not descriptive and not very accurate as iopl is a Linux mechanism. > return 0; > } Thanks