From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f52.google.com (mail-pa0-f52.google.com [209.85.220.52]) by dpdk.org (Postfix) with ESMTP id 7CB10568F for ; Mon, 16 Mar 2015 04:36:52 +0100 (CET) Received: by pagr17 with SMTP id r17so51587317pag.0 for ; Sun, 15 Mar 2015 20:36:51 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=PzheHbHrqPgnduJNmSJ3wkhOUfuV/0F89oY64RhFa/s=; b=fTSkx8arZsrArmgyaT6s8doAOHdByjQ3EJ15K1uTstNr2qo69Vj1YqvG8bgbsSE0G1 r9Z6vtBZKr0REum9hTOLuvEfm7yWJubUvqbkO8MNG+Ue2ch1gyr2kOW4SB53Xf8x2dxL atOb3kylIMvKPFn6lXq1on6MWtnNaAiCiFGog+OJh5/HiUhMho07FJ0U5sNK5UxbCgp1 UGxyPaTE98GfcB84S/2+CsrCfTWYV773AUUW05m//UhuDKFEvMtw5qe7HTSc+svUd6fa oLnjFahfv2XjPfi6y2J3m+PSlnfxwucrVkJ0sS4coRbRm5hndNodrYa6k1NR7tTiQotL VX9w== X-Gm-Message-State: ALoCoQm3r3jUoSpb2DGJ2uhb8YWw0FxXWQi93EsSgVKe8DejS18a59XejkHsRNAhLfuEzWub7Tbe X-Received: by 10.66.161.194 with SMTP id xu2mr69478333pab.48.1426477011684; Sun, 15 Mar 2015 20:36:51 -0700 (PDT) Received: from [10.16.129.101] (napt.igel.co.jp. [219.106.231.132]) by mx.google.com with ESMTPSA id kn4sm14561650pab.48.2015.03.15.20.36.49 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 15 Mar 2015 20:36:51 -0700 (PDT) Message-ID: <55064FD2.1090403@igel.co.jp> Date: Mon, 16 Mar 2015 12:36:50 +0900 From: Tetsuya Mukawa User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: "Iremonger, Bernard" , "dev@dpdk.org" References: <1425438703-18895-1-git-send-email-mukawa@igel.co.jp> <1426155474-1596-1-git-send-email-mukawa@igel.co.jp> <8CEF83825BEC744B83065625E567D7C2049F2891@IRSMSX108.ger.corp.intel.com> In-Reply-To: <8CEF83825BEC744B83065625E567D7C2049F2891@IRSMSX108.ger.corp.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Subject: Re: [dpdk-dev] [PATCH v2 00/15] eal: Port Hotplug support for BSD 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: Mon, 16 Mar 2015 03:36:52 -0000 On 2015/03/13 20:46, Iremonger, Bernard wrote: > >> -----Original Message----- >> From: Tetsuya Mukawa [mailto:mukawa@igel.co.jp] >> Sent: Thursday, March 12, 2015 10:18 AM >> To: dev@dpdk.org >> Cc: Iremonger, Bernard; Richardson, Bruce; Tetsuya Mukawa >> Subject: [PATCH v2 00/15] eal: Port Hotplug support for BSD >> >> This patch series adds port hotplug support to BSD. >> Also, the patches consolidates pci uio functions of linuxapp and bsdap= p. >> >> Following patches cleanup pci uio code to consolidate later. >> eal: Fix cording style of eal_pci.c and eal_pci_uio.c >> eal: Close file descriptor of uio configuration >> eal: Fix memory leak of pci_uio_map_resource() >> eal: Fix needless incrementation of pci_map_addr >> eal/bsdapp: Change names of pci related data structure >> eal: Use map_idx in pci_uio_map_resource() of bsdapp to work same as= >> linuxapp >> eal: Fix interface of pci_map_resource() of bsdapp >> eal: Add pci_uio_alloc_uio_resource() >> eal: Add pci_uio_map_uio_resource_by_index() >> > Hi Tetsuya, > > It might be better to split this patchset into two patch sets,=20 > A cleanup patchset and a consolidation patchset.=20 > It would make reviewing easier. Hi Bernard, Yes, I will split the series, and will submit cleanup patch set first. Also, I will do followings. - Set Initialized cfg_fd to -1 in below patch [PATCH v2 02/15] eal: Close file descriptor of uio configuration - Squashed below patches. [PATCH v2 03/15] eal: Fix memory leak of pci_uio_map_resource() [PATCH v2 04/15] eal: Fix needless incrementation of pci_map_addr - Removed '#ifdef RTE_LIBRTE_EAL_HOTPLUG' when the functions are consolidated. and remove the last patch. Regards, Tetsuya > Regards, > > Bernard. > > >> Following patches are actually for functions consolidation. >> eal: Consolidate pci_map and mapped_pci_resource of linuxapp and >> bsdapp >> eal: Consolidate rte_eal_pci_probe/close_one_driver() of linuxapp an= d >> bsdapp >> eal: Consolidate pci_map/unmap_device() of linuxapp and bsdapp >> eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp >> eal: Consolidate pci uio functions of linuxapp and bsdapp >> >> The last patch removes CONFIG_RTE_LIBRTE_EAL_HOTPLUG configuration fro= m Linux and BSD. >> And port hotplug is enabled as default with both Linux and BSD. >> eal: Enable Port Hotplug as default in Linux and BSD >> >> >> PATCH v2 changes >> - Consolidate pci uio functions of linuxapp and bsdapp. >> >> >> Tetsuya Mukawa (15): >> eal: Fix cording style of eal_pci.c and eal_pci_uio.c >> eal: Close file descriptor of uio configuration >> eal: Fix memory leak of pci_uio_map_resource() >> eal: Fix needless incrementation of pci_map_addr >> eal/bsdapp: Change names of pci related data structure >> eal: Use map_idx in pci_uio_map_resource() of bsdapp to work same as= >> linuxapp >> eal: Fix interface of pci_map_resource() of bsdapp >> eal: Add pci_uio_alloc_uio_resource() >> eal: Add pci_uio_map_uio_resource_by_index() >> eal: Consolidate pci_map and mapped_pci_resource of linuxapp and >> bsdapp >> eal: Consolidate rte_eal_pci_probe/close_one_driver() of linuxapp an= d >> bsdapp >> eal: Consolidate pci_map/unmap_device() of linuxapp and bsdapp >> eal: Consolidate pci_map/unmap_resource() of linuxapp and bsdapp >> eal: Consolidate pci uio functions of linuxapp and bsdapp >> eal: Enable Port Hotplug as default in Linux and BSD >> >> config/common_bsdapp | 6 - >> config/common_linuxapp | 5 - >> lib/librte_eal/bsdapp/eal/Makefile | 1 + >> lib/librte_eal/bsdapp/eal/eal_pci.c | 292 ++++++------= --------- >> .../bsdapp/eal/include/exec-env/rte_interrupts.h | 1 + >> lib/librte_eal/bsdapp/eal/rte_eal_version.map | 6 + >> lib/librte_eal/common/eal_common_dev.c | 2 - >> lib/librte_eal/common/eal_common_pci.c | 234 ++++++++++++= ++++- >> lib/librte_eal/common/eal_common_pci_uio.c | 224 ++++++++++++= ++++ >> lib/librte_eal/common/eal_private.h | 57 +++- >> lib/librte_eal/common/include/rte_pci.h | 42 ++- >> 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 | 263 +++++-------= ------- >> lib/librte_ether/rte_ethdev.c | 22 +- >> 16 files changed, 702 insertions(+), 729 deletions(-) create mode 10= 0644 >> lib/librte_eal/common/eal_common_pci_uio.c >> >> -- >> 1.9.1