Bug 250323 - [memory][multi-process] Each memory context needs a different MemoryRetrieval
Summary: [memory][multi-process] Each memory context needs a different MemoryRetrieval
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf (show other bugs)
Version: 0 DD 1.1   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 8.3.0   Edit
Assignee: Alvaro Sanchez-Leon CLA
QA Contact: Pawel Piech CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-10-09 14:24 EDT by Marc Khouzam CLA
Modified: 2014-01-18 13:00 EST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Khouzam CLA 2008-10-09 14:24:47 EDT
Currently, we only use a single memory retrieval.  This worked because we only had a single memory context.  With multi-process, we will have multiple memory context and therefore we need a way to support multiple memory retrievals, one per memory context.
Comment 1 Alvaro Sanchez-Leon CLA 2013-12-06 13:14:21 EST
The following change proposes the implementation to use a memory retrieval per memory context.

https://git.eclipse.org/r/#/c/19453/
Comment 2 Alvaro Sanchez-Leon CLA 2013-12-06 13:21:32 EST
The proposed change takes advantage of the current structure used in the Memory Browser's view and Memory View.
  So this implementation allows both views to display information related to a memory context i.e. a Process for Linux. 

This can be tested using the existing capabilities for multi-process support, where multiple programs could be debugged under the same gdb session. 
  Switching between these programs under the same gdb will reflect the memory information corresponding to each memory context (i.e. per process on Linux).
Comment 3 Marc Khouzam CLA 2013-12-11 14:28:49 EST
The patch is approved, I'll just wait to get Mikhail's feedback on the factory use and on having this supported directly in DSF like was done.
Comment 4 Marc Khouzam CLA 2014-01-15 09:12:03 EST
Commmitted to master.
Thanks Alvaro!
Comment 5 Alvaro Sanchez-Leon CLA 2014-01-18 13:00:50 EST
Thanks Marc and Mikhael for your thorough review and valuable comments !