Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1317600 - ocl-icd doesn't depend on any ICD implementation
Summary: ocl-icd doesn't depend on any ICD implementation
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: ocl-icd
Version: 26
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rob Clark
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1317602 1317603 1317605
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-03-14 16:09 UTC by Dominik 'Rathann' Mierzejewski
Modified: 2018-01-08 21:33 UTC (History)
9 users (show)

Fixed In Version: ocl-icd-2.2.8-3.git20151217.0122332.fc24
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-01-08 21:33:50 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)

Description Dominik 'Rathann' Mierzejewski 2016-03-14 16:09:35 UTC 
Description of problem:
ocl-icd doesn't depend on any OpenCL ICD implementation, so after installing it, you still have to install an ICD manually. Currently, there are three ICDs available in Fedora:
mesa-libOpenCL (mainly AMD GPUs)
beignet (Intel GPUs)
pocl (CPU-based implementation)

I'd suggest adding something like:
Requires: opencl-icd
or at least:
Recommends: opencl-icd
to ocl-icd.spec. I'm opening bugs against the three packages mentioned above to include this common provide.

Version-Release number of selected component (if applicable):
2.2.8-2.git20151217.0122332.fc24
Comment 1 Björn 'besser82' Esser 2016-04-08 17:05:01 UTC 
Rebuild with virtual Provides failed (but not related to that) for beignet.  Pocl and Mesa have been rebuilt.  Ocl-icd has been rebuilt with Requires: opencl-icd.
Comment 2 Fedora Update System 2016-04-08 18:04:18 UTC 
mesa-11.2.0-0.devel.13.24ea81a.fc24 ocl-icd-2.2.8-3.git20151217.0122332.fc24 pocl-0.13-4.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-9fa31fd9f3
Comment 3 Fedora Update System 2016-04-09 18:52:22 UTC 
mesa-11.2.0-0.devel.13.24ea81a.fc24, ocl-icd-2.2.8-3.git20151217.0122332.fc24, pocl-0.13-4.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-9fa31fd9f3
Comment 4 Fedora Update System 2016-04-11 09:11:28 UTC 
mesa-11.2.0-0.devel.13.24ea81a.fc24, ocl-icd-2.2.8-3.git20151217.0122332.fc24, pocl-0.13-4.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.
Comment 5 Igor Gnatenko 2016-08-14 11:07:20 UTC 
I reverted this for a while because it creates some problems.
Comment 6 Igor Gnatenko 2016-08-14 11:08:55 UTC 
We should think how to implement something what has been done in SUSE long long time ago. Dependencies based on PCI ID.

Otherwise it just doesn't make sense to add such provides as in 80% cases it will not work.
Comment 7 Benson Muite 2016-08-16 08:42:11 UTC 
This is an interesting idea. May provide a standard build for each architecture and possibly suggestions for how to recompile or optimized builds in copr repositories. For other packages which use OpenCL in libraries built on OpenCL, will build system allow easy use of correct dependencies?
Comment 8 Nicolas Chauvet (kwizart) 2016-09-08 07:32:23 UTC 
Can we have a fixed package (without the Requires) for f24.
Comment 9 Fedora End Of Life 2017-02-28 09:55:39 UTC 
This bug appears to have been reported against 'rawhide' during the Fedora 26 development cycle.
Changing version to '26'.
Comment 10 Nicolas Chauvet (kwizart) 2018-01-08 21:31:39 UTC 
What is the current status of this issue ?
It seems like an openCL implementation is installed in my case (armv7) whereas none is relevant. Unless an OpenCL enabled app really mandate an opencl backend (In think none is in this case, having blender in mind). There should be no Requires or even Suggests that would install one by default.

Best would be to rely on appdata to install any appropriate OpenCL implementation (see how it's done with nvidia on rpmfusion). than rely on any random choice for an implementation.
Comment 11 Igor Gnatenko 2018-01-08 21:33:50 UTC 
I wish dnf would automagically install opencl drivers based on hardware (as thing with metainfo/appdata does)....

So I agree that until dnf supports this, no way to implement this.
Note You need to log in before you can comment on or make changes to this bug.