Reply to thread

Message

<blockquote data-quote="tourettes" data-source="post: 662571" data-attributes="member: 10858">davidf, something to be analyzed. As attachements two logs, one from HTPC where the drifting happens and one from the dev PC where I haven't spotted any drifting.[code]REFERENCE_TIME CSyncClock::GetPrivateTime(){&nbsp; CAutoLock cObjectLock(this);&nbsp; UINT64 qpcNow = GetCurrentTimestamp();&nbsp; DWORD dwTime = timeGetTime();&nbsp; UINT64 hwClock(0);&nbsp; UINT64 hwQpc(0);&nbsp; UINT64 hwClockEnd(0);&nbsp; UINT64 hwQpcEnd(0);&nbsp; HRESULT hr = m_pAudioRenderer-&gt;AudioClock(hwClock, hwQpc);&nbsp; if (hr == S_OK)&nbsp; {&nbsp; &nbsp; if (m_dStartQpcHW == 0)&nbsp; &nbsp; &nbsp; m_dStartQpcHW = hwQpc;&nbsp; &nbsp; if (m_dStartTimeHW == 0)&nbsp; &nbsp; &nbsp; m_dStartTimeHW = hwClock;&nbsp; &nbsp; if (m_dStartTimeSystem == 0)&nbsp; &nbsp; &nbsp; m_dStartTimeSystem = dwTime;&nbsp; &nbsp; if (m_dStartTimeCorrected == 0)&nbsp; &nbsp; &nbsp; m_dStartTimeCorrected = m_rtPrivateTime;&nbsp; &nbsp; m_dDurationHW = (hwClock - m_dStartTimeHW) / 10000;&nbsp; &nbsp; m_dDurationSystem = (dwTime - m_dStartTimeSystem); &nbsp; &nbsp; m_dDurationCorrected = (m_rtPrivateTime - m_dStartTimeCorrected) / 10000;&nbsp; &nbsp; //if (hwQpc &lt; m_dPrevQpcHW)&nbsp; &nbsp; &nbsp; //Log(&quot;%I64d&quot;, hwQpc - m_dPrevQpcHW);&nbsp; &nbsp; if (m_dPrevTimeHW &gt; hwClock)&nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; m_dStartTimeHW = m_dPrevTimeHW = hwClock;&nbsp; &nbsp; &nbsp; m_dStartQpcHW = m_dPrevQpcHW = hwQpc;&nbsp; &nbsp; &nbsp; m_dStartTimeSystem = dwTime;&nbsp; &nbsp; &nbsp; m_dStartTimeCorrected = m_rtPrivateTime;&nbsp; &nbsp; &nbsp; m_llDeltaError = 0;&nbsp; &nbsp; &nbsp; overallCorrection = 0;&nbsp; &nbsp; }&nbsp; &nbsp; else&nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; double clockDiff = hwClock - m_dStartTimeHW;&nbsp; &nbsp; &nbsp; double qpcDiff = hwQpc - m_dStartQpcHW;&nbsp; &nbsp; &nbsp; if (clockDiff &gt; 0 &amp;&amp; qpcDiff &gt; 0)&nbsp; &nbsp; &nbsp; &nbsp; m_dSystemClockMultiplier = clockDiff / qpcDiff;&nbsp; &nbsp; &nbsp; if (m_dSystemClockMultiplier &lt; 0.95 || m_dSystemClockMultiplier &gt; 1.05)&nbsp; &nbsp; &nbsp; &nbsp; m_dSystemClockMultiplier = 1.0;&nbsp; &nbsp; &nbsp; m_dPrevTimeHW = hwClock;&nbsp; &nbsp; &nbsp; m_dPrevQpcHW = hwQpc;&nbsp; &nbsp; }&nbsp; }&nbsp; else&nbsp; {&nbsp; &nbsp; //Log(&quot;AudioClock() returned error (0x%08x)&quot;);&nbsp; }&nbsp; REFERENCE_TIME delta = REFERENCE_TIME(dwTime) - REFERENCE_TIME(m_dwPrevSystemTime);&nbsp; REFERENCE_TIME deltaOrig = delta;&nbsp; if (dwTime &lt; m_dwPrevSystemTime)&nbsp; {&nbsp; &nbsp; delta += REFERENCE_TIME(UINT_MAX) + 1;&nbsp; }&nbsp; m_dwPrevSystemTime = dwTime;&nbsp; delta = (REFERENCE_TIME)(delta * (UNITS / MILLISECONDS));&nbsp; double dAdjustment;&nbsp; if (m_bHWBasedRefClock)&nbsp; {&nbsp; &nbsp; dAdjustment = m_dAdjustment * m_dBias / m_dSystemClockMultiplier;&nbsp; }&nbsp; else&nbsp; {&nbsp; &nbsp; dAdjustment = m_dAdjustment * m_dBias;&nbsp; }&nbsp; double ddelta= ((double) delta)*dAdjustment;&nbsp; delta = (REFERENCE_TIME) ddelta;&nbsp; m_ddeltaError += ddelta - delta;&nbsp; if (m_ddeltaError&gt;1.0)&nbsp; {&nbsp; &nbsp; delta--;&nbsp; &nbsp; m_ddeltaError-=1.0;&nbsp; &nbsp; overallCorrection += 1.0;&nbsp; }&nbsp; &nbsp; if (hwQpc - m_dStartQpcHW &gt; 600000000)&nbsp; {&nbsp; &nbsp; Log(&quot;mul: %.10f de: %6I64d de.orig: %6I64d de.err: %.10f bias: %.10f adj: %.10f hwQpc: %I64d hwClock: %I64d&quot;, &nbsp; &nbsp; &nbsp; m_dSystemClockMultiplier, delta, deltaOrig * 10000, m_ddeltaError, m_dBias, m_dAdjustment, hwQpc, hwClock);&nbsp; }&nbsp; m_rtPrivateTime = m_rtPrivateTime + delta;&nbsp; return m_rtPrivateTime;}[/code]</blockquote>

