[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#601235: live-config: 012-xinit script clash with display manager



Package: live-config
Version: 2.0.8-1
Severity: important


When live-snapshot is used, and live-sn.cpio,gz is created, live-config installs the shell script from /lib/live/config/012-xinit on the system.  If any display manager has been spawned during bootup, 012-xinit spawns a runaway script that:
1.  spikes CPU usage (due to spawning of hundreds of instances of startx)
2.  disables the display by alteringspikes CPU usage (due to spawning of hundreds of instances of startx)
2.  disables the display by altering xauth

suggest:
1. removal of 012-xinit as unnecessary
2. changing 012-xinit:

currently:
if [ -z "\${DISPLAY}" ] && [ \$(tty) = /dev/tty1 ]
then
	while true
	do
		startx
	done
fi

suggest changing to:
if [ -s /etc/X11/default-display-manager ]
then
	if [ ! "$(pidof $(cat /etc/X11/default-display-manager)" ]
		then
			startx
	fi
	else
			startx
fi

--end

this eliminates the while loop, preventing thousands of startx processes from being spawned, and adds a check for a currently running display manager to prevent the behavior seen in this bug report


-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (999, 'testing'), (900, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-686 (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=ANSI_X3.4-1968) (ignored: LC_ALL set to C)
Shell: /bin/sh linked to /bin/bash

Versions of packages live-config depends on:
pn  live-config-sysvinit | live-c <none>     (no description available)

Versions of packages live-config recommends:
ii  sudo                          1.7.4p4-2  Provide limited super user privile
ii  user-setup                    1.34       Set up initial user and password

Versions of packages live-config suggests:
ii  wget                          1.12-2.1   retrieves files from the web



Reply to: