----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3273/#review7956 -----------------------------------------------------------
Any thoughts on this patch? Otherwise we hope to check it in soon. Thanks - Brad Beckmann On Jan. 14, 2016, 9:59 p.m., John Kalamatianos wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3273/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2016, 9:59 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11377:be6977d381ed > --------------------------- > 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. This patch is dependent on the GPU baseline patch (3189). > Legal values of Wavefront size are 16, 32, 64 for now and enforced at > initialization time. > > > Diffs > ----- > > src/gpu-compute/compute_unit.cc PRE-CREATION > src/gpu-compute/dispatcher.hh PRE-CREATION > src/gpu-compute/dispatcher.cc PRE-CREATION > src/gpu-compute/global_memory_pipeline.cc PRE-CREATION > src/gpu-compute/gpu_dyn_inst.hh PRE-CREATION > src/gpu-compute/gpu_dyn_inst.cc PRE-CREATION > src/gpu-compute/local_memory_pipeline.cc PRE-CREATION > src/gpu-compute/misc.hh PRE-CREATION > src/gpu-compute/qstruct.hh PRE-CREATION > src/gpu-compute/vector_register_file.cc PRE-CREATION > src/gpu-compute/vector_register_state.hh PRE-CREATION > src/gpu-compute/vector_register_state.cc PRE-CREATION > src/gpu-compute/wavefront.hh PRE-CREATION > src/gpu-compute/wavefront.cc PRE-CREATION > configs/example/apu_se.py PRE-CREATION > src/gpu-compute/GPU.py PRE-CREATION > src/gpu-compute/arch/hsail/gen.py PRE-CREATION > src/gpu-compute/arch/hsail/insts/branch.hh PRE-CREATION > src/gpu-compute/arch/hsail/insts/main.cc PRE-CREATION > src/gpu-compute/arch/hsail/insts/mem.hh PRE-CREATION > src/gpu-compute/arch/hsail/insts/mem_impl.hh PRE-CREATION > src/gpu-compute/arch/hsail/insts/pseudo_inst.cc PRE-CREATION > src/gpu-compute/arch/hsail/operand.hh PRE-CREATION > src/gpu-compute/cl_driver.cc PRE-CREATION > src/gpu-compute/compute_unit.hh PRE-CREATION > > Diff: http://reviews.gem5.org/r/3273/diff/ > > > Testing > ------- > > > Thanks, > > John Kalamatianos > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
