Re: [PATCH v2] lib/crc16_kunit.c: add KUnit tests for crc16

2024-10-04 Thread Vinicius Peixoto
On 10/4/24 15:40, kernel test robot wrote: Hi Vinicius, kernel test robot noticed the following build warnings: [auto build test WARNING on 9852d85ec9d492ebef56dc5f229416c925758edc] url: https://github.com/intel-lab-lkp/linux/commits/Vinicius-Peixoto/lib-crc16_kunit-c-add-KUnit-tests-for

Re: [PATCH 0/1] Add KUnit tests for lib/crc16.c

2024-09-29 Thread Vinicius Peixoto
Hi David, On 9/25/24 08:26, David Laight wrote: From: Vinicius Peixoto Sent: 23 September 2024 00:27 Hi all, This patch was developed during a hackathon organized by LKCAMP [1], with the objective of writing KUnit tests, both to introduce people to the kernel development process and to learn

Re: [PATCH 0/1] Add KUnit tests for lib/crc16.c

2024-09-29 Thread Vinicius Peixoto
Hi David, On 9/26/24 13:21, David Laight wrote: ... The checksums for the randomly-generated test cases were calculated using a reference implementation [1] and this test compares them against the values yielded by the kernel's implementation. I'd just use a naïve implementation - doesn't rea

[PATCH v2] lib/crc16_kunit.c: add KUnit tests for crc16

2024-10-03 Thread Vinicius Peixoto
ments. Signed-off-by: Vinicius Peixoto Co-developed-by: Enzo Bertoloti Signed-off-by: Enzo Bertoloti Co-developed-by: Fabricio Gasperin Signed-off-by: Fabricio Gasperin Suggested-by: David Laight --- Hi all, This patch was developed during a hackathon organized by LKCAMP [1], with the objecti

[PATCH v3] lib/crc16_kunit.c: add KUnit tests for crc16

2024-10-12 Thread Vinicius Peixoto
ments. Signed-off-by: Vinicius Peixoto Co-developed-by: Enzo Bertoloti Signed-off-by: Enzo Bertoloti Co-developed-by: Fabricio Gasperin Signed-off-by: Fabricio Gasperin Suggested-by: David Laight --- This patch was developed during a hackathon organized by LKCAMP [1], with the objective of wr

[PATCH 0/1] Add KUnit tests for lib/crc16.c

2024-09-22 Thread Vinicius Peixoto
randomly-generated test data that we use to validate the kernel's implementation of CRC-16. We would really appreciate any feedback/suggestions on how to improve this. Thanks! :-) Vinicius Peixoto (1): lib/crc16_kunit.c: add KUnit tests for crc16 lib/Kconfig.debug | 8 + lib/Mak

[PATCH 1/1] lib/crc16_kunit.c: add KUnit tests for crc16

2024-09-22 Thread Vinicius Peixoto
mance measurements. Signed-off-by: Vinicius Peixoto Co-developed-by: Enzo Bertoloti Signed-off-by: Enzo Bertoloti Co-developed-by: Fabricio Gasperin Signed-off-by: Fabricio Gasperin --- lib/Kconfig.debug | 8 + lib/Makefile | 1 + lib/crc16_kunit.c

Re: [PATCH v2 0/1] Add KUnit tests for kfifo

2024-09-21 Thread Vinicius Peixoto
Hi all, On 9/3/24 18:36, Diego Vieira wrote: Hi all, This is part of a hackathon organized by LKCAMP [1], focused on writing tests using KUnit. We reached out a while ago asking for advice on what would be a useful contribution [2] and ended up choosing data structures that did not yet have tes

Re: [PATCH 0/1] Add KUnit tests for lib/crc16.c

2024-09-24 Thread Vinicius Peixoto
Hi André, On 9/24/24 6:33 PM, André Almeida wrote: Hey! On 9/23/24 01:26, Vinicius Peixoto wrote: Hi all, This patch was developed during a hackathon organized by LKCAMP [1], with the objective of writing KUnit tests, both to introduce people to the kernel development process and to learn