Thanks Alan, Staffan,
I'll push the change with the spacing adjusted as suggested by Alan.
Regards,
Volker
On Wed, Oct 8, 2014 at 9:33 AM, Staffan Larsen
<staffan.lar...@oracle.com> wrote:
> Looks good!
>
> Thanks,
> /Staffan
>
> On 7 okt 2014, at 16:26, Thomas Stüfe <thomas.stu...@gmail.com> wrote:
>
> Hi all,
>
> We saw crashes when connecting to a target VM using
> com.sun.tools.attach.WindowsVirtualMachine when injecting VM was compiled
> with /RTC1.
>
> The error turned out to be in VirtualMachineImpl.c: the function
> "jvm_attach_thread_func" - the one which is injected into the target VM and
> used as thread entry point for CreateRemoteThread() - must be compiled with
> runtime checks disabled in order to keep the code-to-inject position
> independent.
>
> Using /rtc1 will cause the Microsoft compiler to generate relative calls to
> a check function ("_RTC_CheckEsp") which will not work if code is planted in
> target process at a different address.
>
> This change adds a pragma to locally disable the runtime checks and
> re-enable them below the function.
>
> http://cr.openjdk.java.net/~simonis/webrevs/8059868/
>
> Kind regards,
>
> Thomas Stuefe
>
>
