mbox

[dpdk-dev,pull-request] next-pipeline 17.11 pre-rc1

Message ID 1507732847-100455-1-git-send-email-cristian.dumitrescu@intel.com (mailing list archive)
State Not Applicable, archived
Headers

Pull-request

http://dpdk.org/git/next/dpdk-next-pipeline

Message

Cristian Dumitrescu Oct. 11, 2017, 2:40 p.m. UTC 
  The following changes since commit d65b3b1668f2037745407c3909c43d85a18692a6:

  vhost: fix false-positive warning from clang 5 (2017-10-11 13:56:34 +0200)

are available in the git repository at:

  http://dpdk.org/git/next/dpdk-next-pipeline 

for you to fetch changes up to 5ba29c202060c8edadbc36d8c9c19be24bd95cac:

  deprecation: removed the librte_table notice (2017-10-11 15:07:45 +0100)

----------------------------------------------------------------
Cristian Dumitrescu (8):
      initial commit
      removing startup file
      Merge branch 'master' of dpdk.org:dpdk-next-pipeline
      table: add key mask for hash tables
      test: update due to api changes in librte_table
      test-pipeline: update due to api changes in librte_table
      ip_pipeline: update due to api changes in librte_table
      deprecation: removed the librte_table notice

 doc/guides/rel_notes/deprecation.rst               |   6 -
 examples/ip_pipeline/pipeline/hash_func.h          | 178 +++--
 .../pipeline/pipeline_flow_classification.c        |  12 +-
 .../pipeline/pipeline_flow_classification_be.c     |  51 +-
 .../ip_pipeline/pipeline/pipeline_passthrough_be.c |  18 +-
 .../ip_pipeline/pipeline/pipeline_routing_be.c     |  18 +-
 lib/librte_table/rte_table_hash.h                  | 306 +--------
 lib/librte_table/rte_table_hash_cuckoo.c           | 205 +++---
 lib/librte_table/rte_table_hash_ext.c              | 417 ++++--------
 lib/librte_table/rte_table_hash_key16.c            | 749 ++++++---------------
 lib/librte_table/rte_table_hash_key32.c            | 435 +++++++-----
 lib/librte_table/rte_table_hash_key8.c             | 715 ++++++--------------
 lib/librte_table/rte_table_hash_lru.c              | 513 ++++++--------
 lib/librte_table/rte_table_version.map             |  23 +-
 test/test-pipeline/main.h                          |   5 +-
 test/test-pipeline/pipeline_hash.c                 | 107 +--
 test/test/test_table.c                             |   1 +
 test/test/test_table.h                             |   3 +-
 test/test/test_table_combined.c                    | 140 ++--
 test/test/test_table_tables.c                      | 148 ++--
 20 files changed, 1444 insertions(+), 2606 deletions(-)

Comments

Thomas Monjalon Oct. 12, 2017, 5:10 p.m. UTC | #1 
Hi,

11/10/2017 16:40, Cristian Dumitrescu:
> The following changes since commit d65b3b1668f2037745407c3909c43d85a18692a6:
> 
>   vhost: fix false-positive warning from clang 5 (2017-10-11 13:56:34 +0200)
> 
> are available in the git repository at:
> 
>   http://dpdk.org/git/next/dpdk-next-pipeline 
> 
> for you to fetch changes up to 5ba29c202060c8edadbc36d8c9c19be24bd95cac:
> 
>   deprecation: removed the librte_table notice (2017-10-11 15:07:45 +0100)
> 
> ----------------------------------------------------------------
> Cristian Dumitrescu (8):
>       initial commit
>       removing startup file
>       Merge branch 'master' of dpdk.org:dpdk-next-pipeline
>       table: add key mask for hash tables
>       test: update due to api changes in librte_table
>       test-pipeline: update due to api changes in librte_table
>       ip_pipeline: update due to api changes in librte_table
>       deprecation: removed the librte_table notice

It cannot be integrated in RC1 because we must be able to compile
between each commit (to avoid breaking git bisect).

The first commit is very big and does several things.
Maybe you can split it and update the applications accordingly.

I have seen also that the .map file has some issues (blocks inheritance
and indents).