Oliver Endriss wrote:
Robert Schedel wrote:
Oliver Endriss wrote:
Robert Schedel wrote:
Enabling debug traces shows that polling for the PSR in function
'saa7146_wait_for_debi_done_sleep' constantly times out after 250x1ms
sleeps:
> saa7146: saa7146_wait_for_debi_done_sleep(): saa7146 (0):
saa7146_wait_for_debi_done_sleep timed out while waiting for transfer
completion
Increasing the 250ms did not avoid the timeout. And as I understood from
previous list mails, this timeout is normal when no CI/CAM is connected
to the DEBI. However, for me the high frequency polling does not make
sense if someone does not plan to plug in a CI/CAM.
When commenting out two lines in 'dvb_ca_en50221_thread_update_delay' to
increase the polling timer for slotstate NONE from 100ms (!) to 60s, the
CPU load went down to 0. So this is some kind of workaround for me.
Afaics the polling interval could be increased to something like 5s or
10s if (and only if) the slot is empty. Could you provide a patch?
Attached a patch for 2.6.24.4. Opinions?
Basically it should work but it has to be tested with CI/CAM, too.
Correct, unfortunately I cannot test it against a CI.
Furthermore it is not sufficient to test with budget-av because many
other drivers will be affected.
So I would prefer a patch which does not touch behaviour for other card
drivers (if possible).
To my understanding of the DVB code dvb_ca_en50221 is only referenced by
budget_av and budget_ci, at least in the vanilla kernel 2.6.25. The
patch only changes the timer for slot state EMPTY if
DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE is not set, which is for
1) budget_av
2) budget_ci if the CI firmware version is 0xa2 (because IRQs for CAM
change are not supported in this version)
And those two cases are probably affected by the load issue and should
be fixed.
Of course, we could add another DVB_CA_EN50221 flag solely for budget_av
to exclude case 2), but does this make sense? Who is available to test
against a budget_ci with FW=0xa2 whether it is affected by the load issue?
Please note for 'final' patches:
Always run 'checkpatch.pl' and fix the errors.
Sorry for that. :-(
Sorry, forgot to call the check script. Attached a revised patch
(against 2.6.25).
Regarding DEBI_E: Just found a av7110 code comment which also reflects
what my recent tests showed:
/* Note: Don't try to handle the DEBI error irq (MASK_18), in
* intel mode the timeout is asserted all the time...
*/
So really only DEBI_S would be left, see below.
Did you check whether DEBI_S and/or DEBI_E are ever asserted with your
setup? If not, an interrupt would never occur anyway...
DEBI_E was always asserted (as described in the av7110 code comment), so
it was worthless. DEBI_S was never asserted without CI (therefore the
250ms timeout), so it would probably only be received when a CI is used.
But as described in my other email with measurements, it seems that
there is no need to optimize the debi_done function further.
4. Are the high timeout periods in debi_done (50ms/250ms) in relation to
the 100ms poll timer intended? (I found the recent patch to this code in
the mailing list end of last year)
That patch was applied to reduce the load on the pci bus in busy-wait
mode. Basically it did not change anything for cam polling. (In fact I
was not aware that the CAM was polled every 100ms. Imho this should be
fixed.)
Only wondered whether the 250ms might have been smaller in former driver
versions.
Iirc it should be even worse with older drivers.
Basically the 250ms timeout is just a last resort to escape from the
loop, if the debi transfer hangs for some reason. We might try to reduce
the timeout but I don't know how far we can go. (Touching 'magic' values
might be dangeous.)
As above, according to my measurements we would not need to change the
250ms timeout.
Regards,
Robert
From: Robert Schedel <r.schedel@xxxxxxxx>
This change addresses kernel bug #10459: In kernel 2.6.25 the
budget_av driver polls for an CI slot in 100ms intervals (because no
interrupt solution for budget_av cards is feasible due to HW reasons).
If no CI/CAM is connected to the DVB card, polling times out only after 250ms.
This periodic polling leads to high CPU load.
The change increases the polling interval for empty slots from 100ms to 5s.
Intervals for remaining slot states (invalid, in progress, ready) are unchanged,
as they are either temporary conditions or no timeout should occur.
Signed-off-by: Robert Schedel <r.schedel@xxxxxxxx>
---
drivers/media/dvb/dvb-core/dvb_ca_en50221.c | 28 ++++++++++--------
1 file changed, 16 insertions(+), 12 deletions(-)
diff -rupN linux-2.6.25-orig/drivers/media/dvb/dvb-core/dvb_ca_en50221.c linux-2.6.25/drivers/media/dvb/dvb-core/dvb_ca_en50221.c
--- linux-2.6.25-orig/drivers/media/dvb/dvb-core/dvb_ca_en50221.c 2008-04-17 04:49:44.000000000 +0200
+++ linux-2.6.25/drivers/media/dvb/dvb-core/dvb_ca_en50221.c 2008-04-20 21:23:12.000000000 +0200
@@ -910,15 +910,21 @@ static void dvb_ca_en50221_thread_update
int curdelay = 100000000;
int slot;
+ /* Beware of too high polling frequency, because one polling
+ * call might take several hundred milliseconds until timeout!
+ */
for (slot = 0; slot < ca->slot_count; slot++) {
switch (ca->slot_info[slot].slot_state) {
default:
case DVB_CA_SLOTSTATE_NONE:
+ delay = HZ * 60; /* 60s */
+ if (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE))
+ delay = HZ * 5; /* 5s */
+ break;
case DVB_CA_SLOTSTATE_INVALID:
- delay = HZ * 60;
- if (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE)) {
- delay = HZ / 10;
- }
+ delay = HZ * 60; /* 60s */
+ if (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE))
+ delay = HZ / 10; /* 100ms */
break;
case DVB_CA_SLOTSTATE_UNINITIALISED:
@@ -926,19 +932,17 @@ static void dvb_ca_en50221_thread_update
case DVB_CA_SLOTSTATE_VALIDATE:
case DVB_CA_SLOTSTATE_WAITFR:
case DVB_CA_SLOTSTATE_LINKINIT:
- delay = HZ / 10;
+ delay = HZ / 10; /* 100ms */
break;
case DVB_CA_SLOTSTATE_RUNNING:
- delay = HZ * 60;
- if (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE)) {
- delay = HZ / 10;
- }
+ delay = HZ * 60; /* 60s */
+ if (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_CAMCHANGE))
+ delay = HZ / 10; /* 100ms */
if (ca->open) {
if ((!ca->slot_info[slot].da_irq_supported) ||
- (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_DA))) {
- delay = HZ / 10;
- }
+ (!(ca->flags & DVB_CA_EN50221_FLAG_IRQ_DA)))
+ delay = HZ / 10; /* 100ms */
}
break;
}
_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
