Hi Richard - Yes you are right. It is a scope typo; Under if(! PERFECT_SUMMARY_FILTER), bool imperfect_result should be imperfect_result
-Haris
On Feb 11, 2008 1:21 PM, Richard Yoo < yoo@xxxxxxxxxxxxxx> wrote:
Hi,
I have a question regarding the SimicsHypervisor::existSummarySignatureConflict() function in
SimicsHypervisor.C.
if(isRead) { // check Summary Write signature bool perfect_result = xact_isol_mgr->isInSummaryWriteSetPerfectFilter(0, addr); bool imperfect_result = perfect_result;
if(! PERFECT_SUMMARY_FILTER) { bool imperfect_result = xact_isol_mgr->isInSummaryWriteSetFilter(0, addr); // no false negatives assert(!(perfect_result && !imperfect_result));
}
if (imperfect_result) { return CONFLICT_IS_READER | CONFLICT_WITH_SUMMARY_WRITE; } else { return 0; } }
Following the above code, imperfect_result variable would always be equal
to perfect_result variable; which means we only use perfect signature in conflict detection, regardless of whether PERFECT_SUMMARY_FILTER is set or not.
Is this intentional? Am I missing something?
Thanks in advance.
-Richard
_______________________________________________ Gems-users mailing list Gems-users@xxxxxxxxxxx https://lists.cs.wisc.edu/mailman/listinfo/gems-users
Use Google to search the GEMS Users mailing list by adding "site:https://lists.cs.wisc.edu/archive/gems-users/" to your search.
|