From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 833265A80 for ; Fri, 16 Jan 2015 12:08:49 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga103.fm.intel.com with ESMTP; 16 Jan 2015 03:03:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,410,1418112000"; d="scan'208";a="671003645" Received: from irsmsx104.ger.corp.intel.com ([163.33.3.159]) by orsmga002.jf.intel.com with ESMTP; 16 Jan 2015 03:08:47 -0800 Received: from irsmsx105.ger.corp.intel.com ([169.254.7.81]) by IRSMSX104.ger.corp.intel.com ([169.254.5.229]) with mapi id 14.03.0195.001; Fri, 16 Jan 2015 11:08:46 +0000 From: "Ananyev, Konstantin" To: "Richardson, Bruce" Thread-Topic: [dpdk-dev] Q on Support for I217 and I218 Intel chipsets. Thread-Index: AQHQKHYtq2//YAz5eE2s33NUueGa/JyxOdCAgACBywCABga8AIAIGsEAgAHXQYCAADTuwIAAuvaAgAADe2A= Date: Fri, 16 Jan 2015 11:08:46 +0000 Message-ID: <2601191342CEEE43887BDE71AB977258213DA3B9@irsmsx105.ger.corp.intel.com> References: <3180122.JKDOmAPjoA@xps13> <2601191342CEEE43887BDE71AB977258213D5104@irsmsx105.ger.corp.intel.com> <20150116105255.GA10516@bricha3-MOBL3> In-Reply-To: <20150116105255.GA10516@bricha3-MOBL3> Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.182] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] Q on Support for I217 and I218 Intel chipsets. 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: Fri, 16 Jan 2015 11:08:50 -0000 > -----Original Message----- > From: Richardson, Bruce > Sent: Friday, January 16, 2015 10:53 AM > To: Ananyev, Konstantin > Cc: Ravi Kerur; Thomas Monjalon; dev@dpdk.org > Subject: Re: [dpdk-dev] Q on Support for I217 and I218 Intel chipsets. >=20 > On Thu, Jan 15, 2015 at 11:54:52PM +0000, Ananyev, Konstantin wrote: > > Hi, > > > > > -----Original Message----- > > > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ravi Kerur > > > Sent: Thursday, January 15, 2015 8:34 PM > > > To: Thomas Monjalon > > > Cc: dev@dpdk.org > > > Subject: Re: [dpdk-dev] Q on Support for I217 and I218 Intel chipsets= . > > > > > > On Wed, Jan 14, 2015 at 8:27 AM, Thomas Monjalon > > > wrote: > > > > > > > 2015-01-09 04:41, Ravi Kerur: > > > > > Thomas, > > > > > > > > > > Please let me know how I can move forward on this. If i confine c= hanges > > > > in > > > > > e1000/ directory to e1000_osdep.h file only and the rest in PMD w= ill that > > > > > work? The reason I ask is because of following comment in README= file. > > > > > > > > > > ... > > > > > Few changes to the original FreeBSD sources were made to: > > > > > - Adopt it for PMD usage mode: > > > > > e1000_osdep.c > > > > > e1000_osdep.h > > > > > ... > > > > Yes, if needed you can modify these files. > > In fact, these files are the only 2 that are allowed to be modified ins= ide e1000 sub-directory. > > As I understand you plan to implement E1000_READ_FLASH_REG and E1000_W= RITE_FLASH_REG > > macros properly, correct? > > Konstantin > > >=20 > As a cleanup we should really look to move these two files out of the e10= 00 > subdirectory (and similarly for the ixgbe versions etc.), so as to give a= cleaner > and more manageable separation between what can be edited or not. It was always like that for all Intel PMDs we have: $ find lib/ -name '*_osdep.*' | grep -v acl lib/librte_pmd_vmxnet3/vmxnet3/vmxnet3_osdep.h lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h lib/librte_eal/linuxapp/kni/ethtool/ixgbe/ixgbe_osdep.h lib/librte_eal/linuxapp/kni/ethtool/igb/e1000_osdep.h lib/librte_pmd_i40e/i40e/i40e_osdep.h lib/librte_pmd_e1000/e1000/e1000_osdep.c lib/librte_pmd_e1000/e1000/e1000_osdep.h As I understand ND has it's own version of _osdep.* for each OS th= ey support. We obviously modify it to fit DPDK purposes. Konstantin >=20 > /Bruce