Xen devel - Jan 2007 - Fwd: Re: [Xen-users] Random swap of interfaces at domU startup

maybe a developer can explain that...

----------  Forwarded Message  ----------

Subject: Re: [Xen-users] Random swap of interfaces at domU startup
Date: Tuesday 23 January 2007 13:24
From: Daniele Palumbo <daniele@retaggio.net>
To: xen-users@lists.xensource.com

On Thursday 18 January 2007 22:24, Frédéric VANNIÈRE
wrote:
> I don''t think so. The problem comes from xend, it doesn''t
plug the
> vif interface on the right bridge.
>
> I guess the problem comes from dict handling in Python, default dicts
> are not ordered
i have same problem too, and 3.0.4 too (but seems, at first look, that also
3.0.3 got it)
and i agree (almost) with you, but i think that is not something
like "unordered" (you can see correct order in vif0.0, vif0.1).
my opinion (without looking at code) is that there is concurrent thread that
setup bridge, and so who is running faster will be setup as eth0.
> This is OK :
>    (XendDomainInfo:1195) {''bridge'':
''xenbr0'', ''mac'': ''00:26:3E:7B:0B:
> 24'', ''uuid'':
''17ccddf1-17ee-87a6-9534-a8439402f38a'',
''ip'':
> ''192.168.2.145''}
>   (XendDomainInfo:1195) {''bridge'':
''xenbr1'', ''mac'': ''00:26:3E:7B:0B:
> 25'', ''uuid'':
''b96f77b6-b310-7f21-cdfc-a9a598c85790'',
''ip'': ''10.1.0.21''}
>
> This is NOT OK :
>   (XendDomainInfo:1195)  {''bridge'':
''xenbr1'', ''mac'': ''00:26:3E:7B:0B:
> 25'', ''uuid'':
''661e6a7e-be77-28ce-180d-fca88aa826ab'',
''ip'': ''10.1.0.21''}
>   (XendDomainInfo:1195)  {''bridge'':
''xenbr0'', ''mac'': ''00:26:3E:7B:0B:
> 24'', ''uuid'':
''87f327a3-0544-f76a-81f8-6edd5ac7b6f5'',
''ip'':
> ''192.168.2.145''}
just other logs:

config file:
vif = [ ''bridge=xenbr1'',
''bridge=xenbr1.600'', ''bridge=xenbr1.14'' ]

