[vdr] Re: Still picture in pre5 - FIX

[Andreas Schultz <aschultz@cs.uni-magdeburg.de>]

Hi Klaus,

On Sunday 07 April 2002 11:45, Klaus Schmidinger wrote:
> Gerald Raaf wrote:
> > Hello (Klaus),
> >
> > encrease the values for usleep() and MIN_IFRAME also didn't work. But if
> > i use your patch as follows it work ):
> >
> > --- vdr-1.0.0pre5-orginal/dvbapi.c      Fri Mar 29 12:32:47 2002
> > +++ vdr-1.0.0pre5/dvbapi.c              Sat Apr  6 21:40:19 2002
> > @@ -1194,8 +1194,10 @@
> >    CHECK(ioctl(videoDev, VIDEO_STILLPICTURE, &sp));
> >  #else
> >  #define MIN_IFRAME 400000
> > -  for (int i = MIN_IFRAME / Length + 1; i > 0; i--)
> > +  for (int i = MIN_IFRAME / Length + 1; i > 0; i--) {
> >        safe_write(videoDev, b, Length);
> > +      usleep(1);
> > +      }
> >  #endif
> >  }
> >
> > Gerald
>
> Ok, with this change it still works for me (and apparently it works for
> you _only_ with this change - strange).
>
> Can the others who have initially reported that my "workaround" doesn't
> work for them please try Gerald's modification (putting the usleep() into
> the for loop) and let me know if it works that way for them?

Something related, i have observer an increased chance of ARM crashes when OSD 
activies happen during or directly after such a still frame workaround. Any 
attempts do write a working demonstration for this problem have failed so 
far. It appears as if only certain types of I-frames trigger the crash.

Andreas

Andreas