Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARM GPU modifications #24

Closed
wants to merge 1 commit into from
Closed

ARM GPU modifications #24

wants to merge 1 commit into from

Conversation

marcino239
Copy link
Contributor

Add function to check if local memory is present. Without this check the OpenCL code generation fails on systems without local memory.

@abergeron
Copy link
Member

This doesn't merge since I've added BIN_ID.

Also, this property is useless since the OpenCL spec requires at least 16Kb of local memory (which may be actually global memory or whatever).

What error are you trying to fix exactly?

@marcino239
Copy link
Contributor Author

Hi - the Mali GPU returns error when calling function lmemory in gpuarray.pyx. I think this might be happening because when asking for CL_DEVICE_LOCAL_MEM_TYPE, the runtime returns CL_GLOBAL. I'll rework the code to resolve the conflicts.

@abergeron
Copy link
Member

No, returning CL_GLOBAL is fine, libgpuarray never checks for that and the spec allows it.

What isn't allowed is to return less than 16Kb for CL_DEVICE_LOCAL_MEM_SIZE or erroring out.

Can you give me more information as to what error exactly is the Mali GPU returning? Can you show me the exact error message you get?

@abergeron
Copy link
Member

This has been on here for over 6 months and I heard nothing back. Closing for now.

@abergeron abergeron closed this Sep 8, 2015
@nouiz nouiz mentioned this pull request Sep 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants