On Wed, 27 May 2020 03:19:18 +0000 Luis Chamberlain wrote: > I read your patch, and granted, I will accept I was under the incorrect > assumption that this can only be used by networking devices, however it > the devlink approach achieves getting userspace the ability with > iproute2 devlink util to query a device health, on to which we can peg > firmware health. But *this* patch series is not about health status and > letting users query it, its about a *critical* situation which has come up > with firmware requiring me to reboot my system, and the lack of *any* > infrastructure in the kernel today to inform userspace about it. > > So say we use netlink to report a critical health situation, how are we > informing userspace with your patch series about requring a reboot? One of main features of netlink is pub/sub model of notifications. Whatever you imagine listening to your uevent can listen to devlink-health notifications via devlink. In fact I've shown this off in the RFC patches I sent to you, see the devlink mon health command being used.