Fix für GetDeviceForTransponder bei Device-Bonding und IPTV

HelmutB hat hier einen Fix für ein Problem bei der Device-Wahl mit IPTV gepostet und darin festgestellt, dass der Code im 'else if' Zweig nie ausgeführt wird, womit er vollkommen recht hat.

Der Grund hierfür war ein "Fix", den ich damals hier gepostet hatte. Der war natürlich Quatsch, denn wenn MaySwitchTransponder() false ist, wird weder der 'if' noch der 'else if' Zweig ausgeführt, weil der Aufruf ja in beiden Bedingungen drin ist und sich das Ergebnis zwischen den beiden Aufrufen auch nicht ändert. Das Problem von damals trat damit nur einfach nicht mehr auf, weil die Auswahl des Devices über die Prioritäten ganz abgeklemmt wurde. Wie kamel5 hier richtig beobachtet hat, hatte sich durch den Patch das eigentlich beabsichtigte Verhalten bei VPS-Aufnahmen verändert. Es wurde, wenn alle Devices belegt waren, nicht mehr für eine höher priorisierte VPS-Aufnahme einige Zeit vor dem geplanten Beginn auf den entsprechenden Transponder geschaltet. Nachdem Device-Bonding für mich immer schon ein "ungeliebtes Feature" war und es damit aber ansonsten zu funktionieren schien, bin ich der Sache dann auch nicht weiter nachgegangen (siehe auch hier und hier).

Anbei ein Patch, der zum Einen die Änderung von HelmutB enthält, dass immer das *erste* freie Device genommen wird, und zum Anderen im Falle von Device-Bonding die Prioritäten aller gebondeten Devices berücksichtigt. Damit sollten eigentlich beide Probleme gelöst und auch die Nebenwirkung bzgl. VPS-Aufnahmen behoben sein.

Bitte testet diesen Patch ausgiebig, insbesondere kamel5 und dile.

Die Änderung an cDevice::Priority() hatte wohl zu viele Seiteneffekte, daher schlage ich folgendes vor:

- für dvbdevice.[hc] und device.h wieder die Versionen aus 2.4.2 nehmen

- in device.c den 'else if'-Zweig löschen:

--- device.c    2020/06/25 13:46:16     4.33
+++ device.c    2020/06/25 14:39:39
@@ -419,7 +419,6 @@

 cDevice *cDevice::GetDeviceForTransponder(const cChannel *Channel, int Priority)
 {
-  cDevice *Device = NULL;
   for (int i = 0; i < cDevice::NumDevices(); i++) {
       if (cDevice *d = cDevice::GetDevice(i)) {
          if (d->IsTunedToTransponder(Channel))
@@ -427,14 +426,10 @@
          if (d->ProvidesTransponder(Channel)) {
             if (d->MaySwitchTransponder(Channel))
                return d; // this device may switch to the transponder without disturbing any receiver or live view
-            else if (!d->Occupied()) { // MaySwitchTransponder() implicitly calls Occupied()
-               if (d->Priority() < Priority && (!Device || d->Priority() < Device->Priority()))
-                  Device = d; // use this one only if no other with less impact can be found
-               }
             }
          }
       }
-  return Device;
+  return NULL;
 }

 bool cDevice::HasCi(void)

Damit bleibt lediglich der Fix von HelmutB, dass das erste Device genommen wird, welches den Transponder ohne Störung von Aufnahmen oder Live-View liefern kann. Da der 'else if' Zweig schon seit 8 Jahren nicht mehr aktiv war, hat er anscheinend niemandem gefehlt. Dieser Zweig sollte dafür sorgen, dass für eine anstehende VPS-Aufnahme der Live-Kanal umgeschaltet wird bzw. eine Aufnahme mit niedrigerer Priorität unterbrochen wird, falls kein freies Device verfügbar ist. So schaltet er halt zur Startzeit der Aufnahme "hart" um.

kamel5: kannst du bitte mit den genannten Änderungen bzw. Rückbauten nochmal testen?

kamel5 es hätte mich ehrlich gesagt auch gewundert, wenn es sich damit anders verhalten hätte als früher, denn ob der 'else if' Zweig wegen der Bedingung, die immer false ist, nie durchlaufen wird, oder weil er ganz weggelassen wird, sollte keinen Unterschied machen.

Aber HelmutB hat natürlich recht, warum sollen User mit nicht gebondeten Devices auf eine eigentlich gewollte Funktionalität verzichten müssen. Ich wollte zwar das Thema Device Bonding vollständig innerhalb cDvbDevice halten und es nicht in device.c haben, aber nachdem die Lösung durch ein einfaches cDevice::IsBonded() machbar ist, hab ich mich halt dazu durchgerungen.

Anbei die hoffentlich endgültige Fassung dieses Fixes. Die Zeilennummern können etwas off sein, ansonsten sollte es aber passen.