Project

General

Profile

Cleanup #1422

Cleanup dangling issues from 64bit merge

Added by Eric Mikida over 2 years ago. Updated over 2 years ago.

Status:
In Progress
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
02/15/2017
Due date:
% Done:

0%


Description

CkLocMgr still has some unnecessary or erroneous bits of code that were left over from the 64bit merge. At a quick glance:

- CmiUInt8 is still only used for chare arrays and isn't well documented

-The CkLocRec hash table is poorly hashed after switching the key from CkArrayIndex to CmiUInt8

-Some fault tolerance code still seems to be using CkArrayIndex where it shouldnt. See createLocal() in cklocation.C

-Since all records are now local, flushLocalRecs() and reclaimRemote(...) in cklocation.C may be no longer needed at all.


Related issues

Related to Charm++ - Bug #1244: mlogft and causalft builds do not compile New 10/04/2016
Related to Charm++ - Feature #178: objid_t: fault tolerance infrastructure should use objid_t New 04/09/2013
Related to Charm++ - Feature #179: objid_t: adapt callbacks to use objid_t In Progress 03/08/2013
Related to Charm++ - Cleanup #165: object location services: separate location caching from msg delivery and buffering In Progress 04/05/2013
Related to Charm++ - Bug #159: Some CkCallback types are not valid across checkpoint/restart New 04/02/2013
Related to Charm++ - Feature #177: objid_t: load balancing infrastructure should use objid_t In Progress 04/09/2013

History

#1 Updated by Eric Mikida over 2 years ago

  • Status changed from New to In Progress

https://charm.cs.illinois.edu/gerrit/#/c/2250/ is being implemented to address the poor hashing issue for CmiUInt8.

#2 Updated by Eric Mikida over 2 years ago

inform(CmiUInt8, CkArrayIdx, int) and inform(CmiUInt8, int) do completely different things and it is undocumented as to why even though they would seem to have the same purpose.

#3 Updated by Eric Mikida over 2 years ago

  • Related to Cleanup #165: object location services: separate location caching from msg delivery and buffering added

#4 Updated by Eric Mikida 4 months ago

  • Related to Bug #159: Some CkCallback types are not valid across checkpoint/restart added

#5 Updated by Eric Mikida 4 months ago

  • Related to Feature #177: objid_t: load balancing infrastructure should use objid_t added

Also available in: Atom PDF