Merge lp:~jamesodhunt/ubuntu/precise/upstart/bug-935585 into lp:ubuntu/precise/upstart

Proposed by James Hunt
Status: Merged
Merged at revision: 1376
Proposed branch: lp:~jamesodhunt/ubuntu/precise/upstart/bug-935585
Merge into: lp:ubuntu/precise/upstart
Diff against target: 45 lines (+18/-2)
2 files modified
debian/changelog (+7/-2)
init/log.c (+11/-0)
To merge this branch: bzr merge lp:~jamesodhunt/ubuntu/precise/upstart/bug-935585
Reviewer Review Type Date Requested Status
Stéphane Graber Pending
Review via email: mp+97937@code.launchpad.net

Description of the change

  [ Steve Langasek ]
  * debian/conf/failsafe.conf: instead of waiting for the 'runlevel' event
    before considering failsafe done, stop this job as soon as we're
    starting rc-sysinit; that way, any delays in /etc/rcS.d will not cause
    confusing messages about networking delays when the network is not the
    problem. LP: #950662.

  [ James Hunt ]
  * init/log.c:log_read_watch(): Set remote_closed for scenarios where error
     handler never called (LP: #935585).

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-03-09 19:43:20 +0000
+++ debian/changelog 2012-03-16 17:35:29 +0000
@@ -1,12 +1,17 @@
1upstart (1.4-0ubuntu9) UNRELEASED; urgency=low1upstart (1.4-0ubuntu9) precise; urgency=low
22
3 [ Steve Langasek ]
3 * debian/conf/failsafe.conf: instead of waiting for the 'runlevel' event4 * debian/conf/failsafe.conf: instead of waiting for the 'runlevel' event
4 before considering failsafe done, stop this job as soon as we're5 before considering failsafe done, stop this job as soon as we're
5 starting rc-sysinit; that way, any delays in /etc/rcS.d will not cause6 starting rc-sysinit; that way, any delays in /etc/rcS.d will not cause
6 confusing messages about networking delays when the network is not the7 confusing messages about networking delays when the network is not the
7 problem. LP: #950662.8 problem. LP: #950662.
89
9 -- Steve Langasek <steve.langasek@ubuntu.com> Fri, 09 Mar 2012 11:41:55 -080010 [ James Hunt ]
11 * init/log.c:log_read_watch(): Set remote_closed for scenarios where error
12 handler never called (LP: #935585).
13
14 -- James Hunt <james.hunt@ubuntu.com> Fri, 16 Mar 2012 17:20:11 +0000
1015
11upstart (1.4-0ubuntu8) precise; urgency=low16upstart (1.4-0ubuntu8) precise; urgency=low
1217
1318
=== modified file 'init/log.c'
--- init/log.c 2012-02-14 16:56:51 +0000
+++ init/log.c 2012-03-16 17:35:29 +0000
@@ -685,6 +685,17 @@
685 break;685 break;
686 }686 }
687 }687 }
688
689 /* Job process has ended and we've drained all the data the job
690 * produced, so remote end must have closed.
691 *
692 * This cannot be handled entirely by log_io_error_handler()
693 * since the job may produce some output prior to disks being
694 * writeable, then end without producing further output.
695 * In this scenario the error handler is never called.
696 *
697 */
698 log->remote_closed = 1;
688}699}
689700
690/**701/**

Subscribers

People subscribed via source and target branches

to all changes: