On 9/28/21 17:32, Maxime Coquelin wrote:


On 9/15/21 16:54, David Marchand wrote:
Caught by code review, this copy is unnecessary.

Signed-off-by: David Marchand <david.march...@redhat.com>
---
  lib/vhost/virtio_net.c | 11 +++--------
  1 file changed, 3 insertions(+), 8 deletions(-)


Applied to dpdk-next-virtio/main.

Thanks,
Maxime


As you suggested on IRC, I added back the comment that got removed:

@@ -3136,6 +3136,12 @@ rte_vhost_dequeue_burst(int vid, uint16_t queue_id,
                        count = 0;
                        goto out;
                }
+               /*
+ * Inject it to the head of "pkts" array, so that switch's mac
+                * learning table will get updated first.
+                */
+               pkts[0] = rarp_mbuf;
+               pkts++;
                count -= 1;
        }

Regards,
Maxime

Reply via email to