net: marvell: prestera: Add neighbour cache accounting
authorYevhen Orlov <yevhen.orlov@plvision.eu>
Sat, 1 Oct 2022 09:34:16 +0000 (12:34 +0300)
committerJakub Kicinski <kuba@kernel.org>
Tue, 4 Oct 2022 00:14:53 +0000 (17:14 -0700)
Move forward and use new PRESTERA_FIB_TYPE_UC_NH to provide basic
nexthop routes support.
Provide deinitialization sequence for all created router objects.

Limitations:
- Only "local" and "main" tables supported
- Only generic interfaces supported for router (no bridges or vlans)

Co-developed-by: Taras Chornyi <tchornyi@marvell.com>
Signed-off-by: Taras Chornyi <tchornyi@marvell.com>
Co-developed-by: Oleksandr Mazur <oleksandr.mazur@plvision.eu>
Signed-off-by: Oleksandr Mazur <oleksandr.mazur@plvision.eu>
Signed-off-by: Yevhen Orlov <yevhen.orlov@plvision.eu>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/marvell/prestera/prestera.h
drivers/net/ethernet/marvell/prestera/prestera_router.c

index 2f2f80e7e358aebf5b6847d8c711a17131904390..540a36069b79bd7579215fdfcbdb3a6262f4db8c 100644 (file)
@@ -316,6 +316,7 @@ struct prestera_router {
        struct rhashtable nh_neigh_ht;
        struct rhashtable nexthop_group_ht;
        struct rhashtable fib_ht;
+       struct rhashtable kern_neigh_cache_ht;
        struct rhashtable kern_fib_cache_ht;
        struct notifier_block inetaddr_nb;
        struct notifier_block inetaddr_valid_nb;
index d31dd1fe66330734a2080d01a2376d8ff4593909..af7d24390d2e050c7f529d7d325837bae956a2ba 100644 (file)
@@ -8,11 +8,30 @@
 #include <net/switchdev.h>
 #include <linux/rhashtable.h>
 #include <net/nexthop.h>
+#include <net/arp.h>
+#include <linux/if_vlan.h>
+#include <linux/if_macvlan.h>
 #include <net/netevent.h>
 
 #include "prestera.h"
 #include "prestera_router_hw.h"
 
+struct prestera_kern_neigh_cache_key {
+       struct prestera_ip_addr addr;
+       struct net_device *dev;
+};
+
+struct prestera_kern_neigh_cache {
+       struct prestera_kern_neigh_cache_key key;
+       struct rhash_head ht_node;
+       struct list_head kern_fib_cache_list;
+       /* Hold prepared nh_neigh info if is in_kernel */
+       struct prestera_neigh_info nh_neigh_info;
+       /* Indicate if neighbour is reachable by direct route */
+       bool reachable;
+       /* Lock cache if neigh is present in kernel */
+       bool in_kernel;
+};
 struct prestera_kern_fib_cache_key {
        struct prestera_ip_addr addr;
        u32 prefix_len;
@@ -25,9 +44,15 @@ struct prestera_kern_fib_cache {
        struct {
                struct prestera_fib_key fib_key;
                enum prestera_fib_type fib_type;
+               struct prestera_nexthop_group_key nh_grp_key;
        } lpm_info; /* hold prepared lpm info */
        /* Indicate if route is not overlapped by another table */
        struct rhash_head ht_node; /* node of prestera_router */
+       struct prestera_kern_neigh_cache_head {
+               struct prestera_kern_fib_cache *this;
+               struct list_head head;
+               struct prestera_kern_neigh_cache *n_cache;
+       } kern_neigh_cache_head[PRESTERA_NHGR_SIZE_MAX];
        union {
                struct fib_notifier_info info; /* point to any of 4/6 */
                struct fib_entry_notifier_info fen4_info;
@@ -35,6 +60,13 @@ struct prestera_kern_fib_cache {
        bool reachable;
 };
 
+static const struct rhashtable_params __prestera_kern_neigh_cache_ht_params = {
+       .key_offset  = offsetof(struct prestera_kern_neigh_cache, key),
+       .head_offset = offsetof(struct prestera_kern_neigh_cache, ht_node),
+       .key_len     = sizeof(struct prestera_kern_neigh_cache_key),
+       .automatic_shrinking = true,
+};
+
 static const struct rhashtable_params __prestera_kern_fib_cache_ht_params = {
        .key_offset  = offsetof(struct prestera_kern_fib_cache, key),
        .head_offset = offsetof(struct prestera_kern_fib_cache, ht_node),
@@ -67,6 +99,268 @@ prestera_util_fen_info2fib_cache_key(struct fib_notifier_info *info,
        key->kern_tb_id = fen_info->tb_id;
 }
 
+static int prestera_util_nhc2nc_key(struct prestera_switch *sw,
+                                   struct fib_nh_common *nhc,
+                                   struct prestera_kern_neigh_cache_key *nk)
+{
+       memset(nk, 0, sizeof(*nk));
+       if (nhc->nhc_gw_family == AF_INET) {
+               nk->addr.v = PRESTERA_IPV4;
+               nk->addr.u.ipv4 = nhc->nhc_gw.ipv4;
+       } else {
+               nk->addr.v = PRESTERA_IPV6;
+               nk->addr.u.ipv6 = nhc->nhc_gw.ipv6;
+       }
+
+       nk->dev = nhc->nhc_dev;
+       return 0;
+}
+
+static void
+prestera_util_nc_key2nh_key(struct prestera_kern_neigh_cache_key *ck,
+                           struct prestera_nh_neigh_key *nk)
+{
+       memset(nk, 0, sizeof(*nk));
+       nk->addr = ck->addr;
+       nk->rif = (void *)ck->dev;
+}
+
+static bool
+prestera_util_nhc_eq_n_cache_key(struct prestera_switch *sw,
+                                struct fib_nh_common *nhc,
+                                struct prestera_kern_neigh_cache_key *nk)
+{
+       struct prestera_kern_neigh_cache_key tk;
+       int err;
+
+       err = prestera_util_nhc2nc_key(sw, nhc, &tk);
+       if (err)
+               return false;
+
+       if (memcmp(&tk, nk, sizeof(tk)))
+               return false;
+
+       return true;
+}
+
+static int
+prestera_util_neigh2nc_key(struct prestera_switch *sw, struct neighbour *n,
+                          struct prestera_kern_neigh_cache_key *key)
+{
+       memset(key, 0, sizeof(*key));
+       if (n->tbl->family == AF_INET) {
+               key->addr.v = PRESTERA_IPV4;
+               key->addr.u.ipv4 = *(__be32 *)n->primary_key;
+       } else {
+               return -ENOENT;
+       }
+
+       key->dev = n->dev;
+
+       return 0;
+}
+
+static bool __prestera_fi_is_direct(struct fib_info *fi)
+{
+       struct fib_nh *fib_nh;
+
+       if (fib_info_num_path(fi) == 1) {
+               fib_nh = fib_info_nh(fi, 0);
+               if (fib_nh->fib_nh_gw_family == AF_UNSPEC)
+                       return true;
+       }
+
+       return false;
+}
+
+static bool prestera_fi_is_direct(struct fib_info *fi)
+{
+       if (fi->fib_type != RTN_UNICAST)
+               return false;
+
+       return __prestera_fi_is_direct(fi);
+}
+
+static bool prestera_fi_is_nh(struct fib_info *fi)
+{
+       if (fi->fib_type != RTN_UNICAST)
+               return false;
+
+       return !__prestera_fi_is_direct(fi);
+}
+
+static bool __prestera_fi6_is_direct(struct fib6_info *fi)
+{
+       if (!fi->fib6_nh->nh_common.nhc_gw_family)
+               return true;
+
+       return false;
+}
+
+static bool prestera_fi6_is_direct(struct fib6_info *fi)
+{
+       if (fi->fib6_type != RTN_UNICAST)
+               return false;
+
+       return __prestera_fi6_is_direct(fi);
+}
+
+static bool prestera_fi6_is_nh(struct fib6_info *fi)
+{
+       if (fi->fib6_type != RTN_UNICAST)
+               return false;
+
+       return !__prestera_fi6_is_direct(fi);
+}
+
+static bool prestera_fib_info_is_direct(struct fib_notifier_info *info)
+{
+       struct fib6_entry_notifier_info *fen6_info =
+               container_of(info, struct fib6_entry_notifier_info, info);
+       struct fib_entry_notifier_info *fen_info =
+               container_of(info, struct fib_entry_notifier_info, info);
+
+       if (info->family == AF_INET)
+               return prestera_fi_is_direct(fen_info->fi);
+       else
+               return prestera_fi6_is_direct(fen6_info->rt);
+}
+
+static bool prestera_fib_info_is_nh(struct fib_notifier_info *info)
+{
+       struct fib6_entry_notifier_info *fen6_info =
+               container_of(info, struct fib6_entry_notifier_info, info);
+       struct fib_entry_notifier_info *fen_info =
+               container_of(info, struct fib_entry_notifier_info, info);
+
+       if (info->family == AF_INET)
+               return prestera_fi_is_nh(fen_info->fi);
+       else
+               return prestera_fi6_is_nh(fen6_info->rt);
+}
+
+/* must be called with rcu_read_lock() */
+static int prestera_util_kern_get_route(struct fib_result *res, u32 tb_id,
+                                       __be32 *addr)
+{
+       struct flowi4 fl4;
+
+       /* TODO: walkthrough appropriate tables in kernel
+        * to know if the same prefix exists in several tables
+        */
+       memset(&fl4, 0, sizeof(fl4));
+       fl4.daddr = *addr;
+       return fib_lookup(&init_net, &fl4, res, 0 /* FIB_LOOKUP_NOREF */);
+}
+
+static bool
+__prestera_util_kern_n_is_reachable_v4(u32 tb_id, __be32 *addr,
+                                      struct net_device *dev)
+{
+       struct fib_nh *fib_nh;
+       struct fib_result res;
+       bool reachable;
+
+       reachable = false;
+
+       if (!prestera_util_kern_get_route(&res, tb_id, addr))
+               if (prestera_fi_is_direct(res.fi)) {
+                       fib_nh = fib_info_nh(res.fi, 0);
+                       if (dev == fib_nh->fib_nh_dev)
+                               reachable = true;
+               }
+
+       return reachable;
+}
+
+/* Check if neigh route is reachable */
+static bool
+prestera_util_kern_n_is_reachable(u32 tb_id,
+                                 struct prestera_ip_addr *addr,
+                                 struct net_device *dev)
+{
+       if (addr->v == PRESTERA_IPV4)
+               return __prestera_util_kern_n_is_reachable_v4(tb_id,
+                                                             &addr->u.ipv4,
+                                                             dev);
+       else
+               return false;
+}
+
+static void prestera_util_kern_set_neigh_offload(struct neighbour *n,
+                                                bool offloaded)
+{
+       if (offloaded)
+               n->flags |= NTF_OFFLOADED;
+       else
+               n->flags &= ~NTF_OFFLOADED;
+}
+
+static void
+prestera_util_kern_set_nh_offload(struct fib_nh_common *nhc, bool offloaded, bool trap)
+{
+               if (offloaded)
+                       nhc->nhc_flags |= RTNH_F_OFFLOAD;
+               else
+                       nhc->nhc_flags &= ~RTNH_F_OFFLOAD;
+
+               if (trap)
+                       nhc->nhc_flags |= RTNH_F_TRAP;
+               else
+                       nhc->nhc_flags &= ~RTNH_F_TRAP;
+}
+
+static struct fib_nh_common *
+prestera_kern_fib_info_nhc(struct fib_notifier_info *info, int n)
+{
+       struct fib6_entry_notifier_info *fen6_info;
+       struct fib_entry_notifier_info *fen4_info;
+       struct fib6_info *iter;
+
+       if (info->family == AF_INET) {
+               fen4_info = container_of(info, struct fib_entry_notifier_info,
+                                        info);
+               return &fib_info_nh(fen4_info->fi, n)->nh_common;
+       } else if (info->family == AF_INET6) {
+               fen6_info = container_of(info, struct fib6_entry_notifier_info,
+                                        info);
+               if (!n)
+                       return &fen6_info->rt->fib6_nh->nh_common;
+
+               list_for_each_entry(iter, &fen6_info->rt->fib6_siblings,
+                                   fib6_siblings) {
+                       if (!--n)
+                               return &iter->fib6_nh->nh_common;
+               }
+       }
+
+       /* if family is incorrect - than upper functions has BUG */
+       /* if doesn't find requested index - there is alsi bug, because
+        * valid index must be produced by nhs, which checks list length
+        */
+       WARN(1, "Invalid parameters passed to %s n=%d i=%p",
+            __func__, n, info);
+       return NULL;
+}
+
+static int prestera_kern_fib_info_nhs(struct fib_notifier_info *info)
+{
+       struct fib6_entry_notifier_info *fen6_info;
+       struct fib_entry_notifier_info *fen4_info;
+
+       if (info->family == AF_INET) {
+               fen4_info = container_of(info, struct fib_entry_notifier_info,
+                                        info);
+               return fib_info_num_path(fen4_info->fi);
+       } else if (info->family == AF_INET6) {
+               fen6_info = container_of(info, struct fib6_entry_notifier_info,
+                                        info);
+               return fen6_info->rt->fib6_nsiblings + 1;
+       }
+
+       return 0;
+}
+
 static unsigned char
 prestera_kern_fib_info_type(struct fib_notifier_info *info)
 {
@@ -89,6 +383,153 @@ prestera_kern_fib_info_type(struct fib_notifier_info *info)
        return RTN_UNSPEC;
 }
 
+/* Decided, that uc_nh route with key==nh is obviously neighbour route */
+static bool
+prestera_fib_node_util_is_neighbour(struct prestera_fib_node *fib_node)
+{
+       if (fib_node->info.type != PRESTERA_FIB_TYPE_UC_NH)
+               return false;
+
+       if (fib_node->info.nh_grp->nh_neigh_head[1].neigh)
+               return false;
+
+       if (!fib_node->info.nh_grp->nh_neigh_head[0].neigh)
+               return false;
+
+       if (memcmp(&fib_node->info.nh_grp->nh_neigh_head[0].neigh->key.addr,
+                  &fib_node->key.addr, sizeof(struct prestera_ip_addr)))
+               return false;
+
+       return true;
+}
+
+static int prestera_dev_if_type(const struct net_device *dev)
+{
+       struct macvlan_dev *vlan;
+
+       if (is_vlan_dev(dev) &&
+           netif_is_bridge_master(vlan_dev_real_dev(dev))) {
+               return PRESTERA_IF_VID_E;
+       } else if (netif_is_bridge_master(dev)) {
+               return PRESTERA_IF_VID_E;
+       } else if (netif_is_lag_master(dev)) {
+               return PRESTERA_IF_LAG_E;
+       } else if (netif_is_macvlan(dev)) {
+               vlan = netdev_priv(dev);
+               return prestera_dev_if_type(vlan->lowerdev);
+       } else {
+               return PRESTERA_IF_PORT_E;
+       }
+}
+
+static int
+prestera_neigh_iface_init(struct prestera_switch *sw,
+                         struct prestera_iface *iface,
+                         struct neighbour *n)
+{
+       struct prestera_port *port;
+
+       iface->vlan_id = 0; /* TODO: vlan egress */
+       iface->type = prestera_dev_if_type(n->dev);
+       if (iface->type != PRESTERA_IF_PORT_E)
+               return -EINVAL;
+
+       if (!prestera_netdev_check(n->dev))
+               return -EINVAL;
+
+       port = netdev_priv(n->dev);
+       iface->dev_port.hw_dev_num = port->dev_id;
+       iface->dev_port.port_num = port->hw_id;
+
+       return 0;
+}
+
+static struct prestera_kern_neigh_cache *
+prestera_kern_neigh_cache_find(struct prestera_switch *sw,
+                              struct prestera_kern_neigh_cache_key *key)
+{
+       struct prestera_kern_neigh_cache *n_cache;
+
+       n_cache =
+        rhashtable_lookup_fast(&sw->router->kern_neigh_cache_ht, key,
+                               __prestera_kern_neigh_cache_ht_params);
+       return IS_ERR(n_cache) ? NULL : n_cache;
+}
+
+static void
+__prestera_kern_neigh_cache_destruct(struct prestera_switch *sw,
+                                    struct prestera_kern_neigh_cache *n_cache)
+{
+       dev_put(n_cache->key.dev);
+}
+
+static void
+__prestera_kern_neigh_cache_destroy(struct prestera_switch *sw,
+                                   struct prestera_kern_neigh_cache *n_cache)
+{
+       rhashtable_remove_fast(&sw->router->kern_neigh_cache_ht,
+                              &n_cache->ht_node,
+                              __prestera_kern_neigh_cache_ht_params);
+       __prestera_kern_neigh_cache_destruct(sw, n_cache);
+       kfree(n_cache);
+}
+
+static struct prestera_kern_neigh_cache *
+__prestera_kern_neigh_cache_create(struct prestera_switch *sw,
+                                  struct prestera_kern_neigh_cache_key *key)
+{
+       struct prestera_kern_neigh_cache *n_cache;
+       int err;
+
+       n_cache = kzalloc(sizeof(*n_cache), GFP_KERNEL);
+       if (!n_cache)
+               goto err_kzalloc;
+
+       memcpy(&n_cache->key, key, sizeof(*key));
+       dev_hold(n_cache->key.dev);
+
+       INIT_LIST_HEAD(&n_cache->kern_fib_cache_list);
+       err = rhashtable_insert_fast(&sw->router->kern_neigh_cache_ht,
+                                    &n_cache->ht_node,
+                                    __prestera_kern_neigh_cache_ht_params);
+       if (err)
+               goto err_ht_insert;
+
+       return n_cache;
+
+err_ht_insert:
+       dev_put(n_cache->key.dev);
+       kfree(n_cache);
+err_kzalloc:
+       return NULL;
+}
+
+static struct prestera_kern_neigh_cache *
+prestera_kern_neigh_cache_get(struct prestera_switch *sw,
+                             struct prestera_kern_neigh_cache_key *key)
+{
+       struct prestera_kern_neigh_cache *n_cache;
+
+       n_cache = prestera_kern_neigh_cache_find(sw, key);
+       if (!n_cache)
+               n_cache = __prestera_kern_neigh_cache_create(sw, key);
+
+       return n_cache;
+}
+
+static struct prestera_kern_neigh_cache *
+prestera_kern_neigh_cache_put(struct prestera_switch *sw,
+                             struct prestera_kern_neigh_cache *n_cache)
+{
+       if (!n_cache->in_kernel &&
+           list_empty(&n_cache->kern_fib_cache_list)) {
+               __prestera_kern_neigh_cache_destroy(sw, n_cache);
+               return NULL;
+       }
+
+       return n_cache;
+}
+
 static struct prestera_kern_fib_cache *
 prestera_kern_fib_cache_find(struct prestera_switch *sw,
                             struct prestera_kern_fib_cache_key *key)
@@ -105,6 +546,17 @@ static void
 __prestera_kern_fib_cache_destruct(struct prestera_switch *sw,
                                   struct prestera_kern_fib_cache *fib_cache)
 {
+       struct prestera_kern_neigh_cache *n_cache;
+       int i;
+
+       for (i = 0; i < PRESTERA_NHGR_SIZE_MAX; i++) {
+               n_cache = fib_cache->kern_neigh_cache_head[i].n_cache;
+               if (n_cache) {
+                       list_del(&fib_cache->kern_neigh_cache_head[i].head);
+                       prestera_kern_neigh_cache_put(sw, n_cache);
+               }
+       }
+
        fib_info_put(fib_cache->fen4_info.fi);
 }
 
@@ -119,6 +571,41 @@ prestera_kern_fib_cache_destroy(struct prestera_switch *sw,
        kfree(fib_cache);
 }
 
+static int
+__prestera_kern_fib_cache_create_nhs(struct prestera_switch *sw,
+                                    struct prestera_kern_fib_cache *fc)
+{
+       struct prestera_kern_neigh_cache_key nc_key;
+       struct prestera_kern_neigh_cache *n_cache;
+       struct fib_nh_common *nhc;
+       int i, nhs, err;
+
+       if (!prestera_fib_info_is_nh(&fc->info))
+               return 0;
+
+       nhs = prestera_kern_fib_info_nhs(&fc->info);
+       if (nhs > PRESTERA_NHGR_SIZE_MAX)
+               return 0;
+
+       for (i = 0; i < nhs; i++) {
+               nhc = prestera_kern_fib_info_nhc(&fc->fen4_info.info, i);
+               err = prestera_util_nhc2nc_key(sw, nhc, &nc_key);
+               if (err)
+                       return 0;
+
+               n_cache = prestera_kern_neigh_cache_get(sw, &nc_key);
+               if (!n_cache)
+                       return 0;
+
+               fc->kern_neigh_cache_head[i].this = fc;
+               fc->kern_neigh_cache_head[i].n_cache = n_cache;
+               list_add(&fc->kern_neigh_cache_head[i].head,
+                        &n_cache->kern_fib_cache_list);
+       }
+
+       return 0;
+}
+
 /* Operations on fi (offload, etc) must be wrapped in utils.
  * This function just create storage.
  */
@@ -146,6 +633,12 @@ prestera_kern_fib_cache_create(struct prestera_switch *sw,
        if (err)
                goto err_ht_insert;
 
+       /* Handle nexthops */
+       err = __prestera_kern_fib_cache_create_nhs(sw, fib_cache);
+       if (err)
+               goto out; /* Not critical */
+
+out:
        return fib_cache;
 
 err_ht_insert:
@@ -155,6 +648,46 @@ err_kzalloc:
        return NULL;
 }
 
+static void
+__prestera_k_arb_fib_nh_offload_set(struct prestera_switch *sw,
+                                   struct prestera_kern_fib_cache *fibc,
+                                   struct prestera_kern_neigh_cache *nc,
+                                   bool offloaded, bool trap)
+{
+       struct fib_nh_common *nhc;
+       int i, nhs;
+
+       nhs = prestera_kern_fib_info_nhs(&fibc->info);
+       for (i = 0; i < nhs; i++) {
+               nhc = prestera_kern_fib_info_nhc(&fibc->info, i);
+               if (!nc) {
+                       prestera_util_kern_set_nh_offload(nhc, offloaded, trap);
+                       continue;
+               }
+
+               if (prestera_util_nhc_eq_n_cache_key(sw, nhc, &nc->key)) {
+                       prestera_util_kern_set_nh_offload(nhc, offloaded, trap);
+                       break;
+               }
+       }
+}
+
+static void
+__prestera_k_arb_n_offload_set(struct prestera_switch *sw,
+                              struct prestera_kern_neigh_cache *nc,
+                              bool offloaded)
+{
+       struct neighbour *n;
+
+       n = neigh_lookup(&arp_tbl, &nc->key.addr.u.ipv4,
+                        nc->key.dev);
+       if (!n)
+               return;
+
+       prestera_util_kern_set_neigh_offload(n, offloaded);
+       neigh_release(n);
+}
+
 static void
 __prestera_k_arb_fib_lpm_offload_set(struct prestera_switch *sw,
                                     struct prestera_kern_fib_cache *fc,
@@ -183,15 +716,187 @@ __prestera_k_arb_fib_lpm_offload_set(struct prestera_switch *sw,
        }
 }
 
+static void
+__prestera_k_arb_n_lpm_set(struct prestera_switch *sw,
+                          struct prestera_kern_neigh_cache *n_cache,
+                          bool enabled)
+{
+       struct prestera_nexthop_group_key nh_grp_key;
+       struct prestera_kern_fib_cache_key fc_key;
+       struct prestera_kern_fib_cache *fib_cache;
+       struct prestera_fib_node *fib_node;
+       struct prestera_fib_key fib_key;
+
+       /* Exception for fc with prefix 32: LPM entry is already used by fib */
+       memset(&fc_key, 0, sizeof(fc_key));
+       fc_key.addr = n_cache->key.addr;
+       fc_key.prefix_len = PRESTERA_IP_ADDR_PLEN(n_cache->key.addr.v);
+       /* But better to use tb_id of route, which pointed to this neighbour. */
+       /* We take it from rif, because rif inconsistent.
+        * Must be separated in_rif and out_rif.
+        * Also note: for each fib pointed to this neigh should be separated
+        *            neigh lpm entry (for each ingress vr)
+        */
+       fc_key.kern_tb_id = l3mdev_fib_table(n_cache->key.dev);
+       fib_cache = prestera_kern_fib_cache_find(sw, &fc_key);
+       memset(&fib_key, 0, sizeof(fib_key));
+       fib_key.addr = n_cache->key.addr;
+       fib_key.prefix_len = PRESTERA_IP_ADDR_PLEN(n_cache->key.addr.v);
+       fib_key.tb_id = prestera_fix_tb_id(fc_key.kern_tb_id);
+       fib_node = prestera_fib_node_find(sw, &fib_key);
+       if (!fib_cache || !fib_cache->reachable) {
+               if (!enabled && fib_node) {
+                       if (prestera_fib_node_util_is_neighbour(fib_node))
+                               prestera_fib_node_destroy(sw, fib_node);
+                       return;
+               }
+       }
+
+       if (enabled && !fib_node) {
+               memset(&nh_grp_key, 0, sizeof(nh_grp_key));
+               prestera_util_nc_key2nh_key(&n_cache->key,
+                                           &nh_grp_key.neigh[0]);
+               fib_node = prestera_fib_node_create(sw, &fib_key,
+                                                   PRESTERA_FIB_TYPE_UC_NH,
+                                                   &nh_grp_key);
+               if (!fib_node)
+                       pr_err("%s failed ip=%pI4n", "prestera_fib_node_create",
+                              &fib_key.addr.u.ipv4);
+               return;
+       }
+}
+
+static void
+__prestera_k_arb_nc_kern_fib_fetch(struct prestera_switch *sw,
+                                  struct prestera_kern_neigh_cache *nc)
+{
+       if (prestera_util_kern_n_is_reachable(l3mdev_fib_table(nc->key.dev),
+                                             &nc->key.addr, nc->key.dev))
+               nc->reachable = true;
+       else
+               nc->reachable = false;
+}
+
+/* Kernel neighbour -> neigh_cache info */
+static void
+__prestera_k_arb_nc_kern_n_fetch(struct prestera_switch *sw,
+                                struct prestera_kern_neigh_cache *nc)
+{
+       struct neighbour *n;
+       int err;
+
+       memset(&nc->nh_neigh_info, 0, sizeof(nc->nh_neigh_info));
+       n = neigh_lookup(&arp_tbl, &nc->key.addr.u.ipv4, nc->key.dev);
+       if (!n)
+               goto out;
+
+       read_lock_bh(&n->lock);
+       if (n->nud_state & NUD_VALID && !n->dead) {
+               err = prestera_neigh_iface_init(sw, &nc->nh_neigh_info.iface,
+                                               n);
+               if (err)
+                       goto n_read_out;
+
+               memcpy(&nc->nh_neigh_info.ha[0], &n->ha[0], ETH_ALEN);
+               nc->nh_neigh_info.connected = true;
+       }
+n_read_out:
+       read_unlock_bh(&n->lock);
+out:
+       nc->in_kernel = nc->nh_neigh_info.connected;
+       if (n)
+               neigh_release(n);
+}
+
+/* neigh_cache info -> lpm update */
+static void
+__prestera_k_arb_nc_apply(struct prestera_switch *sw,
+                         struct prestera_kern_neigh_cache *nc)
+{
+       struct prestera_kern_neigh_cache_head *nhead;
+       struct prestera_nh_neigh_key nh_key;
+       struct prestera_nh_neigh *nh_neigh;
+       int err;
+
+       __prestera_k_arb_n_lpm_set(sw, nc, nc->reachable && nc->in_kernel);
+       __prestera_k_arb_n_offload_set(sw, nc, nc->reachable && nc->in_kernel);
+
+       prestera_util_nc_key2nh_key(&nc->key, &nh_key);
+       nh_neigh = prestera_nh_neigh_find(sw, &nh_key);
+       if (!nh_neigh)
+               goto out;
+
+       /* Do hw update only if something changed to prevent nh flap */
+       if (memcmp(&nc->nh_neigh_info, &nh_neigh->info,
+                  sizeof(nh_neigh->info))) {
+               memcpy(&nh_neigh->info, &nc->nh_neigh_info,
+                      sizeof(nh_neigh->info));
+               err = prestera_nh_neigh_set(sw, nh_neigh);
+               if (err) {
+                       pr_err("%s failed with err=%d ip=%pI4n mac=%pM",
+                              "prestera_nh_neigh_set", err,
+                              &nh_neigh->key.addr.u.ipv4,
+                              &nh_neigh->info.ha[0]);
+                       goto out;
+               }
+       }
+
+out:
+       list_for_each_entry(nhead, &nc->kern_fib_cache_list, head) {
+               __prestera_k_arb_fib_nh_offload_set(sw, nhead->this, nc,
+                                                   nc->in_kernel,
+                                                   !nc->in_kernel);
+       }
+}
+
 static int
 __prestera_pr_k_arb_fc_lpm_info_calc(struct prestera_switch *sw,
                                     struct prestera_kern_fib_cache *fc)
 {
+       struct fib_nh_common *nhc;
+       int nh_cnt;
+
        memset(&fc->lpm_info, 0, sizeof(fc->lpm_info));
 
        switch (prestera_kern_fib_info_type(&fc->info)) {
        case RTN_UNICAST:
-               fc->lpm_info.fib_type = PRESTERA_FIB_TYPE_TRAP;
+               if (prestera_fib_info_is_direct(&fc->info) &&
+                   fc->key.prefix_len ==
+                       PRESTERA_IP_ADDR_PLEN(fc->key.addr.v)) {
+                       /* This is special case.
+                        * When prefix is 32. Than we will have conflict in lpm
+                        * for direct route - once TRAP added, there is no
+                        * place for neighbour entry. So represent direct route
+                        * with prefix 32, as NH. So neighbour will be resolved
+                        * as nexthop of this route.
+                        */
+                       nhc = prestera_kern_fib_info_nhc(&fc->info, 0);
+                       fc->lpm_info.fib_type = PRESTERA_FIB_TYPE_UC_NH;
+                       fc->lpm_info.nh_grp_key.neigh[0].addr =
+                               fc->key.addr;
+                       fc->lpm_info.nh_grp_key.neigh[0].rif =
+                               nhc->nhc_dev;
+
+                       break;
+               }
+
+               /* We can also get nh_grp_key from fi. This will be correct to
+                * because cache not always represent, what actually written to
+                * lpm. But we use nh cache, as well for now (for this case).
+                */
+               for (nh_cnt = 0; nh_cnt < PRESTERA_NHGR_SIZE_MAX; nh_cnt++) {
+                       if (!fc->kern_neigh_cache_head[nh_cnt].n_cache)
+                               break;
+
+                       fc->lpm_info.nh_grp_key.neigh[nh_cnt].addr =
+                               fc->kern_neigh_cache_head[nh_cnt].n_cache->key.addr;
+                       fc->lpm_info.nh_grp_key.neigh[nh_cnt].rif =
+                               fc->kern_neigh_cache_head[nh_cnt].n_cache->key.dev;
+               }
+
+               fc->lpm_info.fib_type = nh_cnt ?
+                                       PRESTERA_FIB_TYPE_UC_NH :
+                                       PRESTERA_FIB_TYPE_TRAP;
                break;
        /* Unsupported. Leave it for kernel: */
        case RTN_BROADCAST:
@@ -231,7 +936,8 @@ static int __prestera_k_arb_f_lpm_set(struct prestera_switch *sw,
                return 0;
 
        fib_node = prestera_fib_node_create(sw, &fc->lpm_info.fib_key,
-                                           fc->lpm_info.fib_type, NULL);
+                                           fc->lpm_info.fib_type,
+                                           &fc->lpm_info.nh_grp_key);
 
        if (!fib_node) {
                dev_err(sw->dev->dev, "fib_node=NULL %pI4n/%d kern_tb_id = %d",
@@ -261,6 +967,8 @@ static int __prestera_k_arb_fc_apply(struct prestera_switch *sw,
 
        switch (fc->lpm_info.fib_type) {
        case PRESTERA_FIB_TYPE_UC_NH:
+               __prestera_k_arb_fib_lpm_offload_set(sw, fc, false,
+                                                    fc->reachable, false);
                break;
        case PRESTERA_FIB_TYPE_TRAP:
                __prestera_k_arb_fib_lpm_offload_set(sw, fc, false,
@@ -313,6 +1021,57 @@ __prestera_k_arb_util_fib_overlapped(struct prestera_switch *sw,
        return rfc;
 }
 
+/* Propagate kernel event to hw */
+static void prestera_k_arb_n_evt(struct prestera_switch *sw,
+                                struct neighbour *n)
+{
+       struct prestera_kern_neigh_cache_key n_key;
+       struct prestera_kern_neigh_cache *n_cache;
+       int err;
+
+       err = prestera_util_neigh2nc_key(sw, n, &n_key);
+       if (err)
+               return;
+
+       n_cache = prestera_kern_neigh_cache_find(sw, &n_key);
+       if (!n_cache) {
+               n_cache = prestera_kern_neigh_cache_get(sw, &n_key);
+               if (!n_cache)
+                       return;
+               __prestera_k_arb_nc_kern_fib_fetch(sw, n_cache);
+       }
+
+       __prestera_k_arb_nc_kern_n_fetch(sw, n_cache);
+       __prestera_k_arb_nc_apply(sw, n_cache);
+
+       prestera_kern_neigh_cache_put(sw, n_cache);
+}
+
+static void __prestera_k_arb_fib_evt2nc(struct prestera_switch *sw)
+{
+       struct prestera_kern_neigh_cache *n_cache;
+       struct rhashtable_iter iter;
+
+       rhashtable_walk_enter(&sw->router->kern_neigh_cache_ht, &iter);
+       rhashtable_walk_start(&iter);
+       while (1) {
+               n_cache = rhashtable_walk_next(&iter);
+
+               if (!n_cache)
+                       break;
+
+               if (IS_ERR(n_cache))
+                       continue;
+
+               rhashtable_walk_stop(&iter);
+               __prestera_k_arb_nc_kern_fib_fetch(sw, n_cache);
+               __prestera_k_arb_nc_apply(sw, n_cache);
+               rhashtable_walk_start(&iter);
+       }
+       rhashtable_walk_stop(&iter);
+       rhashtable_walk_exit(&iter);
+}
+
 static int
 prestera_k_arb_fib_evt(struct prestera_switch *sw,
                       bool replace, /* replace or del */
@@ -370,9 +1129,30 @@ prestera_k_arb_fib_evt(struct prestera_switch *sw,
                        dev_err(sw->dev->dev, "Applying fib_cache failed");
        }
 
+       /* Update all neighs to resolve overlapped and apply related */
+       __prestera_k_arb_fib_evt2nc(sw);
+
        return 0;
 }
 
+static void __prestera_k_arb_abort_neigh_ht_cb(void *ptr, void *arg)
+{
+       struct prestera_kern_neigh_cache *n_cache = ptr;
+       struct prestera_switch *sw = arg;
+
+       if (!list_empty(&n_cache->kern_fib_cache_list)) {
+               WARN_ON(1); /* BUG */
+               return;
+       }
+       __prestera_k_arb_n_offload_set(sw, n_cache, false);
+       n_cache->in_kernel = false;
+       /* No need to destroy lpm.
+        * It will be aborted by destroy_ht
+        */
+       __prestera_kern_neigh_cache_destruct(sw, n_cache);
+       kfree(n_cache);
+}
+
 static void __prestera_k_arb_abort_fib_ht_cb(void *ptr, void *arg)
 {
        struct prestera_kern_fib_cache *fib_cache = ptr;
@@ -381,6 +1161,8 @@ static void __prestera_k_arb_abort_fib_ht_cb(void *ptr, void *arg)
        __prestera_k_arb_fib_lpm_offload_set(sw, fib_cache,
                                             false, false,
                                             false);
+       __prestera_k_arb_fib_nh_offload_set(sw, fib_cache, NULL,
+                                           false, false);
        /* No need to destroy lpm.
         * It will be aborted by destroy_ht
         */
@@ -401,6 +1183,9 @@ static void prestera_k_arb_abort(struct prestera_switch *sw)
        rhashtable_free_and_destroy(&sw->router->kern_fib_cache_ht,
                                    __prestera_k_arb_abort_fib_ht_cb,
                                    sw);
+       rhashtable_free_and_destroy(&sw->router->kern_neigh_cache_ht,
+                                   __prestera_k_arb_abort_neigh_ht_cb,
+                                   sw);
 }
 
 static int __prestera_inetaddr_port_event(struct net_device *port_dev,
@@ -615,12 +1400,13 @@ static void prestera_router_neigh_event_work(struct work_struct *work)
 {
        struct prestera_netevent_work *net_work =
                container_of(work, struct prestera_netevent_work, work);
+       struct prestera_switch *sw = net_work->sw;
        struct neighbour *n = net_work->n;
 
        /* neigh - its not hw related object. It stored only in kernel. So... */
        rtnl_lock();
 
-       /* TODO: handler */
+       prestera_k_arb_n_evt(sw, n);
 
        neigh_release(n);
        rtnl_unlock();
@@ -676,6 +1462,11 @@ int prestera_router_init(struct prestera_switch *sw)
        if (err)
                goto err_kern_fib_cache_ht_init;
 
+       err = rhashtable_init(&router->kern_neigh_cache_ht,
+                             &__prestera_kern_neigh_cache_ht_params);
+       if (err)
+               goto err_kern_neigh_cache_ht_init;
+
        nhgrp_cache_bytes = sw->size_tbl_router_nexthop / 8 + 1;
        router->nhgrp_hw_state_cache = kzalloc(nhgrp_cache_bytes, GFP_KERNEL);
        if (!router->nhgrp_hw_state_cache) {
@@ -715,6 +1506,8 @@ err_register_inetaddr_notifier:
 err_register_inetaddr_validator_notifier:
        kfree(router->nhgrp_hw_state_cache);
 err_nh_state_cache_alloc:
+       rhashtable_destroy(&router->kern_neigh_cache_ht);
+err_kern_neigh_cache_ht_init:
        rhashtable_destroy(&router->kern_fib_cache_ht);
 err_kern_fib_cache_ht_init:
        prestera_router_hw_fini(sw);