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-next>] [day] [month] [year] [list]
Date:   Sat, 1 Oct 2022 16:31:31 +0200
From:   Uwe Kleine-König <>
To:     Douglas Miller <>
        Greg Kroah-Hartman <>,
        "Rafael J. Wysocki" <>,
Subject: Strangeness in ehea network driver's shutdown


while doing some cleanup I stumbled over a problem in the ehea network

In the driver's probe function (ehea_probe_adapter() via
ehea_register_memory_hooks()) a reboot notifier is registered. When this
notifier is triggered (ehea_reboot_notifier()) it unregisters the
driver. I'm unsure what is the order of the actions triggered by that.
Maybe the driver is unregistered twice if there are two bound devices?
Or the reboot notifier is called under a lock and unregistering the
driver (and so the devices) tries to unregister the notifier that is
currently locked and so results in a deadlock? Maybe Greg or Rafael can
tell about the details here?

Whatever the effect is, it's strange. It makes me wonder why it's
necessary to free all the resources of the driver on reboot?! I don't
know anything about the specifics of the affected machines, but I guess
doing just the necessary stuff on reboot would be easier to understand,
quicker to execute and doesn't have such strange side effects.

With my lack of knowledge about the machine, the best I can do is report
my findings. So don't expect a patch or testing from my side.

Best regards

Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | |

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)

Powered by blists - more mailing lists