/*
- * $Id: dsi_stream.c,v 1.16 2009-10-22 13:40:11 franklahm Exp $
+ * $Id: dsi_stream.c,v 1.20 2009-10-26 12:35:56 franklahm Exp $
*
* Copyright (c) 1998 Adrian Sun (asun@zoology.washington.edu)
* All rights reserved. See COPYRIGHT.
#endif
#include <atalk/logger.h>
-
#include <atalk/dsi.h>
#include <netatalk/endian.h>
+#include <atalk/util.h>
#define min(a,b) ((a) < (b) ? (a) : (b))
}
}
-/* ---------------------- */
-static void dsi_buffer(DSI *dsi)
+/* ----------------------
+ afpd is sleeping too much while trying to send something.
+ May be there's no reader or the reader is also sleeping in write,
+ look if there's some data for us to read, hopefully it will wake up
+ the reader
+*/
+static int dsi_buffer(DSI *dsi)
{
fd_set readfds, writefds;
int len;
int maxfd;
+ LOG(log_maxdebug, logtype_dsi, "dsi_buffer: switching to non-blocking IO");
+
+ /* non blocking mode */
+ if (setnonblock(dsi->socket, 1) < 0) {
+ /* can't do it! exit without error it will sleep to death below */
+ LOG(log_error, logtype_dsi, "dsi_buffer: ioctl non blocking mode %s", strerror(errno));
+ return 0;
+ }
+
FD_ZERO(&readfds);
FD_ZERO(&writefds);
FD_SET( dsi->socket, &readfds);
FD_SET( dsi->socket, &readfds);
FD_SET( dsi->socket, &writefds);
if (select( maxfd, &readfds, &writefds, NULL, NULL) <= 0)
- return;
+ break;
if ( !FD_ISSET(dsi->socket, &readfds)) {
/* nothing waiting in the read queue */
- return;
+ break;
}
dsi_init_buffer(dsi);
len = dsi->end - dsi->eof;
* fall back to blocking IO
* could block and disconnect but it's better than a cpu hog
*/
- dsi_block(dsi, 0);
- return;
+ break;
}
len = read(dsi->socket, dsi->eof, len);
if (len <= 0)
- return;
+ break;
dsi->eof += len;
if ( FD_ISSET(dsi->socket, &writefds)) {
- return;
+ /* we can write again at last */
+ break;
}
}
+
+ LOG(log_maxdebug, logtype_dsi, "dsi_buffer: switching back to blocking IO");
+
+ if (setnonblock(dsi->socket, 0) < 0) {
+ /* can't do it! afpd will fail very quickly */
+ LOG(log_error, logtype_dsi, "dsi_buffer: ioctl blocking mode %s", strerror(errno));
+ return -1;
+ }
+ return 0;
}
/* ------------------------------
* write raw data. return actual bytes read. checks against EINTR
* aren't necessary if all of the signals have SA_RESTART
* specified. */
-size_t dsi_stream_write(DSI *dsi, void *data, const size_t length, int mode _U_)
+ssize_t dsi_stream_write(DSI *dsi, void *data, const size_t length, int mode)
{
size_t written;
ssize_t len;
#if 0
/* XXX there's no MSG_DONTWAIT in recv ?? so we have to play with ioctl
*/
- if (dsi->noblocking) {
- flags |= MSG_DONTWAIT;
- }
+ flags |= MSG_DONTWAIT;
#endif
dsi->in_write++;
written = 0;
+
+ LOG(log_maxdebug, logtype_dsi, "dsi_stream_write: sending %u bytes", length);
+
while (written < length) {
- if ((-1 == (len = send(dsi->socket, (u_int8_t *) data + written,
- length - written, flags)) && errno == EINTR) ||
- !len)
- continue;
+ len = send(dsi->socket, (u_int8_t *) data + written, length - written, flags);
+ if ((len == 0) || (len == -1 && errno == EINTR))
+ continue;
if (len < 0) {
- if (dsi->noblocking && errno == EAGAIN) {
- /* non blocking mode but will block
- * read data in input queue.
- *
- */
- dsi_buffer(dsi);
- }
- else {
- LOG(log_error, logtype_default, "dsi_stream_write: %s", strerror(errno));
- break;
+ if (errno == EAGAIN || errno == EWOULDBLOCK) {
+ if (mode == DSI_NOWAIT && written == 0) {
+ /* DSI_NOWAIT is used by attention give up in this case. */
+ return -1;
+ }
+ if (dsi_buffer(dsi)) {
+ /* can't go back to blocking mode, exit, the next read
+ will return with an error and afpd will die.
+ */
+ break;
+ }
+ continue;
}
+ LOG(log_error, logtype_dsi, "dsi_stream_write: %s", strerror(errno));
+ break;
}
else {
written += len;
return written;
}
+
/* ---------------------------------
*/
+#ifdef WITH_SENDFILE
+ssize_t dsi_stream_read_file(DSI *dsi, int fromfd, off_t offset, const size_t length)
+{
+ size_t written;
+ ssize_t len;
+
+ dsi->in_write++;
+ written = 0;
+
+ while (written < length) {
+ len = sys_sendfile(dsi->socket, fromfd, &offset, length - written);
+
+ if (len < 0) {
+ if (errno == EINTR)
+ continue;
+ if (errno == EINVAL || errno == ENOSYS)
+ return -1;
+
+ if (errno == EAGAIN || errno == EWOULDBLOCK) {
+ if (dsi_buffer(dsi)) {
+ /* can't go back to blocking mode, exit, the next read
+ will return with an error and afpd will die.
+ */
+ break;
+ }
+ continue;
+ }
+ LOG(log_error, logtype_dsi, "dsi_stream_write: %s", strerror(errno));
+ break;
+ }
+ else if (!len) {
+ /* afpd is going to exit */
+ errno = EIO;
+ return -1; /* I think we're at EOF here... */
+ }
+ else
+ written += len;
+ }
+
+ dsi->write_count += written;
+ dsi->in_write--;
+ return written;
+}
+#endif
+
+/*
+ * Return all bytes up to count from dsi->buffer if there are any buffered there
+ */
static size_t from_buf(DSI *dsi, u_int8_t *buf, size_t count)
{
size_t nbe = 0;
return nbe;
}
+/*
+ * Get bytes from buffer dsi->buffer or read from socket
+ *
+ * 1. Check if there are bytes in the the dsi->buffer buffer.
+ * 2. Return bytes from (1) if yes.
+ * Note: this may return fewer bytes then requested in count !!
+ * 3. If the buffer was empty, read from the socket.
+ */
static ssize_t buf_read(DSI *dsi, u_int8_t *buf, size_t count)
{
ssize_t nbe;
if (!count)
return 0;
- nbe = from_buf(dsi, buf, count);
+ nbe = from_buf(dsi, buf, count); /* 1. */
if (nbe)
- return nbe;
+ return nbe; /* 2. */
- return read(dsi->socket, buf, count);
-
+ return read(dsi->socket, buf, count); /* 3. */
}
-/* ---------------------------------------
- * read raw data. return actual bytes read. this will wait until
- * it gets length bytes
+/*
+ * Essentially a loop around buf_read() to ensure "length" bytes are read
+ * from dsi->buffer and/or the socket.
*/
size_t dsi_stream_read(DSI *dsi, void *data, const size_t length)
{
else { /* eof or error */
/* don't log EOF error if it's just after connect (OSX 10.3 probe) */
if (len || stored || dsi->read_count) {
- LOG(log_error, logtype_default, "dsi_stream_read(%d): %s", len, (len < 0)?strerror(errno):"unexpected EOF");
+ LOG(log_error, logtype_dsi, "dsi_stream_read(%d): %s", len, (len < 0)?strerror(errno):"unexpected EOF");
}
break;
}
return stored;
}
-/* ---------------------------------------
- * read raw data. return actual bytes read. this will wait until
- * it gets length bytes
+/*
+ * Get "length" bytes from buffer and/or socket. In order to avoid frequent small reads
+ * this tries to read larger chunks (8192 bytes) into a buffer.
*/
static size_t dsi_buffered_stream_read(DSI *dsi, u_int8_t *data, const size_t length)
{
size_t buflen;
dsi_init_buffer(dsi);
- len = from_buf(dsi, data, length);
+ len = from_buf(dsi, data, length); /* read from buffer dsi->buffer */
dsi->read_count += len;
- if (len == length) {
- return len;
+ if (len == length) { /* got enough bytes from there ? */
+ return len; /* yes */
}
-
+
+ /* fill the buffer with 8192 bytes or until buffer is full */
buflen = min(8192, dsi->end - dsi->eof);
if (buflen > 0) {
ssize_t ret;
if (ret > 0)
dsi->eof += ret;
}
- return dsi_stream_read(dsi, data, length -len);
+
+ /* now get the remaining data */
+ len += dsi_stream_read(dsi, data + len, length - len);
+ return len;
}
/* ---------------------------------------
static void block_sig(DSI *dsi)
{
dsi->in_write++;
- if (!dsi->sigblocked) sigprocmask(SIG_BLOCK, &dsi->sigblockset, &dsi->oldset);
}
/* ---------------------------------------
static void unblock_sig(DSI *dsi)
{
dsi->in_write--;
- if (!dsi->sigblocked) sigprocmask(SIG_SETMASK, &dsi->oldset, NULL);
}
/* ---------------------------------------
if ((size_t)len == towrite) /* wrote everything out */
break;
else if (len < 0) { /* error */
- LOG(log_error, logtype_default, "dsi_stream_send: %s", strerror(errno));
+ if (errno == EAGAIN || errno == EWOULDBLOCK) {
+ if (!dsi_buffer(dsi)) {
+ continue;
+ }
+ }
+ LOG(log_error, logtype_dsi, "dsi_stream_send: %s", strerror(errno));
unblock_sig(dsi);
return 0;
}
but we get a server disconnect without reason in the log
*/
if (!block[1]) {
- LOG(log_error, logtype_default, "dsi_stream_receive: invalid packet, fatal");
+ LOG(log_error, logtype_dsi, "dsi_stream_receive: invalid packet, fatal");
return 0;
}