[gem5-dev] Change in gem5/gem5[develop]: mem: Default the SE translating port proxy alloc method to NextPage.

2020-06-09 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29399 )


Change subject: mem: Default the SE translating port proxy alloc method to  
NextPage.

..

mem: Default the SE translating port proxy alloc method to NextPage.

This is what's used in 99% of cases, so it makes sense to make it the
default.

Change-Id: I51535b3387d1c1a0d1d89e77cfca10363388b472
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29399
Reviewed-by: Matthew Poremba 
Maintainer: Gabe Black 
Tested-by: kokoro 
---
M src/mem/se_translating_port_proxy.hh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Matthew Poremba: Looks good to me, approved
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/mem/se_translating_port_proxy.hh  
b/src/mem/se_translating_port_proxy.hh

index 0fe3212..41f92a6 100644
--- a/src/mem/se_translating_port_proxy.hh
+++ b/src/mem/se_translating_port_proxy.hh
@@ -60,7 +60,7 @@
 bool fixupAddr(Addr addr, BaseTLB::Mode mode) const override;

   public:
-SETranslatingPortProxy(ThreadContext *tc, AllocType alloc,
+SETranslatingPortProxy(ThreadContext *tc, AllocType alloc=NextPage,
Request::Flags _flags=0);
 };


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29399
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: I51535b3387d1c1a0d1d89e77cfca10363388b472
Gerrit-Change-Number: 29399
Gerrit-PatchSet: 2
Gerrit-Owner: Gabe Black 
Gerrit-Reviewer: Bradford Beckmann 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Matthew Poremba 
Gerrit-Reviewer: kokoro 
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]: mem: Default the SE translating port proxy alloc method to NextPage.

2020-05-25 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29399 )



Change subject: mem: Default the SE translating port proxy alloc method to  
NextPage.

..

mem: Default the SE translating port proxy alloc method to NextPage.

This is what's used in 99% of cases, so it makes sense to make it the
default.

Change-Id: I51535b3387d1c1a0d1d89e77cfca10363388b472
---
M src/mem/se_translating_port_proxy.hh
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/src/mem/se_translating_port_proxy.hh  
b/src/mem/se_translating_port_proxy.hh

index 0fe3212..41f92a6 100644
--- a/src/mem/se_translating_port_proxy.hh
+++ b/src/mem/se_translating_port_proxy.hh
@@ -60,7 +60,7 @@
 bool fixupAddr(Addr addr, BaseTLB::Mode mode) const override;

   public:
-SETranslatingPortProxy(ThreadContext *tc, AllocType alloc,
+SETranslatingPortProxy(ThreadContext *tc, AllocType alloc=NextPage,
Request::Flags _flags=0);
 };


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29399
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: I51535b3387d1c1a0d1d89e77cfca10363388b472
Gerrit-Change-Number: 29399
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black 
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