annotate agent/src/share/classes/sun/jvm/hotspot/debugger/windbg/amd64/WindbgAMD64Thread.java @ 7974:43badbe2717a

8000973: SA on windows thread inspection is broken Summary: After bug 7161732, On Windows SA could not find correct address of thread_id of OSThread since _thread_id moved to end of the class . The presupposition of the address is following thread handle no longer stands. Fix by adding thread_id field to OSThread and getting the address directly from OSThread. Reviewed-by: nloodin, sspitsyn Contributed-by: yumin.qi@oracle.com
author minqi
date Thu, 31 Jan 2013 17:43:01 -0800
parents c18cbe5936b8
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
7974
43badbe2717a 8000973: SA on windows thread inspection is broken
minqi
parents: 1552
diff changeset
2 * Copyright (c) 2005, 2013, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 0
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
a61af66fc99e Initial load
duke
parents:
diff changeset
25 package sun.jvm.hotspot.debugger.windbg.amd64;
a61af66fc99e Initial load
duke
parents:
diff changeset
26
a61af66fc99e Initial load
duke
parents:
diff changeset
27 import sun.jvm.hotspot.debugger.*;
a61af66fc99e Initial load
duke
parents:
diff changeset
28 import sun.jvm.hotspot.debugger.amd64.*;
a61af66fc99e Initial load
duke
parents:
diff changeset
29 import sun.jvm.hotspot.debugger.windbg.*;
a61af66fc99e Initial load
duke
parents:
diff changeset
30
a61af66fc99e Initial load
duke
parents:
diff changeset
31 class WindbgAMD64Thread implements ThreadProxy {
a61af66fc99e Initial load
duke
parents:
diff changeset
32 private WindbgDebugger debugger;
a61af66fc99e Initial load
duke
parents:
diff changeset
33 private long sysId;
a61af66fc99e Initial load
duke
parents:
diff changeset
34 private boolean gotID;
a61af66fc99e Initial load
duke
parents:
diff changeset
35 private long id;
a61af66fc99e Initial load
duke
parents:
diff changeset
36
7974
43badbe2717a 8000973: SA on windows thread inspection is broken
minqi
parents: 1552
diff changeset
37 // The address argument must be the address of the OSThread::_thread_id
0
a61af66fc99e Initial load
duke
parents:
diff changeset
38 WindbgAMD64Thread(WindbgDebugger debugger, Address addr) {
a61af66fc99e Initial load
duke
parents:
diff changeset
39 this.debugger = debugger;
7974
43badbe2717a 8000973: SA on windows thread inspection is broken
minqi
parents: 1552
diff changeset
40 this.sysId = (long)addr.getCIntegerAt(0, 4, true);
43badbe2717a 8000973: SA on windows thread inspection is broken
minqi
parents: 1552
diff changeset
41 gotID = false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
42 }
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 WindbgAMD64Thread(WindbgDebugger debugger, long sysId) {
a61af66fc99e Initial load
duke
parents:
diff changeset
45 this.debugger = debugger;
a61af66fc99e Initial load
duke
parents:
diff changeset
46 this.sysId = sysId;
a61af66fc99e Initial load
duke
parents:
diff changeset
47 gotID = false;
a61af66fc99e Initial load
duke
parents:
diff changeset
48 }
a61af66fc99e Initial load
duke
parents:
diff changeset
49
a61af66fc99e Initial load
duke
parents:
diff changeset
50 public ThreadContext getContext() throws IllegalThreadStateException {
a61af66fc99e Initial load
duke
parents:
diff changeset
51 long[] data = debugger.getThreadIntegerRegisterSet(getThreadID());
a61af66fc99e Initial load
duke
parents:
diff changeset
52 WindbgAMD64ThreadContext context = new WindbgAMD64ThreadContext(debugger);
a61af66fc99e Initial load
duke
parents:
diff changeset
53 for (int i = 0; i < data.length; i++) {
a61af66fc99e Initial load
duke
parents:
diff changeset
54 context.setRegister(i, data[i]);
a61af66fc99e Initial load
duke
parents:
diff changeset
55 }
a61af66fc99e Initial load
duke
parents:
diff changeset
56 return context;
a61af66fc99e Initial load
duke
parents:
diff changeset
57 }
a61af66fc99e Initial load
duke
parents:
diff changeset
58
a61af66fc99e Initial load
duke
parents:
diff changeset
59 public boolean canSetContext() throws DebuggerException {
a61af66fc99e Initial load
duke
parents:
diff changeset
60 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
61 }
a61af66fc99e Initial load
duke
parents:
diff changeset
62
a61af66fc99e Initial load
duke
parents:
diff changeset
63 public void setContext(ThreadContext thrCtx)
a61af66fc99e Initial load
duke
parents:
diff changeset
64 throws IllegalThreadStateException, DebuggerException {
a61af66fc99e Initial load
duke
parents:
diff changeset
65 throw new DebuggerException("Unimplemented");
a61af66fc99e Initial load
duke
parents:
diff changeset
66 }
a61af66fc99e Initial load
duke
parents:
diff changeset
67
a61af66fc99e Initial load
duke
parents:
diff changeset
68 public boolean equals(Object obj) {
a61af66fc99e Initial load
duke
parents:
diff changeset
69 if ((obj == null) || !(obj instanceof WindbgAMD64Thread)) {
a61af66fc99e Initial load
duke
parents:
diff changeset
70 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
71 }
a61af66fc99e Initial load
duke
parents:
diff changeset
72
a61af66fc99e Initial load
duke
parents:
diff changeset
73 return (((WindbgAMD64Thread) obj).getThreadID() == getThreadID());
a61af66fc99e Initial load
duke
parents:
diff changeset
74 }
a61af66fc99e Initial load
duke
parents:
diff changeset
75
a61af66fc99e Initial load
duke
parents:
diff changeset
76 public int hashCode() {
a61af66fc99e Initial load
duke
parents:
diff changeset
77 return (int) getThreadID();
a61af66fc99e Initial load
duke
parents:
diff changeset
78 }
a61af66fc99e Initial load
duke
parents:
diff changeset
79
a61af66fc99e Initial load
duke
parents:
diff changeset
80 public String toString() {
a61af66fc99e Initial load
duke
parents:
diff changeset
81 return Long.toString(getThreadID());
a61af66fc99e Initial load
duke
parents:
diff changeset
82 }
a61af66fc99e Initial load
duke
parents:
diff changeset
83
a61af66fc99e Initial load
duke
parents:
diff changeset
84 /** Retrieves the thread ID of this thread by examining the Thread
a61af66fc99e Initial load
duke
parents:
diff changeset
85 Information Block. */
a61af66fc99e Initial load
duke
parents:
diff changeset
86 private long getThreadID() {
a61af66fc99e Initial load
duke
parents:
diff changeset
87 if (!gotID) {
a61af66fc99e Initial load
duke
parents:
diff changeset
88 id = debugger.getThreadIdFromSysId(sysId);
a61af66fc99e Initial load
duke
parents:
diff changeset
89 }
a61af66fc99e Initial load
duke
parents:
diff changeset
90
a61af66fc99e Initial load
duke
parents:
diff changeset
91 return id;
a61af66fc99e Initial load
duke
parents:
diff changeset
92 }
a61af66fc99e Initial load
duke
parents:
diff changeset
93 }