首先得给你敲个警钟：这个-race检测到的数据竞争是真实存在且有严重风险的，完全不能忽略。它会导致你的请求计数器状态混乱，甚至可能让限流逻辑彻底失效，必须修复。

问题根源分析 #

你的代码里有两个并发操作在访问hits变量，但只有其中一个（allowCall方法）用了全局锁m保护，而另一个——每秒重置hits的tick协程——直接在无锁的情况下修改subject.hits。这就造成了典型的并发读写竞态：当allowCall正在读取或递增hits时，tick协程可能同时在写这个变量，导致计数器的值出现不可预测的错误（比如递增到一半被重置，或者重置后又被覆盖）。

另外，你的allowCall方法还有个隐藏bug：它在循环等待期间一直持有锁，这会导致tick协程永远拿不到锁去重置hits。一旦请求数达到hitsSecond的上限，所有后续调用都会陷入无限等待，因为tick协程根本没机会把计数器清零——这比数据竞争的问题更致命。

修复方案 #

方案1：给tick协程加上锁（快速修复） #

最简单的修改是让tick协程在重置hits时也获取全局锁m：

func init() {
    subject = httpClientWrapper{
        hits: 0,
        hitsSecond: 1,
    }
    // reset hits every second to 0
    go func() {
        tick := time.Tick(1 * time.Second)
        for range tick {
            m.Lock()
            subject.hits = 0
            m.Unlock()
        }
    }()
}

同时记得修改allowCall里的循环，避免持有锁sleep：

func (c *httpClientWrapper) allowCall() {
    m.Lock()
    for c.hits >= c.hitsSecond {
        m.Unlock() // 等待前释放锁，让tick协程能重置计数器
        time.Sleep(1 * time.Second)
        m.Lock()
    }
    c.hits = c.hits + 1
    m.Unlock()
}

方案2：把锁封装到结构体内部（更优雅的设计） #

全局锁不是好的实践，最好把锁作为httpClientWrapper的成员，实现每个实例独立的锁保护：

type httpClientWrapper struct {
    hits, hitsSecond int
    mu sync.Mutex // 结构体私有锁
}

func init() {
    subject = httpClientWrapper{
        hits: 0,
        hitsSecond: 1,
    }
    go func() {
        tick := time.Tick(1 * time.Second)
        for range tick {
            subject.mu.Lock()
            subject.hits = 0
            subject.mu.Unlock()
        }
    }()
}

func (c *httpClientWrapper) allowCall() {
    c.mu.Lock()
    for c.hits >= c.hitsSecond {
        c.mu.Unlock()
        time.Sleep(1 * time.Second)
        c.mu.Lock()
    }
    c.hits++
    c.mu.Unlock()
}

这个方案更符合Go的封装原则，避免了全局锁带来的潜在冲突，也让代码的职责更清晰。

为什么不能忽略告警？ #

Go的竞争检测器只会报告真实存在的竞态条件，忽略它意味着你愿意承担计数器值错乱、限流逻辑失效的风险。比如：

• 当allowCall正在执行c.hits = c.hits + 1时，tick协程突然重置hits为0，最终hits可能变成1（正常）、0（错误），甚至出现奇怪的中间值（取决于CPU指令的执行顺序）。
• 极端情况下，竞态条件可能导致内存可见性问题：某个协程修改了hits的值，但其他协程看不到最新的结果，导致限流逻辑完全失控。

所以别犹豫，赶紧修复这个问题！

内容的提问来源于stack exchange，提问作者LeniM