All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Knorr <kraxel@bytesex.org>
To: Steve Schmidtke <steve_schmidtke@hotmail.com>
Cc: User-mode-linux-devel@lists.sourceforge.net
Subject: Re: [uml-devel] [PATCH] uml_router: cleanup
Date: 24 Nov 2004 11:03:08 +0100	[thread overview]
Message-ID: <874qjfftsz.fsf@bytesex.org> (raw)
In-Reply-To: <BAY24-F27D8548BC9B2976EFF8B3883B80@phx.gbl>

"Steve Schmidtke" <steve_schmidtke@hotmail.com> writes:

> Hi,
> 
> While I was playing around with uml_switch, I started thinking that
> the code is a bit messy and really could use a facelift.

BTW: not sure I ever mentioned it here, but some time ago I basically
rewrote the switch daemon as well for several reasons.  Main reason is
that debugging network problems with the original one is very hard.

While browsing the code I figured that it likely is easier to largely
rewrite the stuff rather than trying to fix the existing stuff.
Especially the mac handling is broken by design.

The code is here:
        http://www.suse.de/~kraxel/uml/switch.tar.gz

The README is below.

Enjoy,

  Gerd

==============================[ cut here ]==============================
This is a drop-in replacement for the uml_switch daemon from the
uml-utilities package.  It is very loosely based on the original
uml_switch daemon code.

Improvements:

  * Prints nice help text on '-h'.
  * Redesigned MAC handling, it can deal with multiple MAC addresses
    per switch port correctly.
  * Has outgoing package queues per switch port.
  * Drives the switch ports using non-blocking I/O for both reads
    and writes.
  * Pidfile support.

Drawbacks:

  * supports the v3 protocol only.

On my maybe-TODO list:

  * Reintroduce hash table for MAC lookup (not sure through that this
    would noticable speedup the daemon as the number of mac addresses
    usually is small ...).
  * use poll() instead of select() (again this likely doesn't make a
    noticable difference for most users ...).

Have fun,

  Gerd

--
Gerd Knorr <kraxel@bytesex.org> [SUSE Labs]


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

  reply	other threads:[~2004-11-24 10:20 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-24  2:47 [uml-devel] [PATCH] uml_router: cleanup Steve Schmidtke
2004-11-24 10:03 ` Gerd Knorr [this message]
     [not found]   ` <BAY24-F33146F620CDD96B205B7283B80@phx.gbl>
2004-11-24 19:27     ` Gerd Knorr
2004-11-24 20:26   ` Felix Müri
2004-11-24 21:10     ` Gerd Knorr
2004-11-24 22:02     ` Steve Schmidtke

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=874qjfftsz.fsf@bytesex.org \
    --to=kraxel@bytesex.org \
    --cc=User-mode-linux-devel@lists.sourceforge.net \
    --cc=steve_schmidtke@hotmail.com \
    /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.