[5025] in testers
Re: Got it! (Re: console Sun logins)
daemon@ATHENA.MIT.EDU (Mitchell E Berger)
Tue Jul 17 21:17:59 2001
Message-Id: <200107180117.VAA16821@byte-me.mit.edu>
To: Greg Hudson <ghudson@MIT.EDU>
cc: Mitchell E Berger <mitchb@MIT.EDU>, testers@MIT.EDU, mitchb@MIT.EDU
In-Reply-To: Your message of "Tue, 17 Jul 2001 10:20:23 EDT."
<200107171420.KAA17258@egyptian-gods.MIT.EDU>
Date: Tue, 17 Jul 2001 21:17:56 -0400
From: Mitchell E Berger <mitchb@MIT.EDU>
> > 5) Something more clever that hasn't occurred to me.
>
> Try closing console_tty in console_login(), just like we do in
> shutdown().
>
> Another thing to try is not reopening /dev/console in console_login(),
> but instead passing fd 1 (which should be /dev/console, the way we run
> dm) to login_tty(). I don't know if that ought to matter, though.
Neither of those possibilities had any effect (I'm pretty sure closing
console_tty was one of the first things I attempted the other day, but
I've lost track, so I tried again). Bob and I looked at it for a little
while today, and we also tried closing ALL file descriptors (including 0-2)
in console_login, then reopening /dev/console and doing login_tty on it.
This didn't work either.
We do think the problem is sort of likely to go away if we remove the one
second sleep from panel-wrapper when panel dies abnormally, but understand
that this is to prevent eating the cpu if panel permanently loses. One
proposal I've come up with is this: what if we remove the one second pause
from panel-wrapper and reengineer its catastrophy avoidance to be based on
the number of failures to keep panel running (maybe a three strikes and give
up method similar to the way dm treats starting the console app)? This should
be a fairly simple code change (I can probably do it without much work), unless
there was another reason for the pause.
Other than that, I'm out of ideas on the dm side of things... Bob was too when
he left tonight, but said he was going to ponder it some more tonight. If
we don't want to touch panel-wrapper and Bob doesn't come up with anything for
dm, I'm still in favor of having the logout alias kill panel-wrapper if it's
running.
Any other thoughts?
Mitch
