Bug 215416 - CDI/MI framework has hardcoded timeouts for interrupt
Summary: CDI/MI framework has hardcoded timeouts for interrupt
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug (show other bugs)
Version: 4.0.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 4.0.3   Edit
Assignee: Doug Schaefer CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-01-15 17:54 EST by Elena Laskavaia CLA
Modified: 2008-06-22 02:37 EDT (History)
1 user (show)

See Also:

Attachments
4_0 patch (5.09 KB, patch)
2008-01-15 17:56 EST, Elena Laskavaia CLA 		bjorn.freeman-benson: 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 Elena Laskavaia CLA 2008-01-15 17:54:01 EST 
There are at least 4 places in mi.core plugin that does some hardcoded 5 and sometimes 6 seconds wait for interrupt of the process. This should use default timeout, for example MI command timeout, which is 10 seconds by default, but user configurable.

In case of slow targets 5 seconds my be not enough for interrupt to propagate.
Comment 1 Elena Laskavaia CLA 2008-01-15 17:56:58 EST 
Created attachment 86992 [details]
4_0 patch

suggested patch that uses MI command timeout
Comment 2 Doug Schaefer CLA 2008-01-28 14:43:16 EST 
Patch applied. Thanks Elena.
Comment 3 Doug Schaefer CLA 2008-01-28 14:56:45 EST 
Patch applied to 4.0 and HEAD. Thanks Alena!
Comment 4 Doug Schaefer CLA 2008-06-03 15:07:32 EDT 
assigning
Comment 5 Doug Schaefer CLA 2008-06-03 15:07:58 EDT 
done