Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#10279 closed Bug Report - General (fixed)

spamming logs with shutdown messages

Reported by: brian@… Owned by: gigem
Priority: minor Milestone: 0.25.2
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by danielk)

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.

Attachments (1)

10279-v1.patch (742 bytes) - added by danielk 12 years ago.
David's patch + comment ; this should fix the problem it just needs some testing before applying.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 12 years ago by beirdo

Owner: set to gigem
Status: newassigned

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

comment:2 Changed 12 years ago by beirdo

Owner: changed from gigem to 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.

comment:3 Changed 12 years ago by 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

comment:4 Changed 12 years ago by beirdo

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

comment:5 Changed 12 years ago by danielk

Milestone: unknown0.26
Owner: changed from beirdo to danielk
Status: assignedaccepted
Version: UnspecifiedMaster Head

Changed 12 years ago by danielk

Attachment: 10279-v1.patch added

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

comment:6 Changed 12 years ago by danielk

Description: modified (diff)

comment:7 Changed 12 years ago by David Engel <dengel@…>

Resolution: fixed
Status: acceptedclosed

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 .

comment:8 Changed 12 years ago by 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)

comment:9 Changed 12 years ago by Kenni Lund [kenni a kelu dot dk]

Milestone: 0.260.25.2
Owner: changed from danielk to gigem
Note: See TracTickets for help on using tickets.