Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cbfe2e1d authored by Glenn Kasten's avatar Glenn Kasten
Browse files

Use units after all times

Change-Id: I48d3f29c37228b5d03189e4c9600824c9360cac9
parent e61fd281
Loading
Loading
Loading
Loading
+12 −10
Original line number Diff line number Diff line
@@ -80,14 +80,16 @@ static const int8_t kMaxTrackStartupRetries = 50;
static const int8_t kMaxTrackRetriesDirect = 2;

static const int kDumpLockRetries = 50;
static const int kDumpLockSleep = 20000;
static const int kDumpLockSleepUs = 20000;

static const nsecs_t kWarningThrottle = seconds(5);
// don't warn about blocked writes or record buffer overflows more often than this
static const nsecs_t kWarningThrottleNs = seconds(5);

// RecordThread loop sleep time upon application overrun or audio HAL read error
static const int kRecordThreadSleepUs = 5000;

static const nsecs_t kSetParametersTimeout = seconds(2);
// maximum time to wait for setParameters to complete
static const nsecs_t kSetParametersTimeoutNs = seconds(2);

// minimum sleep time for the mixer thread loop when tracks are active but in underrun
static const uint32_t kMinThreadSleepTimeUs = 5000;
@@ -320,7 +322,7 @@ static bool tryLock(Mutex& mutex)
            locked = true;
            break;
        }
        usleep(kDumpLockSleep);
        usleep(kDumpLockSleepUs);
    }
    return locked;
}
@@ -1044,7 +1046,7 @@ status_t AudioFlinger::ThreadBase::setParameters(const String8& keyValuePairs)
    mWaitWorkCV.signal();
    // wait condition with timeout in case the thread loop has exited
    // before the request could be processed
    if (mParamCond.waitRelative(mLock, kSetParametersTimeout) == NO_ERROR) {
    if (mParamCond.waitRelative(mLock, kSetParametersTimeoutNs) == NO_ERROR) {
        status = mParamStatus;
        mWaitWorkCV.signal();
    } else {
@@ -2029,7 +2031,7 @@ bool AudioFlinger::MixerThread::threadLoop()
            nsecs_t delta = now - mLastWriteTime;
            if (!mStandby && delta > maxPeriod) {
                mNumDelayedWrites++;
                if ((now - lastWarning) > kWarningThrottle) {
                if ((now - lastWarning) > kWarningThrottleNs) {
                    LOGW("write blocked for %llu msecs, %d delayed writes, thread %p",
                            ns2ms(delta), mNumDelayedWrites, this);
                    lastWarning = now;
@@ -2415,7 +2417,7 @@ bool AudioFlinger::MixerThread::checkForNewParameters_l()
        mParamCond.signal();
        // wait for condition with time out in case the thread calling ThreadBase::setParameters()
        // already timed out waiting for the status and will never signal the condition.
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeout);
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeoutNs);
    }
    return reconfig;
}
@@ -2891,7 +2893,7 @@ bool AudioFlinger::DirectOutputThread::checkForNewParameters_l()
        mParamCond.signal();
        // wait for condition with time out in case the thread calling ThreadBase::setParameters()
        // already timed out waiting for the status and will never signal the condition.
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeout);
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeoutNs);
    }
    return reconfig;
}
@@ -4411,7 +4413,7 @@ bool AudioFlinger::RecordThread::threadLoop()
            else {
                if (!mActiveTrack->setOverflow()) {
                    nsecs_t now = systemTime();
                    if ((now - lastWarning) > kWarningThrottle) {
                    if ((now - lastWarning) > kWarningThrottleNs) {
                        LOGW("RecordThread: buffer overflow");
                        lastWarning = now;
                    }
@@ -4734,7 +4736,7 @@ bool AudioFlinger::RecordThread::checkForNewParameters_l()
        mParamCond.signal();
        // wait for condition with time out in case the thread calling ThreadBase::setParameters()
        // already timed out waiting for the status and will never signal the condition.
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeout);
        mWaitWorkCV.waitRelative(mLock, kSetParametersTimeoutNs);
    }
    return reconfig;
}