On 8/13/19 1:14 AM, Jiri Pirko wrote: > Mon, Aug 12, 2019 at 05:28:02PM CEST, da...@davemloft.net wrote: >> From: Jiri Pirko <j...@resnulli.us> >> Date: Mon, 12 Aug 2019 10:36:35 +0200 >> >>> I understand it with real devices, but dummy testing device, who's >>> purpose is just to test API. Why? >> >> Because you'll break all of the wonderful testing infrastructure >> people like David have created. > > Are you referring to selftests? There is no such test there :(
I have one now and will be submitting it after net merges with net-next. > But if it would be, could implement the limitations > properly (like using cgroups), change the tests and remove this > code from netdevsim? The intent of this code and test is to have a s/w model similar to how mlxsw works - responding to notifiers and deciding to reject a change. You are currently adding (or trying to) more devlink based s/w tests, so you must see the value of netdevsim as a source of testing.