[Qemu-devel] [RFC 03/38] cpu-exec: set current_cpu at cpu_exec()

2015-08-23 Thread Emilio G. Cota
So that it applies to usermode as well. Signed-off-by: Emilio G. Cota --- cpu-exec.c | 2 ++ cpus.c | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/cpu-exec.c b/cpu-exec.c index b8a11e1..2b9a447 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -386,6 +386,8 @@ int cpu_exec(CP

[Qemu-devel] [RFC 04/38] translate-all: remove volatile from have_tb_lock

2015-08-23 Thread Emilio G. Cota
This is a thread-local variable and therefore all changes to it will be seen in order by the owning thread. There is no need for it to be volatile. Signed-off-by: Emilio G. Cota --- translate-all.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/translate-all.c b/translate-al

[Qemu-devel] [RFC 07/38] seqlock: read sequence number atomically

2015-08-23 Thread Emilio G. Cota
With this change we make sure that the compiler will not optimise the read of the sequence number in any way. Signed-off-by: Emilio G. Cota --- include/qemu/seqlock.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/qemu/seqlock.h b/include/qemu/seqlock.h index f

[Qemu-devel] [RFC 08/38] rcu: init rcu_registry_lock after fork

2015-08-23 Thread Emilio G. Cota
We were unlocking this lock after fork, which is wrong since only the thread that holds a mutex is allowed to unlock it. Signed-off-by: Emilio G. Cota --- util/rcu.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/util/rcu.c b/util/rcu.c index 8ba304d..47c2bce 100644 --

[Qemu-devel] [RFC 06/38] seqlock: add missing 'inline' to seqlock_read_retry

2015-08-23 Thread Emilio G. Cota
Signed-off-by: Emilio G. Cota --- include/qemu/seqlock.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/qemu/seqlock.h b/include/qemu/seqlock.h index 3ff118a..f1256f5 100644 --- a/include/qemu/seqlock.h +++ b/include/qemu/seqlock.h @@ -62,7 +62,7 @@ static inline unsi

[Qemu-devel] [RFC 13/38] cputlb: add physical address to CPUTLBEntry

2015-08-23 Thread Emilio G. Cota
Having the physical address in the TLB entry will allow us to portably obtain the physical address of a memory access, which will prove useful when implementing a scalable emulation of atomic instructions. Signed-off-by: Emilio G. Cota --- cputlb.c| 1 + include/exec/cpu-defs.h |

[Qemu-devel] [RFC 01/38] cpu-exec: add missing mmap_lock in tb_find_slow

2015-08-23 Thread Emilio G. Cota
Signed-off-by: Emilio G. Cota --- cpu-exec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cpu-exec.c b/cpu-exec.c index f53475c..b8a11e1 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -330,6 +330,7 @@ static TranslationBlock *tb_find_slow(CPUState *cpu, if (!tb) { tb = t

[Qemu-devel] [RFC 10/38] translate-all: remove obsolete comment about l1_map

2015-08-23 Thread Emilio G. Cota
l1_map is based on physical addresses in full-system mode, as pointed out in an earlier comment. Said comment also mentions that virtual addresses are only used in l1_map in user-only mode. Signed-off-by: Emilio G. Cota --- translate-all.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)

[Qemu-devel] [PATCH 1/2] linux-user: remove --enable-guest-base/--disable-guest-base

2015-08-23 Thread Laurent Vivier
All tcg host architectures now support the guest base and as there is no real performance lost, it can be always enabled. Anyway, guest base use can be disabled lively by setting guest base to 0. CONFIG_USE_GUEST_BASE is defined as (USE_GUEST_BASE && USER_ONLY), it should have to be replaced by C

[Qemu-devel] [PATCH 2/2] linux-user: remove useless macros GUEST_BASE and RESERVED_VA

2015-08-23 Thread Laurent Vivier
Signed-off-by: Laurent Vivier --- include/exec/cpu-all.h | 4 +--- include/exec/cpu_ldst.h | 8 linux-user/mmap.c| 20 ++-- tcg/aarch64/tcg-target.c | 10 +- tcg/arm/tcg-target.c | 8 tcg/i386/tcg-target.c| 22 +++---

[Qemu-devel] [PATCH 0/2] linux-user: remove guest-base configure parameter

2015-08-23 Thread Laurent Vivier
These two patches remove the parameters --enable-guest-base/--disable-guest-base from the configure options. So, in linux-user mode, guest base is always enabled, and can be disabled by setting the guest base to 0. I've checked some softmmu targets with BIOS/firmware, and booted a raspbian distro

Re: [Qemu-devel] [PATCH v7 RESEND 10/11] filter/buffer: update command description and help

2015-08-23 Thread Thomas Huth
On 21/08/15 00:43, Yang Hongyang wrote: > now that we have a buffer netfilter, update the command > description and help. > > Signed-off-by: Yang Hongyang > CC: Luiz Capitulino > CC: Markus Armbruster > --- > hmp-commands.hx | 2 +- > qemu-options.hx | 5 - > qmp-commands.hx | 2 +- > 3 fi

Re: [Qemu-devel] [PATCH v7 RESEND 03/11] netfilter: add netfilter_{add|del} commands

2015-08-23 Thread Thomas Huth
On 21/08/15 00:43, Yang Hongyang wrote: > add netfilter_{add|del} commands > This is mostly the same with netdev_{add|del} commands. > > When we delete the netdev, we also delete the netfilter object > attached to it, because if the netdev is removed, the filters > which attached to it is useless.

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

2015-08-23 Thread Peter Crosthwaite
Ping^2! On Sat, Aug 15, 2015 at 7:38 PM, Peter Crosthwaite wrote: > Ping! > > This rebases cleanly following the unfreeze. > > Regards, > Peter > > On Sun, Jul 12, 2015 at 5:18 AM, Andreas Färber wrote: >> Am 12.07.2015 um 03:59 schrieb Peter Crosthwaite: >>> Continue QOMifying target-specific d

Re: [Qemu-devel] [PATCH RFC] pseries: define coldplugged devices as "configured"

2015-08-23 Thread Michael Roth
Quoting Laurent Vivier (2015-08-14 02:46:49) > > > On 14/08/2015 07:20, Bharata B Rao wrote: > > On Thu, Aug 13, 2015 at 02:53:02PM +0200, Laurent Vivier wrote: > >> When a device is hotplugged, attach() sets "configured" to > >> false, waiting an action from the OS to configure it and then > >>

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

2015-08-23 Thread Peter Maydell
On 23 August 2015 at 07:49, Peter Crosthwaite wrote: > just periodically sync the headers from binutils Careful, binutils is GPLv3, which isn't compatible with QEMU's overall GPLv2 license. It's OK to look at binutils to find the information to write the relevant QEMU code, but copying it wholesa

Re: [Qemu-devel] [PATCH v7 RESEND 02/11] init/cleanup of netfilter object

2015-08-23 Thread Thomas Huth
On 21/08/15 00:43, Yang Hongyang wrote: > QTAILQ_ENTRY global_list but used by filter layer, so that we can > manage all filters together. > QTAILQ_ENTRY next used by netdev, filter belongs to the specific netdev is > in this queue. > This is mostly the same with init/cleanup of netdev object. > >

Re: [Qemu-devel] [PATCH RFC 4/4] PPC: fix CUDA packet header size

2015-08-23 Thread Mark Cave-Ayland
On 22/08/15 16:32, Cormac O'Brien wrote: > Change the CUDA packet model to use a three-byte header as in real hardware. > Also add handlers for CUDA_COMBINED_FORMAT_IIC and CUDA_GET_SET_IIC. I think it would make sense to split this patch into 2 - one for the CUDA packet changes, and another to a

Re: [Qemu-devel] [PATCH RFC 0/4] Mac OS 9 compatibility improvements

2015-08-23 Thread Mark Cave-Ayland
On 22/08/15 16:32, Cormac O'Brien wrote: > This patchset greatly improves QEMU's ability to run Mac OS 9 on the mac99 > machine, allowing progress past the bootloader and partial progress through > the load screen. > > Signed-off-by: Cormac O'Brien > --- > Alexander Graf (3): > PPC: Allow Rc b

Re: [Qemu-devel] Debian 7.8.0 SPARC64 on qemu - anything i can do to speedup the emulation?

2015-08-23 Thread Dennis Luehring
Am 22.08.2015 um 20:53 schrieb Artyom Tarasenko: Compared with #undef USE_TCG_OPTIMIZATIONS , they are similar, yes. Compared with vanilla master I get a more noticeable improvement. my test suffering less from the Aurelien Jarno described Sparc32->x86_64 "translation" if you're still using de

[Qemu-devel] [PATCH] mac_dbdma: always clear FLUSH bit once DBDMA channel flush is complete

2015-08-23 Thread Mark Cave-Ayland
The code to flush the DBDMA channel was effectively duplicated in dbdma_control_write(), except for the fact that the copy executed outside of a RUN bit transition was broken by not clearing the FLUSH bit once the flush was complete. Newer PPC Linux kernels would timeout waiting for the FLUSH bit