Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-03 Thread Bjoern A. Zeeb
On 2 Feb 2017, at 16:31, Bjoern A. Zeeb wrote: Hi, OK, I updated the diff. Can everyone please check if the diff from http://reviews.gem5.org/r/3803/ works for you now? /bz ___ gem5-dev mailing list gem5-dev@gem5.org

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-02 Thread Bjoern A. Zeeb
On 1 Feb 2017, at 21:41, Matthias Jung wrote: Moin, on my MAC I run into the same issue: http://qa.gem5.org//1905/compiling-problem-gem5-mac-os-10-11-6-scons-build-arm-gem5-opt When I use your patch I run into some other errors: In file included from build/ARM/arch/arm/linux/process.cc:56:

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-02 Thread Beckmann, Brad
--Original Message- From: gem5-dev [mailto:gem5-dev-boun...@gem5.org] On Behalf Of Andreas Hansson Sent: Thursday, February 2, 2017 1:35 AM To: gem5 Developer List <gem5-dev@gem5.org>; Potter, Brandon <brandon.pot...@amd.com> Subject: Re: [gem5-dev] Compilation error for gem5 after

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-02 Thread Andreas Hansson
ev-boun...@gem5.org] On Behalf Of Jason >Lowe-Power >Sent: Thursday, January 26, 2017 3:55 PM >To: gem5 Developer List <gem5-dev@gem5.org>; Potter, Brandon ><brandon.pot...@amd.com> >Subject: Re: [gem5-dev] Compilation error for gem5 after statfs change > >Hi Brandon

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-02-01 Thread Matthias Jung
Hi Bjoern, on my MAC I run into the same issue: http://qa.gem5.org//1905/compiling-problem-gem5-mac-os-10-11-6-scons-build-arm-gem5-opt When I use your patch I run into some other errors: In file included from build/ARM/arch/arm/linux/process.cc:56: build/ARM/sim/syscall_emul.hh:545:41: error:

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-28 Thread Beckmann, Brad
-dev [mailto:gem5-dev-boun...@gem5.org] On Behalf Of Jason Lowe-Power Sent: Thursday, January 26, 2017 3:55 PM To: gem5 Developer List <gem5-dev@gem5.org>; Potter, Brandon <brandon.pot...@amd.com> Subject: Re: [gem5-dev] Compilation error for gem5 after statfs change Hi Brandon,

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-26 Thread Jason Lowe-Power
Hi Brandon, BTW, here's a Travis-CI config file that will run tests on GCC and Clang on both Mac and Linux hosts. We can add many more parameters for different versions of compilers, etc., but this is something that should allow you to test changes on more than just your current platform. If you

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-26 Thread Andreas Hansson
gem5-dev [mailto:gem5-dev-boun...@gem5.org] On Behalf Of Andreas >Hansson >Sent: Thursday, January 26, 2017 2:18 AM >To: gem5 Developer List <gem5-dev@gem5.org> >Subject: Re: [gem5-dev] Compilation error for gem5 after statfs change > >Hi all, > >While I agree

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-26 Thread Bjoern A. Zeeb
, 2017 2:18 AM To: gem5 Developer List <gem5-dev@gem5.org> Subject: Re: [gem5-dev] Compilation error for gem5 after statfs change Hi all, While I agree about the end goal (test every commit on every platform) I would suggest we go about this one step at a time. The likelihood of breakage is onl

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-26 Thread Potter, Brandon
To: gem5 Developer List <gem5-dev@gem5.org> Subject: Re: [gem5-dev] Compilation error for gem5 after statfs change Hi all, While I agree about the end goal (test every commit on every platform) I would suggest we go about this one step at a time. The likelihood of breakage is only high for an

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-26 Thread Andreas Hansson
Hi all, While I agree about the end goal (test every commit on every platform) I would suggest we go about this one step at a time. The likelihood of breakage is only high for anything related to syscall emulation, so it really only affects a small part of the developer community. Agreed? With

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-25 Thread Jason Lowe-Power
Yeah, it's a major problem that we say that we support macOS and others when we allow commits that break builds on these other OSes. If we are going to say that we have support for OSes other than Linux, we need to at least verify gem5 builds on these OSes, preferably before accepting a commit.

Re: [gem5-dev] Compilation error for gem5 after statfs change

2017-01-24 Thread Bjoern A. Zeeb
On 24 Jan 2017, at 22:08, Jason Lowe-Power wrote: Hi Brandon, I think this is a "real" bug: http://qa.gem5.org//1905/compiling-problem-gem5-mac-os-10-11-6-scons-build-arm-gem5-opt. I think there are a few more places that need an #ifdef NO_STATFS. Could you look into it and post a patch if

[gem5-dev] Compilation error for gem5 after statfs change

2017-01-24 Thread Jason Lowe-Power
Hi Brandon, I think this is a "real" bug: http://qa.gem5.org//1905/compiling-problem-gem5-mac-os-10-11-6-scons-build-arm-gem5-opt. I think there are a few more places that need an #ifdef NO_STATFS. Could you look into it and post a patch if there's a problem? If not, please reply to the gem5 QA