From: Alex Bennée alex.ben...@linaro.org
This allows the perf tool to map samples to each individual translation
block. This could be expanded for user space but currently it gives
enough information to find any hotblocks by other means.
---
qemu-options.hx | 10 ++
tcg/tcg.c | 21
From: Alex Bennée alex.ben...@linaro.org
Hi,
While working on aarch64 stuff I've made the following changes in my
local tree to help me debug. The first is a simple documentation fix.
The second is very helpful when your debugging missing instructions
while several shells deep in a build system.
From: Alex Bennée alex.ben...@linaro.org
This ensures the code generation debug code will honour -dfilter if set.
diff --git a/cpu-exec.c b/cpu-exec.c
index 0914d3c..9aa3f3f 100644
--- a/cpu-exec.c
+++ b/cpu-exec.c
@@ -610,7 +610,7 @@ int cpu_exec(CPUArchState *env)
next_tb
From: Alex Bennée alex.ben...@linaro.org
When debugging stuff that occurs over several forks it would be useful
not to keep overwriting the one logfile you've set-up. This allows a
simple %d to be included once in the logfile parameter which is
substituted with getpid().
Signed-off-by: Alex
From: Alex Bennée alex.ben...@linaro.org
This doesn't just dump CPU state on translation but on every block
entrance.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
diff --git a/qemu-log.c b/qemu-log.c
index 797f2af..35bbb56 100644
--- a/qemu-log.c
+++ b/qemu-log.c
@@ -105,7 +105,7 @@ const
From: Alex Bennée alex.ben...@linaro.org
When debugging big programs or system emulation sometimes you want both
the verbosity of cpu,exec et all but don't want to generate lots of logs
for unneeded stuff. This patch adds a new option -dfilter which allows
you to specify interesting address
From: Alex Bennée alex.ben...@linaro.org
This makes the UST backend pay attention to the format string arguments
that are defined when defining payload data. With this you can now
ensure integers are reported in hex mode if you want.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
---
From: Alex Bennée alex.ben...@linaro.org
This makes the UST backend pay attention to the format string arguments
that are defined when defining payload data. With this you can now
ensure integers are reported in hex mode if you want.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
---
v2
-
From: Alex Bennée alex.ben...@linaro.org
Hi,
These two patches have been sitting in my personal tree for a while
and I thought it was worth soliciting feedback as to their wider usefulness.
The first is simply an attempt to make tcg abort failures a little
less terse.
The second I found useful
From: Alex Benn??e alex.ben...@linaro.org
The helpers are useful for debugging if you want to inspect interim
values of tcg temp variables while executing TCG generated code. This is
an alternative to tracing with logs or inspecting with GDB.
The functions do take format strings but to prevent
From: Alex Bennée alex.ben...@linaro.org
There are times the tcg aborts with a fatal but terse error which isn't
overly helpful. This adds an alternative macro that can be used to show
a little more helper information when an abort occurs.
diff --git a/tcg/i386/tcg-target.c
From: Alex Bennée alex.ben...@linaro.org
No changes from the original version posted except to add Stefan's
reviewed-by tags. I thought it was worth submitting this before we
froze for 2.0!
The following changes since commit be813ef02d18ba58e1ff32f1706bcacb88f1f764:
Merge remote-tracking
From: Alex Bennée alex.ben...@linaro.org
The current builds don't include all the features which are
auto-detected and then disabled when the appropriate test packages don't
exist. I've added another target that enables all known additional
packages for increased coverage. I didn't add it to the
From: Alex Bennée alex.ben...@linaro.org
Purely cosmetic but satisfies my OCD.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
diff --git a/.travis.yml b/.travis.yml
index 1d78421..0dbf2da 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,23
From: Alex Bennée alex.ben...@linaro.org
This build was disabled while the lttng tracing was broken. Stefan has
recently submitted a pull request with it re-enabled.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
diff --git a/.travis.yml
From: Alex Bennée alex.ben...@linaro.org
I'm trying to avoid spamming the IRC channel (not overly likely as
builds take a while). So failure will always be reported but if the
build continues to work then the IRC notifications will be quiet.
Note any GitHub based repository with Travis enabled
From: Alex Bennée alex.ben...@linaro.org
Hi,
These two patches have been sitting in my personal tree for a while
and I thought it was worth soliciting feedback as to their wider usefulness.
The first is simply an attempt to make tcg abort failures a little
less terse.
The second I found useful
From: Alex Benn??e alex.ben...@linaro.org
The helpers are useful for debugging if you want to inspect interim
values of tcg temp variables while executing TCG generated code. This is
an alternative to tracing with logs or inspecting with GDB.
The functions do take format strings but to prevent
From: Alex Bennée alex.ben...@linaro.org
Purely cosmetic but satisfies my OCD.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
diff --git a/.travis.yml b/.travis.yml
index 1d78421..0dbf2da 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,23
From: Alex Bennée alex.ben...@linaro.org
No changes from the original version posted except to add Stefan's
reviewed-by tags. I thought it was worth submitting this before we
froze for 2.0!
The following changes since commit be813ef02d18ba58e1ff32f1706bcacb88f1f764:
Merge remote-tracking
From: Alex Bennée alex.ben...@linaro.org
I'm trying to avoid spamming the IRC channel (not overly likely as
builds take a while). So failure will always be reported but if the
build continues to work then the IRC notifications will be quiet.
Note any GitHub based repository with Travis enabled
From: Alex Bennée alex.ben...@linaro.org
The current builds don't include all the features which are
auto-detected and then disabled when the appropriate test packages don't
exist. I've added another target that enables all known additional
packages for increased coverage. I didn't add it to the
From: Alex Bennée alex.ben...@linaro.org
There are times the tcg aborts with a fatal but terse error which isn't
overly helpful. This adds an alternative macro that can be used to show
a little more helper information when an abort occurs.
diff --git a/tcg/i386/tcg-target.c
From: Alex Bennée alex.ben...@linaro.org
This build was disabled while the lttng tracing was broken. Stefan has
recently submitted a pull request with it re-enabled.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
diff --git a/.travis.yml
From: Alex Bennée alex.ben...@linaro.org
This build was disabled while the lttng tracing was broken. Stefan has
recently submitted a pull request with it re-enabled.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
diff --git a/.travis.yml b/.travis.yml
index 286cf62..1d78421 100644
---
From: Alex Bennée alex.ben...@linaro.org
I'm trying to avoid spamming the IRC channel (not overly likely as
builds take a while). So failure will always be reported but if the
build continues to work then the IRC notifications will be quiet.
Note any GitHub based repository with Travis enabled
From: Alex Bennée alex.ben...@linaro.org
Purely cosmetic but satisfies my OCD.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
diff --git a/.travis.yml b/.travis.yml
index 1d78421..0dbf2da 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,23 +14,23 @@ env:
- GUI_PKGS=libgtk-3-dev
From: Alex Bennée alex.ben...@linaro.org
Hi,
I've now enabled Travis testing on the official QEMU GitHub mirror.
Of the following patches two expand the testing, one is a cosmetic
whitespace fix and the final one enabled IRC notification when the
build fails. This is potentially controversial
From: Alex Bennée alex.ben...@linaro.org
The current builds don't include all the features which are
auto-detected and then disabled when the appropriate test packages don't
exist. I've added another target that enables all known additional
packages for increased coverage. I didn't add it to the
Hi,
While working on my aarch64 work I found setting up binfmt was more of
a pain than it could have been. Specifically:
* hard-coded for /usr/local installs
* no help
* no error checking
Tellingly the script doesn't seem to be used by the distros who have
rolled their own binfmt_misc stuff
From: Alex Bennée alex.ben...@linaro.org
I was looking to set-up for development but found the script made some
hard-coded assumptions. It doesn't seem the script is used by the
distros but if it had a little more love maybe it would be ;-)
* Add usage() instructions
* Move all registering to a
/python
+#
+# binfmt check script
+#
+# Copyright 2014 Linaro
+#
+# Authors:
+# Alex Bennee alex.ben...@linaro.org
+#
+# This work is licensed under the terms of the GNU GPL, version 2. See
+# the COPYING file in the top-level directory.
+
+import os
+import re
+import binascii
+
+re_int = re.compile
From: Alex Bennée a...@bennee.com
This adds a build matrix definition for travis-ci.org continuous
integration service. It is usable on any public repository hosted on
GitHub. Once you have created an account signed into Travis you can
enable it on selected projects via travis-ci.org/profile.
From: Alex Bennée a...@bennee.com
Hi,
I think this patch is ready to be merged via trivial. While I was
testing it last night it caught yet another regression
(register_subpage: Assertion, since fixed) so I think it proves it's
worth with that alone.
Since v5
- Forced the python to 2.4
From: Alex Bennée a...@bennee.com
Commit 9b8c69243 (since reverted) broke the ability to boot the kernel
as the value returned by unassigned_mem_read returned non-zero and left
the kernel looping forever waiting for it to change (see
integrator_led_set in the kernel code).
Relying on a varying
From Alex Bennée alex.ben...@linaro.org # This line is ignored.
Hi,
I finally got a chance to follow up on the review comments from Peter
and Andreas.
Changes for v3:
* Moved into hw/arm/integrator_debug.c
* Expanded QOM symbol to INTEGRATOR_DEBUG, moved to header
* Use __func__, HWADDR_PRIx
From: Alex Bennée a...@bennee.com
Commit 9b8c69243 (since reverted) broke the ability to boot the kernel
as the value returned by unassigned_mem_read returned non-zero and left
the kernel looping forever waiting for it to change (see
integrator_led_set in the kernel code).
Relying on a varying
From: Alex Bennée a...@bennee.com
This adds a build matrix definition for travis-ci.org continuous
integration service. It is usable on any public repository hosted on
GitHub. Once you have created an account signed into Travis you can
enable it on selected projects via travis-ci.org/profile.
Hi Anthony,
Here is my first pull request to add a simple .travis.yml profile to the
code base. I'm hoping to expand the range of testing once this is merged
(the tcg code gen tests are next on my list). However as it stands this
already catches build failures and regressions.
Since v4:
-
From: Alex Bennée a...@bennee.com
Currently master is broken as the wanted commit doesn't exist in
qemu's mirror of SeaBIOS.
---
.gitmodules | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.gitmodules b/.gitmodules
index d7e3f3c..d5d5417 100644
--- a/.gitmodules
+++
From: Alex Bennée a...@bennee.com
This adds a build matrix definition for travis-ci.org continuous
integration service. It is usable on any public repository hosted on
GitHub. Once you have created an account signed into Travis you can
enable it on selected projects via travis-ci.org/profile.
Hi,
While testing after Peter's review comments Travis detected a
regression in the tree due to the updating of SeaBIOS (also found by a
bunch of others in the following days) which I take to be a good
indication of the usefulness of these tests.
v4:
- split debug build for with/without TCG
From: Alex Bennée a...@bennee.com
This adds a build matrix definition for travis-ci.org continuous
integration service. It is usable on any public repository hosted on
GitHub. Once you have created an account signed into Travis you can
enable it on selected projects via travis-ci.org/profile.
Assuming there are no further comments from this submission I shall
likely make my next posting a PULL request.
v3
- re-based on current origin/master
- squashed all .travis commits and re-worded commit
- added clang builds
- disabled make check for ftrace back-end
- fixed broken build
Hi,
I've updated the .travis.yml following feedback from Stefan to greatly
increase the number of compiles it does. In the process:
* Disabled ust backend (horribly broken)
* Found gov/gprof builds don't enforce --disable-pie
* Fixed a minor compiler warning when stderr/ftrace back-end enabled
From: Alex Bennée a...@bennee.com
While QEMU already has various continuous integration set-ups in
buildbot and commercial Jenkins setups there is some value in supporting
travis-ci as well. It is well integrated into GitHub work flow and will
trigger on all branch pushes and pull requests. This
From: Alex Bennée a...@bennee.com
I've made the definitions more modular to allow for greater build
control in each part of the matrix. I've confined the additional stuff
to x86/x86_64 for convenience.
---
.travis.yml | 38 +-
1 file changed, 37 insertions(+),
From: Alex Bennée a...@bennee.com
This only showed up when compiling with
--enable-trace-backend=stderr|ftrace at which point the compiler
complains with the following:
block/stream.c: In function ‘stream_run’:
block/stream.c:141:22: error: ‘copy’ may be used uninitialized in this function
From: Alex Bennée a...@bennee.com
Commit 9b8c69243 broke the ability to boot the kernel as the value
returned by unassigned_mem_read returned non-zero and left the kernel
looping forever waiting for it to change (see integrator_led_set in
the kernel code).
Relying on a varying implementation
From: Alex Bennée a...@bennee.com
Commit 9b8c69243 broke the ability to boot the kernel as the value
returned by unassigned_mem_read returned non-zero and left the kernel
looping forever waiting for it to change (see integrator_led_set in
the kernel code).
Relying on a varying implementation
Hi,
I mistakenly appended this to the previous patch revision. I'm now
sending for further review.
Since v1:
- Updated with Peter Maydell's review comments.
--
Alex Bennée
From: Alex Bennée a...@bennee.com
Commit 9b8c69243 broke the ability to boot the kernel as the value
returned by unassigned_mem_read returned non-zero and left the kernel
looping forever waiting for it to change (see integrator_led_set in
the kernel code).
Relying on a varying implementation
Hi,
I've applied the review comments from Peter.
Alex.
From: Alex Benn??e a...@bennee.com
Commit 9b8c69243 broke the ability to boot the kernel as the value
returned by unassigned_mem_read returned non-zero and left the kernel
looping forever waiting for it to change (see integrator_led_set in
the kernel code).
Relying on a varying implementation
From: Alex Bennée a...@bennee.com
While QEMU already has various continuous integration set-ups in
buildbot and commercial Jenkins setups there is some value in supporting
travis-ci as well. It is well integrated into GitHub work flow and will
trigger on all branch pushes and pull requests. This
55 matches
Mail list logo
