From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f48.google.com (mail-pa0-f48.google.com [209.85.220.48]) by dpdk.org (Postfix) with ESMTP id 7E3E958F3 for ; Thu, 23 Apr 2015 11:48:59 +0200 (CEST) Received: by paboj16 with SMTP id oj16so13906740pab.0 for ; Thu, 23 Apr 2015 02:48:58 -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=P1OCWWF66SfzFOQJ/hXYkXBKidP7QTKyHQRAFS32AEA=; b=GZfYPVxIrcWCcw2shRBhgfd4b+oTqUHava2sURRm/QacTub0Ln8mZaFA+H3stHmK5T WPGYgd3VT0XPmc1y4dbS4/KdocUUVS0J+fJz8ULb9yxHZphox+65xFRv6sHCrtMH7LYl 6RmueThqqMbl+AoemLbpllQRV0pl5I/w4bQxUXQC/cTUUZ2cBJXXksqz+5SEGni3DBdN IStyN/KDawRIICSNfzmgBOdRE8OsNhUQz2FXH65t7TDD8jAZQatIzL4M44foYJVeUkPO Mh1K0HaCuJwNznX+FV8dtQSrqKiQCUs6hyjo+Xyhz3x9GvMfMx6thYg23RF2Ccnjg+pY 39Yw== X-Gm-Message-State: ALoCoQlhLkxVq/U+7aJt1vxoZoET0g1bXIVnIcjal/rRBBunMYfb3YlHMsMQguQyyczyCeKlRVs0 X-Received: by 10.70.35.171 with SMTP id i11mr3650324pdj.103.1429782538723; Thu, 23 Apr 2015 02:48:58 -0700 (PDT) Received: from [10.16.129.101] (napt.igel.co.jp. [219.106.231.132]) by mx.google.com with ESMTPSA id fk4sm7502578pbb.80.2015.04.23.02.48.56 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 23 Apr 2015 02:48:58 -0700 (PDT) Message-ID: <5538C008.4000308@igel.co.jp> Date: Thu, 23 Apr 2015 18:48:56 +0900 From: Tetsuya Mukawa User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: "Iremonger, Bernard" , "dev@dpdk.org" References: <1426584645-28828-7-git-send-email-mukawa@igel.co.jp> <1427445553-3151-1-git-send-email-mukawa@igel.co.jp> <8CEF83825BEC744B83065625E567D7C204A0FF82@IRSMSX108.ger.corp.intel.com> <553873C5.1000909@igel.co.jp> <8CEF83825BEC744B83065625E567D7C204A19A4B@IRSMSX108.ger.corp.intel.com> In-Reply-To: <8CEF83825BEC744B83065625E567D7C204A19A4B@IRSMSX108.ger.corp.intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH v3 0/5] 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: Thu, 23 Apr 2015 09:48:59 -0000 On 2015/04/23 17:12, Iremonger, Bernard wrote: > >> -----Original Message----- >> From: Tetsuya Mukawa [mailto:mukawa@igel.co.jp] >> Sent: Thursday, April 23, 2015 5:24 AM >> To: Iremonger, Bernard; dev@dpdk.org >> Cc: Richardson, Bruce; david.marchand@6wind.com >> Subject: Re: [PATCH v3 0/5] Clean up pci uio implementations >> >> On 2015/04/21 22:15, Iremonger, Bernard wrote: >>>> -----Original Message----- >>>> From: Tetsuya Mukawa [mailto:mukawa@igel.co.jp] >>>> Sent: Friday, March 27, 2015 8:39 AM >>>> To: dev@dpdk.org >>>> Cc: Iremonger, Bernard; Richardson, Bruce; david.marchand@6wind.com; >>>> Tetsuya Mukawa >>>> Subject: [PATCH v3 0/5] Clean up pci uio implementations >>>> >>>> This patch set cleans up pci uio implementation. These clean up are >>>> for consolidating pci uio implementation of linuxapp and bsdapp, and moving consolidated >> functions in eal common. >>>> Because of above, this patch set tries to implement linuxapp and bsdapp almost same. >>>> Actual consolidations will be done later patch set. >>>> >>>> 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 (5): >>>> 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 >>>> >>>> lib/librte_eal/bsdapp/eal/eal_pci.c | 166 +++++++++++++++++------------ >>>> lib/librte_eal/linuxapp/eal/eal_pci_init.h | 1 - >>>> lib/librte_eal/linuxapp/eal/eal_pci_uio.c | 91 +++++++++------- >>>> 3 files changed, 154 insertions(+), 104 deletions(-) >>>> >>>> -- >>>> 1.9.1 >>> Series >>> Acked-by: Bernard Iremonger >> Hi Bernard, >> >> Thanks for your checking. >> It seems I need to rebase this patches for latest matser branch. >> So I will update and submit it soon. >> >> Regards, >> Tetsuya > Hi Tetsuya, > I applied the v3 patches to the latest master branch on Monday last and there were no issues. > The patches built cleanly on Linux and Free BSD. > > Regards, > > Bernard. > Hi Bernard, Thanks. I made sure I could apply my patches to latest. I will prepare bottom half of port hotplug patches for BSD. Regards, Tetsuya