[QUOTE="tourettes, post: 662571, member: 10858"] davidf, something to be analyzed. As attachements two logs, one from HTPC where the drifting happens and one from the dev PC where I haven't spotted any drifting. [code] REFERENCE_TIME CSyncClock::GetPrivateTime() { CAutoLock cObjectLock(this); UINT64 qpcNow = GetCurrentTimestamp(); DWORD dwTime = timeGetTime(); UINT64 hwClock(0); UINT64 hwQpc(0); UINT64 hwClockEnd(0); UINT64 hwQpcEnd(0); HRESULT hr = m_pAudioRenderer->AudioClock(hwClock, hwQpc); if (hr == S_OK) { if (m_dStartQpcHW == 0) m_dStartQpcHW = hwQpc; if (m_dStartTimeHW == 0) m_dStartTimeHW = hwClock; if (m_dStartTimeSystem == 0) m_dStartTimeSystem = dwTime; if (m_dStartTimeCorrected == 0) m_dStartTimeCorrected = m_rtPrivateTime; m_dDurationHW = (hwClock - m_dStartTimeHW) / 10000; m_dDurationSystem = (dwTime - m_dStartTimeSystem); m_dDurationCorrected = (m_rtPrivateTime - m_dStartTimeCorrected) / 10000; //if (hwQpc < m_dPrevQpcHW) //Log("%I64d", hwQpc - m_dPrevQpcHW); if (m_dPrevTimeHW > hwClock) { m_dStartTimeHW = m_dPrevTimeHW = hwClock; m_dStartQpcHW = m_dPrevQpcHW = hwQpc; m_dStartTimeSystem = dwTime; m_dStartTimeCorrected = m_rtPrivateTime; m_llDeltaError = 0; overallCorrection = 0; } else { double clockDiff = hwClock - m_dStartTimeHW; double qpcDiff = hwQpc - m_dStartQpcHW; if (clockDiff > 0 && qpcDiff > 0) m_dSystemClockMultiplier = clockDiff / qpcDiff; if (m_dSystemClockMultiplier < 0.95 || m_dSystemClockMultiplier > 1.05) m_dSystemClockMultiplier = 1.0; m_dPrevTimeHW = hwClock; m_dPrevQpcHW = hwQpc; } } else { //Log("AudioClock() returned error (0x%08x)"); } REFERENCE_TIME delta = REFERENCE_TIME(dwTime) - REFERENCE_TIME(m_dwPrevSystemTime); REFERENCE_TIME deltaOrig = delta; if (dwTime < m_dwPrevSystemTime) { delta += REFERENCE_TIME(UINT_MAX) + 1; } m_dwPrevSystemTime = dwTime; delta = (REFERENCE_TIME)(delta * (UNITS / MILLISECONDS)); double dAdjustment; if (m_bHWBasedRefClock) { dAdjustment = m_dAdjustment * m_dBias / m_dSystemClockMultiplier; } else { dAdjustment = m_dAdjustment * m_dBias; } double ddelta= ((double) delta)*dAdjustment; delta = (REFERENCE_TIME) ddelta; m_ddeltaError += ddelta - delta; if (m_ddeltaError>1.0) { delta--; m_ddeltaError-=1.0; overallCorrection += 1.0; } if (hwQpc - m_dStartQpcHW > 600000000) { Log("mul: %.10f de: %6I64d de.orig: %6I64d de.err: %.10f bias: %.10f adj: %.10f hwQpc: %I64d hwClock: %I64d", m_dSystemClockMultiplier, delta, deltaOrig * 10000, m_ddeltaError, m_dBias, m_dAdjustment, hwQpc, hwClock); } m_rtPrivateTime = m_rtPrivateTime + delta; return m_rtPrivateTime; } [/code] [/QUOTE]

Verification