Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Recording
  • Assigned To No-one
  • Operating System SW-codec
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Toni - 2006-11-11

FS#6316 - Avoid full cpu boost during radio recordings

Avoid full cpu boost during radio recordings (on H1xx and others?)

Synptom:
Entering the recording screen via radio menu leads to cpu full boost recording.
This unnecessary drains the battery during recordings.

Reason for full boost: The thread.c module only unboosts when all threads are sleeping.
Unfortunately the encoder codecs never sleep.

But this works fine, if the latency between the last trigger_cpu_boost() and
loading the codecs is big enough to do the unboost before the encoder codec has been
loaded. The rec_set_source() in recording.c gives this latency if the source is REALLY
changed.

But when starting a recording from the radio screen, the latency is smaller, because
there is no real recording source change. So the new encoder thread will be started
(and never sleep) before thread.c stops boosting.

Solution:
To solve the described problem, this patch applies a small sleep() to the encoders, so
that thread.c gets the chance to do the unboost even after the encoders have been
started.

Closed by  Michael Sevakis
2006-12-01 15:31
Reason for closing:  Fixed
Additional comments about closing:  

Original reporter reports problem fixed.

Michael Sevakis commented on 2006-12-01 01:41

Something must have fixed this because I don't observe 100% boost on any encoder entering through the radio screen even if the HD doesn't need spinup. The most boost I get is 12%-13% for wavpack_enc at 44.1 stereo on H120 (boost ratio displayed in rec screen and prerecording only). x5 exhibits no symptoms either. Perhaps there's a clue in here as to why a stray boost is left after boot when voice is enabled?

Toni commented on 2006-12-01 13:25

I checked with 20061201 build. The removing of SCHEDULER_BOOST_CONTROL seems to have fixed this issue. No problems here anymore.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing