Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Remove clinfo #87

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove clinfo #87

wants to merge 1 commit into from

Conversation

candrews
Copy link

clinfo isn't part of the OpenCL runtime - it should be in a separate repository as a separate project.

Alternatively, rely on an existing distribution of clinfo, such as https://github.com/Oblomov/clinfo

@ranisalt
Copy link

Could we have some attention here? This bloats the package and every distro already packages clinfo.

@baryluk
Copy link

baryluk commented Oct 5, 2020

The clinfo distributed by most distros, is a different complete reimplementation of clinfo. The AMD created clinfo first in July 2014. :)

Oblomov's clinfo was created in June 2013. As a reimplementation of AMD's C++ clinfo, and quickly gained more features.

AMD's clinfo got some extra features (like raw output, human formating for values in some fields in non-raw mode, topology / tree view, more properties and amd / nv extensions support, tested on more platforms in general) later on too, but at this stage the Oblomov's clinfo is maintained by more people and has more features and consistency.

The output format between the two is similar, but actually do have substantial differences, even for simple things.

However, I would say any tools that parse the output from clinfo, expect things in a format of Oblomov's clinfo. They shouldn't really be doing that, but sometimes it is a quick way to diagnose issues or send reports. Also in raw mode it is very valid.

There is yet another clinfo, from 2013 by Simon Leblanc, but that is rather poorly maintained and not future rich.

I think AMD wants to keep own clinfo, because, this way they can support it easier on Windows, and also add own extensions quicker without going via external party reviews, etc.

Moving to a separate repo would be best option IMHO.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants