On Wed, Jan 04, 2023 at 01:17:32PM +0100, Magnus Karlsson wrote: > From: Magnus Karlsson <magnus.karlsson@xxxxxxxxx> > > Submit the correct number of frames in the function > xsk_populate_fill_ring(). For the tests that set the flag > use_addr_for_fill, uninitialized buffers were sent to the fill ring > following the correct ones. This has no impact on the tests, since > they only use the ones that were initialized. But for correctnes, this tiny nit: missing second 's' on correctness > should be fixed. > > Signed-off-by: Magnus Karlsson <magnus.karlsson@xxxxxxxxx> > --- > tools/testing/selftests/bpf/xskxceiver.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/bpf/xskxceiver.c b/tools/testing/selftests/bpf/xskxceiver.c > index 2ff43b22180f..a239e975ab66 100644 > --- a/tools/testing/selftests/bpf/xskxceiver.c > +++ b/tools/testing/selftests/bpf/xskxceiver.c > @@ -1272,7 +1272,7 @@ static void xsk_populate_fill_ring(struct xsk_umem_info *umem, struct pkt_stream > > *xsk_ring_prod__fill_addr(&umem->fq, idx++) = addr; > } > - xsk_ring_prod__submit(&umem->fq, buffers_to_fill); > + xsk_ring_prod__submit(&umem->fq, i); > } > > static void thread_common_ops(struct test_spec *test, struct ifobject *ifobject) > -- > 2.34.1 >
