Re: [gem5-dev] Review Request 3470: gpu-compute: Fixed a bug in global memory pipeline

2016-05-19 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3470/#review8325 --- Ship it! Thanks, Tuan. I will ship this soon. - Tony Gutierrez On

Re: [gem5-dev] Review Request 3470: gpu-compute: Fixed a bug in global memory pipeline

2016-05-19 Thread Tuan Ta
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3470/ --- (Updated May 19, 2016, 9:30 p.m.) Review request for Default. Summary (updated)

Re: [gem5-dev] gem5 not building with gcc-4.7 -- bump required gcc version?

2016-05-19 Thread Steve Reinhardt
Fine with us. Steve On Tue, May 17, 2016 at 1:54 AM Andreas Hansson wrote: > Hi all, > > Did we converge on this? > > Is everyone happy to bump the official requirement to gcc 4.8? > > Andreas > > On 18/04/2016, 14:53, "gem5-dev on behalf of Curtis Dunham" >

[gem5-dev] changeset in gem5: config, x86: Properly space pad the X86IntelM...

2016-05-19 Thread Bjoern A. Zeeb
changeset fc247b9c42b6 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=fc247b9c42b6 description: config, x86: Properly space pad the X86IntelMPBus Entry descriptions According to the Intel Multi Processor Specification rev 1.4 (-006) (*), section

[gem5-dev] changeset in gem5: arm, dev: PL011 UART_FR read status enhancement

2016-05-19 Thread Bjoern A. Zeeb
changeset 2af4c6a4f3f5 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=2af4c6a4f3f5 description: arm,dev: PL011 UART_FR read status enhancement Given we do not simulate a FIFO currently there are only two states we can be in upon read: empty or full.

[gem5-dev] changeset in gem5: x86, dev: properly space the APIC registers

2016-05-19 Thread Bjoern A. Zeeb
changeset 8b23edf06cd3 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=8b23edf06cd3 description: x86, dev: properly space the APIC registers Registers are 0x10 and not 0x8 apart. The latter leads to invalid calculations of index in array which in

[gem5-dev] changeset in gem5: dev, virtio: properly set PCI address space t...

2016-05-19 Thread Bjoern A. Zeeb
changeset c926270c33c8 in /z/repo/gem5 details: http://repo.gem5.org/gem5?cmd=changeset;node=c926270c33c8 description: dev, virtio: properly set PCI address space to use IOREG VirtIO spec < 1.0 demands IOREG to be used on PCI and not memory mapped. Set the correct bit on

Re: [gem5-dev] Review Request 3470: gpu-compute: Fixed a bug in global_memory_pipeline.cc

2016-05-19 Thread Tony Gutierrez
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3470/#review8324 --- The description still needs to be fixed. It should follow this form:

Re: [gem5-dev] Review Request 3465: dist, dev: Fixed the packet ordering in etherswitch

2016-05-19 Thread Gabor Dozsa
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3465/#review8320 --- src/dev/net/etherswitch.hh (line 39)

Re: [gem5-dev] Review Request 3132: misc: Separate stats file is generated for SystemC-gem5 co-simulation

2016-05-19 Thread Andrew Bardsley
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3132/#review8323 --- Ship it! Caveat: make the filename/configuration for stats output

Re: [gem5-dev] Review Request 3132: misc: Separate stats file is generated for SystemC-gem5 co-simulation

2016-05-19 Thread Andrew Bardsley
> On Oct. 16, 2015, 3:15 p.m., Nilay Vaish wrote: > > util/systemc/stats.cc, line 79 > > > > > > The file name can be kept as constant, but at least pass an argument to > > this function that specifies the file path. > >

Re: [gem5-dev] Review Request 3465: dist, dev: Fixed the packet ordering in etherswitch

2016-05-19 Thread Gabor Dozsa
> On May 18, 2016, 3 p.m., Gabor Dozsa wrote: > > src/dev/net/etherswitch.cc, line 87 > > > > > > You can replace std::prev(fifo.end()) by fifo.rbegin() (in the next > > line, too). > > Mohammad Alian wrote: > I think

[gem5-dev] Review Request 3473: systemc example: Fixed an ambiguous call error and a typo in the README

2016-05-19 Thread Christian Menard
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3473/ --- Review request for Default. Repository: gem5 Description --- gcc 6.1.1

[gem5-dev] Cron <m5test@zizzer> /z/m5/regression/do-regression quick

2016-05-19 Thread Cron Daemon
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/o3-timing passed.* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple-atomic passed. * build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/minor-timing passed.*