On 11/10/2017 11:42 PM, Gregory Rose wrote: > On 11/10/2017 11:42 PM, Kevin Traynor wrote: >> 38259bd7eb21 added a memset for the dp_netdev_rxq of new rxq's >> to remove a valgrind warning. Since it is all set to 0, we can >> remove explicit reset of some fields. >> >> Signed-off-by: Kevin Traynor <[email protected]> >> --- >> lib/dpif-netdev.c | 7 ------- >> 1 file changed, 7 deletions(-) >> >> diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c >> index d9e2912..8830c41 100644 >> --- a/lib/dpif-netdev.c >> +++ b/lib/dpif-netdev.c >> @@ -3315,11 +3315,4 @@ 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) { > > Looks good but should it have a "fixes" tag for the patch that did the > memset? >
Thanks for reviewing Greg. I didn't think it should be a fixes, as I viewed this as an optimization/cleanup of already existing code that is possible because of the memset patch. I can try to make commit message a bit more explicit if that's not coming across. Kevin. > Thanks, > > - Greg _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
