[Qemu-devel] [PATCH 12/19] xtensa: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com The bootloaders can just pass EM_XTENSA directly, as that is architecture specific code. This removes another architecture specific definition from the global namespace. Cc: Max Filippov jcmvb...@gmail.com Signed-off-by: Peter Crosthwaite

[Qemu-devel] [PATCH 11/19] tricore: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com The bootloader can just pass EM_TRICORE directly, as that is architecture specific code. This removes another architecture specific definition from the global namespace. Cc: Bastian Koppelmann kbast...@mail.uni-paderborn.de Signed-off-by: Peter

[Qemu-devel] [PATCH 16/19] mips: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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

Re: [Qemu-devel] [PATCH v3 28/35] arm: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
On Sat, Jul 18, 2015 at 5:33 AM, Paolo Bonzini pbonz...@redhat.com wrote: On 18/07/2015 11:40, Peter Crosthwaite wrote: From: Peter Crosthwaite crosthwaitepe...@gmail.com The only generic code relying on this is linux-user. Linux user already has a lot of #ifdef TARGET_ customisation so

[Qemu-devel] [PATCH 13/19] sh4: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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

[Qemu-devel] [PATCH 10/19] or32: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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

[Qemu-devel] [PATCH 15/19] sparc: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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

Re: [Qemu-devel] [Consult] tilegx: About floating point instructions

2015-08-15 Thread Chen Gang
On 8/16/15 09:41, Chen Gang wrote: On 8/16/15 02:16, Chen Gang wrote: On 8/15/15 23:47, Richard Henderson wrote: On Aug 15, 2015 2:56 AM, Chen Gang xili_gchen_5...@hotmail.com Oh, we are unlucky, after continue gcc testsuite, add/sub floating point insns also can be mixed together! The

[Qemu-devel] [PATCH 02/19] linux-user: elfload: Provide default for elf_check_arch

2015-08-15 Thread Peter Crosthwaite
For many arch's this macro is defined as the predicatable behaviour of checking the argument for eqaulity against ELF_ARCH. Provide a default define as such, so only archs with special handling (usually allowing multiple EM values) need to provide a def. Arches that do any of: 1: provide this

[Qemu-devel] [PATCH 05/19] m68k: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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 machine model bootloaders can just pass EM_68K directly, as that is architecture specific code.

[Qemu-devel] [PATCH 00/19] multi-arch+linux-user: Cleanup ELF_MACHINE

2015-08-15 Thread Peter Crosthwaite
Every arch defines ELF_MACHINE in cpu.h to an arch-specific value. This definition is rarely needed by core code (only in a few cases). It conflicts with the multi-arch effort where cpu.h cannot export cpu specifics via macros like this. So remove ELF_MACHINE completely from all the cpu.h's.

[Qemu-devel] [PATCH 18/19] i386: Rename ELF_MACHINE to be x86 specific

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com Rename ELF_MACHINE to be I386 specific. This is used as-is by the multiboot loader. Linux-user previously used this definition but will not anymore, falling back to the default bahaviour of using ELF_ARCH as ELF_MACHINE. This removes another

[Qemu-devel] [PATCH 04/19] mb: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com The only generic code relying on this is linux-user, but linux-users' default behaviour or setting ELF_MACHINE to ELF_ARCH will handle this. The microblaze bootloader can just pass EM_MICROBLAZE directly, as that is architecture specific code.

[Qemu-devel] [PATCH 17/19] alpha: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com ELF_MACHINE is unused by target alpha. Cc: Richard Henderson r...@twiddle.net Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- target-alpha/cpu.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/target-alpha/cpu.h

Re: [Qemu-devel] [PATCH v1 1/3] ahci: Separate the AHCI state structure into the header

2015-08-15 Thread Peter Crosthwaite
On Sat, Aug 15, 2015 at 2:21 PM, Peter Crosthwaite crosthwaitepe...@gmail.com wrote: On Mon, Jul 27, 2015 at 11:37 AM, Alistair Francis alistair.fran...@xilinx.com wrote: Pull the AHCI state structure out into the header. This allows other containers to access the struct. This is required to

