Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting

2017-01-16 Thread Doug Goldstein
On 1/16/17 5:46 AM, Stefan Hajnoczi wrote: > On Fri, Jan 13, 2017 at 09:15:49AM -0600, Doug Goldstein wrote: >> On 1/13/17 6:02 AM, Stefan Hajnoczi wrote: >>> On Thu, Jan 12, 2017 at 10:57:53AM -0600, Doug Goldstein wrote: >>>> On 1/12/17

Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting

2017-01-13 Thread Doug Goldstein
On 1/13/17 6:02 AM, Stefan Hajnoczi wrote: > On Thu, Jan 12, 2017 at 10:57:53AM -0600, Doug Goldstein wrote: >> On 1/12/17 5:46 AM, Stefan Hajnoczi wrote: >>> The virtio_queue_set_notification() nesting introduced for AioContext >>> polling >>> raised an as

Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting

2017-01-13 Thread Doug Goldstein
On 1/12/17 2:05 PM, Michael S. Tsirkin wrote: > On Thu, Jan 12, 2017 at 10:57:53AM -0600, Doug Goldstein wrote: >> On 1/12/17 5:46 AM, Stefan Hajnoczi wrote: >>> The virtio_queue_set_notification() nesting introduced for AioContext >>> polling >>> raised an as

Re: [Qemu-devel] [PATCH 0/2] virtio: revert virtio_queue_set_notification() nesting

2017-01-12 Thread Doug Goldstein
DHCP address the screen immediately flashes over to the UEFI shell. Its like a timeout is getting hit and just dropping me to the shell. -- Doug Goldstein signature.asc Description: OpenPGP digital signature

Re: [Qemu-devel] assert hit on master with q35 machine type

2017-01-10 Thread Doug Goldstein
pu_exec (cpu=cpu@entry=0x565bc7f0) at /home/doug/work/qemu/kvm-all.c:1968 #20 0x55797d48 in qemu_kvm_cpu_thread_fn (arg=0x565bc7f0) at /home/doug/work/qemu/cpus.c:998 #21 0x748726ca in start_thread (arg=0x7fffead75700) at pthread_create.c:333 #22 0x745ac0af in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:105 -- Doug Goldstein signature.asc Description: OpenPGP digital signature

[Qemu-devel] assert hit on master with q35 machine type

2017-01-07 Thread Doug Goldstein
rder=n -device qxl-vga -gdb tcp::1234 It happens almost right away. Let me know how I can help track this down further. Thanks. -- Doug Goldstein signature.asc Description: OpenPGP digital signature

Re: [Qemu-devel] [PATCH] ehci: Fix crash with isoc usb packets

2013-09-25 Thread Doug Goldstein
) { /* transfer crosses page border */ uint32_t len2 = off + len - 4096; -- 1.8.3.1 Ping. Don't see this in master (and as such its missing from mdroth's 1.6.1 patch set). -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] [PATCH 13/38] block: expect errors from bdrv_co_is_allocated

2013-09-25 Thread Doug Goldstein
)); +return 0; +} sector_num += num; remaining -= num; if (ret) { -- 1.7.9.5 -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] [PATCH for-1.6] isapc: disable kvmvapic

2013-09-25 Thread Doug Goldstein
the property name to 'vapic' fixes both issues for me. Thanks, Cole Ping this patch. I had to make the same change as Cole for Gentoo as well. -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] Patch Round-up for stable 1.6.1, freeze on 2013-09-30

2013-09-25 Thread Doug Goldstein
pinged on the first one since we had a similar bug in Gentoo and I've carried that patch as well. I'm also pinging a number of other patches I've been carrying. -- Doug Goldstein

Re: [Qemu-devel] [PATCH] configure: detect endian via compile test

2013-09-25 Thread Doug Goldstein
this patch? Richard already reviewed it. See also http://patchwork.ozlabs.org/patch/268687/ for another configure patch waiting for a commit. Regards, Stefan Ping on getting this into master (and then over to stable). -- Doug Goldstein

Re: [Qemu-devel] ARM virtio-serial not showing up (was: Re: ARM virtio-scsi - disks never show up)

