enic: enable rq before updating rq descriptors

rq should be enabled before posting the buffers to rq desc. If not hw sees
stale value and casuses DMAR errors.

Signed-off-by: Govindarajulu Varadarajan <gvaradar@cisco.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c
index 5213bc01a6e901a2ea578a750762e172e304d135..243d5c5fd5e328c6d7804aac449ff479eed7a632 100644 (file)
--- a/drivers/net/ethernet/cisco/enic/enic_main.c
+++ b/drivers/net/ethernet/cisco/enic/enic_main.c
@@ -1939,6 +1939,8 @@ static int enic_open(struct net_device *netdev)
        }
 
        for (i = 0; i < enic->rq_count; i++) {
+               /* enable rq before updating rq desc */
+               vnic_rq_enable(&enic->rq[i]);
                vnic_rq_fill(&enic->rq[i], enic_rq_alloc_buf);
                /* Need at least one buffer on ring to get going */
                if (vnic_rq_desc_used(&enic->rq[i]) == 0) {
@@ -1950,8 +1952,6 @@ static int enic_open(struct net_device *netdev)
 
        for (i = 0; i < enic->wq_count; i++)
                vnic_wq_enable(&enic->wq[i]);
-       for (i = 0; i < enic->rq_count; i++)
-               vnic_rq_enable(&enic->rq[i]);
 
        if (!enic_is_dynamic(enic) && !enic_is_sriov_vf(enic))
                enic_dev_add_station_addr(enic);
@@ -1977,8 +1977,12 @@ static int enic_open(struct net_device *netdev)
        return 0;
 
 err_out_free_rq:
-       for (i = 0; i < enic->rq_count; i++)
+       for (i = 0; i < enic->rq_count; i++) {
+               err = vnic_rq_disable(&enic->rq[i]);
+               if (err)
+                       return err;
                vnic_rq_clean(&enic->rq[i], enic_free_rq_buf);
+       }
        enic_dev_notify_unset(enic);
 err_out_free_intr:
        enic_unset_affinity_hint(enic);