Thank you all!
On Wed, Oct 8, 2014 at 9:51 AM, Volker Simonis <volker.simo...@gmail.com>
wrote:
> 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
> >
> >
>
