[linux-dvb] [PATCH 1/2] make gnutv more resilient to temporary slow reads.

Andrea mariofutire at googlemail.com
Fri Jul 11 00:29:02 CEST 2008


I have been using gnutv to save data to my network drive.
Event if the average speed required to save is way less than the bandwidth, gnutv sometimes does not
read fast enough to avoid overflow of the dvb ring buffer.

In that case gnutv prints an error message and stops recording.

In my opinion, it would be better to bite the bullet, loose a bit of the stream, and carry on, in
the hope that the system will not be too busy.

Here is the patch

diff -r 73b910014d07 util/gnutv/gnutv_data.c
--- a/util/gnutv/gnutv_data.c   Sat Jul 05 16:38:32 2008 +0200
+++ b/util/gnutv/gnutv_data.c   Thu Jul 10 23:19:48 2008 +0100
@@ -222,19 +238,26 @@
          while(!outputthread_shutdown) {
                  if (poll(&pollfd, 1, 1000) != 1)
                          continue;
+
+               // in case of read error, we should not abort in order to minimize the loss of data.
+
                  if (pollfd.revents & POLLERR) {
                          if (errno == EINTR)
                                  continue;
-                       fprintf(stderr, "DVR device read failure\n");
-                       return 0;
+                       fprintf(stderr, "DVR device poll failure\n");
+
+                       // This is tipically EOVERFLOW, i.e. dvb internal buffer run out of space
+
+                       // The next read will fail too, but it will also clear the error flag in the
driver
+                       // so that following reads will succeed
                  }

                  int size = read(dvrfd, buf, sizeof(buf));
                  if (size < 0) {
                          if (errno == EINTR)
                                  continue;
-                       fprintf(stderr, "DVR device read failure\n");
-                       return 0;
+                       fprintf(stderr, "DVR device read failure. Possible loss of data.\n");
+                       // The error flag has been cleared, next read should succeed.
                  }

                  written = 0;
@@ -243,6 +266,7 @@
                          if (tmp == -1) {
                                  if (errno != EINTR) {
                                          fprintf(stderr, "Write error: %m\n");
+                                       // abort loop
                                          break;
                                  }
                          } else {


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: gnutv.diff3
Url: http://www.linuxtv.org/pipermail/linux-dvb/attachments/20080710/0fb48984/attachment.txt 


More information about the linux-dvb mailing list