From: Peter Crosthwaite
EM_MOXIE now has a proper assigned elf code. Use it. Register the old
interim value as EM_MOXIE_OLD and accept either in elf loading.
Cc: Anthony Green
Reviewed-by: Richard Henderson
Signed-off-by: Peter Crosthwaite
Signed-off-by: Paolo Bonzini
---
hw/moxie/moxiesim.
From: Peter Crosthwaite
The only generic code relying on this is linux-user, but linux users'
default behaviour of defaulting ELF_MACHINE to ELF_ARCH will handle
this.
This removes another architecture specific definition from the global
namespace.
Cc: Aurelien Jarno
Acked-by: Aurelien Jarno
Thomas Huth wrote:
> On 22/09/15 11:13, Markus Armbruster wrote:
>
>> docs/migration.txt
>
> Juan?
Reviewed-by: Juan Quintela
O;-)
From: Pavel Butsykin
Added the hmp command to query io apic state, may be usefull after guest
crashes to understand IRQ routing in guest.
Implementation is only for kvm here. The dump will look like
(qemu) info ioapic
ioapic id=0x00 sel=0x26 (redir[11])
pin 0 0x0001 dest=0 vec=0 a
From: Peter Crosthwaite
The only generic code relying on this is linux-user. Linux user
already has a lot of #ifdef TARGET_ customisation so instead, define
ELF_ARCH as either EM_ARM or EM_AARCH64 appropriately.
The armv7m bootloader can just pass EM_ARM directly, as that
is architecture specifi
From: "Daniel P. Berrange"
Steve Ellcey / Leon Alrae reported that QEMU fails to build when
the VPATH directory is outside of the GIT tree, and the system
emulators & tools build is disabled. eg
cd ..
mkdir build
cd build
../qemu/configure --disable-system --disable-tools
make
Signed-off-by: Paolo Bonzini
---
MAINTAINERS | 23 +++
1 file changed, 23 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 7c81848..af4952c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -77,6 +77,7 @@ S: Maintained
F: target-alpha/
F: hw/alpha/
F: tests/tcg/alpha
From: Peter Crosthwaite
Fix the coding style for these cases as per CODING_STYLE. Reverse the
Yoda conditions and add missing if braces.
Reviewed-by: Richard Henderson
Signed-off-by: Peter Crosthwaite
Signed-off-by: Paolo Bonzini
---
include/hw/elf_ops.h | 15 +--
1 file changed,
From: Marc-André Lureau
Replace ad-hoc declarations with the linux header.
Signed-off-by: Marc-André Lureau
Message-Id: <1442585920-28373-1-git-send-email-marcandre.lur...@redhat.com>
Signed-off-by: Paolo Bonzini
---
hw/scsi/vhost-scsi.c | 1 +
include/hw/virtio/vhost-scsi.h | 25 -
Am 22.09.2015 um 17:05 hat Max Reitz geschrieben:
> On 22.09.2015 16:30, Kevin Wolf wrote:
> > Am 18.09.2015 um 17:22 hat Max Reitz geschrieben:
> >> If there is no BlockDriverState in a BlockBackend or if the tray of the
> >> guest device is open, fail all requests (where that is possible) with
>
From: Pavel Butsykin
monitor_fprintf and mon_get_cpu will be used in the target-specific monitor,
so it is advisable to make it external.
Signed-off-by: Pavel Butsykin
Signed-off-by: Denis V. Lunev
CC: Paolo Bonzini
CC: Andreas Färber
Message-Id: <1442927901-1084-6-git-send-email-...@openvz
From: Pavel Butsykin
These constants are needed for optimal access to
bit fields local apic registers without magic numbers.
Signed-off-by: Pavel Butsykin
Signed-off-by: Denis V. Lunev
CC: Paolo Bonzini
CC: Andreas Färber
Message-Id: <1442927901-1084-4-git-send-email-...@openvz.org>
Signed-
From: Marc-André Lureau
Not only it makes sense, but it gets rid of checkpatch warning:
WARNING: consider using qemu_strtosz in preference to strtosz
Also remove get rid of tabs to please checkpatch.
Signed-off-by: Marc-André Lureau
Message-Id: <1442419377-9309-1-git-send-email-marcandre.lur..
From: Peter Crosthwaite
The bootloader can just pass EM_MOXIE directly, as that is architecture
specific code.
This removes another architecture specific definition from the global
namespace.
Cc: Anthony Green
Reviewed-by: Richard Henderson
Acked-By: Riku Voipio
Signed-off-by: Peter Crosthwa
Only "Odd Fixes" status, but let's add a point of contact.
Signed-off-by: Paolo Bonzini
---
MAINTAINERS | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 5317ce1..14c0374 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -724,6 +724,11 @@ L: qe
From: Marc-André Lureau
While reading the function I decided to write some tests.
Signed-off-by: Marc-André Lureau
Message-Id: <1442419377-9309-2-git-send-email-marcandre.lur...@redhat.com>
Signed-off-by: Paolo Bonzini
---
tests/test-cutils.c | 91 +
The arbitration register should read to the same value as the
IOAPIC id register. Fixes kvm-unit-tests ioapic.flat.
Signed-off-by: Paolo Bonzini
---
hw/intc/ioapic.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/hw/intc/ioapic.c b/hw/intc/ioapic.c
index 6ad3c66..bde52e8
From: Pavel Butsykin
Added support emulator for the hmp command "info ioapic"
Signed-off-by: Pavel Butsykin
Signed-off-by: Denis V. Lunev
CC: Paolo Bonzini
CC: Andreas Färber
Message-Id: <1442927901-1084-10-git-send-email-...@openvz.org>
Signed-off-by: Paolo Bonzini
---
hw/intc/ioapic.c
From: Peter Crosthwaite
The bootloaders can just pass EM_SPARC or EM_SPARCV9 directly, as
they are architecture specific code (to one or the other).
This removes another architecture specific definition from the global
namespace.
Cc: Mark Cave-Ayland
Reviewed-by: Richard Henderson
Acked-By: R
Hi
On Tue, Sep 22, 2015 at 4:59 PM, Claudio Fontana
wrote:
> This one does not seem particularly valuable to me.. btw isn't it slightly
> easier to debug with the vector in the structure?
Yeah, it's just is a bit superflous to me store the vector index in
all the elements.
You get the vector i
From: Peter Crosthwaite
The bootloaders can just pass EM_LATTICEMICO32 directly, as that is
architecture specific code.
This removes another architecture specific definition from the global
namespace.
Cc: Michael Walle
Acked-By: Michael Walle
Reviewed-by: Richard Henderson
Acked-By: Riku Voi
From: Pavel Butsykin
Added prefix APIC_ for determining the constant of a particular subsystem,
improve the overall readability and match other constant names.
Signed-off-by: Pavel Butsykin
Signed-off-by: Denis V. Lunev
CC: Paolo Bonzini
CC: Andreas Färber
Message-Id: <1442927901-1084-3-git
If a level-triggered interrupt goes down and back up before the
corresponding EOI, it should be coalesced. This fixes one testcase
in kvm-unit-tests' ioapic.flat.
Signed-off-by: Paolo Bonzini
---
hw/intc/ioapic.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/intc/ioa
From: Pavel Butsykin
Move apic_get_bit(), apic_set_bit() to apic_internal.h, make the apic_get_ppr
symbol external. It's necessary to work with isr, tmr, irr and ppr outside
hw/intc/apic.c
Signed-off-by: Pavel Butsykin
Signed-off-by: Denis V. Lunev
CC: Paolo Bonzini
CC: Andreas Färber
Messa
From: Peter Crosthwaite
The only generic code relying on this is linux-user, but linux users'
default behaviour of defaulting ELF_MACHINE to ELF_ARCH will handle
this.
The bootloader can just pass EM_OPENRISC directly, as that is
architecture specific code.
This removes another architecture spe
Eric Blake writes:
> Expose some weaknesses in the generator: we don't always forbid
> the generation of structs that contain multiple members that map
Slightly misleading. args-name-clash is a clash between command
arguments. These are a struct internally, but we don't currently
generate an a
On 22 September 2015 at 07:52, Paolo Bonzini wrote:
>
>
> On 14/05/2015 18:43, Peter Maydell wrote:
>> Add two new commands to our gdb support:
>> qemu trace-enable eventname
>> qemu trace-disable eventname
>>
>> which allow dynamically enabling and disabling printing of QEMU
>> trace events dur
Hi
- Original Message -
> On 15.09.2015 18:07, marcandre.lur...@redhat.com wrote:
> > From: Marc-André Lureau
> >
> > qemu_chr_fe_get_msgfd() transfers ownership, there is no need to dup the
> > fd.
> >
>
> Are you sure? (tested?)
> There is a specific comment that the dup is done beca
On 22.09.2015 16:42, Kevin Wolf wrote:
> Am 18.09.2015 um 17:22 hat Max Reitz geschrieben:
>> This function associates the given BlockDriverState with the given
>> BlockBackend.
>>
>> Signed-off-by: Max Reitz
>> Reviewed-by: Eric Blake
>> Reviewed-by: Alberto Garcia
>> ---
>> block/block-backen
Dear all,
I am running my vm with this config:
2015-09-22 15:21:56.487+: starting up libvirt version: 1.2.20, qemu
version: 2.4.0
LC_ALL=C PATH=/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin
QEMU_AUDIO_DRV=none /usr/bin/kvm -name testU4 -S -machine
pc-i440fx-2.4,accel=kvm,usb=off
On 22 September 2015 at 07:58, Daniel P. Berrange wrote:
> On Tue, Sep 22, 2015 at 11:13:57AM +0200, Markus Armbruster wrote:
>> Paolo asked for an update.
>>
>> Markus Armbruster writes:
>>
>> > Markus Armbruster writes:
>> Different tack: what are the unmaintained files we actually change?
>>
ok, i've found a better place for patchset download:
https://patchwork.ozlabs.org/project/qemu-
devel/list/?submitter=Timothy+Baldwin&q=linux-user
unfortunately cmake still hangs in a way that even sending SIGCHLD
doesn't wake it up, i have to send SIGKILL to stop it and consequently
breaking the
From: Peter Crosthwaite
ELF_MACHINE is unused by target alpha.
Cc: Richard Henderson
Reviewed-by: Richard Henderson
Acked-By: Riku Voipio
Signed-off-by: Peter Crosthwaite
Signed-off-by: Paolo Bonzini
---
target-alpha/cpu.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/target-alpha/c
On 06/25/2015 07:18 AM, Stefan Hajnoczi wrote:
> On Tue, Jun 23, 2015 at 10:12:15AM +0200, Peter Lieven wrote:
>> upcoming libnfs versions will support logging debug messages. Add
>> support for it in qemu through an URL parameter.
>>
>> Signed-off-by: Peter Lieven
>> ---
>> block/nfs.c | 4
Developers who are new to QEMU, or have a background familiarity
with GNU autotools can have trouble getting their head around the
home-grown QEMU build system. This document attempts to explain
the structure / design of the configure script and the various
Makefile pieces that live across the sour
From: Peter Crosthwaite
The only generic code relying on this is linux-user, but linux users'
default behaviour of defaulting ELF_MACHINE to ELF_ARCH will handle
this.
The bootloaders can just pass EM_MIPS directly, as that is
architecture specific code.
This removes another architecture specif
On 09/21/2015 08:46 PM, Fam Zheng wrote:
> From: Stefan Hajnoczi
>
> Join the transaction when the 'transactional-cancel' QMP argument is
> true.
>
> This ensures that the sync bitmap is not thrown away if another block
> job in the transaction is cancelled or fails. This is critical so
> incre
On 22.09.2015 15:28, Alberto Garcia wrote:
> One of the limitations of the 'blockdev-snapshot-sync' command is that
> it does not allow passing BlockdevOptions to the newly created
> snapshots, so they are always opened using the default values.
>
> Extending the command to allow passing options i
On 2015-09-17 21:55, Richard Henderson wrote:
> This perhaps isn't ideal in terms of (ab)using the "pc" field
> to encode both pc and ppc + delay branch state, as one has to
> be aware of this when examining opcode dumps.
>
> But it preserves existing logic, which will be good for bisection,
> and
On 22.09.2015 15:28, Alberto Garcia wrote:
> Signed-off-by: Alberto Garcia
> Cc: Max Reitz
> ---
> tests/qemu-iotests/085 | 102
> ++---
> tests/qemu-iotests/085.out | 34 ++-
> 2 files changed, 128 insertions(+), 8 deletions(-)
Reviewed
On 09/22/2015 09:23 AM, Kevin Wolf wrote:
> Am 18.09.2015 um 15:22 hat Markus Armbruster geschrieben:
>> From: Eric Blake
>>
>> Commits 7216ae3d and d2828429 disabled some error message hints,
>> all because a change to use modern error reporting meant that the
>> hint would be output prior to the
On 17.09.2015 15:48, Kevin Wolf wrote:
> Store the BdrvChild for bs->file. At this point, bs->file_child->bs just
> duplicates the bs->file pointer. Later, it will completely replace it.
>
> Signed-off-by: Kevin Wolf
> ---
> block.c | 12 +---
> include/block/block_int.
On 22.09.2015 19:14, Max Reitz wrote:
> On 17.09.2015 15:48, Kevin Wolf wrote:
>> Store the BdrvChild for bs->file. At this point, bs->file_child->bs just
>> duplicates the bs->file pointer. Later, it will completely replace it.
>>
>> Signed-off-by: Kevin Wolf
>> ---
>> block.c
On 17.09.2015 15:48, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> block/vmdk.c | 99
> +++-
> 1 file changed, 51 insertions(+), 48 deletions(-)
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP digital signature
Hi,
I suggest cleaning some comments, mostly using the "official"
spellings for MinGW and Mingw-w64.
Am 22.09.2015 um 16:13 schrieb Daniel P. Berrange:
> The oslib-win32 file currently provides a localtime_r and
> gmtime_r replacement unconditionally. Some versions of
> Mingw64 would provide crud
On 17.09.2015 15:48, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> block/blkverify.c | 41 +
> 1 file changed, 21 insertions(+), 20 deletions(-)
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP digital signature
On Tue, 2015-09-22 at 18:08 +0800, Cao jin wrote:
> Hi Alex
>
> On 09/22/2015 02:00 AM, Alex Williamson wrote:
> >
> > Please use different subjects that uniquely identify what each patch
> > does, don't simply re-use the subject for the cover patch on each.
>
> OK, will change it in next version
On 09/22/2015 09:23 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> Expose some weaknesses in the generator: we don't always forbid
>> the generation of structs that contain multiple members that map
>
> Slightly misleading. args-name-clash is a clash between command
> arguments. These
On 17.09.2015 15:48, Kevin Wolf wrote:
> Signed-off-by: Kevin Wolf
> ---
> block/quorum.c | 63
> ++
> 1 file changed, 33 insertions(+), 30 deletions(-)
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP digital signature
On 09/22/2015 03:13 AM, Markus Armbruster wrote:
>> Full list of unmaintained files now:
> [...]
>
> docs/bitmaps.md
> docs/blkdebug.txt
> docs/blkverify.txt
> docs/live-block-ops.txt
> docs/qcow2-cache.txt
> docs/qdev-device-use.txt
> docs/specs/qcow2.txt
> docs/specs/qed_spec.txt
Kevin, St
Reviewed from an en_US perspective, though I left alone things that are
clearly regional (e.g. 'behaviour' vs 'behavior')
On 09/22/2015 12:35 PM, Daniel P. Berrange wrote:
> Developers who are new to QEMU, or have a background familiarity
> with GNU autotools can have trouble getting their head ar
On 09/22/15 18:35, Daniel P. Berrange wrote:
> Developers who are new to QEMU, or have a background familiarity
> with GNU autotools can have trouble getting their head around the
> home-grown QEMU build system. This document attempts to explain
> the structure / design of the configure script and
On 22 September 2015 at 05:01, Markus Armbruster wrote:
> The following changes since commit 9e72681d16792d0ffc42bab634b1753ff299bdfd:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-09-21'
> into staging (2015-09-21 22:33:51 +0100)
>
> are available in the git repository a
On 09/22/2015 10:35 AM, Daniel P. Berrange wrote:
> Developers who are new to QEMU, or have a background familiarity
> with GNU autotools can have trouble getting their head around the
s/autotools/autotools,/
> home-grown QEMU build system. This document attempts to explain
> the structure / desi
meta review of your review:
On 09/22/15 20:11, John Snow wrote:
> Reviewed from an en_US perspective, though I left alone things that are
> clearly regional (e.g. 'behaviour' vs 'behavior')
>
> On 09/22/2015 12:35 PM, Daniel P. Berrange wrote:
>> + - Add information to the help output message to
On 17.09.2015 15:48, Kevin Wolf wrote:
> This patch removes the temporary duplication between bs->file and
> bs->file_child by converting everything to BdrvChild.
>
> Signed-off-by: Kevin Wolf
> ---
> block.c | 61 ++
> block/blkdebug
On 09/22/2015 05:13 AM, Markus Armbruster wrote:
> Paolo asked for an update.
>
> Markus Armbruster writes:
>>
>> Full list of unmaintained files now:
> [...]
>
Full list edited down to 'sections' with replies in-line...
[ top level confetti/detritus ]
[ backends/* ]
[ coroutine-* ]
[ defaul
On 17.09.2015 15:48, Kevin Wolf wrote:
> It is unused now.
>
> Signed-off-by: Kevin Wolf
> ---
> block.c | 34 --
> include/block/block.h | 4
> 2 files changed, 38 deletions(-)
Reviewed-by: Max Reitz
signature.asc
Description: OpenPGP dig
On 17 September 2015 at 10:27, Peter Maydell wrote:
> On 9 September 2015 at 08:49, Pavel Fedin wrote:
>> This series introduces support for GICv3 by KVM. Software emulation is
>> currently not supported.
>>
>> v13 => v14
>>
>> - Rebased on the latest master, fixed conflicts in hw/arm/virt.c
>>
>
On 09/22/2015 12:28 PM, Laszlo Ersek wrote:
> meta review of your review:
>
>
>>> +Each system/userspace emulation target needs to have a slightly
>>> +different set of make rules / variables. Thus, make will be recursively
>>> +invoked for each of the emulation targets.
>>> +
>>> +The recursive
On Fri, Sep 18, 2015 at 10:58:43PM +0800, Yuanhan Liu wrote:
[...]
> diff --git a/net/vhost-user.c b/net/vhost-user.c
> index 93dcecd..4fa3d64 100644
> --- a/net/vhost-user.c
> +++ b/net/vhost-user.c
[...]
> +if (i == 0) {
> +max_queues = vhost_net_get_max_queues(s->vhost_net);
On 09/22/15 20:51, Eric Blake wrote:
> On 09/22/2015 12:28 PM, Laszlo Ersek wrote:
>> meta review of your review:
>>
>
>>
+Each system/userspace emulation target needs to have a slightly
+different set of make rules / variables. Thus, make will be recursively
+invoked for each of th
On 09/21/2015 10:46 PM, Fam Zheng wrote:
> Sometimes block jobs must execute as a transaction group. Finishing
> jobs wait until all other jobs are ready to complete successfully.
> Failure or cancellation of one job cancels the other jobs in the group.
>
> Signed-off-by: Stefan Hajnoczi
> [Re
On 09/21/2015 10:46 PM, Fam Zheng wrote:
> From: Stefan Hajnoczi
>
> Provide a BlockJobTxn to actions executed in a qmp 'transaction'
> command. This allows actions to make their block jobs either complete
> as a group or fail/cancel together.
>
> The next patch adds the first user.
>
> Sign
On 17.09.2015 15:48, Kevin Wolf wrote:
> This is the final step in converting all of the BlockDriverState
> pointers that block drivers use to BdrvChild.
>
> After this patch, bs->children contains the full list of child nodes
> that are referenced by a given BDS, and these children are only
> ref
On 3 September 2015 at 06:02, Gerd Hoffmann wrote:
> Hi,
>
> Here comes the ipxe update pull. Thanks to Stefan all bits are upstream
> now. Also upstream has a named configuration for qemu. That simplifies
> the whole process, I could drop some patches and so there are only 4 of
> them left,
On 09/21/15 21:04, PeteVine wrote:
> Public bug reported:
>
> Trying to boot debian-hurd-20150320.img ends with:
>
> qemu-system-i386: qemu-coroutine-lock.c:91: qemu_co_queue_restart_all:
> Assertion `qemu_in_coroutine()' failed.
>
> Program received signal SIGABRT, Aborted.
> __libc_do_syscall
On 22 September 2015 at 08:05, Paolo Bonzini wrote:
> The following changes since commit 9e72681d16792d0ffc42bab634b1753ff299bdfd:
>
> Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2015-09-21'
> into staging (2015-09-21 22:33:51 +0100)
>
> are available in the git repository at:
>
On 22/09/2015 20:11, John Snow wrote:
>> +such as error handling infrastructure, standard data structures,
>> +platform portability wrapper functions, etc. This code can be compiled
>> +once only and the .o files linked into all output binaries.
>> +
>> +In the target dependant set lives CPU emul
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
On 22/09/2015 20:07, Eric Blake wrote:
> On 09/22/2015 03:13 AM, Markus Armbruster wrote:
>
>>> Full list of unmaintained files now:
>> [...]
>>
>
>> docs/bitmaps.md docs/blkdebug.txt docs/blkverify.txt
>
>> docs/live-block-ops.txt
>
>> docs/q
On 22/09/2015 21:41, Peter Maydell wrote:
> I'm afraid this doesn't build on 32-bit:
>
> /root/qemu/hw/intc/ioapic_common.c: In function ‘ioapic_print_redtbl’:
> /root/qemu/hw/intc/ioapic_common.c:80:24: error: format ‘%lu’ expects
> argument of type ‘long unsigned int’, but argument 6 has type
On 22/09/2015 10:16, Markus Armbruster wrote:
>> "To build QEMU, you'd generally [some basic steps that have generally
>> worked for a long time], but for up-to-date authoritative information,
>> please do:
>>
>> mkdir path_to/build_dir
>> cd path_to/build_dir
>> ../../configure
>> make qemu-doc.
All right, I'll rebuild and try qemu-gdb.py if still necessary.
Do any fancy CFLAGS make a difference in performance or should they be avoided
altogether on ARM?
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.
In 2012, QEMU had a bug where it exposed QEMU version information to the
guest, meaning a QEMU upgrade would expose different hardware to the
guest OS even if the same machine-type is being used.
The bug was fixed by commit 93bfef4c6e4b23caea9d51e1099d06433d8835a4, on
all machines up to pc-1.0. Bu
In 2012, QEMU had a bug where it exposed QEMU version information
to the guest, meaning a QEMU upgrade would expose different
hardware to the guest OS even if the same machine-type is being
used.
The bug was fixed by commit 93bfef4c6e4b23caea9d51e1099d06433d8835a4,
on all machines up to pc-1.0. Bu
Guest visible data shouldn't change with a simple QEMU upgrade, so use
qemu_hw_version() to ensure it won't change (as long as the machine
class being used has hw_version set).
Cc: Hannes Reinecke
Cc: Paolo Bonzini
Cc: qemu-bl...@nongnu.org
Signed-off-by: Eduardo Habkost
---
hw/scsi/megasas.c
This makes the purpose of the function clearer: it is not about the
version of QEMU that's running, but the version string exposed in the
emulated hardware.
Cc: Andrzej Zaborowski
Cc: Peter Maydell
Cc: John Snow
Cc: Paolo Bonzini
Signed-off-by: Eduardo Habkost
---
hw/arm/nseries.c | 2 +
On 09/22/2015 10:16 PM, Eduardo Habkost wrote:
> Guest visible data shouldn't change with a simple QEMU upgrade, so use
> qemu_hw_version() to ensure it won't change (as long as the machine
> class being used has hw_version set).
>
> Cc: Hannes Reinecke
> Cc: Paolo Bonzini
> Cc: qemu-bl...@nongn
On 09/22/2015 04:16 PM, Eduardo Habkost wrote:
> This makes the purpose of the function clearer: it is not about the
> version of QEMU that's running, but the version string exposed in the
> emulated hardware.
>
> Cc: Andrzej Zaborowski
> Cc: Peter Maydell
> Cc: John Snow
> Cc: Paolo Bonzini
With an eye toward making it mandatory.
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-alpha/translate.c | 2 +-
target-arm/translate-a64.c| 2 +-
target-arm/translate.c| 2 +-
target-cris/translate.c | 4 ++--
target-c
Version 3. Notable changes:
(1) Add a guard page at the end of the code_gen_buffer.
We will segv insted of silently corrupting memory
if we overrun the buffer.
The win32 bits tested under wine only; I haven't put together
all the right bits under my win7 vm yet. Althou
While we're at it, emit the opcode adjacent to where we currently
record data for search_pc. This puts gen_io_start et al on the
"correct" side of the marker.
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-alpha/translate.c | 6 ++
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-arm/cpu.h | 1 +
target-arm/translate-a64.c | 2 +-
target-arm/translate.c | 3 ++-
3 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/target-arm/cpu.h b/target-arm/cpu.h
Unify three copies of this code from different
branch types. Fix the case when npc == DYNAMIC_PC,
i.e. a branch within a delay slot.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-sparc/translate.c | 55
1 file changed, 2
With an eye toward having this data replace the gen_opc_* arrays
that each target collects in order to enable restore_state_from_tb.
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
tcg/tcg-op.h | 52 --
This does tidy the icount test common to all targets.
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-alpha/translate.c | 4 ++--
target-arm/translate-a64.c| 6 +++---
target-arm/translate.c| 7 ---
target-cris/translate.c
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-i386/cpu.h | 1 +
target-i386/translate.c | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/target-i386/cpu.h b/target-i386/cpu.h
index 5231e8c..717d558 100644
--- a/targ
Reduce the boilerplate required for each target. At the same time,
move the test for breakpoint after calling tcg_gen_insn_start.
Note that arm and aarch64 do not use cpu_breakpoint_test, but still
move the inline test down after tcg_gen_insn_start.
Signed-off-by: Richard Henderson
---
include
As it's only caller, this tidies things a bit.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
include/exec/exec-all.h | 2 -
translate-all.c | 131 ++--
2 files changed, 59 insertions(+), 74 deletions(-)
diff --git a/includ
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-s390x/cpu.h | 1 +
target-s390x/translate.c | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/target-s390x/cpu.h b/target-s390x/cpu.h
index 9aeb024..68d6528 100644
--- a/
By putting the prologue at the end, we risk overwriting the
prologue should our estimate of maximum TB size. Given the
two different placements of the call to tcg_prologue_init,
move the high water mark computation into tcg_prologue_init.
Signed-off-by: Richard Henderson
---
tcg/tcg.c | 2
We always pass pc2 == dc->npc and r_cond == cpu_cond,
and always set is_br afterward. Infer all of that.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-sparc/translate.c | 21 ++---
1 file changed, 10 insertions(+), 11 deletions(-)
diff --git a/target-s
The gen_opc_* arrays are already redundant with the data stored in
the insn_start arguments. Transition restore_state_to_opc to use
data from the latter.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
include/exec/exec-all.h | 2 +-
target-alpha/translate.c | 5 ++
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-mips/cpu.h | 1 +
target-mips/translate.c | 3 ++-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/target-mips/cpu.h b/target-mips/cpu.h
index ed7d86d..fd23832 100644
--- a/tar
This will catch any overflow of the buffer.
Add a native win32 alternative for alloc_code_gen_buffer;
remove the malloc alternative.
Signed-off-by: Richard Henderson
---
translate-all.c | 210
1 file changed, 119 insertions(+), 91 deletio
Reviewed-by: Aurelien Jarno
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-sh4/cpu.h | 1 +
target-sh4/translate.c | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/target-sh4/cpu.h b/target-sh4/cpu.h
index 1f68b27..ea854cb 100644
--- a/target-s
We currently pre-compute an worst case code size for any TB, which
works out to be 122kB. Since the average TB size is near 1kB, this
wastes quite a lot of storage.
Instead, check for overflow in between generating code for each opcode.
The overhead of the check isn't measurable and wastage is mi
This symbol no longer exists.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
target-alpha/cpu.h | 1 -
target-arm/cpu.h| 1 -
target-cris/cpu.h | 1 -
target-i386/cpu.h | 1 -
target-lm32/cpu.h | 1 -
target-m68k/cpu.h | 1 -
target-microbla
It is no longer used, so tidy up everything reached by it.
This includes the gen_opc_* arrays, the search_pc parameter
and the inline gen_intermediate_code_internal functions.
Reviewed-by: Peter Maydell
Signed-off-by: Richard Henderson
---
include/exec/exec-all.h | 1 -
target-alpha/tran
On 09/22/15 22:16, Eduardo Habkost wrote:
> In 2012, QEMU had a bug where it exposed QEMU version information to the
> guest, meaning a QEMU upgrade would expose different hardware to the
> guest OS even if the same machine-type is being used.
>
> The bug was fixed by commit 93bfef4c6e4b23caea9d51
201 - 300 of 390 matches
Mail list logo
