From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6514 invoked by alias); 16 Sep 2016 01:08:46 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 39355 Received: (qmail 17866 invoked from network); 16 Sep 2016 01:08:46 -0000 X-Qmail-Scanner-Diagnostics: from mail-pa0-f49.google.com by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1. Clear:RC:0(209.85.220.49):SA:0(0.0/5.0):. Processed in 0.138209 secs); 16 Sep 2016 01:08:46 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=0.0 required=5.0 tests=T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.1 X-Envelope-From: schaefer@brasslantern.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | Received-SPF: none (ns1.primenet.com.au: domain at brasslantern.com does not designate permitted sender hosts) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=brasslantern-com.20150623.gappssmtp.com; s=20150623; h=from:message-id:date:in-reply-to:comments:references:to:subject :mime-version; bh=Yvw2dZ8WLgeuEnLm+kJaTUfM8SaXrXrrUoigTpbxZYM=; b=RXJaz9T0axVb2lL2JRNI0MJhMamW6q7JuAh73EWmTETO77Qwwfun+ZHF6rfsu9o0AX uD7j4IfSP9mXqP+nMvD3i5Fdo9W49ZNWJseVam/JdP+8BXJZFL6//MOsG95TGkXBrnnf n/QmZmaFG1zDqHlqPVgku1E5FF5uGo4LIpfp5rxbxHSH0+ps7NM03z6iVs7Wm0tZ9KB+ bXl1gwMolmo1OJ4aKbPvGSs1HuPGnCJUUgRPSqJHcd6OM/qxvBEf1QP79cFt7kzeDwWx T4iZwrvJ9r9iNTmeHk73lkGubaJO4F6h+kBBnCkHCj4DbwRe4MhXWLS5tQ3fhuQn3yYF hD7w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:message-id:date:in-reply-to:comments :references:to:subject:mime-version; bh=Yvw2dZ8WLgeuEnLm+kJaTUfM8SaXrXrrUoigTpbxZYM=; b=RZaer+qj0zkorBlcUmU/4/NfVxUoryNmYOZVFvHmOuCGjbDUfk0J73x2hAmYr0U1rY zkz2Gn8A9eORhMiRREVyTTPKp/HDFgQB/fC6wg75n1ATDjyJcN8YQHjOB5GBP7tX1z4t BoNTmBsPpYEm9/jjlAhniZmpSN8DMh0ibbMipfSqOVgrueo4/GdfDh7ZlyORqBV+zI+b lboIy1srBqx6383e290LnqSK4OGlum4mvxlmpPwegTCWwrYYYJCzcAy2+gb6p0cDfBOp qKJZ2jzBPg9VB85DZ+4SNsToPHdQmoO0sDwfgvDZXS6QdiEydDBFnbVFSd1sFl4RBNqo GO3w== X-Gm-Message-State: AE9vXwNHMOyT4Jtpb12PAm/7yGvEVSWn/XOL22FoAF0Z7gBPb8pVLakSt2zN6OB5iwvmUw== X-Received: by 10.66.144.164 with SMTP id sn4mr19309976pab.0.1473988120898; Thu, 15 Sep 2016 18:08:40 -0700 (PDT) From: Bart Schaefer Message-Id: <160915180855.ZM3928@torch.brasslantern.com> Date: Thu, 15 Sep 2016 18:08:55 -0700 In-Reply-To: <20160915184040.5fc1b9d0@pwslap01u.europe.root.pri> Comments: In reply to Peter Stephenson "Re: Bug related to stdin/always/jobcontrol" (Sep 15, 6:40pm) References: <87r392jgd0.fsf@juno.home.vuxu.org> <20160905164207.4630643b@pwslap01u.europe.root.pri> <20160914183105.69862fa9@pwslap01u.europe.root.pri> <20160914223553.3173c8ca@ntlworld.com> <160914202450.ZM32479@torch.brasslantern.com> <20160915092721.3d669bc9@pwslap01u.europe.root.pri> <20160915113315.1655019c@pwslap01u.europe.root.pri> <20160915184040.5fc1b9d0@pwslap01u.europe.root.pri> X-Mailer: OpenZMail Classic (0.9.2 24April2005) To: zsh-workers@zsh.org Subject: Re: Bug related to stdin/always/jobcontrol MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii On Sep 15, 6:40pm, Peter Stephenson wrote: } Subject: Re: Bug related to stdin/always/jobcontrol } } The race is related to the fact that when the new SUPERJOB takes over } it's actually in the same process group as the SUBJOB it's taking over, } which is different from it's own PID, list_pipe_pid [...] } } However, fixing that up doesn't help --- apparently attaching it to the } TTY after the SUBJOB is finished works OK, sending it SIGCONT seems to } be OK, but then it always gets SIGSTOP Is it definitely getting SIGSTOP and not SIGTSTP ? The operating system will never[*] automatically send a true STOP signal because it's un-catchable. If a STOP is being received, it's probably being sent from the "for (; !nowait;)" loop in execpline(), either at line 1674 [ killpg(jobtab[list_pipe_job].gleader, SIGSTOP); ] or at line 1688 [ kill(getpid(), SIGSTOP); ]. [*] To my knowledge, anyway. } I think the code is enough better that I may submit it and wonder what } to do with the remaining problem later, or let Bart have a go when he } gets some time. That seems reasonable. This isn't any worse than it ever was, so it shouldn't hold up the release.