Re: [PATCH net-next v6 00/12] selftests: ncdevmem: Add ncdevmem to ksft

From: Mina Almasry
Date: Thu Oct 31 2024 - 13:20:48 EST


On Thu, Oct 31, 2024 at 9:45 AM Mina Almasry <almasrymina@xxxxxxxxxx> wrote:
>
...
>
> Sorry, 2 issues testing this series:
>
...
>
> 2. Validation is now broken:
>

Validation is re-fixed with this diff:

diff --git a/tools/testing/selftests/drivers/net/hw/ncdevmem.c
b/tools/testing/selftests/drivers/net/hw/ncdevmem.c
index 692c189bb5cc..494ae66d8abf 100644
--- a/tools/testing/selftests/drivers/net/hw/ncdevmem.c
+++ b/tools/testing/selftests/drivers/net/hw/ncdevmem.c
@@ -568,8 +568,7 @@ int do_server(struct memory_buffer *mem)

if (do_validation)
validate_buffer(
- ((unsigned char *)tmp_mem) +
- dmabuf_cmsg->frag_offset,
+ ((unsigned char *)tmp_mem),
dmabuf_cmsg->frag_size);
else
print_nonzero_bytes(tmp_mem,
dmabuf_cmsg->frag_size);

Since memcpy_from_device copies to the beginning of tmp_mem, then the
beginning tmp_mem should be passed to the validation.

--
Thanks,
Mina