spamming logs with shutdown messages

[brian@…]

I have noticed recently on v0.25pre-3930-g074d3e2 when the backend is logging shutdown events it spams the log with the same message hundreds of times over the span of a second or two such as:

2012-01-23 07:29:53.423755 N [1595/2013] Scheduler scheduler.cpp:2624 (HandleIdleShutdown) - I'm idle now... shutdown will occur in 300 seconds.
2012-01-23 07:29:53.425268 N [1595/2013] Scheduler scheduler.cpp:2624 (HandleIdleShutdown) - I'm idle now... shutdown will occur in 300 seconds.
<<<snip>>>

This seems unnecessary.

[beirdo]

It wouldn't surprise me if this were fixed now, but let's hand it to the person who knows the scheduler best.

[beirdo]

Could you run the backend with -v schedule please? This will turn on some more logging right where the timeouts are happening, and I'd like to see what code path it's following when in this state.

Thanks.

[George Nassas <gnassas@…>]

Here are two logs which have -v schedule and exhibit the problem. The first is >4MB and the second ~ 600K. Search for "(run) - sleeping for 0 ms".

​http://files.nassas.com/mythbackend.20120707095825.2687.log.bz2 ​http://files.nassas.com/mythbackend.20120708075339.2420.log.bz2

[beirdo]

Ahhh, very helpful. OK, that makes it easier to determine the actual issue.

[danielk]

David's patch + comment ; this should fix the problem it just needs some testing before applying.

[David Engel <dengel@…>]

In 88817964d66d0a18c43f3ff370994108ebe46c13/mythtv:

Fix 0ms sleep time calculation in scheduler.

Don't check the pending list for programs that won't record. They'll
never be there and don't need the 30 second padding.

Fixes #10279 .

[David Engel <dengel@…>]

In 157fe18bf710b4b35d0756503712f3d7e93b5024/mythtv:

Fix 0ms sleep time calculation in scheduler.

Don't check the pending list for programs that won't record. They'll
never be there and don't need the 30 second padding.

Fixes #10279 .
(cherry picked from commit 88817964d66d0a18c43f3ff370994108ebe46c13)