Re: [Qemu-devel] [PATCH] vhost-net: Move asserts to after check for end < start

2012-04-01 Thread Michael S. Tsirkin
On Fri, Mar 30, 2012 at 07:21:22PM -0700, Josh Durgin wrote: > On 12/16/2011 12:33 PM, Bruce Rogers wrote: > >When migrating a vm using vhost-net we hit the following assertion: > > > >qemu-kvm: /usr/src/packages/BUILD/qemu-kvm-0.15.1/hw/vhost.c:30: > >vhost_dev_sync_region: Assertion `start / (0x1

Re: [Qemu-devel] [PATCH] vhost-net: Move asserts to after check for end < start

2012-03-30 Thread Josh Durgin
On 12/16/2011 12:33 PM, Bruce Rogers wrote: When migrating a vm using vhost-net we hit the following assertion: qemu-kvm: /usr/src/packages/BUILD/qemu-kvm-0.15.1/hw/vhost.c:30: vhost_dev_sync_region: Assertion `start / (0x1000 * (8 * sizeof(vhost_log_chunk_t)))< dev->log_size' failed. I consi

[Qemu-devel] [PATCH] vhost-net: Move asserts to after check for end < start

2011-12-16 Thread Bruce Rogers
When migrating a vm using vhost-net we hit the following assertion: qemu-kvm: /usr/src/packages/BUILD/qemu-kvm-0.15.1/hw/vhost.c:30: vhost_dev_sync_region: Assertion `start / (0x1000 * (8 * sizeof(vhost_log_chunk_t))) < dev->log_size' failed. The cases which the end < start check is intended to c