search

LLNL / STAT

STAT - the Stack Trace Analysis Tool
Other
63 stars 26 forks source link

Cray XT configure flag apply to XE/XK/XC? #5

Closed jeffhammond closed 8 years ago

jeffhammond commented 8 years ago

Is flag truly specific to Cray XT or is it the flag for Cray X[TEKC] supercomputers in general?

--with-cray-xt          Add the flags to run on a Cray XT system

From the source, it would appear to be the flag to use with the Cray ALPS scheduler, which was used on Cray X[TEKC] systems, but has been phased out in favor of SLURM, at least on NERSC Cray XC systems.

If it is a flag for Cray ALPS, it would be helpful to users to rename it to --with-cray-alps to ensure that it is:

Thanks,

Jeff

lee218llnl commented 8 years ago

@jeffhammond Good point, I will updated this in the next release. Thanks.

lee218llnl commented 8 years ago

I renamed this to --with-cray-alps in https://github.com/LLNL/STAT/commit/8b059441ad261206dd80cb8d26347cef54e74ce9.

dongahn commented 8 years ago

@lee218llnl: I think it would be ideal if can make this flag go away since LaunchMON should be able to detect the underlying RM at runtime on a given platform. What are the STAT features enabled when this config flag is given?

lee218llnl commented 8 years ago

@dongahn This commit (https://github.com/LLNL/STAT/commit/52d57e0f6fa2473dedb3c63bcbcdeba967701880) replaces the static RM setting with runtime detection. I have tested this on LLNL Linux clusters and BG/Q systems and there is no regression there. After MUCH toil, I was able to get this commit of STAT and all its deps built on LANL's trinitite Cray system (using alps) and got it to run OK there too.

dongahn commented 8 years ago

Nice work! Thanks.