[Qemu-devel] Open Source Virtualization and Cloud at FOSDEM

2011-11-11 Thread Mark McLoughlin
Hey, Here's one for the calendar - a 2 day track at FOSDEM devoted to Open Source Virtualization and Cloud. Hope to see you there! :) Cheers, Mark. Forwarded Message From: Thierry Carrez thie...@openstack.org To: openst...@lists.launchpad.net openst...@lists.launchpad.net

Re: [Qemu-devel] [PATCH 0/2] netdev fixes

2011-07-20 Thread Mark McLoughlin
/get_maintainer.pl -f net.c net/ Anthony Liguori aligu...@us.ibm.com (maintainer:Network device layer,commit_signer:4/16=25%) Mark McLoughlin mar...@redhat.com (maintainer:Network device layer) This is clearly out of date - feel free to remove it Cheers, Mark.

[Qemu-devel] Re: [PATCH 0/6] Assorted netdev fixes

2010-02-11 Thread Mark McLoughlin
net: Monitor command set_link finds only VLAN clients, fix Nice. They all look good to me Acked-by: Mark McLoughlin mar...@redhat.com Cheers, Mark.

Re: [Qemu-devel] Re: Spice project is now open

2009-12-14 Thread Mark McLoughlin
On Mon, 2009-12-14 at 15:10 +, Daniel P. Berrange wrote: The model I had in mind was for the proxy to define a VNC extension that allows the client to query what 'desktops' are available and request switching between them at any time. The list of desktop would of course be authorized per

Re: [Qemu-devel] Spice project is now open

2009-12-11 Thread Mark McLoughlin
On Fri, 2009-12-11 at 13:04 -0600, Anthony Liguori wrote: But to introduce another protocol where a user has to make a choice to use Spice over VNC, I think we need a really good justification for that. It's really about complexity. A user shouldn't have to know about Spice or VNC. They

Re: [Qemu-devel] vmware video broken in git

2009-12-10 Thread Mark McLoughlin
Hi Dave, On Fri, 2009-12-11 at 14:59 +1000, Dave Airlie wrote: Hi guys, Building 0.11.0, qemu -vga vmware -sdl works fine. with git I just get a blank screen. I'm trying to bisect but I keep running into a big chunk of NIC changes where none of the drivers compile, so I thought maybe

Re: [Qemu-devel] [PATCH, try 2] qemu/tap: add -net tap,dev= option

2009-12-09 Thread Mark McLoughlin
-off-by: Arnd Bergmann a...@arndb.de Looks fine to me Acked-by: Mark McLoughlin mar...@redhat.com Cheers, Mark.

Re: [Qemu-devel] [PATCH 09/44] net: introduce qemu_new_net_client()