2013-09-06 Thread Doug Goldstein
? Are virtio-mmio devices self-describing or do they have to be listed in the device tree? (it's going to really complicate things if the latter ...) Attached below is the command line and kernel output. Rich. -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] [PATCH] qapi-types.py: Fix enum struct sizes on i686

2013-09-01 Thread Doug Goldstein
in gnu99, not sure as to what GCC added them but I know GCC 3.2 didn't have them. -- Doug Goldstein

Re: [Qemu-devel] [PATCH v2 2/2] osdep: warn if opening a file O_DIRECT on tmpfs fails

2013-08-21 Thread Doug Goldstein
] (or get_xip_mem but that's another story). [1] https://www.redhat.com/archives/libvir-list/2013-August/msg00768.html [2] http://lxr.linux.no/#linux+v3.10.9/fs/open.c#L651 -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] Patch Round-up for stable 1.5.3, freeze on 2013-08-16

2013-08-13 Thread Doug Goldstein
-off-by: Luiz Capitulino lcapitul...@redhat.com -- Doug Goldstein

Re: [Qemu-devel] [PATCH 02/11] iov: handle EOF in iov_send_recv

2013-08-02 Thread Doug Goldstein
... and should also be in 1.5.2. Paolo Nudge so this doesn't get forgotten about. It hasn't hit master yet. -- Doug Goldstein

Re: [Qemu-devel] [PATCH for-1.6 01/11] ignore SIGPIPE in qemu-img and qemu-io

2013-08-02 Thread Doug Goldstein
for this one. Nudge so this isn't forgotten about since it hasn't hit master yet. -- Doug Goldstein

Re: [Qemu-devel] Patch Round-up for stable 1.4.2, freeze on Monday

2013-05-17 Thread Doug Goldstein
Hajnoczi stefa...@redhat.com --- It only adds a test for something that was fixed in 1.4.1 (maybe was fixed by the final 1.4.0 release I can't recall). -- Doug Goldstein

[Qemu-devel] [PATCH] Advertise --libdir in configure --help output

2013-03-17 Thread Doug Goldstein
The configure script allows you to supply a libdir via --libdir but was not advertising this in --help. Signed-off-by: Doug Goldstein car...@cardoe.com CC: qemu-triv...@nongnu.org --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 46a7594..497ce29

[Qemu-devel] [PATCH v3 0/2] bridge helper: includedir conf arg

2013-03-17 Thread Doug Goldstein
changes from Stefan Hajnoczi Doug Goldstein (2): bridge helper: unified error cleanup for parse_acl_file bridge helper: support conf dirs qemu-bridge-helper.c | 75 +++- 1 file changed, 63 insertions(+), 12 deletions(-) -- 1.8.1.5

[Qemu-devel] [PATCH v3 1/2] bridge helper: unified error cleanup for parse_acl_file

2013-03-17 Thread Doug Goldstein
Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein car...@cardoe.com Reviewed-by: Stefan Hajnoczi stefa...@redhat.com Reviewed-by: Corey Bryant cor...@linux.vnet.ibm.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar

[Qemu-devel] [PATCH v3 2/2] bridge helper: support conf dirs

2013-03-17 Thread Doug Goldstein
Allow the bridge helper to take a config directory rather than having to specify every file in the directory manually via an include statement. Signed-off-by: Doug Goldstein car...@cardoe.com Reviewed-by: Stefan Hajnoczi stefa...@redhat.com Reviewed-by: Corey Bryant cor...@linux.vnet.ibm.com CC

Re: [Qemu-devel] [Qemu-stable] [SeaBIOS] problems with freeBSD

2013-03-06 Thread Doug Goldstein
after some troubleshooting help from Peter. Similarly our Ubuntu OpenStack machines at work had quirks resolved by dropping the coreboot seabios images on them. -- Doug Goldstein

[Qemu-devel] [PATCHv3 0/2] bridge helper: includedir conf arg

2013-03-06 Thread Doug Goldstein
v2: - Integrated review changes from Corey Bryant - Integrated review changes from Stefan Hajnoczi Change from v1: - Reversed patch order to make the series clearer - Integrated review changes from Corey Bryant - Integrated review changes from Stefan Hajnoczi Doug Goldstein (2): bridge helper

[Qemu-devel] [PATCHv3 2/2] bridge helper: support conf dirs

2013-03-06 Thread Doug Goldstein
Allow the bridge helper to take a config directory rather than having to specify every file in the directory manually via an include statement. Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor

[Qemu-devel] [PATCHv3 1/2] bridge helper: unified error cleanup for parse_acl_file

2013-03-06 Thread Doug Goldstein
Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor...@linux.vnet.ibm.com TO: qemu-devel@nongnu.org --- qemu

Re: [Qemu-devel] [PATCHv2 1/2] bridge helper: unified error cleanup for parse_acl_file

2013-03-04 Thread Doug Goldstein
On Mon, Mar 4, 2013 at 10:27 AM, Corey Bryant cor...@linux.vnet.ibm.com wrote: On 03/02/2013 01:58 AM, Doug Goldstein wrote: Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu

Re: [Qemu-devel] [PATCH] configure: require spice-protocol = 0.12.3

2013-03-03 Thread Doug Goldstein
as well. Reviewed-by: Doug Goldstein car...@cardoe.com -- Doug Goldstein

[Qemu-devel] [Bug 1130769] Re: [RESOLVED] VirtFS (virtio-9p-pci) error: Parameter 'driver' expects device type

2013-03-01 Thread Doug Goldstein
I've updated the Gentoo docs for the flag to read: Enable VirtFS via fsdev (host) and virtio-9p-pci (guest). See http://wiki.qemu.org/Documentation/9psetup Hopefully that's a bit more clear that you need to enable that for this support. -- You received this bug notification because you are a

[Qemu-devel] [PATCHv2 0/2] bridge helper: includedir conf arg

2013-03-01 Thread Doug Goldstein
v1: - Reversed patch order to make the series clearer - Integrated review changes from Corey Bryant - Integrated review changes from Stefan Hajnoczi Doug Goldstein (2): bridge helper: unified error cleanup for parse_acl_file bridge helper: support conf dirs qemu-bridge-helper.c | 75

[Qemu-devel] [PATCHv2 2/2] bridge helper: support conf dirs

2013-03-01 Thread Doug Goldstein
Allow the bridge helper to take a config directory rather than having to specify every file in the directory manually via an include statement. Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor

[Qemu-devel] [PATCHv2 1/2] bridge helper: unified error cleanup for parse_acl_file

2013-03-01 Thread Doug Goldstein
Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor...@linux.vnet.ibm.com TO: qemu-devel@nongnu.org --- qemu

[Qemu-devel] [PATCH 0/2] bridge helper: includedir conf arg

2013-02-24 Thread Doug Goldstein
Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor...@linux.vnet.ibm.com TO: qemu-devel@nongnu.org Doug Goldstein (2): bridge helper: support conf dirs bridge helper: unified error cleanup for parse_acl_file qemu-bridge-helper.c | 73

[Qemu-devel] [PATCH 1/2] bridge helper: support conf dirs

2013-02-24 Thread Doug Goldstein
Allow the bridge helper to take a config directory rather than having to specify every file in the directory manually via an include statement. Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor

[Qemu-devel] [PATCH 2/2] bridge helper: unified error cleanup for parse_acl_file

2013-02-24 Thread Doug Goldstein
Handle errors and cleanup from the error in a unified place for parse_acl_file(). Signed-off-by: Doug Goldstein car...@cardoe.com CC: Anthony Liguori aligu...@us.ibm.com CC: Richa Marwaha rmar...@linux.vnet.ibm.com CC: Corey Bryant cor...@linux.vnet.ibm.com TO: qemu-devel@nongnu.org --- qemu

Re: [Qemu-devel] [PATCH] move qemu-ga from bin to libexec dir, use $HELPERS

2013-02-21 Thread Doug Goldstein
. QEMU can't spawn it, it is spawned in *guest* by a startup script or some event daemon (such as systemd or udev). Thanks, /mjt -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] Patch queue for qemu-1.1.3 stable release

2013-02-19 Thread Doug Goldstein
other than ah that's a good patch to use. -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] Patch queue for qemu-1.1.3 stable release

2013-02-04 Thread Doug Goldstein
for what would amount to a 1.2.3 release as well. But I'll gladly test your branch and provide some feedback when possible. -- Doug Goldstein

Re: [Qemu-devel] [PATCH] link seccomp only with softmmu targets

2013-01-19 Thread Doug Goldstein
.x queue as well. -- Doug Goldstein

Re: [Qemu-devel] qemu-kvm command-line compat

2013-01-18 Thread Doug Goldstein
necessary. And as mentioned in [2], I'm not sure such a wrapper is welcome generally, due to different reception of defaults by different people. Comments? [1] http://patchwork.ozlabs.org/patch/205676/ [2] https://patchwork.kernel.org/patch/1531761/ Thanks, /mjt -- Doug Goldstein

Re: [Qemu-devel] [Qemu-stable] Patch Round-up for stable 1.3.1, freeze Monday

2013-01-18 Thread Doug Goldstein
this increases your maintenance burden but it would enable more distros to provide you more meaningful feedback on stable releases so that they are truly stable. -- Doug Goldstein

Re: [Qemu-devel] [Bug 1033727] Re: USB passthrough doesn't work anymore with qemu-kvm 1.1.1

2013-01-14 Thread Doug Goldstein
right before a new stable tarball is released. -- Doug Goldstein

Re: [Qemu-devel] [libvirt][PATCH v2 1/2] add pci-bridge controller type

2013-01-07 Thread Doug Goldstein
. This change also needs an update to the XML schemas in docs/schemas/basictypes.rng -- Doug Goldstein

Re: [Qemu-devel] 1.1.1 - 1.1.2 migrate /managedsave issue

2012-11-04 Thread Doug Goldstein
On Sun, Nov 4, 2012 at 3:51 PM, Anthony Liguori anth...@codemonkey.ws wrote: Avi Kivity a...@redhat.com writes: On 10/22/2012 09:04 AM, Philipp Hahn wrote: Hello Doug, On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote: I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my

[Qemu-devel] [PATCH] migrate: error message for ram_load size

2012-11-03 Thread Doug Goldstein
Report an error when ramblock's sizes mismatch with a suggestion to the user as to what went wrong. If a user has a managedsave state by libvirt, which is the default now, and upgrades their distro, which in turn upgrades QEMU, they will be surprised by the fact that their VMs fail to start. The

Re: [Qemu-devel] [PATCH] libseccomp: require version 1.0.0

2012-11-03 Thread Doug Goldstein
; then LIBS=`$pkg_config --libs libseccomp` seccomp=yes else -- 1.7.2.5 I can confirm this will fix the compile issue when you've got libseccomp 0.1.0, so ACK from a non-maintainer. -- Doug Goldstein

Re: [Qemu-devel] 1.1.1 - 1.1.2 migrate /managedsave issue

2012-10-29 Thread Doug Goldstein
On Mon, Oct 22, 2012 at 6:23 AM, Avi Kivity a...@redhat.com wrote: On 10/22/2012 09:04 AM, Philipp Hahn wrote: Hello Doug, On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote: I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my VMs. ... I had upgraded to qemu-kvm 1.1.2

[Qemu-devel] [PATCH] migrate: error message for ram_load size

2012-10-29 Thread Doug Goldstein
Report an error when ramblock's sizes mismatch with a suggestion to the user as to what went wrong. --- libvirt uses migration to save the state, however when performing a distro upgrade you might get an error starting your VMs up again without much detail. This patch attempts to remedy that with

Re: [Qemu-devel] 1.1.1 - 1.1.2 migrate /managedsave issue

2012-10-23 Thread Doug Goldstein
On Mon, Oct 22, 2012 at 6:23 AM, Avi Kivity a...@redhat.com wrote: On 10/22/2012 09:04 AM, Philipp Hahn wrote: Hello Doug, On Saturday 20 October 2012 00:46:43 Doug Goldstein wrote: I'm using libvirt 0.10.2 and I had qemu-kvm 1.1.1 running all my VMs. ... I had upgraded to qemu-kvm 1.1.2

[Qemu-devel] 1.1.1 - 1.1.2 migrate / managed save issue

2012-10-18 Thread Doug Goldstein
-vga.vram_size=67108864 -incoming fd:20 -device virtio-balloon-pci,id=balloon0,bus=pci.0,addr=0x5 char device redirected to /dev/pts/7 qemu: warning: error while loading state for instance 0x0 of device 'ram' load of migration failed -- Doug Goldstein

[Qemu-devel] [Bug 816370] Re: compile error in QEMU 0.15.0-rc0 and 0.15.0-rc1

2012-10-17 Thread Doug Goldstein
Fixed by: http://lists.freedesktop.org/archives/spice- devel/2011-January/002259.html ** Changed in: qemu Status: New = Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/816370

[Qemu-devel] [Bug 816370] Re: compile error in QEMU 0.15.0-rc0 and 0.15.0-rc1

2012-10-17 Thread Doug Goldstein
Which has also been released. ** Changed in: qemu Status: Fix Committed = Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/816370 Title: compile error in QEMU 0.15.0-rc0 and

Re: [Qemu-devel] [libvirt] Problem setting CPU topology

2012-07-10 Thread Doug Goldstein
days ACPI provides most of that information so its likely that QEMU doesn't support plain old MPS + APIC. Its also possible that QEMU does support this but the kernel got confused because I see the CPUID +acpi specified in the command line while -no-acpi is on the command line as you noted. -- Doug

Re: [Qemu-devel] [libvirt] Modern CPU models cannot be used with libvirt

2012-03-11 Thread Doug Goldstein
must use Penryn for example. If ovirt and by extension libvirt let me know that guest X is running on CPU-A, I know I could migrate it to any other machine supporting CPU-A or CPU-B (assuming B is a super set of A). -- Doug Goldstein

[Qemu-devel] [Bug 816370]

2011-08-22 Thread Doug Goldstein
*** This bug has been marked as a duplicate of bug 378907 *** -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/816370 Title: compile error in QEMU 0.15.0-rc0 and 0.15.0-rc1 Status in QEMU: New

[Qemu-devel] Re: [PATCH 1/2] kvm: Use kvm-kmod headers if available

2010-01-13 Thread Doug Goldstein
say ./configure --with-kvm-kmod{=optional/path/to} because otherwise this adds a potential automagical dependency onto kvm-kmod (i.e. if the user had kvm-kmod installed at the time of build but then removed them and went back with their kernel provided version) Thanks. -- Doug Goldstein