mbox series

[v5,0/2] add unit test cases for mbuf library

Message ID 1567174395-27432-1-git-send-email-lavanyax.govindarajan@intel.com (mailing list archive)
Headers
Series add unit test cases for mbuf library |

Message

Lavanya Govindarajan Aug. 30, 2019, 2:13 p.m. UTC 
  This patchset contains unit testcases added to increase the
functional and decision coverage for the library functions
defined in rte_mbuf.h and rte_mbuf.c

1/2: unit test cases added for rte_mbuf.h
2/2: unit test cases added for rte_mbuf.c

Patch 2/2 depends on 1/2

Signed-off-by: Lavanya Govindarajan <lavanyax.govindarajan@intel.com>
Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
Reviewed-by: Reshma Pattan <reshma.pattan@intel.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
---
v5: Made minor coding improvisations as suggested.

v4: Fixed cosmetic errors.

v3: Fixed build issue for environment FD30-64.
Improvised error log messages.

v2: Addressed v1's comments.
Removed rte prefix from the test function names.
Fixed comments given for the below test functions
test_mbuf_validate_tx_offload
test_neg_pktmbuf_alloc_bulk
test_pktmbuf_read_from_offset
test_pktmbuf_read_from_chain
---

Lavanya Govindarajan (1):
  app/test: add unit test cases for mbuf library APIs

Pallantla Poornima (1):
  app/test: add unit test for mbuf flag names

 app/test/test_mbuf.c | 1030 +++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 1027 insertions(+), 3 deletions(-)

Comments

David Marchand Oct. 24, 2019, 7:54 a.m. UTC | #1 
On Fri, Aug 30, 2019 at 4:14 PM Lavanya Govindarajan
<lavanyax.govindarajan@intel.com> wrote:
>
> This patchset contains unit testcases added to increase the
> functional and decision coverage for the library functions
> defined in rte_mbuf.h and rte_mbuf.c
>
> 1/2: unit test cases added for rte_mbuf.h
> 2/2: unit test cases added for rte_mbuf.c
>
> Patch 2/2 depends on 1/2
>
> Signed-off-by: Lavanya Govindarajan <lavanyax.govindarajan@intel.com>
> Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
> Reviewed-by: Reshma Pattan <reshma.pattan@intel.com>
> Acked-by: Olivier Matz <olivier.matz@6wind.com>

Applied, thanks.


--
David Marchand