Looks like I somehow missed adding default reject policies to commands in families using split Netlink ops. I realized this randomly trying to dump page pools for a specific device and always getting all of them back. The per-device dump is simply not implemented so the request should have been rejected. Patch 2 is the real change, the rest is just accompaniment.
v2: - add patch 1 to avoid breaking devlink - add a lot more tests v1: https://lore.kernel.org/[email protected] Jakub Kicinski (4): genetlink: use maxattr of 0 for the reject policy genetlink: apply reject policy for split ops on the dispatch path selftests: net: make sure that Netlink rejects unknown attrs in dump selftests: net: add test for Netlink policy dumps tools/testing/selftests/net/Makefile | 1 + net/netlink/genetlink.c | 20 +-- net/netlink/policy.c | 4 +- .../testing/selftests/net/lib/py/__init__.py | 5 +- tools/testing/selftests/net/lib/py/ynl.py | 10 +- tools/testing/selftests/net/nl_netdev.py | 32 ++++- tools/testing/selftests/net/nl_nlctrl.py | 135 ++++++++++++++++++ 7 files changed, 186 insertions(+), 21 deletions(-) create mode 100755 tools/testing/selftests/net/nl_nlctrl.py -- 2.53.0
