Hi Julien,

Julien Cristau wrote:
If logical interfaces are used with ifup, they're not correctly
reenabled by 62-ifup.sh, because 55-down-interfaces.sh loses that
information.  The following patch fixed that for me:

OK, so you're saying that logical interfaces are somehow different from other interfaces? Just to help me understand the situation here (I don't completely get it yet), could you tell me what's in /etc/network/run/ifstate on your machine? And is it correct that this patch makes it call ifup on the full string from the /etc/network/run/ifstate file (stored in IFUP_INTERFACES), while calling ifdown on only the part on the left side of the =? Could you explain a bit why it should work like that?

Cheers,
Bart


diff -Nru acpi-support-0.103/debian/changelog 
acpi-support-0.103/debian/changelog
--- acpi-support-0.103/debian/changelog 2007-12-13 20:34:28.000000000 +0100
+++ acpi-support-0.103/debian/changelog 2008-04-08 13:51:17.000000000 +0200
@@ -1,3 +1,11 @@
+acpi-support (0.103-5.1) UNRELEASED; urgency=high
+
+  * Non-maintainer upload.
+  * Keep track of logical interfaces when downing interfaces on suspend, so we
+    can up them correctly on resume.
+
+ -- Julien Cristau <[EMAIL PROTECTED]>  Tue, 08 Apr 2008 13:50:29 +0200
+
 acpi-support (0.103-5) unstable; urgency=low
[ Bart Samwel ]
diff -Nru /tmp/PFuFwIn8Bc/acpi-support-0.103/suspend.d/55-down-interfaces.sh 
/tmp/75Ea1WPLtc/acpi-support-0.103/suspend.d/55-down-interfaces.sh
--- acpi-support-0.103/suspend.d/55-down-interfaces.sh  2007-06-15 
18:59:55.000000000 +0200
+++ acpi-support-0.103/suspend.d/55-down-interfaces.sh  2008-04-08 
13:49:27.000000000 +0200
@@ -9,7 +9,8 @@
 # the only interfaces that we bring up on resume (the other ones are probably
 # managed by other tools, such as NetworkManager, and will be brought up
 # automatically).
-IFUP_INTERFACES="`cat /etc/network/run/ifstate | sed 's/=.*//'`"
+IFUP_INTERFACES="`cat /etc/network/run/ifstate`"
+IFDOWN_INTERFACES="`cat /etc/network/run/ifstate | sed 's/=.*//'`"
must_control_interface()
 {
@@ -24,7 +25,7 @@
# Shut down the interfaces (except lo, which can and should be kept up)
-for x in $IFUP_INTERFACES; do
+for x in $IFDOWN_INTERFACES; do
     if must_control_interface $x ; then
       ifdown $x
     fi

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.24-1-686 (SMP w/2 CPU cores)
Locale: LANG=C, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages acpi-support depends on:
ii  acpi-support-base             0.103-5    scripts for handling base ACPI eve
ii  acpid                         1.0.6-5.1  Utilities for using ACPI power man
ii dmidecode 2.9-1 Dump Desktop Management Interface ii finger 0.17-12 user information lookup program
ii  hdparm                        8.6-1      tune hard disk parameters for high
ii  laptop-detect                 0.13.5     attempt to detect a laptop
ii  libc6                         2.7-10     GNU C Library: Shared libraries
ii  lsb-base                      3.2-7      Linux Standard Base 3.2 init scrip
ii  nvclock                       0.8b3-1    Allows you to overclock your nVidi
ii  powermgmt-base                1.30       Common utils and configs for power
ii  radeontool                    1.5-5      utility to control ATI Radeon back
ii toshset 1.73-1 Access much of the Toshiba laptop ii vbetool 1.0-1.1 run real-mode video BIOS code to a
ii  x11-xserver-utils             7.3+2      X server utilities

acpi-support recommends no packages.

-- no debconf information






--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to