[Qemu-devel] [Bug 1179357] Re: Assigned ID could not be used for net-client name

[Amos Kong]

** Changed in: qemu
       Status: Confirmed => Fix Released

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1179357

Title:
  Assigned ID could not be used for net-client name

Status in QEMU:
  Fix Released

Bug description:
  As discussed in maillist:  http://lists.nongnu.org/archive/html/qemu-
  devel/2013-04/msg03582.html

  [this problem still existed in latest qemu.git]

  qemu -device virtio-net-pci,netdev=ndev1,id=id1 -netdev tap,id=ndev1 \
                -device e1000,netdev=ndev2,id=id2 -netdev tap,id=ndev2

  (qemu) info network
  virtio-net-device.0: 
index=0,type=nic,model=virtio-net-device,macaddr=52:54:00:12:34:56
   \ ndev1: 
index=0,type=tap,ifname=tap0,script=/etc/qemu-ifup,downscript=/etc/qemu-ifdown
  id2: index=0,type=nic,model=e1000,macaddr=52:54:00:12:34:57
   \ ndev2: 
index=0,type=tap,ifname=tap1,script=/etc/qemu-ifup,downscript=/etc/qemu-ifdown

  net-client name of virtio-nic should be 'id1'

  (qemu) device_del virtio-net-device.0
  Device 'virtio-net-device.0' not found
  (qemu) device_del id1  (will cause the hot-removing action)

  
  It's a regression caused by commit: e37da3945fa2fde161e1b217f937fc318c4b7639

  commit e37da3945fa2fde161e1b217f937fc318c4b7639
  Author: KONRAD Frederic <address@hidden>
  Date:   Thu Apr 11 16:29:58 2013 +0200

      virtio-net-pci: switch to the new API.
      
      Here the virtio-net-pci is modified for the new API. The device
      virtio-net-pci extends virtio-pci. It creates and connects a
      virtio-net-device during the init. The properties are not changed.
      
      Signed-off-by: KONRAD Frederic <address@hidden>
      Tested-by: Cornelia Huck <address@hidden>
      Message-id: address@hidden
      Signed-off-by: Anthony Liguori <address@hidden>

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1179357/+subscriptions