Re: [PATCH] netxen_nic: use spin_[un]lock_bh around tx_clean_lock (2)

From: David Miller
Date: Sat May 09 2015 - 16:45:20 EST


From: Tony Camuso <tcamuso@xxxxxxxxxx>
Date: Wed, 6 May 2015 09:09:18 -0400

> This patch should have been part of the previous patch having the
> same summary. See http://marc.info/?l=linux-kernel&m=143039470103795&w=2
> Unfortunately, I didn't check to see where else this lock was used before
> submitting that patch. This should take care of it for netxen_nic, as I
> did a thorough search this time.
>
> To recap from the original patch; although testing this driver with
> DEBUG_LOCKDEP and DEBUG_SPINLOCK enabled did not produce any traces,
> it would be more prudent in the case of tx_clean_lock to use _bh
> versions of spin_[un]lock, since this lock is manipulated in both
> the process and softirq contexts.
>
> This patch was tested for functionality and regressions with netperf
> and DEBUG_LOCKDEP and DEBUG_SPINLOCK enabled.
>
> Signed-off-by: Tony Camuso <tcamuso@xxxxxxxxxx>

Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/