[gem5-dev] Re: Upstreaming power-gem5
Hi Boris, Gabe, I've just uploaded patchset 4 with these changes: [1] switch back to the use of INTREG_* for the special-purpose registers https://gem5-review.googlesource.com/c/public/gem5/+/40882/ [2] rebase on top of the latest "develop" branch [3] use of arch/power/regs/*.h as a result of the rebase Please have a look. As for [1], as promised, I will revisit it once I have a clear view of how all the SPRs in question are read or modified by different instructions. - Sandipan ___ 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]: tests, arch-power: Add 64-bit hello binaries
Boris Shingarov has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40951 ) Change subject: tests, arch-power: Add 64-bit hello binaries .. tests, arch-power: Add 64-bit hello binaries This adds 64-bit statically linked big and little endian binaries for the hello test program. It should be noted that all possible combinations of ABI version and endianness are possible for 64-bit binaries. However, standard toolchains always use ELF ABI v1 for big endian and ELF ABI v2 for little endian binaries. Change-Id: I2dca7eaa2b04a7b68b117ada799d4c3bb69368be Signed-off-by: Sandipan Das Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40951 Reviewed-by: Boris Shingarov Reviewed-by: Bobby R. Bruce Tested-by: kokoro Maintainer: Bobby R. Bruce --- A tests/test-progs/hello/bin/power/linux/hello64be A tests/test-progs/hello/bin/power/linux/hello64le 2 files changed, 0 insertions(+), 0 deletions(-) Approvals: Boris Shingarov: Looks good to me, approved Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved kokoro: Regressions pass diff --git a/tests/test-progs/hello/bin/power/linux/hello64be b/tests/test-progs/hello/bin/power/linux/hello64be new file mode 100644 index 000..53604fb --- /dev/null +++ b/tests/test-progs/hello/bin/power/linux/hello64be Binary files differ diff --git a/tests/test-progs/hello/bin/power/linux/hello64le b/tests/test-progs/hello/bin/power/linux/hello64le new file mode 100644 index 000..b442d9a --- /dev/null +++ b/tests/test-progs/hello/bin/power/linux/hello64le Binary files differ 4 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/40951 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: I2dca7eaa2b04a7b68b117ada799d4c3bb69368be Gerrit-Change-Number: 40951 Gerrit-PatchSet: 6 Gerrit-Owner: Sandipan Das Gerrit-Reviewer: Bobby R. Bruce Gerrit-Reviewer: Boris Shingarov 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]: tests, arch-power: Move 32-bit hello binary
Boris Shingarov has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/40950 ) Change subject: tests, arch-power: Move 32-bit hello binary .. tests, arch-power: Move 32-bit hello binary This moves the 32-bit hello binary for Power under the linux subdirectory like it was originally before being removed and reintroduced. Change-Id: I5f3da38f9abdda90b31755ce7e7c955838cc7289 Signed-off-by: Sandipan Das Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40950 Reviewed-by: Boris Shingarov Reviewed-by: Bobby R. Bruce Tested-by: kokoro Maintainer: Bobby R. Bruce --- R tests/test-progs/hello/bin/power/linux/hello32 1 file changed, 0 insertions(+), 0 deletions(-) Approvals: Boris Shingarov: Looks good to me, approved Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved kokoro: Regressions pass diff --git a/tests/test-progs/hello/bin/power/hello32 b/tests/test-progs/hello/bin/power/linux/hello32 similarity index 100% rename from tests/test-progs/hello/bin/power/hello32 rename to tests/test-progs/hello/bin/power/linux/hello32 Binary files differ -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/40950 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: I5f3da38f9abdda90b31755ce7e7c955838cc7289 Gerrit-Change-Number: 40950 Gerrit-PatchSet: 6 Gerrit-Owner: Sandipan Das Gerrit-Reviewer: Bobby R. Bruce Gerrit-Reviewer: Boris Shingarov 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]: tests, arch-power: Add support for building hello
Boris Shingarov has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/40949 )
Change subject: tests, arch-power: Add support for building hello
..
tests, arch-power: Add support for building hello
Commit a440108cc ("tests: Add Makefiles for hello")
introduced Makefiles for building the hello test binary
for ARM and x86 using dockcross. Since dockcross also
provides an image with a 64-bit little endian toolchain
for Power, this adds a Makefile for building the hello
binary.
As of this moment, 64-bit little endian (ppc64le) is the
prevalent variant supported by most distributions. Hence,
we are currently limited to only building the binary for
this variant.
Change-Id: Ic20322ca33c69634d9f17d30b29e522cc35742fb
Signed-off-by: Sandipan Das
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/40949
Tested-by: kokoro
Reviewed-by: Bobby R. Bruce
Maintainer: Bobby R. Bruce
---
A tests/test-progs/hello/src/Makefile.power
1 file changed, 11 insertions(+), 0 deletions(-)
Approvals:
Bobby R. Bruce: Looks good to me, approved; Looks good to me, approved
kokoro: Regressions pass
diff --git a/tests/test-progs/hello/src/Makefile.power
b/tests/test-progs/hello/src/Makefile.power
new file mode 100644
index 000..6fb280a
--- /dev/null
+++ b/tests/test-progs/hello/src/Makefile.power
@@ -0,0 +1,11 @@
+all: hello64le-static
+
+hello64le-static: hello.c dockcross-ppc64le
+ ./dockcross-ppc64le bash -c '$$CC hello.c -o hello64le-static -static'
+
+dockcross-ppc64le:
+ docker run --rm dockcross/linux-ppc64le > ./dockcross-ppc64le
+ chmod +x ./dockcross-ppc64le
+
+clean:
+ rm -f dockcross* hello64le-static
2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the
submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/40949
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: Ic20322ca33c69634d9f17d30b29e522cc35742fb
Gerrit-Change-Number: 40949
Gerrit-PatchSet: 6
Gerrit-Owner: Sandipan Das
Gerrit-Reviewer: Bobby R. Bruce
Gerrit-Reviewer: Boris Shingarov
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] mostly unreviewed series
Hi folks. This series about refactoring x86 microop implementations: https://gem5-review.googlesource.com/c/public/gem5/+/42975 Has been mostly unreviewed for about 2 months. For the changes with no review feedback (most of them), if there is no feedback by Friday I'll start self approving them. Gabe ___ 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
