O_DIRECT works!!!

[frescor/ffmpeg.git] / libavformat / aviobuf.c

diff --git a/libavformat/aviobuf.c b/libavformat/aviobuf.c
index 491e7c76d8689ca129261158f8a4c0bcfb90330d..601ea4fc70a949988add8c6ca1c5859be4c02e2a 100644 (file)
--- a/libavformat/aviobuf.c
+++ b/libavformat/aviobuf.c
@@ -81,13 +81,18 @@ ByteIOContext *av_alloc_put_byte(
 
 #define BLOCK_SIZE 512
 
-static void flush_buffer(ByteIOContext *s)
+static void flush_buffer_o_direct(ByteIOContext *s)
 {
     size_t data_len = s->buf_ptr - s->buffer;
-    size_t to_write = (data_len + BLOCK_SIZE - 1) & ~(BLOCK_SIZE - 1);
-    int64_t res = AVERROR(EPIPE);
-    av_log(NULL, AV_LOG_ERROR, "YYY dl=%u, tw=%u\n", data_len, to_write);
-    if (s->buf_ptr > s->buffer) {
+    size_t to_write = data_len & ~(BLOCK_SIZE - 1);
+
+    if (s->flush_all) {
+       s->flush_all = 0;
+       to_write = (data_len + BLOCK_SIZE - 1) & ~(BLOCK_SIZE - 1);
+    }
+
+    av_log(NULL, AV_LOG_ERROR, "          really writing %zx\n", to_write);
+    if (data_len > 0) {
         if (s->write_packet && !s->error){
             int ret= s->write_packet(s->opaque, s->buffer, to_write);
             if(ret < 0){
@@ -99,18 +104,41 @@ static void flush_buffer(ByteIOContext *s)
             s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_ptr - s->checksum_ptr);
             s->checksum_ptr= s->buffer;
         }
-        s->pos += to_write - BLOCK_SIZE;
-       memcpy(s->buffer, &s->buffer[to_write], data_len - to_write + BLOCK_SIZE);
-       s->buf_ptr -= to_write - BLOCK_SIZE;
-
-        if (s->seek)
-           res = s->seek(s->opaque, s->pos, SEEK_SET);
-       if (res < 0)
-           av_log(NULL, AV_LOG_ERROR, "seek error inside flush_buffer: %lld\n", res);
-       av_log(NULL, AV_LOG_ERROR, "ZZZ tw=%u, pos=%d\n", to_write, s->pos);
+        s->pos += to_write;
+       if (data_len > to_write) {
+           /* flush_all was not set */
+           memcpy(s->buffer, &s->buffer[to_write], data_len - to_write);
+           s->buf_ptr -= to_write;
+       } else
+           s->buf_ptr = s->buffer;
+
+       //av_log(NULL, AV_LOG_ERROR, "ZZZ tw=%zu, pos=%zd\n", to_write, s->pos);
     }
 }
 
+static void flush_buffer(ByteIOContext *s)
+{
+    av_log(NULL, AV_LOG_ERROR, "WRITE %p: pos=%zx len=%zx\n", s, s->pos, s->buf_ptr - s->buffer);
+    if (s->o_direct_flag) {
+       flush_buffer_o_direct(s);
+       return;
+    }
+    if (s->buf_ptr > s->buffer) {
+        if (s->write_packet && !s->error){
+            int ret= s->write_packet(s->opaque, s->buffer, s->buf_ptr - s->buffer);
+            if(ret < 0){
+                s->error = ret;
+            }
+        }
+        if(s->update_checksum){
+            s->checksum= s->update_checksum(s->checksum, s->checksum_ptr, s->buf_ptr - s->checksum_ptr);
+            s->checksum_ptr= s->buffer;
+        }
+        s->pos += s->buf_ptr - s->buffer;
+    }
+    s->buf_ptr = s->buffer;
+}
+
 void put_byte(ByteIOContext *s, int b)
 {
     *(s->buf_ptr)++ = b;
@@ -162,13 +190,16 @@ int64_t url_fseek(ByteIOContext *s, int64_t offset, int whence)
             return offset1;
         offset += offset1;
     }
-    av_log(NULL, AV_LOG_ERROR, "SEEK to %llu\n", offset);
+    //if (s->o_direct_flag)
+    av_log(NULL, AV_LOG_ERROR, "SEEK  %p: from %llx to %llx\n", s, pos, offset);
 
     offset1 = offset - pos;
     if (!s->must_flush &&
         offset1 >= 0 && offset1 <= (s->buf_end - s->buffer)) {
         /* can do the seek inside the buffer */
         s->buf_ptr = s->buffer + offset1;
+       if (s->o_direct_flag)
+           av_log(NULL, AV_LOG_ERROR, "Internal seek by %llx\n", offset1);
     } else if(s->is_streamed && !s->write_flag &&
               offset1 >= 0 && offset1 < (s->buf_end - s->buffer) + (1<<16)){
         while(s->pos < offset && !s->eof_reached)
@@ -181,17 +212,31 @@ int64_t url_fseek(ByteIOContext *s, int64_t offset, int whence)
 
 #if CONFIG_MUXERS || CONFIG_NETWORK
         if (s->write_flag) {
+           s->flush_all = 1;
             flush_buffer(s);
             s->must_flush = 1;
         }
 #endif /* CONFIG_MUXERS || CONFIG_NETWORK */
-#warning TODO read the whole block from the file to the buffer
-        if (!s->seek || (res = s->seek(s->opaque, offset, SEEK_SET)) < 0)
-            return res;
-        if (!s->write_flag)
-            s->buf_end = s->buffer;
-        s->buf_ptr = s->buffer;
-        s->pos = offset;
+       if (s->o_direct_flag) {
+           /* Read the whole block from the seek destination */
+           offset1 = offset & ~(BLOCK_SIZE - 1);
+           if (!s->seek || (res = s->seek(s->opaque, offset1, SEEK_SET)) < 0)
+               return res;
+           av_log(NULL, AV_LOG_ERROR, "READ block from %llx\n", offset1);
+           if ((res = s->read_packet(s->opaque, s->buffer, BLOCK_SIZE)) < 0)
+               return res;
+           if (!s->seek || (res = s->seek(s->opaque, offset1, SEEK_SET)) < 0)
+               return res;
+           s->buf_ptr = s->buffer + offset - offset1;
+           s->pos = offset1;
+       } else {
+           if (!s->seek || (res = s->seek(s->opaque, offset, SEEK_SET)) < 0)
+               return res;
+           if (!s->write_flag)
+               s->buf_end = s->buffer;
+           s->buf_ptr = s->buffer;
+           s->pos = offset;
+       }
     }
     s->eof_reached = 0;
     return offset;
@@ -557,8 +602,14 @@ int url_fdopen(ByteIOContext **s, URLContext *h)
     } else {
         buffer_size = IO_BUFFER_SIZE;
     }
-    buffer_size = 2*buffer_size+BLOCK_SIZE;
-    buffer = memalign(sysconf(_SC_PAGESIZE), buffer_size);
+    if (h->flags & URL_DIRECT) {
+       if ((h->flags & URL_RDWR) == 0)
+           return AVERROR(EINVAL); /* We need read beacuse of url_fseek() */
+       buffer_size = 2*buffer_size+BLOCK_SIZE;
+       buffer = memalign(sysconf(_SC_PAGESIZE), buffer_size);
+    } else {
+       buffer = av_malloc(buffer_size);
+    }
     if (!buffer)
         return AVERROR(ENOMEM);
 
@@ -577,6 +628,7 @@ int url_fdopen(ByteIOContext **s, URLContext *h)
     }
     (*s)->is_streamed = h->is_streamed;
     (*s)->max_packet_size = max_packet_size;
+    (*s)->o_direct_flag = !!(h->flags & URL_DIRECT);
     if(h->prot) {
         (*s)->read_pause = (int (*)(void *, int))h->prot->url_read_pause;
         (*s)->read_seek  = (int64_t (*)(void *, int, int64_t, int))h->prot->url_read_seek;