At Fri, 20 Jul 2012 14:47:24 -0500, Pierre-Louis Bossart wrote:
On 7/18/2012 12:10 PM, Takashi Iwai wrote:
Sorry for the late reply. I've looked over the things around my vacation... This version looks better, so if other people have no objection, I can merge it. The extension of mmap_status seems still fitting with 64bit char size, so it should be OK. But, one thing -- can't we reuse the normal tstamp field for the audio wallclock? That is, add a new tstamp mode, and then put wallclock tstamp there instead of the normal system tstamp. Just an idea. thanks, Takashi
Thanks for reviewing the updated patches. No, the audio wall clock needs to come in addition to the regular system tstamp. This is needed for applications such as PulseAudio who try to estimate the drift between system time and audio time. If you remove the system time information, they can't program a timer any longer. Better wait a bit to merge, I need to work on all the TODOs, complete some additional validation for corner cases and get your feedback live at LPC.
Alright. Let's cook a bit longer.
Takashi