#1728: Remove CACHE_SIZE etc. from configure -------------------------+-------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: new Priority: optional | Milestone: Component: Cactus | Version: development version Keywords: | -------------------------+-------------------------------------------------- Configure cannot reliably detect CACHE_SIZE since this may be different on build nodes and compute nodes. Thorn hwloc provides a reliable, portable run-time solution. CACHE_SIZE is not currently used either. I suggest to remove it from configure.
#1728: Remove CACHE_SIZE etc. from configure --------------------------+------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: new Priority: optional | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: --------------------------+-------------------------------------------------
Comment (by eschnett):
I also propose to remove the functionality in src/util/Cache.c; it is similarly unused, and has an interface that is much poorer than that of hwloc.
#1728: Remove CACHE_SIZE etc. from configure --------------------------+------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: review Priority: optional | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: --------------------------+------------------------------------------------- Changes (by eschnett):
* status: new => review
#1728: Remove CACHE_SIZE etc. from configure --------------------------+------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: review Priority: optional | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: --------------------------+-------------------------------------------------
Comment (by knarf):
Sounds good, especially because it is unused anyway.
#1728: Remove CACHE_SIZE etc. from configure --------------------------+------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: reviewed_ok Priority: optional | Milestone: Component: Cactus | Version: development version Resolution: | Keywords: --------------------------+------------------------------------------------- Changes (by rhaas):
* status: review => reviewed_ok
Comment:
Is there a patch for this to apply?
#1728: Remove CACHE_SIZE etc. from configure --------------------------+------------------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: closed Priority: optional | Milestone: Component: Cactus | Version: development version Resolution: fixed | Keywords: --------------------------+------------------------------------------------- Changes (by eschnett):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
I removed these features. (No, there was no patch ahead of time.)
trac@lists.einsteintoolkit.org
-
Einstein Toolkit