diff options
author | phk <phk@FreeBSD.org> | 2009-06-28 11:28:14 +0000 |
---|---|---|
committer | phk <phk@FreeBSD.org> | 2009-06-28 11:28:14 +0000 |
commit | ed08f853a1af78b712c83924957402ea1a1cf56c (patch) | |
tree | 12880e3a07e15788493652721212c558d8c339f1 | |
parent | 73ae1e3978fb839f852978f567e2cf682e501b57 (diff) | |
download | FreeBSD-src-ed08f853a1af78b712c83924957402ea1a1cf56c.zip FreeBSD-src-ed08f853a1af78b712c83924957402ea1a1cf56c.tar.gz |
There are a number of ways an application can check if there are
inbound data waiting on a filedescriptor, such as a pipe or a socket,
for instance by using select(2), poll(2), kqueue(2), ioctl(FIONREAD)
etc.
But we have no way of finding out if written data have yet to be
disposed of, for instance, transmitted (and ack'ed!) to some remote
host, or read by the applicantion at the far end of the pipe.
The closest we get, is calling shutdown(2) on a TCP socket in
non-blocking mode, but this has the undesirable sideeffect of
preventing future communication.
Add a complement to FIONREAD, called FIONWRITE, which returns the
number of bytes not yet properly disposed of. Implement it for
all sockets.
Background:
A HTTP server will want to time out connections, if no new request
arrives within a certain period after the last transmitted response
has actually been sent (and ack'ed).
For a busy HTTP server, this timeout can be subsecond duration.
In order to signal to a load-balancer that the connection is truly
dead, TCP_RST will be the preferred method, as this avoids the need
for a RTT delay for FIN handshaking, with a client which, surprisingly
often, no longer at the remote IP number.
If a slow, distant client is being served a response which is big
enough to fill the window, but small enough to fit in the socket
buffer, the write(2) call will return immediately.
If the session timeout is armed at that time, all bytes in the
response may not have been transmitted by the time it fires.
FIONWRITE allows the timeout to check that no data is outstanding
on the connection, before it TCP_RST's it.
Input & Idea from: rwatson
Approved by: re (kib)
-rw-r--r-- | sys/dev/ath/ath_hal/ar5416/ar5416_reset.c | 2 | ||||
-rw-r--r-- | sys/kern/sys_socket.c | 5 | ||||
-rw-r--r-- | sys/sys/filio.h | 1 |
3 files changed, 7 insertions, 1 deletions
diff --git a/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c b/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c index 4593588..584b558 100644 --- a/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c +++ b/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c @@ -124,7 +124,7 @@ ar5416Reset(struct ath_hal *ah, HAL_OPMODE opmode, if (!ar5416SetPowerMode(ah, HAL_PM_AWAKE, AH_TRUE)) { HALDEBUG(ah, HAL_DEBUG_ANY, "%s: chip did not wakeup\n", __func__); - FAIL(HAL_EIO); + // FAIL(HAL_EIO); } /* diff --git a/sys/kern/sys_socket.c b/sys/kern/sys_socket.c index 93da80d..ecfb4ad 100644 --- a/sys/kern/sys_socket.c +++ b/sys/kern/sys_socket.c @@ -169,6 +169,11 @@ soo_ioctl(struct file *fp, u_long cmd, void *data, struct ucred *active_cred, *(int *)data = so->so_rcv.sb_cc; break; + case FIONWRITE: + /* Unlocked read. */ + *(int *)data = so->so_snd.sb_cc; + break; + case FIOSETOWN: error = fsetown(*(int *)data, &so->so_sigio); break; diff --git a/sys/sys/filio.h b/sys/sys/filio.h index 5f13288..68e7852 100644 --- a/sys/sys/filio.h +++ b/sys/sys/filio.h @@ -55,6 +55,7 @@ struct fiodgname_arg { void *buf; }; #define FIODGNAME _IOW('f', 120, struct fiodgname_arg) /* get dev. name */ +#define FIONWRITE _IOR('f', 119, int) /* get # bytes (yet) to write */ /* Handle lseek SEEK_DATA and SEEK_HOLE for holey file knowledge. */ #define FIOSEEKDATA _IOWR('f', 97, off_t) /* SEEK_DATA */ #define FIOSEEKHOLE _IOWR('f', 98, off_t) /* SEEK_HOLE */ |