Consolidate cache info parsing / setup
Open
@shenki is already working on this.
Since Oct 12, 2018.
bug
medium
- Dominant language
- No language data
- Stars
- 14
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
We have to places in the code that look at CPU cache info.
There's the code in `setup_64.c`, `init_cache_info()` etc., which is used to set the `AT_L2_CACHESHAPE` etc. properties. And then the topology related code in `cacheinfo.c`.
These should be consolidated. Probably the `setup_64.c` code should be moved into `cacheinfo.c`, and then the topology code should be updated to use the info it generates.
That will then mean the workaround for bad cache info in `init_cache_info()` will be effective for both the `AT` properties and the sysfs info.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.