From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 6A44320F for ; Wed, 11 Nov 2015 02:40:28 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga102.jf.intel.com with ESMTP; 10 Nov 2015 17:40:28 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,273,1444719600"; d="scan'208";a="682634488" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga003.jf.intel.com with ESMTP; 10 Nov 2015 17:40:28 -0800 Received: from fmsmsx117.amr.corp.intel.com (10.18.116.17) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.248.2; Tue, 10 Nov 2015 17:40:26 -0800 Received: from shsmsx151.ccr.corp.intel.com (10.239.6.50) by fmsmsx117.amr.corp.intel.com (10.18.116.17) with Microsoft SMTP Server (TLS) id 14.3.248.2; Tue, 10 Nov 2015 17:40:26 -0800 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.138]) by SHSMSX151.ccr.corp.intel.com ([169.254.3.88]) with mapi id 14.03.0248.002; Wed, 11 Nov 2015 09:40:24 +0800 From: "Cao, Waterman" To: "Liu, Yong" , "Mcnamara, John" , Thomas Monjalon Thread-Topic: [dpdk-dev] [PATCH v5 1/7] ethdev: add additional ieee1588 support functions Thread-Index: AQHRG8HIhSZIgaYNfEeWS6BmbPoqg56U2GIAgAEzf3A= Date: Wed, 11 Nov 2015 01:40:24 +0000 Message-ID: References: <1443799208-9408-1-git-send-email-danielx.t.mrzyglod@intel.com> <2257223.vnG0RKMETz@xps13> <3893351.MZRdjpoyuB@xps13> <86228AFD5BCD8E4EBFD2B90117B5E81E10F639CA@SHSMSX103.ccr.corp.intel.com> In-Reply-To: <86228AFD5BCD8E4EBFD2B90117B5E81E10F639CA@SHSMSX103.ccr.corp.intel.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v5 1/7] ethdev: add additional ieee1588 support functions 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: Wed, 11 Nov 2015 01:40:29 -0000 Hi Thomas, Marvin has already tested the patches with a recent HEAD release of DPDK an= d are ready to test them as part of RC2. As you say the changes are localiz= ed to the Intel drivers and from our review they look like low risk of brea= king anything else. Thanks Waterman -----Original Message----- From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Liu, Yong Sent: Tuesday, November 10, 2015 11:18 PM To: Mcnamara, John; Thomas Monjalon Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v5 1/7] ethdev: add additional ieee1588 supp= ort functions Hi Thomas& John, Some update from validation team. > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Mcnamara, John > Sent: Tuesday, November 10, 2015 10:12 PM > To: Thomas Monjalon > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v5 1/7] ethdev: add additional ieee1588=20 > support functions > =09 > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > Sent: Tuesday, November 10, 2015 11:58 AM > > To: Mcnamara, John > > Cc: Mrzyglod, DanielX T; dev@dpdk.org > > Subject: Re: [dpdk-dev] [PATCH v5 1/7] ethdev: add additional=20 > > ieee1588 support functions > > > > 2015-11-10 11:36, Mcnamara, John: > > > From: Thomas Monjalon > > > > I'll try to fix it now to be sure it will be one of the first=20 > > > > series ready for the 2.3 cycle. > > > > > > These comments are minor and could be fixed now. > > > > After having a closer look in the drivers change, it seems to be=20 > > restricted to the PTP functions of the Intel drivers. > > So you can ask to the Intel validation team if they are OK to add it=20 > > in RC2. > > I think it would be a wrong idea because we need to stop moving the > ethdev > > and drivers code, and focus on other DPDK areas for the RC2. >=20 > Hi Thomas, >=20 > Ok. I'll ask the validation team to evaluate the effect of the patches. >=20 >=20 For our validation team's view, in this patch set implemented some APIs whi= ch used to support Precision Time Protocol. The sample based those APIs can work as real ptp client. We have verified i= t work fine with linux ptp server.