From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id B17451B608 for ; Fri, 13 Oct 2017 04:20:02 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Oct 2017 19:20:01 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,369,1503385200"; d="scan'208";a="909482690" Received: from unknown (HELO [10.241.225.21]) ([10.241.225.21]) by FMSMGA003.fm.intel.com with ESMTP; 12 Oct 2017 19:20:01 -0700 To: "Yang, Zhiyong" , "dev@dpdk.org" , "thomas@monjalon.net" , "Richardson, Bruce" References: <20171012093252.53100-1-zhiyong.yang@intel.com> <3a39bb25-ded4-4d7c-1408-6fc20820ce70@intel.com> From: Ferruh Yigit Message-ID: Date: Fri, 13 Oct 2017 03:20:01 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 0/6] fix port id type X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 13 Oct 2017 02:20:03 -0000 On 10/13/2017 3:04 AM, Yang, Zhiyong wrote: > Hi Ferruh, > > The patch (ethdev: increase port_id range) tend to increase port id range in ethdev, not eventdev, > Do you mean we should change eventdev port id ? > > Before V2, that patchset includes eventdev port id extension. > > but Bruce reminded me about it in that patchset v2 thread and suggest to send a > separate patch/patchset to fix it if required. So, I didn’t touch the event related files after v2. > > If required, I can send eventdev port id range patch soon. Sorry I just copy-paste the command output without parsing. No we are not interested in eventdev ones, please ignore them, but in the list there can be still ethdev related ones, you will need to check them first. Thanks, ferruh > > FYI. > Discussion in V2 patch > >>>> Fix some checkpatch issues from the original code and remove some >>>> unnecessary cast operations. >>>> >>>> Signed-off-by: Zhiyong Yang >>>> --- >>>> app/pdump/main.c | 2 +- >>>> app/test-eventdev/test_perf_common.h | 6 +- >>>> app/test-pmd/cmdline.c | 4 +- >>>> app/test-pmd/config.c | 4 +- >>>> app/test-pmd/testpmd.c | 18 +- >>>> app/test-pmd/testpmd.h | 2 +- >>>> drivers/event/octeontx/ssovf_evdev.c | 6 +- >>>> drivers/event/octeontx/ssovf_evdev.h | 4 +- >>>> drivers/event/skeleton/skeleton_eventdev.c | 4 +- >>>> drivers/event/skeleton/skeleton_eventdev.h | 2 +- >>>> drivers/event/sw/sw_evdev.c | 4 +- >>>> drivers/event/sw/sw_evdev.h | 6 +- >>>> drivers/event/sw/sw_evdev_xstats.c | 8 +- >>> >>> These are not ethdev drivers, but eventdev ones. I don't think >>> eventdev needs to go to 16-bit port numbers, but there is no harm in doing so. >>> However, it would be best to modify eventdev drivers in a separate >>> patch - or even patchset. >>> >> >> I will remove them and focus on ethdev in the patchset. > > Thanks > Zhiyong > > >> -----Original Message----- >> From: Yigit, Ferruh >> Sent: Friday, October 13, 2017 4:13 AM >> To: Yang, Zhiyong ; dev@dpdk.org >> Subject: Re: [dpdk-dev] [PATCH 0/6] fix port id type >> >> On 10/12/2017 7:03 PM, Ferruh Yigit wrote: >>> On 10/12/2017 10:32 AM, Zhiyong Yang wrote: >>>> Now ethdev port_id has been increased to uint16_t from uint8_t in the >>>> below patch http://www.dpdk.org/dev/patchwork/patch/29418/ >>>> >>>> but >>>> 1. Some features which are applied were still developed based on >>>> older version uint8_t port_id. >>>> 2. Some places using port id such as testpmd are omitted in previous patch. >>>> The patchset fixes some port id type issues. >>>> >>>> Zhiyong Yang (6): >>>> lib/librte_ether: fix port id type >>>> net/bnxt: fix port id type >>>> net/liquidio: fix port id type >>>> app/pdump: fix port id type >>>> app/proc_info: fix port id type >>>> app/testpmd: fix port id type >>> >>> Series applied to dpdk-next-net/master, thanks. >>> >>> >>> Thanks for the patches! >>> But I can see there are still more uint8_t usage [1], some looks valid. >>> >>> I aware this looks like never it will never end but would you mind >>> doing one more go? It would be nice to get all fixed before rc1. >>> And I hope we won't need to change port_id storage type again... >>> >>> >>> [1] >>> $ git grep -c "uint8_t port_id" >> >> And (git grep "uint8_t port") yields even more result. >> >>> app/test-eventdev/test_order_common.h:2 >>> app/test-eventdev/test_perf_common.h:2 >>> doc/guides/prog_guide/eventdev.rst:1 >>> doc/guides/prog_guide/poll_mode_drv.rst:1 >>> doc/guides/prog_guide/rte_flow.rst:6 >>> doc/guides/sample_app_ug/kernel_nic_interface.rst:4 >>> doc/guides/sample_app_ug/link_status_intr.rst:1 >>> drivers/event/dpaa2/dpaa2_eventdev.c:2 >>> drivers/event/octeontx/ssovf_evdev.c:2 >>> drivers/event/skeleton/skeleton_eventdev.c:2 >>> drivers/event/skeleton/skeleton_eventdev.h:1 >>> drivers/event/sw/sw_evdev.c:2 >>> drivers/event/sw/sw_evdev_xstats.c:1 >>> drivers/net/bonding/rte_eth_bond_pmd.c:1 >>> drivers/net/i40e/i40e_ethdev.c:1 >>> examples/ethtool/lib/rte_ethtool.c:21 >>> examples/ethtool/lib/rte_ethtool.h:21 >>> examples/eventdev_pipeline_sw_pmd/main.c:6 >>> examples/ipsec-secgw/ipsec-secgw.c:2 >>> examples/l2fwd-jobstats/main.c:1 >>> examples/quota_watermark/qw/init.c:2 >>> examples/quota_watermark/qw/init.h:2 >>> examples/quota_watermark/qw/main.c:1 >>> examples/server_node_efd/node/node.c:2 >>> examples/tep_termination/vxlan_setup.c:1 >>> examples/tep_termination/vxlan_setup.h:2 >>> lib/librte_eventdev/rte_event_eth_rx_adapter.c:1 >>> lib/librte_eventdev/rte_eventdev.c:7 >>> lib/librte_eventdev/rte_eventdev.h:11 >>> lib/librte_eventdev/rte_eventdev_pmd.h:2 >>> test/test/test_link_bonding.c:1 >>> test/test/test_link_bonding_mode4.c:2 >>> test/test/test_link_bonding_rssconf.c:3 >>> >