All of lore.kernel.org
 help / color / mirror / Atom feed
From: Declan Doherty <declan.doherty@intel.com>
To: "dev@dpdk.org" <dev@dpdk.org>
Subject: Change new libraries to have dpdk_ prefix instead of rte_
Date: Tue, 5 Apr 2016 14:29:23 +0100	[thread overview]
Message-ID: <5703BDB3.3010501@intel.com> (raw)
In-Reply-To: <1828032.Xk8haRDBUM@xps13>

I'd like people opinion of Thomas proposal to have all new libraries use 
a dpdk_ prefix instead of rte_*. Although I agree that dpdk_ would 
probably make sense, I don't like the ascetics of inconsistent prefixes 
on dpdk libraries. Any comments?




2016-04-05 09:48, Trahe, Fiona:
> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> > 2016-04-05 08:53, Fiona Trahe:
> > > The cryptodev API was introduced in the DPDK 2.2 release.
> > > Since then it has
> > >  - been reviewed and iterated for the DPDK 16.04 release
> > >  - had extensive use by the l2fwd-crypto app,
> > > 			the ipsec-secgw example app,
> > > 			the test app.
> > > We believe it is now stable and the EXPERIMENTAL label should be removed.
> >
> > Are you sure sure? :)
> > It means you will try hard to not change the API anymore or you'll need a
> > deprecation notice strongly agreed (outside of your team).
>
> We're sure sure :)

I think we could change the namespace before making this API stable.
What about using a dpdk_ prefix instead of rte_ ?
(and some macros have CRYPTODEV or CDEV prefixes)

  reply	other threads:[~2016-04-05 13:32 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-05  7:53 [PATCH] cryptodev: Remove EXPERIMENTAL label Fiona Trahe
2016-04-05  8:47 ` Thomas Monjalon
2016-04-05  9:48   ` Trahe, Fiona
2016-04-05 10:11     ` Thomas Monjalon
2016-04-05 13:29       ` Declan Doherty [this message]
2016-04-05 14:03         ` Change new libraries to have dpdk_ prefix instead of rte_ Ananyev, Konstantin
2016-04-05 14:22           ` Thomas Monjalon
2016-04-05 16:45           ` Wiles, Keith
2016-04-06 11:05 ` [PATCH] cryptodev: Remove EXPERIMENTAL label Fiona Trahe
2016-04-06 16:37   ` Thomas Monjalon
2016-04-06 16:46     ` Trahe, Fiona
2016-04-06 16:49       ` Thomas Monjalon
2016-04-06 17:07     ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5703BDB3.3010501@intel.com \
    --to=declan.doherty@intel.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.