Re: [gem5-dev] Review Request 2624: mem: mmap the backing store with MAP_NORESERVE

2015-02-08 Thread Andreas Hansson via gem5-dev
On Feb. 3, 2015, 9:37 p.m., Brandon Potter wrote: The patch seems like a landmine for an unsuspecting user as it would be difficult to diagnose if swap space is exhausted. It will probably be evident that memory exhaustion caused the runtime error (segmentation fault), but tracking

Re: [gem5-dev] Review Request 2646: mem: Split port retry for all different packet classes

2015-02-08 Thread Andreas Hansson via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2646/ --- (Updated Feb. 8, 2015, 5:48 p.m.) Review request for Default. Repository: gem5

Re: [gem5-dev] Review Request 2646: mem: Split port retry for all different packet classes

2015-02-08 Thread Andreas Hansson via gem5-dev
On Feb. 7, 2015, 6:14 p.m., Steve Reinhardt wrote: Awesome! This lack of separate channels for requests responses has bugged me for years, I'm really glad to see it addressed. One minor suggestion: to me, terms like recvRetryReq and recvRetryResp seem backward, since 'retry' is

[gem5-dev] Review Request 2650: cache: add local var in recvTimingResp()

2015-02-08 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2650/ --- Review request for Default. Repository: gem5 Description --- Changeset

[gem5-dev] Review Request 2651: cache: remove redundant test in recvTimingResp()

2015-02-08 Thread Steve Reinhardt via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2651/ --- Review request for Default. Repository: gem5 Description --- Changeset

[gem5-dev] Cron m5test@zizzer /z/m5/regression/do-regression --scratch all

2015-02-08 Thread Cron Daemon via gem5-dev
* build/ALPHA/tests/opt/quick/se/30.eio-mp/alpha/eio/simple-timing-mp passed. * build/ALPHA/tests/opt/quick/se/20.eio-short/alpha/eio/simple-timing passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/tru64/minor-timing passed. *