Re: [Qemu-devel] [PATCH v8 02/14] block/dirty-bitmap: add locked version of bdrv_release_dirty_bitmap

2017-11-10 Thread John Snow
On 10/30/2017 12:32 PM, Vladimir Sementsov-Ogievskiy wrote: > It is needed to realize bdrv_dirty_bitmap_release_successor in > the following patch. > OK, but... > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/dirty-bitmap.c | 25 - > 1 file changed, 20 ins

Re: [Qemu-devel] [PATCH for 2.11 4/5] xlnx-zcu102: Specify the max number of CPUs for the EP108

2017-11-10 Thread Alistair Francis
On Fri, Nov 10, 2017 at 11:53 AM, Emilio G. Cota wrote: > Just like the zcu102, the ep108 can instantiate several CPUs. > > Signed-off-by: Emilio G. Cota I completely missed this, thanks for the patch. Reviewed-by: Alistair Francis Alistair > --- > hw/arm/xlnx-zcu102.c | 1 + > 1 file chang

Re: [Qemu-devel] [PATCH for 2.11 1/5] qom: move CPUClass.tcg_initialize to a global

2017-11-10 Thread Alistair Francis
On Fri, Nov 10, 2017 at 12:23 PM, Eduardo Habkost wrote: > On Fri, Nov 10, 2017 at 02:53:42PM -0500, Emilio G. Cota wrote: >> 55c3cee ("qom: Introduce CPUClass.tcg_initialize", 2017-10-24) >> introduces a per-CPUClass bool that we check so that the target CPU >> is initialized for TCG only once. T

Re: [Qemu-devel] [Qemu-block] [PATCH v2 0/1] Add 8-byte wide AMD flash support, partial interleaving

2017-11-10 Thread Paolo Bonzini
On 10/11/2017 21:25, Mike Nawrocki wrote: > This patch set does a few things. First, it switches the AMD CFI flash MMIO > operations from the old MMIO API to the new one. Second, it enables 8-byte > wide > flash arrays. Finally, it adds flash interleaving using the "device-width" and > "max-device

Re: [Qemu-devel] [PATCH for 2.11 5/5] hw: add .min_cpus and .default_cpus fields to machine_class

2017-11-10 Thread Alistair Francis
On Fri, Nov 10, 2017 at 11:53 AM, Emilio G. Cota wrote: > max_cpus needs to be an upper bound on the number of vCPUs > initialized; otherwise TCG region initialization breaks. > > Some boards initialize a hard-coded number of vCPUs, which is not > captured by the global max_cpus and therefore brea

[Qemu-devel] [Bug 1563887] Re: qemu-system-ppc64 freezes on starting image on ppc64le

2017-11-10 Thread Robin H. Johnson
TL;DR: pass '-vga none' with -nographic, or redirect the screen somewhere! I ended up digging into this after it was mentioned by smoser. The bug is invalid because of a bad assumption in the QEMU inputs. smoser's workaround of usb=off removes USB as a workaround. The kernel, OpenFirmware, and QE

Re: [Qemu-devel] [PATCH v2] linux-user: fix is_proc_myself to check the paths via realpath

2017-11-10 Thread Laurent Vivier
Le 25/10/2017 à 05:34, Zach Riggle a écrit : > Previously, it was possible to get a handle to the "real" /proc/self/mem > by creating a symlink to it and opening the symlink, or opening e.g. > "./mem" after chdir'ing to "/proc/self". > > $ ln -s /proc/self self > $ cat self/maps > 6000

[Qemu-devel] UT Austin Virtualization project

2017-11-10 Thread william lin
Hi all, My name is William Lin. I am an undergrad student at UT Austin and for our virtualization class we have to contribute to a open source repo related to virtualization. I am working in a group of two with about a month of time. We are both comfortable with virtualization concepts and systems

Re: [Qemu-devel] [PATCH v2 for-2.11] block: Make bdrv_next() keep strong references

2017-11-10 Thread Paolo Bonzini
On 10/11/2017 18:25, Max Reitz wrote: > if (bs) { > +bdrv_ref(bs); > +bdrv_unref(old_bs); > return bs; > } Maybe instead goto... > it->phase = BDRV_NEXT_MONITOR_OWNED; > +} else { > +old_bs = it->bs; > } > > /

Re: [Qemu-devel] Yet another git submodule rant

2017-11-10 Thread Alexey Kardashevskiy
On 11/11/17 01:01, Peter Maydell wrote: > On 10 November 2017 at 13:46, Alexey Kardashevskiy wrote: >> And it will still be >> better than changing the $SRC_PATH when a user specifically asked not to do >> that by calling "./configure --source-path='. > > I'm not terribly happy with the submodule

