Instead of resetting memory provider parameters one by one in
__net_mp_{open,close}_rxq, memzero the entire structure. It'll be used
to extend the structure.

Signed-off-by: Pavel Begunkov <[email protected]>
---
 net/core/netdev_rx_queue.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/net/core/netdev_rx_queue.c b/net/core/netdev_rx_queue.c
index c7d9341b7630..a0083f176a9c 100644
--- a/net/core/netdev_rx_queue.c
+++ b/net/core/netdev_rx_queue.c
@@ -139,10 +139,9 @@ int __net_mp_open_rxq(struct net_device *dev, unsigned int 
rxq_idx,
 
        rxq->mp_params = *p;
        ret = netdev_rx_queue_restart(dev, rxq_idx);
-       if (ret) {
-               rxq->mp_params.mp_ops = NULL;
-               rxq->mp_params.mp_priv = NULL;
-       }
+       if (ret)
+               memset(&rxq->mp_params, 0, sizeof(rxq->mp_params));
+
        return ret;
 }
 
@@ -179,8 +178,7 @@ void __net_mp_close_rxq(struct net_device *dev, unsigned 
int ifq_idx,
                         rxq->mp_params.mp_priv != old_p->mp_priv))
                return;
 
-       rxq->mp_params.mp_ops = NULL;
-       rxq->mp_params.mp_priv = NULL;
+       memset(&rxq->mp_params, 0, sizeof(rxq->mp_params));
        err = netdev_rx_queue_restart(dev, ifq_idx);
        WARN_ON(err && err != -ENETDOWN);
 }
-- 
2.52.0


Reply via email to