All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Dor Laor" <dor.laor-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
To: "Avi Kivity" <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>,
	"Baruch Even" <baruch-6P1Dz+XQpLLYtjvyW6yDsg@public.gmane.org>
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: kvm-ifup bug with complex default routes
Date: Mon, 2 Apr 2007 00:55:08 -0700	[thread overview]
Message-ID: <64F9B87B6B770947A9F8391472E032160B21F029@ehost011-8.exch011.intermedia.net> (raw)
In-Reply-To: <4610B478.3030107-atKUWr5tajBWk0Htik3J/w@public.gmane.org>

>Baruch Even wrote:
>> * Dor Laor <dor.laor-atKUWr5tajBWk0Htik3J/w@public.gmane.org> [070402 00:43]:
>>
>>>> A user just submitted a bug report against the kvm-18 debian
package.
>>>> You can find the bug report at
>>>> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=417151
>>>>
>>>> The user has a default route with extra features and the kvm-ifup
>>>>
>>> script
>>>
>>>> fails for him, his default route looks like:
>>>> default via 10.0.0.138 dev br0  metric 1 realm 10
>>>>
>>>> His suggestion to fix is:
>>>> -switch=$(/sbin/ip route list | awk '/^default / { print $NF }')
>>>> +switch=$(ip route ls | awk '/^default / { for(i=0;i<$NF;i++) { if
>>>>
>>> ($(i) ==
>>>
>>>> "dev") print $(i+1) }}')
>>>>
>>> Actually using the default route in order to get the bridge name is
a
>>> bad thing. The best is to use brctl show and pick the bridge that
has
>>> the interface attached.
>>>
>>
>> The current approach has its problems, especially if there is no
bridge
>> defined. But what if I have multiple bridges? You will need to figure
>> out which one of them to use.
>>
>> But then, this script can't take care of all situations, only of
simple
>> and common ones, the fix suggested above just improves coverage for a
>> few more cases.
>>
>
>I think the correct solution is to set up the bridge where and when
>networking is configured, using the regular configuration files and
>supported by the regular tools.  That's the only way we can deal with
>all the various options.  If there's only one bridge, qemu can use it;
>if there's more, qemu-ifup should read some configuration file to
select
>the bridge.
>
>Fedora has some support for bridging in initscripts: I have an
ifcfg-sw0
>with TYPE=Bridge and ifcfg-eth0 with BRIDGE=sw0.  Seems to work fine,
>and no need to transfer addresses and routes and dhcp client daemons
and
>whatnot.

In general you're right. The only problem is that the ifcfg-XX scripts 
do not work well while having bridge over bond interface.
If someone of our Redhat friends and also at debian will pick up the
glove 
and fix it we will do the same.
Anyway I think qemu guys use such call to external script because there
are
plenty of distributions.

So first the script should use the existing bridge, then the standard
bridge
creation tools can be used.


>
>--
>Do not meddle in the internals of kernels, for they are subtle and
quick to
>panic.


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

      parent reply	other threads:[~2007-04-02  7:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-01 18:28 kvm-ifup bug with complex default routes Baruch Even
     [not found] ` <20070401182810.GT25760-xGn4Jn0woyz+OtfAA3OxFg@public.gmane.org>
2007-04-01 21:30   ` Dor Laor
     [not found]     ` <64F9B87B6B770947A9F8391472E032160B21EF6C-yEcIvxbTEBqsx+V+t5oei8rau4O3wl8o3fe8/T/H7NteoWH0uzbU5w@public.gmane.org>
2007-04-02  6:53       ` Baruch Even
     [not found]         ` <20070402065330.GU25760-xGn4Jn0woyz+OtfAA3OxFg@public.gmane.org>
2007-04-02  7:44           ` Avi Kivity
     [not found]             ` <4610B478.3030107-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-04-02  7:55               ` Dor Laor [this message]

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=64F9B87B6B770947A9F8391472E032160B21F029@ehost011-8.exch011.intermedia.net \
    --to=dor.laor-atkuwr5tajbwk0htik3j/w@public.gmane.org \
    --cc=avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org \
    --cc=baruch-6P1Dz+XQpLLYtjvyW6yDsg@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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.