[patch 28/35] net: iwlwifi: Remove in_interrupt() from tracing macro.

Coelho, Luciano luciano.coelho at intel.com
Mon Sep 28 02:13:14 EDT 2020


On Sun, 2020-09-27 at 21:49 +0200, Thomas Gleixner wrote:
> From: Sebastian Andrzej Siewior <bigeasy at linutronix.de>
> 
> The usage of in_interrupt) in driver code is phased out.
> 
> The iwlwifi_dbg tracepoint records in_interrupt() seperately, but that's
> superfluous because the trace header already records all kind of state and
> context information like hardirq status, softirq status, preemption count
> etc.
> 
> Aside of that the recording of in_interrupt() as boolean does not allow to
> distinguish between the possible contexts (hard interrupt, soft interrupt,
> bottom half disabled) while the trace header gives precise information.
> 
> Remove the duplicate information from the tracepoint and fixup the caller.
> 
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy at linutronix.de>
> Signed-off-by: Thomas Gleixner <tglx at linutronix.de>
> Cc: Johannes Berg <johannes.berg at intel.com>
> Cc: Emmanuel Grumbach <emmanuel.grumbach at intel.com>
> Cc: Luca Coelho <luciano.coelho at intel.com>
> Cc: Intel Linux Wireless <linuxwifi at intel.com>
> Cc: Kalle Valo <kvalo at codeaurora.org>
> Cc: "David S. Miller" <davem at davemloft.net>
> Cc: Jakub Kicinski <kuba at kernel.org>
> Cc: linux-wireless at vger.kernel.org
> Cc: netdev at vger.kernel.org

Acked-by: Luca Coelho <luca at coelho.fi>

--
Luca.


More information about the libertas-dev mailing list