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

Maybe an error in warps.py #2

Open
ConvolutedDog opened this issue Oct 30, 2023 · 1 comment
Open

Maybe an error in warps.py #2

ConvolutedDog opened this issue Oct 30, 2023 · 1 comment

Comments

@ConvolutedDog
Copy link

ConvolutedDog commented Oct 30, 2023

There maybe an error in https://github.com/NMSU-PEARL/PPT-GPU/blob/0351db7b62007aed0752900eee98b3a2e4f753b0/src/warps.py#L66.

When I simulate the cublas benchmark, I encountered this bug:

Traceback (most recent call last):
  File "ppt.py", line 330, in <module>
    main()
  File "ppt.py", line 307, in main
    simianEngine.run()
  File "/home/xxx/Github/PPT-GPU/simian.py", line 1730, in run
    service(event["data"], event["tx"], event["txId"]) #Receive TO BE CGECJED
  File "/home/xxx/Github/PPT-GPU/src/kernels.py", line 290, in kernel_call
    instructions_executed = self.warp_scheduler.step(current_warp_list, pred_out["active_cycles"]) 
  File "/home/xxx/Github/PPT-GPU/src/warp_scheduler.py", line 33, in step
    return self.step_LRR(warp_list, cycles)
  File "/home/xxx/Github/PPT-GPU/src/warp_scheduler.py", line 55, in step_LRR
    current_inst_executed = warp.step(cycles)
  File "/home/xxx/Github/PPT-GPU/src/warps.py", line 50, in step
    if self.process_inst(cycles):
  File "/home/xxx/Github/PPT-GPU/src/warps.py", line 69, in process_inst
    inst = self.tasklist[self.current_inst]
IndexError: list index out of range

Maybe this line should use >= like:

elif self.current_inst >= len(self.tasklist):

All traces and other files that needed to reproduce this bug have been uploaded in the next comment.

@ConvolutedDog
Copy link
Author

ConvolutedDog commented Oct 30, 2023

The sass_traces, memory_traces, app_config.py and QV100.py are in the attachment.

traces_and_others.zip

And the simulation command is:

mpiexec -n 10 python3 ppt.py --app ./traces_and_others --sass --config QV100 --granularity 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant