lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 23 Mar 2022 13:43:53 +0100
From:   Alexander Lobakin <>
To:     "David S. Miller" <>,
        Jakub Kicinski <>,
        Paolo Abeni <>
Cc:     Maciej Fijalkowski <>,
        Michal Kubiak <>,
        Jesse Brandeburg <>,
        Tony Nguyen <>,
        Shiraz Saleem <>,
        Dave Ertman <>,,,,
        Alexander Lobakin <>
Subject: [PATCH net 2/2] ice: don't allow to run ice_send_event_to_aux() in atomic ctx

ice_send_event_to_aux() eventually descends to mutex_lock()
(-> might_sched()), so it must not be called under non-task
context. However, at least two fixes have happened already for the
bug splats occurred due to this function being called from atomic
To make the emergency landings softer, bail out early when executed
in non-task context emitting a warn splat only once. This way we
trade some events being potentially lost for system stability and
avoid any related hangs and crashes.

Fixes: 348048e724a0e ("ice: Implement iidc operations")
Signed-off-by: Alexander Lobakin <>
Tested-by: Michal Kubiak <>
Reviewed-by: Maciej Fijalkowski <>
Acked-by: Tony Nguyen <>
 drivers/net/ethernet/intel/ice/ice_idc.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/ethernet/intel/ice/ice_idc.c b/drivers/net/ethernet/intel/ice/ice_idc.c
index fc3580167e7b..5559230eff8b 100644
--- a/drivers/net/ethernet/intel/ice/ice_idc.c
+++ b/drivers/net/ethernet/intel/ice/ice_idc.c
@@ -34,6 +34,9 @@ void ice_send_event_to_aux(struct ice_pf *pf, struct iidc_event *event)
 	struct iidc_auxiliary_drv *iadrv;
+	if (WARN_ON_ONCE(!in_task()))
+		return;
 	if (!pf->adev)

Powered by blists - more mailing lists