[dpdk-dev] [PATCH v8 2/3] ring: introduce new header file to include common functions
Stephen Hemminger
stephen at networkplumber.org
Mon Jan 22 17:54:03 CET 2018
On Fri, 19 Jan 2018 16:47:36 +0000
Hemant Agrawal <hemant.agrawal at nxp.com> wrote:
> Hi Olivier,
>
> > On Fri, Jan 19, 2018 at 07:45:30PM +0530, Hemant Agrawal wrote:
> > > Hi Jia,
> > >
> > > On 1/17/2018 9:33 AM, Jia He wrote:
> > > > Move the common part of rte_ring.h into rte_ring_generic.h.
> > > > Move the memory barrier part into update_tail().
> > > >
> > > > No functional changes here.
> > > >
> > > > Signed-off-by: Jia He <jia.he at hxt-semitech.com>
> > > > Suggested-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> > > > Suggested-by: Ananyev Konstantin <konstantin.ananyev at intel.com>
> > > > Acked-by: Jerin Jacob <jerin.jacob at caviumnetworks.com>
> > > > Acked-by: Olivier Matz <olivier.matz at 6wind.com>
> > > > ---
> > > > diff --git a/lib/librte_ring/rte_ring_generic.h
> > > > b/lib/librte_ring/rte_ring_generic.h
> > > > new file mode 100644
> > > > index 0000000..01f2cae
> > > > --- /dev/null
> > > > +++ b/lib/librte_ring/rte_ring_generic.h
> > > > @@ -0,0 +1,202 @@
> > > > +/*-
> > > > + * Copyright(c) 2017 Intel Corporation. All rights reserved.
> > > > + * All rights reserved.
> > > > + *
> > > > + * SPDX-License-Identifier: BSD-3-Clause
> > >
> > > The SPDX should be first line. See other files for Intel or NXP.
>
> [Hemant] Don't add SPDX to this file.
> This file is not BSD-3 licensed. Please keep the full text as in the original file.
If it can't be BSD-3 licensed, then it needs to be removed. Mixing licenses is a real problem
More information about the dev
mailing list