All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wen Congyang <wency@cn.fujitsu.com>
To: William Dauchy <wdauchy@gmail.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] null mac address
Date: Fri, 25 Feb 2011 09:32:05 +0800	[thread overview]
Message-ID: <4D670695.7040404@cn.fujitsu.com> (raw)
In-Reply-To: <AANLkTik8Lyynj2mgNF0xmMXJ_5mzMS7MTPZk1joN-XfW@mail.gmail.com>

At 02/24/2011 10:40 PM, William Dauchy Write:
> Hi,
> 
> I got some troubles hot plugging network pci devices. An attach works
> as expected but the mac address is still set to "00:00:00:00:00:00" on
> the guest machine. I have to reboot the guest to get the correct mac
> address.
> I first tried through libvirt with:
> # virsh attach-interface dom0 network default --mac 52:54:00:f6:84:ba
> 
> and then through qemu monitor to make sure that it wasn't a libvirt issue:
> device_add rtl8139
> or
> device_add rtl8139,mac=01:02:03:04:05:06
> 
> Always the same result on the guest. A device info on qemu give the
> correct result, that is to say, with a correct mac address.
> I went through rtl8139.c and saw that the mac address is set in `rtl8139_reset`.
> This function was called in `pci_rtl8139_init` but removed since
> c169998802505c244b8bcad562633f29de7d74a4 commit, because it doesn't
> make sense to call it when the virtual machine is shutdown.
> I'm now wondering where I am supposed to call this reset function when
> live attaching a pci device. I think it could fix the mac address
> issue.
> I will be very pleased to receive some tips to create a patch for this issue.

I got the same troubles, but I don't notice commit c1699988, and I sent a
patch(call reset function in init) some days before. So I think this patch is
wrong.

I think the following method can solve this problem:
move eeprom init from reset function into init function, as it is read only,
and does not need init again in reset function.

I will test this method. If it's OK, I will send a new patch.

Thnaks
Wen Congyang

> 
> Regards,

  reply	other threads:[~2011-02-25  1:33 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-24 14:40 [Qemu-devel] null mac address William Dauchy
2011-02-25  1:32 ` Wen Congyang [this message]
2011-02-25  2:55 ` Wen Congyang
2011-02-25 18:15   ` Blue Swirl
2011-02-28  2:09     ` [Qemu-devel] [PATCH v2] move eeprom init from reset function to init function Wen Congyang
2011-03-02 13:36     ` [Qemu-devel] [PATCH] moving eeprom initialization William Dauchy
2011-03-02 15:25       ` [Qemu-devel] " William Dauchy
2011-03-02 18:28         ` Gerhard Wiesinger
2011-03-02 20:54           ` William Dauchy
2011-03-05 12:15       ` Blue Swirl
2011-03-06 19:23         ` [Qemu-devel] " William Dauchy
2011-03-06 20:38           ` Gerhard Wiesinger
2011-03-06 21:12             ` William Dauchy
2011-03-06 21:27         ` William Dauchy
2011-03-07  2:38           ` [Qemu-devel] " Wen Congyang
2011-03-13 13:33           ` Blue Swirl

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D670695.7040404@cn.fujitsu.com \
    --to=wency@cn.fujitsu.com \
    --cc=qemu-devel@nongnu.org \
    --cc=wdauchy@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.