2004-12-30 23:43:55 +01:00
|
|
|
/*
|
2009-02-13 14:02:39 +01:00
|
|
|
* $Id: filter.c,v 1.14 2009/02/13 13:02:39 schmirl Exp $
|
2004-12-30 23:43:55 +01:00
|
|
|
*/
|
|
|
|
|
|
|
|
#include "client/filter.h"
|
|
|
|
#include "client/socket.h"
|
|
|
|
#include "tools/select.h"
|
|
|
|
#include "common.h"
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
#include <sys/ioctl.h>
|
|
|
|
#include <string.h>
|
2004-12-30 23:43:55 +01:00
|
|
|
|
2005-11-06 17:43:58 +01:00
|
|
|
#include <vdr/device.h>
|
2004-12-30 23:43:55 +01:00
|
|
|
|
2008-10-13 13:30:05 +02:00
|
|
|
#define PID_MASK_HI 0x1F
|
2007-04-23 13:23:15 +02:00
|
|
|
// --- cStreamdevFilter ------------------------------------------------------
|
|
|
|
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
static int FilterSockBufSize_warn = 0;
|
|
|
|
|
2007-04-23 13:23:15 +02:00
|
|
|
class cStreamdevFilter: public cListObject {
|
|
|
|
private:
|
|
|
|
uchar m_Buffer[4096];
|
|
|
|
int m_Used;
|
|
|
|
int m_Pipe[2];
|
|
|
|
u_short m_Pid;
|
|
|
|
u_char m_Tid;
|
|
|
|
u_char m_Mask;
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
#ifdef TIOCOUTQ
|
|
|
|
unsigned long m_maxq;
|
|
|
|
unsigned long m_flushed;
|
|
|
|
#endif
|
2007-04-23 13:23:15 +02:00
|
|
|
|
|
|
|
public:
|
|
|
|
cStreamdevFilter(u_short Pid, u_char Tid, u_char Mask);
|
|
|
|
virtual ~cStreamdevFilter();
|
|
|
|
|
|
|
|
bool Matches(u_short Pid, u_char Tid);
|
2007-04-24 13:23:16 +02:00
|
|
|
bool PutSection(const uchar *Data, int Length, bool Pusi);
|
2007-04-23 13:23:15 +02:00
|
|
|
int ReadPipe(void) const { return m_Pipe[0]; }
|
|
|
|
|
|
|
|
void Reset(void);
|
|
|
|
|
|
|
|
u_short Pid(void) const { return m_Pid; }
|
|
|
|
u_char Tid(void) const { return m_Tid; }
|
|
|
|
u_char Mask(void) const { return m_Mask; }
|
|
|
|
};
|
|
|
|
|
|
|
|
inline bool cStreamdevFilter::Matches(u_short Pid, u_char Tid) {
|
|
|
|
return m_Pid == Pid && m_Tid == (Tid & m_Mask);
|
|
|
|
}
|
|
|
|
|
2004-12-30 23:43:55 +01:00
|
|
|
cStreamdevFilter::cStreamdevFilter(u_short Pid, u_char Tid, u_char Mask) {
|
|
|
|
m_Used = 0;
|
|
|
|
m_Pid = Pid;
|
|
|
|
m_Tid = Tid;
|
|
|
|
m_Mask = Mask;
|
2007-04-23 13:25:59 +02:00
|
|
|
m_Pipe[0] = m_Pipe[1] = -1;
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
#ifdef TIOCOUTQ
|
|
|
|
m_flushed = 0;
|
|
|
|
m_maxq = 0;
|
|
|
|
#endif
|
2004-12-30 23:43:55 +01:00
|
|
|
|
2007-04-23 13:25:59 +02:00
|
|
|
#ifdef SOCK_SEQPACKET
|
|
|
|
// SOCK_SEQPACKET (since kernel 2.6.4)
|
|
|
|
if (socketpair(AF_UNIX, SOCK_SEQPACKET, 0, m_Pipe) != 0) {
|
2007-04-23 14:01:33 +02:00
|
|
|
esyslog("streamdev-client: socketpair(SOCK_SEQPACKET) failed: %m, trying SOCK_DGRAM");
|
2007-04-23 13:25:59 +02:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
if (m_Pipe[0] < 0 && socketpair(AF_UNIX, SOCK_DGRAM, 0, m_Pipe) != 0) {
|
2007-04-23 14:01:33 +02:00
|
|
|
esyslog("streamdev-client: couldn't open section filter socket: %m");
|
2007-04-23 13:25:59 +02:00
|
|
|
}
|
|
|
|
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
// Set buffer for socketpair. During certain situations, such as startup, channel/transponder
|
|
|
|
// change, VDR may lag in reading data. Instead of discarding it, we can buffer it.
|
|
|
|
// Buffer size required may be up to 4MByte.
|
|
|
|
|
|
|
|
if(StreamdevClientSetup.FilterSockBufSize) {
|
|
|
|
int sbs = StreamdevClientSetup.FilterSockBufSize;
|
|
|
|
int sbs2;
|
|
|
|
unsigned int sbss = sizeof(sbs);
|
|
|
|
int r;
|
|
|
|
|
|
|
|
r = setsockopt(m_Pipe[1], SOL_SOCKET, SO_SNDBUF, (char *)&sbs, sbss);
|
|
|
|
|
|
|
|
if(r < 0) {
|
|
|
|
isyslog("streamdev-client: setsockopt(SO_SNDBUF, %d) = %s", sbs, strerror(errno));
|
|
|
|
}
|
|
|
|
sbs2 = 0;
|
|
|
|
r = getsockopt(m_Pipe[1], SOL_SOCKET, SO_SNDBUF, (char *)&sbs2, &sbss);
|
|
|
|
if(r < 0 || !sbss || !sbs2) {
|
|
|
|
isyslog("streamdev-client: getsockopt(SO_SNDBUF, &%d, &%d) = %s", sbs2, sbss, strerror(errno));
|
|
|
|
} else {
|
|
|
|
// Linux actually returns double the requested size
|
|
|
|
// if everything works fine. And it actually buffers up to that double amount
|
|
|
|
// as can be seen from observing TIOCOUTQ (kernel 3.7/2014).
|
|
|
|
|
|
|
|
if(sbs2 > sbs)
|
|
|
|
sbs2 /= 2;
|
|
|
|
if(sbs2 < sbs) {
|
|
|
|
if(FilterSockBufSize_warn != sbs2) {
|
|
|
|
isyslog("streamdev-client: ******************************************************");
|
|
|
|
isyslog("streamdev-client: getsockopt(SO_SNDBUF) = %d < %d (configured).", sbs2, sbs);
|
|
|
|
isyslog("streamdev-client: Consider increasing system buffer size:");
|
|
|
|
isyslog("streamdev-client: 'sysctl net.core.wmem_max=%d'", sbs);
|
|
|
|
isyslog("streamdev-client: ******************************************************");
|
|
|
|
FilterSockBufSize_warn = sbs2;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if(fcntl(m_Pipe[0], F_SETFL, O_NONBLOCK) != 0 ||
|
2007-04-23 13:25:59 +02:00
|
|
|
fcntl(m_Pipe[1], F_SETFL, O_NONBLOCK) != 0) {
|
2007-04-23 14:01:33 +02:00
|
|
|
esyslog("streamdev-client: couldn't set section filter socket to non-blocking mode: %m");
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
cStreamdevFilter::~cStreamdevFilter() {
|
|
|
|
Dprintf("~cStreamdevFilter %p\n", this);
|
2007-04-23 13:33:26 +02:00
|
|
|
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
if (m_Pipe[0] >= 0) {
|
|
|
|
close(m_Pipe[0]);
|
|
|
|
}
|
|
|
|
if (m_Pipe[1] >= 0) {
|
2004-12-30 23:43:55 +01:00
|
|
|
close(m_Pipe[1]);
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
}
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
2007-04-24 13:23:16 +02:00
|
|
|
bool cStreamdevFilter::PutSection(const uchar *Data, int Length, bool Pusi) {
|
|
|
|
|
|
|
|
if (!m_Used && !Pusi) /* wait for payload unit start indicator */
|
|
|
|
return true;
|
|
|
|
if (m_Used && Pusi) /* reset at payload unit start */
|
|
|
|
Reset();
|
|
|
|
|
2004-12-30 23:43:55 +01:00
|
|
|
if (m_Used + Length >= (int)sizeof(m_Buffer)) {
|
|
|
|
esyslog("ERROR: Streamdev: Section handler buffer overflow (%d bytes lost)",
|
|
|
|
Length);
|
2007-04-23 14:00:27 +02:00
|
|
|
Reset();
|
2004-12-30 23:43:55 +01:00
|
|
|
return true;
|
|
|
|
}
|
2007-04-23 14:00:27 +02:00
|
|
|
|
2004-12-30 23:43:55 +01:00
|
|
|
memcpy(m_Buffer + m_Used, Data, Length);
|
|
|
|
m_Used += Length;
|
|
|
|
if (m_Used > 3) {
|
|
|
|
int length = (((m_Buffer[1] & 0x0F) << 8) | m_Buffer[2]) + 3;
|
|
|
|
if (m_Used == length) {
|
|
|
|
m_Used = 0;
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
#ifdef TIOCOUTQ
|
|
|
|
// If we can determine the queue size of the socket,
|
|
|
|
// we flush rather then let the socket drop random packets.
|
|
|
|
// This ensures that we have more contiguous set of packets
|
|
|
|
// on the receiver side.
|
|
|
|
if(m_flushed) {
|
|
|
|
unsigned long queue = 0;
|
|
|
|
ioctl(m_Pipe[1], TIOCOUTQ, &queue);
|
|
|
|
if(queue > m_maxq)
|
|
|
|
m_maxq = queue;
|
|
|
|
if(queue * 2 < m_maxq) {
|
|
|
|
dsyslog("cStreamdevFilter::PutSection(Pid:%d Tid: %d): "
|
|
|
|
"Flushed %ld bytes, max queue: %ld",
|
|
|
|
m_Pid, m_Tid, m_flushed, m_maxq);
|
|
|
|
m_flushed = m_maxq = 0;
|
|
|
|
|
|
|
|
} else {
|
|
|
|
m_flushed += length;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if(!m_flushed)
|
|
|
|
#endif
|
|
|
|
if(write(m_Pipe[1], m_Buffer, length) < 0) {
|
|
|
|
if(errno != EAGAIN && errno != EWOULDBLOCK) {
|
|
|
|
dsyslog("cStreamdevFilter::PutSection(Pid:%d Tid: %d): error: %s",
|
|
|
|
m_Pid, m_Tid, strerror(errno));
|
2007-04-23 14:00:27 +02:00
|
|
|
return false;
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
} else {
|
|
|
|
#ifdef TIOCOUTQ
|
|
|
|
m_flushed += length;
|
|
|
|
#else
|
|
|
|
dsyslog("cStreamdevFilter::PutSection(Pid:%d Tid: %d): "
|
|
|
|
"Dropping packet %ld bytes (queue overflow)",
|
|
|
|
m_Pid, m_Tid, length);
|
|
|
|
#endif
|
|
|
|
}
|
2007-04-23 14:00:27 +02:00
|
|
|
}
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2007-04-23 14:00:27 +02:00
|
|
|
|
|
|
|
if (m_Used > length) {
|
|
|
|
dsyslog("cStreamdevFilter::PutSection: m_Used > length ! Pid %2d, Tid%2d "
|
|
|
|
"(len %3d, got %d/%d)", m_Pid, m_Tid, Length, m_Used, length);
|
|
|
|
if(Length < TS_SIZE-5) {
|
|
|
|
// TS packet not full -> this must be last TS packet of section data -> safe to reset now
|
|
|
|
Reset();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2007-04-23 13:23:15 +02:00
|
|
|
void cStreamdevFilter::Reset(void) {
|
|
|
|
if(m_Used)
|
|
|
|
dsyslog("cStreamdevFilter::Reset skipping %d bytes", m_Used);
|
|
|
|
m_Used = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
// --- cStreamdevFilters -----------------------------------------------------
|
|
|
|
|
2013-01-29 00:02:17 +01:00
|
|
|
cStreamdevFilters::cStreamdevFilters(cClientSocket *ClientSocket):
|
2004-12-30 23:43:55 +01:00
|
|
|
cThread("streamdev-client: sections assembler") {
|
2013-01-29 00:02:17 +01:00
|
|
|
m_ClientSocket = ClientSocket;
|
2007-04-24 13:23:16 +02:00
|
|
|
m_TSBuffer = NULL;
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
cStreamdevFilters::~cStreamdevFilters() {
|
2007-04-24 13:23:16 +02:00
|
|
|
SetConnection(-1);
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
int cStreamdevFilters::OpenFilter(u_short Pid, u_char Tid, u_char Mask) {
|
|
|
|
cStreamdevFilter *f = new cStreamdevFilter(Pid, Tid, Mask);
|
2007-04-23 14:52:28 +02:00
|
|
|
int fh = f->ReadPipe();
|
|
|
|
|
|
|
|
Lock();
|
2004-12-30 23:43:55 +01:00
|
|
|
Add(f);
|
2007-04-23 14:52:28 +02:00
|
|
|
Unlock();
|
|
|
|
|
|
|
|
return fh;
|
|
|
|
}
|
|
|
|
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
void cStreamdevFilters::CloseFilter(int Handle) {
|
2007-04-23 14:52:28 +02:00
|
|
|
LOCK_THREAD;
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
|
|
|
|
for (cStreamdevFilter *fi = First(); fi; fi = Next(fi)) {
|
|
|
|
if(fi->ReadPipe() == Handle) {
|
|
|
|
// isyslog("cStreamdevFilters::CloseFilter(%d): Pid %4d, Tid %3d, Mask %2x (%d filters left)\n",
|
|
|
|
// Handle, (int)fi->Pid(), (int)fi->Tid(), fi->Mask(), Count()-1);
|
|
|
|
Del(fi);
|
|
|
|
return;
|
2007-04-23 14:52:28 +02:00
|
|
|
}
|
|
|
|
}
|
Fixed problems related to VTP filter streaming like ringbuffer overflows,
stuttering or aborting video stream (refs #2045)
Toerless Eckert wrote:
This patch tries to resolve problems in streamdev-client that
can occur when enabling "StreamFilters". Enabling this option
is necessary to receive certain programs with dynamic PIDs such as
some german "regional" broadcast (eg: NDR).
Problem:
Without this fix, the following behavior was observed on a Raspberry
PI running streamdev-0.6.1-git with VDR-2.6.1:
- Buffer overflows of filter data
- Stop/go video on channels
- Total stopping of video
More logs in:
http://www.vdr-portal.de/board16-video-disk-recorder/board55-vdr-plugins/125237-
streamdev-client-filter-daten-streamen-ndr-raspberry-haengt/
Analysis:
VDR expect section data from filters separately from the
main program stream. Historically, it received each filter data
via a separate file descriptor from the DVB card. In the streamdev-client
module, a socketpair is used to feed filter data to the main VDR code.
During certain operations in VDR, such as startup or channel change
(depending also on the speed of initialization of the video output driver),
VDR does not consume the filter data as fast as it is provided by
streamdev-client, resulting in overflow of the default socket buffers
used by streamdev-client.
To add to the problem of overflowing the socketpair buffers, the
streamdev-client code sends several times a second short packets into
the socketpair to determine if the receiving side (VDR) has closed
the socketpair (IsClosed(), CarbageCollect()). This further clogs
up the socketpair() buffer.
The raspberry PI socketpair buffering behavior seems to be the same
as that of other 3.x linux systems, the socket buffer size is by
default 163840, and it can be increased via sysctl net.core.wmem_max.
During startup, it can take up to 10 seconds before VDR will consume
filter data, so the socketpair buffer can fill up with 10 seconds worth
of data.
Solution
1. IsClosed()/CarbageCollect() where removed from client/filter.c
and replaced by explicitly tracking when VDR closes a filter socket.
This alone seems to already resolve the problem of hanging or stop&go
video and seems to be sufficient to receive dynamic-PID channels reliably.
2. filter.c was enhanced to request a larger socket buffer size
if config option FilterSockBufSize is set.
3. If supported (if streamdev-client runs on linux), the socketpair
queue is "flushed" to reduce the amount of "random" packet drop messages
and to rather drop sequential messages.
2015-01-24 00:19:04 +01:00
|
|
|
esyslog("cStreamdevFilters::CloseFilter(%d): failed (%d filters left)\n", Handle, Count()-1);
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
2007-04-24 13:23:16 +02:00
|
|
|
bool cStreamdevFilters::ReActivateFilters(void)
|
|
|
|
{
|
|
|
|
LOCK_THREAD;
|
|
|
|
|
|
|
|
bool res = true;
|
|
|
|
for (cStreamdevFilter *fi = First(); fi; fi = Next(fi)) {
|
2013-01-29 00:02:17 +01:00
|
|
|
res = m_ClientSocket->SetFilter(fi->Pid(), fi->Tid(), fi->Mask(), true) && res;
|
2007-04-24 13:23:16 +02:00
|
|
|
Dprintf("ReActivateFilters(%d, %d, %d) -> %s", fi->Pid(), fi->Tid(), fi->Mask(), res ? "Ok" :"FAIL");
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2007-04-24 13:23:16 +02:00
|
|
|
return res;
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
2007-04-24 13:23:16 +02:00
|
|
|
void cStreamdevFilters::SetConnection(int Handle) {
|
|
|
|
|
|
|
|
Cancel(2);
|
|
|
|
DELETENULL(m_TSBuffer);
|
|
|
|
|
|
|
|
if (Handle >= 0) {
|
|
|
|
m_TSBuffer = new cTSBuffer(Handle, MEGABYTE(1), 1);
|
|
|
|
ReActivateFilters();
|
|
|
|
Start();
|
|
|
|
}
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
void cStreamdevFilters::Action(void) {
|
2007-04-24 13:23:16 +02:00
|
|
|
int fails = 0;
|
2004-12-30 23:43:55 +01:00
|
|
|
|
2007-04-24 13:23:16 +02:00
|
|
|
while (Running()) {
|
|
|
|
const uchar *block = m_TSBuffer->Get();
|
|
|
|
if (block) {
|
2004-12-30 23:43:55 +01:00
|
|
|
u_short pid = (((u_short)block[1] & PID_MASK_HI) << 8) | block[2];
|
|
|
|
u_char tid = block[3];
|
2007-04-24 13:23:16 +02:00
|
|
|
bool Pusi = block[1] & 0x40;
|
2009-02-13 14:02:39 +01:00
|
|
|
// proprietary extension
|
2007-04-24 13:23:16 +02:00
|
|
|
int len = block[4];
|
|
|
|
#if 0
|
|
|
|
if (block[1] == 0xff &&
|
|
|
|
block[2] == 0xff &&
|
|
|
|
block[3] == 0xff &&
|
|
|
|
block[4] == 0x7f)
|
|
|
|
isyslog("*********** TRANSPONDER -> %s **********", block+5);
|
|
|
|
#endif
|
|
|
|
LOCK_THREAD;
|
|
|
|
cStreamdevFilter *f = First();
|
|
|
|
while (f) {
|
|
|
|
cStreamdevFilter *next = Next(f);
|
|
|
|
if (f->Matches(pid, tid)) {
|
|
|
|
|
|
|
|
if (f->PutSection(block + 5, len, Pusi))
|
|
|
|
break;
|
|
|
|
|
|
|
|
if (errno != ECONNREFUSED &&
|
|
|
|
errno != ECONNRESET &&
|
|
|
|
errno != EPIPE) {
|
2004-12-30 23:43:55 +01:00
|
|
|
Dprintf("FATAL ERROR: %m\n");
|
2007-04-24 13:23:16 +02:00
|
|
|
esyslog("streamdev-client: couldn't send section packet: %m");
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2013-01-29 00:02:17 +01:00
|
|
|
m_ClientSocket->SetFilter(f->Pid(), f->Tid(), f->Mask(), false);
|
2004-12-30 23:43:55 +01:00
|
|
|
Del(f);
|
2007-04-24 13:23:16 +02:00
|
|
|
// Filter was closed.
|
|
|
|
// - need to check remaining filters for another match
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2007-04-24 13:23:16 +02:00
|
|
|
f = next;
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2007-04-24 13:23:16 +02:00
|
|
|
} else {
|
|
|
|
#if 1 // TODO: this should be fixed in vdr cTSBuffer
|
|
|
|
// Check disconnection
|
2013-01-29 00:02:17 +01:00
|
|
|
int fd = *m_ClientSocket->DataSocket(siLiveFilter);
|
2007-04-24 13:23:16 +02:00
|
|
|
if(fd < 0)
|
|
|
|
break;
|
|
|
|
cPoller Poller(fd);
|
|
|
|
if (Poller.Poll()) {
|
|
|
|
char tmp[1];
|
|
|
|
errno = 0;
|
|
|
|
Dprintf("cStreamdevFilters::Action(): checking connection");
|
|
|
|
if (recv(fd, tmp, 1, MSG_PEEK) == 0 && errno != EAGAIN) {
|
|
|
|
++fails;
|
|
|
|
if (fails >= 10) {
|
|
|
|
esyslog("cStreamdevFilters::Action(): stream disconnected ?");
|
2013-01-29 00:02:17 +01:00
|
|
|
m_ClientSocket->CloseDataConnection(siLiveFilter);
|
2007-04-24 13:23:16 +02:00
|
|
|
break;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
fails = 0;
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
fails = 0;
|
|
|
|
}
|
|
|
|
cCondWait::SleepMs(10);
|
|
|
|
#endif
|
|
|
|
}
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|
2007-04-24 13:23:16 +02:00
|
|
|
|
|
|
|
DELETENULL(m_TSBuffer);
|
|
|
|
dsyslog("StreamdevFilters::Action() ended");
|
2004-12-30 23:43:55 +01:00
|
|
|
}
|