From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f46.google.com (mail-pa0-f46.google.com [209.85.220.46]) by dpdk.org (Postfix) with ESMTP id 056F75686 for ; Thu, 15 Jan 2015 21:34:20 +0100 (CET) Received: by mail-pa0-f46.google.com with SMTP id lf10so19629066pab.5 for ; Thu, 15 Jan 2015 12:34:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=25xQDOu56BmCv/BJ6kdIYeo1scHyjjiy6E8c1IXwyk8=; b=DHJyNCWovYN0ziDDziZBr0SMrLka+WQcLd2F404tfoG7mBxWRH3ya0Eh+UQQ477WD8 cKS6df+BeoR1PbeHUdQHCI2hscKPm/7hCPEtp5nrvXgQVHGpHDZ0F8I1ZaNPgiBQQtk3 z72MH4lHWI7GdInmnErYb3Fw081uTFMa1ECQuHKpnrXcfasib8ZpWjdDcByU6ndiyLA8 ZPqSxgbzFwIeF1IozR7hDPzmYfn9hjprJ6UhIV5gtOTiLxan1x5tz9hFCUmRkna7XZfl 9e8FvWSP85HH6DJKD4rOSOFLb6T0UVHpiJNp3oMwKdl0V+XflxQfDGN+sZIgyO06bxZC v0SQ== MIME-Version: 1.0 X-Received: by 10.68.194.106 with SMTP id hv10mr16965274pbc.50.1421354059231; Thu, 15 Jan 2015 12:34:19 -0800 (PST) Received: by 10.70.114.233 with HTTP; Thu, 15 Jan 2015 12:34:19 -0800 (PST) In-Reply-To: <3180122.JKDOmAPjoA@xps13> References: <3180122.JKDOmAPjoA@xps13> Date: Thu, 15 Jan 2015 12:34:19 -0800 Message-ID: From: Ravi Kerur To: Thomas Monjalon Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 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: Thu, 15 Jan 2015 20:34:20 -0000 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 changes > in > > e1000/ directory to e1000_osdep.h file only and the rest in PMD will 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 > > ... > > This is an Intel driver so you should ask to the responsible of this code > at Intel. > The problem is that there is not really an identified responsible for this > driver. > > The rule is to not change the base driver, even osdep files. > But it would be better to have an exception here. > > > PS: please avoid top-posting. > Please let me know who is the contact person from Intel so I can add him/her to "To" list when I send the patch or Should I contact Jim St Leger and ask him about this? Thanks. > > > On Mon, Jan 5, 2015 at 8:40 AM, Ravi Kerur wrote: > > > > > Inline > > > > > > On Mon, Jan 5, 2015 at 12:55 AM, Thomas Monjalon < > > > thomas.monjalon@6wind.com> wrote: > > > > > >> 2015-01-04 15:28, Ravi Kerur: > > >> > We have a Gigabyte H97N motherboard which has I217 Intel chipset > which > > >> uses > > >> > e100e drivers. I looked into lib/librte_pmd_e1000 directory and I > do see > > >> > that e1000e code is integrated but missing some support for > read/write > > >> from > > >> > flash_address and other minor things. I have made changes shown > below > > >> and > > >> > have done some testing with testpmd utility and now have following > > >> questions > > >> > > > >> > 1. What amount of testing is required to qualify patch as > successfully > > >> > tested on new chipsets > > >> > > >> There is no good answer to this question. Generally, you must be sure > that > > >> you don't break anything. > > >> So you must test the code paths you have changed. > > >> > > > > > > yes I have done testing on Ubuntu for I217 using testpmd. > > > > > >> > > >> > 2. FreeBSD testing, currently we have Ubuntu 14.04 installed on > existing > > >> > H97N motherboard and testing is done solely on Linux. We plan to get > > >> > another motherboard which will have I218 chipset and still deciding > > >> whether > > >> > to go with FreeBSD or Ubuntu. So the question I have is what amount > of > > >> > testing should be done on FreeBSD? I don't think > > >> setup.sh/dpdk_nic_bind.py > > >> > works on FreeBSD yet hence the question on testing. > > >> > > >> FreeBSD testing is required when patching common EAL, scripts or > > >> makefiles. > > >> > > >> > > lib/librte_pmd_e1000/e1000/e1000_api.c | 21 > > >> +++++++++++++++++++++ > > >> > > lib/librte_pmd_e1000/e1000/e1000_api.h | 1 + > > >> > > lib/librte_pmd_e1000/e1000/e1000_osdep.h | 24 > > >> +++++++++++++++++++----- > > >> > > >> These files are part of the base driver. > > >> The rule is to not patch them and try to do the changes in PMD only. > > >> There can be exceptions if an Intel maintainer acknowledges it. > > >> > > > > > > Changes in these files are modifying existing macros > > > > > > E1000_READ_FLASH_REG, > > > E1000_WRITE_FLASH_REG > > > ... > > > > > > If it is not recommended to modify these files, should I move macros > into > > > some PMD file? > > > > > > Thanks. > >