Somehow one of our boards has 9-space indentation.  Fix this.

CRs-Fixed: 1013327
Change-Id: Id0d4f51105e954567c745a64d35367159d50a6c4

Update the cache dump table version to be the supported version,
and Use the correct function to do ICache parsing.

CRs-Fixed: 1013312
Change-Id: I77f1e5edb106a2d906856f82295c6f9e4b573008

Timerlist structure changed for kernel 4.2. Added extra
parsing function to get timerlist dumps for kernel 4.2
Commit ID for kernel changes:

1dabbcec2: timer: Use hlist for the timer wheel hash buckets
0eeda71bc: timer: Replace timer base by a cpu index

CRs-Fixed: 981273
Change-Id: I163ef37d118914486c6e9b77ee80b2317c37c5b6

Debug image parsing currently includes QDSS ETM and ETR binary data
parsing by default. However, these two parsing operations can take a
significant amount of time if either of their respective data buffers
are very large. Furthermore, this binary data is not needed for most
common debug image parsing use cases.

Add an argument to optionally disable QDSS ETM and ETR binary data
parsing when debug image parsing has also been selected.

Change-Id: Iab25c15d00dde1bb5dfafb3149e6e55760fd6291

As RSS values are read as unsigned, will get a large possive value.

Change-Id: Ieedc4223293cce40c295c692d32b6bb8498bafa1

Add module to get information from MDP dumps, which can occur under
situations where MDP cannot recover.

Change-Id: I8ecdced37206947c434c7ddf4d7d61979c9cf92f

Print timestamps in seconds to make it easily readable.

Change-Id: Ie50f8b6e0adf24e5a9b3b18c6118000348b29507

Print total, free, slab, SHMEM, RSS memory details.

Change-Id: I63b3faf2375ebb6631309fd9cf00fd080ccfe5b4

Allow easily iterating through the list by implementing ListWalker
iterator functions.

Change-Id: Idba8cbcb8e3399e6e33f99b993554aeef7420a53

Describe the cache format used in msmcobalt.

CRs-Fixed: 983532
Change-Id: I4dc88fc7f29770e892a29a7e48e8a79d883c9b06

Describe the cache format used for 8937, 8953 and 8917.

Change-Id: Ie877173bc0fa6ea01e7daf1825720c5601a6f82a

Add support for detecting msm8917 target
and parsing the ramdumps.

Change-Id: I8014abc0f52a459bdcb7d94b09f4826ded74542c

Add support for detecting APQ8953 target
and parsing the ramdumps.

Change-Id: I9529d2187ce70401d57ca14c4294c4a7cee2c810

Currently we split each config line on the equals sign, assuming that
there's only one equals sign there.  This isn't a valid assumption for
cases like:

    CONFIG_CMDLINE="console=ttyAMA0"

Fix this by splitting at the first equals sign, rather than splitting
at *all* equals signs.

Change-Id: I01e44fe2e9a0f09a8bebdd76715002f6b12cf0b5

Add support for detecting msm8953 target and parsing the ramdumps.

Change-Id: Ie0ddf18e46ab20d0419609c1f84f2d5b0667be9c

Add '--kaslr-offset' option.  This specifies the offset for loading symbols
for a KASLR-enabled kernel.

Change-Id: I208d964ea3ca58536a0b80a32b4e334c73aae534

The register_parser decorator function is one of the most important
public functions in the codebase.  Improve its documentation by adhering
to Python docstring standards, including parameter documentation.

Change-Id: Ibbde495e799dcd550810635e591b24e9eba8cb28

Some of our module documentation is a bit lacking.  Fix this by filling
in some docstrings and add then adding some pointers to existing
modules/functions in the docs.

Change-Id: Ibf714225f1fc185be916e514861540fd98cb2de5

Support is required for version 3 of IPC Logging, which increases the
maximum size of a logging context name from 20 to 32.

Add this support.

Change-Id: Id7c4b30efd2499cbad99db5d00a5008198047c72

Along with the unpack() functions which are present for parsing, the
classes which parse individual log messages also have pack() functions
which are used for recreating the log messages in their binary form.
These functions are used for unit testing.

In the TSVQTimer pack() function, incorrect math is performed on the
result before it is returned. In the LogMessageString pack() function, a
QTimer object is manipulated before it is created.

Fix these errors.

Change-Id: I5c288ddd53e468368c3663bdc9c1134a5f694865

Parse out the APM state that has been configured so that it can
be compared against the VDD supply voltage to ensure correct
functionality.

