Gmail Kalender Dokument Reader Nätet mer »
Nyligen besökta grupper | Hjälp | Logga in
Startsida för Google-grupper
Bug#569032: Sends SIGUSR1 to init unconditionally, conflicts with upstart
Det är för många ämnen i denna grupp som visas först. För att visa detta ämne först, så måste inställningen tas bort från något annat ämne.
Det uppstod ett fel när din begäran skulle bearbetas. Försök igen.
flagga
  3 meddelanden - Komprimera alla  -  Översätt allt till Översatt (visa alla ursprungstexter)
Gruppen som du skickar meddelanden till är en Usenet-grupp. Meddelanden som skickas till den här gruppen gör ditt mail synligt för alla på Internet.
Ditt svarsmeddelande har inte skickats.
Ditt meddelande har publicerats
 
Från:
Till:
Kopia:
Uppföljning på:
Lägg till kopia | Lägg till uppföljning | Redigera ämne
Ämne:
Validering:
Av verifieringsskäl ber vi dig att skriva in de bokstäver du ser i bilden nedan eller de siffror som du hör om du klickar på tillgänglighetsikonen. Lyssna och skriv talen du hör
 
Michael Biebl  
Visa profil   Översätt till Översatt (visa ursprungstexten)
 Fler alternativ 9 Feb, 17:10
Nyhetsgrupper: linux.debian.bugs.dist
Från: Michael Biebl <bi...@debian.org>
Datum: Tue, 09 Feb 2010 17:10:02 +0100
Lokalt: Tis 9 Feb 2010 17:10
Ämne: Bug#569032: Sends SIGUSR1 to init unconditionally, conflicts with upstart

Package: initscripts
Version: 2.87dsf-8
Severity: important
File: /etc/init.d/mountall.sh
Tags: patch

Hi,

mountall.sh contains this code:
        #
        # We might have mounted something over /dev, see if
        # /dev/initctl is there.
        #
        if [ ! -p $INITCTL ]
        then
                rm -f $INITCTL
                mknod -m 600 $INITCTL p
        fi
        kill -USR1 1

In upstart, the SIGUSR1 signal is used, to tell init  to reconnect to
the D-Bus system bus. This obviously conflicts with the above.
At the point where mountall.sh is run, dbus is not yet running, leading
to an error message when upstart is used.

Reading through the changelog, the above was apparently added when devfs
was used, so I'm unsure if it is still required today.

That said, attached is a proposed patch, which moves the kill inside the
if clause and additionally checks for /sbin/initctl, a binary that only
exists in the upstart package.

upstart does not require /dev/initctl, so the patch ensures what we
simply skip this step.

Please review and consider it for the next upload.

Cheers,
Michael

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

Kernel: Linux 2.6.33-rc7 (SMP w/1 CPU core)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages initscripts depends on:
ii  coreutils                     8.4-1      GNU core utilities
ii  debianutils                   3.2.2      Miscellaneous utilities specific t
ii  libc6                         2.10.2-5   Embedded GNU C Library: Shared lib
ii  lsb-base                      3.2-23     Linux Standard Base 3.2 init scrip
ii  mount                         2.16.2-0   Tools for mounting and manipulatin
ii  sysv-rc                       2.87dsf-8  System-V-like runlevel change mech
ii  sysvinit-utils                2.87dsf-8  System-V-like utilities

Versions of packages initscripts recommends:
ii  e2fsprogs                     1.41.9-1   ext2/ext3/ext4 file system utiliti
ii  psmisc                        22.10-1    utilities that use the proc file s

initscripts suggests no packages.

-- no debconf information

  initctl-sigusr1.patch
< 1K Hämta

    Vidarebefordra  
Du måste Logga in innan du kan skicka meddelanden.
Om du vill skicka ett meddelande måste du först delta i den här gruppen.
Uppdatera ditt smeknamn på sidan Prenumerationsinställningar innan du skickar.
Du har inte behörighet att skicka meddelanden.
Diskussionsämnet har ändrats till "Bug#569032: Acknowledgement (Sends SIGUSR1 to init unconditionally, conflicts with upstart)" av Michael Biebl
Michael Biebl  
Visa profil   Översätt till Översatt (visa ursprungstexten)
 Fler alternativ 9 Feb, 17:20
Nyhetsgrupper: linux.debian.bugs.dist
Från: Michael Biebl <bi...@debian.org>
Datum: Tue, 09 Feb 2010 17:20:02 +0100
Lokalt: Tis 9 Feb 2010 17:20
Ämne: Bug#569032: Acknowledgement (Sends SIGUSR1 to init unconditionally, conflicts with upstart)

There was a bug in my previous patch (a missing '!'), noticed it after hitting
the send button.

Please find attached an updated patch.

Cheers,
Michael

--
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

  initctl-sigusr1.patch
< 1K Hämta

  signature.asc
< 1K Hämta

    Vidarebefordra  
Du måste Logga in innan du kan skicka meddelanden.
Om du vill skicka ett meddelande måste du först delta i den här gruppen.
Uppdatera ditt smeknamn på sidan Prenumerationsinställningar innan du skickar.
Du har inte behörighet att skicka meddelanden.
Diskussionsämnet har ändrats till "Bug#569032: Sends SIGUSR1 to init unconditionally, conflicts with upstart" av Petter Reinholdtsen
Petter Reinholdtsen  
Visa profil   Översätt till Översatt (visa ursprungstexten)
 Fler alternativ 20 mar, 11:10
Nyhetsgrupper: linux.debian.bugs.dist
Från: Petter Reinholdtsen <p...@hungry.com>
Datum: Sat, 20 Mar 2010 11:10:01 +0100
Lokalt: Lör 20 mar 2010 11:10
Ämne: Bug#569032: Sends SIGUSR1 to init unconditionally, conflicts with upstart
[Michael Biebl]

> There was a bug in my previous patch (a missing '!'), noticed it
> after hitting the send button.

> Please find attached an updated patch.

I see the point of only signaling init when the socket changed, but am
unsure if we want to look for upstart specific binaries in the generic
scripts.  Is there still a bug with upstart if this change is
implemented instead?

Index: debian/src/initscripts/etc/init.d/mountall.sh
===================================================================
--- debian/src/initscripts/etc/init.d/mountall.sh       (revision 1827)
+++ debian/src/initscripts/etc/init.d/mountall.sh       (working copy)
@@ -59,8 +59,8 @@
        then
                rm -f $INITCTL
                mknod -m 600 $INITCTL p
+               kill -USR1 1
        fi
-       kill -USR1 1

        #
        # Execute swapon command again, in case we want to swap to

I am not sure if this code is still needed, but believe we should keep
it until someone can verify that it is obsolete.

Happy hacking,
--
Petter Reinholdtsen

--
To UNSUBSCRIBE, email to debian-bugs-dist-REQU...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org


    Vidarebefordra  
Du måste Logga in innan du kan skicka meddelanden.
Om du vill skicka ett meddelande måste du först delta i den här gruppen.
Uppdatera ditt smeknamn på sidan Prenumerationsinställningar innan du skickar.
Du har inte behörighet att skicka meddelanden.
Slut på meddelanden
« Tillbaka till diskussioner « Nyare ämnen     Äldre ämnen »

Skapa en grupp - Google-grupper - Googles startsida - Användarvillkor - Sekretesspolicy
©2010 Google