Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error running latter #9

Open
zhqingg opened this issue Mar 29, 2019 · 1 comment
Open

Error running latter #9

zhqingg opened this issue Mar 29, 2019 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@zhqingg
Copy link

zhqingg commented Mar 29, 2019

Hi @johnjmar
Can you let me know if you've run into error with Memory/Latter
Build the binaries of stream and lat_mem_rd as the instruction. When executed latter.sh, the process stuck in this line “lat_mem_rd 5i3000 128 0 0 72”.
analysis the usage of lat_men_rd

Usage: ./lat_mem_rd [-P <parallelism>] [-W <warmup>] [-N <repetitions>] [-t] len [stride...]

So the _ARGS for lat_men_rd is wrong I think
So we should Modify some part

@@ -96,7 +96,7 @@

     # Store the current prefetch and SMT settings
     CURR_PREFETCH=`ppc64_cpu --dscr | sed 's/DSCR is //'`
-    CURR_SMT=`ppc64_cpu --smt | sed "s/SMT=//" | xargs`
+    CURR_SMT=`ppc64_cpu --smt | awk -F "is|=" '{print $2}'`

     for _PREFETCH in ${LTPREFETCH_SET}
     do
@@ -120,7 +120,7 @@

             for _CHIP in ${LTCHIP_SET}
             do
-                _ARGS="${LTITERS}i${LT_BUFSIZE} ${LT_STRIDESIZE} ${_CHIP} ${LTCHIP_SET}"
+                _ARGS="-N ${LTITERS} ${LT_BUFSIZE} ${LT_STRIDESIZE}"
                 _OF1="${LTNAME_LAT}.${_CHIP}.${LTNAME_OUT}"

                 if [[ ${LTRUN_PMCOUNT} == "TRUE" ]]
@@ -148,7 +148,7 @@

     for _CHIP in ${LTCHIP_SET}
     do
-        _ARGS="${LTPATH}/${LTCMD} 60 128 ${_CHIP}"
+        _ARGS="${LTPATH}/${LTCMD} 60 128"
         _OF3="${_CHIP}.cache.out"
         Log "${_ARGS}"
         ${_ARGS} >> ${_OF3} 2>&1
@@ -615,3 +615,4 @@
     rm -r ${LTDATADIR}
 fi
 exit 0
+

And the usage of lat_mem_rd do not support to choose chip.maybe,the source code need to modify again.

@johnjmar johnjmar self-assigned this Mar 29, 2019
@johnjmar johnjmar added the bug Something isn't working label Mar 29, 2019
@johnjmar
Copy link
Member

@zhqingg you're correct. This is indeed a bug.
For the time being would you please replace function LTlat_mem_rd in the original script with the one below in the included file?
latter.tmp.txt

Also, you are correct in that the the ARGs should be changed to lat_mem_rd ${LT_BUFSIZE} ${LT_STRIDESIZE}

FYI, the owner of the benchmark in no longer in the company and the script may require more work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants