* [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. @ 2012-02-14 3:10 bo.novell 2012-05-09 20:20 ` Andreas Färber 0 siblings, 1 reply; 8+ messages in thread From: bo.novell @ 2012-02-14 3:10 UTC (permalink / raw) To: qemu-devel; +Cc: Bo Yang From: Bo Yang <boyang@suse.com> Windows 2008 sp2 tries to read mac address from phys and then write the read value into it. This patch is a workaround for the issue. Signed-off-by: Bo Yang <boyang@suse.com> --- hw/rtl8139.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/hw/rtl8139.c b/hw/rtl8139.c index 1668390..074a14a 100644 --- a/hw/rtl8139.c +++ b/hw/rtl8139.c @@ -3476,6 +3476,8 @@ static int pci_rtl8139_init(PCIDevice *dev) s->eeprom.contents[7] = s->conf.macaddr.a[0] | s->conf.macaddr.a[1] << 8; s->eeprom.contents[8] = s->conf.macaddr.a[2] | s->conf.macaddr.a[3] << 8; s->eeprom.contents[9] = s->conf.macaddr.a[4] | s->conf.macaddr.a[5] << 8; + /* workaround broken windows 2008 sp2 driver. */ + memcpy(s->phys, s->conf.macaddr.a, 6); s->nic = qemu_new_nic(&net_rtl8139_info, &s->conf, object_get_typename(OBJECT(dev)), dev->qdev.id, s); -- 1.6.0.2 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-02-14 3:10 [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2 bo.novell @ 2012-05-09 20:20 ` Andreas Färber 2012-05-09 21:00 ` Alexander Graf 0 siblings, 1 reply; 8+ messages in thread From: Andreas Färber @ 2012-05-09 20:20 UTC (permalink / raw) To: qemu-devel Cc: Bo Yang, Mark McLoughlin, Alexander Graf, Anthony Liguori, Bruce Rogers Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: > From: Bo Yang <boyang@suse.com> > > Windows 2008 sp2 tries to read mac address from phys > and then write the read value into it. This patch > is a workaround for the issue. > > Signed-off-by: Bo Yang <boyang@suse.com> > --- > hw/rtl8139.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) Ping! Who feels responsible for reviewing this? It would be a candidate for 1.1 and 0.15.x (BNC#722643). Only issue I see is that the subject could be improved (saying what it does rather than what was broken before, e.g. "rtl8139: Init phys with MAC address"). Regards, Andreas > diff --git a/hw/rtl8139.c b/hw/rtl8139.c > index 1668390..074a14a 100644 > --- a/hw/rtl8139.c > +++ b/hw/rtl8139.c > @@ -3476,6 +3476,8 @@ static int pci_rtl8139_init(PCIDevice *dev) > s->eeprom.contents[7] = s->conf.macaddr.a[0] | s->conf.macaddr.a[1] << 8; > s->eeprom.contents[8] = s->conf.macaddr.a[2] | s->conf.macaddr.a[3] << 8; > s->eeprom.contents[9] = s->conf.macaddr.a[4] | s->conf.macaddr.a[5] << 8; > + /* workaround broken windows 2008 sp2 driver. */ > + memcpy(s->phys, s->conf.macaddr.a, 6); > > s->nic = qemu_new_nic(&net_rtl8139_info, &s->conf, > object_get_typename(OBJECT(dev)), dev->qdev.id, s); -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 20:20 ` Andreas Färber @ 2012-05-09 21:00 ` Alexander Graf 2012-05-09 21:05 ` Andreas Färber 0 siblings, 1 reply; 8+ messages in thread From: Alexander Graf @ 2012-05-09 21:00 UTC (permalink / raw) To: Andreas Färber Cc: Bo Yang, Mark McLoughlin, qemu-devel, Anthony Liguori, Bruce Rogers On 09.05.2012, at 22:20, Andreas Färber wrote: > Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >> From: Bo Yang <boyang@suse.com> >> >> Windows 2008 sp2 tries to read mac address from phys >> and then write the read value into it. This patch >> is a workaround for the issue. >> >> Signed-off-by: Bo Yang <boyang@suse.com> >> --- >> hw/rtl8139.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) > > Ping! Who feels responsible for reviewing this? > It would be a candidate for 1.1 and 0.15.x (BNC#722643). > > Only issue I see is that the subject could be improved (saying what it > does rather than what was broken before, e.g. "rtl8139: Init phys with > MAC address"). I thought the real bug was that the reset function didn't get called? Alex ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 21:00 ` Alexander Graf @ 2012-05-09 21:05 ` Andreas Färber 2012-05-09 21:06 ` Alexander Graf 0 siblings, 1 reply; 8+ messages in thread From: Andreas Färber @ 2012-05-09 21:05 UTC (permalink / raw) To: Alexander Graf, Bo Yang Cc: Mark McLoughlin, qemu-devel, Anthony Liguori, Bruce Rogers Am 09.05.2012 23:00, schrieb Alexander Graf: > > On 09.05.2012, at 22:20, Andreas Färber wrote: > >> Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >>> From: Bo Yang <boyang@suse.com> >>> >>> Windows 2008 sp2 tries to read mac address from phys >>> and then write the read value into it. This patch >>> is a workaround for the issue. >>> >>> Signed-off-by: Bo Yang <boyang@suse.com> >>> --- >>> hw/rtl8139.c | 2 ++ >>> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> Ping! Who feels responsible for reviewing this? >> It would be a candidate for 1.1 and 0.15.x (BNC#722643). >> >> Only issue I see is that the subject could be improved (saying what it >> does rather than what was broken before, e.g. "rtl8139: Init phys with >> MAC address"). > > I thought the real bug was that the reset function didn't get called? That was for the e1000 that I was working on (BNC#722958). I believe Bo tested the combination of both patches? Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 21:05 ` Andreas Färber @ 2012-05-09 21:06 ` Alexander Graf 2012-05-09 21:27 ` Bruce Rogers 2012-05-10 16:31 ` Bo Yang 0 siblings, 2 replies; 8+ messages in thread From: Alexander Graf @ 2012-05-09 21:06 UTC (permalink / raw) To: Andreas Färber Cc: Bo Yang, Mark McLoughlin, qemu-devel, Anthony Liguori, Bruce Rogers On 09.05.2012, at 23:05, Andreas Färber wrote: > Am 09.05.2012 23:00, schrieb Alexander Graf: >> >> On 09.05.2012, at 22:20, Andreas Färber wrote: >> >>> Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >>>> From: Bo Yang <boyang@suse.com> >>>> >>>> Windows 2008 sp2 tries to read mac address from phys >>>> and then write the read value into it. This patch >>>> is a workaround for the issue. >>>> >>>> Signed-off-by: Bo Yang <boyang@suse.com> >>>> --- >>>> hw/rtl8139.c | 2 ++ >>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>> >>> Ping! Who feels responsible for reviewing this? >>> It would be a candidate for 1.1 and 0.15.x (BNC#722643). >>> >>> Only issue I see is that the subject could be improved (saying what it >>> does rather than what was broken before, e.g. "rtl8139: Init phys with >>> MAC address"). >> >> I thought the real bug was that the reset function didn't get called? > > That was for the e1000 that I was working on (BNC#722958). I believe Bo > tested the combination of both patches? IIRC the bug was the same, just that you found the root cause and Bo came up with this patch :). Alex ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 21:06 ` Alexander Graf @ 2012-05-09 21:27 ` Bruce Rogers 2012-05-09 21:38 ` Andreas Färber 2012-05-10 16:31 ` Bo Yang 1 sibling, 1 reply; 8+ messages in thread From: Bruce Rogers @ 2012-05-09 21:27 UTC (permalink / raw) To: afaerber, Alexander Graf Cc: Bo Yang, Mark McLoughlin, qemu-devel, Anthony Liguori >>> On 5/9/2012 at 03:06 PM, Alexander Graf <agraf@suse.de> wrote: > On 09.05.2012, at 23:05, Andreas Färber wrote: > >> Am 09.05.2012 23:00, schrieb Alexander Graf: >>> >>> On 09.05.2012, at 22:20, Andreas Färber wrote: >>> >>>> Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >>>>> From: Bo Yang <boyang@suse.com> >>>>> >>>>> Windows 2008 sp2 tries to read mac address from phys >>>>> and then write the read value into it. This patch >>>>> is a workaround for the issue. >>>>> >>>>> Signed-off-by: Bo Yang <boyang@suse.com> >>>>> --- >>>>> hw/rtl8139.c | 2 ++ >>>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>>> >>>> Ping! Who feels responsible for reviewing this? >>>> It would be a candidate for 1.1 and 0.15.x (BNC#722643). >>>> >>>> Only issue I see is that the subject could be improved (saying what it >>>> does rather than what was broken before, e.g. "rtl8139: Init phys with >>>> MAC address"). >>> >>> I thought the real bug was that the reset function didn't get called? >> >> That was for the e1000 that I was working on (BNC#722958). I believe Bo >> tested the combination of both patches? > > IIRC the bug was the same, just that you found the root cause and Bo came up > with this patch :). > > > Alex That is my recollection as well. Bruce ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 21:27 ` Bruce Rogers @ 2012-05-09 21:38 ` Andreas Färber 0 siblings, 0 replies; 8+ messages in thread From: Andreas Färber @ 2012-05-09 21:38 UTC (permalink / raw) To: Bruce Rogers, Bo Yang Cc: Mark McLoughlin, Alexander Graf, Anthony Liguori, qemu-devel Am 09.05.2012 23:27, schrieb Bruce Rogers: > >>> On 5/9/2012 at 03:06 PM, Alexander Graf <agraf@suse.de> wrote: > >> On 09.05.2012, at 23:05, Andreas Färber wrote: >> >>> Am 09.05.2012 23:00, schrieb Alexander Graf: >>>> >>>> On 09.05.2012, at 22:20, Andreas Färber wrote: >>>> >>>>> Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >>>>>> From: Bo Yang <boyang@suse.com> >>>>>> >>>>>> Windows 2008 sp2 tries to read mac address from phys >>>>>> and then write the read value into it. This patch >>>>>> is a workaround for the issue. >>>>>> >>>>>> Signed-off-by: Bo Yang <boyang@suse.com> >>>>>> --- >>>>>> hw/rtl8139.c | 2 ++ >>>>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>>>> >>>>> Ping! Who feels responsible for reviewing this? >>>>> It would be a candidate for 1.1 and 0.15.x (BNC#722643). >>>> >>>> I thought the real bug was that the reset function didn't get called? >>> >>> That was for the e1000 that I was working on (BNC#722958). I believe Bo >>> tested the combination of both patches? >> >> IIRC the bug was the same, just that you found the root cause and Bo came up >> with this patch :). > > That is my recollection as well. Even better then, less work. Bo, in such a case, the patch should've been recalled (by a reply). Cheers, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2. 2012-05-09 21:06 ` Alexander Graf 2012-05-09 21:27 ` Bruce Rogers @ 2012-05-10 16:31 ` Bo Yang 1 sibling, 0 replies; 8+ messages in thread From: Bo Yang @ 2012-05-10 16:31 UTC (permalink / raw) To: Alexander Graf Cc: Mark McLoughlin, Bruce Rogers, afaerber, Anthony Liguori, qemu-devel On 05/10/2012 05:06 AM, Alexander Graf wrote: > > On 09.05.2012, at 23:05, Andreas Färber wrote: > >> Am 09.05.2012 23:00, schrieb Alexander Graf: >>> >>> On 09.05.2012, at 22:20, Andreas Färber wrote: >>> >>>> Am 14.02.2012 04:10, schrieb bo.novell@gmail.com: >>>>> From: Bo Yang <boyang@suse.com> >>>>> >>>>> Windows 2008 sp2 tries to read mac address from phys >>>>> and then write the read value into it. This patch >>>>> is a workaround for the issue. >>>>> >>>>> Signed-off-by: Bo Yang <boyang@suse.com> >>>>> --- >>>>> hw/rtl8139.c | 2 ++ >>>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>>> >>>> Ping! Who feels responsible for reviewing this? >>>> It would be a candidate for 1.1 and 0.15.x (BNC#722643). >>>> >>>> Only issue I see is that the subject could be improved (saying what it >>>> does rather than what was broken before, e.g. "rtl8139: Init phys with >>>> MAC address"). >>> >>> I thought the real bug was that the reset function didn't get called? >> >> That was for the e1000 that I was working on (BNC#722958). I believe Bo >> tested the combination of both patches? > > IIRC the bug was the same, just that you found the root cause and Bo came up with this patch :). yes. > > > Alex > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2012-05-10 16:49 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-02-14 3:10 [Qemu-devel] [PATCH] RTL8139 hotplug is broken on windows 2008 sp2 bo.novell 2012-05-09 20:20 ` Andreas Färber 2012-05-09 21:00 ` Alexander Graf 2012-05-09 21:05 ` Andreas Färber 2012-05-09 21:06 ` Alexander Graf 2012-05-09 21:27 ` Bruce Rogers 2012-05-09 21:38 ` Andreas Färber 2012-05-10 16:31 ` Bo Yang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).