Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 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 3 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 3 years 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 3 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 3 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 3 years ago by beirdo

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

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

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

comment:6 Changed 3 years ago by danielk

  • Description modified (diff)

comment:7 Changed 3 years 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 3 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 3 years 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 owner will remain gigem.
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.