[Qemu-devel] [PULL 0/4] Net patches

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4:

  Merge branch 'master' of git.qemu.org:/pub/git/qemu (2012-10-07 18:42:18 +0000)

are available in the git repository at:


  git://github.com/stefanha/qemu.git net

for you to fetch changes up to a245fc18352fe286ba45ae0661a73b3841514889:

  net: consolidate NetClientState header files into one (2012-10-08 13:59:40 +0200)

----------------------------------------------------------------
Amos Kong (2):
      e1000: update nc.link_down in e1000_post_load()
      virtio-net: update nc.link_down in virtio_net_load()

Jason Wang (1):
      rtl8139: implement 8139cp link status

Paolo Bonzini (1):
      net: consolidate NetClientState header files into one

 hw/e1000.c                  | 12 ++++++++++++
 hw/rtl8139.c                | 24 ++++++++++++++++++++++--
 hw/virtio-net.c             |  5 +++++
 net.c                       | 11 ++++-------
 net/{socket.h => clients.h} | 28 +++++++++++++++++++++++++---
 net/dump.c                  |  2 +-
 net/dump.h                  | 33 ---------------------------------
 net/hub.c                   |  1 +
 net/hub.h                   |  2 --
 net/slirp.c                 |  3 ++-
 net/slirp.h                 |  3 ---
 net/socket.c                |  3 +--
 net/tap-win32.c             |  2 +-
 net/tap.c                   |  3 ++-
 net/tap.h                   |  6 ------
 net/vde.c                   |  3 +--
 net/vde.h                   | 37 -------------------------------------
 17 files changed, 77 insertions(+), 101 deletions(-)
 rename net/{socket.h => clients.h} (62%)
 delete mode 100644 net/dump.h
 delete mode 100644 net/vde.h

-- 
1.7.11.4

Re: [Qemu-devel] [PULL 0/4] Net patches

[Anthony Liguori]

Stefan Hajnoczi <stefanha@redhat.com> writes:

> The following changes since commit 4bb26682f70a5f626cad3e0ac82bf4b6252ea7a4:
>
>   Merge branch 'master' of git.qemu.org:/pub/git/qemu (2012-10-07 18:42:18 +0000)
>
> are available in the git repository at:
>
>
>   git://github.com/stefanha/qemu.git net
>
> for you to fetch changes up to a245fc18352fe286ba45ae0661a73b3841514889:
>
>   net: consolidate NetClientState header files into one (2012-10-08 13:59:40 +0200)
>

Pulled. Thanks.

Regards,

Anthony Liguori

> ----------------------------------------------------------------
> Amos Kong (2):
>       e1000: update nc.link_down in e1000_post_load()
>       virtio-net: update nc.link_down in virtio_net_load()
>
> Jason Wang (1):
>       rtl8139: implement 8139cp link status
>
> Paolo Bonzini (1):
>       net: consolidate NetClientState header files into one
>
>  hw/e1000.c                  | 12 ++++++++++++
>  hw/rtl8139.c                | 24 ++++++++++++++++++++++--
>  hw/virtio-net.c             |  5 +++++
>  net.c                       | 11 ++++-------
>  net/{socket.h => clients.h} | 28 +++++++++++++++++++++++++---
>  net/dump.c                  |  2 +-
>  net/dump.h                  | 33 ---------------------------------
>  net/hub.c                   |  1 +
>  net/hub.h                   |  2 --
>  net/slirp.c                 |  3 ++-
>  net/slirp.h                 |  3 ---
>  net/socket.c                |  3 +--
>  net/tap-win32.c             |  2 +-
>  net/tap.c                   |  3 ++-
>  net/tap.h                   |  6 ------
>  net/vde.c                   |  3 +--
>  net/vde.h                   | 37 -------------------------------------
>  17 files changed, 77 insertions(+), 101 deletions(-)
>  rename net/{socket.h => clients.h} (62%)
>  delete mode 100644 net/dump.h
>  delete mode 100644 net/vde.h
>
> -- 
> 1.7.11.4

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 6801038bc52d61f81ac8a25fbe392f1bad982887:

  target-mips: fix wrong microMIPS opcode encoding (2012-11-15 14:48:16 +0100)

are available in the git repository at:

  git://github.com/stefanha/qemu.git net

for you to fetch changes up to 315d78d7f9d70c44eecfdad7f0cb5d51c612f884:

  tap: reset vnet header size on open (2012-11-16 14:28:33 +0100)

----------------------------------------------------------------
Michael S. Tsirkin (1):
      tap: reset vnet header size on open

Stefan Hajnoczi (3):
      net: add public qemu_net_poll() function
      net: extract notify_link_status_changed() function
      net: convert NetClientState to QOM

 hw/cadence_gem.c        |  27 +++++++---
 hw/dp8393x.c            |  41 ++++++++++----
 hw/e1000.c              |  32 +++++++----
 hw/eepro100.c           |  27 +++++++---
 hw/etraxfs_eth.c        |  27 +++++++---
 hw/lan9118.c            |  27 +++++++---
 hw/lance.c              |  18 +------
 hw/mcf_fec.c            |  32 ++++++++---
 hw/milkymist-minimac2.c |  26 ++++++---
 hw/mipsnet.c            |  25 ++++++---
 hw/musicpal.c           |  26 ++++++---
 hw/ne2000-isa.c         |  17 +-----
 hw/ne2000.c             |  23 +++++---
 hw/ne2000.h             |   2 +
 hw/opencores_eth.c      |  28 +++++++---
 hw/pcnet-pci.c          |  18 +------
 hw/pcnet.c              |  36 +++++++++++--
 hw/pcnet.h              |   3 +-
 hw/rtl8139.c            |  28 +++++++---
 hw/smc91c111.c          |  26 ++++++---
 hw/spapr_llan.c         |  24 ++++++---
 hw/stellaris_enet.c     |  26 ++++++---
 hw/usb/dev-network.c    |  25 ++++++---
 hw/vhost_net.c          |  11 ++--
 hw/virtio-net.c         |  42 ++++++++++-----
 hw/xen_nic.c            |  29 +++++++---
 hw/xgmac.c              |  26 ++++++---
 hw/xilinx_axienet.c     |  26 ++++++---
 hw/xilinx_ethlite.c     |  26 ++++++---
 net.c                   | 140 +++++++++++++++++++++++++++++++++---------------
 net.h                   |  35 ++++++++----
 net/dump.c              |  30 ++++++++---
 net/hub.c               |  64 +++++++++++++---------
 net/slirp.c             |  29 +++++++---
 net/socket.c            |  74 ++++++++++++++++---------
 net/tap-win32.c         |  27 ++++++++--
 net/tap.c               |  84 ++++++++++++++++-------------
 net/tap.h               |   4 ++
 net/vde.c               |  28 ++++++++--
 39 files changed, 850 insertions(+), 389 deletions(-)

-- 
1.8.0

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 8e4a424b305e29dc0e454f52df3b35577f342975:

  Revert "virtio-pci: replace byte swap hack" (2013-01-06 18:30:17 +0000)

are available in the git repository at:

  git://github.com/stefanha/qemu.git net

for you to fetch changes up to 83f58e570f21c3e7227e7fbef1fc0e18b5ed7ea9:

  rtl8139: preserve link state across device reset (2013-01-07 10:43:21 +0100)

----------------------------------------------------------------
Amos Kong (3):
      net: clean up network at qemu process termination
      e1000: no need auto-negotiation if link was down
      rtl8139: preserve link state across device reset

Michael Contreras (1):
      e1000: Discard oversized packets based on SBP|LPE

 hw/e1000.c   | 12 ++++++++++--
 hw/rtl8139.c |  3 ++-
 vl.c         |  4 +++-
 3 files changed, 15 insertions(+), 4 deletions(-)

