[gem5-dev] Change in gem5/gem5[develop]: gpu-compute: Set scratch_base, lds_base for gfx902

2022-02-17 Thread Matt Sinclair (Gerrit) via gem5-dev
Matt Sinclair has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54663 )


Change subject: gpu-compute: Set scratch_base, lds_base for gfx902
..

gpu-compute: Set scratch_base, lds_base for gfx902

When updating how scratch_base and lds_base were set, gfx902 was left out.
This adds in gfx902 to the case statement, allowing the apertures to be set
and for simulations using gfx902 to not error out

Change-Id: I0e1adbdf63f7c129186fb835e30adac9cd4b72d0
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/54663
Reviewed-by: Matt Sinclair 
Maintainer: Matt Sinclair 
Reviewed-by: Matthew Poremba 
Maintainer: Matthew Poremba 
Tested-by: kokoro 
---
M src/gpu-compute/gpu_compute_driver.cc
1 file changed, 21 insertions(+), 0 deletions(-)

Approvals:
  Matthew Poremba: Looks good to me, approved; Looks good to me, approved
  Matt Sinclair: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/src/gpu-compute/gpu_compute_driver.cc  
b/src/gpu-compute/gpu_compute_driver.cc

index e908f4e..d98f4c6 100644
--- a/src/gpu-compute/gpu_compute_driver.cc
+++ b/src/gpu-compute/gpu_compute_driver.cc
@@ -331,6 +331,7 @@
 ldsApeBase(i + 1);
 break;
   case GfxVersion::gfx900:
+  case GfxVersion::gfx902:
 args->process_apertures[i].scratch_base =
 scratchApeBaseV9();
 args->process_apertures[i].lds_base =
@@ -631,6 +632,7 @@
 ape_args->lds_base = ldsApeBase(i + 1);
 break;
   case GfxVersion::gfx900:
+  case GfxVersion::gfx902:
 ape_args->scratch_base = scratchApeBaseV9();
 ape_args->lds_base = ldsApeBaseV9();
 break;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/54663
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I0e1adbdf63f7c129186fb835e30adac9cd4b72d0
Gerrit-Change-Number: 54663
Gerrit-PatchSet: 2
Gerrit-Owner: Kyle Roarty 
Gerrit-Reviewer: Matt Sinclair 
Gerrit-Reviewer: Matthew Poremba 
Gerrit-Reviewer: kokoro 
Gerrit-CC: Bobby Bruce 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: gpu-compute: Set scratch_base, lds_base for gfx902

2021-12-21 Thread Kyle Roarty (Gerrit) via gem5-dev
Kyle Roarty has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54663 )



Change subject: gpu-compute: Set scratch_base, lds_base for gfx902
..

gpu-compute: Set scratch_base, lds_base for gfx902

When updating how scratch_base and lds_base were set, gfx902 was left out.
This adds in gfx902 to the case statement, allowing the apertures to be set
and for simulations using gfx902 to not error out

Change-Id: I0e1adbdf63f7c129186fb835e30adac9cd4b72d0
---
M src/gpu-compute/gpu_compute_driver.cc
1 file changed, 15 insertions(+), 0 deletions(-)



diff --git a/src/gpu-compute/gpu_compute_driver.cc  
b/src/gpu-compute/gpu_compute_driver.cc

index e908f4e..d98f4c6 100644
--- a/src/gpu-compute/gpu_compute_driver.cc
+++ b/src/gpu-compute/gpu_compute_driver.cc
@@ -331,6 +331,7 @@
 ldsApeBase(i + 1);
 break;
   case GfxVersion::gfx900:
+  case GfxVersion::gfx902:
 args->process_apertures[i].scratch_base =
 scratchApeBaseV9();
 args->process_apertures[i].lds_base =
@@ -631,6 +632,7 @@
 ape_args->lds_base = ldsApeBase(i + 1);
 break;
   case GfxVersion::gfx900:
+  case GfxVersion::gfx902:
 ape_args->scratch_base = scratchApeBaseV9();
 ape_args->lds_base = ldsApeBaseV9();
 break;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/54663
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I0e1adbdf63f7c129186fb835e30adac9cd4b72d0
Gerrit-Change-Number: 54663
Gerrit-PatchSet: 1
Gerrit-Owner: Kyle Roarty 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s