[vdr] Re: vdr-1.3.13 with NTPL

[Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de>]

Klaus Schmidinger wrote:
> 
> Alfred Zastrow wrote:
> >
> > Hi,
> >
> > since I don't have NTPL-less systems anymore, I'm forced to run vdr with
> > NTPL.
> >
> > My vdr-build-system in short:
> >
> > linux-2.6.8.1
> > glibc-2.3.4-20040701 (tarball is from Linuxfromscratch)
> > gcc-3.4.2
> > binutils-2.15
> >
> > With the folloing two changes above (commenting out the NTPL-exit in
> > vdr.c and the usleep in dvb-player.c) vdr runs flawless with NTPL.
> > Also jumping between cutting marks appears normal.
> > Right now I'm not able to meassure the cpu-load, but in my case
> > (dedicated system) this is not an issue.
> >
> > --- vdr.orig.c  Sun Oct 17 13:50:21 2004
> > +++ vdr.c       Wed Oct 20 21:40:39 2004
> > @@ -91,7 +91,7 @@
> >     if (confstr(_CS_GNU_LIBPTHREAD_VERSION, LibPthreadVersion,
> > sizeof(LibPthreadVersion)) > 0) {
> >        if (strstr(LibPthreadVersion, "NPTL")) {
> >           fprintf(stderr, "vdr: please turn off NPTL by setting 'export
> > LD_ASSUME_KERNEL=2.4.1' before starting VDR\n");
> > -        return 2;
> > +//        return 2;
> >           }
> >        }
> >   #endif
> >
> > --- dvbplayer.orig.c    Fri Oct 15 15:07:55 2004
> > +++ dvbplayer.c Thu Oct 21 08:11:28 2004
> > @@ -490,8 +490,8 @@
> >                       break;
> >                       }
> >                    }
> > -              else//XXX
> > -                 usleep(1); // this keeps the CPU load low
> > +//              else//XXX
> > +//                 usleep(1); // this keeps the CPU load low
> >                 }
> >
> >              // Store the frame in the buffer:
> >
> > regards
> > Alfred
> 
> Great!
> 
> I'll take it from here and will see what can be done (if at all necessary)
> to keep the CPU load low without that usleep().

Well, apparently the CPU load is the same with or without this usleep()
(at least here on my system). So I'll just remove it, together with the
whole NPTL check in vdr.c.

Klaus