Hi Mike,

Thanks for your comments.

We implement “table” related code from the scratch. The main purpose is to 
propose the “table” APIs, the code part is just to help people to understand 
what we try to propose and to prove the feasibility of the proposal. Maybe some 
ideas are similar but the code itself has no relationship with other projects.

BTW:  I am the colleague of Huanggaoyang, we work together.

BR / Jianhua

From: lng-odp [mailto:[email protected]] On Behalf Of Mike Holmes
Sent: 2015年10月16日 20:00
To: huanggaoyang (A)
Cc: Wangjun (N); Fengqingqing; Chenweilong; Sunzhaonian; lng-odp
Subject: Re: [lng-odp] [PATCH][PATCHv1 0/5] helper: add table module

For code we have migrated from another project, we should put in the commit ID 
it came from in our commit message so that we can trace it back.
They may they fix bugs etc and it would be good to understand how our code 
relates.
Mike

On 16 October 2015 at 05:35, huanggaoyang 
<[email protected]<mailto:[email protected]>> wrote:
v1: helper: added base api of Table
            added two impls of Table:hash table & linear table

Table designed to provide a standard interface to implement different
 types of tables for data processing.
 Use case:
 Table is publicly used in the following scenarios in data plane:
 ARP Table, IP Routing Table, MAC address filtering, ACL, interworking etc.
 The features of the "table":
 1) All these different types of "table" have the common operations:
     Create a table, destroy a table, add an entry (key/value pairs),
     delete an entry, and lookup the value via the key.
     Usually these operations are software based, but also can be
     hardware accelerated if cost, power consumption are not the concern
     in your platform.
 2) Different types of "table" can use different algorithms to
     add/delete/lookup the entry.

huanggaoyang (5):
  helper: add table api
  helper: table: add impl of hashtable
  helper: table: add impl of lineartable
  helper: add the new source files to Makefile
  helper: test add a simple test case for hashtable

 helper/Makefile.am                |   5 +-
 helper/hashtable.c                | 346 ++++++++++++++++++++++++++++++++++++++
 helper/include/odp/helper/table.h | 199 ++++++++++++++++++++++
 helper/lineartable.c              | 211 +++++++++++++++++++++++
 helper/odph_hashtable.h           |  42 +++++
 helper/odph_lineartable.h         |  42 +++++
 helper/odph_list_internal.h       |  85 ++++++++++
 helper/test/Makefile.am           |   5 +-
 helper/test/odp_table.c           | 134 +++++++++++++++
 9 files changed, 1067 insertions(+), 2 deletions(-)
 create mode 100644 helper/hashtable.c
 create mode 100644 helper/include/odp/helper/table.h
 create mode 100644 helper/lineartable.c
 create mode 100644 helper/odph_hashtable.h
 create mode 100644 helper/odph_lineartable.h
 create mode 100644 helper/odph_list_internal.h
 create mode 100644 helper/test/odp_table.c

--
1.9.1


_______________________________________________
lng-odp mailing list
[email protected]<mailto:[email protected]>
https://lists.linaro.org/mailman/listinfo/lng-odp



--
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org<http://www.linaro.org/> │ Open source software for ARM SoCs

_______________________________________________
lng-odp mailing list
[email protected]
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to