-- 
1.8.0.2

Re: [Qemu-devel] [PULL 0/4] Net patches

[Anthony Liguori]

Pulled, thanks.

Regards,

Anthony Liguori

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07:

  Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500)

are available in the git repository at:


  git://github.com/stefanha/qemu.git net

for you to fetch changes up to e21f6f23606e90f5cc6df2d9b1d8b862b54d2796:

  vmxnet3: Eliminate __packed redefined warning (2013-09-02 13:35:50 +0200)

----------------------------------------------------------------
Brad Smith (2):
      tap: Use numbered tap/tun devices on all *BSD OS's
      vmxnet3: Eliminate __packed redefined warning

Jan Kiszka (1):
      net: Rename send_queue to incoming_queue

Vincenzo Maffione (1):
      e1000: add interrupt mitigation support

 hw/i386/pc_piix.c    |  18 ++++++-
 hw/i386/pc_q35.c     |  16 ++++++-
 hw/net/e1000.c       | 131 +++++++++++++++++++++++++++++++++++++++++++++++++--
 hw/net/vmxnet3.h     |   2 -
 include/hw/i386/pc.h |   8 ++++
 include/net/net.h    |   2 +-
 net/hub.c            |   2 +-
 net/net.c            |  14 +++---
 net/tap-bsd.c        |  11 -----
 9 files changed, 176 insertions(+), 28 deletions(-)

-- 
1.8.3.1

Re: [Qemu-devel] [PULL 0/4] Net patches

[Anthony Liguori]

Stefan Hajnoczi <stefanha@redhat.com> writes:

> The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07:
>
>   Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500)
>
> are available in the git repository at:

Sorry, I forgot to send you this feedback earlier.

This conflicts badly with a pull request from Michael Tsirkin that I
merged already.  Can you rebase against master and resubmit?

Thanks!

Regards,

Anthony Liguori

>
>
>   git://github.com/stefanha/qemu.git net
>
> for you to fetch changes up to e21f6f23606e90f5cc6df2d9b1d8b862b54d2796:
>
>   vmxnet3: Eliminate __packed redefined warning (2013-09-02 13:35:50 +0200)
>
> ----------------------------------------------------------------
> Brad Smith (2):
>       tap: Use numbered tap/tun devices on all *BSD OS's
>       vmxnet3: Eliminate __packed redefined warning
>
> Jan Kiszka (1):
>       net: Rename send_queue to incoming_queue
>
> Vincenzo Maffione (1):
>       e1000: add interrupt mitigation support
>
>  hw/i386/pc_piix.c    |  18 ++++++-
>  hw/i386/pc_q35.c     |  16 ++++++-
>  hw/net/e1000.c       | 131 +++++++++++++++++++++++++++++++++++++++++++++++++--
>  hw/net/vmxnet3.h     |   2 -
>  include/hw/i386/pc.h |   8 ++++
>  include/net/net.h    |   2 +-
>  net/hub.c            |   2 +-
>  net/net.c            |  14 +++---
>  net/tap-bsd.c        |  11 -----
>  9 files changed, 176 insertions(+), 28 deletions(-)
>
> -- 
> 1.8.3.1

Re: [Qemu-devel] [PULL 0/4] Net patches

[Paolo Bonzini]

Il 02/09/2013 13:38, Stefan Hajnoczi ha scritto:
> The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07:
> 
>   Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500)
> 
> are available in the git repository at:
> 
> 
>   git://github.com/stefanha/qemu.git net
> 
> for you to fetch changes up to e21f6f23606e90f5cc6df2d9b1d8b862b54d2796:
> 
>   vmxnet3: Eliminate __packed redefined warning (2013-09-02 13:35:50 +0200)

Is Aurelien going to apply the ne2k patch himself?

Paolo

Re: [Qemu-devel] [PULL 0/4] Net patches

[Aurelien Jarno]

On Thu, Sep 05, 2013 at 09:49:46PM +0200, Paolo Bonzini wrote:
> Il 02/09/2013 13:38, Stefan Hajnoczi ha scritto:
> > The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07:
> > 
> >   Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500)
> > 
> > are available in the git repository at:
> > 
> > 
> >   git://github.com/stefanha/qemu.git net
> > 
> > for you to fetch changes up to e21f6f23606e90f5cc6df2d9b1d8b862b54d2796:
> > 
> >   vmxnet3: Eliminate __packed redefined warning (2013-09-02 13:35:50 +0200)
> 
> Is Aurelien going to apply the ne2k patch himself?

It would be nice if you can get the pcnet fix in your tree. For the ne2k
patch, as reported it might break the non-PCI version. I have to look at
that more in details, probably this week-end.

-- 
Aurelien Jarno	                        GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

Re: [Qemu-devel] [PULL 0/4] Net patches

[Paolo Bonzini]

Il 05/09/2013 23:02, Aurelien Jarno ha scritto:
> It would be nice if you can get the pcnet fix in your tree. For the ne2k
> patch, as reported it might break the non-PCI version. I have to look at
> that more in details, probably this week-end.

No, that's fine.  The non-PCI version is also little endian.

Paolo

