Le 1 juin 2017 03:03:39 "Wang, Yipeng1" <yipeng1.w...@intel.com> a écrit :

Hi Vincent,

Thanks for the comments and some quick responses below:

- DPDK Bloom Filter is derived from the libbloom (as shown by the included BSD license), but optimized for performance with various DPDK goodness such as rte_zmalloc, rte_bitmap, rte_jhash, etc. It doesn't seem appropriate to bring those DPDK specific features into libbloom, which is designed for more generic environments.

Did you try and ask to the original author?

- DPDK Bloom Filter is just one very basic mode among other more sophisticated ones in the DPDK Membership Library, which supports a different set of APIs (e.g., bulk lookup, multi-match, etc.) than those supported in libbloom.

How does it justify forking the code?

- It may make sense to bring some generic (i.e., non DPDK specific) improvements back to libbloom once identified.

So let's start first with contributions to libbloom, please.



Thanks
Yipeng

-----Original Message-----
From: Vincent Jardin [mailto:vincent.jar...@6wind.com]
Sent: Saturday, May 27, 2017 2:42 AM
To: Wang, Yipeng1 <yipeng1.w...@intel.com>
Cc: dev@dpdk.org; Gobriel, Sameh <sameh.gobr...@intel.com>; Wang, Ren
<ren.w...@intel.com>; Tai, Charlie <charlie....@intel.com>
Subject: Re: [dpdk-dev] [RFC] Add Membership Library

Why duplicating Jyri's libbloom - https://github.com/jvirkki/libbloom - for this
DPDK capability? Why not showing that you can contribute to libbloom and
make it linkable with the DPDK?

There are so many duplicated code...

Thank you,
  Vincent




Reply via email to