diff options
author | imp <imp@FreeBSD.org> | 2003-01-19 00:34:17 +0000 |
---|---|---|
committer | imp <imp@FreeBSD.org> | 2003-01-19 00:34:17 +0000 |
commit | c4382c90c1c40b6c974e885d944fe12334f1cf29 (patch) | |
tree | b588be657a21709534de5f97afe2dee8d44f8392 | |
parent | 73809496b45d21085a6db24febd200f02c70f935 (diff) | |
download | FreeBSD-src-c4382c90c1c40b6c974e885d944fe12334f1cf29.zip FreeBSD-src-c4382c90c1c40b6c974e885d944fe12334f1cf29.tar.gz |
Fix comment about what we do when there are no listeners.
-rw-r--r-- | sys/kern/subr_bus.c | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/sys/kern/subr_bus.c b/sys/kern/subr_bus.c index d562a43..90b432b 100644 --- a/sys/kern/subr_bus.c +++ b/sys/kern/subr_bus.c @@ -381,19 +381,9 @@ devpoll(dev_t dev, int events, d_thread_t *td) * Common routine that tries to make sending messages as easy as possible. * We allocate memory for the data, copy strings into that, but do not * free it unless there's an error. The dequeue part of the driver should - * free the data. We do not send any data if there is no listeners on the - * /dev/devctl device. We assume that on startup, any program that wishes - * to do things based on devices that have attached before it starts will - * query the tree to find out its current state. This decision may - * be revisited if there are difficulties determining if one should do an - * action or not (eg, are all actions that the listening program idempotent - * or not). This may also open up races as well (say if the listener - * dies just before a device goes away, and is run again just after, no - * detach action would happen). The flip side would be that we'd need to - * limit the size of the queue because otherwise if no listener is running - * then we'd have unbounded growth. Most systems have less than 100 (maybe - * even less than 50) devices, so maybe a limit of 200 or 300 wouldn't be - * too horrible. XXX + * free the data. We don't send data when the device is disabled. We do + * send data, even when we have no listeners, because we wish to avoid + * races relating to startup and restart of listening applications. */ static void devaddq(const char *type, const char *what, device_t dev) |