Re: [Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

On Fri, Sep 06, 2013 at 08:03:27AM +0200, Paolo Bonzini wrote:
> Il 05/09/2013 23:02, Aurelien Jarno ha scritto:
> > It would be nice if you can get the pcnet fix in your tree. For the ne2k
> > patch, as reported it might break the non-PCI version. I have to look at
> > that more in details, probably this week-end.
> 
> No, that's fine.  The non-PCI version is also little endian.

Good.  I just wanted confirmation because I'm not up-to-speed on device
endianness.

I'll merge the patch.

Stefan

Re: [Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

On Thu, Sep 05, 2013 at 01:38:23PM -0500, Anthony Liguori wrote:
> Stefan Hajnoczi <stefanha@redhat.com> writes:
> 
> > The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07:
> >
> >   Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500)
> >
> > are available in the git repository at:
> 
> Sorry, I forgot to send you this feedback earlier.
> 
> This conflicts badly with a pull request from Michael Tsirkin that I
> merged already.  Can you rebase against master and resubmit?

Sure, will do.

Stefan

Re: [Qemu-devel] [PULL 0/4] Net patches

[Aurelien Jarno]

On Fri, Sep 06, 2013 at 04:59:06PM +0200, Stefan Hajnoczi wrote:
> On Fri, Sep 06, 2013 at 08:03:27AM +0200, Paolo Bonzini wrote:
> > Il 05/09/2013 23:02, Aurelien Jarno ha scritto:
> > > It would be nice if you can get the pcnet fix in your tree. For the ne2k
> > > patch, as reported it might break the non-PCI version. I have to look at
> > > that more in details, probably this week-end.
> > 
> > No, that's fine.  The non-PCI version is also little endian.
> 
> Good.  I just wanted confirmation because I'm not up-to-speed on device
> endianness.
> 
> I'll merge the patch.
> 

Thanks. Would it be possible to also merge the following patch:

http://patchwork.ozlabs.org/patch/270487/

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

Re: [Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

On Mon, Sep 09, 2013 at 04:49:06PM +0200, Aurelien Jarno wrote:
> On Fri, Sep 06, 2013 at 04:59:06PM +0200, Stefan Hajnoczi wrote:
> > On Fri, Sep 06, 2013 at 08:03:27AM +0200, Paolo Bonzini wrote:
> > > Il 05/09/2013 23:02, Aurelien Jarno ha scritto:
> > > > It would be nice if you can get the pcnet fix in your tree. For the ne2k
> > > > patch, as reported it might break the non-PCI version. I have to look at
> > > > that more in details, probably this week-end.
> > > 
> > > No, that's fine.  The non-PCI version is also little endian.
> > 
> > Good.  I just wanted confirmation because I'm not up-to-speed on device
> > endianness.
> > 
> > I'll merge the patch.
> > 
> 
> Thanks. Would it be possible to also merge the following patch:
> 
> http://patchwork.ozlabs.org/patch/270487/

Merged onto my net-next tree.  I'll send a pull request at the end of
the week.

Stefan

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 1680d485777ecf436d724631ea8722cc0c66990e:

  Merge remote-tracking branch 'rth/tcg-ldst-6' into staging (2013-10-14 09:59:59 -0700)

are available in the git repository at:


  git://github.com/stefanha/qemu.git net

for you to fetch changes up to 6e6247e54da991cbc7e0c05040bd6a14d505fc89:

  net: disallow to specify multicast MAC address (2013-10-18 13:28:09 +0200)

----------------------------------------------------------------
Amos Kong (3):
      net: update nic info during device reset
      net/e1000: update network information when macaddr is changed in guest
      net/rtl8139: update network information when macaddr is changed in guest

Dmitry Krivenok (1):
      net: disallow to specify multicast MAC address

 hw/net/e1000.c   | 9 +++++++++
 hw/net/rtl8139.c | 7 ++++++-
 net/net.c        | 5 +++++
 net/util.c       | 5 +++++
 net/util.h       | 2 ++
 5 files changed, 27 insertions(+), 1 deletion(-)

-- 
1.8.3.1

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit a1d22a367d5780c9553b2cd5a24f665534ce6ed6:

  target-cris: Use new qemu_ld/st opcodes (2013-12-08 09:36:02 +0100)

are available in the git repository at:

  git://github.com/stefanha/qemu.git net-next

for you to fetch changes up to 02d38fcb2caa4454cf4ed728d5908c3cc9ba47be:

  net: Update netdev peer on link change (2013-12-09 13:33:20 +0100)

----------------------------------------------------------------
Amos Kong (1):
      virtio-net: don't update mac_table in error state

Vincenzo Maffione (2):
      net: Adding netmap network backend
      MAINTAINERS: Add netmap maintainers

Vlad Yasevich (1):
      net: Update netdev peer on link change

 MAINTAINERS         |   8 +
 configure           |  32 ++++
 hmp-commands.hx     |   4 +-
 hw/net/virtio-net.c |  35 +++--
 net/Makefile.objs   |   1 +
 net/clients.h       |   5 +
 net/net.c           |  32 ++--
 net/netmap.c        | 435 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 qapi-schema.json    |  24 ++-
 qemu-options.hx     |   9 ++
 10 files changed, 558 insertions(+), 27 deletions(-)
 create mode 100644 net/netmap.c

-- 
1.8.4.2

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f:

  Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-01-24 15:52:44 -0800)

are available in the git repository at:


  git://github.com/stefanha/qemu.git tags/net-pull-request

for you to fetch changes up to 1f149e721feb617d14a3ee4f5911c47b29866a54:

  tap-linux: Get features once and use it many times (2014-01-27 15:44:06 +0100)

----------------------------------------------------------------
Net patches

----------------------------------------------------------------
Hani Benhabiles (1):
      net: Use g_strdup_printf instead of snprintf.

Kusanagi Kouichi (1):
      tap-linux: Get features once and use it many times

Roy Franz (2):
      Fix lan9118 TX "CMD A" handling
      Fix lan9118 buffer length handling

 hw/net/lan9118.c |  6 +++---
 net/net.c        |  5 +----
 net/tap-linux.c  | 14 ++++++++------
 3 files changed, 12 insertions(+), 13 deletions(-)

-- 
1.8.4.2

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 27 January 2014 14:48, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 0169c511554cb0014a00290b0d3d26c31a49818f:
>
>   Merge remote-tracking branch 'qemu-kvm/uq/master' into staging (2014-01-24 15:52:44 -0800)
>
> are available in the git repository at:
>
>
>   git://github.com/stefanha/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 1f149e721feb617d14a3ee4f5911c47b29866a54:
>
>   tap-linux: Get features once and use it many times (2014-01-27 15:44:06 +0100)

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 64ea8038ffbf703dcd438a108d2d5499c8ff95d9:

  Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20150109.0' into staging (2015-01-10 22:29:09 +0000)

are available in the git repository at:

  git://github.com/stefanha/qemu.git tags/net-pull-request

for you to fetch changes up to a39d97c7becca5fd679d70d17ae5b1d62b9b5da6:

  hw/net/xen_nic.c: Set 'netdev->mac' to NULL after free it (2015-01-12 10:16:23 +0000)

----------------------------------------------------------------

----------------------------------------------------------------

Chen Gang (3):
  hw/net/xen_nic.c: Free 'netdev->txs' when map 'netdev->rxs' fails
  hw/net/xen_nic.c: Need free 'netdev->nic' in net_free() instead of
    net_disconnect()
  hw/net/xen_nic.c: Set 'netdev->mac' to NULL after free it

Paolo Bonzini (1):
  net: remove all cleanup methods from NIC NetClientInfos

 hw/net/allwinner_emac.c     |  8 --------
 hw/net/cadence_gem.c        |  9 ---------
 hw/net/dp8393x.c            | 11 -----------
 hw/net/e1000.c              |  9 ---------
 hw/net/eepro100.c           |  8 --------
 hw/net/etraxfs_eth.c        | 13 -------------
 hw/net/fsl_etsec/etsec.c    |  6 ------
 hw/net/lan9118.c            |  8 --------
 hw/net/lance.c              |  8 --------
 hw/net/mcf_fec.c            |  8 --------
 hw/net/milkymist-minimac2.c |  8 --------
 hw/net/mipsnet.c            |  8 --------
 hw/net/ne2000-isa.c         |  8 --------
 hw/net/ne2000.c             |  8 --------
 hw/net/opencores_eth.c      |  5 -----
 hw/net/pcnet-pci.c          |  8 --------
 hw/net/pcnet.c              |  5 -----
 hw/net/pcnet.h              |  1 -
 hw/net/rtl8139.c            |  8 --------
 hw/net/smc91c111.c          |  8 --------
 hw/net/spapr_llan.c         |  8 --------
 hw/net/stellaris_enet.c     |  8 --------
 hw/net/virtio-net.c         |  8 --------
 hw/net/vmxnet3.c            |  7 -------
 hw/net/xen_nic.c            | 16 +++++++++++-----
 hw/net/xgmac.c              |  8 --------
 hw/net/xilinx_axienet.c     |  9 ---------
 hw/net/xilinx_ethlite.c     |  8 --------
 28 files changed, 11 insertions(+), 216 deletions(-)

-- 
2.1.0

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 12 January 2015 at 10:27, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 64ea8038ffbf703dcd438a108d2d5499c8ff95d9:
>
>   Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20150109.0' into staging (2015-01-10 22:29:09 +0000)
>
> are available in the git repository at:
>
>   git://github.com/stefanha/qemu.git tags/net-pull-request
>
> for you to fetch changes up to a39d97c7becca5fd679d70d17ae5b1d62b9b5da6:
>
>   hw/net/xen_nic.c: Set 'netdev->mac' to NULL after free it (2015-01-12 10:16:23 +0000)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------
>
>  28 files changed, 11 insertions(+), 216 deletions(-)

If only all pullreqs could have diffstats like this :-)

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 3b730f570c5872ceea2137848f1d4554d4847441:

  Merge remote-tracking branch 'remotes/agraf/tags/signed-ppc-for-upstream' into staging (2015-06-04 14:04:14 +0100)

