All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean-Christian de Rivaz <jc@eclis.ch>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] tun/tap networking: patch for existing tun
Date: Sun, 02 Oct 2005 22:23:41 +0200	[thread overview]
Message-ID: <434041CD.9080507@eclis.ch> (raw)
In-Reply-To: <20051002193912.GB13825@jbrown.mylinuxbox.org>

Jim C. Brown a écrit :
> On Sun, Oct 02, 2005 at 01:45:16PM -0500, Anthony Liguori wrote:
> 
>>I don't understand, why is this patch needed?
>>
> 
> 
> It makes qemu easier to use.
> 
> A lot easier to use a persistent tap by doing "qemu -use-already-open-tap tap1"
> instead of hacking around with persistenttapdev.c

I am happy to see your comment, realy :-)

>>It's a pretty simple C program to create a tun device by whatever name 
>>you want and just pass the fd to qemu via -tun-fd.  I think it's 
>>generally better to have the least number of options necessary to make 
>>things easier to understand.
>>
> 
> 
> Like the way vdeq/vdeqemu does it? That works, but is that really the best way
> to handle it?
> 
> vdeq works the way it does because the odds of getting a special "-vde-socket"
> option in qemu were moot. And perhaps so the author of VDE could have control
> over what options vdeq supported. (In the case of vdeq, its a clever hack: both
> tuntap devices and sockets are controlled via fds, so vdeq sends a socket fd
> instead of a tuntap fd and qemu is none the wiser. Hypothetically one could
> even pass a regular file via -tun-fd.)

VDE is a very useful code to complete project like qemu. It requiere 
special code to connect to the vde_switch, but this not a complexe code 
(see how vde_plug make that). Since VDE is higly likly used with qemu, I 
see a good thing that qemu have ditrectly an -vde option and a -tun 
option. This will corver a large part of real use and still be dead 
simple to understand for the user.

> Having an option for specifying tuntap devices by name on the command line
> (persistent or not) is the cleanest way to do it, and also the easiest for the
> user. Maybe even make it so we just pass an option "-tap tap0": if tap0 doesnt
> exist then qemu creates a new device with that name, if it does exist then
> qemu opens it as if it were a persistent tuntap.

It's already the case with at least my proposed patch. I have't tested 
the patch written by Henrik Nordstrom or Lars Munch but it's likly that 
there work the same way since this feature come from the Linux kernel 
tun code.

> In fact, if qemu supported both these things, then I don't see a reason for
> -tun-fd at all (except for something like VDE).

Agree, and a -vde option will go forward in this direction.

>>Is it really something that so many people would want to use that it 
>>warrants making it an option?  Is there a concrete use-case that this 
>>enables?
> 
> 
> What it really boils down to is cleaning up the command line options for the
> network interface(s), which up to now have been added in a hackish, piece-wise
> manner.

So an open question: is the -tun and -vde options a good idea to cleanup 
the network interface options ? To be clear, I don't propose to remove 
option at this point, but just to make qemu more easy to use for simple 
and most common setup.

-- 
Jean-Christian de Rivaz

  reply	other threads:[~2005-10-02 20:33 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-30 22:13 [Qemu-devel] tun/tap networking Don Kitchen
2005-09-30 22:21 ` Henrik Nordstrom
2005-09-30 22:21 ` Paul Brook
2005-09-30 23:01 ` Jim C. Brown
2005-10-01  8:12   ` Jean-Christian de Rivaz
2005-10-01 13:12     ` Jim C. Brown
2005-10-01 20:24       ` Jean-Christian de Rivaz
2005-10-01 21:09         ` Jim C. Brown
2005-10-01 21:17           ` Jean-Christian de Rivaz
2005-10-01 20:47       ` [Qemu-devel] tun/tap networking: patch for existing tun Jean-Christian de Rivaz
2005-10-02  2:42         ` Henrik Nordstrom
2005-10-02  7:56           ` Jean-Christian de Rivaz
2005-10-02 10:24             ` Henrik Nordstrom
2005-10-02 16:53               ` Lars Munch
2005-10-02 17:50                 ` Jean-Christian de Rivaz
2005-10-02 19:47                   ` Jim C. Brown
2005-10-02 20:27                     ` Jean-Christian de Rivaz
2005-10-02 18:45             ` Anthony Liguori
2005-10-02 19:39               ` Jim C. Brown
2005-10-02 20:23                 ` Jean-Christian de Rivaz [this message]
2005-10-02 22:37                   ` Jim C. Brown
2005-10-03  9:46                     ` Jean-Christian de Rivaz
2005-10-03 12:04                       ` Jim C. Brown
2005-10-03 13:10                         ` Jean-Christian de Rivaz
2005-10-03 13:19                         ` Henrik Nordstrom
2005-10-03 13:13                       ` Henrik Nordstrom
2005-10-03 14:14                         ` Jean-Christian de Rivaz
2005-10-03 13:07                     ` Henrik Nordstrom
2005-10-03 14:00                       ` Jean-Christian de Rivaz
2005-10-03 15:04                       ` Jim C. Brown
2005-10-03 13:01                   ` Henrik Nordstrom
2005-10-03 13:58                     ` Jean-Christian de Rivaz
2005-10-03 15:06                     ` Jim C. Brown
2005-10-03 12:54                 ` Henrik Nordstrom
2005-10-03 15:14                   ` Jim C. Brown
2005-10-03 18:29                     ` Fabrice Bellard
2005-10-03 19:22                       ` Christian MICHON
2005-10-03 20:29                         ` Jean-Christian de Rivaz
2005-10-04  7:09                           ` Christian MICHON
2005-10-04  7:56                             ` Jean-Christian de Rivaz
2005-10-03 21:36                       ` Jim C. Brown
2005-10-04  8:23                       ` Matteo
2005-10-04 11:34                         ` Jim C. Brown
2005-10-01 17:49     ` [Qemu-devel] tun/tap networking Henrik Nordstrom
2005-10-01 20:54       ` Jean-Christian de Rivaz
2005-10-01 11:30   ` Oliver Gerlich
2005-10-01 13:07     ` Jim C. Brown
2005-10-01 13:50       ` Paul Brook
2005-10-01 21:15         ` Jim C. Brown
2005-10-02  2:21           ` Henrik Nordstrom
2005-10-01 17:52     ` Henrik Nordstrom

Reply instructions:

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

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

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

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

  git send-email \
    --in-reply-to=434041CD.9080507@eclis.ch \
    --to=jc@eclis.ch \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

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

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