[lkml]   [2009]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] kdesu broken
    On Mon, 27 Jul 2009 12:40:11 -0700 (PDT)
    Linus Torvalds <> wrote:

    > On Tue, 28 Jul 2009, OGAWA Hirofumi wrote:
    > >
    > > If I read that part of emacs correctly, it seems to be assuming the data
    > > was already sent to master side if the child process was exited.
    > That sounds like a rather obvious assumption.

    > Aren't pty's flushing the data at flush() time? Which should be happening
    > when the child process exits and closes the pty slave.

    When you close the slave the device all the data has been queued to the

    > So at what point do we just admit that the commit that caused all this was
    > a buggy pile of sh*t and just revert it?

    If we could "just revert it" and get a sane tree I'd have asked you to do
    so quite some time ago and readdressed it later.

    We can't "just revert it" or I'd have deferred it for the next release. If
    you revert it you

    - introduce a DoS attack
    - break ppp
    - introduce a pile of other tty races including at least one where the
    right timings should let you jump through null pointers
    - put back all sorts of random obscure hangs caused by all the lock

    and you'll note I pointed out this was a late change I was forced to make
    and really didn't want to in the original commit.

    Nor can we revert several patches because the ppp stuff means going back
    to about 2.6.28 or so for the entire tty layer plus some of the DoS and
    null pointer races go back to 2.2 or 2.0 8(.

    We can use the two line slightly imperfect quickfix which people reported
    does fix their problem and I'm tempted to go with that for 2.6.31 because
    it works for the real world cases that matter.


     \ /
      Last update: 2009-07-27 22:55    [W:0.020 / U:13.548 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site