are available in the git repository at:

  git://github.com/stefanha/qemu.git tags/net-pull-request

for you to fetch changes up to 73e47f4bc1a1aac59dba3f3a5feb27fbcb1cde50:

  qmp/hmp: add rocker device support (2015-06-05 11:19:53 +0100)

----------------------------------------------------------------

----------------------------------------------------------------

David Ahern (1):
  rocker: Add support for phys name

Scott Feldman (3):
  rocker: update tests using hw-derived interface names
  rocker: bring link up/down on PHY enable/disable
  qmp/hmp: add rocker device support

 docs/specs/rocker.txt         |   1 +
 hmp-commands.hx               |  24 ++++
 hmp.c                         | 303 ++++++++++++++++++++++++++++++++++++++++
 hmp.h                         |   4 +
 hw/net/Makefile.objs          |   1 +
 hw/net/rocker/qmp-norocker.c  |  50 +++++++
 hw/net/rocker/rocker.c        |  68 ++++++++-
 hw/net/rocker/rocker_fp.c     |  29 +++-
 hw/net/rocker/rocker_fp.h     |   2 +
 hw/net/rocker/rocker_hw.h     |   1 +
 hw/net/rocker/rocker_of_dpa.c | 312 ++++++++++++++++++++++++++++++++++++++++++
 monitor.c                     |  28 ++++
 qapi-schema.json              |   3 +
 qapi/rocker.json              | 286 ++++++++++++++++++++++++++++++++++++++
 qmp-commands.hx               |  97 +++++++++++++
 tests/rocker/bridge           |  25 ++--
 tests/rocker/bridge-stp       |  25 ++--
 tests/rocker/bridge-vlan      |  37 +++--
 tests/rocker/bridge-vlan-stp  |  37 +++--
 tests/rocker/port             |   8 +-
 20 files changed, 1263 insertions(+), 78 deletions(-)
 create mode 100644 hw/net/rocker/qmp-norocker.c
 create mode 100644 qapi/rocker.json

-- 
2.4.2

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 5 June 2015 at 11:22, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 3b730f570c5872ceea2137848f1d4554d4847441:
>
>   Merge remote-tracking branch 'remotes/agraf/tags/signed-ppc-for-upstream' into staging (2015-06-04 14:04:14 +0100)
>
> are available in the git repository at:
>
>   git://github.com/stefanha/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 73e47f4bc1a1aac59dba3f3a5feb27fbcb1cde50:
>
>   qmp/hmp: add rocker device support (2015-06-05 11:19:53 +0100)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------
>
> David Ahern (1):
>   rocker: Add support for phys name
>
> Scott Feldman (3):
>   rocker: update tests using hw-derived interface names
>   rocker: bring link up/down on PHY enable/disable
>   qmp/hmp: add rocker device support

Eric has some review comments on the qmp/hmp patch which haven't
been addressed yet, so I'm not applying this one. (I noticed the
commit message misformatting too...)

thanks
-- PMM

[Qemu-devel] [PULL 0/4] Net patches

[Stefan Hajnoczi]

The following changes since commit 619622424dba749feef752d76d79ef2569f7f250:

  Merge remote-tracking branch 'remotes/berrange/tags/vnc-crypto-v9-for-upstream' into staging (2015-09-15 15:42:58 +0100)

are available in the git repository at:

  git://github.com/stefanha/qemu.git tags/net-pull-request

for you to fetch changes up to 271a234a2359975101396916f37f3c7d347c61b8:

  net: smc91c111: flush packets on RCR register changes (2015-09-17 12:36:03 +0100)

----------------------------------------------------------------

----------------------------------------------------------------

Jason Wang (1):
  MAINTAINERS: Stefan will not maintain net subsystem

Peter Crosthwaite (3):
  net: smc91c111: guard flush_queued_packets() on can_rx()
  net: smc91c111: gate can_receive() on rx FIFO having a slot
  net: smc91c111: flush packets on RCR register changes

 MAINTAINERS        |  3 +--
 hw/net/smc91c111.c | 33 +++++++++++++++++++++++++--------
 2 files changed, 26 insertions(+), 10 deletions(-)

-- 
2.4.3

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 17 September 2015 at 12:43, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> The following changes since commit 619622424dba749feef752d76d79ef2569f7f250:
>
>   Merge remote-tracking branch 'remotes/berrange/tags/vnc-crypto-v9-for-upstream' into staging (2015-09-15 15:42:58 +0100)
>
> are available in the git repository at:
>
>   git://github.com/stefanha/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 271a234a2359975101396916f37f3c7d347c61b8:
>
>   net: smc91c111: flush packets on RCR register changes (2015-09-17 12:36:03 +0100)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/4] Net patches

[Jason Wang]

The following changes since commit c74e62ee3e2dc2955e07d004c71badecb68a84eb:

  Merge remote-tracking branch 'remotes/rth/tags/cota-target-pull-request' into staging (2018-05-11 15:41:29 +0100)

are available in the git repository at:

  https://github.com/jasowang/qemu.git tags/net-pull-request

for you to fetch changes up to 0e0266c2e475b82b39a757c875fa03e64272fbe7:

  net: Get rid of 'vlan' terminology and use 'hub' instead in the doc files (2018-05-14 15:47:14 +0800)

----------------------------------------------------------------

----------------------------------------------------------------
Thomas Huth (4):
      net: Fix memory leak in net_param_nic()
      net: Remove the deprecated "vlan" parameter
      net: Get rid of 'vlan' terminology and use 'hub' instead in the source files
      net: Get rid of 'vlan' terminology and use 'hub' instead in the doc files

 docs/qdev-device-use.txt         |  3 --
 hw/core/qdev-properties-system.c | 80 ----------------------------------------
 include/hw/qdev-properties.h     |  3 --
 include/net/net.h                |  1 -
 net/hub.c                        |  7 ++--
 net/net.c                        | 18 ++++-----
 net/slirp.c                      |  8 ++--
 net/tap.c                        |  4 +-
 qapi/net.json                    | 15 ++++----
 qemu-doc.texi                    | 51 ++++++++++++-------------
 qemu-options.hx                  | 29 ++++++---------
 11 files changed, 58 insertions(+), 161 deletions(-)

[Qemu-devel] [PULL 1/4] net: Fix memory leak in net_param_nic()

[Jason Wang]

From: Thomas Huth <thuth@redhat.com>

The early exits in case of errors leak the memory allocated for nd_id.
Fix it by using a "goto out" to the cleanup at the end of the function
instead.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 net/net.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/net/net.c b/net/net.c
index 29f8398..65457b7 100644
--- a/net/net.c
+++ b/net/net.c
@@ -1502,11 +1502,12 @@ static int net_param_nic(void *dummy, QemuOpts *opts, Error **errp)
         g_free(mac);
         if (ret) {
             error_setg(errp, "invalid syntax for ethernet address");
-            return -1;
+            goto out;
         }
         if (is_multicast_ether_addr(ni->macaddr.a)) {
             error_setg(errp, "NIC cannot have multicast MAC address");
-            return -1;
+            ret = -1;
+            goto out;
         }
     }
     qemu_macaddr_default_if_unset(&ni->macaddr);
@@ -1518,6 +1519,7 @@ static int net_param_nic(void *dummy, QemuOpts *opts, Error **errp)
         nb_nics++;
     }
 
+out:
     g_free(nd_id);
     return ret;
 }
-- 
2.7.4

[Qemu-devel] [PULL 2/4] net: Remove the deprecated "vlan" parameter

[Jason Wang]

From: Thomas Huth <thuth@redhat.com>

