From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f179.google.com (mail-pd0-f179.google.com [209.85.192.179]) by dpdk.org (Postfix) with ESMTP id 390615A7A for ; Tue, 30 Jun 2015 10:25:07 +0200 (CEST) Received: by pdbci14 with SMTP id ci14so2142371pdb.2 for ; Tue, 30 Jun 2015 01:25:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=N+pW3MHuU45voHJIgXNcSqJP72Kbrex9VqiE+R8MMJo=; b=YnZ7mCOMHfnuj19D4m3O/mM9RPqpQRwdMac6rzz75HmLuVjRaCM4YilTdSl6582I+P hWKCl8PvsO5lV193LZrSlWfDptR7Lvn7UBL78Syd/tZU90Xx4Rixr+IHhdssZ/HZEpm1 R/W90KCaU1BbnX+3jsFSNcm2d7dw5xnTXg25+0jhivIORcAAKoeuE732U0TRD8DQdBsQ VJEJ3pADbvX5sIbfrgi+swMORluhYVzmNQ/K6CEN8HoAuGbcg4VUKgxIFMt7PXovdaI7 60GY867NYyg8Qa2ZYs/s8GvsB+vEsNVIri8Fo+vOAqBPD4/GayISRC0cjmA7ZxXf5t5P Tr6Q== X-Gm-Message-State: ALoCoQmrO/KHPPJBK0jg4I95hLS3xeeCzAealJBFPN359mnH7Oxqco7NBeeeKqnQ/xE9xbzQFPH6 X-Received: by 10.70.129.143 with SMTP id nw15mr40560890pdb.101.1435652706612; Tue, 30 Jun 2015 01:25:06 -0700 (PDT) Received: from localhost.localdomain (napt.igel.co.jp. [219.106.231.132]) by mx.google.com with ESMTPSA id k9sm44733339pdp.60.2015.06.30.01.25.04 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 30 Jun 2015 01:25:05 -0700 (PDT) From: Tetsuya Mukawa To: dev@dpdk.org Date: Tue, 30 Jun 2015 17:24:16 +0900 Message-Id: <1435652668-3380-1-git-send-email-mukawa@igel.co.jp> X-Mailer: git-send-email 2.1.4 In-Reply-To: <1435306705-11645-4-git-send-email-mukawa@igel.co.jp> References: <1435306705-11645-4-git-send-email-mukawa@igel.co.jp> Subject: [dpdk-dev] [PATCH v7 00/12] Clean up pci uio implementations 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: Tue, 30 Jun 2015 08:25:07 -0000 Currently Linux implementation and BSD implementation have almost same code about pci uio. This patch series cleans up it. PATCH v7 changes: - Add below patches. Also, the order of patches are changed. - eal: Add pci_uio_alloc_resource() - eal: Add pci_uio_map_resource_by_index() - eal: Consolidate pci_map and mapped_pci_resource of linuxapp and bsdapp - eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp - eal: Consolidate pci uio functions of linuxapp and bsdapp - eal: Consolidate pci_map/unmap_device() of linuxapp and bsdapp - eal: Consolidate rte_eal_pci_probe/close_one_driver() of linuxapp and bsdapp (Thanks to Bruce Richardson) - While adding above, below patches are not changed at all. - eal: Fix coding style of eal_pci.c and eal_pci_uio.c - eal: Close file descriptor of uio configuration - eal: Fix memory leaks and needless increment of pci_map_addr - eal/bsdapp: Change names of pci related data structure - eal: Fix uio mapping differences between linuxapp and bsdapp - some function names are changed like below. - pci_uio_alloc_uio_resource() to pci_uio_alloc_resource(). - pci_uio_map_uio_resource_by_index() to pci_uio_map_resource_by_index(). (Thanks to Iremonger, Bernard) PATCH v6 changes: - Free mapped resources in pci_uio_map_resource(). - Fix error handling in pci_uio_map_resource(). (Thanks to David, Marchand) PATCH v5 changes: - Rebase to latest master branch. PATCH v4 changes: - Rebase to latest master branch. - Fix bug in pci_uio_map_resource() of BSD code. 'maps[i].path' shouldn't be freed. Fixed in below patch: [PATCH 3/5] eal: Fix memory leaks and needless increment of pci_map_addr - 'path' member of 'struct mapped_pci_resource' should not be removed because it will be used in BSD code. Fixed in below patch: [PATCH 5/5] eal: Fix uio mapping differences between linuxapp and bsdapp PATCH v3 changes: - Squash patches related with pci_map_resource(). - Free maps[].path to easy to understand. (Thanks to Iremonger, Bernard) - Close fds opened in this function. - Remove unused path variable from mapped_pci_resource structure. PATCH v2 changes: - Move 'if-condition' to later patch series. - Fix memory leaks of path. - Fix typos. (Thanks to David Marchand) - Fix commit title and body. - Fix pci_map_resource() to handle MAP_FAILED. (Thanks to Iremonger, Bernard) Changes: - This patch set is derived from below. "[PATCH v2] eal: Port Hotplug support for BSD" - Set cfg_fd as -1, when cfg_fd is closed. (Thanks to Iremonger, Bernard) - Remove needless coding style fixings. - Fix coding style of if-else condition. (Thanks to Richardson, Bruce) Tetsuya.Mukawa (12): eal: Fix coding style of eal_pci.c and eal_pci_uio.c eal: Close file descriptor of uio configuration eal: Fix memory leaks and needless increment of pci_map_addr eal/bsdapp: Change names of pci related data structure eal: Fix uio mapping differences between linuxapp and bsdapp eal: Add pci_uio_alloc_resource() eal: Add pci_uio_map_resource_by_index() eal: Consolidate pci_map and mapped_pci_resource of linuxapp and bsdapp eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp eal: Consolidate pci uio functions of linuxapp and bsdapp eal: Consolidate pci_map/unmap_device() of linuxapp and bsdapp eal: Consolidate rte_eal_pci_probe/close_one_driver() of linuxapp and bsdapp lib/librte_eal/bsdapp/eal/Makefile | 1 + lib/librte_eal/bsdapp/eal/eal_pci.c | 283 ++++++-------------- .../bsdapp/eal/include/exec-env/rte_interrupts.h | 1 + lib/librte_eal/common/eal_common_pci.c | 225 ++++++++++++++++ lib/librte_eal/common/eal_common_pci_uio.c | 270 +++++++++++++++++++ lib/librte_eal/common/eal_private.h | 59 ++++- lib/librte_eal/common/include/rte_pci.h | 40 +++ lib/librte_eal/linuxapp/eal/Makefile | 1 + lib/librte_eal/linuxapp/eal/eal_pci.c | 236 +---------------- lib/librte_eal/linuxapp/eal/eal_pci_init.h | 39 +-- lib/librte_eal/linuxapp/eal/eal_pci_uio.c | 290 ++++++--------------- lib/librte_ether/rte_ethdev.c | 1 + 12 files changed, 753 insertions(+), 693 deletions(-) create mode 100644 lib/librte_eal/common/eal_common_pci_uio.c -- 2.1.4