diff options
author | Nicholas Marriott <nicm@cvs.openbsd.org> | 2009-11-09 17:53:40 +0000 |
---|---|---|
committer | Nicholas Marriott <nicm@cvs.openbsd.org> | 2009-11-09 17:53:40 +0000 |
commit | 9473d5ca6183970c256efe065773dda18e76cac4 (patch) | |
tree | c9ec3b4b0898967b165f4e25400b8686f69727b1 /sys/dev/pci/bktr | |
parent | 8314ec33d28a7808b9db09541a81f1ac0496c799 (diff) |
Every selwakeup() should have a matching KNOTE() (even if kqueue isn't
supported it doesn't do any harm), so put the KNOTE() in selwakeup() itself and
remove it from any occurences where both are used, except one for kqueue itself
and one in sys_pipe.c (where the selwakeup is under a PIPE_SEL flag).
Based on a diff from tedu.
ok deraadt
Diffstat (limited to 'sys/dev/pci/bktr')
-rw-r--r-- | sys/dev/pci/bktr/bktr_core.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/sys/dev/pci/bktr/bktr_core.c b/sys/dev/pci/bktr/bktr_core.c index a4cde5e1ea9..11c5da0b264 100644 --- a/sys/dev/pci/bktr/bktr_core.c +++ b/sys/dev/pci/bktr/bktr_core.c @@ -1,4 +1,4 @@ -/* $OpenBSD: bktr_core.c,v 1.27 2009/10/31 06:40:17 deraadt Exp $ */ +/* $OpenBSD: bktr_core.c,v 1.28 2009/11/09 17:53:39 nicm Exp $ */ /* $FreeBSD: src/sys/dev/bktr/bktr_core.c,v 1.114 2000/10/31 13:09:56 roger Exp $ */ /* @@ -653,12 +653,11 @@ common_bktr_intr( void *arg ) /* If someone has a select() on /dev/vbi, inform them */ #ifndef __OpenBSD__ if (bktr->vbi_select.si_pid) { -#else - if (bktr->vbi_select.si_selpid) { -#endif selwakeup(&bktr->vbi_select); - KNOTE(&bktr->vbi_select.si_note, 0); } +#else + selwakeup(&bktr->vbi_select); +#endif } |