From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stephen Hemminger <stephen@networkplumber.org>
Subject: Re: [PATCH v6 2/3] eal: add u64 bit variant for
 reciprocal
Date: Wed, 6 Sep 2017 08:37:15 -0700
Message-ID: <20170906083715.2084ec8d@xeon-e3>
References: <1504693294-2100-1-git-send-email-pbhagavatula@caviumnetworks.com>
 <1504693294-2100-2-git-send-email-pbhagavatula@caviumnetworks.com>
 <11668986-444f-4238-147f-4baa82d15285@redhat.com>
 <20170906144133.GA21468@PBHAGAVATULA-LT>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Cc: Kevin Traynor <ktraynor@redhat.com>, cristian.dumitrescu@intel.com,
 dev@dpdk.org
To: Pavan Nikhilesh Bhagavatula <pbhagavatula@caviumnetworks.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-pf0-f175.google.com (mail-pf0-f175.google.com
 [209.85.192.175]) by dpdk.org (Postfix) with ESMTP id D035F37AC
 for <dev@dpdk.org>; Wed,  6 Sep 2017 17:37:23 +0200 (CEST)
Received: by mail-pf0-f175.google.com with SMTP id g13so13365056pfm.2
 for <dev@dpdk.org>; Wed, 06 Sep 2017 08:37:23 -0700 (PDT)
In-Reply-To: <20170906144133.GA21468@PBHAGAVATULA-LT>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Wed, 6 Sep 2017 20:11:34 +0530
Pavan Nikhilesh Bhagavatula <pbhagavatula@caviumnetworks.com> wrote:

> On Wed, Sep 06, 2017 at 01:28:24PM +0100, Kevin Traynor wrote:
> > On 09/06/2017 11:21 AM, Pavan Nikhilesh wrote:  
> > > From: Pavan Bhagavatula <pbhagavatula@caviumnetworks.com>
> > >
> > > Currently, rte_reciprocal only supports unsigned 32bit divisors. This
> > > commit adds support for unsigned 64bit divisors.
> > >
> > > Rename unsigned 32bit specific functions appropriately and update
> > > librte_sched accordingly.
> > >
> > > Signed-off-by: Pavan Nikhilesh <pbhagavatula@caviumnetworks.com>
> > > ---
> > >  lib/librte_eal/bsdapp/eal/rte_eal_version.map   |   3 +-
> > >  lib/librte_eal/common/include/rte_reciprocal.h  | 109 ++++++++++++++++++++--
> > >  lib/librte_eal/common/rte_reciprocal.c          | 116 +++++++++++++++++++++---
> > >  lib/librte_eal/linuxapp/eal/rte_eal_version.map |   3 +-
> > >  lib/librte_sched/Makefile                       |   4 +-
> > >  lib/librte_sched/rte_sched.c                    |   9 +-
> > >  6 files changed, 219 insertions(+), 25 deletions(-)
> > >
> > > diff --git a/lib/librte_eal/bsdapp/eal/rte_eal_version.map b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
> > > index 90d7258..59a85bb 100644
> > > --- a/lib/librte_eal/bsdapp/eal/rte_eal_version.map
> > > +++ b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
> > > @@ -241,6 +241,7 @@ EXPERIMENTAL {
> > >  DPDK_17.11 {
> > >  	global:
> > >
> > > -	rte_reciprocal_value;
> > > +	rte_reciprocal_value_u32;
> > > +	rte_reciprocal_value_u64;
> > >
> > >  } DPDK_17.08;
> > > diff --git a/lib/librte_eal/common/include/rte_reciprocal.h b/lib/librte_eal/common/include/rte_reciprocal.h
> > > index b6d752f..85599e6 100644
> > > --- a/lib/librte_eal/common/include/rte_reciprocal.h
> > > +++ b/lib/librte_eal/common/include/rte_reciprocal.h  
> >
> > Hi Pavan, sorry for commenting late but the license in v1 of this file
> > states it cannot be removed. It is not included in later versions - can
> > you explain why?
> >  
> Hi Kevin,
> 
> I have misinterpreted this mail
> http://dpdk.org/ml/archives/dev/2017-August/073781.html,
> any suggestion on how to proceed on this further?
> 
> Thanks,
> Pavan

License issues need legal advice (TAB could ask LF if required).
Sorry, I am a cynic engineer not a lawyer.

Easiest solution is to find equivalent code in FreeBSD or some other project
which does not have the restrictions.