TigerVNC项目中gettimeofday计时器异常问题分析与修复

问题背景

在TigerVNC项目的最新代码提交后，用户在使用x0vncserver时发现控制台输出了大量"Timer: gettimeofday is broken..."的错误信息。这些错误信息表明系统计时器功能出现了异常，可能会影响VNC服务器的正常运行和性能表现。

问题现象

错误信息显示系统认为gettimeofday系统调用存在问题，具体表现为计时器计算出现偏差。开发人员通过调试发现，每次触发错误时，计算得到的等待时间(toWait)与超时时间(timeoutMs)之间总是相差1毫秒，例如：

toWait=51, timeoutMs=50
toWait=9, timeoutMs=8

问题根源分析

经过深入排查，发现问题源于时间差计算函数diffTimeMillis的实现方式。原始代码将秒和微秒分开计算后再合并，这种计算方式在微秒部分发生溢出时会导致精度丢失。具体来说，当微秒差值计算出现负值时，加上999再进行除法运算会产生不准确的结果。

技术细节

在Linux系统中，gettimeofday系统调用返回一个timeval结构体，包含tv_sec(秒)和tv_usec(微秒)两个字段。原始的时间差计算方法是：

((later.tv_sec - earlier.tv_sec) * 1000) + ((later.tv_usec - earlier.tv_usec + 999) / 1000)

这种方法存在两个潜在问题：

1. 秒和微秒部分分开计算，忽略了微秒部分可能的溢出
2. 在微秒差为负值时，直接加999会导致计算结果偏差

解决方案

修复方案是将时间差统一转换为微秒单位计算，然后再转换为毫秒：

long udiff;
udiff = ((later.tv_sec - earlier.tv_sec) * 1000000) +
        (later.tv_usec - earlier.tv_usec);
return (udiff + 999) / 1000;

这种改进后的算法具有以下优点：

1. 统一使用微秒作为计算单位，避免中间结果溢出
2. 整体计算时间差后再进行毫秒转换，保证精度
3. 正确处理负时间差情况

验证与测试

修复后经过严格测试，特别是在触发大量计时器事件的场景下(如打开文件管理器)，确认：

1. 不再出现"gettimeofday is broken"错误
2. 计时器功能正常工作
3. 系统性能稳定

总结

这次问题修复展示了时间处理在系统编程中的重要性，特别是在需要高精度计时的网络应用中。通过将时间差计算统一到更小的单位(微秒)后再转换，可以有效避免边界条件导致的精度问题。这一改进不仅解决了当前问题，也为TigerVNC项目的计时器可靠性提供了更好的保障。