Re: [vdr 2.3.9] Setting a mark is sluggish

Klaus Schmidinger <[email protected]>
Newsgroups gmane.linux.vdr
Message-ID <[email protected]>
On 18.03.2018 20:39, Oliver Endriss wrote:
> Am Sonntag, den 18.03.2018, 19:15 +0100 schrieb Klaus Schmidinger:
>> On 18.03.2018 18:55, Oliver Endriss wrote:
>> > Hi,
>> > 
>> > just installed vdr 2.3.9 and noticed that there is a delay
>> > when I try to set a recording mark, compared with vdr 2.2.0.
>> > 
>> > Steps to reproduce:
>> > - Play a recording.
>> > - Press ok to display the progress bar.
>> > - Press 0 to set a mark.
>> > 
>> > There is a notable delay between the keypress
>> > and the mark showing up.
>> > 
>> > Can someone confirm this?
>> 
>> Tried it while replaying on a Raspberry Pi, with the video directory
>> mounted via NFS, and had no unusual delay.
>> 
>> - Which skin are you using?
> Classic skin.
> 
>> - If applicable: Does it also happen with the LCARS skin?
> Yes.
> 
>> - Are you running any plugins that utilize the cStatus::MarksModified() function?
> No. Test setup: vdr + dvbsddevice + remote.

I'm afraid I don't have a working VDR with the old FF card any more, so
I can't test on that hardware. It doesn't happen with the Raspberry Pi, though.

Does it make a difference whether the progress display is active or not
when you set the mark?

>> - If none of the above: can you determine which version between 2.2.0 and
>>    2.3.9 introduced this?
> 
> Ok. I went back in time and installed the older versions.
> Problem appeared with vdr-2.3.2, vdr-2.3.1 tested ok.

The only change that was introduced in that area between these two versions
is in cReplayControl::ShowProgress():

   if (Initial || time(NULL) - lastProgressUpdate >= 1) {

Please try commenting out that line and the corresponding closing '}'.
While I don't see why this would only be a problem on dvbsddevice and not
on rpihddevice, I strongly suspect it to be the culprit.

If you like, we can continue this in private.

Klaus

_______________________________________________
vdr mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.