[Qemu-devel] [Bug 1731588] [NEW] qemu-system-arm black screen and keyboard not detected

2017-11-10 Thread Kevin
Public bug reported: Hi guys, I try to emulate FreeRTOS with this guide : http://wiki.csie.ncku.edu.tw/embedded/Lab32 But, the keys on my keyboard are not taken into account. - Command line : qemu_stm32/arm-softmmu/qemu-system-arm -M stm32-p103 -monitor stdio -kernel build/main.bin -semihosti

Re: [Qemu-devel] Yet another git submodule rant

2017-11-10 Thread Alexey Kardashevskiy
On 11/11/17 01:22, Daniel P. Berrange wrote: > On Sat, Nov 11, 2017 at 12:46:36AM +1100, Alexey Kardashevskiy wrote: >> On 10/11/17 21:41, Daniel P. Berrange wrote: >>> On Fri, Nov 10, 2017 at 09:35:54PM +1100, Alexey Kardashevskiy wrote: On 09/11/17 00:01, Daniel P. Berrange wrote: > On W

Re: [Qemu-devel] [PATCH v2] linux-user: fix is_proc_myself to check the paths via realpath

2017-11-10 Thread Zach Riggle
Good catch. Relying on realpath() for *exe* does cause issues. A better general solution (which handles the "exe" case) is to use open(2) with O_PATH | O_NOFOLLOW for the candidate path (e.g. /proc/self/exe) and to do the same for the path we're testing along with readlink(). If, in the process

Re: [Qemu-devel] [PATCH v2] linux-user: fix is_proc_myself to check the paths via realpath

2017-11-10 Thread Zach Riggle
I wrote up a quick example to show that this should work specifically for /proc/self/exe: #define _GNU_SOURCE #include #include #include #include int main(int argc, char** argv) { int fd = open("/proc/self/exe", O_NOFOLLOW | O_PATH); system("ls -la /proc/$PPID/fd/"); } *Zach Riggle*

Re: [Qemu-devel] [Qemu-block] [PATCH] block: all I/O should be completed before removing throttle timers.

2017-11-10 Thread l00284672
ok,thanks ! On 2017/11/10 23:33, Stefan Hajnoczi wrote: On Sat, Oct 21, 2017 at 01:34:00PM +0800, Zhengui Li wrote: From: Zhengui In blk_remove_bs, all I/O should be completed before removing throttle timers. If there has inflight I/O, removing throttle timers here will cause the inflight I/

[Qemu-devel] [PATCH 0/2] tpm: Handle failure mode of backend device better

2017-11-10 Thread Stefan Berger
The following two patches fix the case that the backend device, e.g., tpm_emulator, could not be initialized and the TIS frontend needs to go into failure mode. Stefan Stefan Berger (2): tpm_tis: Return TPM_VERSION_UNSPEC in case of BE failure tpm_tis: Return 0 for every register in case o

[Qemu-devel] [PATCH 2/2] tpm_tis: Return 0 for every register in case of failure mode

2017-11-10 Thread Stefan Berger
Rather than returning ~0, return 0 for every register in case of failure mode. The '0' is better to indicate that there's no device there. Signed-off-by: Stefan Berger --- hw/tpm/tpm_tis.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/tpm/tpm_tis.c b/hw/tpm/tpm_tis.c ind

[Qemu-devel] [PATCH 1/2] tpm_tis: Return TPM_VERSION_UNSPEC in case of BE failure

2017-11-10 Thread Stefan Berger
In case the backend has a failure, such as the tpm_emulator's CMD_INIT failing, the TIS goes into failure mode and does not respond to reads or writes to MMIO registers. In this case we need to prevent the ACPI table from being added and the straight-forward way is to indicate that there's no known

Re: [Qemu-devel] [PATCH] linux-user: Support explicit targets for PowerPC

2017-11-10 Thread no-reply
Hi, This series failed build test on s390x host. Please find the details below. Message-id: 492245211.936616.1510023015797.javamail.zim...@xes-inc.com Type: series Subject: [Qemu-devel] [PATCH] linux-user: Support explicit targets for PowerPC === TEST SCRIPT BEGIN === #!/bin/bash # Testing scrip

Re: [Qemu-devel] kvm_intel fails to load on Conroe CPUs running Linux 4.12

2017-11-10 Thread Gerhard Wiesinger
On 10.11.2017 17:29, Paolo Bonzini wrote: On 10/11/2017 16:33, Gerhard Wiesinger wrote: Hello Paolo, Any update for a new patch? Yes, https://marc.info/?l=kvm&m=150997149623548&w=2 Paolo Works also for Fedora 26: https://koji.fedoraproject.org/koji/buildinfo?buildID=996781 kernel-4.13.12

<    1   2   3