From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: Re: [PATCH v7 09/12] eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp Date: Thu, 2 Jul 2015 12:11:09 +0100 Message-ID: <20150702111109.GD3828@bricha3-MOBL3> References: <1435306705-11645-4-git-send-email-mukawa@igel.co.jp> <1435652668-3380-1-git-send-email-mukawa@igel.co.jp> <1435652668-3380-10-git-send-email-mukawa@igel.co.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: dev@dpdk.org To: Tetsuya Mukawa Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 5DD4411F5 for ; Thu, 2 Jul 2015 13:11:15 +0200 (CEST) Content-Disposition: inline In-Reply-To: <1435652668-3380-10-git-send-email-mukawa@igel.co.jp> 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" On Tue, Jun 30, 2015 at 05:24:25PM +0900, Tetsuya Mukawa wrote: > From: "Tetsuya.Mukawa" > > The patch consolidates below functions, and implemented in common > eal code. > - pci_map_resource() > - pci_unmap_resource() > > Signed-off-by: Tetsuya Mukawa > --- > lib/librte_eal/bsdapp/eal/eal_pci.c | 22 ---------------- > lib/librte_eal/common/eal_common_pci.c | 39 ++++++++++++++++++++++++++++ > lib/librte_eal/common/include/rte_pci.h | 11 ++++++++ > lib/librte_eal/linuxapp/eal/eal_pci.c | 41 ------------------------------ > lib/librte_eal/linuxapp/eal/eal_pci_init.h | 5 ---- > 5 files changed, 50 insertions(+), 68 deletions(-) > > diff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h > index 0a2ef09..56dcb46 100644 > --- a/lib/librte_eal/common/include/rte_pci.h > +++ b/lib/librte_eal/common/include/rte_pci.h > @@ -364,6 +364,17 @@ int rte_eal_pci_scan(void); > */ > int rte_eal_pci_probe(void); > > +/** > + * Map pci resouce. > + */ > +void *pci_map_resource(void *requested_addr, int fd, off_t offset, > + size_t size, int additional_flags); > + > +/** > + * Map pci resouce. > + */ > +void pci_unmap_resource(void *requested_addr, size_t size); > + These functions should probably be marked "@internal", right?