[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: Remove METHOD_ENTRY and METHOD_EXIT events from needed_capabilities in JDWP Agent
From:       "Ronald Servant" <Ronald_Servant () ca ! ibm ! com>
Date:       2017-06-21 14:35:15
Message-ID: OFAF5D7ACD.CA57A818-ON00258146.004FDD28-00258146.005021DC () notes ! na ! collabserv ! com
[Download RAW message or body]

<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial, Helvetica, \
sans-serif;font-size:10.5pt" ><blockquote data-history-content-modified="1" \
data-history-expanded="1" dir="ltr" style="border-left:solid #aaaaaa 2px; \
margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" ><blockquote \
cite="mid:OF17185C46.8474EA24-ON00258145.006B861B-00258145.006C82BF@notes.na.collabserv.com" \
type="cite" ><div dir="ltr" style="font-family:Arial, Helvetica,        \
sans-serif;font-size:10.5pt" >&nbsp; <div dir="ltr" >Looking at the Agent_OnLoad() \
code in debuginit.c of libjdwp, I see that both the METHOD_ENTRY and METHOD_EXIT are \
listed as required, but I don't see where they are ever enabled in the agent.</div> \
<div dir="ltr" >&nbsp;</div> <div dir="ltr" >In the J9 JVM making it possible to \
enable these events has severe performance penalties. Since the events are not \
enabled in the JDWP agent, simply reserved, we were hoping you'd be open to removing \
the reservation of the capabilities.</div></div></blockquote>If the JDWP agent never \
enables these events then it shouldn't need to obtain the capabilities. I have a \
vague recollection that they are enabled when single stepping, are you sure this \
isn't the case? Have you run the tests with without enabling the \
capabilities?<br><br>-Alan</blockquote> <div dir="ltr" >You are right, it is used for \
single stepping.&nbsp; I missed that in my earlier scan of the code. Ignore my \
request. <div>&nbsp;</div>-- Ron</div>
<div dir="ltr" >&nbsp;</div></div><BR>


[prev in list] [next in list] [prev in thread] [next in thread]