gem5.git
11 years agomem: Merge ranges that are part of the conf table
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
mem: Merge ranges that are part of the conf table

This patch adds basic merging of address ranges when determining which
address ranges should be reported in the configuration table. By
performing this merging it is possible to distribute an address range
across many memory channels (controllers). This is essential to enable
address interleaving.

11 years agobase: Add support for merging of interleaved address ranges
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
base: Add support for merging of interleaved address ranges

This patch adds support for merging a vector of interleaved address
ranges into a contigous range. The functionality will be used in the
interconnect and the PhysicalMemory to transform interleaved memory
ranges to contigous ranges before passing them on.

The actual use of the merging is appearing in future patches.

11 years agomem: Add interleaving bits to the address ranges
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
mem: Add interleaving bits to the address ranges

This patch adds support for interleaving bits for the address
ranges. What was previously just a start and end address, now has an
additional three fields, for the high bit, and number of bits to use
for interleaving, and a match value to compare against. If the number
of interleaving bits is set to zero it is effectively disabled.

A number of convenience functions are added to the range to enquire
about the interleaving, its granularity and the number of stripes it
is part of.

11 years agoconfig: Traverse lists when visiting children in all proxy
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
config: Traverse lists when visiting children in all proxy

This patch makes the all proxy traverse any potential list that is
encountered in the object hierarchy instead of only looking at
children that are SimObjects. An example of where this is useful is
when creating a multi-channel memory system as a list of controllers,
whilst ensuring that the memories are still visible in the system.

11 years agobase: Simplify the AddrRangeMap by removing unused code
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
base: Simplify the AddrRangeMap by removing unused code

This patch cleans up the AddrRangeMap in preparation for the addition
of interleaving by removing unused code. The non-const editions of
find are never used, and hence the duplication is not needed.

11 years agoconfig: Do not use hardcoded physmem in fs script
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
config: Do not use hardcoded physmem in fs script

This patch generalises the address range resolution for the I/O cache
and I/O bridge such that they do not assume a single memory. The patch
involves adding a parameter to the system which is then defined based
on the memories that are to be visible from the I/O subsystem, whether
behind a cache or a bridge.

The change is needed to allow interleaved memory controllers in the
system.

11 years agomem: Tidy up bus addr range debug messages
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
mem: Tidy up bus addr range debug messages

This patch tidies up a number of the bus DPRINTFs related to range
manipulation. In particular, it shifts the message about range changes
to the start of the member function, and also adds information about
when all ranges are received.

11 years agomem: Skip address mapper range checks to allow more flexibility
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
mem: Skip address mapper range checks to allow more flexibility

This patch makes the address mapper less stringent about checking the
before and after ranges, i.e. the original and remapped ranges. The
checks were not really necessary, and there are situations when the
previous checks were too strict.

11 years agobase: Encapsulate the underlying fields in AddrRange
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
base: Encapsulate the underlying fields in AddrRange

This patch makes the start and end address private in a move to
prevent direct manipulation and matching of ranges based on these
fields. This is done so that a transition to ranges with interleaving
support is possible.

As a result of hiding the start and end, a number of member functions
are needed to perform the comparisons and manipulations that
previously took place directly on the members. An accessor function is
provided for the start address, and a function is added to test if an
address is within a range. As a result of the latter the != and ==
operator is also removed in favour of the member function. A member
function that returns a string representation is also created to allow
debug printing.

In general, this patch does not add any functionality, but it does
take us closer to a situation where interleaving (and more cleverness)
can be added under the bonnet without exposing it to the user. More on
that in a later patch.

11 years agomem: Remove the joining of neighbouring ranges
Andreas Hansson [Mon, 7 Jan 2013 18:05:38 +0000 (13:05 -0500)]
mem: Remove the joining of neighbouring ranges

This patch temporarily removes the joining of ranges when creating the
backing store, to reserve this functionality for the interleaved
ranges that are about to be introduced.

When creating the mmaps for the backing store, there is no point in
creating larger contigous chunks that what is necessary. The larger
chunks will only make life more difficult for the host.

Merging will be re-added later, but then only for interleaved ranges.

11 years agocpu: Share the send functionality between traffic generators
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
cpu: Share the send functionality between traffic generators

This patch moves the packet creating and sending to a member function
in the shared base class to avoid code duplication.

11 years agocpu: Add support for protobuf input for the trace generator
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
cpu: Add support for protobuf input for the trace generator

This patch adds support for reading input traces encoded using
protobuf according to what is done in the CommMonitor.

A follow-up patch adds a Python script that can be used to convert the
previously used ASCII traces to protobuf equivalents. The appropriate
regression input is updated as part of this patch.

