Move chunk_queue stuff to own files chunk_queue.[ch].
[paraslash.git] / http_send.c
index 1f28f72..bbf6341 100644 (file)
@@ -18,6 +18,7 @@
 #include "net.h"
 #include "string.h"
 #include "fd.h"
+#include "chunk_queue.h"
 
 /** \cond convert sock_addr_in to ascii */
 #define CLIENT_ADDR(hc) inet_ntoa((hc)->addr.sin_addr)
@@ -49,14 +50,6 @@ static struct list_head clients;
 /** The whitelist/blacklist. */
 static struct list_head access_perm_list;
 
-struct chunk_queue{
-       /** The list of pending chunks for this client. */
-       struct list_head q;
-       /** The number of pending bytes for this client. */
-       unsigned long num_pending;
-       unsigned long max_pending;
-};
-
 /** Describes one client that connected the tcp port of the http sender. */
 struct http_client {
        /** The file descriptor of the client. */
@@ -75,22 +68,6 @@ struct http_client {
        struct chunk_queue *cq;
 };
 
-/**
- * Describes one queued chunk of the chunk queue.
- *
- * The send function of the http sender checks each client fd for writing. If a
- * client fd is not ready, it tries to queue that chunk for this client until
- * the number of queued bytes exceeds \p MAX_BACKLOG.
- */
-struct queued_chunk {
-       /** The number of the queued chunk, -1U means header. */
-       unsigned chunk_num;
-       /** The number of bytes already sent. */
-       unsigned sent;
-       /** Position of the chunk in the chunk queue. */
-       struct list_head node;
-};
-
 /**
  * Describes one entry in the blacklist/whitelist of the http sender.
  */
@@ -107,90 +84,6 @@ static int server_fd = -1, numclients;
 static struct sender *self;
 
 
-static int cq_enqueue(struct chunk_queue *cq, long unsigned chunk_num,
-       size_t sent)
-{
-       struct queued_chunk *qc;
-       char *buf;
-       size_t len;
-       int ret;
-
-       if (chunk_num != -1U) {
-               ret = vss_get_chunk(chunk_num, &buf, &len);
-               if (ret < 0)
-                       return ret;
-       } else
-               buf = vss_get_header(&len);
-       if (cq->num_pending + len > cq->max_pending)
-               return -E_QUEUE;
-       qc = para_malloc(sizeof(struct queued_chunk));
-       cq->num_pending += len;
-       qc->chunk_num = chunk_num;
-       qc->sent = sent;
-       list_add_tail(&qc->node, &cq->q);
-       PARA_DEBUG_LOG("%lu bytes queued for %p\n", cq->num_pending, &cq->q);
-       return 1;
-}
-
-static struct queued_chunk *cq_peek(struct chunk_queue *cq)
-{
-       if (list_empty(&cq->q))
-               return NULL;
-       return list_entry(cq->q.next, struct queued_chunk, node);
-}
-
-int cq_dequeue(struct chunk_queue *cq)
-{
-       struct queued_chunk *qc = cq_peek(cq);
-       assert(qc);
-       list_del(&qc->node);
-       free(qc);
-       return 1;
-}
-
-void cq_update(struct chunk_queue *cq, size_t sent)
-{
-       struct queued_chunk *qc = cq_peek(cq);
-       assert(qc);
-       qc->sent += sent;
-       cq->num_pending -= sent;
-}
-
-int cq_get(struct queued_chunk *qc, char **buf, size_t *len)
-{
-       int ret;
-
-       if (qc->chunk_num != -1U) {
-               ret = vss_get_chunk(qc->chunk_num, buf, len);
-               if (ret < 0)
-                       return ret;
-       } else
-               *buf = vss_get_header(len);
-       assert(*len > qc->sent);
-       *buf += qc->sent;
-       *len -= qc->sent;
-       return 1;
-}
-
-struct chunk_queue *cq_init(size_t max_pending)
-{
-       struct chunk_queue *cq = para_malloc(sizeof(*cq));
-       INIT_LIST_HEAD(&cq->q);
-       cq->max_pending = max_pending;
-       cq->num_pending = 0;
-       return cq;
-}
-
-void cq_destroy(struct chunk_queue *cq)
-{
-       struct queued_chunk *qc, *tmp;
-       list_for_each_entry_safe(qc, tmp, &cq->q, node) {
-               list_del(&qc->node);
-               free(qc);
-       }
-       free(cq);
-}
-
 static void http_shutdown_client(struct http_client *hc, const char *msg)
 {
        PARA_INFO_LOG("shutting down %s on fd %d (%s)\n", CLIENT_ADDR(hc),
@@ -231,7 +124,6 @@ static int http_send_err_msg(struct http_client *hc)
        return http_send_msg(hc, HTTP_ERR_MSG);
 }
 
-
 static int send_queued_chunks(struct http_client *hc)
 {
        struct queued_chunk *qc;
@@ -244,7 +136,7 @@ static int send_queued_chunks(struct http_client *hc)
                cq_get(qc, &buf, &len);
                ret = write(hc->fd, buf, len);
                if (ret < 0)
-                       return -1; /* FIXME */
+                       return -E_SEND_QUEUED_CHUNK;
                cq_update(hc->cq, ret);
                if (ret != len)
                        return 1;
@@ -383,7 +275,7 @@ static void http_post_select(fd_set *rfds, fd_set *wfds)
                goto err_out;
        }
        hc->status = HTTP_CONNECTED;
-       hc->cq = cq_init(MAX_BACKLOG);
+       hc->cq = cq_new(MAX_BACKLOG);
        PARA_INFO_LOG("accepted client #%d: %s (fd %d)\n", numclients,
                CLIENT_ADDR(hc), hc->fd);
        numclients++;