Re: [Qemu-devel] [PATCH 2/4] e1000e: No need to validate configuration on migration

2016-10-27 Thread Eduardo Habkost
On Thu, Oct 27, 2016 at 01:22:29PM +0100, Dr. David Alan Gilbert wrote: > * Eduardo Habkost (ehabk...@redhat.com) wrote: > > On Thu, Oct 27, 2016 at 09:42:53AM +0300, Dmitry Fleytman wrote: > > > > > > > On 26 Oct 2016, at 22:21 PM, Eduardo Habkost > > > > wrote: > > > > > > > > The user (or ma

Re: [Qemu-devel] [PATCH 2/4] e1000e: No need to validate configuration on migration

2016-10-27 Thread Dr. David Alan Gilbert
* Eduardo Habkost (ehabk...@redhat.com) wrote: > On Thu, Oct 27, 2016 at 09:42:53AM +0300, Dmitry Fleytman wrote: > > > > > On 26 Oct 2016, at 22:21 PM, Eduardo Habkost wrote: > > > > > > The user (or management software) is responsible for keeping the > > > same configuration on both sides whil

Re: [Qemu-devel] [PATCH 2/4] e1000e: No need to validate configuration on migration

2016-10-27 Thread Eduardo Habkost
On Thu, Oct 27, 2016 at 09:42:53AM +0300, Dmitry Fleytman wrote: > > > On 26 Oct 2016, at 22:21 PM, Eduardo Habkost wrote: > > > > The user (or management software) is responsible for keeping the > > same configuration on both sides while migrating. Remove the > > configuration validation code a

Re: [Qemu-devel] [PATCH 2/4] e1000e: No need to validate configuration on migration

2016-10-26 Thread Dmitry Fleytman
> On 26 Oct 2016, at 22:21 PM, Eduardo Habkost wrote: > > The user (or management software) is responsible for keeping the > same configuration on both sides while migrating. Remove the > configuration validation code at e1000e_post_load, and the > unnecessary subsys_used/subsys_ven_used fields.

[Qemu-devel] [PATCH 2/4] e1000e: No need to validate configuration on migration

2016-10-26 Thread Eduardo Habkost
The user (or management software) is responsible for keeping the same configuration on both sides while migrating. Remove the configuration validation code at e1000e_post_load, and the unnecessary subsys_used/subsys_ven_used fields. Signed-off-by: Eduardo Habkost --- hw/net/e1000e.c | 14 ---