From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v8 08/14] eal/linux/pci: Add functions for unmapping igb_uio resources Date: Wed, 18 Feb 2015 02:09:38 +0100 Message-ID: <3062823.xy5khOG9OG@xps13> References: <1423470639-15744-2-git-send-email-mukawa@igel.co.jp> <2770102.I27SuxhSqW@xps13> <54E2DC82.8070901@igel.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Tetsuya Mukawa Return-path: In-Reply-To: <54E2DC82.8070901-AlSX/UN32fvPDbFq/vQRIQ@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" 2015-02-17 15:15, Tetsuya Mukawa: > On 2015/02/17 10:11, Thomas Monjalon wrote: > > 2015-02-16 13:14, Tetsuya Mukawa: > >> +#ifdef ENABLE_HOTPLUG > > Please avoid using #ifdef if not really necessary. > > I agree with you. > In this case, only hotplug functions call pci_unmap_resource(). > So this will be needed. Why is it needed?