It's been marked as deprecated since QEMU v2.9.0, so that should have
been enough time for everybody to either just drop unnecessary "vlan=0"
parameters, to switch to the modern -device + -netdev syntax for connecting
guest NICs with host network backends, or to switch to the "hubport" netdev
in case hubs are really wanted instead.

Buglink: https://bugs.launchpad.net/qemu/+bug/658904
Signed-off-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 docs/qdev-device-use.txt         |  3 --
 hw/core/qdev-properties-system.c | 80 ----------------------------------------
 include/hw/qdev-properties.h     |  3 --
 include/net/net.h                |  1 -
 net/net.c                        | 12 ++----
 qapi/net.json                    | 15 ++++----
 qemu-doc.texi                    |  9 -----
 qemu-options.hx                  | 29 ++++++---------
 8 files changed, 22 insertions(+), 130 deletions(-)

diff --git a/docs/qdev-device-use.txt b/docs/qdev-device-use.txt
index 8f188d1..98229b3 100644
--- a/docs/qdev-device-use.txt
+++ b/docs/qdev-device-use.txt
@@ -277,9 +277,6 @@ devices and ne2k_isa are.
 
 Some PCI devices aren't available with -net nic, e.g. i82558a.
 
-To connect to a VLAN instead of an ordinary host part, replace
-netdev=NET-ID by vlan=VLAN.
-
 === Graphics Devices ===
 
 Host and guest part of graphics devices have always been separate.
diff --git a/hw/core/qdev-properties-system.c b/hw/core/qdev-properties-system.c
index 1d3ba72..8b22fb5 100644
--- a/hw/core/qdev-properties-system.c
+++ b/hw/core/qdev-properties-system.c
@@ -321,86 +321,6 @@ const PropertyInfo qdev_prop_netdev = {
     .set   = set_netdev,
 };
 
-/* --- vlan --- */
-
-static int print_vlan(DeviceState *dev, Property *prop, char *dest, size_t len)
-{
-    NetClientState **ptr = qdev_get_prop_ptr(dev, prop);
-
-    if (*ptr) {
-        int id;
-        if (!net_hub_id_for_client(*ptr, &id)) {
-            return snprintf(dest, len, "%d", id);
-        }
-    }
-
-    return snprintf(dest, len, "<null>");
-}
-
-static void get_vlan(Object *obj, Visitor *v, const char *name, void *opaque,
-                     Error **errp)
-{
-    DeviceState *dev = DEVICE(obj);
-    Property *prop = opaque;
-    NetClientState **ptr = qdev_get_prop_ptr(dev, prop);
-    int32_t id = -1;
-
-    if (*ptr) {
-        int hub_id;
-        if (!net_hub_id_for_client(*ptr, &hub_id)) {
-            id = hub_id;
-        }
-    }
-
-    visit_type_int32(v, name, &id, errp);
-}
-
-static void set_vlan(Object *obj, Visitor *v, const char *name, void *opaque,
-                     Error **errp)
-{
-    DeviceState *dev = DEVICE(obj);
-    Property *prop = opaque;
-    NICPeers *peers_ptr = qdev_get_prop_ptr(dev, prop);
-    NetClientState **ptr = &peers_ptr->ncs[0];
-    Error *local_err = NULL;
-    int32_t id;
-    NetClientState *hubport;
-
-    if (dev->realized) {
-        qdev_prop_set_after_realize(dev, name, errp);
-        return;
-    }
-
-    visit_type_int32(v, name, &id, &local_err);
-    if (local_err) {
-        error_propagate(errp, local_err);
-        return;
-    }
-    if (id == -1) {
-        *ptr = NULL;
-        return;
-    }
-    if (*ptr) {
-        error_set_from_qdev_prop_error(errp, -EINVAL, dev, prop, name);
-        return;
-    }
-
-    hubport = net_hub_port_find(id);
-    if (!hubport) {
-        error_setg(errp, QERR_INVALID_PARAMETER_VALUE,
-                   name, prop->info->name);
-        return;
-    }
-    *ptr = hubport;
-}
-
-const PropertyInfo qdev_prop_vlan = {
-    .name  = "int32",
-    .description = "Integer VLAN id to connect to",
-    .print = print_vlan,
-    .get   = get_vlan,
-    .set   = set_vlan,
-};
 
 void qdev_prop_set_drive(DeviceState *dev, const char *name,
                          BlockBackend *value, Error **errp)
diff --git a/include/hw/qdev-properties.h b/include/hw/qdev-properties.h
index b2ad8e9..4f60cc8 100644
--- a/include/hw/qdev-properties.h
+++ b/include/hw/qdev-properties.h
@@ -29,7 +29,6 @@ extern const PropertyInfo qdev_prop_bios_chs_trans;
 extern const PropertyInfo qdev_prop_fdc_drive_type;
 extern const PropertyInfo qdev_prop_drive;
 extern const PropertyInfo qdev_prop_netdev;
-extern const PropertyInfo qdev_prop_vlan;
 extern const PropertyInfo qdev_prop_pci_devfn;
 extern const PropertyInfo qdev_prop_blocksize;
 extern const PropertyInfo qdev_prop_pci_host_devaddr;
@@ -195,8 +194,6 @@ extern const PropertyInfo qdev_prop_off_auto_pcibar;
     DEFINE_PROP(_n, _s, _f, qdev_prop_string, char*)
 #define DEFINE_PROP_NETDEV(_n, _s, _f)             \
     DEFINE_PROP(_n, _s, _f, qdev_prop_netdev, NICPeers)
-#define DEFINE_PROP_VLAN(_n, _s, _f)             \
-    DEFINE_PROP(_n, _s, _f, qdev_prop_vlan, NICPeers)
 #define DEFINE_PROP_DRIVE(_n, _s, _f) \
     DEFINE_PROP(_n, _s, _f, qdev_prop_drive, BlockBackend *)
 #define DEFINE_PROP_MACADDR(_n, _s, _f)         \
diff --git a/include/net/net.h b/include/net/net.h
index 1f7341e..1425960 100644
--- a/include/net/net.h
+++ b/include/net/net.h
@@ -37,7 +37,6 @@ typedef struct NICConf {
 
 #define DEFINE_NIC_PROPERTIES(_state, _conf)                            \
     DEFINE_PROP_MACADDR("mac",   _state, _conf.macaddr),                \
-    DEFINE_PROP_VLAN("vlan",     _state, _conf.peers),                   \
     DEFINE_PROP_NETDEV("netdev", _state, _conf.peers)
 
 
diff --git a/net/net.c b/net/net.c
index 65457b7..efb9eaf 100644
--- a/net/net.c
+++ b/net/net.c
@@ -965,7 +965,6 @@ static int net_client_init1(const void *object, bool is_netdev, Error **errp)
     const Netdev *netdev;
     const char *name;
     NetClientState *peer = NULL;
-    static bool vlan_warned;
 
     if (is_netdev) {
         netdev = object;
@@ -1036,15 +1035,10 @@ static int net_client_init1(const void *object, bool is_netdev, Error **errp)
             return -1;
         }
 
-        /* Do not add to a vlan if it's a nic with a netdev= parameter. */
+        /* Do not add to a hub if it's a nic with a netdev= parameter. */
         if (netdev->type != NET_CLIENT_DRIVER_NIC ||
             !opts->u.nic.has_netdev) {
-            peer = net_hub_add_port(net->has_vlan ? net->vlan : 0, NULL, NULL);
-        }
-
-        if (net->has_vlan && !vlan_warned) {
-            error_report("'vlan' is deprecated. Please use 'netdev' instead.");
-            vlan_warned = true;
+            peer = net_hub_add_port(0, NULL, NULL);
         }
     }
 
