[gem5-dev] Change in gem5/gem5[develop]: mem-cache: Clear prefetch bit for CleanEvict before propagating to do...

2021-12-19 Thread Huang Jiasen (Gerrit) via gem5-dev
Huang Jiasen has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54364 )



Change subject: mem-cache: Clear prefetch bit for CleanEvict before  
propagating to downstream caches

..

mem-cache: Clear prefetch bit for CleanEvict before propagating to  
downstream caches


Change-Id: Iab56b9266eb64baf972b160774aca0823faea458
---
M src/mem/cache/base.cc
1 file changed, 14 insertions(+), 0 deletions(-)



diff --git a/src/mem/cache/base.cc b/src/mem/cache/base.cc
index dc21151..69c01cb 100644
--- a/src/mem/cache/base.cc
+++ b/src/mem/cache/base.cc
@@ -1310,6 +1310,11 @@
 // propagating further down the hierarchy. Returning true will
 // treat the CleanEvict like a satisfied write request and  
delete

 // it.
+
+if (blk->wasPrefetched()) {
+blk->clearPrefetched();
+}
+
 return true;
 }
 // We didn't find the block here, propagate the CleanEvict further

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/54364
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: Iab56b9266eb64baf972b160774aca0823faea458
Gerrit-Change-Number: 54364
Gerrit-PatchSet: 1
Gerrit-Owner: Huang Jiasen 
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

[gem5-dev] Change in gem5/gem5[release-staging-v21-2]: configs,tests: Update refs to deprecated 'x86-ubuntu-img'

2021-12-19 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54583 )



Change subject: configs,tests: Update refs to deprecated 'x86-ubuntu-img'
..

configs,tests: Update refs to deprecated 'x86-ubuntu-img'

This resource has been deprecated in favor of 'x86-ubuntu-18.04-img'. In
practise, these are the same resource (i.e., the same disk image), but
we deprecate 'x86-ubuntu-img' as 'x86-ubuntu-18.04-img' is more explicit
on what the disk image contains. Going forward we may want different
disk image resources with different Ubuntu versions, and we want it to
be clear what the user is retrieving.

Change-Id: I06b08e6b65cf431067ffda35ebd10d419068d073
---
M tests/gem5/configs/boot_kvm_fork_run.py
M tests/gem5/configs/boot_kvm_switch_exit.py
M configs/example/gem5_library/x86-ubuntu-run-with-kvm.py
M configs/example/gem5_library/x86-ubuntu-run.py
M tests/gem5/configs/x86_boot_exit_run.py
5 files changed, 21 insertions(+), 5 deletions(-)



diff --git a/configs/example/gem5_library/x86-ubuntu-run-with-kvm.py  
b/configs/example/gem5_library/x86-ubuntu-run-with-kvm.py

index fa84960..1aea258 100644
--- a/configs/example/gem5_library/x86-ubuntu-run-with-kvm.py
+++ b/configs/example/gem5_library/x86-ubuntu-run-with-kvm.py
@@ -121,7 +121,7 @@
 kernel=Resource("x86-linux-kernel-5.4.49"),
 # The x86 ubuntu image will be automatically downloaded to the if not
 # already present.
-disk_image=Resource("x86-ubuntu-img"),
+disk_image=Resource("x86-ubuntu-18.04-img"),
 readfile_contents=command,
 )

diff --git a/configs/example/gem5_library/x86-ubuntu-run.py  
b/configs/example/gem5_library/x86-ubuntu-run.py

index c6f6f83..2aee8c7 100644
--- a/configs/example/gem5_library/x86-ubuntu-run.py
+++ b/configs/example/gem5_library/x86-ubuntu-run.py
@@ -58,7 +58,7 @@
 # downloaded.
 board.set_kernel_disk_workload(
 kernel=Resource("x86-linux-kernel-5.4.49"),
-disk_image=Resource("x86-ubuntu-img"),
+disk_image=Resource("x86-ubuntu-18.04-img"),
 )

 simulator = Simulator(board=board)
diff --git a/tests/gem5/configs/boot_kvm_fork_run.py  
b/tests/gem5/configs/boot_kvm_fork_run.py

