Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Armstrong
Package: debian-installer-launcher Version: 14 Followup-For: Bug #702335 The attached patch fixes the problem. The old code assumed /live/installer was on a separate mount so that fuser -m would return only a handful of processes, but using the new live-boot, that is not the case, so we have to

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Armstrong
Package: debian-installer-launcher Version: 14 Followup-For: Bug #702335 I have made some small progress in narrowing down where the failure occurs. It appears that in plugins/live, fuser is identifying a much larger list of PIDs than it looks like was intended: live_cleanup () { # Stopping

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Hutchings
On Sun, 2013-03-24 at 15:04 -0300, Ben Armstrong wrote: [...] At this point, I am stuck. How do we make this right so that only PIDs launched by the installer are cleaned up afterwards? Use a cgroup. Ben. -- Ben Hutchings The two most common things in the universe are hydrogen and stupidity.

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Armstrong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/25/2013 10:02 AM, Ben Hutchings wrote: Use a cgroup. Is there a compelling reason for me to switch my approach right now, since I have already solved it another way? If not, I will throw this on the TODO and investigate post-release. I'm

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Hutchings
On Mon, 2013-03-25 at 10:13 -0300, Ben Armstrong wrote: On 03/25/2013 10:02 AM, Ben Hutchings wrote: Use a cgroup. Is there a compelling reason for me to switch my approach right now, since I have already solved it another way? If not, I will throw this on the TODO and investigate

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-25 Thread Ben Armstrong
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/25/2013 10:25 AM, Ben Hutchings wrote: The reason is, the kernel can make this much easier for you! I agree I want things to be easier. :) (That said, I don't know how you're supposed to create a simple cgroup in a shell script. cgexec

Bug#702335: debian-installer-launcher: freezes after exiting

2013-03-05 Thread Ben Armstrong
Package: debian-installer-launcher Version: 14 Severity: normal Steps to reproduce: Boot http://live.debian.net/cdimage/release/next/i386/iso-hybrid/debian-wheezy-live-rc1-i386-lxde-desktop.iso Download and install debian-installer-launcher 14, as this image only contains 13 which does not