Change-Id: Iabf78a6799015861379a39d93e59289a489e1510

Some of the documentation in register.py doesn't render nicely when
built with Sphinx.  Fix this.

Change-Id: Ifdb0da6c21f092b58c51be8f566a7a4e5cab984b

There's a comment in gdbmi that doesn't adhere to the canonical Python
prompt format.  Fix this.

Change-Id: I0a369669cc4c5aa5ac318da0f4949aba84bed02d

Add support for detecting 8937 target and parsing the ramdumps.

Change-Id: If487b01d02c634b8e9788ada07d408cd516af556

Most read functions that take a virtual address also understand string
variable names, for convenience.  Add this support to the `hexdump'
function as well.

Change-Id: I773992ba3793a68226b49525b66141b021924387

Although the `--shell' option can currently be used for ad-hoc
debugging, it's not ideal for scripting since it prints out a bunch of
other stuff during initialization.  Add a dedicated `--eval' option that
executes arbitrary Python code without printing anything else.

Change-Id: I9e24e6d140dd49208b850b2a17ac2f599a35657b

Most of the time 100 is a good default max length when reading any old
C-string.  Use it.

Change-Id: I1805b0b4b2b9bc18c06f4ca0e7cd6cd70b854620

GdbMI.symbol_at returns a string, not a GdbSymbol object, so the extra
field traversal is unnecessary.

Change-Id: I85acd83657e4b6055bd1f7945da73dcf9d4839f1

The name symbol_at gives a better idea of what the function does, and it
more closely mirrors the corresponding gdbmi function.

Change-Id: If2f3df505bc129a3b963ea363e4c544599626948

In lieu of a more generic way of reading fields from structures,
introduce another specialized structure field reader for reading C
strings.

Change-Id: I6aba392f59179fa70a065a08f19a694e87f683dd

Rename addr_lookup to address_of to match common terminology more
closely.

Change-Id: I7fc744ce88d1f8880f13dfbf83fb0efcbe9ba0f2

The cmdParse() function has excessive cyclomatic complexity.

Reduce the cyclomatic complexity by writing helper functions to obtain
version info, handle processing log contexts, and handle processing log
pages.

Change-Id: I15b8da9b20bb7b20daad3f0f0ac460b6cfe8ad89

LogPage_v1.sortAndLink() has excessive cyclomatic complexity. Reduce the
complexity by refactoring sortAndLink() into several smaller functions.

This commit is to be squashed into the commit containing all other
style fixes.

Change-Id: I52ac6027e5f9d45af7d44d4b24beff7a28baea05

The LogPage_v1's class documentation says that the class represents a
version 0 IPC log page, but the class represents a version 1 IPC log
page.

Correct the error.

Change-Id: If6ca9a66d486f36d5a2078ff7f78850c35ad65f3

IPC Logging currently displays the kernel log timestamp, which is based
on a timer that does not increase when the Apps processor is in suspend.
This means that the kernel log timestamp "loses time" and cannot be
correlated with events on other processors. The source of the QTimer
timestamp is always running and has the same source other logs in the
system, which makes it convenient to correlate logs across the system.

Update the IPC Logging extraction script to handle version 2 of IPC
Logging, which includes the QTimer timestamp.

Change-Id: Ia4534a53012e29efe5ac647b6212530f84c07f46

The coding style of the IPC Logging extraction script is not compliant
with the Python coding style guide (PEP 0008). In order to be merged
into the Linux Ram Dump Parser, the code must be made compliant with PEP
0008.

Bring the code into compliance with PEP 0008.

Change-Id: Idf587a55670a5b30c7668c78b1625100d6128c9c

Pointers can be either 32-bits or 64-bits, depending on the
architecture.  Add an accessor function to abstract this away.

Change-Id: I64e2c18c85ec3d9ca7304163dc5a42edc83f1443

A few reader functions currently assume that a virtual address (a
number) was passed in, but those functions also support reading variable
names.  Resolve the address or name to a virtual address before doing
arithmetic on it.

Change-Id: I55aa649ff203926565f8eaf3cb234677ef6b6104

Some of the return sequences in the memory accessor code are
unnecessarily complex.  Simplify them.

Change-Id: Iefe489b3ce4a4591574a2cc8d51aa23ce425b2b1

Currently there are "trace" print statements scattered throughout the
code.  These make the code harder to follow and aren't terribly useful.
Remove them in favor of on-demand instrumentation or more sophisticated
debugging methods (like a debugger).

Change-Id: I70a91b8dca34566d48ed1f4902cba5b9f9b4de40