[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tap: forbid creating multiqueue tap when hub is
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] tap: forbid creating multiqueue tap when hub is used |
Date: |
Wed, 20 Feb 2013 10:45:25 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 20/02/2013 10:18, Jason Wang ha scritto:
> Obviously, hub does not support multiqueue tap. So this patch forbids creating
> multiple queue tap when hub is used to prevent the crash when command line
> such
> as "-net tap,queues=2" is used.
>
> Signed-off-by: Jason Wang <address@hidden>
> ---
> This patch is needed for 1.4 stable also.
Stefan, please add Cc when committing to the net branch.
Jason, next time please add the Cc yourself to the body.
Paolo
> ---
> net/tap.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/net/tap.c b/net/tap.c
> index 48c254e..1e14f59 100644
> --- a/net/tap.c
> +++ b/net/tap.c
> @@ -693,6 +693,12 @@ int net_init_tap(const NetClientOptions *opts, const
> char *name,
> queues = tap->has_queues ? tap->queues : 1;
> vhostfdname = tap->has_vhostfd ? tap->vhostfd : NULL;
>
> + if (peer && (tap->has_queues || tap->has_fds || tap->has_vhostfds)) {
> + error_report("This configuration is not compatiable with multiqueue"
> + " tap");
> + return -1;
> + }
> +
> if (tap->has_fd) {
> if (tap->has_ifname || tap->has_script || tap->has_downscript ||
> tap->has_vnet_hdr || tap->has_helper || tap->has_queues ||
>