From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4AB78A0555; Wed, 19 Feb 2020 11:56:38 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 1AF751BF7C; Wed, 19 Feb 2020 11:56:38 +0100 (CET) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 330D81BF91 for ; Wed, 19 Feb 2020 11:56:36 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Feb 2020 02:56:34 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.70,459,1574150400"; d="scan'208,217";a="283077936" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by FMSMGA003.fm.intel.com with ESMTP; 19 Feb 2020 02:56:34 -0800 Received: from fmsmsx157.amr.corp.intel.com (10.18.116.73) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 19 Feb 2020 02:56:33 -0800 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX157.amr.corp.intel.com (10.18.116.73) with Microsoft SMTP Server (TLS) id 14.3.439.0; Wed, 19 Feb 2020 02:56:33 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.196]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.222]) with mapi id 14.03.0439.000; Wed, 19 Feb 2020 18:56:29 +0800 From: "Wang, Xiao W" To: Xiaojun Liu CC: "dev@dpdk.org" , "Zhang, Qi Z" , "Kwan, Ngai-mint" , "Fornal, Jakub" , "Keller, Jacob E" , Jeff Zheng , Eyal Cohen Thread-Topic: [PATCH v2 0/7] support switch management Thread-Index: AQHVsAigriNscwtQbUeBgjjdweYzFKf0qTOggAA4zbCAIUVWkIAEOYeLgAfnJ1OAAHv0cA== Date: Wed, 19 Feb 2020 10:56:28 +0000 Message-ID: References: <1576057875-7677-1-git-send-email-xiaojun.liu@silicom.co.il> , , In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH v2 0/7] support switch management 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi Xiaojun, 1. No, don't base on the old patch. Just send new version. 2. Yes. 3. No need, just send to the same names as this version. Please remember= to add "-in-reply-to " when sending new version. The Message-ID can be found in the email header of the previous patch or in= http://patches.dpdk.org/project/dpdk/list/?state=3D*&archive=3Dboth. Also, please document what change you made from previous version. You can r= efer to others' patches in the mailing list. Best Regards, Xiao From: Xiaojun Liu Sent: Wednesday, February 19, 2020 1:58 PM To: Wang, Xiao W Cc: dev@dpdk.org; Zhang, Qi Z ; Kwan, Ngai-mint ; Fornal, Jakub ; Keller, Jaco= b E ; Jeff Zheng ; Eyal= Cohen Subject: Re: [PATCH v2 0/7] support switch management Hi Xiao, I have already finished the fixing. Thank you for your helping so much. I still have some questions. 1. Shall I generate the patch based on old patches ? 2. I used checkpatches.sh+check-git-log.sh+test-build.sh to check the pa= tch, is that enough ? 3. May I send the patch to you first ? So you can help checking it befor= e I submit. Best regards, Xiaojun ________________________________ From: dev > on behalf of = Xiaojun Liu > Sent: Friday, February 14, 2020 10:46 AM To: Wang, Xiao W > Cc: dev@dpdk.org >; = Zhang, Qi Z >; Kwan, Ngai= -mint >; Fornal, = Jakub >; Keller, Jaco= b E >; Jeff Zheng= >; Eyal Cohen > Subject: Re: [dpdk-dev] [PATCH v2 0/7] support switch management hi Xiao, Thanks for your comments! I will fix them ASAP. Best regards, Xiaojun ________________________________ From: Wang, Xiao W > Sent: Tuesday, February 11, 2020 6:31 PM To: Xiaojun Liu > Cc: dev@dpdk.org >; = Zhang, Qi Z >; Kwan, Ngai= -mint >; Fornal, = Jakub >; Keller, Jaco= b E >; Jeff Zheng= >; Eyal Cohen > Subject: RE: [PATCH v2 0/7] support switch management Since this is a big code change, so just some general comments/suggestions = for your next version patch set: - please clean up comments like "XXX" in the code. - It's better to define MACRO for all register addrs and bit shift, please = try to avoid magic number (e.g. in the serdes part). - There're threads created for handling events and timers, we'd better use = pthread_join() to recycle them in fm10k_sm_detach(). - set "tapstop" and "shiftwidth" to 8 in vimrc, then recheck the macros and= keep them aligned. - There're some "printf" for debugging, please use dpdk logging API instead= . Best Regards, Xiao > -----Original Message----- > From: Xiaojun Liu > > Sent: Tuesday, January 21, 2020 2:15 PM > To: Wang, Xiao W > > Cc: dev@dpdk.org; Zhang, Qi Z >; Kwan, Ngai-mint > >; jakub.fornal= @intel.co; Keller, Jacob E > >; Jeff Zheng <= jeff.zheng@silicom.co.il>; Eyal > Cohen > > Subject: RE: [PATCH v2 0/7] support switch management > > Hi Xiao, > > Thank you! I will update the commit log and prepare a document to describ= e > the design and implementation. > > Best regards, > Xiaojun > > -----Original Message----- > From: Wang, Xiao W [mailto:xiao.w.wang@intel.com] > Sent: Tuesday, January 21, 2020 10:53 AM > To: Xiaojun Liu > Cc: dev@dpdk.org; Zhang, Qi Z; Kwan, Ngai-mint; jaku= b.fornal@intel.co; > Keller, Jacob E > Subject: RE: [PATCH v2 0/7] support switch management > > Hi Xiaojun, > > Could you please help to improve the commit logs of all the 7 patches? Th= ey > look very similar, and info like below is not very helpful for reviewer, = since > we already know which file you are adding. > "To support switch management, add the following files: > Add fm10k/switch/fm10k_debug.h(define log Macros). > Add fm10k/switch/fm10k_regs.h(define all the registers)." > > Please talk more about the design and implementation details in the commi= t > log. Refer to history patches if you need a sample. > > Also please help to address the compile error reported by automation in l= ink > http://patches.dpdk.org/patch/63742/: > "ci/Intel-compilation fail Compilation issues" > > I would look deeper into your change, and you can address above comments > simultaneously. > > Best Regards, > Xiao > > > -----Original Message----- > > From: Xiaojun Liu > > > Sent: Wednesday, December 11, 2019 5:52 PM > > To: Wang, Xiao W >;= Zhang, Qi Z > > >; Kwan, Ngai-mint >; > > jakub.fornal@intel.co; Keller, Jacob E > > > Cc: dev@dpdk.org; Xiaojun Liu > > > Subject: [PATCH v2 0/7] support switch management > > > > To avoid configuration for both kernel driver > > and userspace SDK outside DPDK, we add switch > > management in FM10K DPDK PMD driver. > > To enable switch management, you need add > > CONFIG_RTE_FM10K_MANAGEMENT=3Dy in > > config/common_linux when building. > > > > > > Xiaojun Liu (7): > > net/fm10k: add i2c sbus registers definition > > net/fm10k: add some modules of port > > net/fm10k: add config ffu statistics support > > net/fm10k: add flow and switch management > > net/fm10k: add switch initialization > > net/fm10k: add mirror and filter ctrl > > net/fm10k: add dpdk port mapping > > > > drivers/net/fm10k/Makefile | 22 + > > drivers/net/fm10k/fm10k_ethdev.c | 580 +++++- > > drivers/net/fm10k/switch/fm10k_config.c | 855 ++++++++ > > drivers/net/fm10k/switch/fm10k_config.h | 171 ++ > > drivers/net/fm10k/switch/fm10k_debug.h | 19 + > > drivers/net/fm10k/switch/fm10k_ext_port.c | 841 ++++++++ > > drivers/net/fm10k/switch/fm10k_ext_port.h | 136 ++ > > drivers/net/fm10k/switch/fm10k_ffu.c | 1209 +++++++++++ > > drivers/net/fm10k/switch/fm10k_ffu.h | 31 + > > drivers/net/fm10k/switch/fm10k_flow.c | 872 ++++++++ > > drivers/net/fm10k/switch/fm10k_flow.h | 26 + > > drivers/net/fm10k/switch/fm10k_i2c.c | 310 +++ > > drivers/net/fm10k/switch/fm10k_i2c.h | 54 + > > drivers/net/fm10k/switch/fm10k_regs.h | 2202 > ++++++++++++++++++++ > > drivers/net/fm10k/switch/fm10k_sbus.c | 292 +++ > > drivers/net/fm10k/switch/fm10k_sbus.h | 40 + > > drivers/net/fm10k/switch/fm10k_serdes.c | 1886 +++++++++++++++++ > > drivers/net/fm10k/switch/fm10k_serdes.h | 32 + > > drivers/net/fm10k/switch/fm10k_sm.c | 182 ++ > > drivers/net/fm10k/switch/fm10k_sm.h | 78 + > > drivers/net/fm10k/switch/fm10k_spico_code.c | 2966 > > +++++++++++++++++++++++++++ > > drivers/net/fm10k/switch/fm10k_spico_code.h | 21 + > > drivers/net/fm10k/switch/fm10k_stats.c | 1242 +++++++++++ > > drivers/net/fm10k/switch/fm10k_stats.h | 257 +++ > > drivers/net/fm10k/switch/fm10k_switch.c | 2562 > > +++++++++++++++++++++++ > > drivers/net/fm10k/switch/fm10k_switch.h | 336 +++ > > 26 files changed, 17188 insertions(+), 34 deletions(-) > > create mode 100644 drivers/net/fm10k/switch/fm10k_config.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_config.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_debug.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_ext_port.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_ext_port.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_ffu.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_ffu.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_flow.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_flow.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_i2c.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_i2c.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_regs.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_sbus.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_sbus.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_serdes.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_serdes.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_sm.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_sm.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_spico_code.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_spico_code.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_stats.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_stats.h > > create mode 100644 drivers/net/fm10k/switch/fm10k_switch.c > > create mode 100644 drivers/net/fm10k/switch/fm10k_switch.h > > > > -- > > 1.8.3.1