I have more questions/issues on this topic of protocol specific profiling. I
do not believe this issues should be fixed by having SLICC understand more STL
types. I should have pointed out before that many times it is not one specific
protocol that needs special profiling, but rather a set of
Hi Nilay,
Could you explain the motivation behind this change? What was wrong with the
previous notation that data member declarations are separated by commas rather
than semi-colons?
This is the type of seemingly unnecessary change that make it difficult for
users to stay in sync with the pu
> On Nov. 25, 2014, 10:02 p.m., Nilay Vaish wrote:
> > I suggest the additions being made to the Simulation.py file be made by
> > creating new functions.
> > I think the run function is already too big and confusing.
I can work on cleaning up the patch by creating new functions as suggested.
On Tue, 2 Dec 2014, Beckmann, Brad wrote:
Hi Nilay,
Could you explain the motivation behind this change? What was wrong
with the previous notation that data member declarations are separated
by commas rather than semi-colons?
I think in most places in SLICC we use comma to separate the att
Yes. The source repos used to make those kernels is the same as the source
repos referenced here:
http://gem5.org/ARM_Linux_Kernel
Cheers,
Anthony Gutierrez
http://web.eecs.umich.edu/~atgutier
On Tue, Dec 2, 2014 at 1:02 PM, Guru Prasad wrote:
> Hi,
>
> Are the kernel sources also available? I
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2541/#review5605
---
Ship it!
Ship It!
- Nilay Vaish
On Nov. 25, 2014, 11:47 a.m., Gabe Bl
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2538/#review5604
---
Ship it!
Seems fine other than the one comment I have made below.
src/
---
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/2528/#review5603
---
Ship it!
Ship It!
- Nilay Vaish
On Nov. 24, 2014, 2:48 a.m., Steve Re
changeset c848de089432 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=c848de089432
description:
mem: Clean up packet data allocation
This patch attempts to make the rules for data allocation in the
packet explicit, understandable, and easy to verify.
changeset fc4c90a7d2f5 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=fc4c90a7d2f5
description:
mem: Relax packet src/dest check and shift onus to crossbar
This patch allows objects to get the src/dest of a packet even if it
is not set to a valid port
changeset c99c8d2a7c31 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=c99c8d2a7c31
description:
mem: Assume all dynamic packet data is array allocated
This patch simplifies how we deal with dynamically allocated data in
the packet, always assuming tha
changeset 997be6ba467e in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=997be6ba467e
description:
config: Fix to SystemC example's event handling
This patch fixes checkpoint restore in the SystemC hosting example by
handling
early PollEvent events corr
changeset e622a3e2ed14 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=e622a3e2ed14
description:
arm: Fix TLB ignoring faults when table walking
This patch fixes a case where the Minor CPU can deadlock due to the lack
of a response to TLB request becau
changeset d1e1e851 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=d1e1e851
description:
mem: Support WriteInvalidate (again)
This patch takes a clean-slate approach to providing WriteInvalidate
(write streaming, full cache line writes without
changeset c04dc66e4316 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=c04dc66e4316
description:
mem: Remove WriteInvalidate support
Prepare for a different implementation following in the next patch
diffstat:
src/mem/cache/base.hh |1 -
src/mem/c
changeset babb40bd2fc6 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=babb40bd2fc6
description:
scons: Ensure dictionary iteration is sorted by key
This patch adds sorting based on the SimObject name or parameter name
for all situations where we itera
changeset 7c4f1d0a8cff in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=7c4f1d0a8cff
description:
cpu: Fix retries on barrier/store in Minor's store buffer
This patch fixes a case where a store in Minor's store buffer never
leaves the store buffer as it
changeset 1c9d5d9417b3 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=1c9d5d9417b3
description:
stats: Bump stats for fixes, mostly TLB and WriteInvalidate
diffstat:
tests/long/fs/10.linux-boot/ref/alpha/linux/tsunami-minor/stats.txt
| 1869 +-
tes
changeset a8c16e2d466a in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=a8c16e2d466a
description:
mem: Use const pointers for port proxy write functions
This patch changes the various write functions in the port proxies
to use const pointers for all sou
changeset ffd46545b284 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=ffd46545b284
description:
mem: Make the requests carried by packets const
This adds a basic level of sanity checking to the packet by ensuring
that a request is not modified once th
changeset de2979ff873a in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=de2979ff873a
description:
stats: Bump stats for o3 LSQ changes
diffstat:
tests/long/fs/10.linux-boot/ref/alpha/linux/tsunami-o3-dual/stats.txt |
1594 +-
tests/long/fs/10.linux-boot/ref/ar
changeset dcb908e40547 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=dcb908e40547
description:
mem: Cleanup Packet::checkFunctional and hasData usage
This patch cleans up the use of hasData and checkFunctional in the
packet. The hasData function is u
changeset 3b405d11d6dc in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=3b405d11d6dc
description:
cpu: Move packet deallocation to recvTimingResp in the O3 CPU
Move the packet deallocations in the O3 CPU so that the
completeDataAccess
deals only with t
changeset a8d612fa170b in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=a8d612fa170b
description:
cpu, o3: Ignored invalidate causing same-address load reordering
In case the memory subsystem sends a combined response with invalidate
(e.g. ReadRespWith
changeset e1a853349529 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=e1a853349529
description:
mem: Add a GDDR5 DRAM config
This patch adds a first cut GDDR5 config to accommodate the users
combining gem5 and GPUSim. The config is based on a SK Hynix
changeset e70523bd0d26 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=e70523bd0d26
description:
mem: Make Request getters const
This patch tidies up the Request class, making all getters const. The
odd one out is incAccessDepth which is called by the
changeset 95297ec0f14b in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=95297ec0f14b
description:
cpu: Always mask the snoop address when performing lock check
Ensure the snoop address check is always using a cache-block aligned
address. This patch upda
changeset 5d7af9fa9809 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=5d7af9fa9809
description:
config: SystemC Gem5Control top level additions
This patch cleans up a few style issues and adds a few capabilities to
the
SystemC top level 'Gem5Control/
changeset 953d7b741619 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=953d7b741619
description:
cpu: Fix memoryIssueLimit checking in Minor
This patch fixes the checking of the number of memory instructions
issued
per cycles in the Minor CPU.
diffst
changeset b99fdc295c34 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=b99fdc295c34
description:
mem: Remove null-check bypassing in Packet::getPtr
This patch removes the parameter that enables bypassing the null check
in the Packet::getPtr method. A n
changeset 926802ed1536 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=926802ed1536
description:
mem: Add checks and explanation for assertMemInhibit usage
diffstat:
src/mem/cache/cache_impl.hh | 7 ++-
src/mem/packet.hh | 7 ++-
2 files changed,
changeset 23593fdaadcd in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=23593fdaadcd
description:
mem: Remove redundant Packet::allocate calls
This patch cleans up the packet memory allocation confusion. The data
is always allocated at the requesting si
changeset 755b18321206 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=755b18321206
description:
mem: Add const getters for write packet data
This patch takes a first step in tightening up how we use the data
pointer in write packets. A const getter is
X86LinuxSESystem is a better name.
Gabe
On Mon, Dec 1, 2014 at 3:05 AM, Gabe Black wrote:
> One of the last things I did the last time I was working on gem5 was to
> merge the SE and FS builds of the simulator into one. I didn't really
> finish it before I left, though, and in a lot of ways it'
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/minor-timing passed.
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/o3-timing passed.
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple-timing passed.
* build/ALPHA/tests/opt/quick/se/00.hello/alpha/linux/simple
35 matches
Mail list logo