On Fri, Dec 13, 2024 at 07:22:42AM -0800, Jakub Kicinski wrote:
> recv_size parameter allows constraining the buffer size for dumps.
> It's useful in testing kernel handling of dump continuation,
> IOW testing dumps which span multiple skbs.
>
> Let the tests set this parameter when initializing the YNL family.
> Keep the normal default, we don't want tests to unintentionally
> behave very differently than normal code.
>
> Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx>
> ---
> CC: shuah@xxxxxxxxxx
> CC: jiri@xxxxxxxxxxx
> CC: petrm@xxxxxxxxxx
> CC: linux-kselftest@xxxxxxxxxxxxxxx
> ---
> tools/testing/selftests/net/lib/py/ynl.py | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/tools/testing/selftests/net/lib/py/ynl.py b/tools/testing/selftests/net/lib/py/ynl.py
> index a0d689d58c57..076a7e8dc3eb 100644
> --- a/tools/testing/selftests/net/lib/py/ynl.py
> +++ b/tools/testing/selftests/net/lib/py/ynl.py
> @@ -32,23 +32,23 @@ from .ksft import ksft_pr, ktap_result
> # Set schema='' to avoid jsonschema validation, it's slow
> #
> class EthtoolFamily(YnlFamily):
> - def __init__(self):
> + def __init__(self, recv_size=0):
> super().__init__((SPEC_PATH / Path('ethtool.yaml')).as_posix(),
> - schema='')
> + schema='', recv_size=recv_size)
>
>
> class RtnlFamily(YnlFamily):
> - def __init__(self):
> + def __init__(self, recv_size=0):
> super().__init__((SPEC_PATH / Path('rt_link.yaml')).as_posix(),
> - schema='')
> + schema='', recv_size=recv_size)
>
>
> class NetdevFamily(YnlFamily):
> - def __init__(self):
> + def __init__(self, recv_size=0):
> super().__init__((SPEC_PATH / Path('netdev.yaml')).as_posix(),
> - schema='')
> + schema='', recv_size=recv_size)
>
> class NetshaperFamily(YnlFamily):
> - def __init__(self):
> + def __init__(self, recv_size=0):
> super().__init__((SPEC_PATH / Path('net_shaper.yaml')).as_posix(),
> - schema='')
> + schema='', recv_size=recv_size)
Reviewed-by: Joe Damato <jdamato@xxxxxxxxxx>