Re: [Qemu-devel] [PATCH 8/8] disas: alpha: QOMify target specific disas setup

2015-08-15 Thread Peter Crosthwaite
Ping! On Sat, Jul 11, 2015 at 7:00 PM, Peter Crosthwaite crosthwaitepe...@gmail.com wrote: From: Peter Crosthwaite crosthwaitepe...@gmail.com Move the target_disas() alpha specifics to the QOM disas_set_info hook and delete the #ifdef specific code in disas.c. This also makes

Re: [Qemu-devel] [PATCH 1/8] disas: s390x: QOMify target specific disas setup

2015-08-15 Thread Peter Crosthwaite
Ping! On Sat, Jul 11, 2015 at 6:59 PM, Peter Crosthwaite crosthwaitepe...@gmail.com wrote: From: Peter Crosthwaite crosthwaitepe...@gmail.com Move the target_disas() s390 specifics to the QOM disas_set_info hook and delete the #ifdef specific code in disas.c. Cc: Richard Henderson

Re: [Qemu-devel] [PATCH 0/8] Disas QOMification, round 2

2015-08-15 Thread Peter Crosthwaite
Ping! This rebases cleanly following the unfreeze. Regards, Peter On Sun, Jul 12, 2015 at 5:18 AM, Andreas Färber afaer...@suse.de wrote: Am 12.07.2015 um 03:59 schrieb Peter Crosthwaite: Continue QOMifying target-specific disassembly. Convert all arches except for X86 and PPC. They will be

Re: [Qemu-devel] [PATCH v1 1/3] ahci: Separate the AHCI state structure into the header

2015-08-15 Thread Peter Crosthwaite
On Mon, Jul 27, 2015 at 11:37 AM, Alistair Francis alistair.fran...@xilinx.com wrote: Pull the AHCI state structure out into the header. This allows other containers to access the struct. This is required to add the device to modern SoC containers. Signed-off-by: Alistair Francis

Re: [Qemu-devel] [PATCH v1 2/3] object.c: object_class_dynamic_cast return NULL if the class has no type

2015-08-15 Thread Peter Crosthwaite
On Mon, Jul 27, 2015 at 11:37 AM, Alistair Francis alistair.fran...@xilinx.com wrote: If the ObjectClass has no type return NULL instead of trying to compare the type name. What was the issue? Regards, Peter Signed-off-by: Alistair Francis alistair.fran...@xilinx.com Reviewed-by: Sai Pavan

Re: [Qemu-devel] guest: NO_HZ FULL will not work with unstable sched clock

