Re: [lng-odp] [PATCH API-NEXT v2] api: comp: compression API set and implementation

2017-10-02 Thread Github ODP bot
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page: include/odp/api/spec/comp.h @@ -21,8 +21,8 @@ extern "C" { Comment: API changes really should be a separate commit. > Bill Fischofer(Bill-Fischofer-Linaro) wrote: > This should not be part of this PR. Neither should the extraneo

Re: [lng-odp] [PATCH API-NEXT v2] api: comp: compression API set and implementation

2017-10-02 Thread Github ODP bot
Bill Fischofer(Bill-Fischofer-Linaro) replied on github web page: .travis.yml line 19 @@ -251,11 +251,17 @@ jobs: fi - export PATH=$HOME/doxygen-install/bin:$PATH - # doxygen does not trap on warnings, check for the

[lng-odp] [Bug 3262] Missing doxygen detected by Travis

2017-10-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3262 Bill Fischofer changed: What|Removed |Added Status|IN_PROGRESS |RESOLVED Resolution|---

[lng-odp] [Bug 3241] codecov: _odp_packet_cmp_data is not covered

2017-10-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3241 Bill Fischofer changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[lng-odp] [Linaro/odp] fdc44d: linux-gen: drop unused _odp_packet_cmp_data() func...

2017-10-02 Thread GitHub
Branch: refs/heads/api-next Home: https://github.com/Linaro/odp Commit: fdc44dd4322f624c2a5d8c0be5306f7c45364520 https://github.com/Linaro/odp/commit/fdc44dd4322f624c2a5d8c0be5306f7c45364520 Author: Dmitry Eremin-Solenikov Date: 2017-09-26 (Tue, 26 Sep 2017) Changed paths:

[lng-odp] [Bug 3241] codecov: _odp_packet_cmp_data is not covered

2017-10-02 Thread bugzilla-daemon
https://bugs.linaro.org/show_bug.cgi?id=3241 --- Comment #12 from Maxim Uvarov --- https://github.com/Linaro/odp/commit/fdc44dd4322f624c2a5d8c0be5306f7c45364520 refs/heads/api-next 2017-09-26T21:05:04+03:00 Dmitry Eremin-Solenikov dmitry.ereminsoleni...@linaro.org linux-gen: drop unused _odp_pack

[lng-odp] [Linaro/odp] b4d17b: tests: fix validation tests being skipped by defau...

2017-10-02 Thread GitHub
Branch: refs/heads/master Home: https://github.com/Linaro/odp Commit: b4d17b1f6807cd980a1b2dd30573f17677ea371b https://github.com/Linaro/odp/commit/b4d17b1f6807cd980a1b2dd30573f17677ea371b Author: Dmitry Eremin-Solenikov Date: 2017-10-02 (Mon, 02 Oct 2017) Changed paths:

[lng-odp] [PATCH API-NEXT v1 1/1] linux-gen: pktio: add MAC address set function

2017-10-02 Thread Github ODP bot
From: Bogdan Pricope Signed-off-by: Bogdan Pricope --- /** Email created from pull request 203 (bogdanPricope:set_mac_pr) ** https://github.com/Linaro/odp/pull/203 ** Patch: https://github.com/Linaro/odp/pull/203.patch ** Base sha: e04e5f90df69e3031622b77fb5273b85d47eb966 ** Merge commit sha

[lng-odp] [PATCH API-NEXT v1 0/1] linux-gen: pktio: add MAC address set function

2017-10-02 Thread Github ODP bot
Signed-off-by: Bogdan Pricope bogdan.pric...@linaro.org github /** Email created from pull request 203 (bogdanPricope:set_mac_pr) ** https://github.com/Linaro/odp/pull/203 ** Patch: https://github.com/Linaro/odp/pull/203.patch ** Base sha: e04e5f90df69e30

[lng-odp] Code review and discussion on mailing list only

2017-10-02 Thread Savolainen, Petri (Nokia - FI/Espoo)
Hi, Let's stop using Github for code review and design discussions. Mailing list is far more flexible and robust for human-to-human interaction than a web-based tool. We can continue to use Github for sending/bookkeeping pull requests (and run Travis on those), but all discussions should happen

Re: [lng-odp] [PATCH] Fix doc errors

2017-10-02 Thread Bill Fischofer
On Mon, Oct 2, 2017 at 7:05 AM, Szymon Sliwa wrote: > odp_pull_head() -> odp_packet_pull_head() > odp_push_head() -> odp_packet_push_head() > > and paragraph reformat to comply with the > 80 character line limit > > Signed-off-by: Szymon Sliwa > Reviewed-by: Bill Fischofer > --- > doc/users

[lng-odp] [PATCH] Fix doc errors

2017-10-02 Thread Szymon Sliwa
odp_pull_head() -> odp_packet_pull_head() odp_push_head() -> odp_packet_push_head() and paragraph reformat to comply with the 80 character line limit Signed-off-by: Szymon Sliwa --- doc/users-guide/users-guide-packet.adoc | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions

Re: [lng-odp] odp_pktin_queue_param_t - classifier_enable & num_queues

2017-10-02 Thread Bill Fischofer
Thanks Liron. This appears to be a bug in the implementation. It's only an error for num_queues to be zero if hash_enabled is set. On Mon, Oct 2, 2017 at 6:02 AM, Liron Himi wrote: > Hi, > > According to the API if 'classifier_enable' is enabled than 'num_queues' > and 'queue_param' should be ig

[lng-odp] odp_pktin_queue_param_t - classifier_enable & num_queues

2017-10-02 Thread Liron Himi
Hi, According to the API if 'classifier_enable' is enabled than 'num_queues' and 'queue_param' should be ignored. But looking at linux-generic implementation those parameters are always being checked regardless of 'classifier_enable' value. 'classifier_enable' is actually ignore at linux-generic