Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 2 years ago

Closed 21 months ago

Last modified 21 months 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 21 months 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 22 months ago by beirdo

  • Owner set to gigem
  • Status changed from new to assigned

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 22 months 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 22 months 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 22 months ago by beirdo

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

comment:5 Changed 22 months ago by danielk

  • Milestone changed from unknown to 0.26
  • Owner changed from beirdo to danielk
  • Status changed from assigned to accepted
  • Version changed from Unspecified to Master Head

Changed 21 months ago by danielk

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

comment:6 Changed 21 months ago by danielk

  • Description modified (diff)

comment:7 Changed 21 months ago by David Engel <dengel@…>

  • Resolution set to fixed
  • Status changed from accepted to closed

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 21 months 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 21 months ago by kenni

  • Milestone changed from 0.26 to 0.25.2
  • Owner changed from danielk to gigem

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.