]> git.proxmox.com Git - mirror_ovs.git/commitdiff
dpif-netdev: Remove unnecessary resets on new rxqs.
authorKevin Traynor <ktraynor@redhat.com>
Sun, 12 Nov 2017 11:47:18 +0000 (11:47 +0000)
committerBen Pfaff <blp@ovn.org>
Sun, 12 Nov 2017 22:44:12 +0000 (14:44 -0800)
Commit 38259bd7eb21 (dpif-netdev: Initialize new rxqs in
port_reconfigure().) added a memset for the dp_netdev_rxq of new rxq's
to remove a valgrind warning for an index field in that struct.  With
the addition of that memset, it also means there are some existing
resets on other fields in that struct that are no longer needed and
gives the opportunity to simplify by removing them.

Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
lib/dpif-netdev.c

index 599308d257a8eeb8048e6ab7233c554a0cb60d74..db7831874feda298ce89a6f532e6e5e31c5327ec 100644 (file)
@@ -3314,13 +3314,6 @@ port_reconfigure(struct dp_netdev_port *port)
 
         port->rxqs[i].port = port;
 
-        if (new_queue) {
-            dp_netdev_rxq_set_cycles(&port->rxqs[i], RXQ_CYCLES_PROC_CURR, 0);
-            dp_netdev_rxq_set_cycles(&port->rxqs[i], RXQ_CYCLES_PROC_HIST, 0);
-            for (unsigned j = 0; j < PMD_RXQ_INTERVAL_MAX; j++) {
-                dp_netdev_rxq_set_intrvl_cycles(&port->rxqs[i], 0);
-            }
-        }
         err = netdev_rxq_open(netdev, &port->rxqs[i].rx, i);
         if (err) {
             return err;