AltraMayor / AltraMayor/gatekeeper

Review `time_resolution_init()`

未关闭
#82 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
enhancement good first issue
主要语言
C
星标
1.6k
派生
252
PR 合并指标
30 天内没有已合并 PR

描述

Function `time_resolution_init()` located in file `main/main.c` estimates the frequency of the TSC register and sets its result in a couple global variable in different units (e.g. `cycles_per_sec`). DPDK has function `rte_get_tsc_hz()`, which is equivalent to the global variable `cycles_per_sec`.

Which function is more precise, `time_resolution_init()` or `rte_get_tsc_hz()`? If `time_resolution_init()`, we should submit a patch to DPDK. In any case, we should adopt `rte_get_tsc_hz()` in `time_resolution_init()` to simplify our code.

We should also investigate if the global variables are still needed. For example, should we replace variable `cycles_per_sec` with `rte_get_tsc_hz()` or keep it?

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。