Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-29 Thread Peter Crosthwaite
On Fri, May 30, 2014 at 2:00 AM, Gabriel L. Somlo wrote: > Hi Peter, > > Thanks for the QOM crash course ! > > So I did s/TYPE_E1000/TYPE_E1000_BASE/ as suggested by Stefan, and > also moved PCI_VENDOR_ID_INTEL back into e1000_class_init() and away > from e1000_devices[] as you suggested. That's g

Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-29 Thread Gabriel L. Somlo
Hi Peter, Thanks for the QOM crash course ! So I did s/TYPE_E1000/TYPE_E1000_BASE/ as suggested by Stefan, and also moved PCI_VENDOR_ID_INTEL back into e1000_class_init() and away from e1000_devices[] as you suggested. That's going to be part of v3, as soon as we're done talking about QOM, see be

Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-28 Thread Peter Crosthwaite
On Thu, May 22, 2014 at 4:27 AM, Gabriel L. Somlo wrote: > Allow selection of different card models from the qemu > command line, to better accomodate a wider range of guests. > > Signed-off-by: Romain Dolbeau > Signed-off-by: Gabriel Somlo > --- > hw/net/e1000.c | 95 > +++

Re: [Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-28 Thread Stefan Hajnoczi
On Wed, May 21, 2014 at 02:27:43PM -0400, Gabriel L. Somlo wrote: > -#define TYPE_E1000 "e1000" > +#define TYPE_E1000 "e1000-base" Please rename to TYPE_E1000_BASE. It's confusing to call it TYPE_E1000 but then actually have a different type use the "e1000" string. Besides that, I'm happy. Stef

[Qemu-devel] [PATCH v2 2/3] e1000: allow command-line selection of card model

2014-05-21 Thread Gabriel L. Somlo
Allow selection of different card models from the qemu command line, to better accomodate a wider range of guests. Signed-off-by: Romain Dolbeau Signed-off-by: Gabriel Somlo --- hw/net/e1000.c | 95 -- 1 file changed, 79 insertions(+), 16