2009-11-26 Thread Mark McLoughlin
On Wed, 2009-11-25 at 20:36 +, Blue Swirl wrote: On Wed, Nov 25, 2009 at 6:49 PM, Mark McLoughlin mar...@redhat.com wrote: A replacement for qemu_new_vlan_client(), using NetClientInfo to replace most arguments. +VLANClientState *qemu_new_net_client(NetClientInfo *info

Re: [Qemu-devel] Re: [PATCH] Fix TAP networking on host kernels without IFF_VNET_HDR support

2009-11-25 Thread Mark McLoughlin
On Tue, 2009-11-24 at 22:27 +0100, Pierre Riteau wrote: On 24 nov. 2009, at 12:22, Mark McLoughlin wrote: On Tue, 2009-11-24 at 12:17 +0100, Pierre Riteau wrote: Isn't there a way to detect whether the kernel supports the TUNSETOFFLOAD ioctl at all? The kernel will set errno

Re: [Qemu-devel] [PATCH] Check for TUNSETOFFLOAD support before trying to enable offload features

2009-11-25 Thread Mark McLoughlin
On Wed, 2009-11-25 at 11:04 +0100, Pierre Riteau wrote: This avoids the TUNSETOFFLOAD ioctl() failed: Invalid argument message on kernels without TUNSETOFFLOAD support. Signed-off-by: Pierre Riteau pierre.rit...@irisa.fr Acked-by: Mark McLoughlin mar...@redhat.com Thanks, Mark. --- net

[Qemu-devel] [PATCH 03/44] net: move socket backend code from net.c to net/socket.c

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile |1 + net.c| 530 +-- net/socket.c | 561 ++ net/socket.h | 33 4 files changed, 596 insertions

[Qemu-devel] [PATCH 00/44] Fix announce_self(), misc net fixes and cleanups

2009-11-25 Thread Mark McLoughlin
Hi, Here's a new series of patches mainly to fix qemu_announce_self() with -netdev as discussed earlier on the list. I would have liked to have split the series in three, but there would be conflicts between each, so here goes: 1) Patches 01 to 05 - net.c cleanups These

[Qemu-devel] [PATCH 02/44] net: move vde code from net.c to net/vde.c

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile |1 + net.c | 102 +- net/vde.c | 124 + net/vde.h | 36 ++ 4 files changed, 162 insertions

[Qemu-devel] [PATCH 04/44] net: move dump backend code from net.c to net/dump.c

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile |1 + net.c | 127 +-- net/dump.c | 151 net/dump.h | 33 + 4 files changed, 186 insertions(+), 126

[Qemu-devel] [PATCH 01/44] net: move slirp code from net.c to net/slirp.c

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile|1 + monitor.c |1 + net.c | 718 +--- net.h |8 +- net/slirp.c | 752 +++ net/slirp.h | 51

[Qemu-devel] [PATCH 06/44] net: remove NICInfo::vc

2009-11-25 Thread Mark McLoughlin
Since 1cc33683, this field is not set for most devices, so just remove it and its remaining few uses. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c | 10 +- hw/etraxfs_eth.c | 10 +- hw/mcf_fec.c | 10 +- hw/mipsnet.c | 10

[Qemu-devel] [PATCH 17/44] net: introduce NICState and qemu_new_nic()

2009-11-25 Thread Mark McLoughlin
Common state for all NICs. The opaque member will replace the opaque member in VLANClientState since only NICs need it. The conf member will allow us to iterate over NICs, access the MAC addr for the NIC and send a packet from each NIC in qemu_announce_self(). Signed-off-by: Mark McLoughlin mar

[Qemu-devel] [PATCH 10/44] qdev: move DO_UPCAST() into osdep.h

2009-11-25 Thread Mark McLoughlin
Nothing qdev specific about this, make it available throughtout. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/qdev.h | 10 -- osdep.h | 10 ++ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/hw/qdev.h b/hw/qdev.h index 41642ee..8d53754 100644

[Qemu-devel] [PATCH 11/44] net: convert tap to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap.c | 82 +--- 1 files changed, 45 insertions(+), 37 deletions(-) diff --git a/net/tap.c b/net/tap.c index b2e5908..d34feec 100644 --- a/net/tap.c +++ b/net/tap.c @@ -47,7 +47,7

[Qemu-devel] [PATCH 28/44] net: convert mipsnet to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/mipsnet.c | 45 +++-- 1 files changed, 27 insertions(+), 18 deletions(-) diff --git a/hw/mipsnet.c b/hw/mipsnet.c index 67160a4..a066f63 100644 --- a/hw/mipsnet.c +++ b/hw/mipsnet.c @@ -35,7 +35,8

[Qemu-devel] [PATCH 34/44] net: move parse_macaddr() to net/util.[ch]

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile |2 +- net.c | 35 ++- net/util.c | 60 net/util.h | 32 4 files changed, 95 insertions

[Qemu-devel] [PATCH 35/44] net: convert xen to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/xen_nic.c | 46 +- 1 files changed, 29 insertions(+), 17 deletions(-) diff --git a/hw/xen_nic.c b/hw/xen_nic.c index bcf161c..08055b8 100644 --- a/hw/xen_nic.c +++ b/hw/xen_nic.c @@ -41,6 +41,7

[Qemu-devel] [PATCH 12/44] net: convert tap-win32 to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap-win32.c | 39 +++ 1 files changed, 23 insertions(+), 16 deletions(-) diff --git a/net/tap-win32.c b/net/tap-win32.c index ea66471..ef63782 100644 --- a/net/tap-win32.c +++ b/net/tap-win32.c

[Qemu-devel] [PATCH 21/44] net: convert ne2000 to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/ne2000-isa.c | 22 ++ hw/ne2000.c | 35 +-- hw/ne2000.h |2 +- 3 files changed, 36 insertions(+), 23 deletions(-) diff --git a/hw/ne2000-isa.c b/hw/ne2000-isa.c index

[Qemu-devel] [PATCH 07/44] net: remove NICInfo::private

2009-11-25 Thread Mark McLoughlin
Was used by hot-unplug, but not anymore. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/net.h b/net.h index bc8392b..9ffcfe3 100644 --- a/net.h +++ b/net.h @@ -124,7 +124,6 @@ struct NICInfo { char *devaddr

[Qemu-devel] [PATCH 13/44] net: convert slirp to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/slirp.c | 61 +++--- 1 files changed, 37 insertions(+), 24 deletions(-) diff --git a/net/slirp.c b/net/slirp.c index d6d5772..3f91c4b 100644 --- a/net/slirp.c +++ b/net/slirp.c @@ -64,8

[Qemu-devel] [PATCH 39/44] net: fix qemu_announce_self()

2009-11-25 Thread Mark McLoughlin
Now that we have a way to iterate NICs. Signed-off-by: Mark McLoughlin mar...@redhat.com --- savevm.c | 26 +- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/savevm.c b/savevm.c index 4668843..3cfc5aa 100644 --- a/savevm.c +++ b/savevm.c @@ -123,24

[Qemu-devel] [PATCH 41/44] net: fix TAP networking on host kernels without IFF_VNET_HDR support

2009-11-25 Thread Mark McLoughlin
From: Pierre Riteau pierre.rit...@irisa.fr vnet_hdr is initialized at 1 by default. We need to reset it to 0 if the kernel doesn't support IFF_VNET_HDR. Signed-off-by: Pierre Riteau pierre.rit...@irisa.fr Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap-linux.c |2 ++ 1 files

[Qemu-devel] [PATCH 14/44] net: convert vde to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/vde.c | 45 - 1 files changed, 28 insertions(+), 17 deletions(-) diff --git a/net/vde.c b/net/vde.c index 4d09967..42b4633 100644 --- a/net/vde.c +++ b/net/vde.c @@ -34,7 +34,7 @@ #include

[Qemu-devel] [PATCH 33/44] net: convert xilinx_ethlite to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/xilinx_ethlite.c | 35 --- 1 files changed, 20 insertions(+), 15 deletions(-) diff --git a/hw/xilinx_ethlite.c b/hw/xilinx_ethlite.c index b7129d5..37e33ec 100644 --- a/hw/xilinx_ethlite.c +++ b/hw

[Qemu-devel] [PATCH 36/44] net: remove qemu_new_vlan_client()

2009-11-25 Thread Mark McLoughlin
... and VLANClientState::opaque and qemu_find_vlan_client(). All of these are now unused Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 44 net.h | 13 - 2 files changed, 0 insertions(+), 57 deletions(-) diff --git

[Qemu-devel] [PATCH 38/44] net: add qemu_foreach_nic()

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 20 net.h |2 ++ 2 files changed, 22 insertions(+), 0 deletions(-) diff --git a/net.c b/net.c index a1ec243..c71699d 100644 --- a/net.c +++ b/net.c @@ -326,6 +326,26 @@ qemu_find_vlan_client_by_name(Monitor

[Qemu-devel] [PATCH 40/44] net: print correct error for '-netdev '

2009-11-25 Thread Mark McLoughlin
Reported-by: Markus Armbruster arm...@redhat.com Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 16 +++- 1 files changed, 11 insertions(+), 5 deletions(-) diff --git a/net.c b/net.c index c71699d..13bdbb2 100644 --- a/net.c +++ b/net.c @@ -1055,12 +1055,18 @@ int

[Qemu-devel] [PATCH 32/44] net: convert usb-net to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/usb-net.c | 57 +++-- 1 files changed, 31 insertions(+), 26 deletions(-) diff --git a/hw/usb-net.c b/hw/usb-net.c index 7b8cc7a..2556e05 100644 --- a/hw/usb-net.c +++ b/hw/usb-net.c

[Qemu-devel] [PATCH 43/44] net: initialize vnet_hdr in net_init_tap()

2009-11-25 Thread Mark McLoughlin
net_tap_init() always sets vnet_hdr using qemu_opt_get_bool(), but initialize it in net_init_tap() just to reduce confusion. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/tap.c b/net/tap.c index

[Qemu-devel] [PATCH 25/44] net: convert etrax to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/etraxfs_eth.c | 45 +++-- 1 files changed, 27 insertions(+), 18 deletions(-) diff --git a/hw/etraxfs_eth.c b/hw/etraxfs_eth.c index 2cc2332..dedd107 100644 --- a/hw/etraxfs_eth.c +++ b/hw

[Qemu-devel] [PATCH 42/44] net: check for TUNSETOFFLOAD support before trying to enable offload features

2009-11-25 Thread Mark McLoughlin
From: Pierre Riteau pierre.rit...@irisa.fr This avoids the TUNSETOFFLOAD ioctl() failed: Invalid argument message on kernels without TUNSETOFFLOAD support. Signed-off-by: Pierre Riteau pierre.rit...@irisa.fr Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap-linux.c |5 + 1

[Qemu-devel] [PATCH 27/44] net: convert mcf_fec to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/mcf_fec.c | 58 +- 1 files changed, 33 insertions(+), 25 deletions(-) diff --git a/hw/mcf_fec.c b/hw/mcf_fec.c index 8242c8a..4e7fbed 100644 --- a/hw/mcf_fec.c +++ b/hw/mcf_fec.c

[Qemu-devel] [PATCH 37/44] net: remove VLANClientState members now in NetClientInfo

2009-11-25 Thread Mark McLoughlin
Add a NetClientInfo pointer to VLANClientState and use that for the typecode and function pointers. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c|4 ++-- hw/virtio-net.c |2 +- net.c | 45 - net.h

[Qemu-devel] [PATCH 23/44] net: convert eepro100 to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/eepro100.c | 43 --- 1 files changed, 24 insertions(+), 19 deletions(-) diff --git a/hw/eepro100.c b/hw/eepro100.c index 8734907..eb16a51 100644 --- a/hw/eepro100.c +++ b/hw/eepro100.c @@ -195,7

[Qemu-devel] [PATCH 30/44] net: convert smc91c111 to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/smc91c111.c | 35 --- 1 files changed, 20 insertions(+), 15 deletions(-) diff --git a/hw/smc91c111.c b/hw/smc91c111.c index b7398c9..c1a88c9 100644 --- a/hw/smc91c111.c +++ b/hw/smc91c111.c @@ -18,7 +18,7

[Qemu-devel] [PATCH 19/44] net: convert e1000 to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/e1000.c | 47 ++- 1 files changed, 26 insertions(+), 21 deletions(-) diff --git a/hw/e1000.c b/hw/e1000.c index 00f6a57..683fdcc 100644 --- a/hw/e1000.c +++ b/hw/e1000.c @@ -75,7 +75,7 @@ enum

[Qemu-devel] [PATCH 20/44] net: convert rtl8139 to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/rtl8139.c | 43 --- 1 files changed, 24 insertions(+), 19 deletions(-) diff --git a/hw/rtl8139.c b/hw/rtl8139.c index c166db0..931cde9 100644 --- a/hw/rtl8139.c +++ b/hw/rtl8139.c @@ -459,7 +459,7

[Qemu-devel] [PATCH 15/44] net: convert socket to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/socket.c | 74 +++-- 1 files changed, 45 insertions(+), 29 deletions(-) diff --git a/net/socket.c b/net/socket.c index 551fc2b..7331d87 100644 --- a/net/socket.c +++ b/net/socket.c

[Qemu-devel] [PATCH 18/44] net: convert virtio to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/virtio-net.c | 73 +- 1 files changed, 39 insertions(+), 34 deletions(-) diff --git a/hw/virtio-net.c b/hw/virtio-net.c index 2f147e5..4f8d89e 100644 --- a/hw/virtio-net.c +++ b/hw

[Qemu-devel] [PATCH 08/44] net: introduce NetClientInfo

2009-11-25 Thread Mark McLoughlin
This structure holds data which is common to all instances of a given net client type/model. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.h | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/net.h b/net.h index 9ffcfe3..56c5849 100644 --- a/net.h

[Qemu-devel] [PATCH 24/44] net: convert dp8393x to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c | 41 + 1 files changed, 25 insertions(+), 16 deletions(-) diff --git a/hw/dp8393x.c b/hw/dp8393x.c index 5143cc8..be9714d 100644 --- a/hw/dp8393x.c +++ b/hw/dp8393x.c @@ -154,7 +154,8

[Qemu-devel] [PATCH 16/44] net: convert dump to NetClientInfo

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/dump.c | 47 +++ 1 files changed, 27 insertions(+), 20 deletions(-) diff --git a/net/dump.c b/net/dump.c index 05a102b..4ed3f5f 100644 --- a/net/dump.c +++ b/net/dump.c @@ -28,7 +28,7

[Qemu-devel] [PATCH 29/44] net: convert musicpal to NICState

2009-11-25 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/musicpal.c | 31 ++- 1 files changed, 18 insertions(+), 13 deletions(-) diff --git a/hw/musicpal.c b/hw/musicpal.c index 264669f..4a33e28 100644 --- a/hw/musicpal.c +++ b/hw/musicpal.c @@ -151,7 +151,7

[Qemu-devel] [PATCH 44/44] net: fix vnet_hdr handling in solaris tap code

2009-11-25 Thread Mark McLoughlin
Print an error if the user specifies vnet_hdr=1 on the cmdline. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap-solaris.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/net/tap-solaris.c b/net/tap-solaris.c index ef4e60c..e14fe36 100644 --- a/net

[Qemu-devel] [PATCH] net: initialize vnet_hdr in net_tap_init()

2009-11-24 Thread Mark McLoughlin
. From: Mark McLoughlin mar...@redhat.com Subject: [PATCH] net: initialize vnet_hdr in net_tap_init() Don't assume that all tap_open() implementations will set it to zero if VNET_HDR support isn't found. Fixes tap networking on host kernels lacking IFF_VNET_HDR support. Reported-by: Pierre Riteau

[Qemu-devel] Re: [PATCH] Fix TAP networking on host kernels without IFF_VNET_HDR support

2009-11-24 Thread Mark McLoughlin
On Tue, 2009-11-24 at 10:06 +0100, Pierre Riteau wrote: vnet_hdr is initialized at 1 by default. We need to reset it to 0 if the kernel doesn't support IFF_VNET_HDR. Signed-off-by: Pierre Riteau pierre.rit...@irisa.fr Thanks Pierre, I see why this is needed now Acked-by: Mark McLoughlin mar

Re: [Qemu-devel] Re: [PATCH] Fix TAP networking on host kernels without IFF_VNET_HDR support

2009-11-24 Thread Mark McLoughlin
On Tue, 2009-11-24 at 12:17 +0100, Pierre Riteau wrote: On 24 nov. 2009, at 11:28, Mark McLoughlin wrote: On Tue, 2009-11-24 at 10:06 +0100, Pierre Riteau wrote: vnet_hdr is initialized at 1 by default. We need to reset it to 0 if the kernel doesn't support IFF_VNET_HDR. Signed-off

[Qemu-devel] Re: [PATCH 5/7] qdev: add qdev_foreach()

2009-11-20 Thread Mark McLoughlin
Hi Gerd, Thanks for looking, you're absolutely right that this approach just isn't right with e.g. -device On Mon, 2009-11-16 at 10:28 +0100, Gerd Hoffmann wrote: On 11/12/09 21:29, Mark McLoughlin wrote: No doubt this is the worst idea ever, but the requirement is simple - some way

[Qemu-devel] [PATCH 5/7] qdev: add qdev_foreach()

2009-11-12 Thread Mark McLoughlin
No doubt this is the worst idea ever, but the requirement is simple - some way to iterate all NICs in the system. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/qdev.c | 20 hw/qdev.h |3 +++ 2 files changed, 23 insertions(+), 0 deletions(-) diff --git a/hw

[Qemu-devel] [PATCH 6/7] qdev: add qdev_prop_get_{macaddr, net_client}()

2009-11-12 Thread Mark McLoughlin
Probably the second worst idea in the world, ever. While iterating over all NICs in the system, we need to be able to query their 'net-client' and 'macaddr' properties. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/qdev-properties.c | 31 --- hw/qdev.h

[Qemu-devel] [PATCH 1/7] net: remove NICInfo::vc

2009-11-12 Thread Mark McLoughlin
Since 1cc33683, this field is not set for most devices, so just remove it and its remaining few uses. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c | 10 +- hw/etraxfs_eth.c | 10 +- hw/mcf_fec.c | 10 +- hw/mipsnet.c | 10

[Qemu-devel] [RFC PATCH 0/7] Fix qemu_announce_self() properly

2009-11-12 Thread Mark McLoughlin
Hi, The recent fix to qemu_announce_self() will segfault if you use -netdev. The real root cause of the regression was the removal of the assignment to NICInfo::vc that used to live in qdev_get_vlan_client(). Now that NICs are qdev-ified, we should just iterate over the qdev tree

[Qemu-devel] [PATCH 7/7] net: fix qemu_announce_self()

2009-11-12 Thread Mark McLoughlin
Now that we have a sane way of iterating over NICs. Signed-off-by: Mark McLoughlin mar...@redhat.com --- savevm.c | 43 ++- 1 files changed, 26 insertions(+), 17 deletions(-) diff --git a/savevm.c b/savevm.c index 039740c..3736588 100644 --- a/savevm.c

[Qemu-devel] [PATCH 2/7] qdev: add net-client property

2009-11-12 Thread Mark McLoughlin
advantage is that we can easily get the client associated with a qdev. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/qdev-properties.c | 25 + hw/qdev.h|5 + net.h|8 +--- 3 files changed, 35 insertions(+), 3 deletions

[Qemu-devel] [PATCH 3/7] net: create the VLANClientState for NICs early

2009-11-12 Thread Mark McLoughlin
Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c| 10 +- hw/e1000.c | 10 +- hw/eepro100.c | 10 +- hw/etraxfs_eth.c| 11 +-- hw/mcf_fec.c| 10 +- hw/mipsnet.c| 15 ++- hw

[Qemu-devel] [PATCH 4/7] net: kill off NICInfo/NICConf::vlan/netdev

2009-11-12 Thread Mark McLoughlin
No valid users left except the announce_self() code which currently crashes when used with -netdev. Note the assertion in the xen code. Need to figure out how to support the -netdev arg with xen. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/integratorcp.c|3 +- hw/mcf5208.c

Re: [Qemu-devel] [PATCH 7/7] net: fix qemu_announce_self()

2009-11-12 Thread Mark McLoughlin
On Thu, 2009-11-12 at 20:29 +, Mark McLoughlin wrote: ... +printf(qemu_announce_self_iter() mac = %p\n, mac); + +len = announce_self_create(buf, mac); + +printf(sending packet from %s\n, client-name); Ooops, some debugging leftovers. removed in my tree. Cheers, Mark.

Re: [Qemu-devel] Problems with bridge Networking

2009-11-11 Thread Mark McLoughlin
On Thu, 2009-11-12 at 03:31 +, Armin Garcia wrote: I have the next problem I configure the tun/tap and all great any error, I have an IP from my dhcp, I can see my virtual machine (winxp) from my other computers, but In my virtual machine I cant connect to internet, I mean, when I

Re: [Qemu-devel] [PATCH 00/16] Add a -netdev option

2009-11-10 Thread Mark McLoughlin
On Tue, 2009-11-10 at 15:44 +, Paul Brook wrote: On Thursday 08 October 2009, Mark McLoughlin wrote: Hi, Here's a series of patches which gets the ball rolling on adding a -netdev option. ... The idea is to de-emphasise the vlan support, and instead make a nic

[Qemu-devel] Re: [PATCH] Fix incoming migration

2009-11-10 Thread Mark McLoughlin
On Tue, 2009-11-10 at 18:03 +0100, Juan Quintela wrote: Mark McLoughlin mar...@redhat.com wrote: On Fri, 2009-11-06 at 15:58 +0100, Juan Quintela wrote: commit b04c4134d6de28c249277de19e523bfbe4aebbd6 broke incoming migration. After talking with Gleb, code was intended to be the way

[Qemu-devel] preadv/pwritev data corruption on 32 bit

2009-11-05 Thread Mark McLoughlin
Hey, Just a heads up - on 32 bit hosts where qemu was using preadv and pwritev, we were seeing data corruption due to file offset being truncated: https://bugzilla.redhat.com/533063 https://bugzilla.redhat.com/526549 The glibc headers fix is posted here:

[Qemu-devel] Re: [PATCH] whitelist host virtio networking features [was Re: qemu-kvm-0.11 regression, crashes on older ...]

2009-11-02 Thread Mark McLoughlin
On Fri, 2009-10-30 at 16:15 -0500, Dustin Kirkland wrote: On Thu, Oct 29, 2009 at 10:34 AM, Dustin Kirkland kirkl...@canonical.com wrote: whitelist host virtio networking features This patch is a followup to 8eca6b1bc770982595db2f7207c65051572436cb, fixing crashes when guests with 2.6.25

Re: [Qemu-devel] [PATCH 5/5] tap: drain queue in tap_send()

2009-10-30 Thread Mark McLoughlin
On Fri, 2009-10-30 at 11:21 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: Okay, let's try re-enabling the drain-entire-queue behaviour, with a difference - before each subsequent packet, use qemu_can_send_packet() to check that we can send it. This is similar to how we check before

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
Hi Dustin, On Wed, 2009-10-28 at 14:22 -0500, Dustin Kirkland wrote: I believe that we have identified a regression in qemu-kvm-0.11.0. Regression versus which previous version of qemu-kvm? The kvm process crashes for older guests with virtio networking, when the guest's incoming network

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 20:00 +0800, Scott Tsai wrote: Excerpts from Mark McLoughlin's message of Thu Oct 29 17:16:43 +0800 2009: Assuming this is something like the virtio-net in 2.6.26, there was no receivable buffers support so (as Scott points out) it must be that we've read a packet from

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 09:11 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: tap_set_offload(csum: 1, tso4: 1, tso6: 1, ecn: 1) being called and get an mtu of 1500 on virbr0 using his birdge.sh script. virtio_net_receive2 was trying to transfer a 1534 byte packet (1524 'size

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 09:39 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: On Thu, 2009-10-29 at 09:11 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: tap_set_offload(csum: 1, tso4: 1, tso6: 1, ecn: 1) being called and get an mtu of 1500 on virbr0 using his

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 09:46 -0500, Dustin Kirkland wrote: On Thu, 2009-10-29 at 09:34 -0500, Dustin Kirkland wrote: In the mean time, Hardy's kernel is in git here: http://kernel.ubuntu.com/git?p=ubuntu/ubuntu-hardy.git;a=summary I'll save you a few clicks...

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 10:01 -0500, Dustin Kirkland wrote: On Thu, 2009-10-29 at 14:48 +, Mark McLoughlin wrote: Ah, it all makes sense now. I was getting confused between HOST_* and GUEST_* this should have been: features |= (1 VIRTIO_NET_F_MAC); features |= (1

Re: [Qemu-devel] qemu-kvm-0.11 regression, crashes on older guests with virtio network

2009-10-29 Thread Mark McLoughlin
On Thu, 2009-10-29 at 10:13 -0500, Dustin Kirkland wrote: On Thu, 2009-10-29 at 15:01 +, Mark McLoughlin wrote: Sorry, should be VIRTIO_NET_F_CSUM ... the rest is correct Brilliant! Works like a champ. I'll send a patch in a subsequent email. Would you add a signed-off

[Qemu-devel] Re: [PATCH 0/5] Fix packet queueing to allow full tap queue drain

2009-10-28 Thread Mark McLoughlin
On Wed, 2009-10-28 at 11:36 +0800, Scott Tsai wrote: Scott, Sven, if you could test the tap-drain-queue branch from my tree, that would be great: http://repo.or.cz/w/qemu/markmc.git Thanks, Mark. Mark, I've tested http://repo.or.cz/w/qemu/markmc.git and it indeed works

[Qemu-devel] [PATCH] virtio-net: fix macaddr config regression

2009-10-28 Thread Mark McLoughlin
that virtio_net_get_config() uses. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/virtio-net.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/hw/virtio-net.c b/hw/virtio-net.c index 93294af..4b09a93 100644 --- a/hw/virtio-net.c +++ b/hw/virtio-net.c @@ -837,6

Handling merge conflicts [was Re: [Qemu-devel] [PATCH 00/19 v2] Add virtio-net/tap support for partial csums and GSO]

2009-10-28 Thread Mark McLoughlin
Hi Anthony, Thanks for merging this stuff ... In the process of merging it all into qemu-kvm, I noticed a couple of problems: 1) bb6e63644 lacked the change to add the type code to qemu_new_vlan_client() so that and the subsequent 14 commits are unbuildable 2)

[Qemu-devel] [PATCH 3/5] net/queue: queue packets even if sender doesn't supply a callback

2009-10-27 Thread Mark McLoughlin
Now that we disable any receiver whose queue is full, we do not require senders to handle a zero return by supplying a sent callback. This is a second step towards allowing can_receive() handlers to return true even if no buffer space is available. Signed-off-by: Mark McLoughlin mar

[Qemu-devel] [PATCH 1/5] tap: disable draining queue in one go

2009-10-27 Thread Mark McLoughlin
-by: Scott Tsai scottt...@gmail.com Tested-by: Sven Rudolph sven_rudo...@drewag.de Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap.c | 33 +++-- 1 files changed, 15 insertions(+), 18 deletions(-) diff --git a/net/tap.c b/net/tap.c index 60354e4..f32226b

[Qemu-devel] [PATCH 0/5] Fix packet queueing to allow full tap queue drain

2009-10-27 Thread Mark McLoughlin
Hey, Scott points out that the current behaviour of draining the entire queue in tap_send() doesn't work with non-virtio NICs because we drop a packet when the NIC queue fills up. This series of patches first disables this behaviour, fixes various aspects of the queueing logic

[Qemu-devel] [PATCH 2/5] net: disable receiving if client returns zero

2009-10-27 Thread Mark McLoughlin
if no buffer space is available. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 49 ++--- net.h |1 + 2 files changed, 39 insertions(+), 11 deletions(-) diff --git a/net.c b/net.c index 661bbc1..9dea615 100644 --- a/net.c +++ b/net.c

[Qemu-devel] [PATCH 5/5] tap: drain queue in tap_send()

2009-10-27 Thread Mark McLoughlin
it. This patch should be a performance improvement since we no longer have to go through the mainloop for each packet. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net/tap.c | 33 ++--- 1 files changed, 18 insertions(+), 15 deletions(-) diff --git a/net/tap.c b

[Qemu-devel] [PATCH 4/5] virtio-net: split the has_buffers() logic from can_receive()

2009-10-27 Thread Mark McLoughlin
just allow receive() to return zero when this condition occurs and have the caller handle queueing the packet. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/virtio-net.c | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/hw/virtio-net.c b/hw/virtio

[Qemu-devel] [PATCH, stable-0.11] net: disable draining tap queue in one go

2009-10-27 Thread Mark McLoughlin
-by: Scott Tsai scottt...@gmail.com Tested-by: Sven Rudolph sven_rudo...@drewag.de Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 18 -- 1 files changed, 8 insertions(+), 10 deletions(-) diff --git a/net.c b/net.c index 3d3829d..7466961 100644 --- a/net.c +++ b/net.c

Re: [Qemu-devel] [PATCH 00/15] Some networking code re-organization

2009-10-23 Thread Mark McLoughlin
On Thu, 2009-10-22 at 15:34 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: Hey, We've been meaning to split net.c up for quite a while now, so here goes with a first cut at. There shouldn't be anything too controversial here, apart from CONFIG_LINUX maybe

[Qemu-devel] Re: [PATCH 00/15] Some networking code re-organization

2009-10-23 Thread Mark McLoughlin
On Thu, 2009-10-22 at 20:53 +0200, Juan Quintela wrote: Mark McLoughlin mar...@redhat.com wrote: Hey, We've been meaning to split net.c up for quite a while now, so here goes with a first cut at. If you have to respin this series, please use git mv foo.c net/foo.c I did

[Qemu-devel] audio segfault in qemu-kvm-0.11.0

2009-10-23 Thread Mark McLoughlin
Hi, Any ideas on this segfault a Fedora 12 user (Gene, cc-ed) is seeing? Thread 1 (Thread 2849): #0 0x7f25fcd10f70 in memset () from /lib64/libc.so.6 No symbol table info available. #1 0x004babc6 in audio_capture_mix_and_clear (samples=-1099358712, rpos=value optimized out,

Re: [Qemu-devel] Re: audio segfault in qemu-kvm-0.11.0

2009-10-23 Thread Mark McLoughlin
On Fri, 2009-10-23 at 17:41 +0400, malc wrote: On Fri, 23 Oct 2009, malc wrote: On Fri, 23 Oct 2009, Mark McLoughlin wrote: Hi, Any ideas on this segfault a Fedora 12 user (Gene, cc-ed) is seeing? [..snip..] Summary: No idea. FWIW, there's no information

Re: [Qemu-devel] net packet storms with multiple NICs

2009-10-23 Thread Mark McLoughlin
On Fri, 2009-10-23 at 20:25 +0400, Michael Tokarev wrote: I've two questions: o what's the intended usage of all-vlan-equal case, when kvm (or qemu) reflects packets from one interface to another? It's what bridge in linux is for, I think. I don't think it's necessarily an intended

Re: [Qemu-devel] [PATCH 00/15] Some networking code re-organization

2009-10-23 Thread Mark McLoughlin
On Fri, 2009-10-23 at 08:44 -0500, Anthony Liguori wrote: Mark McLoughlin wrote: On Thu, 2009-10-22 at 15:34 -0500, Anthony Liguori wrote: This series doesn't build for me. I get dependency errors even after a full rebuild. I'm building from a separate directory fwiw

[Qemu-devel] [PATCH 01/15 v2] net: move net-queue.[ch] under net/

2009-10-23 Thread Mark McLoughlin
[v2: handle building in a separate dir] Signed-off-by: Mark McLoughlin mar...@redhat.com --- Makefile | 10 +++--- configure |2 +- net.h |2 +- net-queue.c = net/queue.c |2 +- net-queue.h = net/queue.h |0 5 files

[Qemu-devel] [PATCH 01/19] net: remove unused includes of if_tun.h and if_tap.h

2009-10-22 Thread Mark McLoughlin
Looks like these are just artifacts of vl.c being split up. Signed-off-by: Mark McLoughlin mar...@redhat.com --- qemu-char.c |6 -- savevm.c|6 -- vl.c|6 -- 3 files changed, 0 insertions(+), 18 deletions(-) diff --git a/qemu-char.c b/qemu-char.c index

[Qemu-devel] [PATCH 07/19] net: add a client type code

2009-10-22 Thread Mark McLoughlin
This is so as to allow APIs which operate on specific client types without having to add a function table entry which is only implemented by a single client type. Signed-off-by: Mark McLoughlin mar...@redhat.com --- hw/dp8393x.c |3 ++- hw/etraxfs_eth.c |3 ++- hw/mcf_fec.c

[Qemu-devel] [PATCH 13/19] net: implement tap support for receive_raw()

2009-10-22 Thread Mark McLoughlin
tap_receive_raw() always prepends a vnet header if IFF_VNET_HDR is enabled. tap_receive() only prepends when the a header is required but the NIC doesn't supply one. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 23 +++ 1 files changed, 19 insertions(+), 4

[Qemu-devel] [PATCH 02/19] net: import linux tap ioctl definitions

2009-10-22 Thread Mark McLoughlin
Making features dependent on the availability of newer versions if_tun.h is going to get seriously clumsy, so let's just import the definitions we need. It's only a small handful. If and when we're comfortable depending on 2.6.30 headers, we can remove this again. Signed-off-by: Mark McLoughlin

[Qemu-devel] [PATCH 10/19] net: add an API for 'raw' packets

2009-10-22 Thread Mark McLoughlin
In the case where a NIC and backend agree on a packet header format, this API allows injecting packets which lack the agreed upon header. We need this for sending our gratuitous ARP. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net-queue.h |1 + net.c | 37

[Qemu-devel] [PATCH 08/19] net: add tap_has_vnet_hdr() and tap_using_vnet_hdr() APIs

2009-10-22 Thread Mark McLoughlin
These lamely named functions allow virtio-net to query whether IFF_VNET_HDR is enabled on a tap interface and inform the tap code that virtio-net will supply packets with a vnet header. Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 39

[Qemu-devel] [PATCH 05/19] net: refactor tap initialization

2009-10-22 Thread Mark McLoughlin
with such things ending up in writeconfig output? Signed-off-by: Mark McLoughlin mar...@redhat.com --- net.c | 97 +++-- 1 files changed, 52 insertions(+), 45 deletions(-) diff --git a/net.c b/net.c index 356a280..fccabdb 100644 --- a/net.c

  1   2   >