Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions

2015-10-04 Thread Markus Armbruster
Peter Maydell writes: > On 2 October 2015 at 18:20, Markus Armbruster wrote: >> QMP command device-list-properties regressed in 2.1: it can crash or >> leave dangling pointers behind. >> >> -device FOO,help regressed in 2.2: it no longer works for >> non-pluggable devices. I tried to fix that s

Re: [Qemu-devel] [PATCH v8 04/54] Move configuration section writing

2015-10-04 Thread Amit Shah
On (Tue) 29 Sep 2015 [09:37:28], Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > The vmstate_configuration is currently written > in 'qemu_savevm_state_begin', move it to > 'qemu_savevm_state_header' since it's got a hard > requirement that it must be the 1st thing after

Re: [Qemu-devel] [PATCH] Add syscalls for -runas and -chroot tothe seccomp sandbox

2015-10-04 Thread Markus Armbruster
"Namsun Ch'o" writes: >> Our intention since the beginning was to protect the host from the >> illegal guest operations. But you do have an interesting point about >> flaws on qemu itself. Perhaps this might be something I could work on to >> improve (start a bigger whitelist and get it tighter b

Re: [Qemu-devel] [PATCH] Add syscalls for -runas and -chroot tothe seccomp sandbox

2015-10-04 Thread Markus Armbruster
"Namsun Ch'o" writes: >> If we intend seccomp to protect against flaws during QEMU setup, then having >> it earlier is neccessary. eg QEMU opening a corrupt qcow2 image which might >> exploit QEMU before the guest CPUs start. > >> If the latter is the case, then we could start with a relaxed secc

Re: [Qemu-devel] [PATCH 1/3] Target-microblaze: Remove unnecessary variable

2015-10-04 Thread Markus Armbruster
Michael Tokarev writes: > 25.09.2015 11:37, Shraddha Barke wrote: >> Compress lines and remove the variable . > > Applied to -trivial, removing this piece of commit message: > > --- >> Change made using Coccinelle script >> >> @@ >> expression ret; >> @@ >> - if (ret) return ret; >> - return 0;

Re: [Qemu-devel] [PATCH 21/36] misc: spelling

2015-10-04 Thread Markus Armbruster
Michael Tokarev writes: > 25.09.2015 19:08, Eric Blake wrote: >> On 09/25/2015 08:03 AM, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau >>> >>> --- >>> monitor.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> Trivial, can be applied now without waiting for pend

Re: [Qemu-devel] [PATCH v8 00/54] Postcopy implementation

2015-10-04 Thread Bharata B Rao
On Mon, Sep 28, 2015 at 05:51:39PM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > This is the 8th cut of my version of postcopy. > > The userfaultfd linux kernel code is now in the upstream kernel > tree, and so 4.3-rc3 can be used without modification. > > Th

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Guenter Roeck
On 10/04/2015 07:21 PM, Peter Crosthwaite wrote: On Sun, Oct 4, 2015 at 6:08 PM, Guenter Roeck wrote: On 10/04/2015 02:38 PM, Beniamino Galvani wrote: On Sun, Oct 04, 2015 at 02:11:35PM -0700, Guenter Roeck wrote: What is your qemu command line ? qemu-system-arm \ -M cubieb

[Qemu-devel] [PATCH v4] linux-user/syscall.c: malloc()/calloc() to g_malloc()/g_try_malloc()/g_new0()

2015-10-04 Thread Harmandeep Kaur
Convert malloc()/calloc() calls to g_malloc()/g_try_malloc()/g_new0() in linux-user/syscall.c file Signed-off-by: Harmandeep Kaur --- v1->v2 convert the free() call in host_to_target_semarray() to g_free() and calls g_try_malloc(count) instead of g_try_malloc(sizeof(count)) v2->v3 used g_try_n

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Peter Crosthwaite
On Sun, Oct 4, 2015 at 6:08 PM, Guenter Roeck wrote: > On 10/04/2015 02:38 PM, Beniamino Galvani wrote: >> >> On Sun, Oct 04, 2015 at 02:11:35PM -0700, Guenter Roeck wrote: >>> >>> What is your qemu command line ? >> >> >> qemu-system-arm \ >> -M cubieboard \ >> -kernel .

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Guenter Roeck
On 10/04/2015 02:38 PM, Beniamino Galvani wrote: On Sun, Oct 04, 2015 at 02:11:35PM -0700, Guenter Roeck wrote: What is your qemu command line ? qemu-system-arm \ -M cubieboard \ -kernel ../linux/zImage-dtb \ -serial stdio \ -append "console=

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Peter Maydell
On 4 October 2015 at 22:39, Peter Crosthwaite wrote: > On Sun, Oct 4, 2015 at 12:56 PM, Beniamino Galvani > wrote: >> Ignoring attempt to switch CPSR_A flag from non-secure world with SCR.AW bit >> clear >> Ignoring attempt to switch CPSR_F flag from non-secure world with SCR.FW bit >> clear >

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Beniamino Galvani
On Sun, Oct 04, 2015 at 02:11:35PM -0700, Guenter Roeck wrote: > What is your qemu command line ? qemu-system-arm \ -M cubieboard \ -kernel ../linux/zImage-dtb \ -serial stdio \ -append "console=ttyS0 rw root=/dev/nfs nfsroot=10.0.0.1:/nfs,v3 ip=10.

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Peter Crosthwaite
On Sun, Oct 4, 2015 at 12:56 PM, Beniamino Galvani wrote: > On Sat, Oct 03, 2015 at 02:31:08PM -0700, Peter Crosthwaite wrote: >> QEMU cubieboard has no usable storage media, but the real hardware >> does have AHCI sata. I added sysbus-ahci at the right place but turns >> out the SATA controller h

Re: [Qemu-devel] [PULL 00/10] Fix device introspection regressions

2015-10-04 Thread Peter Maydell
On 2 October 2015 at 18:20, Markus Armbruster wrote: > QMP command device-list-properties regressed in 2.1: it can crash or > leave dangling pointers behind. > > -device FOO,help regressed in 2.2: it no longer works for > non-pluggable devices. I tried to fix that some time ago[*], but my > fix f

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Guenter Roeck
On 10/04/2015 12:56 PM, Beniamino Galvani wrote: On Sat, Oct 03, 2015 at 02:31:08PM -0700, Peter Crosthwaite wrote: QEMU cubieboard has no usable storage media, but the real hardware does have AHCI sata. I added sysbus-ahci at the right place but turns out the SATA controller has some custom pow

Re: [Qemu-devel] [PATCH v3 2/4] firmware: use acpi to detect QEMU fw_cfg device for sysfs fw_cfg driver

2015-10-04 Thread Gabriel L. Somlo
On Sun, Oct 04, 2015 at 04:24:00PM -0400, Gabriel L. Somlo wrote: > On Sun, Oct 04, 2015 at 10:54:57AM +0300, Michael S. Tsirkin wrote: > > On Sat, Oct 03, 2015 at 07:28:07PM -0400, Gabriel L. Somlo wrote: > > > > > > Instead of blindly probing fw_cfg registers at known IOport and MMIO > > > locat

Re: [Qemu-devel] [PATCH v3 2/4] firmware: use acpi to detect QEMU fw_cfg device for sysfs fw_cfg driver

2015-10-04 Thread Gabriel L. Somlo
On Sun, Oct 04, 2015 at 10:54:57AM +0300, Michael S. Tsirkin wrote: > On Sat, Oct 03, 2015 at 07:28:07PM -0400, Gabriel L. Somlo wrote: > > From: Gabriel Somlo > > > > Instead of blindly probing fw_cfg registers at known IOport and MMIO > > locations, use the ACPI subsystem to determine whether a

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Beniamino Galvani
On Sat, Oct 03, 2015 at 02:31:08PM -0700, Peter Crosthwaite wrote: > QEMU cubieboard has no usable storage media, but the real hardware > does have AHCI sata. I added sysbus-ahci at the right place but turns > out the SATA controller has some custom power/clock (not really > sure??) registers speci

Re: [Qemu-devel] [PATCH v4] target-arm: Break the TB after ISB to execute self-modified code correctly

2015-10-04 Thread Peter Maydell
On 2 October 2015 at 13:38, Sergey Sorokin wrote: > If any store instruction writes the code inside the same TB > after this store insn, the execution of the TB must be stopped > to execute new code correctly. > As described in ARMv8 manual D3.4.6 a self-modified code need to do > IC invalidation

Re: [Qemu-devel] [PATCH 3/4] checkpatch: adapt some tests to QEMU

2015-10-04 Thread Peter Maydell
On 17 September 2015 at 17:32, Paolo Bonzini wrote: > > > On 17/09/2015 18:16, Peter Maydell wrote: >> On 17 September 2015 at 17:00, Paolo Bonzini wrote: >>> >>> >>> On 17/09/2015 16:24, Peter Maydell wrote: Can we revert this one, please? Checkpatch now warns about constructs like >>>

[Qemu-devel] 'make check' now has "Warning: path not on HugeTLBFS: /tmp/vhost-test-MJ3mXo" warnings

2015-10-04 Thread Peter Maydell
I noticed that 'make check' now produces a warning while running the check-qtest-i386 and -x86_64 tests: "Warning: path not on HugeTLBFS: /tmp/vhost-test-MJ3mXo" which I think is a relatively recent development. Does anybody know what's causing this and if we can reasonably suppress it or turn it

Re: [Qemu-devel] [PATCH v3] linux-user/syscall.c: malloc()/calloc() to g_malloc()/g_try_malloc()/g_new0()

2015-10-04 Thread Stefan Hajnoczi
On Thu, Oct 1, 2015 at 8:15 PM, Harmandeep Kaur wrote: > @@ -1881,7 +1881,7 @@ static struct iovec *lock_iovec(int type, abi_ulong > target_addr, > return NULL; > } > > -vec = calloc(count, sizeof(struct iovec)); > +vec = g_try_new0(struct iovec, count); > if (vec == NUL

Re: [Qemu-devel] [PATCH] MAINTAINERS: Small IDE/FDC touchup

2015-10-04 Thread Michael Tokarev
24.09.2015 20:50, John Snow wrote: > libqos/ahci and tests/fdc-test are under my purview also, > include them in the appropriate stanzas. Applied to -trivial too, just in case :) Thank you for adding more files to MAINTAINERS! /mjt

Re: [Qemu-devel] [PATCH] qtest/ahci: fix redundant assertion

2015-10-04 Thread Michael Tokarev
21.09.2015 23:55, John Snow wrote: > Fixes https://bugs.launchpad.net/qemu/+bug/1497711 > > (!ncq || (ncq && lba48)) is the same as > (!ncq || lba48). Applied to -trivial too, just in case :) Thank you! /mjt

Re: [Qemu-devel] [PATCH] linux-user: Remove type casts to union type

2015-10-04 Thread Michael Tokarev
08.02.2015 17:40, Stefan Weil wrote: > Casting to a union type is a gcc (and clang) extension. Other compilers > might not support it. This is not a problem today, but the type casts > can be removed easily. Smatch now no longer complains like before: I've applied this patch with the following cha

Re: [Qemu-devel] [PATCH v3 1/4] firmware: introduce sysfs driver for QEMU's fw_cfg device

2015-10-04 Thread kbuild test robot
Hi Gabriel, [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) drivers/firmware/qemu_fw_cfg.c:

[Qemu-devel] [Bug 1502613] [NEW] [Feature Request] Battery Status / Virtual Battery

2015-10-04 Thread Wolfgang Andreas
Public bug reported: When using virtualization on notebooks heavily then virtual machines do not realize that they're running on a notebook device causing high power consumption because they're not switching into a optimized "laptop mode". This leads to the circumstance that they are trying to do

[Qemu-devel] CharUDP - Connection refused

2015-10-04 Thread poma
Hi Fi With or without the patch[1] the same result - "Connection refused". UDP network console, the character device acts as a UDP netconsole service, sending and receiving packets. This is a lossy service. = HOST: virt-manager: Add New Virtual Serial Device Device Type: UDP net cosnole (ud

[Qemu-devel] [PATCH v5 1/1] intel_iommu: Add support for translation for devices behind bridges

2015-10-04 Thread Knut Omang
- Use a hash table indexed on bus pointers to store information about buses instead of using the bus numbers. Bus pointers are stored in a new VTDBus struct together with the vector of device address space pointers indexed by devfn. - The bus number is still used for lookup for selective SID

[Qemu-devel] [PATCH v5 0/1] intel_iommu: Add support for translation for devices behind bridges

2015-10-04 Thread Knut Omang
This patch set has been completely reimplemented according to ideas from the discussion of v2. It still solves the same problem, but does so only within the Intel IOMMU code and Q35, without changing the IOMMU interface. This eliminates the need for any separate interface change patch. This is t

Re: [Qemu-devel] [PATCH v4 1/1] intel_iommu: Add support for translation for devices behind bridges

2015-10-04 Thread Knut Omang
On Sun, 2015-09-27 at 13:07 +0300, Michael S. Tsirkin wrote: > On Sat, Sep 26, 2015 at 08:09:56AM +0200, Knut Omang wrote: > > - Use a hash table indexed on bus pointers to store information > > about buses > > instead of using the bus numbers. > > Bus pointers are stored in a new VTDBus struct

Re: [Qemu-devel] [PATCH] target-tilegx: Implement floating point temporarily

2015-10-04 Thread Chen Gang
After the temporary floating point patch, our tilegx qemu can test the gcc testsuite successfully (no any unimplementation issues). The test result is:             === gcc Summary ===   # of expected passes          77012   # of unexpected failures      622   # of unexpected successes     7   # o

[Qemu-devel] [PATCH] target-tilegx: Implement floating point temporarily

2015-10-04 Thread Chen Gang
>From 4d12af14f361fb5e3a893fc68a599be9ea17d1dc Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sun, 4 Oct 2015 18:00:53 +0800 Subject: [PATCH] target-tilegx: Implement floating point temporarily It is a temporary implementation, but it can pass gcc testsuite. Signed-off-by: Chen Gang ---  target

[Qemu-devel] [PATCH] target-tilegx: Let prefetch nop instructions return before allocating dest temporary register

2015-10-04 Thread Chen Gang
>From 40ec3f1c75b4c97e3e0495c9e465be898f48a652 Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sun, 4 Oct 2015 17:34:17 +0800 Subject: [PATCH] target-tilegx: Let prefetch nop instructions return before allocating dest temporary register Or it will cause issue by the dest temporary registers. Sig

[Qemu-devel] [PATCH v4] target-tilegx: Support iret instruction and related special registers

2015-10-04 Thread Chen Gang
>From 8e8d35fffd735df997c78324b301f22cf270b515 Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sun, 4 Oct 2015 17:41:14 +0800 Subject: [PATCH v4] target-tilegx: Support iret instruction and related special registers Acording to the __longjmp tilegx libc implementation, and reference from tilegx I

[Qemu-devel] [PATCH v2] target-tilegx: Use TILEGX_EXCP_OPCODE_UNKNOWN and TILEGX_EXCP_OPCODE_UNIMPLEMENTED correctly

2015-10-04 Thread Chen Gang
>From 0f53a45b3c29e3355cc6b2183ee084e62b86e5fe Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sun, 4 Oct 2015 13:34:33 +0800 Subject: [PATCH v2] target-tilegx: Use TILEGX_EXCP_OPCODE_UNKNOWN and TILEGX_EXCP_OPCODE_UNIMPLEMENTED correctly For some cases, they are for TILEGX_EXCP_OPCODE_UNKNOWN, n

[Qemu-devel] [PATCH v2] target-tilegx: Implement v2mults instruction

2015-10-04 Thread Chen Gang
>From 298aa5e9be6373fea7b30236bd3e90352c6e693a Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Sat, 3 Oct 2015 10:42:01 +0800 Subject: [PATCH v2] target-tilegx: Implement v2mults instruction Just according to v1multu instruction implementation. Signed-off-by: Chen Gang ---  target-tilegx/helper.

[Qemu-devel] [PATCH v2] target-tilegx: Implement v?int_* instructions

2015-10-04 Thread Chen Gang
>From 418c1600c481f0acbde42987db286b48f1848399 Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Fri, 2 Oct 2015 11:00:37 +0800 Subject: [PATCH v2] target-tilegx: Implement v?int_* instructions. Signed-off-by: Chen Gang ---  target-tilegx/helper.h      |  5  target-tilegx/simd_helper.c | 56 ++

[Qemu-devel] [PATCH v2] target-tilegx: Implement v2sh* instructions

2015-10-04 Thread Chen Gang
>From be4b6be54c79d9ca22431f749f31e0c7b9fdd091 Mon Sep 17 00:00:00 2001 From: Chen Gang Date: Fri, 2 Oct 2015 09:19:56 +0800 Subject: [PATCH v2] target-tilegx: Implement v2sh* instructions It is just according to v1sh* instructions implementation. Signed-off-by: Chen Gang ---  target-tilegx/tra

[Qemu-devel] [PATCH v2] target-tilegx: Implement v?int_* instructions.

2015-10-04 Thread gang . chen . 5i5j
From: Chen Gang Signed-off-by: Chen Gang --- target-tilegx/helper.h | 5 target-tilegx/simd_helper.c | 56 + target-tilegx/translate.c | 14 3 files changed, 75 insertions(+) diff --git a/target-tilegx/helper.h b/target-tile

[Qemu-devel] [PATCH v2] target-tilegx: Implement v2sh* instructions

2015-10-04 Thread gang . chen . 5i5j
From: Chen Gang It is just according to v1sh* instructions implementation. Signed-off-by: Chen Gang --- target-tilegx/translate.c | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/target-tilegx/translate.c b/target-tilegx/translate.c index 6ab66f9..9bb8857

[Qemu-devel] [PATCH v2] target-tilegx: Use TILEGX_EXCP_OPCODE_UNKNOWN and TILEGX_EXCP_OPCODE_UNIMPLEMENTED correctly

2015-10-04 Thread gang . chen . 5i5j
From: Chen Gang For some cases, they are for TILEGX_EXCP_OPCODE_UNKNOWN, not for TILEGX_EXCP_OPCODE_UNIMPLEMENTED. Also for some cases, they are for TILEGX_EXCP_OPCODE_UNIMPLEMENTED, not for TILEGX_EXCP_OPCODE_UNKNOWN. When analyzing issues, the correct printing information is necessary, e.g. g

[Qemu-devel] [PATCH v2] target-tilegx: Implement v2mults instruction

2015-10-04 Thread gang . chen . 5i5j
From: Chen Gang Just according to v1multu instruction implementation. Signed-off-by: Chen Gang --- target-tilegx/helper.h | 1 + target-tilegx/simd_helper.c | 13 + target-tilegx/translate.c | 4 3 files changed, 18 insertions(+) diff --git a/target-tilegx/helper.h b

Re: [Qemu-devel] [Qemu-discuss] TCP options ipv4 and ipv6 have no effect

2015-10-04 Thread Paolo Bonzini
On 03/10/2015 00:36, Peter Maydell wrote: > > I agree about the (!ipv4 || !ipv6) condition though. > The three states I listed above ought to correspond > to "qemu_opt not set", "qemu_opt set to false" and > "qemu_opt set to true", The problem is that the underlying QemuOpts-based code treats "qe

Re: [Qemu-devel] QEMU+Linux ARMv7A current state

2015-10-04 Thread Peter Maydell
On 3 October 2015 at 23:14, Peter Crosthwaite wrote: > On Sat, Oct 3, 2015 at 2:51 PM, Peter Maydell > wrote: >> Did you build your kernel with LPAE or not? I think an LPAE >> config ought to avoid the PCI highmem bug (and it's definitely >> what you want for anything that's Cortex-A15 based). >

[Qemu-devel] [Bug 921208] Re: win7/x64 installer hangs on startup with 0x0000005d.

2015-10-04 Thread Ludovic
Any news regarding this issue ? I applied the first patch, but I'm unsure how to apply the second patch. I get either DRIVER_IRQL_NOT_LESS_OR_EQUAL or KMODE_EXCEPTION_NOT_HANDLED during the Windows 7 x64 install. -- You received this bug notification because you are a member of qemu- devel-ml, w

Re: [Qemu-devel] [PATCH v7 03/24] util: add linux-only memfd fallback

2015-10-04 Thread Michael S. Tsirkin
On Fri, Oct 02, 2015 at 05:57:34PM +0200, Paolo Bonzini wrote: > > Will be easier to apply if this refactoring is a separate patch. > > Is someone doing anything conflicting in this area? (My answer: not > that I know of). Yes - I'm trying to refactor mmap-alloc, that's why I asked for this. Con

Re: [Qemu-devel] [PATCH v2] exec: factor out duplicate mmap code

2015-10-04 Thread Michael S. Tsirkin
On Fri, Oct 02, 2015 at 10:48:13AM +1000, Richard Henderson wrote: > On 10/01/2015 10:58 PM, Michael S. Tsirkin wrote: > >Anonymous and file-backed RAM allocation are now almost exactly the same. > > > >Reduce code duplication by moving RAM mmap code out of oslib-posix.c and > >exec.c. > > > >Repor

Re: [Qemu-devel] [PATCH v3 2/4] firmware: use acpi to detect QEMU fw_cfg device for sysfs fw_cfg driver

2015-10-04 Thread Michael S. Tsirkin
On Sat, Oct 03, 2015 at 07:28:07PM -0400, Gabriel L. Somlo wrote: > From: Gabriel Somlo > > Instead of blindly probing fw_cfg registers at known IOport and MMIO > locations, use the ACPI subsystem to determine whether a QEMU fw_cfg > device is present, and, if found, to initialize it. > > This l