Re: [Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Fri, Jun 27, 2014 at 03:42:10PM +0100, Dr. David Alan Gilbert wrote: > > * Michael S. Tsirkin (m...@redhat.com) wrote: > > > On Fri, Jun 27, 2014 at 09:34:38AM +0100, Dr. David Alan Gilbert (git) > > > wrote: > > > > From: "Dr. David Alan Gilbert"

Re: [Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Michael S. Tsirkin
On Fri, Jun 27, 2014 at 03:42:10PM +0100, Dr. David Alan Gilbert wrote: > * Michael S. Tsirkin (m...@redhat.com) wrote: > > On Fri, Jun 27, 2014 at 09:34:38AM +0100, Dr. David Alan Gilbert (git) > > wrote: > > > From: "Dr. David Alan Gilbert" > > > > > > Commit 'virtio: validate config_len on lo

Re: [Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Fri, Jun 27, 2014 at 09:34:38AM +0100, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Commit 'virtio: validate config_len on load' restricted config_len > > loaded from the wire to match the config_len that the dev

Re: [Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Michael S. Tsirkin
On Fri, Jun 27, 2014 at 09:34:38AM +0100, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Commit 'virtio: validate config_len on load' restricted config_len > loaded from the wire to match the config_len that the device had. > > Unfortunately, there are cases where this

Re: [Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Paolo Bonzini
Il 27/06/2014 10:34, Dr. David Alan Gilbert (git) ha scritto: From: "Dr. David Alan Gilbert" Commit 'virtio: validate config_len on load' restricted config_len loaded from the wire to match the config_len that the device had. Unfortunately, there are cases where this isn't true, the one we fou

[Qemu-devel] [PATCH] Allow mismatched virtio config-len

2014-06-27 Thread Dr. David Alan Gilbert (git)
From: "Dr. David Alan Gilbert" Commit 'virtio: validate config_len on load' restricted config_len loaded from the wire to match the config_len that the device had. Unfortunately, there are cases where this isn't true, the one we found it on was the wqe addition in virtio-blk. Allow mismatched c