On 4/13/19 10:20 AM, Jiri Pirko wrote: > From: Jiri Pirko <j...@mellanox.com> > > Currently there is one devlink instance created per network namespace. > That is quite odd considering the fact that devlink instance should > represent an ASIC. The following patches are going to move the devlink > instance even more down to a bus device, but until then, have one > devlink instance per netdevsim instance. Struct nsim_devlink is > introduced to hold fib setting. > > Signed-off-by: Jiri Pirko <j...@mellanox.com> > --- > drivers/net/netdevsim/devlink.c | 179 +++++++++++------------------- > drivers/net/netdevsim/fib.c | 103 +++++++---------- > drivers/net/netdevsim/netdev.c | 27 ++--- > drivers/net/netdevsim/netdevsim.h | 18 +-- > 4 files changed, 125 insertions(+), 202 deletions(-)
you are making a lot of changes to how fib events are handled and not explaining what you are doing.