--- libaitio/src/bpf.c 2013/06/25 16:42:41 1.1.2.12 +++ libaitio/src/bpf.c 2013/06/26 12:30:17 1.1.2.15 @@ -164,17 +164,6 @@ io_etherOpen(const char *csIface, int flags, int whdr, #endif } - if (ioctl(eth, BIOCIMMEDIATE, &n) == -1) { - LOGERR; - close(eth); - return -1; - } - if (whdr && ioctl(eth, BIOCSHDRCMPLT, &n) == -1) { - LOGERR; - close(eth); - return -1; - } - if (csIface) strlcpy(szStr, csIface, sizeof szStr); else if (io_get1stiface(szStr, sizeof szStr) == -1) { @@ -189,6 +178,18 @@ io_etherOpen(const char *csIface, int flags, int whdr, return -1; } + n = 1; + if (whdr && ioctl(eth, BIOCSHDRCMPLT, &n) == -1) { + LOGERR; + io_etherClose(eth, zcbuf); + return -1; + } + if (ioctl(eth, BIOCIMMEDIATE, &n) == -1) { + LOGERR; + io_etherClose(eth, zcbuf); + return -1; + } + return eth; } @@ -246,7 +247,7 @@ nextZCbuf(void ** __restrict zcache, struct bpf_zbuf * buf = ((caddr_t) *zcache) + sizeof(struct bpf_zbuf_header); ackZCbuf(bzh); } else - io_SetErr(EAGAIN, "kernel owns the buffer"); + io_SetErr(EAGAIN, "kernel owns the buffer A"); } else if (*zcache == zbuf->bz_bufa) { bzh = (struct bpf_zbuf_header *) zbuf->bz_bufb; if (chkZCbuf(bzh)) { @@ -256,7 +257,7 @@ nextZCbuf(void ** __restrict zcache, struct bpf_zbuf * buf = ((caddr_t) *zcache) + sizeof(struct bpf_zbuf_header); ackZCbuf(bzh); } else - io_SetErr(EAGAIN, "kernel owns the buffer"); + io_SetErr(EAGAIN, "kernel owns the buffer B"); } return rlen; @@ -276,7 +277,7 @@ ssize_t io_etherRecv(int eth, void * __restrict buf, size_t buflen, void * __restrict zcbuf) { ssize_t rlen = 0; - void **zcache = NULL; + void *zcache = NULL; if (!buf || !buflen) { io_SetErr(EINVAL, "invalid arguments"); @@ -289,7 +290,7 @@ io_etherRecv(int eth, void * __restrict buf, size_t bu LOGERR; } else { #ifdef __FreeBSD__ - rlen = nextZCbuf(zcache, (struct bpf_zbuf*) zcbuf, buf); + rlen = nextZCbuf(&zcache, (struct bpf_zbuf*) zcbuf, buf); #else io_SetErr(ENOTSUP, "bpf zero copy buffer mode is not supported"); #endif