index 2cd180a..662ef23 100644
--- a/tests/gem5/configs/boot_kvm_fork_run.py
+++ b/tests/gem5/configs/boot_kvm_fork_run.py
@@ -203,7 +203,7 @@
 resource_directory=args.resource_directory,
 ),
 disk_image=Resource(
-"x86-ubuntu-img",
+"x86-ubuntu-18.04-img",
 resource_directory=args.resource_directory,
 ),
 readfile_contents=dedent(
diff --git a/tests/gem5/configs/boot_kvm_switch_exit.py  
b/tests/gem5/configs/boot_kvm_switch_exit.py

index 9f5f7ee..5cfee40 100644
--- a/tests/gem5/configs/boot_kvm_switch_exit.py
+++ b/tests/gem5/configs/boot_kvm_switch_exit.py
@@ -188,7 +188,7 @@
 resource_directory=args.resource_directory,
 ),
 disk_image=Resource(
-"x86-ubuntu-img",
+"x86-ubuntu-18.04-img",
 resource_directory=args.resource_directory,
 ),
 # The first exit signals to switch processors.
diff --git a/tests/gem5/configs/x86_boot_exit_run.py  
b/tests/gem5/configs/x86_boot_exit_run.py

index 5c8b025..93c9028 100644
--- a/tests/gem5/configs/x86_boot_exit_run.py
+++ b/tests/gem5/configs/x86_boot_exit_run.py
@@ -207,7 +207,7 @@
 resource_directory=args.resource_directory,
 ),
 disk_image=Resource(
-"x86-ubuntu-img",
+"x86-ubuntu-18.04-img",
 resource_directory=args.resource_directory,
 ),
 kernel_args=kernal_args,

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


Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v21-2
Gerrit-Change-Id: I06b08e6b65cf431067ffda35ebd10d419068d073
Gerrit-Change-Number: 54583
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce 
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

[gem5-dev] Change in gem5/gem5[release-staging-v21-2]: scons: Remove the '-Wno-c99-designator' Cxx clang flag

2021-12-19 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54543 )



Change subject: scons: Remove the '-Wno-c99-designator' Cxx clang flag
..

scons: Remove the '-Wno-c99-designator' Cxx clang flag

The following warning was being generated when using clang-9:

```
warning: unknown warning option '-Wno-c99-designator'; did you  
mean '-Wno-gnu-designator'? [-Wunknown-warning-option]

```

The simplest solution, for the gem5 21-2 release was to remove this
flag.

Change-Id: I87b460388a1af6e6b72d36caf32b2fb949fdc8ca
---
M SConstruct
1 file changed, 18 insertions(+), 1 deletion(-)



diff --git a/SConstruct b/SConstruct
index ceeb1ba..3e7a593 100755
--- a/SConstruct
+++ b/SConstruct
@@ -403,7 +403,6 @@

 # clang has a few additional warnings that we disable.
 with gem5_scons.Configure(main) as conf:
-conf.CheckCxxFlag('-Wno-c99-designator')
 conf.CheckCxxFlag('-Wno-defaulted-function-deleted')

 main.Append(TCMALLOC_CCFLAGS=['-fno-builtin'])

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


Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v21-2
Gerrit-Change-Id: I87b460388a1af6e6b72d36caf32b2fb949fdc8ca
Gerrit-Change-Number: 54543
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce 
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

[gem5-dev] Change in gem5/gem5[develop]: base: Finish deprecating SatCounter

2021-12-19 Thread Daniel Carvalho (Gerrit) via gem5-dev
Daniel Carvalho has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/54523 )



Change subject: base: Finish deprecating SatCounter
..

base: Finish deprecating SatCounter

SatCounter has been marked as deprecated for at least 2 versions,
so it can be removed.

Change-Id: Iffb75822cc0d09d8b7d9b86828b26198865ce407
Signed-off-by: Daniel R. Carvalho 
---
M src/base/sat_counter.hh
1 file changed, 13 insertions(+), 3 deletions(-)



diff --git a/src/base/sat_counter.hh b/src/base/sat_counter.hh
index 6644b05..a607c4c 100644
--- a/src/base/sat_counter.hh
+++ b/src/base/sat_counter.hh
@@ -340,9 +340,6 @@
 typedef GenericSatCounter SatCounter64;
 /** @} */

-[[deprecated("Use SatCounter8 (or variants) instead")]]
-typedef SatCounter8 SatCounter;
-
 } // namespace gem5

 #endif // __BASE_SAT_COUNTER_HH__

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/54523
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: Iffb75822cc0d09d8b7d9b86828b26198865ce407
Gerrit-Change-Number: 54523
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Carvalho 
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