@@ -1365,7 +1359,7 @@ void qmp_set_link(const char *name, bool up, Error **errp)
          * If the peer is a HUBPORT or a backend, we do not change the
          * link status.
          *
-         * This behavior is compatible with qemu vlans where there could be
+         * This behavior is compatible with qemu hubs where there could be
          * multiple clients that can still communicate with each other in
          * disconnected mode. For now maintain this compatibility.
          */
diff --git a/qapi/net.json b/qapi/net.json
index 9117c56..b4fe4b6 100644
--- a/qapi/net.json
+++ b/qapi/net.json
@@ -209,7 +209,7 @@
 ##
 # @NetdevTapOptions:
 #
-# Connect the host TAP network interface name to the VLAN.
+# Used to configure a host TAP network interface backend.
 #
 # @ifname: interface name
 #
@@ -267,8 +267,8 @@
 ##
 # @NetdevSocketOptions:
 #
-# Connect the VLAN to a remote VLAN in another QEMU virtual machine using a TCP
-# socket connection.
+# Socket netdevs are used to establish a network connection to another
+# QEMU virtual machine via a TCP socket.
 #
 # @fd: file descriptor of an already opened socket
 #
@@ -296,7 +296,7 @@
 ##
 # @NetdevL2TPv3Options:
 #
-# Connect the VLAN to Ethernet over L2TPv3 Static tunnel
+# Configure an Ethernet over L2TPv3 tunnel.
 #
 # @src: source address
 #
@@ -352,7 +352,7 @@
 ##
 # @NetdevVdeOptions:
 #
-# Connect the VLAN to a vde switch running on the host.
+# Connect to a vde switch running on the host.
 #
 # @sock: socket path
 #
@@ -490,8 +490,6 @@
 #
 # Captures the configuration of a network device; legacy.
 #
-# @vlan: vlan number
-#
 # @id: identifier for monitor commands
 #
 # @name: identifier for monitor commands, ignored if @id is present
@@ -499,10 +497,11 @@
 # @opts: device type specific properties (legacy)
 #
 # Since: 1.2
+#
+# 'vlan' - removed with 2.12
 ##
 { 'struct': 'NetLegacy',
   'data': {
-    '*vlan': 'int32',
     '*id':   'str',
     '*name': 'str',
     'opts':  'NetLegacyOptions' } }
diff --git a/qemu-doc.texi b/qemu-doc.texi
index 0ed0f19..9216848 100644
--- a/qemu-doc.texi
+++ b/qemu-doc.texi
@@ -2855,15 +2855,6 @@ with ``-device ...,netdev=x''), or ``-nic user,smb=/some/dir''
 (for embedded NICs). The new syntax allows different settings to be
 provided per NIC.
 
-@subsection -net vlan (since 2.9.0)
-
-The ``-net vlan=NN'' argument was mostly used to attach separate
-network backends to different virtual NICs.  This is the default
-behavior for ``-netdev'' and ``-nic''.  You can connect multiple
-``-netdev'' and ``-nic'' devices to the same network using the
-"hubport" network backend, created with ``-netdev hubport,hubid=NN,...''
-and ``-nic hubport,hubid=NN''.
-
 @subsection -drive cyls=...,heads=...,secs=...,trans=... (since 2.10.0)
 
 The drive geometry arguments are replaced by the the geometry arguments
diff --git a/qemu-options.hx b/qemu-options.hx
index c611766..3588b04 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2003,7 +2003,7 @@ DEF("netdev", HAS_ARG, QEMU_OPTION_netdev,
     "                configure a vhost-user network, backed by a chardev 'dev'\n"
 #endif
     "-netdev hubport,id=str,hubid=n[,netdev=nd]\n"
-    "                configure a hub port on QEMU VLAN 'n'\n", QEMU_ARCH_ALL)
+    "                configure a hub port on the hub with ID 'n'\n", QEMU_ARCH_ALL)
 DEF("nic", HAS_ARG, QEMU_OPTION_nic,
     "--nic [tap|bridge|"
 #ifdef CONFIG_SLIRP
@@ -2028,10 +2028,9 @@ DEF("nic", HAS_ARG, QEMU_OPTION_nic,
     "                provided a 'user' network connection)\n",
     QEMU_ARCH_ALL)
 DEF("net", HAS_ARG, QEMU_OPTION_net,
-    "-net nic[,vlan=n][,netdev=nd][,macaddr=mac][,model=type][,name=str][,addr=str][,vectors=v]\n"
+    "-net nic[,macaddr=mac][,model=type][,name=str][,addr=str][,vectors=v]\n"
     "                configure or create an on-board (or machine default) NIC and\n"
-    "                connect it either to VLAN 'n' or the netdev 'nd' (for pluggable\n"
-    "                NICs please use '-device devtype,netdev=nd' instead)\n"
+    "                connect it to hub 0 (please use -nic unless you need a hub)\n"
     "-net ["
 #ifdef CONFIG_SLIRP
     "user|"
@@ -2044,7 +2043,7 @@ DEF("net", HAS_ARG, QEMU_OPTION_net,
 #ifdef CONFIG_NETMAP
     "netmap|"
 #endif
-    "socket][,vlan=n][,option][,option][,...]\n"
+    "socket][,option][,option][,...]\n"
     "                old way to initialize a host network interface\n"
     "                (use the -netdev option if possible instead)\n", QEMU_ARCH_ALL)
 STEXI
@@ -2462,17 +2461,14 @@ qemu -m 512 -object memory-backend-file,id=mem,size=512M,mem-path=/hugetlbfs,sha
 Create a hub port on the emulated hub with ID @var{hubid}.
 
 The hubport netdev lets you connect a NIC to a QEMU emulated hub instead of a
-single netdev. @code{-net} and @code{-device} with the parameter @option{vlan}
-(deprecated), or @code{-nic hubport} can also be used to connect a
-network device or a NIC to a hub. Alternatively, you can also connect the
-hubport to another netdev with ID @var{nd} by using the @option{netdev=@var{nd}}
-option.
+single netdev. Alternatively, you can also connect the hubport to another
+netdev with ID @var{nd} by using the @option{netdev=@var{nd}} option.
 
-@item -net nic[,vlan=@var{n}][,netdev=@var{nd}][,macaddr=@var{mac}][,model=@var{type}] [,name=@var{name}][,addr=@var{addr}][,vectors=@var{v}]
+@item -net nic[,netdev=@var{nd}][,macaddr=@var{mac}][,model=@var{type}] [,name=@var{name}][,addr=@var{addr}][,vectors=@var{v}]
 @findex -net
 Legacy option to configure or create an on-board (or machine default) Network
-Interface Card(NIC) and connect it either to the emulated hub port ("vlan")
-with number @var{n} (@var{n} = 0 is the default), or to the netdev @var{nd}.
+Interface Card(NIC) and connect it either to the emulated hub with ID 0 (i.e.
+the default hub), or to the netdev @var{nd}.
 The NIC is an e1000 by default on the PC target. Optionally, the MAC address
 can be changed to @var{mac}, the device address set to @var{addr} (PCI cards
 only), and a @var{name} can be assigned for use in monitor commands.
@@ -2482,11 +2478,10 @@ that the card should have; this option currently only affects virtio cards; set
 NIC is created.  QEMU can emulate several different models of network card.
 Use @code{-net nic,model=help} for a list of available devices for your target.
 
-@item -net user|tap|bridge|socket|l2tpv3|vde[,...][,vlan=@var{n}][,name=@var{name}]
+@item -net user|tap|bridge|socket|l2tpv3|vde[,...][,name=@var{name}]
 Configure a host network backend (with the options corresponding to the same
-@option{-netdev} option) and connect it to the emulated hub ("vlan") with the
-number @var{n} (default is number 0). Use @var{name} to specify the name of the
-hub port.
+@option{-netdev} option) and connect it to the emulated hub 0 (the default
+hub). Use @var{name} to specify the name of the hub port.
 ETEXI
 
 STEXI
-- 
2.7.4

[Qemu-devel] [PULL 3/4] net: Get rid of 'vlan' terminology and use 'hub' instead in the source files

[Jason Wang]

From: Thomas Huth <thuth@redhat.com>

'vlan' is very confusing since it does not mean something like IEEE
802.1Q, but rather emulated hubs, so let's switch to that terminology
instead.

Buglink: https://bugs.launchpad.net/qemu/+bug/658904
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 net/hub.c   | 7 +++----
 net/slirp.c | 8 ++++----
 net/tap.c   | 4 ++--
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/net/hub.c b/net/hub.c
index 5e84a9a..78b671e 100644
--- a/net/hub.c
+++ b/net/hub.c
@@ -23,8 +23,7 @@
 
 /*
  * A hub broadcasts incoming packets to all its ports except the source port.
- * Hubs can be used to provide independent network segments, also confusingly
- * named the QEMU 'vlan' feature.
+ * Hubs can be used to provide independent emulated network segments.
  */
 
 typedef struct NetHub NetHub;
@@ -345,10 +344,10 @@ void net_hub_check_clients(void)
             }
         }
         if (has_host_dev && !has_nic) {
-            warn_report("vlan %d with no nics", hub->id);
+            warn_report("hub %d with no nics", hub->id);
         }
         if (has_nic && !has_host_dev) {
-            warn_report("vlan %d is not connected to host network", hub->id);
+            warn_report("hub %d is not connected to host network", hub->id);
         }
     }
 }
