[gem5-dev] Change in gem5/gem5[develop]: util: Add dockerfile for GCN3 w/machine learning

2020-05-20 Thread Kyle Roarty (Gerrit) via gem5-dev
Kyle Roarty has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29192 )


Change subject: util: Add dockerfile for GCN3 w/machine learning
..

util: Add dockerfile for GCN3 w/machine learning

This dockerfile creates an image that installs the software stack needed
to run both machine learning and non-machine learning applications using
the GCN3 gpu model, while also applying patches to the software stack to
optimize machine learning applications, as well as APUs, which is the
current type of GPU in the GCN3 GPU model.

Change-Id: If36c2df1c00c895e27e9d741027fd10c17bf224e
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29192
Reviewed-by: Matt Sinclair 
Reviewed-by: Jason Lowe-Power 
Maintainer: Bobby R. Bruce 
Tested-by: kokoro 
---
A util/dockerfiles/gcn-gpu/Dockerfile
A util/dockerfiles/gcn-gpu/README.md
2 files changed, 159 insertions(+), 0 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  Matt Sinclair: Looks good to me, approved
  Bobby R. Bruce: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/util/dockerfiles/gcn-gpu/Dockerfile  
b/util/dockerfiles/gcn-gpu/Dockerfile

new file mode 100644
index 000..485a406
--- /dev/null
+++ b/util/dockerfiles/gcn-gpu/Dockerfile
@@ -0,0 +1,132 @@
+FROM ubuntu:16.04
+
+# Should be minimal needed packages
+RUN apt-get update && apt-get install -y --no-install-recommends \
+findutils \
+file \
+libunwind8 \
+libunwind-dev \
+pkg-config \
+build-essential \
+gcc-multilib \
+g++-multilib \
+git \
+ca-certificates \
+m4 \
+scons \
+zlib1g \
+zlib1g-dev \
+libprotobuf-dev \
+protobuf-compiler \
+libprotoc-dev \
+libgoogle-perftools-dev \
+python-dev \
+python \
+python-yaml \
+wget \
+libpci3 \
+libelf1 \
+libelf-dev \
+cmake \
+openssl \
+libssl-dev \
+libboost-filesystem-dev \
+libboost-system-dev \
+libboost-dev
+
+ARG gem5_dist=http://dist.gem5.org/dist/current
+
+# Install ROCm 1.6 binaries
+RUN wget -qO- ${gem5_dist}/apt_1.6.2.tar.bz2 \
+| tar -xjv \
+&& cd apt_1.6.2/pool/main/ \
+&& dpkg -i h/hsakmt-roct-dev/* \
+&& dpkg -i h/hsa-ext-rocr-dev/* \
+&& dpkg -i h/hsa-rocr-dev/* \
+&& dpkg -i r/rocm-utils/* \
+&& dpkg -i h/hcc/* \
+&& dpkg -i r/rocm-opencl/* \
+&& dpkg -i r/rocm-opencl-dev/*
+
+# Get ROCm libraries we need to compile from source (and ROCm-profiler)
+RUN git clone --single-branch https://github.com/ROCm-Developer-Tools/HIP/  
&& \
+git clone --single-branch  
https://github.com/ROCmSoftwarePlatform/hipBLAS/ && \
+git clone --single-branch  
https://github.com/ROCmSoftwarePlatform/rocBLAS/ && \
+git clone --single-branch  
https://github.com/ROCmSoftwarePlatform/MIOpenGEMM/ && \
+git clone --single-branch  
https://github.com/ROCmSoftwarePlatform/MIOpen/ && \
+git clone --single-branch  
https://github.com/RadeonOpenCompute/rocm-cmake/ && \
+git clone --single-branch  
https://github.com/rocmarchive/ROCm-Profiler.git

+
+# Apply patches to various repos
+RUN mkdir -p /patch && cd /patch && \
+wget ${gem5_dist}/rocm_patches/hipBLAS.patch && \
+wget ${gem5_dist}/rocm_patches/hip.patch && \
+wget ${gem5_dist}/rocm_patches/miopen.patch && \
+wget ${gem5_dist}/rocm_patches/rocBLAS.patch
+
+RUN git -C /HIP/ checkout 0e3d824e && git -C /HIP/ apply /patch/hip.patch  
&& \
+git -C /hipBLAS/ checkout ee57787e && git -C /hipBLAS/ apply  
/patch/hipBLAS.patch && \
+git -C /rocBLAS/ checkout cbff4b4e && git -C /rocBLAS/ apply  
/patch/rocBLAS.patch && \

+git -C /MIOpenGEMM/ checkout 9547fb9e && \
+git -C /MIOpen/ checkout a9949e30 && git -C /MIOpen/ apply  
/patch/miopen.patch

+
+ENV ROCM_PATH /opt/rocm
+ENV HCC_HOME ${ROCM_PATH}/hcc
+ENV HSA_PATH ${ROCM_PATH}/hsa
+ENV HIP_PATH ${ROCM_PATH}/hip
+ENV HIP_PLATFORM hcc
+ENV PATH  
${ROCM_PATH}/bin:${HCC_HOME}/bin:${HSA_PATH}/bin:${HIP_PATH}/bin:${PATH}

+ENV HCC_AMDGPU_TARGET gfx801
+
+# Create build dirs for machine learning ROCm installs
+RUN mkdir -p /HIP/build && \
+mkdir -p /rocBLAS/build && \
+mkdir -p /hipBLAS/build && \
+mkdir -p /rocm-cmake/build && \
+mkdir -p /MIOpenGEMM/build && \
+mkdir -p /MIOpen/build
+
+# Do the builds, empty build dir to trim image size
+WORKDIR /HIP/build
+RUN cmake .. && make -j$(nproc) && make install && rm -rf *
+
+WORKDIR /rocBLAS/build
+RUN CXX=/opt/rocm/bin/hcc cmake  
-DCMAKE_CXX_FLAGS="--amdgpu-target=gfx801" .. && \

+make -j$(nproc) && make install && rm -rf *
+
+WORKDIR /hipBLAS/build
+RUN CXX=/opt/rocm/bin/hcc cmake  
-DCMAKE_CXX_FLAGS="--amdgpu-target=gfx801" .. && \

+make -j$(nproc) && make install && rm -rf *
+
+WORKDIR /rocm-cmake/build
+RUN cmake .. && cmake --build . --target install && rm -rf *
+
+WORKDIR /MIOpenGEMM/build
+RUN cmake .. && make miopengemm && 

[gem5-dev] Re: Debugging application inside gem5 (m5term)

2020-05-20 Thread GAURAV JAIN via gem5-dev
Hi Ciro,
 runn
Thanks for the response. Yeah, even we couldn't make the gdbserver approach 
work.

For the non-gdbserver approach you described, we tried it but we are still not 
able to hit the breakpoints on the app (by running gdb on the host system).

I think I might have missed this earlier but using gdb inside m5term with the 
application is not an issue and works perfectly fine. It was that the libraries 
I was trying to step into didn't contain debug symbols on the guest OS, I 
thought connecting it with the host gdb and debugging the application on the 
host would be a better approach. (because on the host, I have seen that a few, 
if not all of the libraries, have been built with debug symbols)


Thanks,
Gaurav

From: ciro.santi...@gmail.com 
Sent: Tuesday, May 19, 2020 3:24 PM
To: gem5 Developer List 
Cc: Beckmann, Brad ; GAURAV JAIN 
Subject: Re: [gem5-dev] Debugging application inside gem5 (m5term)

For the gdbserver approach, I haven't found a guest to host networking
possibility in gem5 yet:
https://stackoverflow.com/questions/48941494/how-to-do-port-forwarding-from-guest-to-host-and-vice-versa-in-gem5
but maybe I missed it.

For a non-gdbserver approach, have you tried add-symbol-file as
mentioned at 
https://stackoverflow.com/questions/26271901/is-it-possible-to-use-gdb-and-qemu-to-debug-linux-user-space-programs-and-kernel/46636070#46636070
and 
https://stackoverflow.com/questions/20380204/how-to-load-multiple-symbol-files-in-gdb

BTW, have a look into lx-symbols to help with kernel modules in case
you are not using it yet:
https://cirosantilli.com/linux-kernel-module-cheat/#gdb-step-debug-kernel-module

BTW2 Dealing with userland apps perfectly from gem5 GDB stub
(including multiple userland programs with same address) asked at:
https://stackoverflow.com/questions/9561546/thread-aware-gdb-for-the-linux-kernel

On Tue, May 19, 2020 at 8:03 PM GAURAV JAIN via gem5-dev
 wrote:
>
> Hi All,
>
> We are trying to debug an application running inside gem5 full system.
>
> After running gem5, we are using 'gdb vmlinux' to attach to it so as to be 
> able to debug the kernel modules.
>
> One approach we tried was using the 'file' command to load the symbol table 
> for the application and continue. But then that approach would overwrite the 
> existing symbol table (for the kernel modules) with that of the application 
> which is not what we desire.
>
> The second approach we tried was running 'gdbserver' inside m5term and then 
> connect to it through a gdb instance running on the host.
>
> We did - 'gdbserver localhost:1234 ./app', which seemed to be successful in 
> launching a process and Listening on the port 1234.
>
> On the host, we did - 'gdb ./app' and from inside it, 'target remote :1234'.
> Doing this did not work and we got the generic error - 'remote:1234 
> connection timed out'
>
> It'd be great if someone could provide pointers on how can we debug an 
> application running in gem5 full system and if either of the two approaches 
> mentioned above would work.
>
> Thanks,
> Gaurav
> ___
> 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 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]: Merge branch 'develop' of https://gem5.googlesource.com/public/gem5 i...

2020-05-20 Thread Polydoros Petrakis (Gerrit) via gem5-dev
Polydoros Petrakis has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29255 )



Change subject: Merge branch 'develop' of  
https://gem5.googlesource.com/public/gem5 into develop

..

Merge branch 'develop' of https://gem5.googlesource.com/public/gem5 into  
develop


Change-Id: I49a0acde1af5c5dc7bc6bc7869bf9809d9d9c9a3
---
M src/mem/ruby/network/garnet2.0/GarnetNetwork.cc
2 files changed, 4 insertions(+), 0 deletions(-)



diff --git a/src/mem/ruby/network/garnet2.0/GarnetNetwork.cc  
b/src/mem/ruby/network/garnet2.0/GarnetNetwork.cc

index 83ec4a1..5f4b54f 100644
--- a/src/mem/ruby/network/garnet2.0/GarnetNetwork.cc
+++ b/src/mem/ruby/network/garnet2.0/GarnetNetwork.cc
@@ -423,6 +423,10 @@
 for (int i = 0; i < m_networklinks.size(); i++) {
 m_networklinks[i]->resetStats();
 }
+for (int i = 0; i < m_creditlinks.size(); i++) {
+m_creditlinks[i]->resetStats();
+}
+}
 }

 void

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29255
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: I49a0acde1af5c5dc7bc6bc7869bf9809d9d9c9a3
Gerrit-Change-Number: 29255
Gerrit-PatchSet: 1
Gerrit-Owner: Polydoros Petrakis 
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]: tests: add ruby_mem_test.py to the tests

2020-05-20 Thread Ciro Santilli (Gerrit) via gem5-dev
Ciro Santilli has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/26805 )


Change subject: tests: add ruby_mem_test.py to the tests
..

tests: add ruby_mem_test.py to the tests

This catches ruby functional memory errors we have observed, and ensures
that ruby_mem_test.py itself won't be broken.

The test duration is about 10 seconds, and it can be run as:

./main.py run --uid SuiteUID:tests/gem5/test_ruby_mem_test.py:test-ruby\
_mem_test-NULL-x86_64-opt

Change-Id: I39bc559aaea3ebb41217a96cd4e8dae46271ea1f
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/26805
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M tests/gem5/memory/test.py
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/tests/gem5/memory/test.py b/tests/gem5/memory/test.py
index 2a4eeb3..5b6294e 100644
--- a/tests/gem5/memory/test.py
+++ b/tests/gem5/memory/test.py
@@ -69,6 +69,8 @@
 null_tests = [
 ('garnet_synth_traffic', ['--sim-cycles', '500']),
 ('memcheck', ['--maxtick', '20', '--prefetchers']),
+('ruby_mem_test', ['--abs-max-tick', '2000',
+'--functional', '10']),
 ('ruby_random_test', ['--maxloads', '5000']),
 ('ruby_direct_test', ['--requests', '5']),
 ]

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/26805
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: I39bc559aaea3ebb41217a96cd4e8dae46271ea1f
Gerrit-Change-Number: 26805
Gerrit-PatchSet: 5
Gerrit-Owner: Ciro Santilli 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Jason Lowe-Power 
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]: python: Make DOT config generation optional

2020-05-20 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29232 )


Change subject: python: Make DOT config generation optional
..

python: Make DOT config generation optional

By default, DOT configs are always generated when pydot is present.
This change allows a user to pass an empty --dot-config='' to disable
generating the DOT configuration. This can be useful to save space, or
to reduce Gem5 startup time when running many small regression tests.

This brings the behavior in-line with providing an empty
--dump_config='' and/or --json_config='' which similarly disables
generation of those output files.

Change-Id: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29232
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/python/m5/simulate.py
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/python/m5/simulate.py b/src/python/m5/simulate.py
index 3317ae8..698dfbc 100644
--- a/src/python/m5/simulate.py
+++ b/src/python/m5/simulate.py
@@ -107,8 +107,9 @@
 except ImportError:
 pass

-do_dot(root, options.outdir, options.dot_config)
-do_ruby_dot(root, options.outdir, options.dot_config)
+if options.dot_config:
+do_dot(root, options.outdir, options.dot_config)
+do_ruby_dot(root, options.outdir, options.dot_config)

 # Initialize the global statistics
 stats.initSimStats()

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29232
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: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
Gerrit-Change-Number: 29232
Gerrit-PatchSet: 3
Gerrit-Owner: Giacomo Travaglini 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Daniel Carvalho 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Nikos Nikoleris 
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]: sim-se: implement the getcpu syscall

2020-05-20 Thread Ciro Santilli (Gerrit) via gem5-dev
Ciro Santilli has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/28590 )


Change subject: sim-se: implement the getcpu syscall
..

sim-se: implement the getcpu syscall

Change-Id: I63a1384646829b8cf68453c42aed6a7d12172787
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/28590
Reviewed-by: Giacomo Travaglini 
Maintainer: Giacomo Travaglini 
Tested-by: kokoro 
---
M src/arch/arm/linux/process.cc
M src/arch/x86/linux/process.cc
M src/sim/syscall_emul.cc
M src/sim/syscall_emul.hh
4 files changed, 32 insertions(+), 6 deletions(-)

Approvals:
  Giacomo Travaglini: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/arch/arm/linux/process.cc b/src/arch/arm/linux/process.cc
index b5b6553..55141ba 100644
--- a/src/arch/arm/linux/process.cc
+++ b/src/arch/arm/linux/process.cc
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2010-2013, 2015 ARM Limited
+ * Copyright (c) 2010-2013, 2015, 2020 ARM Limited
  * All rights reserved
  *
  * The license below extends only to copyright in the software and shall
@@ -476,7 +476,7 @@
 { base + 342, "tee" },
 { base + 343, "vmsplice" },
 { base + 344, "move_pages" },
-{ base + 345, "getcpu" },
+{ base + 345, "getcpu", getcpuFunc },
 { base + 346, "epoll_pwait" },
 { base + 347, "sys_kexec_load" },
 { base + 348, "sys_utimensat" },
@@ -681,7 +681,7 @@
 {  base + 165, "getrusage", getrusageFunc },
 {  base + 166, "umask" },
 {  base + 167, "prctl" },
-{  base + 168, "getcpu" },
+{  base + 168, "getcpu", getcpuFunc },
 {  base + 169, "gettimeofday", gettimeofdayFunc },
 {  base + 170, "settimeofday" },
 {  base + 171, "adjtimex" },
diff --git a/src/arch/x86/linux/process.cc b/src/arch/x86/linux/process.cc
index 6b50dbf..2c594e7 100644
--- a/src/arch/x86/linux/process.cc
+++ b/src/arch/x86/linux/process.cc
@@ -566,7 +566,7 @@
 { 306, "syncfs" },
 { 307, "sendmmsg" },
 { 308, "setns" },
-{ 309, "getcpu" },
+{ 309, "getcpu", getcpuFunc },
 { 310, "proess_vm_readv" },
 { 311, "proess_vm_writev" },
 { 312, "kcmp" },
@@ -914,7 +914,7 @@
 { 315, "tee" },
 { 316, "vmsplice" },
 { 317, "move_pages" },
-{ 318, "getcpu" },
+{ 318, "getcpu", getcpuFunc },
 { 319, "epoll_pwait" },
 { 320, "utimensat" },
 { 321, "signalfd" },
diff --git a/src/sim/syscall_emul.cc b/src/sim/syscall_emul.cc
index bffedfd..6d39823 100644
--- a/src/sim/syscall_emul.cc
+++ b/src/sim/syscall_emul.cc
@@ -1633,3 +1633,26 @@
 return (status == -1) ? -errno : status;
 }

+SyscallReturn
+getcpuFunc(SyscallDesc *desc, ThreadContext *tc,
+   Addr cpu_ptr, Addr node_ptr, Addr tcache_ptr)
+{
+bool error = false;
+
+// unsigned is the same size (4) on all Linux supported ISAs.
+if (cpu_ptr != 0) {
+TypedBufferArg result(cpu_ptr);
+*result = htog(tc->contextId(),
+tc->getSystemPtr()->getGuestByteOrder());
+error |= !result.copyOut(tc->getVirtProxy());
+}
+
+// Set a fixed NUMA node 0.
+if (node_ptr != 0) {
+TypedBufferArg result(node_ptr);
+*result = 0;
+error |= !result.copyOut(tc->getVirtProxy());
+}
+
+return error ? -EFAULT : 0;
+}
diff --git a/src/sim/syscall_emul.hh b/src/sim/syscall_emul.hh
index 4a37e99..55d30f3 100644
--- a/src/sim/syscall_emul.hh
+++ b/src/sim/syscall_emul.hh
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2012-2013, 2015, 2019 ARM Limited
+ * Copyright (c) 2012-2013, 2015, 2019-2020 ARM Limited
  * Copyright (c) 2015 Advanced Micro Devices, Inc.
  * All rights reserved
  *
@@ -354,6 +354,9 @@
  int tgt_fd, int level, int optname,
  Addr valPtr, socklen_t len);

+SyscallReturn getcpuFunc(SyscallDesc *desc, ThreadContext *tc,
+ Addr cpu_ptr, Addr node_ptr, Addr tcache_ptr);
+
 // Target getsockname() handler.
 SyscallReturn getsocknameFunc(SyscallDesc *desc, ThreadContext *tc,
   int tgt_fd, Addr addrPtr, Addr lenPtr);

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/28590
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: I63a1384646829b8cf68453c42aed6a7d12172787
Gerrit-Change-Number: 28590
Gerrit-PatchSet: 4
Gerrit-Owner: Ciro Santilli 
Gerrit-Reviewer: Brandon Potter 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Jason Lowe-Power 
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] Change in gem5/gem5[develop]: sim-se: ignore all scheduler related syscalls for arm

2020-05-20 Thread Ciro Santilli (Gerrit) via gem5-dev
Ciro Santilli has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/28591 )


Change subject: sim-se: ignore all scheduler related syscalls for arm
..

sim-se: ignore all scheduler related syscalls for arm

With the simplistic syscall emulation fork algorithm that we currently have
of running one thread per call, those calls simply cannot be reasonably
implemented.

However, content can often still work without them.

Change-Id: Iac88dfd055564c47b7a7b6898b7582cf4087f708
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/28591
Maintainer: Giacomo Travaglini 
Tested-by: kokoro 
Reviewed-by: Jason Lowe-Power 
---
M src/arch/arm/linux/process.cc
1 file changed, 18 insertions(+), 18 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  Giacomo Travaglini: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/arch/arm/linux/process.cc b/src/arch/arm/linux/process.cc
index 55141ba..4c679b3 100644
--- a/src/arch/arm/linux/process.cc
+++ b/src/arch/arm/linux/process.cc
@@ -290,14 +290,14 @@
 { base + 151, "munlock" },
 { base + 152, "mlockall" },
 { base + 153, "munlockall" },
-{ base + 154, "sched_setparam" },
-{ base + 155, "sched_getparam" },
-{ base + 156, "sched_setscheduler" },
-{ base + 157, "sched_getscheduler" },
-{ base + 158, "sched_yield" },
-{ base + 159, "sched_get_priority_max" },
-{ base + 160, "sched_get_priority_min" },
-{ base + 161, "sched_rr_get_interval" },
+{ base + 154, "sched_setparam", ignoreWarnOnceFunc },
+{ base + 155, "sched_getparam", ignoreWarnOnceFunc },
+{ base + 156, "sched_setscheduler", ignoreWarnOnceFunc },
+{ base + 157, "sched_getscheduler", ignoreWarnOnceFunc },
+{ base + 158, "sched_yield", ignoreWarnOnceFunc },
+{ base + 159, "sched_get_priority_max", ignoreWarnOnceFunc },
+{ base + 160, "sched_get_priority_min", ignoreWarnOnceFunc },
+{ base + 161, "sched_rr_get_interval", ignoreWarnOnceFunc },
 { base + 162, "nanosleep", ignoreWarnOnceFunc },
 { base + 163, "mremap", mremapFunc }, // ARM-specific
 { base + 164, "setresuid" },
@@ -375,7 +375,7 @@
 { base + 238, "tkill" },
 { base + 239, "sendfile64" },
 { base + 240, "futex", futexFunc },
-{ base + 241, "sched_setaffinity" },
+{ base + 241, "sched_setaffinity", ignoreWarnOnceFunc },
 { base + 242, "sched_getaffinity", ignoreFunc },
 { base + 243, "io_setup" },
 { base + 244, "io_destroy" },
@@ -631,16 +631,16 @@
 {  base + 115, "clock_nanosleep" },
 {  base + 116, "syslog" },
 {  base + 117, "ptrace" },
-{  base + 118, "sched_setparam" },
-{  base + 119, "sched_setscheduler" },
-{  base + 120, "sched_getscheduler" },
-{  base + 121, "sched_getparam" },
-{  base + 122, "sched_setaffinity" },
+{  base + 118, "sched_setparam", ignoreWarnOnceFunc },
+{  base + 119, "sched_setscheduler", ignoreWarnOnceFunc },
+{  base + 120, "sched_getscheduler", ignoreWarnOnceFunc },
+{  base + 121, "sched_getparam", ignoreWarnOnceFunc },
+{  base + 122, "sched_setaffinity", ignoreWarnOnceFunc },
 {  base + 123, "sched_getaffinity", ignoreFunc },
-{  base + 124, "sched_yield" },
-{  base + 125, "sched_get_priority_max" },
-{  base + 126, "sched_get_priority_min" },
-{  base + 127, "sched_rr_get_interval" },
+{  base + 124, "sched_yield", ignoreWarnOnceFunc },
+{  base + 125, "sched_get_priority_max", ignoreWarnOnceFunc },
+{  base + 126, "sched_get_priority_min", ignoreWarnOnceFunc },
+{  base + 127, "sched_rr_get_interval", ignoreWarnOnceFunc },
 {  base + 128, "restart_syscall" },
 {  base + 129, "kill", ignoreFunc },
 {  base + 130, "tkill" },

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/28591
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: Iac88dfd055564c47b7a7b6898b7582cf4087f708
Gerrit-Change-Number: 28591
Gerrit-PatchSet: 4
Gerrit-Owner: Ciro Santilli 
Gerrit-Reviewer: Brandon Potter 
Gerrit-Reviewer: Ciro Santilli 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Jason Lowe-Power 
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