search

intel / compute-runtime

Intel® Graphics Compute Runtime for oneAPI Level Zero and OpenCL™ Driver
MIT License
1.14k stars 233 forks source link

Add Gen6 and Gen7 GPU support (currently provided by Intel's Beignet driver and Mesa's Clover) #507

Closed hilga007 closed 2 years ago

hilga007 commented 2 years ago

Beignet provides OpenCL 1.2 support for Gen6 and Gen7 graphics. Please add these for OpenCL 3.0 (feature level 1.2) support.

NUCs and Compute Sticks were the "Next Unit of Computing" but the "next" level of OpenCL support and commitment to a platform seems to be lacking

hilga007 commented 2 years ago

Who to contact to request beignet simply be merged into mesa for mesa-opencl which could include its AMD backends + a bringup to modern llvm for beignet. It is for "outdated" tech (Ivy Bridge, Haswell) but still has uses.

For example: budget-strapped healthcare organizations could benefit from some ultrasound imaging that has a huge OpenCL perf boost vs. CPU software computations. These organizations are stuck on Sandy Bridge / Ivy Bridge / Haswell, etc.

Thoughts?

On Sun, Apr 17, 2022 at 10:31 PM antermin @.***> wrote:

Apparently Beignet does not support Gen6 (Sandy Bridge). Also, from ArchWiki and Mesa source code, apparently Clover only works for AMD (and limited support for iris+clover), while crocus+clover (Gen7 and Gen8LP use crocus) is not supported.

For Gen8LP, there was a feature request, but they decided https://github.com/intel/compute-runtime/issues/317#issuecomment-688236589 not to support Gen8LP in NEO Compute Runtime.

Maybe you will also want to check out the WIP rusticl in Mesa https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15439 and this comment https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15439#note_1304834 .

— Reply to this email directly, view it on GitHub https://github.com/intel/compute-runtime/issues/507#issuecomment-1101110103, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIGW35YYTFOKA4WRY7FXXFDVFTXTZANCNFSM5N4BZW7A . You are receiving this because you authored the thread.Message ID: @.***>

eero-t commented 2 years ago

Old drivers (in this case beignet) are for supporting old HW. It does not make sense to have single driver supporting all HW due to differences in how the HW is programmed and what features that HW supports. At some point there just needs to be clean break to keep driver maintainable.

Btw. It's not just compute where the driver changed between GEN8 <-> GEN9 (beignet -> compute-runtime), also Mesa (i965/crocus -> Iris OpenGL driver) and Media (i965 -> iHD VA-API driver) did that.

And questions about Beignet and Mesa belong to those projects, not here. Please close this ticket.

JablonskiMateusz commented 2 years ago

Hi @hilga007, Gen6 and Gen7 GPUs are supported by beignet driver, please create an issue in their repo https://github.com/intel/beignet.

I'm closing this issue as we are not planning to support these GPUs.

hilga007 commented 2 years ago

Apparently they no longer are, either, hence asking to have them included in the supported driver.

On Tue, May 17, 2022 at 11:46 AM Mateusz Jablonski @.***> wrote:

Closed #507 https://github.com/intel/compute-runtime/issues/507.

— Reply to this email directly, view it on GitHub https://github.com/intel/compute-runtime/issues/507#event-6626824820, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIGW355B5HIUDLKKRPWTR33VKPSQJANCNFSM5N4BZW7A . You are receiving this because you were mentioned.Message ID: @.***>