OpenCL interoperability does not work on 64-bit OS
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 48/100
Direzione di ricerca
I punti di ingresso interessati si trovano in arrayfire/opencl.py intorno alle righe 88 e 156, oltre agli altri utilizzi di c_int_t in quel file. Leggi prima queste dichiarazioni ctypes, quindi riproduci la configurazione OpenCL/PyOpenCL a 64 bit su Windows; il lavoro è completato quando gli ID dei dispositivi e i valori condivisi di dispositivo, contesto e coda non vanno più in overflow e l’interoperabilità funziona.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Currently the OpenCL interoperability does not work on 64-bit Windows machine. The reason for this is the use of c_int_t which defaults to long on Windows (which on the other hand is 32-bit). This can be problematic with things like device IDs as they can easily go over 2^32/2-1. For example https://github.com/arrayfire/arrayfire-python/blob/master/arrayfire/opencl.py#L88 uses c_int_t, as well as a few other functions in the same file. This can be fixed by replacing the c_int_t with, for example, c_void_ptr_t as it is used with the context. Is there by the way some specific reason why context is treated differently?
There is also another issue. When the context is created by PyOpenCL, the input device, context and queue values are input as python ints in order to use the same context and queue, but on Windows you get <class 'OverflowError'>: int too long to convert errors then. These can be avoided by casting them to, for example, c_int64-types. For example in https://github.com/arrayfire/arrayfire-python/blob/master/arrayfire/opencl.py#L156, replace safe_call(backend.get().afcl_add_device_context(dev, ctx, que)) with safe_call(backend.get().afcl_add_device_context(ct.c_int64(dev), ct.c_int64(ctx), ct.c_int64(que))) (using c_void_p seems to work too).
I haven't tested these changes extensively, but they do seem to get things working on 64-bit Windows. I can provide a PR if needed.
- Lingua principale
- Python
- Stelle
- 422
- Fork
- 63
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di arrayfire/arrayfire-python
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
arrayfire/arrayfire-python#271 ·
-
Unwrap phase function Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
arrayfire/arrayfire-python#270 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
arrayfire/arrayfire-python#269 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
arrayfire/arrayfire-python#268 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
arrayfire/arrayfire-python#267 · 1 commento · 1 reazione ·
Tutte le issue di arrayfire/arrayfire-python
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
learningequality/ricecooker#747 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
run-llama/llama_index#23199 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
KhronosGroup/glTF-Blender-IO#2769 ·