2015-08-15 Thread Stefan Priebe
same with kvm clock: [0.195073] Switched to clocksource kvm-clock [0.195509] [ cut here ] [0.195509] WARNING: CPU: 1 PID: 1 at kernel/time/tick-sched.c:192 can_stop_full_tick+0x1c1/0x200() [0.195515] NO_HZ FULL will not work with unstable sched clock [

Re: [Qemu-devel] [PATCH v1 1/3] ahci: Separate the AHCI state structure into the header

2015-08-15 Thread Peter Crosthwaite
On Wed, Jul 29, 2015 at 5:12 PM, Alistair Francis alistair.fran...@xilinx.com wrote: On Wed, Jul 29, 2015 at 3:21 PM, John Snow js...@redhat.com wrote: On 07/27/2015 02:37 PM, Alistair Francis wrote: Pull the AHCI state structure out into the header. This allows other containers to access

Re: [Qemu-devel] [PATCH v1 3/3] xlnx-zynqmp: Connect the sysbus AHCI to ZynqMP

2015-08-15 Thread Peter Crosthwaite
On Mon, Jul 27, 2015 at 11:37 AM, Alistair Francis alistair.fran...@xilinx.com wrote: Connect the Sysbus AHCI device to ZynqMP. Signed-off-by: Alistair Francis alistair.fran...@xilinx.com Reviewed-by: Sai Pavan Boddu saip...@xilinx.com --- hw/arm/xlnx-zynqmp.c | 20

[Qemu-devel] [PATCH 03/19] arm: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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

[Qemu-devel] [PATCH 06/19] cris: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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_CRIS directly, as that is architecture specific code. This removes

[Qemu-devel] [PATCH 01/19] linux_user: elfload: Default ELF_MACHINE to ELF_ARCH

2015-08-15 Thread Peter Crosthwaite
In most (but not all) cases, ELF_MACHINE and ELF_ARCH are safely the same. Default ELF_MACHINE to ELF_ARCH. This makes defining ELF_MACHINE optional for target-*/cpu.h when they are known to match. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- linux-user/elfload.c | 4 1

[Qemu-devel] [PATCH 14/19] s390: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com The bootloader can just pass EM_S390 directly, as that is architecture specific code. This removes another architecture specific definition from the global namespace. Cc: Richard Henderson r...@twiddle.net Cc: Alexander Graf ag...@suse.de

Re: [Qemu-devel] [PATCH] Move RAMBlock and ram_list to ram_addr.h

2015-08-15 Thread Peter Crosthwaite
Is there a functional reason for making this change, or pure organisational? Either way the patch is ok, I'm just looking for the motivation as this stuff did pop up in the multi-arch refactorings at one stage and I did think about moving it. Can we add a one-liner to commit message? Regards,

Re: [Qemu-devel] [PATCH v1 1/3] ahci: Separate the AHCI state structure into the header

2015-08-15 Thread Peter Maydell
On 15 August 2015 at 22:18, Peter Crosthwaite crosthwaitepe...@gmail.com wrote: This has come up a few times, and the conclusion was to do the inlining by preference. I vaguely remember PMM started an effort to build infrastructure to hide the internals from use by outsiders, but the struct

[Qemu-devel] [PATCH 19/19] ppc: Rename ELF_MACHINE to be PPC specific

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com Rename ELF_MACHINE to be PPC specific. This is used as-is by the various PPC bootloaders and is locally defined to ELF_MACHINE in linux user in PPC specific ifdeffery. This removes another architecture specific definition from the global

[Qemu-devel] [PATCH 07/19] moxie: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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 gr...@moxielogic.com Signed-off-by: Peter Crosthwaite

[Qemu-devel] [PATCH 08/19] unicore: Remove ELF_MACHINE from cpu.h

2015-08-15 Thread Peter Crosthwaite
From: Peter Crosthwaite crosthwaitepe...@gmail.com 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: Guan Xuetao

Re: [Qemu-devel] [Consult] tilegx: About floating point instructions

2015-08-15 Thread Chen Gang
On 8/16/15 02:16, Chen Gang wrote: On 8/15/15 23:47, Richard Henderson wrote: On Aug 15, 2015 2:56 AM, Chen Gang xili_gchen_5...@hotmail.com Oh, we are unlucky, after continue gcc testsuite, add/sub floating point insns also can be mixed together! The related C code, -save-temps, and

Re: [Qemu-devel] [Consult] tilegx: About floating point instructions

2015-08-15 Thread Chen Gang
On 8/13/15 22:59, Chen Gang wrote: Hello all: For me, I guess for single insns, they are simple, and each calculation insns group can not be mixed with each other. So current implementation should be OK. For double insns, I guess, only mul calculation can be mixed with other calculation

Re: [Qemu-devel] [PATCH v16 00/21] Deterministic replay core

2015-08-15 Thread Pavel Dovgalyuk
Hi, Paolo! Will you apply these patches to 2.5? Pavel Dovgalyuk Sent using CloudMagic [https://cloudmagic.com/k/d/mailapp?ct=tacv=5.1.11.3pv=4.4.2] On вт, Авг 04, 2015 at 11:43 AM, Pavel Dovgalyuk pavel.dovga...@ispras.ru [pavel.dovga...@ispras.ru] wrote: This set of patches is related

Re: [Qemu-devel] [PATCH v16 00/21] Deterministic replay core

2015-08-15 Thread Paolo Bonzini
On 15/08/2015 11:57, Pavel Dovgalyuk wrote: Hi, Paolo! Will you apply these patches to 2.5? Yes, I'll put them in my next pull request. Paolo

Re: [Qemu-devel] [PATCH 03/11] target-m68k: Remove incorrect clearing of cc_x

2015-08-15 Thread Andreas Schwab
Richard Henderson r...@twiddle.net writes: Ah. Except placed here it also applies to and/or/move too. Well, none of those ever overflow in the first place. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 And now for

Re: [Qemu-devel] [Consult] tilegx: About floating point instructions

2015-08-15 Thread Chen Gang
On 8/15/15 23:47, Richard Henderson wrote: On Aug 15, 2015 2:56 AM, Chen Gang xili_gchen_5...@hotmail.com Oh, we are unlucky, after continue gcc testsuite, add/sub floating point insns also can be mixed together! The related C code, -save-temps, and objdump files are in attachments (is it

[Qemu-devel] guest: NO_HZ FULL will not work with unstable sched clock

2015-08-15 Thread Stefan Priebe
Hi, while switching to a FULL tickless kernel i detected that all our VMs produce the following stack trace while running under qemu 2.3.0. [0.195160] HPET: 3 timers in total, 0 timers will be used for per-cpu timer [0.195181] hpet0: at MMIO 0xfed0, IRQs 2, 8, 0 [0.195188]

[Qemu-devel] [PATCH] linux-user: remove MAX_ARG_PAGES, allow dynamic growth of env/argv space

2015-08-15 Thread Stefan Brüns
qemu currently limits the space for the evironment and arguments to 32 * PAGE_SIZE. Linux limits the argument space to 1/4 of the stack size. A program trying to detect this with a getrlimit(RLIMIT_STACK) syscall will typically get a much larger limit than qemus current 128kB. The current limit

[Qemu-devel] [PATCH] Fix Argument list to long errors

2015-08-15 Thread Stefan Brüns
Build environments often invoke commands with huge argument lists. For native builds, the limit on current linux (2.6.33) is 1/4th of the stack size, typically 2MByte, so this works fine. For chroots with qemu emulation, the limit currently is set by MAX_ARG_PAGES, i.e. 32*4kByte. Without the

Re: [Qemu-devel] [Qemu-block] RFC cdrom in own thread?

2015-08-15 Thread Peter Lieven
Am 14.08.2015 um 16:45 schrieb Peter Lieven: Am 14.08.2015 um 16:08 schrieb Kevin Wolf: Am 14.08.2015 um 15:43 hat Peter Lieven geschrieben: Am 22.06.2015 um 23:54 schrieb John Snow: On 06/22/2015 09:09 AM, Peter Lieven wrote: Am 22.06.2015 um 11:25 schrieb Stefan Hajnoczi: On Fri, Jun 19,

[Qemu-devel] [Bug 1485180] [NEW] Ctrl Alt G -- Multiple Virtual Machines

2015-08-15 Thread hotbot200
Public bug reported: I'm using Fedora 22. Firstly, what works: A single VM instance, running Windows. Although, I am keeping this (GTK) window focused. What really fails: If I have two running VM's, WIndows XP and Windows Vista: 1. I press Ctrl-Alt-G to get the focus. 2. That works first time.

Re: [Qemu-devel] [Consult] tilegx: About floating point instructions

2015-08-15 Thread Richard Henderson
On Aug 15, 2015 2:56 AM, Chen Gang xili_gchen_5...@hotmail.com Oh, we are unlucky, after continue gcc testsuite, add/sub floating point insns also can be mixed together! The related C code, -save-temps, and objdump files are in attachments (is it gcc's issue? I guess not). So, I guess, we