Commit e348885b authored Jun 11, 2018 by Jon Mason Committed by Greg Kroah-Hartman Nov 28, 2019

ntb_netdev: fix sleep time mismatch



[ Upstream commit a861594b1b7ffd630f335b351c4e9f938feadb8e ]

The tx_time should be in usecs (according to the comment above the
variable), but the setting of the timer during the rearming is done in
msecs.  Change it to match the expected units.

Fixes: e74bfeed ("NTB: Add flow control to the ntb_netdev")
Suggested-by: Gerd W. Haeussler <gerd.haeussler@cesys-it.com>
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Acked-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent eb517baf

drivers/net/ntb_netdev.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -236,7 +236,7 @@ static void ntb_netdev_tx_timer(unsigned long data)
		struct ntb_netdev *dev = netdev_priv(ndev);

		if (ntb_transport_tx_free_entry(dev->qp) < tx_stop) {
		mod_timer(&dev->tx_timer, jiffies + msecs_to_jiffies(tx_time));
		mod_timer(&dev->tx_timer, jiffies + usecs_to_jiffies(tx_time));
		} else {
		/* Make sure anybody stopping the queue after this sees the new
		* value of ntb_transport_tx_free_entry()