rocker: move MAC learn event back to 'no wait' processing
[cascardo/linux.git] / drivers / net / ethernet / rocker / rocker.c
index 4818f2b..8430cb3 100644 (file)
@@ -295,7 +295,7 @@ static bool rocker_vlan_id_is_internal(__be16 vlan_id)
        return (_vlan_id >= start && _vlan_id <= end);
 }
 
-static __be16 rocker_port_vid_to_vlan(struct rocker_port *rocker_port,
+static __be16 rocker_port_vid_to_vlan(const struct rocker_port *rocker_port,
                                      u16 vid, bool *pop_vlan)
 {
        __be16 vlan_id;
@@ -312,7 +312,7 @@ static __be16 rocker_port_vid_to_vlan(struct rocker_port *rocker_port,
        return vlan_id;
 }
 
-static u16 rocker_port_vlan_to_vid(struct rocker_port *rocker_port,
+static u16 rocker_port_vlan_to_vid(const struct rocker_port *rocker_port,
                                   __be16 vlan_id)
 {
        if (rocker_vlan_id_is_internal(vlan_id))
@@ -321,15 +321,23 @@ static u16 rocker_port_vlan_to_vid(struct rocker_port *rocker_port,
        return ntohs(vlan_id);
 }
 
-static bool rocker_port_is_bridged(struct rocker_port *rocker_port)
+static bool rocker_port_is_bridged(const struct rocker_port *rocker_port)
 {
        return !!rocker_port->bridge_dev;
 }
 
+#define ROCKER_OP_FLAG_REMOVE          BIT(0)
+#define ROCKER_OP_FLAG_NOWAIT          BIT(1)
+#define ROCKER_OP_FLAG_LEARNED         BIT(2)
+#define ROCKER_OP_FLAG_REFRESH         BIT(3)
+
 static void *__rocker_port_mem_alloc(struct rocker_port *rocker_port,
-                                    enum switchdev_trans trans, size_t size)
+                                    enum switchdev_trans trans, int flags,
+                                    size_t size)
 {
        struct list_head *elem = NULL;
+       gfp_t gfp_flags = (flags & ROCKER_OP_FLAG_NOWAIT) ?
+                         GFP_ATOMIC : GFP_KERNEL;
 
        /* If in transaction prepare phase, allocate the memory
         * and enqueue it on a per-port list.  If in transaction
@@ -342,7 +350,7 @@ static void *__rocker_port_mem_alloc(struct rocker_port *rocker_port,
 
        switch (trans) {
        case SWITCHDEV_TRANS_PREPARE:
-               elem = kzalloc(size + sizeof(*elem), GFP_KERNEL);
+               elem = kzalloc(size + sizeof(*elem), gfp_flags);
                if (!elem)
                        return NULL;
                list_add_tail(elem, &rocker_port->trans_mem);
@@ -353,7 +361,7 @@ static void *__rocker_port_mem_alloc(struct rocker_port *rocker_port,
                list_del_init(elem);
                break;
        case SWITCHDEV_TRANS_NONE:
-               elem = kzalloc(size + sizeof(*elem), GFP_KERNEL);
+               elem = kzalloc(size + sizeof(*elem), gfp_flags);
                if (elem)
                        INIT_LIST_HEAD(elem);
                break;
@@ -365,16 +373,17 @@ static void *__rocker_port_mem_alloc(struct rocker_port *rocker_port,
 }
 
 static void *rocker_port_kzalloc(struct rocker_port *rocker_port,
-                                enum switchdev_trans trans, size_t size)
+                                enum switchdev_trans trans, int flags,
+                                size_t size)
 {
-       return __rocker_port_mem_alloc(rocker_port, trans, size);
+       return __rocker_port_mem_alloc(rocker_port, trans, flags, size);
 }
 
 static void *rocker_port_kcalloc(struct rocker_port *rocker_port,
-                                enum switchdev_trans trans, size_t n,
-                                size_t size)
+                                enum switchdev_trans trans, int flags,
+                                size_t n, size_t size)
 {
-       return __rocker_port_mem_alloc(rocker_port, trans, n * size);
+       return __rocker_port_mem_alloc(rocker_port, trans, flags, n * size);
 }
 
 static void rocker_port_kfree(enum switchdev_trans trans, const void *mem)
@@ -397,11 +406,13 @@ static void rocker_port_kfree(enum switchdev_trans trans, const void *mem)
 struct rocker_wait {
        wait_queue_head_t wait;
        bool done;
+       bool nowait;
 };
 
 static void rocker_wait_reset(struct rocker_wait *wait)
 {
        wait->done = false;
+       wait->nowait = false;
 }
 
 static void rocker_wait_init(struct rocker_wait *wait)
@@ -411,11 +422,12 @@ static void rocker_wait_init(struct rocker_wait *wait)
 }
 
 static struct rocker_wait *rocker_wait_create(struct rocker_port *rocker_port,
-                                             enum switchdev_trans trans)
+                                             enum switchdev_trans trans,
+                                             int flags)
 {
        struct rocker_wait *wait;
 
-       wait = rocker_port_kzalloc(rocker_port, trans, sizeof(*wait));
+       wait = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*wait));
        if (!wait)
                return NULL;
        rocker_wait_init(wait);
@@ -443,18 +455,18 @@ static void rocker_wait_wake_up(struct rocker_wait *wait)
        wake_up(&wait->wait);
 }
 
-static u32 rocker_msix_vector(struct rocker *rocker, unsigned int vector)
+static u32 rocker_msix_vector(const struct rocker *rocker, unsigned int vector)
 {
        return rocker->msix_entries[vector].vector;
 }
 
-static u32 rocker_msix_tx_vector(struct rocker_port *rocker_port)
+static u32 rocker_msix_tx_vector(const struct rocker_port *rocker_port)
 {
        return rocker_msix_vector(rocker_port->rocker,
                                  ROCKER_MSIX_VEC_TX(rocker_port->port_number));
 }
 
-static u32 rocker_msix_rx_vector(struct rocker_port *rocker_port)
+static u32 rocker_msix_rx_vector(const struct rocker_port *rocker_port)
 {
        return rocker_msix_vector(rocker_port->rocker,
                                  ROCKER_MSIX_VEC_RX(rocker_port->port_number));
@@ -473,9 +485,9 @@ static u32 rocker_msix_rx_vector(struct rocker_port *rocker_port)
  * HW basic testing functions
  *****************************/
 
-static int rocker_reg_test(struct rocker *rocker)
+static int rocker_reg_test(const struct rocker *rocker)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct pci_dev *pdev = rocker->pdev;
        u64 test_reg;
        u64 rnd;
 
@@ -503,12 +515,12 @@ static int rocker_reg_test(struct rocker *rocker)
        return 0;
 }
 
-static int rocker_dma_test_one(struct rocker *rocker, struct rocker_wait *wait,
-                              u32 test_type, dma_addr_t dma_handle,
-                              unsigned char *buf, unsigned char *expect,
-                              size_t size)
+static int rocker_dma_test_one(const struct rocker *rocker,
+                              struct rocker_wait *wait, u32 test_type,
+                              dma_addr_t dma_handle, const unsigned char *buf,
+                              const unsigned char *expect, size_t size)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct pci_dev *pdev = rocker->pdev;
        int i;
 
        rocker_wait_reset(wait);
@@ -532,7 +544,7 @@ static int rocker_dma_test_one(struct rocker *rocker, struct rocker_wait *wait,
 #define ROCKER_TEST_DMA_BUF_SIZE (PAGE_SIZE * 4)
 #define ROCKER_TEST_DMA_FILL_PATTERN 0x96
 
-static int rocker_dma_test_offset(struct rocker *rocker,
+static int rocker_dma_test_offset(const struct rocker *rocker,
                                  struct rocker_wait *wait, int offset)
 {
        struct pci_dev *pdev = rocker->pdev;
@@ -592,7 +604,8 @@ free_alloc:
        return err;
 }
 
-static int rocker_dma_test(struct rocker *rocker, struct rocker_wait *wait)
+static int rocker_dma_test(const struct rocker *rocker,
+                          struct rocker_wait *wait)
 {
        int i;
        int err;
@@ -614,9 +627,9 @@ static irqreturn_t rocker_test_irq_handler(int irq, void *dev_id)
        return IRQ_HANDLED;
 }
 
-static int rocker_basic_hw_test(struct rocker *rocker)
+static int rocker_basic_hw_test(const struct rocker *rocker)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct pci_dev *pdev = rocker->pdev;
        struct rocker_wait wait;
        int err;
 
@@ -749,7 +762,7 @@ static u64 rocker_tlv_get_u64(const struct rocker_tlv *tlv)
        return *(u64 *) rocker_tlv_data(tlv);
 }
 
-static void rocker_tlv_parse(struct rocker_tlv **tb, int maxtype,
+static void rocker_tlv_parse(const struct rocker_tlv **tb, int maxtype,
                             const char *buf, int buf_len)
 {
        const struct rocker_tlv *tlv;
@@ -762,19 +775,19 @@ static void rocker_tlv_parse(struct rocker_tlv **tb, int maxtype,
                u32 type = rocker_tlv_type(tlv);
 
                if (type > 0 && type <= maxtype)
-                       tb[type] = (struct rocker_tlv *) tlv;
+                       tb[type] = tlv;
        }
 }
 
-static void rocker_tlv_parse_nested(struct rocker_tlv **tb, int maxtype,
+static void rocker_tlv_parse_nested(const struct rocker_tlv **tb, int maxtype,
                                    const struct rocker_tlv *tlv)
 {
        rocker_tlv_parse(tb, maxtype, rocker_tlv_data(tlv),
                         rocker_tlv_len(tlv));
 }
 
-static void rocker_tlv_parse_desc(struct rocker_tlv **tb, int maxtype,
-                                 struct rocker_desc_info *desc_info)
+static void rocker_tlv_parse_desc(const struct rocker_tlv **tb, int maxtype,
+                                 const struct rocker_desc_info *desc_info)
 {
        rocker_tlv_parse(tb, maxtype, desc_info->data,
                         desc_info->desc->tlv_size);
@@ -859,9 +872,9 @@ static void rocker_tlv_nest_end(struct rocker_desc_info *desc_info,
 }
 
 static void rocker_tlv_nest_cancel(struct rocker_desc_info *desc_info,
-                                  struct rocker_tlv *start)
+                                  const struct rocker_tlv *start)
 {
-       desc_info->tlv_size = (char *) start - desc_info->data;
+       desc_info->tlv_size = (const char *) start - desc_info->data;
 }
 
 /******************************************
@@ -873,7 +886,7 @@ static u32 __pos_inc(u32 pos, size_t limit)
        return ++pos == limit ? 0 : pos;
 }
 
-static int rocker_desc_err(struct rocker_desc_info *desc_info)
+static int rocker_desc_err(const struct rocker_desc_info *desc_info)
 {
        int err = desc_info->desc->comp_err & ~ROCKER_DMA_DESC_COMP_ERR_GEN;
 
@@ -901,31 +914,31 @@ static int rocker_desc_err(struct rocker_desc_info *desc_info)
        return -EINVAL;
 }
 
-static void rocker_desc_gen_clear(struct rocker_desc_info *desc_info)
+static void rocker_desc_gen_clear(const struct rocker_desc_info *desc_info)
 {
        desc_info->desc->comp_err &= ~ROCKER_DMA_DESC_COMP_ERR_GEN;
 }
 
-static bool rocker_desc_gen(struct rocker_desc_info *desc_info)
+static bool rocker_desc_gen(const struct rocker_desc_info *desc_info)
 {
        u32 comp_err = desc_info->desc->comp_err;
 
        return comp_err & ROCKER_DMA_DESC_COMP_ERR_GEN ? true : false;
 }
 
-static void *rocker_desc_cookie_ptr_get(struct rocker_desc_info *desc_info)
+static void *rocker_desc_cookie_ptr_get(const struct rocker_desc_info *desc_info)
 {
        return (void *)(uintptr_t)desc_info->desc->cookie;
 }
 
-static void rocker_desc_cookie_ptr_set(struct rocker_desc_info *desc_info,
+static void rocker_desc_cookie_ptr_set(const struct rocker_desc_info *desc_info,
                                       void *ptr)
 {
        desc_info->desc->cookie = (uintptr_t) ptr;
 }
 
 static struct rocker_desc_info *
-rocker_desc_head_get(struct rocker_dma_ring_info *info)
+rocker_desc_head_get(const struct rocker_dma_ring_info *info)
 {
        static struct rocker_desc_info *desc_info;
        u32 head = __pos_inc(info->head, info->size);
@@ -937,15 +950,15 @@ rocker_desc_head_get(struct rocker_dma_ring_info *info)
        return desc_info;
 }
 
-static void rocker_desc_commit(struct rocker_desc_info *desc_info)
+static void rocker_desc_commit(const struct rocker_desc_info *desc_info)
 {
        desc_info->desc->buf_size = desc_info->data_size;
        desc_info->desc->tlv_size = desc_info->tlv_size;
 }
 
-static void rocker_desc_head_set(struct rocker *rocker,
+static void rocker_desc_head_set(const struct rocker *rocker,
                                 struct rocker_dma_ring_info *info,
-                                struct rocker_desc_info *desc_info)
+                                const struct rocker_desc_info *desc_info)
 {
        u32 head = __pos_inc(info->head, info->size);
 
@@ -970,8 +983,8 @@ rocker_desc_tail_get(struct rocker_dma_ring_info *info)
        return desc_info;
 }
 
-static void rocker_dma_ring_credits_set(struct rocker *rocker,
-                                       struct rocker_dma_ring_info *info,
+static void rocker_dma_ring_credits_set(const struct rocker *rocker,
+                                       const struct rocker_dma_ring_info *info,
                                        u32 credits)
 {
        if (credits)
@@ -984,7 +997,7 @@ static unsigned long rocker_dma_ring_size_fix(size_t size)
                   min(roundup_pow_of_two(size), ROCKER_DMA_SIZE_MAX));
 }
 
-static int rocker_dma_ring_create(struct rocker *rocker,
+static int rocker_dma_ring_create(const struct rocker *rocker,
                                  unsigned int type,
                                  size_t size,
                                  struct rocker_dma_ring_info *info)
@@ -1020,8 +1033,8 @@ static int rocker_dma_ring_create(struct rocker *rocker,
        return 0;
 }
 
-static void rocker_dma_ring_destroy(struct rocker *rocker,
-                                   struct rocker_dma_ring_info *info)
+static void rocker_dma_ring_destroy(const struct rocker *rocker,
+                                   const struct rocker_dma_ring_info *info)
 {
        rocker_write64(rocker, DMA_DESC_ADDR(info->type), 0);
 
@@ -1031,7 +1044,7 @@ static void rocker_dma_ring_destroy(struct rocker *rocker,
        kfree(info->desc_info);
 }
 
-static void rocker_dma_ring_pass_to_producer(struct rocker *rocker,
+static void rocker_dma_ring_pass_to_producer(const struct rocker *rocker,
                                             struct rocker_dma_ring_info *info)
 {
        int i;
@@ -1046,8 +1059,8 @@ static void rocker_dma_ring_pass_to_producer(struct rocker *rocker,
        rocker_desc_commit(&info->desc_info[i]);
 }
 
-static int rocker_dma_ring_bufs_alloc(struct rocker *rocker,
-                                     struct rocker_dma_ring_info *info,
+static int rocker_dma_ring_bufs_alloc(const struct rocker *rocker,
+                                     const struct rocker_dma_ring_info *info,
                                      int direction, size_t buf_size)
 {
        struct pci_dev *pdev = rocker->pdev;
@@ -1084,7 +1097,7 @@ static int rocker_dma_ring_bufs_alloc(struct rocker *rocker,
 
 rollback:
        for (i--; i >= 0; i--) {
-               struct rocker_desc_info *desc_info = &info->desc_info[i];
+               const struct rocker_desc_info *desc_info = &info->desc_info[i];
 
                pci_unmap_single(pdev, dma_unmap_addr(desc_info, mapaddr),
                                 desc_info->data_size, direction);
@@ -1093,15 +1106,15 @@ rollback:
        return err;
 }
 
-static void rocker_dma_ring_bufs_free(struct rocker *rocker,
-                                     struct rocker_dma_ring_info *info,
+static void rocker_dma_ring_bufs_free(const struct rocker *rocker,
+                                     const struct rocker_dma_ring_info *info,
                                      int direction)
 {
        struct pci_dev *pdev = rocker->pdev;
        int i;
 
        for (i = 0; i < info->size; i++) {
-               struct rocker_desc_info *desc_info = &info->desc_info[i];
+               const struct rocker_desc_info *desc_info = &info->desc_info[i];
                struct rocker_desc *desc = &info->desc[i];
 
                desc->buf_addr = 0;
@@ -1114,7 +1127,7 @@ static void rocker_dma_ring_bufs_free(struct rocker *rocker,
 
 static int rocker_dma_rings_init(struct rocker *rocker)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct pci_dev *pdev = rocker->pdev;
        int err;
 
        err = rocker_dma_ring_create(rocker, ROCKER_DMA_CMD,
@@ -1171,11 +1184,11 @@ static void rocker_dma_rings_fini(struct rocker *rocker)
        rocker_dma_ring_destroy(rocker, &rocker->cmd_ring);
 }
 
-static int rocker_dma_rx_ring_skb_map(struct rocker *rocker,
-                                     struct rocker_port *rocker_port,
+static int rocker_dma_rx_ring_skb_map(const struct rocker_port *rocker_port,
                                      struct rocker_desc_info *desc_info,
                                      struct sk_buff *skb, size_t buf_len)
 {
+       const struct rocker *rocker = rocker_port->rocker;
        struct pci_dev *pdev = rocker->pdev;
        dma_addr_t dma_handle;
 
@@ -1195,13 +1208,12 @@ tlv_put_failure:
        return -EMSGSIZE;
 }
 
-static size_t rocker_port_rx_buf_len(struct rocker_port *rocker_port)
+static size_t rocker_port_rx_buf_len(const struct rocker_port *rocker_port)
 {
        return rocker_port->dev->mtu + ETH_HLEN + ETH_FCS_LEN + VLAN_HLEN;
 }
 
-static int rocker_dma_rx_ring_skb_alloc(struct rocker *rocker,
-                                       struct rocker_port *rocker_port,
+static int rocker_dma_rx_ring_skb_alloc(const struct rocker_port *rocker_port,
                                        struct rocker_desc_info *desc_info)
 {
        struct net_device *dev = rocker_port->dev;
@@ -1218,8 +1230,7 @@ static int rocker_dma_rx_ring_skb_alloc(struct rocker *rocker,
        skb = netdev_alloc_skb_ip_align(dev, buf_len);
        if (!skb)
                return -ENOMEM;
-       err = rocker_dma_rx_ring_skb_map(rocker, rocker_port, desc_info,
-                                        skb, buf_len);
+       err = rocker_dma_rx_ring_skb_map(rocker_port, desc_info, skb, buf_len);
        if (err) {
                dev_kfree_skb_any(skb);
                return err;
@@ -1228,8 +1239,8 @@ static int rocker_dma_rx_ring_skb_alloc(struct rocker *rocker,
        return 0;
 }
 
-static void rocker_dma_rx_ring_skb_unmap(struct rocker *rocker,
-                                        struct rocker_tlv **attrs)
+static void rocker_dma_rx_ring_skb_unmap(const struct rocker *rocker,
+                                        const struct rocker_tlv **attrs)
 {
        struct pci_dev *pdev = rocker->pdev;
        dma_addr_t dma_handle;
@@ -1243,10 +1254,10 @@ static void rocker_dma_rx_ring_skb_unmap(struct rocker *rocker,
        pci_unmap_single(pdev, dma_handle, len, PCI_DMA_FROMDEVICE);
 }
 
-static void rocker_dma_rx_ring_skb_free(struct rocker *rocker,
-                                       struct rocker_desc_info *desc_info)
+static void rocker_dma_rx_ring_skb_free(const struct rocker *rocker,
+                                       const struct rocker_desc_info *desc_info)
 {
-       struct rocker_tlv *attrs[ROCKER_TLV_RX_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_RX_MAX + 1];
        struct sk_buff *skb = rocker_desc_cookie_ptr_get(desc_info);
 
        if (!skb)
@@ -1256,15 +1267,15 @@ static void rocker_dma_rx_ring_skb_free(struct rocker *rocker,
        dev_kfree_skb_any(skb);
 }
 
-static int rocker_dma_rx_ring_skbs_alloc(struct rocker *rocker,
-                                        struct rocker_port *rocker_port)
+static int rocker_dma_rx_ring_skbs_alloc(const struct rocker_port *rocker_port)
 {
-       struct rocker_dma_ring_info *rx_ring = &rocker_port->rx_ring;
+       const struct rocker_dma_ring_info *rx_ring = &rocker_port->rx_ring;
+       const struct rocker *rocker = rocker_port->rocker;
        int i;
        int err;
 
        for (i = 0; i < rx_ring->size; i++) {
-               err = rocker_dma_rx_ring_skb_alloc(rocker, rocker_port,
+               err = rocker_dma_rx_ring_skb_alloc(rocker_port,
                                                   &rx_ring->desc_info[i]);
                if (err)
                        goto rollback;
@@ -1277,10 +1288,10 @@ rollback:
        return err;
 }
 
-static void rocker_dma_rx_ring_skbs_free(struct rocker *rocker,
-                                        struct rocker_port *rocker_port)
+static void rocker_dma_rx_ring_skbs_free(const struct rocker_port *rocker_port)
 {
-       struct rocker_dma_ring_info *rx_ring = &rocker_port->rx_ring;
+       const struct rocker_dma_ring_info *rx_ring = &rocker_port->rx_ring;
+       const struct rocker *rocker = rocker_port->rocker;
        int i;
 
        for (i = 0; i < rx_ring->size; i++)
@@ -1326,7 +1337,7 @@ static int rocker_port_dma_rings_init(struct rocker_port *rocker_port)
                goto err_dma_rx_ring_bufs_alloc;
        }
 
-       err = rocker_dma_rx_ring_skbs_alloc(rocker, rocker_port);
+       err = rocker_dma_rx_ring_skbs_alloc(rocker_port);
        if (err) {
                netdev_err(rocker_port->dev, "failed to alloc rx dma ring skbs\n");
                goto err_dma_rx_ring_skbs_alloc;
@@ -1352,7 +1363,7 @@ static void rocker_port_dma_rings_fini(struct rocker_port *rocker_port)
 {
        struct rocker *rocker = rocker_port->rocker;
 
-       rocker_dma_rx_ring_skbs_free(rocker, rocker_port);
+       rocker_dma_rx_ring_skbs_free(rocker_port);
        rocker_dma_ring_bufs_free(rocker, &rocker_port->rx_ring,
                                  PCI_DMA_BIDIRECTIONAL);
        rocker_dma_ring_destroy(rocker, &rocker_port->rx_ring);
@@ -1361,7 +1372,8 @@ static void rocker_port_dma_rings_fini(struct rocker_port *rocker_port)
        rocker_dma_ring_destroy(rocker, &rocker_port->tx_ring);
 }
 
-static void rocker_port_set_enable(struct rocker_port *rocker_port, bool enable)
+static void rocker_port_set_enable(const struct rocker_port *rocker_port,
+                                  bool enable)
 {
        u64 val = rocker_read64(rocker_port->rocker, PORT_PHYS_ENABLE);
 
@@ -1379,14 +1391,19 @@ static void rocker_port_set_enable(struct rocker_port *rocker_port, bool enable)
 static irqreturn_t rocker_cmd_irq_handler(int irq, void *dev_id)
 {
        struct rocker *rocker = dev_id;
-       struct rocker_desc_info *desc_info;
+       const struct rocker_desc_info *desc_info;
        struct rocker_wait *wait;
        u32 credits = 0;
 
        spin_lock(&rocker->cmd_ring_lock);
        while ((desc_info = rocker_desc_tail_get(&rocker->cmd_ring))) {
                wait = rocker_desc_cookie_ptr_get(desc_info);
-               rocker_wait_wake_up(wait);
+               if (wait->nowait) {
+                       rocker_desc_gen_clear(desc_info);
+                       rocker_wait_destroy(SWITCHDEV_TRANS_NONE, wait);
+               } else {
+                       rocker_wait_wake_up(wait);
+               }
                credits++;
        }
        spin_unlock(&rocker->cmd_ring_lock);
@@ -1395,22 +1412,22 @@ static irqreturn_t rocker_cmd_irq_handler(int irq, void *dev_id)
        return IRQ_HANDLED;
 }
 
-static void rocker_port_link_up(struct rocker_port *rocker_port)
+static void rocker_port_link_up(const struct rocker_port *rocker_port)
 {
        netif_carrier_on(rocker_port->dev);
        netdev_info(rocker_port->dev, "Link is up\n");
 }
 
-static void rocker_port_link_down(struct rocker_port *rocker_port)
+static void rocker_port_link_down(const struct rocker_port *rocker_port)
 {
        netif_carrier_off(rocker_port->dev);
        netdev_info(rocker_port->dev, "Link is down\n");
 }
 
-static int rocker_event_link_change(struct rocker *rocker,
+static int rocker_event_link_change(const struct rocker *rocker,
                                    const struct rocker_tlv *info)
 {
-       struct rocker_tlv *attrs[ROCKER_TLV_EVENT_LINK_CHANGED_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_EVENT_LINK_CHANGED_MAX + 1];
        unsigned int port_number;
        bool link_up;
        struct rocker_port *rocker_port;
@@ -1437,45 +1454,19 @@ static int rocker_event_link_change(struct rocker *rocker,
        return 0;
 }
 
-#define ROCKER_OP_FLAG_REMOVE          BIT(0)
-#define ROCKER_OP_FLAG_LEARNED         BIT(1)
-#define ROCKER_OP_FLAG_REFRESH         BIT(2)
-
 static int rocker_port_fdb(struct rocker_port *rocker_port,
                           enum switchdev_trans trans,
                           const unsigned char *addr,
                           __be16 vlan_id, int flags);
 
-struct rocker_mac_vlan_seen_work {
-       struct work_struct work;
-       struct rocker_port *rocker_port;
-       int flags;
-       unsigned char addr[ETH_ALEN];
-       __be16 vlan_id;
-};
-
-static void rocker_event_mac_vlan_seen_work(struct work_struct *work)
-{
-       struct rocker_mac_vlan_seen_work *sw =
-               container_of(work, struct rocker_mac_vlan_seen_work, work);
-
-       rtnl_lock();
-       rocker_port_fdb(sw->rocker_port, SWITCHDEV_TRANS_NONE,
-                       sw->addr, sw->vlan_id, sw->flags);
-       rtnl_unlock();
-
-       kfree(work);
-}
-
-static int rocker_event_mac_vlan_seen(struct rocker *rocker,
+static int rocker_event_mac_vlan_seen(const struct rocker *rocker,
                                      const struct rocker_tlv *info)
 {
-       struct rocker_mac_vlan_seen_work *sw;
-       struct rocker_tlv *attrs[ROCKER_TLV_EVENT_MAC_VLAN_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_EVENT_MAC_VLAN_MAX + 1];
        unsigned int port_number;
        struct rocker_port *rocker_port;
-       unsigned char *addr;
-       int flags = ROCKER_OP_FLAG_LEARNED;
+       const unsigned char *addr;
+       int flags = ROCKER_OP_FLAG_NOWAIT | ROCKER_OP_FLAG_LEARNED;
        __be16 vlan_id;
 
        rocker_tlv_parse_nested(attrs, ROCKER_TLV_EVENT_MAC_VLAN_MAX, info);
@@ -1497,27 +1488,15 @@ static int rocker_event_mac_vlan_seen(struct rocker *rocker,
            rocker_port->stp_state != BR_STATE_FORWARDING)
                return 0;
 
-       sw = kmalloc(sizeof(*sw), GFP_ATOMIC);
-       if (!sw)
-               return -ENOMEM;
-
-       INIT_WORK(&sw->work, rocker_event_mac_vlan_seen_work);
-
-       sw->rocker_port = rocker_port;
-       sw->flags = flags;
-       ether_addr_copy(sw->addr, addr);
-       sw->vlan_id = vlan_id;
-
-       schedule_work(&sw->work);
-
-       return 0;
+       return rocker_port_fdb(rocker_port, SWITCHDEV_TRANS_NONE,
+                              addr, vlan_id, flags);
 }
 
-static int rocker_event_process(struct rocker *rocker,
-                               struct rocker_desc_info *desc_info)
+static int rocker_event_process(const struct rocker *rocker,
+                               const struct rocker_desc_info *desc_info)
 {
-       struct rocker_tlv *attrs[ROCKER_TLV_EVENT_MAX + 1];
-       struct rocker_tlv *info;
+       const struct rocker_tlv *attrs[ROCKER_TLV_EVENT_MAX + 1];
+       const struct rocker_tlv *info;
        u16 type;
 
        rocker_tlv_parse_desc(attrs, ROCKER_TLV_EVENT_MAX, desc_info);
@@ -1541,8 +1520,8 @@ static int rocker_event_process(struct rocker *rocker,
 static irqreturn_t rocker_event_irq_handler(int irq, void *dev_id)
 {
        struct rocker *rocker = dev_id;
-       struct pci_dev *pdev = rocker->pdev;
-       struct rocker_desc_info *desc_info;
+       const struct pci_dev *pdev = rocker->pdev;
+       const struct rocker_desc_info *desc_info;
        u32 credits = 0;
        int err;
 
@@ -1586,38 +1565,43 @@ static irqreturn_t rocker_rx_irq_handler(int irq, void *dev_id)
  * Command interface
  ********************/
 
-typedef int (*rocker_cmd_cb_t)(struct rocker *rocker,
-                              struct rocker_port *rocker_port,
-                              struct rocker_desc_info *desc_info,
-                              void *priv);
+typedef int (*rocker_cmd_prep_cb_t)(const struct rocker_port *rocker_port,
+                                   struct rocker_desc_info *desc_info,
+                                   void *priv);
 
-static int rocker_cmd_exec(struct rocker *rocker,
-                          struct rocker_port *rocker_port,
-                          enum switchdev_trans trans,
-                          rocker_cmd_cb_t prepare, void *prepare_priv,
-                          rocker_cmd_cb_t process, void *process_priv)
+typedef int (*rocker_cmd_proc_cb_t)(const struct rocker_port *rocker_port,
+                                   const struct rocker_desc_info *desc_info,
+                                   void *priv);
+
+static int rocker_cmd_exec(struct rocker_port *rocker_port,
+                          enum switchdev_trans trans, int flags,
+                          rocker_cmd_prep_cb_t prepare, void *prepare_priv,
+                          rocker_cmd_proc_cb_t process, void *process_priv)
 {
+       struct rocker *rocker = rocker_port->rocker;
        struct rocker_desc_info *desc_info;
        struct rocker_wait *wait;
-       unsigned long flags;
+       bool nowait = !!(flags & ROCKER_OP_FLAG_NOWAIT);
+       unsigned long lock_flags;
        int err;
 
-       wait = rocker_wait_create(rocker_port, trans);
+       wait = rocker_wait_create(rocker_port, trans, flags);
        if (!wait)
                return -ENOMEM;
+       wait->nowait = nowait;
 
-       spin_lock_irqsave(&rocker->cmd_ring_lock, flags);
+       spin_lock_irqsave(&rocker->cmd_ring_lock, lock_flags);
 
        desc_info = rocker_desc_head_get(&rocker->cmd_ring);
        if (!desc_info) {
-               spin_unlock_irqrestore(&rocker->cmd_ring_lock, flags);
+               spin_unlock_irqrestore(&rocker->cmd_ring_lock, lock_flags);
                err = -EAGAIN;
                goto out;
        }
 
-       err = prepare(rocker, rocker_port, desc_info, prepare_priv);
+       err = prepare(rocker_port, desc_info, prepare_priv);
        if (err) {
-               spin_unlock_irqrestore(&rocker->cmd_ring_lock, flags);
+               spin_unlock_irqrestore(&rocker->cmd_ring_lock, lock_flags);
                goto out;
        }
 
@@ -1626,7 +1610,10 @@ static int rocker_cmd_exec(struct rocker *rocker,
        if (trans != SWITCHDEV_TRANS_PREPARE)
                rocker_desc_head_set(rocker, &rocker->cmd_ring, desc_info);
 
-       spin_unlock_irqrestore(&rocker->cmd_ring_lock, flags);
+       spin_unlock_irqrestore(&rocker->cmd_ring_lock, lock_flags);
+
+       if (nowait)
+               return 0;
 
        if (trans != SWITCHDEV_TRANS_PREPARE)
                if (!rocker_wait_event_timeout(wait, HZ / 10))
@@ -1637,7 +1624,7 @@ static int rocker_cmd_exec(struct rocker *rocker,
                return err;
 
        if (process)
-               err = process(rocker, rocker_port, desc_info, process_priv);
+               err = process(rocker_port, desc_info, process_priv);
 
        rocker_desc_gen_clear(desc_info);
 out:
@@ -1646,8 +1633,7 @@ out:
 }
 
 static int
-rocker_cmd_get_port_settings_prep(struct rocker *rocker,
-                                 struct rocker_port *rocker_port,
+rocker_cmd_get_port_settings_prep(const struct rocker_port *rocker_port,
                                  struct rocker_desc_info *desc_info,
                                  void *priv)
 {
@@ -1667,14 +1653,13 @@ rocker_cmd_get_port_settings_prep(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_get_port_settings_ethtool_proc(struct rocker *rocker,
-                                         struct rocker_port *rocker_port,
-                                         struct rocker_desc_info *desc_info,
+rocker_cmd_get_port_settings_ethtool_proc(const struct rocker_port *rocker_port,
+                                         const struct rocker_desc_info *desc_info,
                                          void *priv)
 {
        struct ethtool_cmd *ecmd = priv;
-       struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
-       struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
+       const struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
        u32 speed;
        u8 duplex;
        u8 autoneg;
@@ -1706,15 +1691,14 @@ rocker_cmd_get_port_settings_ethtool_proc(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_get_port_settings_macaddr_proc(struct rocker *rocker,
-                                         struct rocker_port *rocker_port,
-                                         struct rocker_desc_info *desc_info,
+rocker_cmd_get_port_settings_macaddr_proc(const struct rocker_port *rocker_port,
+                                         const struct rocker_desc_info *desc_info,
                                          void *priv)
 {
        unsigned char *macaddr = priv;
-       struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
-       struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
-       struct rocker_tlv *attr;
+       const struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
+       const struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
+       const struct rocker_tlv *attr;
 
        rocker_tlv_parse_desc(attrs, ROCKER_TLV_CMD_MAX, desc_info);
        if (!attrs[ROCKER_TLV_CMD_INFO])
@@ -1739,17 +1723,16 @@ struct port_name {
 };
 
 static int
-rocker_cmd_get_port_settings_phys_name_proc(struct rocker *rocker,
-                                           struct rocker_port *rocker_port,
-                                           struct rocker_desc_info *desc_info,
+rocker_cmd_get_port_settings_phys_name_proc(const struct rocker_port *rocker_port,
+                                           const struct rocker_desc_info *desc_info,
                                            void *priv)
 {
-       struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
-       struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
+       const struct rocker_tlv *info_attrs[ROCKER_TLV_CMD_PORT_SETTINGS_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
        struct port_name *name = priv;
-       struct rocker_tlv *attr;
+       const struct rocker_tlv *attr;
        size_t i, j, len;
-       char *str;
+       const char *str;
 
        rocker_tlv_parse_desc(attrs, ROCKER_TLV_CMD_MAX, desc_info);
        if (!attrs[ROCKER_TLV_CMD_INFO])
@@ -1781,8 +1764,7 @@ rocker_cmd_get_port_settings_phys_name_proc(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_set_port_settings_ethtool_prep(struct rocker *rocker,
-                                         struct rocker_port *rocker_port,
+rocker_cmd_set_port_settings_ethtool_prep(const struct rocker_port *rocker_port,
                                          struct rocker_desc_info *desc_info,
                                          void *priv)
 {
@@ -1812,12 +1794,11 @@ rocker_cmd_set_port_settings_ethtool_prep(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_set_port_settings_macaddr_prep(struct rocker *rocker,
-                                         struct rocker_port *rocker_port,
+rocker_cmd_set_port_settings_macaddr_prep(const struct rocker_port *rocker_port,
                                          struct rocker_desc_info *desc_info,
                                          void *priv)
 {
-       unsigned char *macaddr = priv;
+       const unsigned char *macaddr = priv;
        struct rocker_tlv *cmd_info;
 
        if (rocker_tlv_put_u16(desc_info, ROCKER_TLV_CMD_TYPE,
@@ -1837,8 +1818,7 @@ rocker_cmd_set_port_settings_macaddr_prep(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_set_port_learning_prep(struct rocker *rocker,
-                                 struct rocker_port *rocker_port,
+rocker_cmd_set_port_learning_prep(const struct rocker_port *rocker_port,
                                  struct rocker_desc_info *desc_info,
                                  void *priv)
 {
@@ -1863,8 +1843,7 @@ rocker_cmd_set_port_learning_prep(struct rocker *rocker,
 static int rocker_cmd_get_port_settings_ethtool(struct rocker_port *rocker_port,
                                                struct ethtool_cmd *ecmd)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                              SWITCHDEV_TRANS_NONE,
+       return rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                               rocker_cmd_get_port_settings_prep, NULL,
                               rocker_cmd_get_port_settings_ethtool_proc,
                               ecmd);
@@ -1873,8 +1852,7 @@ static int rocker_cmd_get_port_settings_ethtool(struct rocker_port *rocker_port,
 static int rocker_cmd_get_port_settings_macaddr(struct rocker_port *rocker_port,
                                                unsigned char *macaddr)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                              SWITCHDEV_TRANS_NONE,
+       return rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                               rocker_cmd_get_port_settings_prep, NULL,
                               rocker_cmd_get_port_settings_macaddr_proc,
                               macaddr);
@@ -1883,8 +1861,7 @@ static int rocker_cmd_get_port_settings_macaddr(struct rocker_port *rocker_port,
 static int rocker_cmd_set_port_settings_ethtool(struct rocker_port *rocker_port,
                                                struct ethtool_cmd *ecmd)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                              SWITCHDEV_TRANS_NONE,
+       return rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                               rocker_cmd_set_port_settings_ethtool_prep,
                               ecmd, NULL, NULL);
 }
@@ -1892,8 +1869,7 @@ static int rocker_cmd_set_port_settings_ethtool(struct rocker_port *rocker_port,
 static int rocker_cmd_set_port_settings_macaddr(struct rocker_port *rocker_port,
                                                unsigned char *macaddr)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                              SWITCHDEV_TRANS_NONE,
+       return rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                               rocker_cmd_set_port_settings_macaddr_prep,
                               macaddr, NULL, NULL);
 }
@@ -1901,13 +1877,14 @@ static int rocker_cmd_set_port_settings_macaddr(struct rocker_port *rocker_port,
 static int rocker_port_set_learning(struct rocker_port *rocker_port,
                                    enum switchdev_trans trans)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port, trans,
+       return rocker_cmd_exec(rocker_port, trans, 0,
                               rocker_cmd_set_port_learning_prep,
                               NULL, NULL, NULL);
 }
 
-static int rocker_cmd_flow_tbl_add_ig_port(struct rocker_desc_info *desc_info,
-                                          struct rocker_flow_tbl_entry *entry)
+static int
+rocker_cmd_flow_tbl_add_ig_port(struct rocker_desc_info *desc_info,
+                               const struct rocker_flow_tbl_entry *entry)
 {
        if (rocker_tlv_put_u32(desc_info, ROCKER_TLV_OF_DPA_IN_PPORT,
                               entry->key.ig_port.in_pport))
@@ -1922,8 +1899,9 @@ static int rocker_cmd_flow_tbl_add_ig_port(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_add_vlan(struct rocker_desc_info *desc_info,
-                                       struct rocker_flow_tbl_entry *entry)
+static int
+rocker_cmd_flow_tbl_add_vlan(struct rocker_desc_info *desc_info,
+                            const struct rocker_flow_tbl_entry *entry)
 {
        if (rocker_tlv_put_u32(desc_info, ROCKER_TLV_OF_DPA_IN_PPORT,
                               entry->key.vlan.in_pport))
@@ -1945,8 +1923,9 @@ static int rocker_cmd_flow_tbl_add_vlan(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_add_term_mac(struct rocker_desc_info *desc_info,
-                                           struct rocker_flow_tbl_entry *entry)
+static int
+rocker_cmd_flow_tbl_add_term_mac(struct rocker_desc_info *desc_info,
+                                const struct rocker_flow_tbl_entry *entry)
 {
        if (rocker_tlv_put_u32(desc_info, ROCKER_TLV_OF_DPA_IN_PPORT,
                               entry->key.term_mac.in_pport))
@@ -1982,7 +1961,7 @@ static int rocker_cmd_flow_tbl_add_term_mac(struct rocker_desc_info *desc_info,
 
 static int
 rocker_cmd_flow_tbl_add_ucast_routing(struct rocker_desc_info *desc_info,
-                                     struct rocker_flow_tbl_entry *entry)
+                                     const struct rocker_flow_tbl_entry *entry)
 {
        if (rocker_tlv_put_be16(desc_info, ROCKER_TLV_OF_DPA_ETHERTYPE,
                                entry->key.ucast_routing.eth_type))
@@ -2003,8 +1982,9 @@ rocker_cmd_flow_tbl_add_ucast_routing(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_add_bridge(struct rocker_desc_info *desc_info,
-                                         struct rocker_flow_tbl_entry *entry)
+static int
+rocker_cmd_flow_tbl_add_bridge(struct rocker_desc_info *desc_info,
+                              const struct rocker_flow_tbl_entry *entry)
 {
        if (entry->key.bridge.has_eth_dst &&
            rocker_tlv_put(desc_info, ROCKER_TLV_OF_DPA_DST_MAC,
@@ -2036,8 +2016,9 @@ static int rocker_cmd_flow_tbl_add_bridge(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_add_acl(struct rocker_desc_info *desc_info,
-                                      struct rocker_flow_tbl_entry *entry)
+static int
+rocker_cmd_flow_tbl_add_acl(struct rocker_desc_info *desc_info,
+                           const struct rocker_flow_tbl_entry *entry)
 {
        if (rocker_tlv_put_u32(desc_info, ROCKER_TLV_OF_DPA_IN_PPORT,
                               entry->key.acl.in_pport))
@@ -2102,12 +2083,11 @@ static int rocker_cmd_flow_tbl_add_acl(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_add(struct rocker *rocker,
-                                  struct rocker_port *rocker_port,
+static int rocker_cmd_flow_tbl_add(const struct rocker_port *rocker_port,
                                   struct rocker_desc_info *desc_info,
                                   void *priv)
 {
-       struct rocker_flow_tbl_entry *entry = priv;
+       const struct rocker_flow_tbl_entry *entry = priv;
        struct rocker_tlv *cmd_info;
        int err = 0;
 
@@ -2160,8 +2140,7 @@ static int rocker_cmd_flow_tbl_add(struct rocker *rocker,
        return 0;
 }
 
-static int rocker_cmd_flow_tbl_del(struct rocker *rocker,
-                                  struct rocker_port *rocker_port,
+static int rocker_cmd_flow_tbl_del(const struct rocker_port *rocker_port,
                                   struct rocker_desc_info *desc_info,
                                   void *priv)
 {
@@ -2197,7 +2176,7 @@ rocker_cmd_group_tbl_add_l2_interface(struct rocker_desc_info *desc_info,
 
 static int
 rocker_cmd_group_tbl_add_l2_rewrite(struct rocker_desc_info *desc_info,
-                                   struct rocker_group_tbl_entry *entry)
+                                   const struct rocker_group_tbl_entry *entry)
 {
        if (rocker_tlv_put_u32(desc_info, ROCKER_TLV_OF_DPA_GROUP_ID_LOWER,
                               entry->l2_rewrite.group_id))
@@ -2220,7 +2199,7 @@ rocker_cmd_group_tbl_add_l2_rewrite(struct rocker_desc_info *desc_info,
 
 static int
 rocker_cmd_group_tbl_add_group_ids(struct rocker_desc_info *desc_info,
-                                  struct rocker_group_tbl_entry *entry)
+                                  const struct rocker_group_tbl_entry *entry)
 {
        int i;
        struct rocker_tlv *group_ids;
@@ -2246,7 +2225,7 @@ rocker_cmd_group_tbl_add_group_ids(struct rocker_desc_info *desc_info,
 
 static int
 rocker_cmd_group_tbl_add_l3_unicast(struct rocker_desc_info *desc_info,
-                                   struct rocker_group_tbl_entry *entry)
+                                   const struct rocker_group_tbl_entry *entry)
 {
        if (!is_zero_ether_addr(entry->l3_unicast.eth_src) &&
            rocker_tlv_put(desc_info, ROCKER_TLV_OF_DPA_SRC_MAC,
@@ -2270,8 +2249,7 @@ rocker_cmd_group_tbl_add_l3_unicast(struct rocker_desc_info *desc_info,
        return 0;
 }
 
-static int rocker_cmd_group_tbl_add(struct rocker *rocker,
-                                   struct rocker_port *rocker_port,
+static int rocker_cmd_group_tbl_add(const struct rocker_port *rocker_port,
                                    struct rocker_desc_info *desc_info,
                                    void *priv)
 {
@@ -2316,8 +2294,7 @@ static int rocker_cmd_group_tbl_add(struct rocker *rocker,
        return 0;
 }
 
-static int rocker_cmd_group_tbl_del(struct rocker *rocker,
-                                   struct rocker_port *rocker_port,
+static int rocker_cmd_group_tbl_del(const struct rocker_port *rocker_port,
                                    struct rocker_desc_info *desc_info,
                                    void *priv)
 {
@@ -2400,7 +2377,8 @@ static void rocker_free_tbls(struct rocker *rocker)
 }
 
 static struct rocker_flow_tbl_entry *
-rocker_flow_tbl_find(struct rocker *rocker, struct rocker_flow_tbl_entry *match)
+rocker_flow_tbl_find(const struct rocker *rocker,
+                    const struct rocker_flow_tbl_entry *match)
 {
        struct rocker_flow_tbl_entry *found;
        size_t key_len = match->key_len ? match->key_len : sizeof(found->key);
@@ -2415,17 +2393,17 @@ rocker_flow_tbl_find(struct rocker *rocker, struct rocker_flow_tbl_entry *match)
 }
 
 static int rocker_flow_tbl_add(struct rocker_port *rocker_port,
-                              enum switchdev_trans trans,
+                              enum switchdev_trans trans, int flags,
                               struct rocker_flow_tbl_entry *match)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_flow_tbl_entry *found;
        size_t key_len = match->key_len ? match->key_len : sizeof(found->key);
-       unsigned long flags;
+       unsigned long lock_flags;
 
        match->key_crc32 = crc32(~0, &match->key, key_len);
 
-       spin_lock_irqsave(&rocker->flow_tbl_lock, flags);
+       spin_lock_irqsave(&rocker->flow_tbl_lock, lock_flags);
 
        found = rocker_flow_tbl_find(rocker, match);
 
@@ -2445,26 +2423,25 @@ static int rocker_flow_tbl_add(struct rocker_port *rocker_port,
        if (trans != SWITCHDEV_TRANS_PREPARE)
                hash_add(rocker->flow_tbl, &found->entry, found->key_crc32);
 
-       spin_unlock_irqrestore(&rocker->flow_tbl_lock, flags);
+       spin_unlock_irqrestore(&rocker->flow_tbl_lock, lock_flags);
 
-       return rocker_cmd_exec(rocker, rocker_port, trans,
-                              rocker_cmd_flow_tbl_add,
-                              found, NULL, NULL);
+       return rocker_cmd_exec(rocker_port, trans, flags,
+                              rocker_cmd_flow_tbl_add, found, NULL, NULL);
 }
 
 static int rocker_flow_tbl_del(struct rocker_port *rocker_port,
-                              enum switchdev_trans trans,
+                              enum switchdev_trans trans, int flags,
                               struct rocker_flow_tbl_entry *match)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_flow_tbl_entry *found;
        size_t key_len = match->key_len ? match->key_len : sizeof(found->key);
-       unsigned long flags;
+       unsigned long lock_flags;
        int err = 0;
 
        match->key_crc32 = crc32(~0, &match->key, key_len);
 
-       spin_lock_irqsave(&rocker->flow_tbl_lock, flags);
+       spin_lock_irqsave(&rocker->flow_tbl_lock, lock_flags);
 
        found = rocker_flow_tbl_find(rocker, match);
 
@@ -2474,12 +2451,12 @@ static int rocker_flow_tbl_del(struct rocker_port *rocker_port,
                found->cmd = ROCKER_TLV_CMD_TYPE_OF_DPA_FLOW_DEL;
        }
 
-       spin_unlock_irqrestore(&rocker->flow_tbl_lock, flags);
+       spin_unlock_irqrestore(&rocker->flow_tbl_lock, lock_flags);
 
        rocker_port_kfree(trans, match);
 
        if (found) {
-               err = rocker_cmd_exec(rocker, rocker_port, trans,
+               err = rocker_cmd_exec(rocker_port, trans, flags,
                                      rocker_cmd_flow_tbl_del,
                                      found, NULL, NULL);
                rocker_port_kfree(trans, found);
@@ -2493,9 +2470,9 @@ static int rocker_flow_tbl_do(struct rocker_port *rocker_port,
                              struct rocker_flow_tbl_entry *entry)
 {
        if (flags & ROCKER_OP_FLAG_REMOVE)
-               return rocker_flow_tbl_del(rocker_port, trans, entry);
+               return rocker_flow_tbl_del(rocker_port, trans, flags, entry);
        else
-               return rocker_flow_tbl_add(rocker_port, trans, entry);
+               return rocker_flow_tbl_add(rocker_port, trans, flags, entry);
 }
 
 static int rocker_flow_tbl_ig_port(struct rocker_port *rocker_port,
@@ -2505,7 +2482,7 @@ static int rocker_flow_tbl_ig_port(struct rocker_port *rocker_port,
 {
        struct rocker_flow_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2527,7 +2504,7 @@ static int rocker_flow_tbl_vlan(struct rocker_port *rocker_port,
 {
        struct rocker_flow_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2554,7 +2531,7 @@ static int rocker_flow_tbl_term_mac(struct rocker_port *rocker_port,
 {
        struct rocker_flow_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2594,7 +2571,7 @@ static int rocker_flow_tbl_bridge(struct rocker_port *rocker_port,
        bool dflt = !eth_dst || (eth_dst && eth_dst_mask);
        bool wild = false;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2644,7 +2621,7 @@ static int rocker_flow_tbl_ucast4_routing(struct rocker_port *rocker_port,
 {
        struct rocker_flow_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2674,7 +2651,7 @@ static int rocker_flow_tbl_acl(struct rocker_port *rocker_port,
        u32 priority;
        struct rocker_flow_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2713,8 +2690,8 @@ static int rocker_flow_tbl_acl(struct rocker_port *rocker_port,
 }
 
 static struct rocker_group_tbl_entry *
-rocker_group_tbl_find(struct rocker *rocker,
-                     struct rocker_group_tbl_entry *match)
+rocker_group_tbl_find(const struct rocker *rocker,
+                     const struct rocker_group_tbl_entry *match)
 {
        struct rocker_group_tbl_entry *found;
 
@@ -2742,14 +2719,14 @@ static void rocker_group_tbl_entry_free(enum switchdev_trans trans,
 }
 
 static int rocker_group_tbl_add(struct rocker_port *rocker_port,
-                               enum switchdev_trans trans,
+                               enum switchdev_trans trans, int flags,
                                struct rocker_group_tbl_entry *match)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_group_tbl_entry *found;
-       unsigned long flags;
+       unsigned long lock_flags;
 
-       spin_lock_irqsave(&rocker->group_tbl_lock, flags);
+       spin_lock_irqsave(&rocker->group_tbl_lock, lock_flags);
 
        found = rocker_group_tbl_find(rocker, match);
 
@@ -2767,23 +2744,22 @@ static int rocker_group_tbl_add(struct rocker_port *rocker_port,
        if (trans != SWITCHDEV_TRANS_PREPARE)
                hash_add(rocker->group_tbl, &found->entry, found->group_id);
 
-       spin_unlock_irqrestore(&rocker->group_tbl_lock, flags);
+       spin_unlock_irqrestore(&rocker->group_tbl_lock, lock_flags);
 
-       return rocker_cmd_exec(rocker, rocker_port, trans,
-                              rocker_cmd_group_tbl_add,
-                              found, NULL, NULL);
+       return rocker_cmd_exec(rocker_port, trans, flags,
+                              rocker_cmd_group_tbl_add, found, NULL, NULL);
 }
 
 static int rocker_group_tbl_del(struct rocker_port *rocker_port,
-                               enum switchdev_trans trans,
+                               enum switchdev_trans trans, int flags,
                                struct rocker_group_tbl_entry *match)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_group_tbl_entry *found;
-       unsigned long flags;
+       unsigned long lock_flags;
        int err = 0;
 
-       spin_lock_irqsave(&rocker->group_tbl_lock, flags);
+       spin_lock_irqsave(&rocker->group_tbl_lock, lock_flags);
 
        found = rocker_group_tbl_find(rocker, match);
 
@@ -2793,12 +2769,12 @@ static int rocker_group_tbl_del(struct rocker_port *rocker_port,
                found->cmd = ROCKER_TLV_CMD_TYPE_OF_DPA_GROUP_DEL;
        }
 
-       spin_unlock_irqrestore(&rocker->group_tbl_lock, flags);
+       spin_unlock_irqrestore(&rocker->group_tbl_lock, lock_flags);
 
        rocker_group_tbl_entry_free(trans, match);
 
        if (found) {
-               err = rocker_cmd_exec(rocker, rocker_port, trans,
+               err = rocker_cmd_exec(rocker_port, trans, flags,
                                      rocker_cmd_group_tbl_del,
                                      found, NULL, NULL);
                rocker_group_tbl_entry_free(trans, found);
@@ -2812,9 +2788,9 @@ static int rocker_group_tbl_do(struct rocker_port *rocker_port,
                               struct rocker_group_tbl_entry *entry)
 {
        if (flags & ROCKER_OP_FLAG_REMOVE)
-               return rocker_group_tbl_del(rocker_port, trans, entry);
+               return rocker_group_tbl_del(rocker_port, trans, flags, entry);
        else
-               return rocker_group_tbl_add(rocker_port, trans, entry);
+               return rocker_group_tbl_add(rocker_port, trans, flags, entry);
 }
 
 static int rocker_group_l2_interface(struct rocker_port *rocker_port,
@@ -2824,7 +2800,7 @@ static int rocker_group_l2_interface(struct rocker_port *rocker_port,
 {
        struct rocker_group_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2837,19 +2813,19 @@ static int rocker_group_l2_interface(struct rocker_port *rocker_port,
 static int rocker_group_l2_fan_out(struct rocker_port *rocker_port,
                                   enum switchdev_trans trans,
                                   int flags, u8 group_count,
-                                  u32 *group_ids, u32 group_id)
+                                  const u32 *group_ids, u32 group_id)
 {
        struct rocker_group_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
        entry->group_id = group_id;
        entry->group_count = group_count;
 
-       entry->group_ids = rocker_port_kcalloc(rocker_port, trans, group_count,
-                                              sizeof(u32));
+       entry->group_ids = rocker_port_kcalloc(rocker_port, trans, flags,
+                                              group_count, sizeof(u32));
        if (!entry->group_ids) {
                rocker_port_kfree(trans, entry);
                return -ENOMEM;
@@ -2862,7 +2838,7 @@ static int rocker_group_l2_fan_out(struct rocker_port *rocker_port,
 static int rocker_group_l2_flood(struct rocker_port *rocker_port,
                                 enum switchdev_trans trans, int flags,
                                 __be16 vlan_id, u8 group_count,
-                                u32 *group_ids, u32 group_id)
+                                const u32 *group_ids, u32 group_id)
 {
        return rocker_group_l2_fan_out(rocker_port, trans, flags,
                                       group_count, group_ids,
@@ -2871,12 +2847,12 @@ static int rocker_group_l2_flood(struct rocker_port *rocker_port,
 
 static int rocker_group_l3_unicast(struct rocker_port *rocker_port,
                                   enum switchdev_trans trans, int flags,
-                                  u32 index, u8 *src_mac, u8 *dst_mac,
+                                  u32 index, const u8 *src_mac, const u8 *dst_mac,
                                   __be16 vlan_id, bool ttl_check, u32 pport)
 {
        struct rocker_group_tbl_entry *entry;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -2893,7 +2869,7 @@ static int rocker_group_l3_unicast(struct rocker_port *rocker_port,
 }
 
 static struct rocker_neigh_tbl_entry *
-       rocker_neigh_tbl_find(struct rocker *rocker, __be32 ip_addr)
+rocker_neigh_tbl_find(const struct rocker *rocker, __be32 ip_addr)
 {
        struct rocker_neigh_tbl_entry *found;
 
@@ -2909,10 +2885,10 @@ static void _rocker_neigh_add(struct rocker *rocker,
                              enum switchdev_trans trans,
                              struct rocker_neigh_tbl_entry *entry)
 {
-       entry->index = rocker->neigh_tbl_next_index;
+       if (trans != SWITCHDEV_TRANS_COMMIT)
+               entry->index = rocker->neigh_tbl_next_index++;
        if (trans == SWITCHDEV_TRANS_PREPARE)
                return;
-       rocker->neigh_tbl_next_index++;
        entry->ref_count++;
        hash_add(rocker->neigh_tbl, &entry->entry,
                 be32_to_cpu(entry->ip_addr));
@@ -2931,7 +2907,7 @@ static void _rocker_neigh_del(enum switchdev_trans trans,
 
 static void _rocker_neigh_update(struct rocker_neigh_tbl_entry *entry,
                                 enum switchdev_trans trans,
-                                u8 *eth_dst, bool ttl_check)
+                                const u8 *eth_dst, bool ttl_check)
 {
        if (eth_dst) {
                ether_addr_copy(entry->eth_dst, eth_dst);
@@ -2943,7 +2919,7 @@ static void _rocker_neigh_update(struct rocker_neigh_tbl_entry *entry,
 
 static int rocker_port_ipv4_neigh(struct rocker_port *rocker_port,
                                  enum switchdev_trans trans,
-                                 int flags, __be32 ip_addr, u8 *eth_dst)
+                                 int flags, __be32 ip_addr, const u8 *eth_dst)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_neigh_tbl_entry *entry;
@@ -2959,7 +2935,7 @@ static int rocker_port_ipv4_neigh(struct rocker_port *rocker_port,
        bool removing;
        int err = 0;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -3075,7 +3051,7 @@ static int rocker_port_ipv4_nh(struct rocker_port *rocker_port,
        bool resolved = true;
        int err = 0;
 
-       entry = rocker_port_kzalloc(rocker_port, trans, sizeof(*entry));
+       entry = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*entry));
        if (!entry)
                return -ENOMEM;
 
@@ -3125,15 +3101,15 @@ static int rocker_port_vlan_flood_group(struct rocker_port *rocker_port,
                                        int flags, __be16 vlan_id)
 {
        struct rocker_port *p;
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        u32 group_id = ROCKER_GROUP_L2_FLOOD(vlan_id, 0);
        u32 *group_ids;
        u8 group_count = 0;
        int err = 0;
        int i;
 
-       group_ids = rocker_port_kcalloc(rocker_port, trans, rocker->port_count,
-                                       sizeof(u32));
+       group_ids = rocker_port_kcalloc(rocker_port, trans, flags,
+                                       rocker->port_count, sizeof(u32));
        if (!group_ids)
                return -ENOMEM;
 
@@ -3144,6 +3120,8 @@ static int rocker_port_vlan_flood_group(struct rocker_port *rocker_port,
 
        for (i = 0; i < rocker->port_count; i++) {
                p = rocker->ports[i];
+               if (!p)
+                       continue;
                if (!rocker_port_is_bridged(p))
                        continue;
                if (test_bit(ntohs(vlan_id), p->vlan_bitmap)) {
@@ -3171,7 +3149,7 @@ static int rocker_port_vlan_l2_groups(struct rocker_port *rocker_port,
                                      enum switchdev_trans trans, int flags,
                                      __be16 vlan_id, bool pop_vlan)
 {
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        struct rocker_port *p;
        bool adding = !(flags & ROCKER_OP_FLAG_REMOVE);
        u32 out_pport;
@@ -3203,7 +3181,7 @@ static int rocker_port_vlan_l2_groups(struct rocker_port *rocker_port,
 
        for (i = 0; i < rocker->port_count; i++) {
                p = rocker->ports[i];
-               if (test_bit(ntohs(vlan_id), p->vlan_bitmap))
+               if (p && test_bit(ntohs(vlan_id), p->vlan_bitmap))
                        ref++;
        }
 
@@ -3269,13 +3247,13 @@ static struct rocker_ctrl {
 
 static int rocker_port_ctrl_vlan_acl(struct rocker_port *rocker_port,
                                     enum switchdev_trans trans, int flags,
-                                    struct rocker_ctrl *ctrl, __be16 vlan_id)
+                                    const struct rocker_ctrl *ctrl, __be16 vlan_id)
 {
        u32 in_pport = rocker_port->pport;
        u32 in_pport_mask = 0xffffffff;
        u32 out_pport = 0;
-       u8 *eth_src = NULL;
-       u8 *eth_src_mask = NULL;
+       const u8 *eth_src = NULL;
+       const u8 *eth_src_mask = NULL;
        __be16 vlan_id_mask = htons(0xffff);
        u8 ip_proto = 0;
        u8 ip_proto_mask = 0;
@@ -3302,7 +3280,7 @@ static int rocker_port_ctrl_vlan_acl(struct rocker_port *rocker_port,
 
 static int rocker_port_ctrl_vlan_bridge(struct rocker_port *rocker_port,
                                        enum switchdev_trans trans, int flags,
-                                       struct rocker_ctrl *ctrl,
+                                       const struct rocker_ctrl *ctrl,
                                        __be16 vlan_id)
 {
        enum rocker_of_dpa_table_id goto_tbl =
@@ -3327,7 +3305,7 @@ static int rocker_port_ctrl_vlan_bridge(struct rocker_port *rocker_port,
 
 static int rocker_port_ctrl_vlan_term(struct rocker_port *rocker_port,
                                      enum switchdev_trans trans, int flags,
-                                     struct rocker_ctrl *ctrl, __be16 vlan_id)
+                                     const struct rocker_ctrl *ctrl, __be16 vlan_id)
 {
        u32 in_pport_mask = 0xffffffff;
        __be16 vlan_id_mask = htons(0xffff);
@@ -3351,7 +3329,7 @@ static int rocker_port_ctrl_vlan_term(struct rocker_port *rocker_port,
 
 static int rocker_port_ctrl_vlan(struct rocker_port *rocker_port,
                                 enum switchdev_trans trans, int flags,
-                                struct rocker_ctrl *ctrl, __be16 vlan_id)
+                                const struct rocker_ctrl *ctrl, __be16 vlan_id)
 {
        if (ctrl->acl)
                return rocker_port_ctrl_vlan_acl(rocker_port, trans, flags,
@@ -3388,7 +3366,7 @@ static int rocker_port_ctrl_vlan_add(struct rocker_port *rocker_port,
 
 static int rocker_port_ctrl(struct rocker_port *rocker_port,
                            enum switchdev_trans trans, int flags,
-                           struct rocker_ctrl *ctrl)
+                           const struct rocker_ctrl *ctrl)
 {
        u16 vid;
        int err = 0;
@@ -3506,7 +3484,7 @@ struct rocker_fdb_learn_work {
 
 static void rocker_port_fdb_learn_work(struct work_struct *work)
 {
-       struct rocker_fdb_learn_work *lw =
+       const struct rocker_fdb_learn_work *lw =
                container_of(work, struct rocker_fdb_learn_work, work);
        bool removing = (lw->flags & ROCKER_OP_FLAG_REMOVE);
        bool learned = (lw->flags & ROCKER_OP_FLAG_LEARNED);
@@ -3556,7 +3534,7 @@ static int rocker_port_fdb_learn(struct rocker_port *rocker_port,
        if (!rocker_port_is_bridged(rocker_port))
                return 0;
 
-       lw = rocker_port_kzalloc(rocker_port, trans, sizeof(*lw));
+       lw = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*lw));
        if (!lw)
                return -ENOMEM;
 
@@ -3577,7 +3555,8 @@ static int rocker_port_fdb_learn(struct rocker_port *rocker_port,
 }
 
 static struct rocker_fdb_tbl_entry *
-rocker_fdb_tbl_find(struct rocker *rocker, struct rocker_fdb_tbl_entry *match)
+rocker_fdb_tbl_find(const struct rocker *rocker,
+                   const struct rocker_fdb_tbl_entry *match)
 {
        struct rocker_fdb_tbl_entry *found;
 
@@ -3599,7 +3578,7 @@ static int rocker_port_fdb(struct rocker_port *rocker_port,
        bool removing = (flags & ROCKER_OP_FLAG_REMOVE);
        unsigned long lock_flags;
 
-       fdb = rocker_port_kzalloc(rocker_port, trans, sizeof(*fdb));
+       fdb = rocker_port_kzalloc(rocker_port, trans, flags, sizeof(*fdb));
        if (!fdb)
                return -ENOMEM;
 
@@ -3637,12 +3616,11 @@ static int rocker_port_fdb(struct rocker_port *rocker_port,
 }
 
 static int rocker_port_fdb_flush(struct rocker_port *rocker_port,
-                                enum switchdev_trans trans)
+                                enum switchdev_trans trans, int flags)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_fdb_tbl_entry *found;
        unsigned long lock_flags;
-       int flags = ROCKER_OP_FLAG_REMOVE;
        struct hlist_node *tmp;
        int bkt;
        int err = 0;
@@ -3651,6 +3629,8 @@ static int rocker_port_fdb_flush(struct rocker_port *rocker_port,
            rocker_port->stp_state == BR_STATE_FORWARDING)
                return 0;
 
+       flags |= ROCKER_OP_FLAG_REMOVE;
+
        spin_lock_irqsave(&rocker->fdb_tbl_lock, lock_flags);
 
        hash_for_each_safe(rocker->fdb_tbl, bkt, tmp, found, entry) {
@@ -3707,13 +3687,12 @@ static int rocker_port_router_mac(struct rocker_port *rocker_port,
 }
 
 static int rocker_port_fwding(struct rocker_port *rocker_port,
-                             enum switchdev_trans trans)
+                             enum switchdev_trans trans, int flags)
 {
        bool pop_vlan;
        u32 out_pport;
        __be16 vlan_id;
        u16 vid;
-       int flags = 0;
        int err;
 
        /* Port will be forwarding-enabled if its STP state is LEARNING
@@ -3747,12 +3726,12 @@ static int rocker_port_fwding(struct rocker_port *rocker_port,
 }
 
 static int rocker_port_stp_update(struct rocker_port *rocker_port,
-                                 enum switchdev_trans trans, u8 state)
+                                 enum switchdev_trans trans, int flags,
+                                 u8 state)
 {
        bool want[ROCKER_CTRL_MAX] = { 0, };
        bool prev_ctrls[ROCKER_CTRL_MAX];
        u8 prev_state;
-       int flags;
        int err;
        int i;
 
@@ -3788,8 +3767,9 @@ static int rocker_port_stp_update(struct rocker_port *rocker_port,
 
        for (i = 0; i < ROCKER_CTRL_MAX; i++) {
                if (want[i] != rocker_port->ctrls[i]) {
-                       flags = (want[i] ? 0 : ROCKER_OP_FLAG_REMOVE);
-                       err = rocker_port_ctrl(rocker_port, trans, flags,
+                       int ctrl_flags = flags |
+                                        (want[i] ? 0 : ROCKER_OP_FLAG_REMOVE);
+                       err = rocker_port_ctrl(rocker_port, trans, ctrl_flags,
                                               &rocker_ctrls[i]);
                        if (err)
                                goto err_out;
@@ -3797,11 +3777,11 @@ static int rocker_port_stp_update(struct rocker_port *rocker_port,
                }
        }
 
-       err = rocker_port_fdb_flush(rocker_port, trans);
+       err = rocker_port_fdb_flush(rocker_port, trans, flags);
        if (err)
                goto err_out;
 
-       err = rocker_port_fwding(rocker_port, trans);
+       err = rocker_port_fwding(rocker_port, trans, flags);
 
 err_out:
        if (trans == SWITCHDEV_TRANS_PREPARE) {
@@ -3813,29 +3793,31 @@ err_out:
 }
 
 static int rocker_port_fwd_enable(struct rocker_port *rocker_port,
-                                 enum switchdev_trans trans)
+                                 enum switchdev_trans trans, int flags)
 {
        if (rocker_port_is_bridged(rocker_port))
                /* bridge STP will enable port */
                return 0;
 
        /* port is not bridged, so simulate going to FORWARDING state */
-       return rocker_port_stp_update(rocker_port, trans, BR_STATE_FORWARDING);
+       return rocker_port_stp_update(rocker_port, trans, flags,
+                                     BR_STATE_FORWARDING);
 }
 
 static int rocker_port_fwd_disable(struct rocker_port *rocker_port,
-                                  enum switchdev_trans trans)
+                                  enum switchdev_trans trans, int flags)
 {
        if (rocker_port_is_bridged(rocker_port))
                /* bridge STP will disable port */
                return 0;
 
        /* port is not bridged, so simulate going to DISABLED state */
-       return rocker_port_stp_update(rocker_port, trans, BR_STATE_DISABLED);
+       return rocker_port_stp_update(rocker_port, trans, flags,
+                                     BR_STATE_DISABLED);
 }
 
 static struct rocker_internal_vlan_tbl_entry *
-rocker_internal_vlan_tbl_find(struct rocker *rocker, int ifindex)
+rocker_internal_vlan_tbl_find(const struct rocker *rocker, int ifindex)
 {
        struct rocker_internal_vlan_tbl_entry *found;
 
@@ -3890,8 +3872,9 @@ found:
        return found->vlan_id;
 }
 
-static void rocker_port_internal_vlan_id_put(struct rocker_port *rocker_port,
-                                            int ifindex)
+static void
+rocker_port_internal_vlan_id_put(const struct rocker_port *rocker_port,
+                                int ifindex)
 {
        struct rocker *rocker = rocker_port->rocker;
        struct rocker_internal_vlan_tbl_entry *found;
@@ -3921,10 +3904,10 @@ not_found:
 
 static int rocker_port_fib_ipv4(struct rocker_port *rocker_port,
                                enum switchdev_trans trans, __be32 dst,
-                               int dst_len, struct fib_info *fi, u32 tb_id,
-                               int flags)
+                               int dst_len, const struct fib_info *fi,
+                               u32 tb_id, int flags)
 {
-       struct fib_nh *nh;
+       const struct fib_nh *nh;
        __be16 eth_type = htons(ETH_P_IP);
        __be32 dst_mask = inet_make_mask(dst_len);
        __be16 internal_vlan_id = rocker_port->internal_vlan_id;
@@ -3994,7 +3977,7 @@ static int rocker_port_open(struct net_device *dev)
                goto err_request_rx_irq;
        }
 
-       err = rocker_port_fwd_enable(rocker_port, SWITCHDEV_TRANS_NONE);
+       err = rocker_port_fwd_enable(rocker_port, SWITCHDEV_TRANS_NONE, 0);
        if (err)
                goto err_fwd_enable;
 
@@ -4021,7 +4004,7 @@ static int rocker_port_stop(struct net_device *dev)
        rocker_port_set_enable(rocker_port, false);
        napi_disable(&rocker_port->napi_rx);
        napi_disable(&rocker_port->napi_tx);
-       rocker_port_fwd_disable(rocker_port, SWITCHDEV_TRANS_NONE);
+       rocker_port_fwd_disable(rocker_port, SWITCHDEV_TRANS_NONE, 0);
        free_irq(rocker_msix_rx_vector(rocker_port), rocker_port);
        free_irq(rocker_msix_tx_vector(rocker_port), rocker_port);
        rocker_port_dma_rings_fini(rocker_port);
@@ -4029,12 +4012,12 @@ static int rocker_port_stop(struct net_device *dev)
        return 0;
 }
 
-static void rocker_tx_desc_frags_unmap(struct rocker_port *rocker_port,
-                                      struct rocker_desc_info *desc_info)
+static void rocker_tx_desc_frags_unmap(const struct rocker_port *rocker_port,
+                                      const struct rocker_desc_info *desc_info)
 {
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        struct pci_dev *pdev = rocker->pdev;
-       struct rocker_tlv *attrs[ROCKER_TLV_TX_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_TX_MAX + 1];
        struct rocker_tlv *attr;
        int rem;
 
@@ -4042,7 +4025,7 @@ static void rocker_tx_desc_frags_unmap(struct rocker_port *rocker_port,
        if (!attrs[ROCKER_TLV_TX_FRAGS])
                return;
        rocker_tlv_for_each_nested(attr, attrs[ROCKER_TLV_TX_FRAGS], rem) {
-               struct rocker_tlv *frag_attrs[ROCKER_TLV_TX_FRAG_ATTR_MAX + 1];
+               const struct rocker_tlv *frag_attrs[ROCKER_TLV_TX_FRAG_ATTR_MAX + 1];
                dma_addr_t dma_handle;
                size_t len;
 
@@ -4059,11 +4042,11 @@ static void rocker_tx_desc_frags_unmap(struct rocker_port *rocker_port,
        }
 }
 
-static int rocker_tx_desc_frag_map_put(struct rocker_port *rocker_port,
+static int rocker_tx_desc_frag_map_put(const struct rocker_port *rocker_port,
                                       struct rocker_desc_info *desc_info,
                                       char *buf, size_t buf_len)
 {
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        struct pci_dev *pdev = rocker->pdev;
        dma_addr_t dma_handle;
        struct rocker_tlv *frag;
@@ -4168,35 +4151,6 @@ static int rocker_port_set_mac_address(struct net_device *dev, void *p)
        return 0;
 }
 
-static int rocker_port_vlan_rx_add_vid(struct net_device *dev,
-                                      __be16 proto, u16 vid)
-{
-       struct rocker_port *rocker_port = netdev_priv(dev);
-       int err;
-
-       err = rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE, 0, vid);
-       if (err)
-               return err;
-
-       return rocker_port_router_mac(rocker_port, SWITCHDEV_TRANS_NONE,
-                                     0, htons(vid));
-}
-
-static int rocker_port_vlan_rx_kill_vid(struct net_device *dev,
-                                       __be16 proto, u16 vid)
-{
-       struct rocker_port *rocker_port = netdev_priv(dev);
-       int err;
-
-       err = rocker_port_router_mac(rocker_port, SWITCHDEV_TRANS_NONE,
-                                    ROCKER_OP_FLAG_REMOVE, htons(vid));
-       if (err)
-               return err;
-
-       return rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE,
-                               ROCKER_OP_FLAG_REMOVE, vid);
-}
-
 static int rocker_port_get_phys_port_name(struct net_device *dev,
                                          char *buf, size_t len)
 {
@@ -4204,8 +4158,7 @@ static int rocker_port_get_phys_port_name(struct net_device *dev,
        struct port_name name = { .buf = buf, .len = len };
        int err;
 
-       err = rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                             SWITCHDEV_TRANS_NONE,
+       err = rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                              rocker_cmd_get_port_settings_prep, NULL,
                              rocker_cmd_get_port_settings_phys_name_proc,
                              &name);
@@ -4218,8 +4171,6 @@ static const struct net_device_ops rocker_port_netdev_ops = {
        .ndo_stop                       = rocker_port_stop,
        .ndo_start_xmit                 = rocker_port_xmit,
        .ndo_set_mac_address            = rocker_port_set_mac_address,
-       .ndo_vlan_rx_add_vid            = rocker_port_vlan_rx_add_vid,
-       .ndo_vlan_rx_kill_vid           = rocker_port_vlan_rx_kill_vid,
        .ndo_bridge_getlink             = switchdev_port_bridge_getlink,
        .ndo_bridge_setlink             = switchdev_port_bridge_setlink,
        .ndo_bridge_dellink             = switchdev_port_bridge_dellink,
@@ -4236,8 +4187,8 @@ static const struct net_device_ops rocker_port_netdev_ops = {
 static int rocker_port_attr_get(struct net_device *dev,
                                struct switchdev_attr *attr)
 {
-       struct rocker_port *rocker_port = netdev_priv(dev);
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker_port *rocker_port = netdev_priv(dev);
+       const struct rocker *rocker = rocker_port->rocker;
 
        switch (attr->id) {
        case SWITCHDEV_ATTR_PORT_PARENT_ID:
@@ -4254,7 +4205,7 @@ static int rocker_port_attr_get(struct net_device *dev,
        return 0;
 }
 
-static void rocker_port_trans_abort(struct rocker_port *rocker_port)
+static void rocker_port_trans_abort(const struct rocker_port *rocker_port)
 {
        struct list_head *mem, *tmp;
 
@@ -4302,6 +4253,7 @@ static int rocker_port_attr_set(struct net_device *dev,
        switch (attr->id) {
        case SWITCHDEV_ATTR_PORT_STP_STATE:
                err = rocker_port_stp_update(rocker_port, attr->trans,
+                                            ROCKER_OP_FLAG_NOWAIT,
                                             attr->u.stp_state);
                break;
        case SWITCHDEV_ATTR_PORT_BRIDGE_FLAGS:
@@ -4327,12 +4279,17 @@ static int rocker_port_vlan_add(struct rocker_port *rocker_port,
        if (err)
                return err;
 
-       return rocker_port_router_mac(rocker_port, trans, 0, htons(vid));
+       err = rocker_port_router_mac(rocker_port, trans, 0, htons(vid));
+       if (err)
+               rocker_port_vlan(rocker_port, trans,
+                                ROCKER_OP_FLAG_REMOVE, vid);
+
+       return err;
 }
 
 static int rocker_port_vlans_add(struct rocker_port *rocker_port,
                                 enum switchdev_trans trans,
-                                struct switchdev_obj_vlan *vlan)
+                                const struct switchdev_obj_vlan *vlan)
 {
        u16 vid;
        int err;
@@ -4349,7 +4306,7 @@ static int rocker_port_vlans_add(struct rocker_port *rocker_port,
 
 static int rocker_port_fdb_add(struct rocker_port *rocker_port,
                               enum switchdev_trans trans,
-                              struct switchdev_obj_fdb *fdb)
+                              const struct switchdev_obj_fdb *fdb)
 {
        __be16 vlan_id = rocker_port_vid_to_vlan(rocker_port, fdb->vid, NULL);
        int flags = 0;
@@ -4364,7 +4321,7 @@ static int rocker_port_obj_add(struct net_device *dev,
                               struct switchdev_obj *obj)
 {
        struct rocker_port *rocker_port = netdev_priv(dev);
-       struct switchdev_obj_ipv4_fib *fib4;
+       const struct switchdev_obj_ipv4_fib *fib4;
        int err = 0;
 
        switch (obj->trans) {
@@ -4415,7 +4372,7 @@ static int rocker_port_vlan_del(struct rocker_port *rocker_port,
 }
 
 static int rocker_port_vlans_del(struct rocker_port *rocker_port,
-                                struct switchdev_obj_vlan *vlan)
+                                const struct switchdev_obj_vlan *vlan)
 {
        u16 vid;
        int err;
@@ -4431,7 +4388,7 @@ static int rocker_port_vlans_del(struct rocker_port *rocker_port,
 
 static int rocker_port_fdb_del(struct rocker_port *rocker_port,
                               enum switchdev_trans trans,
-                              struct switchdev_obj_fdb *fdb)
+                              const struct switchdev_obj_fdb *fdb)
 {
        __be16 vlan_id = rocker_port_vid_to_vlan(rocker_port, fdb->vid, NULL);
        int flags = ROCKER_OP_FLAG_REMOVE;
@@ -4446,7 +4403,7 @@ static int rocker_port_obj_del(struct net_device *dev,
                               struct switchdev_obj *obj)
 {
        struct rocker_port *rocker_port = netdev_priv(dev);
-       struct switchdev_obj_ipv4_fib *fib4;
+       const struct switchdev_obj_ipv4_fib *fib4;
        int err = 0;
 
        switch (obj->id) {
@@ -4471,7 +4428,7 @@ static int rocker_port_obj_del(struct net_device *dev,
        return err;
 }
 
-static int rocker_port_fdb_dump(struct rocker_port *rocker_port,
+static int rocker_port_fdb_dump(const struct rocker_port *rocker_port,
                                struct switchdev_obj *obj)
 {
        struct rocker *rocker = rocker_port->rocker;
@@ -4501,7 +4458,7 @@ static int rocker_port_fdb_dump(struct rocker_port *rocker_port,
 static int rocker_port_obj_dump(struct net_device *dev,
                                struct switchdev_obj *obj)
 {
-       struct rocker_port *rocker_port = netdev_priv(dev);
+       const struct rocker_port *rocker_port = netdev_priv(dev);
        int err = 0;
 
        switch (obj->id) {
@@ -4585,8 +4542,7 @@ static void rocker_port_get_strings(struct net_device *netdev, u32 stringset,
 }
 
 static int
-rocker_cmd_get_port_stats_prep(struct rocker *rocker,
-                              struct rocker_port *rocker_port,
+rocker_cmd_get_port_stats_prep(const struct rocker_port *rocker_port,
                               struct rocker_desc_info *desc_info,
                               void *priv)
 {
@@ -4610,14 +4566,13 @@ rocker_cmd_get_port_stats_prep(struct rocker *rocker,
 }
 
 static int
-rocker_cmd_get_port_stats_ethtool_proc(struct rocker *rocker,
-                                      struct rocker_port *rocker_port,
-                                      struct rocker_desc_info *desc_info,
+rocker_cmd_get_port_stats_ethtool_proc(const struct rocker_port *rocker_port,
+                                      const struct rocker_desc_info *desc_info,
                                       void *priv)
 {
-       struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
-       struct rocker_tlv *stats_attrs[ROCKER_TLV_CMD_PORT_STATS_MAX + 1];
-       struct rocker_tlv *pattr;
+       const struct rocker_tlv *attrs[ROCKER_TLV_CMD_MAX + 1];
+       const struct rocker_tlv *stats_attrs[ROCKER_TLV_CMD_PORT_STATS_MAX + 1];
+       const struct rocker_tlv *pattr;
        u32 pport;
        u64 *data = priv;
        int i;
@@ -4651,8 +4606,7 @@ rocker_cmd_get_port_stats_ethtool_proc(struct rocker *rocker,
 static int rocker_cmd_get_port_stats_ethtool(struct rocker_port *rocker_port,
                                             void *priv)
 {
-       return rocker_cmd_exec(rocker_port->rocker, rocker_port,
-                              SWITCHDEV_TRANS_NONE,
+       return rocker_cmd_exec(rocker_port, SWITCHDEV_TRANS_NONE, 0,
                               rocker_cmd_get_port_stats_prep, NULL,
                               rocker_cmd_get_port_stats_ethtool_proc,
                               priv);
@@ -4703,8 +4657,8 @@ static struct rocker_port *rocker_port_napi_tx_get(struct napi_struct *napi)
 static int rocker_port_poll_tx(struct napi_struct *napi, int budget)
 {
        struct rocker_port *rocker_port = rocker_port_napi_tx_get(napi);
-       struct rocker *rocker = rocker_port->rocker;
-       struct rocker_desc_info *desc_info;
+       const struct rocker *rocker = rocker_port->rocker;
+       const struct rocker_desc_info *desc_info;
        u32 credits = 0;
        int err;
 
@@ -4739,11 +4693,11 @@ static int rocker_port_poll_tx(struct napi_struct *napi, int budget)
        return 0;
 }
 
-static int rocker_port_rx_proc(struct rocker *rocker,
-                              struct rocker_port *rocker_port,
+static int rocker_port_rx_proc(const struct rocker *rocker,
+                              const struct rocker_port *rocker_port,
                               struct rocker_desc_info *desc_info)
 {
-       struct rocker_tlv *attrs[ROCKER_TLV_RX_MAX + 1];
+       const struct rocker_tlv *attrs[ROCKER_TLV_RX_MAX + 1];
        struct sk_buff *skb = rocker_desc_cookie_ptr_get(desc_info);
        size_t rx_len;
 
@@ -4765,7 +4719,7 @@ static int rocker_port_rx_proc(struct rocker *rocker,
 
        netif_receive_skb(skb);
 
-       return rocker_dma_rx_ring_skb_alloc(rocker, rocker_port, desc_info);
+       return rocker_dma_rx_ring_skb_alloc(rocker_port, desc_info);
 }
 
 static struct rocker_port *rocker_port_napi_rx_get(struct napi_struct *napi)
@@ -4776,7 +4730,7 @@ static struct rocker_port *rocker_port_napi_rx_get(struct napi_struct *napi)
 static int rocker_port_poll_rx(struct napi_struct *napi, int budget)
 {
        struct rocker_port *rocker_port = rocker_port_napi_rx_get(napi);
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        struct rocker_desc_info *desc_info;
        u32 credits = 0;
        int err;
@@ -4816,9 +4770,9 @@ static int rocker_port_poll_rx(struct napi_struct *napi, int budget)
  * PCI driver ops
  *****************/
 
-static void rocker_carrier_init(struct rocker_port *rocker_port)
+static void rocker_carrier_init(const struct rocker_port *rocker_port)
 {
-       struct rocker *rocker = rocker_port->rocker;
+       const struct rocker *rocker = rocker_port->rocker;
        u64 link_status = rocker_read64(rocker, PORT_PHYS_LINK_STATUS);
        bool link_up;
 
@@ -4829,13 +4783,15 @@ static void rocker_carrier_init(struct rocker_port *rocker_port)
                netif_carrier_off(rocker_port->dev);
 }
 
-static void rocker_remove_ports(struct rocker *rocker)
+static void rocker_remove_ports(const struct rocker *rocker)
 {
        struct rocker_port *rocker_port;
        int i;
 
        for (i = 0; i < rocker->port_count; i++) {
                rocker_port = rocker->ports[i];
+               if (!rocker_port)
+                       continue;
                rocker_port_ig_tbl(rocker_port, SWITCHDEV_TRANS_NONE,
                                   ROCKER_OP_FLAG_REMOVE);
                unregister_netdev(rocker_port->dev);
@@ -4843,10 +4799,10 @@ static void rocker_remove_ports(struct rocker *rocker)
        kfree(rocker->ports);
 }
 
-static void rocker_port_dev_addr_init(struct rocker *rocker,
-                                     struct rocker_port *rocker_port)
+static void rocker_port_dev_addr_init(struct rocker_port *rocker_port)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct rocker *rocker = rocker_port->rocker;
+       const struct pci_dev *pdev = rocker->pdev;
        int err;
 
        err = rocker_cmd_get_port_settings_macaddr(rocker_port,
@@ -4859,9 +4815,10 @@ static void rocker_port_dev_addr_init(struct rocker *rocker,
 
 static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
 {
-       struct pci_dev *pdev = rocker->pdev;
+       const struct pci_dev *pdev = rocker->pdev;
        struct rocker_port *rocker_port;
        struct net_device *dev;
+       u16 untagged_vid = 0;
        int err;
 
        dev = alloc_etherdev(sizeof(struct rocker_port));
@@ -4875,7 +4832,7 @@ static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
        rocker_port->brport_flags = BR_LEARNING | BR_LEARNING_SYNC;
        INIT_LIST_HEAD(&rocker_port->trans_mem);
 
-       rocker_port_dev_addr_init(rocker, rocker_port);
+       rocker_port_dev_addr_init(rocker_port);
        dev->netdev_ops = &rocker_port_netdev_ops;
        dev->ethtool_ops = &rocker_port_ethtool_ops;
        dev->switchdev_ops = &rocker_port_switchdev_ops;
@@ -4885,8 +4842,7 @@ static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
                       NAPI_POLL_WEIGHT);
        rocker_carrier_init(rocker_port);
 
-       dev->features |= NETIF_F_NETNS_LOCAL |
-                        NETIF_F_HW_VLAN_CTAG_FILTER;
+       dev->features |= NETIF_F_NETNS_LOCAL;
 
        err = register_netdev(dev);
        if (err) {
@@ -4897,16 +4853,27 @@ static int rocker_probe_port(struct rocker *rocker, unsigned int port_number)
 
        rocker_port_set_learning(rocker_port, SWITCHDEV_TRANS_NONE);
 
-       rocker_port->internal_vlan_id =
-               rocker_port_internal_vlan_id_get(rocker_port, dev->ifindex);
        err = rocker_port_ig_tbl(rocker_port, SWITCHDEV_TRANS_NONE, 0);
        if (err) {
                dev_err(&pdev->dev, "install ig port table failed\n");
                goto err_port_ig_tbl;
        }
 
+       rocker_port->internal_vlan_id =
+               rocker_port_internal_vlan_id_get(rocker_port, dev->ifindex);
+
+       err = rocker_port_vlan_add(rocker_port, SWITCHDEV_TRANS_NONE,
+                                  untagged_vid, 0);
+       if (err) {
+               netdev_err(rocker_port->dev, "install untagged VLAN failed\n");
+               goto err_untagged_vlan;
+       }
+
        return 0;
 
+err_untagged_vlan:
+       rocker_port_ig_tbl(rocker_port, SWITCHDEV_TRANS_NONE,
+                          ROCKER_OP_FLAG_REMOVE);
 err_port_ig_tbl:
        unregister_netdev(dev);
 err_register_netdev:
@@ -4921,7 +4888,7 @@ static int rocker_probe_ports(struct rocker *rocker)
        int err;
 
        alloc_size = sizeof(struct rocker_port *) * rocker->port_count;
-       rocker->ports = kmalloc(alloc_size, GFP_KERNEL);
+       rocker->ports = kzalloc(alloc_size, GFP_KERNEL);
        if (!rocker->ports)
                return -ENOMEM;
        for (i = 0; i < rocker->port_count; i++) {
@@ -4970,7 +4937,7 @@ err_enable_msix:
        return err;
 }
 
-static void rocker_msix_fini(struct rocker *rocker)
+static void rocker_msix_fini(const struct rocker *rocker)
 {
        pci_disable_msix(rocker->pdev);
        kfree(rocker->msix_entries);
@@ -5136,7 +5103,7 @@ static struct pci_driver rocker_pci_driver = {
  * Net device notifier event handler
  ************************************/
 
-static bool rocker_port_dev_check(struct net_device *dev)
+static bool rocker_port_dev_check(const struct net_device *dev)
 {
        return dev->netdev_ops == &rocker_port_netdev_ops;
 }
@@ -5144,46 +5111,55 @@ static bool rocker_port_dev_check(struct net_device *dev)
 static int rocker_port_bridge_join(struct rocker_port *rocker_port,
                                   struct net_device *bridge)
 {
+       u16 untagged_vid = 0;
        int err;
 
-       rocker_port_internal_vlan_id_put(rocker_port,
-                                        rocker_port->dev->ifindex);
-
-       rocker_port->bridge_dev = bridge;
+       /* Port is joining bridge, so the internal VLAN for the
+        * port is going to change to the bridge internal VLAN.
+        * Let's remove untagged VLAN (vid=0) from port and
+        * re-add once internal VLAN has changed.
+        */
 
-       /* Use bridge internal VLAN ID for untagged pkts */
-       err = rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE,
-                              ROCKER_OP_FLAG_REMOVE, 0);
+       err = rocker_port_vlan_del(rocker_port, untagged_vid, 0);
        if (err)
                return err;
+
+       rocker_port_internal_vlan_id_put(rocker_port,
+                                        rocker_port->dev->ifindex);
        rocker_port->internal_vlan_id =
                rocker_port_internal_vlan_id_get(rocker_port, bridge->ifindex);
-       return rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE, 0, 0);
+
+       rocker_port->bridge_dev = bridge;
+
+       return rocker_port_vlan_add(rocker_port, SWITCHDEV_TRANS_NONE,
+                                   untagged_vid, 0);
 }
 
 static int rocker_port_bridge_leave(struct rocker_port *rocker_port)
 {
+       u16 untagged_vid = 0;
        int err;
 
-       rocker_port_internal_vlan_id_put(rocker_port,
-                                        rocker_port->bridge_dev->ifindex);
-
-       rocker_port->bridge_dev = NULL;
-
-       /* Use port internal VLAN ID for untagged pkts */
-       err = rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE,
-                              ROCKER_OP_FLAG_REMOVE, 0);
+       err = rocker_port_vlan_del(rocker_port, untagged_vid, 0);
        if (err)
                return err;
+
+       rocker_port_internal_vlan_id_put(rocker_port,
+                                        rocker_port->bridge_dev->ifindex);
        rocker_port->internal_vlan_id =
                rocker_port_internal_vlan_id_get(rocker_port,
                                                 rocker_port->dev->ifindex);
-       err = rocker_port_vlan(rocker_port, SWITCHDEV_TRANS_NONE, 0, 0);
+
+       rocker_port->bridge_dev = NULL;
+
+       err = rocker_port_vlan_add(rocker_port, SWITCHDEV_TRANS_NONE,
+                                  untagged_vid, 0);
        if (err)
                return err;
 
        if (rocker_port->dev->flags & IFF_UP)
-               err = rocker_port_fwd_enable(rocker_port, SWITCHDEV_TRANS_NONE);
+               err = rocker_port_fwd_enable(rocker_port,
+                                            SWITCHDEV_TRANS_NONE, 0);
 
        return err;
 }
@@ -5242,7 +5218,8 @@ static struct notifier_block rocker_netdevice_nb __read_mostly = {
 static int rocker_neigh_update(struct net_device *dev, struct neighbour *n)
 {
        struct rocker_port *rocker_port = netdev_priv(dev);
-       int flags = (n->nud_state & NUD_VALID) ? 0 : ROCKER_OP_FLAG_REMOVE;
+       int flags = (n->nud_state & NUD_VALID ? 0 : ROCKER_OP_FLAG_REMOVE) |
+                   ROCKER_OP_FLAG_NOWAIT;
        __be32 ip_addr = *(__be32 *)n->primary_key;
 
        return rocker_port_ipv4_neigh(rocker_port, SWITCHDEV_TRANS_NONE,