Review Board 2.0.15


[mq]: jkalamat/wfSizeParam.patch

Review Request #3273 - Created Jan. 13, 2016 and updated

Information
John Kalamatianos
gem5
default
Reviewers
Default

Changeset 11342:3a8eb18b522f
---------------------------
gpu: parametrize Wavefront size
Eliminate VSZ constant that defined the Wavefront size in numbers of work
items. Replaced it with a parameter in the GPU.py configuration script.
Changed all data structures dependent on the Wavefront size to be
dynamically sized.
Legal values of Wavefront size are 16, 32, 64 for now and checked at
initialization time.

Updated the patch so that it builds with latest version of Gem5.


   
Review request changed
Updated (Feb. 16, 2016, 7:56 a.m.)

Change Summary:

Updated patch so that it applies and builds cleanly with latest Gem5

Summary:

-gpu: parametrize Wavefront size
+[mq]: jkalamat/wfSizeParam.patch

Description:

~  

Changeset 11377:be6977d381ed

  ~

Changeset 11342:3a8eb18b522f

    ---------------------------
~   gpu: parametrize Wavefront size

~  
  ~ gpu: parametrize Wavefront size
  ~ Eliminate VSZ constant that defined the Wavefront size in numbers of work
-  

Eliminate VSZ constant that defined the Wavefront size in numbers of work

    items. Replaced it with a parameter in the GPU.py configuration script.
    Changed all data structures dependent on the Wavefront size to be
~   dynamically sized. This patch is dependent on the GPU baseline patch (3189).
~   Legal values of Wavefront size are 16, 32, 64 for now and enforced at
  ~ dynamically sized.
  ~ Legal values of Wavefront size are 16, 32, 64 for now and checked at
    initialization time.

  +
  +

Updated the patch so that it builds with latest version of Gem5.

Diff:

Revision 2 (+258 -192)

Show changes

Ship it!
Posted (Feb. 16, 2016, 2:54 p.m.)

John, I've been using this internally and have even made some modifications to my own branch using the same API calls and it all works just fine.