webrev to remove AMD GPU check

S. Bharadwaj Yadavalli bharadwaj.yadavalli at oracle.com
Fri Nov 22 07:59:48 PST 2013


This change looks good to me. I have successfully run a couple of PTX 
tests on a machine with Intel CPU + Nvidia GPU with this patch.

Thanks,

Bharadwaj

On 11/22/2013 09:13 AM, Caspole, Eric wrote:
> Hi everybody,
> After discussing it with Bharadwaj the other day, I made a webrev to remove AMD GPU detection since for the time being we are only running on the HSAIL simulator, which only depends on shared libraries. I found that since AMD CPUs and GPUs have the same vendor ID, this part was not working as intended if you have a system with an AMD CPU with Nvidia GPU so nothing would be offloaded. We will adjust this AMD APU detection when the real HSA system is generally available.
>
>   http://cr.openjdk.java.net/~ecaspole/no_amd_hw_check/
>
> In the course of this change, I moved the gpu::init during the JVM boot up after the arguments are processed so the +UseHSAILSimulator I added earlier will work correctly.
> Let me know what you think,
> Thanks,
> Eric
>



More information about the graal-dev mailing list