diff --git a/net/slirp.c b/net/slirp.c
index 8991816..6922524 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -415,7 +415,7 @@ static SlirpState *slirp_lookup(Monitor *mon, const char *hub_id,
         if (hub_id) {
             nc = net_hub_find_client_by_name(strtol(hub_id, NULL, 0), name);
             if (!nc) {
-                monitor_printf(mon, "unrecognized (vlan-id, stackname) pair\n");
+                monitor_printf(mon, "unrecognized (hub-id, stackname) pair\n");
                 return NULL;
             }
         } else {
@@ -870,9 +870,9 @@ void hmp_info_usernet(Monitor *mon, const QDict *qdict)
 
     QTAILQ_FOREACH(s, &slirp_stacks, entry) {
         int id;
-        bool got_vlan_id = net_hub_id_for_client(&s->nc, &id) == 0;
-        monitor_printf(mon, "VLAN %d (%s):\n",
-                       got_vlan_id ? id : -1,
+        bool got_hub_id = net_hub_id_for_client(&s->nc, &id) == 0;
+        monitor_printf(mon, "Hub %d (%s):\n",
+                       got_hub_id ? id : -1,
                        s->nc.name);
         slirp_connection_info(s->slirp, mon);
     }
diff --git a/net/tap.c b/net/tap.c
index 2b3a36f..de05f20 100644
--- a/net/tap.c
+++ b/net/tap.c
@@ -766,10 +766,10 @@ int net_init_tap(const Netdev *netdev, const char *name,
     queues = tap->has_queues ? tap->queues : 1;
     vhostfdname = tap->has_vhostfd ? tap->vhostfd : NULL;
 
-    /* QEMU vlans does not support multiqueue tap, in this case peer is set.
+    /* QEMU hubs do not support multiqueue tap, in this case peer is set.
      * For -netdev, peer is always NULL. */
     if (peer && (tap->has_queues || tap->has_fds || tap->has_vhostfds)) {
-        error_setg(errp, "Multiqueue tap cannot be used with QEMU vlans");
+        error_setg(errp, "Multiqueue tap cannot be used with hubs");
         return -1;
     }
 
-- 
2.7.4

[Qemu-devel] [PULL 4/4] net: Get rid of 'vlan' terminology and use 'hub' instead in the doc files

[Jason Wang]

From: Thomas Huth <thuth@redhat.com>

'vlan' is very confusing since it does not mean something like IEEE
802.1Q, but rather emulated hubs, so let's switch to that terminology
instead. While we're at it, move the subsection about hub a little bit
downward in the documentation (it's not as important anymore as it was
before the invention of the -netdev parameter), and extend it a little
bit.

Buglink: https://bugs.launchpad.net/qemu/+bug/658904
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 qemu-doc.texi | 42 +++++++++++++++++++++++-------------------
 1 file changed, 23 insertions(+), 19 deletions(-)

diff --git a/qemu-doc.texi b/qemu-doc.texi
index 9216848..e2acac8 100644
--- a/qemu-doc.texi
+++ b/qemu-doc.texi
@@ -714,20 +714,12 @@ state is not saved or restored properly (in particular USB).
 @node pcsys_network
 @section Network emulation
 
-QEMU can simulate several network cards (PCI or ISA cards on the PC
-target) and can connect them to an arbitrary number of Virtual Local
-Area Networks (VLANs). Host TAP devices can be connected to any QEMU
-VLAN. VLAN can be connected between separate instances of QEMU to
-simulate large networks. For simpler usage, a non privileged user mode
-network stack can replace the TAP device to have a basic network
-connection.
-
-@subsection VLANs
-
-QEMU simulates several VLANs. A VLAN can be symbolised as a virtual
-connection between several network devices. These devices can be for
-example QEMU virtual Ethernet cards or virtual Host ethernet devices
-(TAP devices).
+QEMU can simulate several network cards (e.g. PCI or ISA cards on the PC
+target) and can connect them to a network backend on the host or an emulated
+hub. The various host network backends can either be used to connect the NIC of
+the guest to a real network (e.g. by using a TAP devices or the non-privileged
+user mode network stack), or to other guest instances running in another QEMU
+process (e.g. by using the socket host network backend).
 
 @subsection Using TAP network interfaces
 
@@ -763,7 +755,7 @@ network). The virtual network configuration is the following:
 
 @example
 
-         QEMU VLAN      <------>  Firewall/DHCP server <-----> Internet
+     guest (10.0.2.15)  <------>  Firewall/DHCP server <-----> Internet
                            |          (10.0.2.2)
                            |
                            ---->  DNS server (10.0.2.3)
@@ -798,11 +790,23 @@ When using the @option{'-netdev user,hostfwd=...'} option, TCP or UDP
 connections can be redirected from the host to the guest. It allows for
 example to redirect X11, telnet or SSH connections.
 
-@subsection Connecting VLANs between QEMU instances
+@subsection Hubs
+
+QEMU can simulate several hubs. A hub can be thought of as a virtual connection
+between several network devices. These devices can be for example QEMU virtual
+ethernet cards or virtual Host ethernet devices (TAP devices). You can connect
+guest NICs or host network backends to such a hub using the @option{-netdev
+hubport} or @option{-nic hubport} options. The legacy @option{-net} option
+also connects the given device to the emulated hub with ID 0 (i.e. the default
+hub) unless you specify a netdev with @option{-net nic,netdev=xxx} here.
+
+@subsection Connecting emulated networks between QEMU instances
 
-Using the @option{-net socket} option, it is possible to make VLANs
-that span several QEMU instances. See @ref{sec_invocation} to have a
-basic example.
+Using the @option{-netdev socket} (or @option{-nic socket} or
+@option{-net socket}) option, it is possible to create emulated
+networks that span several QEMU instances.
+See the description of the @option{-netdev socket} option in the
+@ref{sec_invocation,,Invocation chapter} to have a basic example.
 
 @node pcsys_other_devs
 @section Other Devices
-- 
2.7.4

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 14 May 2018 at 08:55, Jason Wang <jasowang@redhat.com> wrote:
> The following changes since commit c74e62ee3e2dc2955e07d004c71badecb68a84eb:
>
>   Merge remote-tracking branch 'remotes/rth/tags/cota-target-pull-request' into staging (2018-05-11 15:41:29 +0100)
>
> are available in the git repository at:
>
>   https://github.com/jasowang/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 0e0266c2e475b82b39a757c875fa03e64272fbe7:
>
>   net: Get rid of 'vlan' terminology and use 'hub' instead in the doc files (2018-05-14 15:47:14 +0800)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------
> Thomas Huth (4):
>       net: Fix memory leak in net_param_nic()
>       net: Remove the deprecated "vlan" parameter
>       net: Get rid of 'vlan' terminology and use 'hub' instead in the source files
>       net: Get rid of 'vlan' terminology and use 'hub' instead in the doc files
>

Applied, thanks.

-- PMM

Re: [Qemu-devel] [PULL 2/4] net: Remove the deprecated "vlan" parameter

[Eric Blake]

On 05/14/2018 02:55 AM, Jason Wang wrote:
> From: Thomas Huth <thuth@redhat.com>
> 
> It's been marked as deprecated since QEMU v2.9.0, so that should have
> been enough time for everybody to either just drop unnecessary "vlan=0"
> parameters, to switch to the modern -device + -netdev syntax for connecting
> guest NICs with host network backends, or to switch to the "hubport" netdev
> in case hubs are really wanted instead.
> 
> Buglink: https://bugs.launchpad.net/qemu/+bug/658904
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
> Signed-off-by: Jason Wang <jasowang@redhat.com>
> ---

> @@ -499,10 +497,11 @@
>   # @opts: device type specific properties (legacy)
>   #
>   # Since: 1.2
> +#
> +# 'vlan' - removed with 2.12

Shouldn't this be 2.13?  Also, while the phrase 'removed with XXX' 
already appears for NetClientDriver::dump, it sounds awkward; better 
would be matching the way we document things for BlockDeviceInfo:

NetClientDriver
2.12: dropped 'dump'
...
NetLegacy
2.13: dropped 'vlan'

or the way we do it for QKeyCode:

NetClientDriver
dump: dropped in 2.12
...
NetLegacy
vlan: dropped in 2.13

As this is already in a pull request, and as NetClientDriver also needs 
a cleanup, that can be a followup patch.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PULL 2/4] net: Remove the deprecated "vlan" parameter

[Thomas Huth]

On 15.05.2018 17:10, Eric Blake wrote:
> On 05/14/2018 02:55 AM, Jason Wang wrote:
>> From: Thomas Huth <thuth@redhat.com>
>>
>> It's been marked as deprecated since QEMU v2.9.0, so that should have
>> been enough time for everybody to either just drop unnecessary "vlan=0"
>> parameters, to switch to the modern -device + -netdev syntax for
>> connecting
>> guest NICs with host network backends, or to switch to the "hubport"
>> netdev
>> in case hubs are really wanted instead.
>>
>> Buglink: https://bugs.launchpad.net/qemu/+bug/658904
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
>> Signed-off-by: Jason Wang <jasowang@redhat.com>
>> ---
> 
>> @@ -499,10 +497,11 @@
>>   # @opts: device type specific properties (legacy)
>>   #
>>   # Since: 1.2
>> +#
>> +# 'vlan' - removed with 2.12
> 
> Shouldn't this be 2.13?

Oh man, yes, of course. Stefan already found that while reviewing my
patches (v3), but apparently I forgot to update it in the final v4 :-(
Sorry for that.

> NetClientDriver
> 2.12: dropped 'dump'
> ...
> NetLegacy
> 2.13: dropped 'vlan'
> 
> or the way we do it for QKeyCode:
> 
> NetClientDriver
> dump: dropped in 2.12
> ...
> NetLegacy
> vlan: dropped in 2.13

I think I slightly prefer the latter variant.

> As this is already in a pull request, and as NetClientDriver also needs
> a cleanup, that can be a followup patch.

The pull request has already been merged. I'll send a followup patch...

 Thomas

[Qemu-devel] [PULL 0/4] Net patches

[Jason Wang]

The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5:

  Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100)

are available in the git repository at:

  https://github.com/jasowang/qemu.git tags/net-pull-request

for you to fetch changes up to c67daf4a24442d1bb404a11a6a54dc45ea10f234:

  vhost-user: delete net client if necessary (2018-06-15 10:39:53 +0800)

----------------------------------------------------------------

----------------------------------------------------------------
Brijesh Singh (1):
      tap: set vhostfd passed from qemu cli to non-blocking

Jan Kiszka (1):
      e1000e: Do not auto-clear ICR bits which aren't set in EIAC

Lin Ma (1):
      net: Fix a potential segfault

linzhecheng (1):
      vhost-user: delete net client if necessary

 hw/net/e1000e_core.c | 4 ----
 net/net.c            | 4 +++-
 net/tap.c            | 2 ++
 net/vhost-user.c     | 3 +++
 4 files changed, 8 insertions(+), 5 deletions(-)

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On 15 June 2018 at 04:01, Jason Wang <jasowang@redhat.com> wrote:
> The following changes since commit 409c241f887a38bb7a2ac12e34d3a8d73922a9a5:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/ui-20180614-pull-request' into staging (2018-06-14 14:04:14 +0100)
>
> are available in the git repository at:
>
>   https://github.com/jasowang/qemu.git tags/net-pull-request
>
> for you to fetch changes up to c67daf4a24442d1bb404a11a6a54dc45ea10f234:
>
>   vhost-user: delete net client if necessary (2018-06-15 10:39:53 +0800)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM

[Qemu-devel] [PULL 0/4] Net patches

[Jason Wang]

The following changes since commit d8276573da58e8ce78dab8c46dd660efd664bcb7:

  Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20190510' into staging (2019-05-16 13:15:08 +0100)

are available in the git repository at:

  https://github.com/jasowang/qemu.git tags/net-pull-request

for you to fetch changes up to 78e4f446d2569210a8558946b2321f9ff2ef47f6:

  net/colo-compare.c: Fix a crash in COLO Primary. (2019-05-17 17:00:12 +0800)

----------------------------------------------------------------

----------------------------------------------------------------
Chris Kenna (1):
      e1000: Never increment the RX undersize count register

Jason Wang (1):
      vhost_net: don't set backend for the uninitialized virtqueue

Lukas Straub (1):
      net/colo-compare.c: Fix a crash in COLO Primary.

Stefano Garzarella (1):
      net/slirp: fix the IPv6 prefix length error message

 hw/net/e1000.c             |  1 -
 hw/net/vhost_net.c         | 10 ++++++++++
 hw/virtio/virtio.c         |  5 +++++
 include/hw/virtio/virtio.h |  1 +
 net/colo-compare.c         |  3 +--
 net/slirp.c                |  3 ++-
 6 files changed, 19 insertions(+), 4 deletions(-)

Re: [Qemu-devel] [PULL 0/4] Net patches

[Peter Maydell]

On Fri, 17 May 2019 at 10:07, Jason Wang <jasowang@redhat.com> wrote:
>
> The following changes since commit d8276573da58e8ce78dab8c46dd660efd664bcb7:
>
>   Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20190510' into staging (2019-05-16 13:15:08 +0100)
>
> are available in the git repository at:
>
>   https://github.com/jasowang/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 78e4f446d2569210a8558946b2321f9ff2ef47f6:
>
>   net/colo-compare.c: Fix a crash in COLO Primary. (2019-05-17 17:00:12 +0800)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.1
for any user-visible changes.

-- PMM