Re: [Qemu-devel] [PATCH 6/8] linux-user: Enable NPTL for OpenRISC

2013-07-13 Thread Jia Liu
Hi Peter, On Sat, Jul 13, 2013 at 4:12 AM, Peter Maydell wrote: > The OpenRISC kernel ignores CLONE_SETTLS in its copy_thread() > implementation, so a cpu_set_tls() implementation is a no-op. > cpu_clone_regs() was setting the syscall return value in the > wrong register -- it is gpr[11], not gpr

Re: [Qemu-devel] [PATCH 6/8] linux-user: Enable NPTL for OpenRISC

2013-07-13 Thread Peter Maydell
On 13 July 2013 08:40, Jia Liu wrote: > Hi Peter, > > On Sat, Jul 13, 2013 at 4:12 AM, Peter Maydell > wrote: >> The OpenRISC kernel ignores CLONE_SETTLS in its copy_thread() >> implementation, so a cpu_set_tls() implementation is a no-op. >> cpu_clone_regs() was setting the syscall return value

Re: [Qemu-devel] [Qemu-trivial] [PATCH] slirp: reorder include to fix FreeBSD build failure

2013-07-13 Thread Michael Tokarev
13.07.2013 00:29, Ed Maste wrote: > Signed-off-by: Ed Maste > --- > The issue comes from slirp/mbuf.h #defining m_flags, which conflicts with > a header included via on FreeBSD. Umgh. How.. disgusting... :( Here's the code in question. /* header at beginning of each mbuf: */ struct m_hdr {

Re: [Qemu-devel] [PATCH v8 1/3] Introduce async_run_on_cpu()

2013-07-13 Thread Orit Wasserman
On 06/24/2013 12:49 PM, Chegu Vinod wrote: > Introduce an asynchronous version of run_on_cpu() i.e. the caller > doesn't have to block till the call back routine finishes execution > on the target vcpu. > > Signed-off-by: Chegu Vinod > Reviewed-by: Paolo Bonzini > --- > cpus.c|

Re: [Qemu-devel] [PATCH v8 2/3] Add 'auto-converge' migration capability

2013-07-13 Thread Orit Wasserman
On 06/24/2013 12:49 PM, Chegu Vinod wrote: > The auto-converge migration capability allows the user to specify if they > choose live migration seqeunce to automatically detect and force convergence. > > Signed-off-by: Chegu Vinod > Reviewed-by: Paolo Bonzini > Reviewed-by: Eric Blake > --- > i

Re: [Qemu-devel] [PATCH] spapr-pci: change endianness for io ports space

2013-07-13 Thread David Gibson
On Fri, Jul 12, 2013 at 05:37:19PM +1000, Alexey Kardashevskiy wrote: > sPAPR PHB emulates IO ports on PCI via a special memory region which > routes all reads/writes further via cpu_in*/cpu_out* which are eventually > processed by MemoryRegionOps implemented by devices. Hrm. That double dispatch

Re: [Qemu-devel] [PATCH v8 3/3] Force auto-convegence of live migration

2013-07-13 Thread Orit Wasserman
On 06/24/2013 12:47 PM, Chegu Vinod wrote: > If a user chooses to turn on the auto-converge migration capability > these changes detect the lack of convergence and throttle down the > guest. i.e. force the VCPUs out of the guest for some duration > and let the migration thread catchup and help conv

Re: [Qemu-devel] [PATCH 8/8] linux-user: Enable NPTL for m68k

2013-07-13 Thread Laurent Vivier
Le 12/07/2013 22:12, Peter Maydell a écrit : For m68k, per-thread data is a purely kernel construct with no CPU level support. Implement it via a field in the TaskState structure, used by cpu_set_tls() and the set_thread_area/get_thread_area syscalls. This allows us to enable compilation with NPT

[Qemu-devel] Idle win7 guest's cpu consumption is 60%

2013-07-13 Thread 楼正伟
Hi, I'm using KVM win7(32bit) guest, and found that, guest cpu was almostly 60% when it's idle. The host is centos6.3, with qemu-1.5.1. In the guest, I found that: 1) All the cpu is eat by one task called "system interrupt"; 2) The virtio net device and USB Uni

Re: [Qemu-devel] [PATCH 8/8] linux-user: Enable NPTL for m68k

