[Bug 389374] MemoryErrorReport constructor may close TCF channel on virtual addresses translations
Bugzilla Link | 389374 |
Status | NEW |
Importance | P3 normal |
Reported | Sep 12, 2012 03:02 EDT |
Modified | Sep 14, 2012 04:35 EDT |
Reporter | Frederic Leger |
Description
Using a simics agent based on TCF, I got a "No translation for virtual address" error :
2012-09-12 08:30:42,627 : ---> C 164 Memory get "tracker0.ctx710" 140736507740133 4 4 2
2012-09-12 08:30:42,632 : <--- R 164 None None "AAAAAA==" {"Code":1,"Time":1347431442627,"Format":"No translation for virtual address"} [{"addr":3314352101,"size":4,"stat":6,"msg":{"Code":1,"Time":1347431442627,"Format":"No translation for virtual address"}}]
I guess that either the "addr" in the error message is wrong, either simics is trying to do a virtual/physical memory mapping, still I get a 'channel closed' because of the
assert r.offs >= 0\
assert r.size >= 0
Lines in the MemoryErrorReport constructor.
I do not think that the assert is needed. Either add something in the error message, or do nothing ... The patch attached just ignores the problem.