PERFORCE change 114152 for review
Christian S.J. Peron
csjp at FreeBSD.org
Wed Feb 7 05:01:34 UTC 2007
http://perforce.freebsd.org/chv.cgi?CH=114152
Change 114152 by csjp at csjp_rnd01 on 2007/02/07 05:01:04
Annotate that in the future before we issue wakesup, we might do something like:
(1) Check to see if this descriptor is operating in immediate mode
(2) Check to see if the holder buffer is NULL
(3) Check to see if we are waking up any sleepers
We do not need to check the store buffer, as the fact that it has data in it
at this point in the code is invariant.
If the following conditions are true, then rotate the buffer. This will save
userspace from issuing an ioctl(2) potentially per wakeup.
Affected files ...
.. //depot/projects/zcopybpf/src/sys/net/bpf.c#14 edit
Differences ...
==== //depot/projects/zcopybpf/src/sys/net/bpf.c#14 (text+ko) ====
@@ -1760,6 +1760,11 @@
(*cpfn)(d, d->bd_sbuf, curlen + hdrlen, pkt, hdr.bh_caplen);
d->bd_slen = curlen + totlen;
+ /*
+ * XXXCSJP we could probably save a syscall per wakeup if we check the
+ * d->bd_immediate flag, hold buffer status and rotate the buffers
+ * before the wakeup.
+ */
if (do_wakeup)
bpf_wakeup(d);
}
More information about the p4-projects
mailing list