2013-07-13 Thread Peter Maydell
On 13 July 2013 11:28, Laurent Vivier wrote: > Le 12/07/2013 22:12, Peter Maydell a écrit : >> --- a/linux-user/syscall.c >> +++ b/linux-user/syscall.c >> @@ -8558,6 +8558,11 @@ abi_long do_syscall(void *cpu_env, int num, >> abi_long arg1, >> #elif defined(TARGET_I386) && defined(TARGET_ABI32) >

Re: [Qemu-devel] [PATCH v3 11/14] ioport: Switch dispatching to memory core layer

2013-07-13 Thread Paolo Bonzini
Il 11/07/2013 14:46, Andreas Färber ha scritto: > sPAPR has its MemoryRegion marked Little Endian: > > http://git.qemu.org/?p=qemu.git;a=blobdiff;f=hw/spapr_pci.c;h=a08ed11166595bdc493065beb64d4ce5b7b0dded;hp=c2c3079d21d5be2647faf85a8c608ac995d2ca62;hb=a3cfa18eb075c7ef78358ca1956fe7b01caa1724;hpb=

Re: [Qemu-devel] [PATCH v3 11/14] ioport: Switch dispatching to memory core layer

2013-07-13 Thread Anthony Liguori
On Sat, Jul 13, 2013 at 9:38 AM, Paolo Bonzini wrote: > Il 11/07/2013 14:46, Andreas Färber ha scritto: >> sPAPR has its MemoryRegion marked Little Endian: >> >> http://git.qemu.org/?p=qemu.git;a=blobdiff;f=hw/spapr_pci.c;h=a08ed11166595bdc493065beb64d4ce5b7b0dded;hp=c2c3079d21d5be2647faf85a8c608a

Re: [Qemu-devel] [PATCH v3 11/14] ioport: Switch dispatching to memory core layer

2013-07-13 Thread Hervé Poussineau
Anthony Liguori a écrit : On Sat, Jul 13, 2013 at 9:38 AM, Paolo Bonzini wrote: Il 11/07/2013 14:46, Andreas Färber ha scritto: sPAPR has its MemoryRegion marked Little Endian: http://git.qemu.org/?p=qemu.git;a=blobdiff;f=hw/spapr_pci.c;h=a08ed11166595bdc493065beb64d4ce5b7b0dded;hp=c2c3079d21

Re: [Qemu-devel] [PATCH 5/8] linux-user: Move includes of target-specific headers to end of qemu.h

2013-07-13 Thread Richard Henderson
On 07/12/2013 01:12 PM, Peter Maydell wrote: > The target-specific headers (target_cpu.h and target_signal.h) > might need to use the target-independent structure and function > definitions of qemu.h; so include them only at the bottom of > qemu.h, not the top. > > Signed-off-by: Peter Maydell R

Re: [Qemu-devel] [PATCH 7/8] linux-user: Enable NPTL for SPARC targets

2013-07-13 Thread Richard Henderson
On 07/12/2013 01:12 PM, Peter Maydell wrote: > Provide the missing cpu_set_tls(), and resolve the FIXME in > cpu_clone_regs() by clearing the carry flag for the child. > This allows us to turn on building with NPTL for SPARC. > > Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [Qemu-trivial] [PATCH] slirp: reorder include to fix FreeBSD build failure

2013-07-13 Thread Ed Maste
On 13 July 2013 05:12, Michael Tokarev wrote: > Remaining: > > struct mbuf { > union M_dat { > charm_dat_[1]; /* ANSI don't like 0 sized arrays */ > char*m_ext_; > } M_dat; > }; > > #define m_dat M_dat.m_dat_ > #define m_ext

Re: [Qemu-devel] [PATCH] atomic: using memory_order_relaxed for refcnt inc/dec ops

2013-07-13 Thread liu ping fan
On Sat, Jul 13, 2013 at 12:24 AM, Richard Henderson wrote: > On 07/11/2013 11:32 PM, Liu Ping Fan wrote: >> Refcnt's atomic inc/dec ops are frequent and its idiom need no seq_cst >> order. So to get better performance, it worth to adopt _relaxed >> other than _seq_cst memory model on them. > > You

[Qemu-devel] [PATCH v2] atomic: using memory_order_relaxed for refcnt inc/dec ops

2013-07-13 Thread Liu Ping Fan
Refcnt's atomic inc/dec ops are frequent and its idiom need no seq_cst order. So to get better performance, it worth to adopt _relaxed other than _seq_cst memory model on them. We resort to gcc builtins. If gcc supports C11 memory model, __atomic_* buitlins is used, otherwise __sync_* builtins. S

Re: [Qemu-devel] [PATCH v5 10/14] pc: Add dimm paravirt SRAT info

2013-07-13 Thread Michael S. Tsirkin
On Fri, Jul 12, 2013 at 09:33:28AM +0800, Hu Tao wrote: > On Thu, Jul 11, 2013 at 11:49:01AM +0300, Michael S. Tsirkin wrote: > > On Thu, Jul 11, 2013 at 07:13:39AM +0200, Igor Mammedov wrote: > > > On Wed, 10 Jul 2013 13:10:03 +0300 > > > "Michael S. Tsirkin" wrote: > > > > > > > On Wed, Jun 26,

Re: [Qemu-devel] [PATCH v2] atomic: using memory_order_relaxed for refcnt inc/dec ops

2013-07-13 Thread Paolo Bonzini
Il 14/07/2013 04:53, Liu Ping Fan ha scritto: > Refcnt's atomic inc/dec ops are frequent and its idiom need no seq_cst > order. So to get better performance, it worth to adopt _relaxed > other than _seq_cst memory model on them. > > We resort to gcc builtins. If gcc supports C11 memory model, __at

Re: [Qemu-devel] [PATCH 0/4] alpha-softmmu fixes

2013-07-13 Thread Rob Landley
On 07/10/2013 08:49:35 AM, Richard Henderson wrote: On 07/09/2013 10:43 AM, Rob Landley wrote: > Do you have a kernel .config that boots on this board? I'd really like to try > this out myself... I haven't worked on this in quite some time. But I've located two branches that look like the

Re: [Qemu-devel] [PATCH v3 11/14] ioport: Switch dispatching to memory core layer

2013-07-13 Thread Paolo Bonzini
Il 13/07/2013 17:22, Anthony Liguori ha scritto: > 1) MMIO request goes to sPAPR PIO area, the vCPU was in BE mode but by > the time the handler is called, the value is in host byte order. > > 2) sPAPR (incorrectly) byte swaps by marking the region as little > endian (data is now garbage) > > 3)

Re: [Qemu-devel] [PATCH 3/4] exec: Support 64-bit operations in address_space_rw

2013-07-13 Thread Paolo Bonzini
Il 09/07/2013 03:01, Richard Henderson ha scritto: > Honor the implementation maximum access size, and at least check > the minimum access size. > > Signed-off-by: Richard Henderson > --- > exec.c | 68 > ++ > 1 file changed, 56 in