l...@redhat.com writes:
On 10/01/2013 04:37 PM, Alex Bennée wrote:
l...@redhat.com writes:
Hello:
In the daily qemu.git autotest run, I started to see the following
problem while checking out the qemu.git tree and checking out the
submodules:
Oh I just noticed that. Is this Stefan's

peter.mayd...@linaro.org writes:
On 3 October 2013 17:58, alex.ben...@linaro.org wrote:
From: Alex Bennée a...@bennee.com
Currently master is broken as the wanted commit doesn't exist in
qemu's mirror of SeaBIOS.
snip
This is wrong. QEMU's submodules should all point to
git.qemu.org

constant is used
twice and should it ever be changed there is a risk of one being missed
(although conceivably thumb encoding could be different?).
I appreciate the disassembler code is already a mass of magic constants
so it's not a major thing for me.
Cheers,
--
Alex Bennée

r...@twiddle.net writes:
There are free scheduling slots between the sequence of
comparison instructions. This requires changing the
register in use to avoid conflict with those compares.
Signed-off-by: Richard Henderson r...@twiddle.net
snip
Looks good to me.
Reviewed-by: Alex Bennée

.. Perhaps a few simple
examples could be added to the document so people can at least confirm
everything is working as it should be. Otherwise I'm happy.
Reviewed-by: Alex Bennée a...@bennee.com
Mohamad
Mohamad Gebai (3):
Fix configure script for LTTng 2.x
Modified the tracetool framework

peter.mayd...@linaro.org writes:
On 16 September 2013 13:50, alex.ben...@linaro.org wrote:
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

mohamad.ge...@polymtl.ca writes:
On 13-10-16 08:05 AM, Alex Bennée wrote:
Running this gives me:
quote
UST events:
-
None
/quote
Before or after running qemu. What is the mechanism lttng expects to
find out all these events?
Either the user should belong the group tracing

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

peter.mayd...@linaro.org writes:
On 18 October 2013 12:45, alex.ben...@linaro.org wrote:
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

these
softfloat functions against unintentional breakage? It would certainly
be worthwhile as soon as multiple arches use these functions as float
errors are often subtle and hard to track down.
--
Alex Bennée

peter.mayd...@linaro.org writes:
On 25 October 2013 12:34, Alex Bennée alex.ben...@linaro.org wrote:
Is it worth adding some sort of test into make check to defend these
softfloat functions against unintentional breakage? It would certainly
be worthwhile as soon as multiple arches use

mohamad.ge...@polymtl.ca writes:
Signed-off-by: Mohamad Gebai mohamad.ge...@polymtl.ca
---
snip
Tested on Ubuntu 12.04 with and without the LTTNG PPA and seems to work
well enough.
Reviewed-by: Alex Bennée a...@bennee.com
Yes, the bug is actually only in the Ubuntu package (missing

of issues while
building. They are mostly -Werror bits and pieces which I can work
through but as it doesn't build with the default QEMU build options I
thought I'd better check if that is the correct latest/greatest version
of the patch set.
Regards,
--
Alex Bennée

stefa...@gmail.com writes:
On Wed, Sep 18, 2013 at 03:35:07PM +0100, Alex Bennée wrote:
stefa...@gmail.com writes:
Looks useful. To get more code coverage, flesh out the build
environment:
Thanks. Am I dreaming or are there some system image tests somewhere as well?
Anthony

;-)
I'm currently pondering what the best way of supporting system images
(i.e. kernel+rootfs) would be to make system regression testing easier.
Unfortunately those images would be far too large to carry in the repo
although there may be some sub-module annex type thing I could try.
--
Alex Bennée

kw...@redhat.com writes:
Am 20.09.2013 um 11:10 hat Alex Bennée geschrieben:
jc...@redhat.com writes:
snip
I'm currently pondering what the best way of supporting system images
(i.e. kernel+rootfs) would be to make system regression testing easier.
Unfortunately those images would

stefa...@redhat.com writes:
On Mon, Sep 23, 2013 at 05:07:30PM +0100, alex.ben...@linaro.org wrote:
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:
nsip
Stefan Weil

of stable API. I hunted around a
bit trying to get it working but realised the script needs fixing up as
well so gave up.
Really ust just needs to be ripped out for now unless someone else wants
to dig into to supporting multiple versions painlessly.
--
Alex Bennée

petar.jovano...@imgtec.com writes:
From: Alex Bennée [alex.ben...@linaro.org]
snip
There is an awful lot of similarity between a lot of the structures
while not being totally identical. Given the syscall munging is common
is there not an argument

patch has been started during Linaro Connect
by me and Alex Bennee.
snip
With the proviso of Richard's decode comment you can add:
Reviewed-by: Alex Bennée a...@bennee.com
Signed-of-by: Alex Bennée a...@bennee.com
--
Alex Bennée

