Hi Jakub!
On Wed, Nov 26, 2025 at 05:36:46PM -0800, Jakub Kicinski wrote:
Netpoll does not seem to handle DOWN events, so I'm guessing your primary use case is that the device had a HW fault and netdev was recreated after device reset?
Correct, this is the intended use case for this series. Handling cases where the device was unregistered and then brought back up.
Should we not be listening for the REGISTER event then? On boot we force UP the device if we find it, theoretically there may be a case where user space is not configured to UP the device, and then we'd never resume the target?
This is indeed a limitation on the current implementation. Based on your feedback, I'm working on a new version of this series handling REGISTER instead of UP and ensuring we force UP the device. This will make it consistent with the boot behavior you described.
Based on my tests, I can't force the device UP while handling the REGISTER event. I believe this is due to dev_open attempting to lock the device which is already held. For this reason, I'm resorting to defering this to a workqueue, similar to my approach on v1 [1] (but correctly handling target_list lock).
Let me know if this approach makes sense or if I'm missing something.
Thanks for the review!
[1] https://lore.kernel.org/all/20250909-netcons-retrigger-v1-4-3aea904926cf@gma...