in attachment: xend.log, notable lines:
--
[''device'', [''vif'',
[''bridge'', ''xenbr1'']]], [''
device'', [''vif'', [''bridge'',
''xenbr1.600'']]], [''device'',
[''vif'',
[''bridge'', ''xenbr1.14'']]]
[...]
createDevice: vif : {''bridge'': ''xenbr1''
[...]
createDevice: vif : {''bridge'': ''xenbr1.14'',
[...]
createDevice: vif : {''bridge'': ''xenbr1.600'',
[...]
--
as you can see, eth1 and eth2 are swapped.
but in a precedent boot, eth0 and eth1 were swapped.

- messages -
Jan 23 12:19:13 xen2 logger: /etc/xen/scripts/block: add
XENBUS_PATH=backend/vbd/2/2049
Jan 23 12:19:13 xen2 logger: /etc/xen/scripts/block: add
XENBUS_PATH=backend/vbd/2/2065
Jan 23 12:19:13 xen2 logger: /etc/xen/scripts/vif-bridge: online
XENBUS_PATH=backend/vif/2/0
Jan 23 12:19:13 xen2 logger: /etc/xen/scripts/vif-bridge: online
XENBUS_PATH=backend/vif/2/1
Jan 23 12:19:13 xen2 logger: /etc/xen/scripts/block: add
XENBUS_PATH=backend/vbd/2/2050
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: online
XENBUS_PATH=backend/vif/2/2
Jan 23 12:19:14 xen2 [ 2832.403309] device vif2.1 entered promiscuous mode
Jan 23 12:19:14 xen2 [ 2832.407399] device vif2.0 entered promiscuous mode
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Successful
vif-bridge online for vif2.1, bridge xenbr1.14.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Writing
backend/vif/2/1/hotplug-status connected to xenstore.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Successful
vif-bridge online for vif2.0, bridge xenbr1.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Writing
backend/vif/2/0/hotplug-status connected to xenstore.
Jan 23 12:19:14 xen2 [ 2832.428236] device vif2.2 entered promiscuous mode
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2065/physical-device fd:5 to xenstore.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Successful
vif-bridge online for vif2.2, bridge xenbr1.600.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2065/hotplug-status connected to xenstore.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/vif-bridge: Writing
backend/vif/2/2/hotplug-status connected to xenstore.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2050/physical-device fd:9 to xenstore.
Jan 23 12:19:14 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2050/hotplug-status connected to xenstore.
Jan 23 12:19:15 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2049/physical-device fd:8 to xenstore.
Jan 23 12:19:15 xen2 logger: /etc/xen/scripts/block: Writing
backend/vbd/2/2049/hotplug-status connected to xenstore.

bye
d.

-------------------------------------------------------


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Wed, Jan 24, 2007 at 01:13:06PM +0100, Daniele Palumbo wrote:
> maybe a developer can explain that...
> 
> ----------  Forwarded Message  ----------
> 
> Subject: Re: [Xen-users] Random swap of interfaces at domU startup
> Date: Tuesday 23 January 2007 13:24
> From: Daniele Palumbo <daniele@retaggio.net>
> To: xen-users@lists.xensource.com
> 
> On Thursday 18 January 2007 22:24, Frédéric VANNIÈRE wrote:
> > I don''t think so. The problem comes from xend, it
doesn''t plug the
> > vif interface on the right bridge.
> >
> > I guess the problem comes from dict handling in Python, default dicts
> > are not ordered
I discovered this problem recently, and have a patch against xen-unstable that
will help -- I''m testing it at the moment, and hopefully will be able
to drop
it in today.

Ewan.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Wednesday 24 January 2007 13:22, Ewan Mellor wrote:
> > On Thursday 18 January 2007 22:24, Frédéric VANNIÈRE wrote:
> > > I guess the problem comes from dict handling in Python, default
dicts
> > > are not ordered
>
> I discovered this problem recently, and have a patch against xen-unstable
> that will help -- I''m testing it at the moment, and hopefully will
be able
> to drop it in today.
is the patch already on the web, so i can test it?
do you plan to write a patch also for 3.0.4?
will you please send in -users thread a mail when the patch is up and 
running? :)

bye
d.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

> -----Original Message-----
> From: xen-devel-bounces@lists.xensource.com 
> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of 
> Daniele Palumbo
> Sent: 24 January 2007 15:05
> To: xen-devel@lists.xensource.com
> Subject: Re: [Xen-devel] Fwd: Re: [Xen-users] Random swap of 
> interfaces at domU startup
> 
> On Wednesday 24 January 2007 13:22, Ewan Mellor wrote:
> > > On Thursday 18 January 2007 22:24, Frédéric VANNIÈRE wrote:
> > > > I guess the problem comes from dict handling in Python, 
> default dicts
> > > > are not ordered
> >
> > I discovered this problem recently, and have a patch 
> against xen-unstable
> > that will help -- I''m testing it at the moment, and 
> hopefully will be able
> > to drop it in today.
> 
> is the patch already on the web, so i can test it?
> do you plan to write a patch also for 3.0.4?
> will you please send in -users thread a mail when the patch is up and 
> running? :)
The patch just turned up in the staging tree (which is "before
it''s been tested"), so I expect it to hit the "unstable"
tomorrow. I can''t comment on the plans to include this in 3.0.4 or not,
but since the patch is relatively simple (about 6-8 lines that change) I
don''t see why it would be a hard task to get it into your own 3.0.4
build even if it''s not in the 3.0.4 official tree.

--
Mats
> 
> bye
> d.
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
> 
> 
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Wed, Jan 24, 2007 at 04:04:53PM +0100, Daniele Palumbo wrote:
> On Wednesday 24 January 2007 13:22, Ewan Mellor wrote:
> > > On Thursday 18 January 2007 22:24, Frédéric VANNIÈRE wrote:
> > > > I guess the problem comes from dict handling in Python,
default dicts
> > > > are not ordered
> >
> > I discovered this problem recently, and have a patch against
xen-unstable
> > that will help -- I''m testing it at the moment, and hopefully
will be able
> > to drop it in today.
> 
> is the patch already on the web, so i can test it?
> do you plan to write a patch also for 3.0.4?
> will you please send in -users thread a mail when the patch is up and 
> running? :)
It''s xen-unstable changeset
13425:4cad44a3ad86ae83a573fa915d980a00ea430f32.

I don''t plan to backport this to 3.0.4, but if someone else did,
I''d apply it.

Thanks,

Ewan.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Friday 26 January 2007 14:54, Ewan Mellor wrote:
> It''s xen-unstable changeset
13425:4cad44a3ad86ae83a573fa915d980a00ea430f32.
>
> I don''t plan to backport this to 3.0.4, but if someone else did,
I''d apply
> it.
that is.
i tested it 10 times on create and shutdown, plus 3 reboot, all fine

Signed-off-by: Daniele Palumbo <daniele@retaggio.net>

bye
d.

p.s.: is "Signed-off-by" correct?


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

On Mon, Jan 29, 2007 at 12:43:36PM +0100, Daniele Palumbo wrote:
> On Friday 26 January 2007 14:54, Ewan Mellor wrote:
> > It''s xen-unstable changeset
13425:4cad44a3ad86ae83a573fa915d980a00ea430f32.
> >
> > I don''t plan to backport this to 3.0.4, but if someone else
did, I''d apply
> > it.
> 
> that is.
> i tested it 10 times on create and shutdown, plus 3 reboot, all fine
> 
> Signed-off-by: Daniele Palumbo <daniele@retaggio.net>
> 
> bye
> d.
> 
> p.s.: is "Signed-off-by" correct?
Yes, that''s right.  I''ve applied this patch, thank you.

Ewan.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel