From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f66.google.com (mail-wm0-f66.google.com [74.125.82.66]) by dpdk.org (Postfix) with ESMTP id 22981C656 for ; Fri, 29 Jan 2016 13:41:13 +0100 (CET) Received: by mail-wm0-f66.google.com with SMTP id r129so9642550wmr.0 for ; Fri, 29 Jan 2016 04:41:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc:content-type; bh=K/6j2DP7F7AzwuaEyLbi0M/SN5Zoj2JTFget5C86oI8=; b=GuRSeeKOOvfEaa7ZTtfhYmtPWR65lCYPDZ40S+LS3eoSip3UmigiXehph9hGiTb3Cw GeYkw07i31urBR7HNVzAT/0dGI+3DYhqnpTxm6ZXDKaDL3D/uTNLftdnKoobLWw9wSkn QHFuawfuKZTi48SOfkwCyfNFu9Y3fp/2RBg8kMSmm+0WgYA7y1Zp1jhhT6cuCF73WtLV Pv2SbnhJ71R3a7sevOWkePLZhy8UXtfJayU6pXLv4ZIX7GyNJ1ni6kXAFtrYoCada5sk Ge2i4+N7S88CMsASClXe5wsdzck2jf6XDbJcAqMYOzENoxoGDfl6HeukMjN0ghIELZjf 6eDQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc:content-type; bh=K/6j2DP7F7AzwuaEyLbi0M/SN5Zoj2JTFget5C86oI8=; b=LNfVXijHgdimNufCX6d5RWDGEK2AX0x8YUDuP37lWIx4u3gMb/bq642s4+RimTbW2Y jPe6/eOtNKdU8YwgdmLG+bjTHzt2105ElkGtybB9RaBOHxpitP9HRcFaqToPK3nBRdY4 BJRGU0ItF0e17wz+jHNxqWkxgGGJsQx1iGm9e1iHOdaVcHepm4oc3Tsy2J1hukX/3PkQ xlPpIdfDZxxIGsoR+tyWnuEb8BYyyjhdHGemlvJNoQ5Gt2r1RzImAFXirsOxCMXt0MDC qOE7aqKLi/M69Z/3aiiniE2s/rlTDYUtMBF2Q1Na0AtHUXBl9y8Gzthu/8USonYvZpcn 3y1g== X-Gm-Message-State: AG10YOTLe4moR8MMhLDdOmSIuHAEsEKVnH9d37TQV9CmFEf70w8tqRD4xgq/nAQ486VKF49w1eaJwqEF5JRgjw== X-Received: by 10.194.171.66 with SMTP id as2mr8543058wjc.73.1454071272981; Fri, 29 Jan 2016 04:41:12 -0800 (PST) MIME-Version: 1.0 Sender: marc.sune@gmail.com Received: by 10.27.95.202 with HTTP; Fri, 29 Jan 2016 04:40:53 -0800 (PST) In-Reply-To: <2601191342CEEE43887BDE71AB97725836B0014E@irsmsx105.ger.corp.intel.com> References: <1445810400-8978-1-git-send-email-marcdevel@gmail.com> <1745156.01DTmyzyO8@xps13> <2601191342CEEE43887BDE71AB97725836B000EC@irsmsx105.ger.corp.intel.com> <14139435.dXsMv1AF6J@xps13> <2601191342CEEE43887BDE71AB97725836B0014E@irsmsx105.ger.corp.intel.com> From: Marc Date: Fri, 29 Jan 2016 13:40:53 +0100 X-Google-Sender-Auth: WzZNPkBO-J-XLpsH2QksN_gfOUE Message-ID: To: "Ananyev, Konstantin" Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH v7 3/5] ethdev: redesign link speed config API 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, 29 Jan 2016 12:41:13 -0000 On 29 January 2016 at 11:17, Ananyev, Konstantin < konstantin.ananyev@intel.com> wrote: > > > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > Sent: Friday, January 29, 2016 9:54 AM > > To: Ananyev, Konstantin > > Cc: dev@dpdk.org; Marc Sune; Lu, Wenzhuo; Zhang, Helin; Harish Patil; > Chen, Jing D; Mcnamara, John > > Subject: Re: [dpdk-dev] [PATCH v7 3/5] ethdev: redesign link speed > config API > > > > 2016-01-29 09:47, Ananyev, Konstantin: > > > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > > > 2016-01-29 09:24, Ananyev, Konstantin: > > > > > Can you avoid modifications in the e1000/base code? > Yes I can. It was just to save intermediate variables and conversions. Previously u16 was used to store numeric throughput in mbps in ethdev, and passed to the inner functions in e1000/base, but it cannot accommodate 100G values, so I moved it to uint32_t. But I can do the conversion outside of e1000/base. > > > > We do not modify (and maintain) that part on our own. > > > > > Instead we take it straight from Intel ND. > > > > > So if you feel like these changes are really necessary - please > submit a patch > > > > > to ND first, and if your changes will be applied, will pick it up > from them. > > > > > > > > I was not aware we can submit a change to ND for Intel base drivers. > > > > What is the procedure please? > > > > > > I meant not to the ND directly, but probably to the freebsd e1000 > kernel driver. > > > As I remember, that is the closest one to what we have. > > > From my understanding (I might be wrong here): > > > If they will be accepted, we should see these changes In next code > drops from ND. > > > > These base drivers are used in several places. > > We are allowed to submit a patch in Linux or FreeBSD but not in DPDK > > where the base driver is verbatim? > > Yes, that's my understanding. > > > We have an agreement to not touch them in DPDK > > Yes. > > > but I still think the > > ND team could consider some patches from dpdk.org. > > I personally think that would be a good thing, > but it is up to ND guys to make such decision. Agree, but: Besides documenting (which is necessary), why not importing the sources via a git submodule pointing to the base project where these files are obtained (e.g. in drivers/net/e1000/ext/ or even in the root folder under ext/ and making symlinks to that in e1000 driver), and just use the raw sources and headers from there? That would make it more explicit that these files should not be modified by DPDK by strictly forbidding so, and that changes should be done via the repository pointed in drivers/net/e1000/ext/, hence following that repository's workflow. marc > > Konstantin > >