11 years agotests: Add support for skipping tests, skip EIO tests if not enabled
Andreas Sandberg [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
tests: Add support for skipping tests, skip EIO tests if not enabled

The EIO tests depend on the EIO support from the "encumbered"
repository, which means that they are not normally built with
gem5. This causes all EIO related tests to fail, which is both
annoying and confusing. This patch addresses this by adding support
for skipping tests if certain conditions (e.g., the presence of a
SimObject) can not be met. It introduces the following Python
functions that can be called from within a test case:

  * skip_test -- Skip a test and optionally print why the test was
                 skipped.

  * has_sim_object -- Test if a SimObject exists.

  * require_sim_object -- Test if a SimObject exists and skip, or
                          optionally fail, the test if not.

Additionally, this patch updates the EIO tests to check for the
presence of EioProcess.

11 years agocpu: Encapsulate traffic generator input in a stream
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
cpu: Encapsulate traffic generator input in a stream

This patch encapsulates the traffic generator input in a stream class
such that the parsing is not visible to the trace generator. The
change takes us one step closer to using protobuf-based input traces
for the trace replay.

The functionality of the current input stream is identical to what it
was, and the ASCII format remains the same for now.

11 years agobase: Add wrapped protobuf input stream
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
base: Add wrapped protobuf input stream

This patch adds support for inputting protobuf messages through a
ProtoInputStream which hides the internal streams used by the
library. The stream is created based on the name of an input file and
optionally includes decompression using gzip.

The input stream will start by getting a magic number from the file,
and also verify that it matches with the expected value. Once opened,
messages can be read incrementally from the stream, returning
true/false until an error occurs or the end of the file is reached.

11 years agomem: Add tracing support in the communication monitor
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
mem: Add tracing support in the communication monitor

This patch adds packet tracing to the communication monitor using a
protobuf as the mechanism for creating the trace.

If no file is specified, then the tracing is disabled. If a file is
specified, then for every packet that is successfully sent, a protobuf
message is serialized to the file.

11 years agobase: Add wrapped protobuf output streams
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
base: Add wrapped protobuf output streams

This patch adds support for outputting protobuf messages through a
ProtoOutputStream which hides the internal streams used by the
library. The stream is created based on the name of an output file and
optionally includes compression using gzip.

The output stream will start by putting a magic number in the file,
and then for every message that is serialized prepend the size such
that the stream can be written and read incrementally. At this point
this merely serves as a proof of concept.

11 years agoscons: Add support for google protobuf building
Andreas Hansson [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
scons: Add support for google protobuf building

This patch enables the use of protobuf input files in the build
process, thus allowing .proto files to be added to input. Each .proto
file is compiled using the protoc tool and the newly created C++
source is added to the list of sources.

The first location where the protobufs will be used is in the
capturing and replay of memory traces, involving the communication
monitor and the trace-generator state of the traffic generator. This
will follow in the next patch.

This patch does add a dependency on the availability of the BSD
licensed protobuf library (and headers), and the protobuf compiler,
protoc. These dependencies are checked in the SConstruct, similar to
e.g. swig. The user can override the use of protoc from the PATH by
specifying the PROTOC environment variable.

Although the dependency on libprotobuf and protoc might seem like a
big step, they add significant value to the project going
forward. Execution traces and other types of traces could easily be
added and parsers for C++ and Python are automatically generated. We
could also envision using protobufs for the checkpoints, description
of the traffic-generator behaviour etc. The sky is the limit. We could
also use the GzipOutputStream from the protobuf library instead of the
current GPL gzstream.

Currently, only the C++ source and header is generated. Going forward
we might want to add the Python output to support simple command-line
tools for displaying and editing the traces.

11 years agoarm: Fix DMA event handling bug in the PL111 model
Andreas Sandberg [Mon, 7 Jan 2013 18:05:37 +0000 (13:05 -0500)]
arm: Fix DMA event handling bug in the PL111 model

The PL111 model currently maintains a list of pre-allocated
DmaDoneEvents to prevent unnecessary heap allocations. This list
effectively works like a stack where the top element is the latest
scheduled event. When an event triggers, the top pointer is moved down
the stack. This obviously breaks since events usually retire from the
bottom (events don't necessarily have to retire in order), which
triggers the following assertion:

gem5.debug: build/ARM/dev/arm/pl111.cc:460: void Pl111::fillFifo(): \
  Assertion `!dmaDoneEvent[dmaPendingNum-1].scheduled()' failed.

This changeset adds a vector listing the currently unused events. This
vector acts like a stack where the an element is popped off the stack
when a new event is needed an pushed on the stack when they trigger.

11 years agodev: Fix the Pl111 timings by separating pixel and DMA clock
Andreas Hansson [Mon, 7 Jan 2013 18:05:36 +0000 (13:05 -0500)]
dev: Fix the Pl111 timings by separating pixel and DMA clock

This patch fixes the Pl111 timings by creating a separate clock for
the pixel timings. The device clock is used for all interactions with
the memory system, just like the AHB clock on the actual module.

The result without this patch is that the module only is allowed to
send one request every tick of the 24MHz clock which causes a huge
backlog.

11 years agostats: Update DRAM regression stats to match new config
Andreas Hansson [Mon, 7 Jan 2013 18:05:36 +0000 (13:05 -0500)]
stats: Update DRAM regression stats to match new config

This patch updates the regression stats to reflect the change in the
traffic gen configuration.

11 years agoconfig: Reduce DRAM controller regression traffic rate
Andreas Hansson [Mon, 7 Jan 2013 18:05:36 +0000 (13:05 -0500)]
config: Reduce DRAM controller regression traffic rate

This patch changes the traffic generator period such that it does not
completely saturate the DRAM controller and create an ever-growing
backlog in the queued port.

A separate patch updates the stats.

11 years agocpu: Fix the traffic gen read percentage
Andreas Hansson [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
cpu: Fix the traffic gen read percentage

This patch fixes the computation that determines whether to perform a
read or a write such that the two corner cases (0 and 100) are both
more efficient and handled correctly.

11 years agomem: Add sanity check to packet queue size
Andreas Hansson [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
mem: Add sanity check to packet queue size

This patch adds a basic check to ensure that the packet queue does not
grow absurdly large. The queue should only be used to store packets
that were delayed due to blocking from the neighbouring port, and not
for actual storage. Thus, a limit of 100 has been chosen for now
(which is already quite substantial).

11 years agoruby: Fix missing cxx_header in Switch
Andreas Hansson [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
ruby: Fix missing cxx_header in Switch

This patch addresses a warning related to the swig interface
generation for the Switch class. The cxx_header is now specified
correctly, and the header in question has got a few includes added to
make it all compile.

11 years agoscons: Fix libelf linking errors when using clang/llvm
Andreas Hansson [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
scons: Fix libelf linking errors when using clang/llvm

This patch fixes a linking error that occurs when using clang/llvm in
combination with older versions of glibc. The fix involves adding
-std=gnu89 to the command line when compiling libelf as clang defaults
to c99, causing issues with the symbols in sysmacros.h being defined
multiple times.

11 years agoconfig: Replace second keyboard with a mouse.
Chris Emmons [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
config:  Replace second keyboard with a mouse.

The platform has two KMI devices that are both setup to be keyboards.  This
patch changes the second keyboard to a mouse.  This patch will allow keyboard
input as usual and additionally provide mouse support.

11 years agomem: Fix a bug in the memory serialization file naming
Andreas Hansson [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
mem: Fix a bug in the memory serialization file naming

This patch fixes a bug that caused multiple systems to overwrite each
other physical memory. The system name is now included in the filename
such that this is avoided.

11 years agoarm: Make ID registers ISA parameters
Andreas Sandberg [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
arm: Make ID registers ISA parameters

This patch makes the values of ID_ISARx, MIDR, and FPSID configurable
as ISA parameter values. Additionally, setMiscReg now ignores writes
to all of the ID registers.

Note: This moves the MIDR parameter from ArmSystem to ArmISA for
consistency.

11 years agoarch: Make the ISA class inherit from SimObject
Andreas Sandberg [Mon, 7 Jan 2013 18:05:35 +0000 (13:05 -0500)]
arch: Make the ISA class inherit from SimObject

The ISA class on stores the contents of ID registers on many
architectures. In order to make reset values of such registers
configurable, we make the class inherit from SimObject, which allows
us to use the normal generated parameter headers.

This patch introduces a Python helper method, BaseCPU.createThreads(),
which creates a set of ISAs for each of the threads in an SMT
system. Although it is currently only needed when creating
multi-threaded CPUs, it should always be called before instantiating
the system as this is an obvious place to configure ID registers
identifying a thread/CPU.

11 years agoo3: Fix issue with LLSC ordering and speculation
Ali Saidi [Mon, 7 Jan 2013 18:05:33 +0000 (13:05 -0500)]
o3: Fix issue with LLSC ordering and speculation

This patch unlocks the cpu-local monitor when the CPU sees a snoop to a locked
address. Previously we relied on the cache to handle the locking for us, however
some users on the gem5 mailing list reported a case where the cpu speculatively
executes a ll operation after a pending sc operation in the pipeline and that
makes the cache monitor valid. This should handle that case by invaliding the
local monitor.

11 years agocpu: rename the misleading inSyscall to noSquashFromTC
Ali Saidi [Mon, 7 Jan 2013 18:05:33 +0000 (13:05 -0500)]
cpu: rename the misleading inSyscall to noSquashFromTC

isSyscall was originally created because during handling of a syscall in SE
mode the threadcontext had to be updated. However, in many places this is used
in FS mode (e.g. fault handlers) and the name doesn't make much sense. The
boolean actually stops gem5 from squashing speculative and non-committed state
when a write to a threadcontext happens, so re-name the variable to something
more appropriate

11 years agotests: Always specify memory mode in every test system.
Ali Saidi [Mon, 7 Jan 2013 18:05:33 +0000 (13:05 -0500)]
tests: Always specify memory mode in every test system.

Previous to this change we didn't always set the memory mode which worked as
long as we never attempted to switch CPUs or checked that a CPU was in a
memory system with the correct mode. Future changes will make CPUs verify
that they're operating in the correct mode and thus we need to always set it.

11 years agotests: Create base classes to encapsulate common test configurations
Andreas Sandberg [Mon, 7 Jan 2013 18:05:33 +0000 (13:05 -0500)]
tests: Create base classes to encapsulate common test configurations

Most of the test cases currently contain a large amount of duplicated
boiler plate code. This changeset introduces a set of classes that
encapsulates most of the functionality when setting up a test
configuration.

The following base classes are introduced:
* BaseSystem - Basic system configuration that can be used for both
               SE and FS simulation.

* BaseFSSystem - Basic FS configuration uni-processor and multi-processor
                 configurations.

* BaseFSSystemUniprocessor - Basic FS configuration for uni-processor
                             configurations. This is provided as a way
     to make existing test cases backwards
     compatible.

Architecture specific implementations are provided for ARM, Alpha, and
X86.

11 years agocache: add note about where conflicts are handled
Ali Saidi [Mon, 7 Jan 2013 18:05:32 +0000 (13:05 -0500)]
cache: add note about where conflicts are handled

11 years agoregressions: stats update due to decoder changes
Nilay Vaish [Sat, 5 Jan 2013 01:00:48 +0000 (19:00 -0600)]
regressions: stats update due to decoder changes

11 years agoDecoder: Remove the thread context get/set from the decoder.
Gabe Black [Sat, 5 Jan 2013 01:00:45 +0000 (19:00 -0600)]
Decoder: Remove the thread context get/set from the decoder.

This interface is no longer used, and getting rid of it simplifies the
decoders and code that sets up the decoders. The thread context had been used
to read architectural state which was used to contextualize the instruction
memory as it came in. That was changed so that the state is now sent to the
decoders to keep locally if/when it changes. That's significantly more
efficient.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

11 years agoX86: Move address based decode caching in front of the predecoder.
Gabe Black [Sat, 5 Jan 2013 01:00:44 +0000 (19:00 -0600)]
X86: Move address based decode caching in front of the predecoder.
The predecoder in x86 does a lot of work, most of which can be skipped if the
decoder cache is put in front of it.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

11 years agoSPARC: Keep a copy of the current ASI in the decoder.
Gabe Black [Sat, 5 Jan 2013 00:09:45 +0000 (18:09 -0600)]
SPARC: Keep a copy of the current ASI in the decoder.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

11 years agoARM: Keep a copy of the fpscr len and stride fields in the decoder.
Gabe Black [Sat, 5 Jan 2013 00:09:35 +0000 (18:09 -0600)]
ARM: Keep a copy of the fpscr len and stride fields in the decoder.
Avoid reading them every instruction, and also eliminate the last use of the
thread context in the decoders.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

11 years agox86 regressions: stats update due to new x87 instructions
Nilay Vaish [Sun, 30 Dec 2012 18:45:52 +0000 (12:45 -0600)]
x86 regressions: stats update due to new x87 instructions

11 years agox86: implement x87 fp instruction fnstsw
Nilay Vaish [Sun, 30 Dec 2012 18:45:50 +0000 (12:45 -0600)]
x86: implement x87 fp instruction fnstsw
This patch implements the fnstsw instruction. The code was originally written
by Vince Weaver. Gabe had made some comments about the code, but those were
never addressed. This patch addresses those comments.

11 years agox86: implement x87 fp instruction fsincos
Nilay Vaish [Sun, 30 Dec 2012 18:45:45 +0000 (12:45 -0600)]
x86: implement x87 fp instruction fsincos
This patch implements the fsincos instruction. The code was originally written
by Vince Weaver. Gabe had made some comments about the code, but those were
never addressed. This patch addresses those comments.

12 years agoarm regressions: updates to config.ini, terminal files
Nilay Vaish [Wed, 12 Dec 2012 15:51:55 +0000 (09:51 -0600)]
arm regressions: updates to config.ini, terminal files

12 years agoarm: set uopSet_uop as conditional or unconditional control
Nathanael Premillieu [Wed, 12 Dec 2012 15:50:33 +0000 (09:50 -0600)]
arm: set uopSet_uop as conditional or unconditional control
uopSet_uop is microop instruction that has the IsControl flags set, but the
IsCondControl or IsUncondControl flags seems not to be set, neither in
the construction nor where the microop is used. This patch adds the the
flags in the constructor of the instruction (MicroUopSetPCCPSR).

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

12 years agoarm: set movret_uop as conditional or unconditional control
Nathanael Premillieu [Wed, 12 Dec 2012 15:50:16 +0000 (09:50 -0600)]
arm: set movret_uop as conditional or unconditional control
A flag was missing for the movret_uop microop instruction. This patch adds
that flag when the instruction is used, not directly in the constructor of
the instruction.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

12 years agoregressions: stats update due to stats from ruby prefetcher
Nilay Vaish [Tue, 11 Dec 2012 16:06:01 +0000 (10:06 -0600)]
regressions: stats update due to stats from ruby prefetcher

12 years agoruby: add support for prefetching to MESI protocol
Nilay Vaish [Tue, 11 Dec 2012 16:05:56 +0000 (10:05 -0600)]
ruby: add support for prefetching to MESI protocol

12 years agoruby: modify the directed tester to read/write streams
Nilay Vaish [Tue, 11 Dec 2012 16:05:55 +0000 (10:05 -0600)]
ruby: modify the directed tester to read/write streams
The directed tester supports only generating only read or only write accesses. The
patch modifies the tester to support streams that have both read and write accesses.

12 years agoruby: change slicc to allow for constructor args
Nilay Vaish [Tue, 11 Dec 2012 16:05:55 +0000 (10:05 -0600)]
ruby: change slicc to allow for constructor args
The patch adds support to slicc for recognizing arguments that should be
passed to the constructor of a class. I did not like the fact that an explicit
check was being carried on the type 'TBETable' to figure out the arguments to
be passed to the constructor.
The patch also moves some of the member variables that are declared for all
the controllers to the base class AbstractController.

12 years agoruby: add a prefetcher
Nilay Vaish [Tue, 11 Dec 2012 16:05:54 +0000 (10:05 -0600)]
ruby: add a prefetcher
This patch adds a prefetcher for the ruby memory system. The prefetcher
is based on a prefetcher implemented by others (well, I don't know
who wrote the original). The prefetcher does stride-based prefetching,
both unit and non-unit. It obseves the misses in the cache and trains on
these. After the training period is over, the prefetcher starts issuing
prefetch requests to the controller.

12 years agoruby: add functions for computing next stride/page address
Nilay Vaish [Tue, 11 Dec 2012 16:05:53 +0000 (10:05 -0600)]
ruby: add functions for computing next stride/page address

12 years agoregression test: update a couple of config.ini files
Nilay Vaish [Thu, 6 Dec 2012 16:26:12 +0000 (10:26 -0600)]
regression test: update a couple of config.ini files

12 years agoTournamentBP: Fix some bugs with table sizes and counters
Erik Tomusk [Thu, 6 Dec 2012 15:31:06 +0000 (09:31 -0600)]
TournamentBP: Fix some bugs with table sizes and counters
globalHistoryBits, globalPredictorSize, and choicePredictorSize are decoupled.
globalHistoryBits controls how much history is kept, global and choice
predictor sizes control how much of that history is used when accessing
predictor tables. This way, global and choice predictors can actually be
different sizes, and it is no longer possible to walk off the predictor arrays
and cause a seg fault.

There are now individual thresholds for choice, global, and local saturating
counters, so that taken/not taken decisions are correct even when the
predictors' counters' sizes are different.

The interface for localPredictorSize has been removed from TournamentBP because
the value can be calculated from localHistoryBits.

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

12 years agoinorder cpu: add missing DPRINTF argument
Malek Musleh [Thu, 6 Dec 2012 11:25:40 +0000 (05:25 -0600)]
inorder cpu: add missing DPRINTF argument

Committed by: Nilay Vaish <nilay@cs.wisc.edu>

12 years agoo3 cpu: remove some unused buggy functions in the lsq
Nathanael Premillieu [Thu, 6 Dec 2012 10:36:51 +0000 (04:36 -0600)]
o3 cpu: remove some unused buggy functions in the lsq
Committed by: Nilay Vaish <nilay@cs.wisc.edu>

12 years agoconfig: Fix description of checkpoint option from cycle to tick
Andreas Hansson [Mon, 19 Nov 2012 16:21:09 +0000 (11:21 -0500)]
config: Fix description of checkpoint option from cycle to tick

This patch merely updates the description of the "take-checkpoints"
option to reflect that it is specified in ticks and not in cycles.

12 years agosim: have a curTick per eventq
Nilay Vaish [Fri, 16 Nov 2012 16:27:47 +0000 (10:27 -0600)]
sim: have a curTick per eventq
This patch adds a _curTick variable to an eventq. This variable is updated
whenever an event is serviced in function serviceOne(), or all events upto
a particular time are processed in function serviceEvents(). This change
helps when there are eventqs that do not make use of curTick for scheduling
events.

12 years agoregressions: stats update due to ruby functional access patch
Nilay Vaish [Sat, 10 Nov 2012 23:18:02 +0000 (17:18 -0600)]
regressions: stats update due to ruby functional access patch

12 years agoruby: support functional accesses in garnet flexible network
Nilay Vaish [Sat, 10 Nov 2012 23:18:01 +0000 (17:18 -0600)]
ruby: support functional accesses in garnet flexible network

12 years agoruby: bug in functionalRead, revert recent changes
Nilay Vaish [Sat, 10 Nov 2012 23:18:00 +0000 (17:18 -0600)]
ruby: bug in functionalRead, revert recent changes
Recent changes to functionalRead() in the memory system was not correct.
The change allowed for returning data from the first message found in
the buffers of the memory system. This is not correct since it is possible
that a timing message has data from an older state of the block.

The changes are being reverted.

12 years agomem: Fix DRAM draining to ensure write queue is empty
Andreas Hansson [Thu, 8 Nov 2012 09:25:06 +0000 (04:25 -0500)]
mem: Fix DRAM draining to ensure write queue is empty

This patch fixes the draining of the SimpleDRAM controller model. The
controller performs buffering of writes and normally there is no need
to ever empty the write buffer (if you have a fast on-chip memory,
then use it). The patch adds checks to ensure the write buffer is
drained when the controller is asked to do so.

12 years agox86, util: add m5_writefile to m5op_x86.S
Lluis Vilanova [Sat, 3 Nov 2012 17:57:28 +0000 (12:57 -0500)]
x86, util: add m5_writefile to m5op_x86.S

Committed by: Nilay Vaish

12 years agoruby: reset and dump stats along with reset of the system
Hamid Reza Khaleghzadeh ext:(%2C%20Lluc%20Alvarez%20%3Clluc.alvarez%40bsc.es%3E%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) [Fri, 2 Nov 2012 17:18:25 +0000 (12:18 -0500)]
ruby: reset and dump stats along with reset of the system
This patch adds support to ruby so that the statistics maintained by ruby
are reset/dumped when the statistics for the rest of the system are
reset/dumped. For resetting the statistics, ruby now provides the
resetStats() function that a sim object can provide. As a consequence, the
clearStats() function has been removed from RubySystem. For dumping stats,
Ruby now adds a callback event to the dumpStatsQueue. The exit callback that
ruby used to add earlier is being removed.

Created by: Hamid Reza Khaleghzadeh.
Improved by: Lluc Alvarez, Nilay Vaish
Committed by: Nilay Vaish

12 years agomem: fix use after free issue in memories until 4-phase work complete.
Ali Saidi [Fri, 2 Nov 2012 16:50:16 +0000 (11:50 -0500)]
mem: fix use after free issue in memories until 4-phase work complete.

12 years agoupdate stats for preceeding changes
Ali Saidi [Fri, 2 Nov 2012 16:50:06 +0000 (11:50 -0500)]
update stats for preceeding changes

12 years agomem: Add support for writing back and flushing caches
Andreas Sandberg [Fri, 2 Nov 2012 16:32:02 +0000 (11:32 -0500)]
mem: Add support for writing back and flushing caches

This patch adds support for the following optional drain methods in
the classical memory system's cache model:

memWriteback() - Write back all dirty cache lines to memory using
functional accesses.

memInvalidate() - Invalidate all cache lines. Dirty cache lines
are lost unless a writeback is requested.

Since memWriteback() is called when checkpointing systems, this patch
adds support for checkpointing systems with caches. The serialization
code now checks whether there are any dirty lines in the cache. If
there are dirty lines in the cache, the checkpoint is flagged as bad
and a warning is printed.

12 years agosim: Add drain methods to request additional cleanup operations
Andreas Sandberg [Fri, 2 Nov 2012 16:32:02 +0000 (11:32 -0500)]
sim: Add drain methods to request additional cleanup operations

This patch adds the following two methods to the Drainable base class:

memWriteback() - Write back all dirty cache lines to memory using
functional accesses.

memInvalidate() - Invalidate memory system buffers. Dirty data
won't be written back.

Specifying calling memWriteback() after draining will allow us to
checkpoint systems with caches. memInvalidate() can be used to drop
memory system buffers in preparation for switching to an accelerated
CPU model that bypasses the gem5 memory system (e.g., hardware
virtualized CPUs).

Note: This patch only adds the methods to Drainable, the code for
flushing the TLB and the cache is committed separately.

12 years agosim: Add SWIG interface for Serializable
Andreas Sandberg [Fri, 2 Nov 2012 16:32:02 +0000 (11:32 -0500)]
sim: Add SWIG interface for Serializable

This changeset adds a SWIG interface for the Serializable class, which
fixes a warning when compiling the SWIG interface for the event
queue. Currently, the only method exported is the name() method.

12 years agopython: Rename doDrain()->drain() and make it do the right thing
Andreas Sandberg [Fri, 2 Nov 2012 16:32:02 +0000 (11:32 -0500)]
python: Rename doDrain()->drain() and make it do the right thing

There is no point in exporting the old drain() method in
Simulate.py. It should only be used internally by doDrain(). This
patch moves the old drain() method into doDrain() and renames
doDrain() to drain().

12 years agosim: Reuse the code to change memory mode.
Andreas Sandberg [Fri, 2 Nov 2012 16:32:02 +0000 (11:32 -0500)]
sim: Reuse the code to change memory mode.

changeToAtomic and changeToTiming both do essentially the same thing,
they check the type of their input argument, drain the system, and
switch to the desired memory mode. This patch moves all of that code
to a separate method (changeMemoryMode) and calls that from both
changeToAtomic and changeToTiming.

12 years agosim: Move the draining interface into a separate base class
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
sim: Move the draining interface into a separate base class

This patch moves the draining interface from SimObject to a separate
class that can be used by any object needing draining. However,
objects not visible to the Python code (i.e., objects not deriving
from SimObject) still depend on their parents informing them when to
drain. This patch also gets rid of the CountedDrainEvent (which isn't
really an event) and replaces it with a DrainManager.

12 years agocpu: O3 add a header declaring the DerivO3CPU
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
cpu: O3 add a header declaring the DerivO3CPU

SWIG needs a complete declaration of all wrapped objects. This patch
adds a header file with the DerivO3CPU class and includes it in the
SWIG interface.

--HG--
rename : src/cpu/o3/cpu_builder.cc => src/cpu/o3/deriv.cc

12 years agocpu: Add header files for checker CPUs
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
cpu: Add header files for checker CPUs

In order to create reliable SWIG wrappers, we need to include the
declaration of the wrapped class in the SWIG file. Previously, we
didn't expose the declaration of checker CPUs. This patch adds header
files for such CPUs and include them in the SWIG wrapper.

--HG--
rename : src/cpu/dummy_checker_builder.cc => src/cpu/dummy_checker.cc
rename : src/cpu/o3/checker_builder.cc => src/cpu/o3/checker.cc

12 years agodev: Fix ethernet device inheritance structure
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
dev: Fix ethernet device inheritance structure

The Python wrappers and the C++ should have the same object
structure. If this is not the case, bad things will happen when the
SWIG wrappers cast between an object and any of its base classes. This
was not the case for NSGigE and Sinic devices. This patch makes NSGigE
and Sinic inherit from the new EtherDevBase class, which in turn
inherits from EtherDevice. As a bonus, this removes some duplicated
statistics from the Sinic device.

12 years agosim: Include object header files in SWIG interfaces
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
sim: Include object header files in SWIG interfaces

When casting objects in the generated SWIG interfaces, SWIG uses
classical C-style casts ( (Foo *)bar; ). In some cases, this can
degenerate into the equivalent of a reinterpret_cast (mainly if only a
forward declaration of the type is available). This usually works for
most compilers, but it is known to break if multiple inheritance is
used anywhere in the object hierarchy.

This patch introduces the cxx_header attribute to Python SimObject
definitions, which should be used to specify a header to include in
the SWIG interface. The header should include the declaration of the
wrapped object. We currently don't enforce header the use of the
header attribute, but a warning will be generated for objects that do
not use it.

12 years agopci: Make Python wrapper cast to the right type
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
pci: Make Python wrapper cast to the right type

The PCI base class is PciDev and not PciDevice, which is used by the
Python world. Make sure this is reflected in the wrapper code.

12 years agomips: Remove unused Python file
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
mips: Remove unused Python file

Remove BISystem.py, BareIronMipsSystem is already implemented in
MipsSystem.py.

12 years agodev: Add missing inline declarations
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
dev: Add missing inline declarations

12 years agobase: Add missing header file to addr_range.hh.
Andreas Sandberg [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
base: Add missing header file to addr_range.hh.

12 years agom5: Expose m5 pseudo-instructions to C/C++ via a static library
James Clarkson [Tue, 9 Oct 2012 11:58:25 +0000 (12:58 +0100)]
m5: Expose m5 pseudo-instructions to C/C++ via a static library

Updated the util/m5/Makefile.arm so that m5op_arm.S is used to create
a static library - libm5.a. Allowing users to insert m5
psuedo-instructions into their applications for fine-grained
checkpointing, switching cpus or dumping statistics. e.g.

#include <m5op.h>

void foo(){
...
m5_reset_stats(<delay>,<period>)
m5_work_begin(<workid>,<threadid>);
...
m5_work_end(<workid>,<threadid>);
m5_dump_stats(<delay>,<period>);
}

12 years agoARM: dump stats and process info on context switches
Dam Sunwoo [Fri, 2 Nov 2012 16:32:01 +0000 (11:32 -0500)]
ARM: dump stats and process info on context switches

This patch enables dumping statistics and Linux process information on
context switch boundaries (__switch_to() calls) that are used for
Streamline integration (a graphical statistics viewer from ARM).

12 years agobase: Fix a few incorrectly handled print format cases
Chander Sudanthi [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
base: Fix a few incorrectly handled print format cases

This patch ensures cases like %0.6u, %06f, and %.6u are processed correctly.
The case like %06f is ambiguous and was made to match printf.  Also, this patch
removes the goto statement in cprintf.cc in favor of a function call.

12 years agobase: split out the VncServer into a VncInput and Server classes
Chander Sudanthi [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
base: split out the VncServer into a VncInput and Server classes

This patch adds a VncInput base class which VncServer inherits from.
Another class can implement the same interface and be used instead
of the VncServer, for example a class that replays Vnc traffic.

--HG--
rename : src/base/vnc/VncServer.py => src/base/vnc/Vnc.py
rename : src/base/vnc/vncserver.cc => src/base/vnc/vncinput.cc
rename : src/base/vnc/vncserver.hh => src/base/vnc/vncinput.hh

12 years agoISA: generic Linux thread info support
Dam Sunwoo [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
ISA: generic Linux thread info support

This patch takes the Linux thread info support scattered across
different ISA implementations (currently in ARM, ALPHA, and MIPS), and
unifies them into a single file.

Adds a few more helper functions to read out TGID, mm, etc.

ISA-specific information (e.g., ALPHA PCBB register) is now moved to
the corresponding isa_traits.hh files.

12 years agosim: Fix as issue where exit events on instr queues are used after freed.
Ali Saidi [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
sim: Fix as issue where exit events on instr queues are used after freed.

12 years agoo3: Fix a couple of issues with the local predictor.
Mrinmoy Ghosh [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
o3: Fix a couple of issues with the local predictor.

Fix some issues with the local predictor and the way it's indexed.

12 years agoPartly revert [4f54b0f229b5] and move draining to m5.changeToTiming
Andreas Sandberg [Fri, 2 Nov 2012 16:32:00 +0000 (11:32 -0500)]
Partly revert [4f54b0f229b5] and move draining to m5.changeToTiming

Changeset 4f54b0f229b5 removed the call to doDrain in changeToTiming
based on the assumption that the system does not need draining when
running in atomic mode. This is a false assumption since at least the
System class requires the system to be drained before it allows
switching of memory modes. This patch reverts that part of the
changeset.

12 years agomem: Fix typo in port comments
Andreas Hansson [Wed, 31 Oct 2012 13:28:23 +0000 (09:28 -0400)]
mem: Fix typo in port comments

This patch merely fixes a few typos in the port comments.

12 years agostats: Update stats for fixed simple-atomic-mp config
Andreas Hansson [Wed, 31 Oct 2012 12:39:45 +0000 (08:39 -0400)]
stats: Update stats for fixed simple-atomic-mp config

This patch updates the stats for the regressions that were affected by
the typo in the simple-atomic-mp configuration.

12 years agoconfig: Fix a typo in the simple-atomic-mp configuration
Andreas Hansson [Wed, 31 Oct 2012 12:39:43 +0000 (08:39 -0400)]
config: Fix a typo in the simple-atomic-mp configuration

This patch fixes a minor typo that managed to sneak into the
simple-atomic-mp regression configuration.

12 years agostats: Update stats for unified cache configuration
Andreas Hansson [Tue, 30 Oct 2012 13:35:32 +0000 (09:35 -0400)]
stats: Update stats for unified cache configuration

This patch updates the stats to reflect the changes in the L2 MSHRs,
as the latter are now uniform across the regressions.

12 years agoconfig: Unify caches used in regressions and adjust L2 MSHRs
Andreas Hansson [Tue, 30 Oct 2012 11:44:08 +0000 (07:44 -0400)]
config: Unify caches used in regressions and adjust L2 MSHRs

This patch unified the L1 and L2 caches used throughout the
regressions instead of declaring different, but very similar,
configurations in the different scripts.

The patch also changes the default L2 configuration to match what it
used to be for the fs and se scripts (until the last patch that
updated the regressions to also make use of the cache config). The
MSHRs and targets per MSHR are now set to a more realistic default of
20 and 12, respectively.

As a result of both the aforementioned changes, many of the regression
stats are changed. A follow-on patch will bump the stats.

12 years agoregressions: update stats for ruby fs test
Nilay Vaish [Sat, 27 Oct 2012 21:05:06 +0000 (16:05 -0500)]
regressions: update stats for ruby fs test

12 years agoruby: set the is_icache param for caches
Malek Musleh [Sat, 27 Oct 2012 21:04:30 +0000 (16:04 -0500)]
ruby: set the is_icache param for caches
This patch sets the is_icache param for the L1 caches used in
the MESI and the MOESI CMP directory protocols.

12 years agoRuby: Use block size in configuring directory bits in address
Jason Power ext:(%2C%20Joel%20Hestness%20%3Chestness%40cs.wisc.edu%3E) [Sat, 27 Oct 2012 21:01:09 +0000 (16:01 -0500)]
Ruby: Use block size in configuring directory bits in address
This patch replaces hard coded values used in Ruby's configuration files
for setting directory bits with values based on the block size in use.

12 years agoconfig: Add a check for fastmem only used with Atomic CPU
Andreas Hansson [Fri, 26 Oct 2012 10:42:45 +0000 (06:42 -0400)]
config: Add a check for fastmem only used with Atomic CPU

This patch adds an additional check to ensure that the fastmem option
is only used if the system is using the Atomic CPU.

12 years agoconfig: Remove unused mem_size in fs.py
Andreas Hansson [Fri, 26 Oct 2012 10:42:43 +0000 (06:42 -0400)]
config: Remove unused mem_size in fs.py

This patch removes a segment of dead code that is never used.

12 years agoconfig: Fix the cache class naming in regression scripts
Andreas Hansson [Fri, 26 Oct 2012 10:42:42 +0000 (06:42 -0400)]
config: Fix the cache class naming in regression scripts

This patch unifies the naming of the default L1 and L2 caches in the
regression configs to be in line with what is used in the se and fs
scripts.

12 years agostats: Update the stats to reflect the 1GHz default system clock
Andreas Hansson [Thu, 25 Oct 2012 17:15:59 +0000 (13:15 -0400)]
stats: Update the stats to reflect the 1GHz default system clock

This patch updates the stats to reflect the change in the default
system clock from 1 THz to 1GHz. The changes are due to the DMA
devices now injecting requests at a lower pace.