Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323276 - [debug view][cdi] "Force thread list update feature" does not work with GDB 7.0 or later
Summary: [debug view][cdi] "Force thread list update feature" does not work with GDB 7...
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: 7.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 7.0.1   Edit
Assignee: Marc Khouzam CLA
QA Contact: Marc Khouzam CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-20 13:44 EDT by Marc Khouzam CLA
Modified: 2010-09-27 14:53 EDT (History)
2 users (show)

See Also:
john.cortell: review+

Attachments
Fix (1.05 KB, patch)
2010-08-20 13:44 EDT, Marc Khouzam CLA 		marc.khouzam: iplog-
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Khouzam CLA 2010-08-20 13:44:53 EDT 
Created attachment 177121 [details]
Fix

The solution to bug 280631 does not work for GDB 7.0 or later.
This is because in that case, when we clear GDBProcesses_7_0.fThreadCommandCache using e.getDMContext(), this context is a Container context; on the other hand, when we use fThreadCommandCache, we use it with a ICommandControlDMContext.  Therefore, the clearing of the cache does not actually clear anything.

I am able to test this fix with gdbserver, as it does not seem to be reporting new thread creation.

I believe the solution is to clear the cache for the ICommandControlDMContext instead.  I've successfully tested this patch which implements this solution.
Comment 1 Marc Khouzam CLA 2010-08-20 14:05:32 EDT 
Committed patch to HEAD and 7_0.

John, can you review?
Comment 2 John Cortell CLA 2010-09-27 14:53:20 EDT 
Looks good to me.