Bugzilla – Full Text Bug Listing
| Summary: | [non-stop] A process that has not been started should be considered as suspended in non-top mode too | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Tools] CDT | Reporter: | Marc Khouzam <marc.khouzam> | ||||||
| Component: | cdt-debug-dsf-gdb | Assignee: | Marc Khouzam <marc.khouzam> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | Marc Khouzam <marc.khouzam> | ||||||
| Severity: | normal | ||||||||
| Priority: | P3 | CC: | cdtdoug, pawel.1.piech | ||||||
| Version: | 7.0 | Flags: | marc.khouzam:
review?
(nobody) |
||||||
| Target Milestone: | 8.0 | ||||||||
| Hardware: | PC | ||||||||
| OS: | Linux | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
Created attachment 192212 [details]
Better fix
Small typo in previous patch
Pawel, can you review? (3 lines) *** cdt cvs genie on behalf of mkhouzam *** Bug 341396: A process that has not been started should be considered as suspended in non-top mode too [*] GDBRunControl_7_0_NS.java 1.30 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/gdb/service/GDBRunControl_7_0_NS.java?root=Tools_Project&r1=1.29&r2=1.30 (In reply to comment #2) > Pawel, can you review? (3 lines) I'm pretty out of touch with the non stop implementation at this point. So I don't think my review would be very valuable. (In reply to comment #4) > (In reply to comment #2) > > Pawel, can you review? (3 lines) > > I'm pretty out of touch with the non stop implementation at this point. So I > don't think my review would be very valuable. I understand. Mikhail, can you review instead? |
Created attachment 192211 [details] Fix In all-stop mode, before we start the process, we consider the process as suspended. This is not currently the case for non-stop and I believe it was an oversight. This is causing me problems when trying to set an initial breakpoint with GDB 7.2 in non-stop. The problem happens because we think the process is running and we try to interrupt one of its threads, so we request the list of threads, and there are none, since the process isn't started. The patch fixes that. Committed to HEAD.