(register) canonical for ORR (rn=31 shift_amount=0), and
MVN
(register) canonical for ORN (rn=31 shift_amount=0), and both therefore
also
worth a special case?
I suspect I'm being overly cheeky to expect the optimiser to detect and
optimise for that case as the ZR is a const ;-)
Cheers,
--
Alex

thought Xin was talking about caching translations between invocations
of QEMU.
I suspect address space randomisation would be another wrinkle in the
side of any such scheme though.
thanks
-- PMM
--
Alex Bennée
QEMU/KVM Hacker for Linaro

ldst patches did get a little gnarly thanks to the
structure of the original decoder and I wanted to avoid things like imm
fields being optionally chopped apart depending on the decode.
--
Alex Bennée
QEMU/KVM Hacker for Linaro

to issue a warning (when not --quiet) when it detects
creation of an image on a partition where performance may not be as
expected due to COW behaviour.
Cheers,
--
Alex Bennée
QEMU/KVM Hacker for Linaro

the information?
The hard dependency on glib has been in quite some time
(e18df14185e817ba735bce57ecdef9a55fb3d093) so I don't think you can
build without it.
However it should be possible to build glib for android and bundle it
with your qemu if needed.
Thanks,
Tervel
--
Alex Bennée
QEMU/KVM

r...@twiddle.net writes:
On 12/10/2013 06:16 AM, Alex Bennée wrote:
However my preference unless there is a strong objection would be to
clean that up in later patches. For one thing the more instructions each
patch handles the longer it takes to run the instruction validation on
the rather

,then Service Hooks and selecting Travis.
Basically you just need to go to http://travis-ci.org and click the
Sign in with GitHub link and your up and running. It's super easy.
--
Alex Bennée
QEMU/KVM Hacker for Linaro

peter.mayd...@linaro.org writes:
On 18 December 2013 11:32, Alex Bennée alex.ben...@linaro.org wrote:
Now we have a .travis.yml merged into master can we enable Travis to run
in the QEMU mirror on github (https://github.com/qemu/qemu)? I'm happy
to talk who ever controls that repo through

Hi,
Not too much has changed:
* added a review tag
* fixed up review comments
* added some notes about benchmark results
* added a patch to disable ARMv5 in AArch64 build
The most important thing is I've measured a 25-30% improvement in
kernel and android boot time.
Alex Bennée (5

According to the ARM ARM we weren't correctly flushing the TLB entries
where bits 63:56 didn't match bit 55 of the virtual address. This
exposed a problem when we switched QEMU's internal TARGET_PAGE_BITS to
12 for aarch64.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
Reviewed-by: Peter

If you attempt to run a system image which uses 1k pages in the
qemu-system-aarch64 build it will fail thanks to the change to 12 bit
pages. The boards are still available for the qemu-system-arm build.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
diff --git a/default-configs/aarch64

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 Bennée alex.ben...@linaro.org
Reviewed

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.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
---
v2:
- hoist up into translate-all.c

-0x9000,0xffc8+0x200,...
Then logging code can use the new qemu_log_in_addr_range() function to
decide if it will output logging information for the given range.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
v2
- More clean-ups to the documentation
v3
- re-base
- use GArray

This ensures the code generation debug code will honour -dfilter if set.
For the exec tracing I've added a new inline macro for efficiency's
sake. I've not touched CPU_LOG_TB_OP as this is buried in each
individual target.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
v2

to compliment the tc_ptr (and the subject pc,
block size). This is set on code generation and then accessed directly
by all the people that need it.
I've also cleaned up some comments and removed un-used return variables.
Signed-off-by: Alex Bennée alex.ben...@linaro.org
---
v1
- checkpatch fixes
diff

From: Peter Maydell peter.mayd...@linaro.org
Make qemu_log_mask() a macro which only calls the function to
do the actual work if the logging is enabled. This avoids making
a function call in possible fast paths where logging is disabled.
Signed-off-by: Peter Maydell peter.mayd...@linaro.org

the glue magic (not your fault
;-) we have in QEMU. However wouldn't it be simpler for the helper
pre-amble code to ensure the subject pc is updated in the CPU
environment?
Can QEMU only rectify the processor state from a TranlationBlock tc address?
--
Alex Bennée

{
+replay_state.skipping_instruction = 1;
+}
I'm not quite following what instructions_count means here. Is it an
actual count of instructions executed? Can you add a comment to the
header where it's defined?
--
Alex Bennée

Alex Bennée writes:
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.
snip
Ping Stefan, can this one at least be slurped up into your tracing tree

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
- remove silly debug statements
v3
- fix

Hi Stefan,
As v3 posted earlier today but with a format string fix which didn't
show up in the ust build I tested it on
Alex Bennée (2):
trace: teach lttng backend to use format strings
trace: add some tcg tracing support
cpu-exec.c | 6 ++
scripts