On 7/25/19 5:13 PM, Saeed Mahameed wrote:
On Mon, 2019-07-22 at 14:40 -0700, Shannon Nelson wrote:

+static void ionic_tx_timeout_work(struct work_struct *ws)
+{
+       struct lif *lif = container_of(ws, struct lif,
tx_timeout_work);
+
+       netdev_info(lif->netdev, "Tx Timeout recovery\n");
+       ionic_reset_queues(lif);
missing rtnl_lock ?

+}
+
  static void ionic_tx_timeout(struct net_device *netdev)
  {
-       netdev_info(netdev, "%s: stubbed\n", __func__);
+       struct lif *lif = netdev_priv(netdev);
+
+       schedule_work(&lif->tx_timeout_work);
  }
missing cancel work ? be careful when combined with the rtnl_lockthough ..



Yep, good catch on both.  I'll take care of those.

sln

Reply via email to