From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Dima Kogan Newsgroups: gmane.emacs.bugs Subject: bug#49954: 28.0.50; TRAMP: cannot kill child processes: "Forbidden reentrant call of Tramp" Date: Mon, 09 Aug 2021 21:26:38 -0700 Message-ID: <874kby7wn5.fsf@secretsauce.net> References: <875ywf9ea7.fsf@secretsauce.net> <871r72ssqi.fsf@gmx.de> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="1250"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: mu4e 1.4.15; emacs 28.0.50 Cc: 49954@debbugs.gnu.org To: Michael Albinus Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Aug 10 06:38:27 2021 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mDJX0-00006E-71 for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 10 Aug 2021 06:38:27 +0200 Original-Received: from localhost ([::1]:56998 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mDJWz-0005eO-32 for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 10 Aug 2021 00:38:25 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:53916) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mDJLz-0006QX-59 for bug-gnu-emacs@gnu.org; Tue, 10 Aug 2021 00:27:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:46638) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1mDJLy-0000ql-SB for bug-gnu-emacs@gnu.org; Tue, 10 Aug 2021 00:27:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1mDJLy-0007UE-G7 for bug-gnu-emacs@gnu.org; Tue, 10 Aug 2021 00:27:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Dima Kogan Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 10 Aug 2021 04:27:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 49954 X-GNU-PR-Package: emacs Original-Received: via spool by 49954-submit@debbugs.gnu.org id=B49954.162856961028757 (code B ref 49954); Tue, 10 Aug 2021 04:27:02 +0000 Original-Received: (at 49954) by debbugs.gnu.org; 10 Aug 2021 04:26:50 +0000 Original-Received: from localhost ([127.0.0.1]:58184 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mDJLm-0007Tl-G2 for submit@debbugs.gnu.org; Tue, 10 Aug 2021 00:26:50 -0400 Original-Received: from wout2-smtp.messagingengine.com ([64.147.123.25]:58053) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1mDJLj-0007TV-Ij for 49954@debbugs.gnu.org; Tue, 10 Aug 2021 00:26:49 -0400 Original-Received: from compute6.internal (compute6.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id 5D609320024A; Tue, 10 Aug 2021 00:26:41 -0400 (EDT) Original-Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Tue, 10 Aug 2021 00:26:41 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=secretsauce.net; h=references:from:to:cc:subject:in-reply-to:date:message-id :mime-version:content-type; s=fm1; bh=2LhiVjNekSL/Z9/WePaWCSRAl1 dC8qdaYRGw5Fts4iI=; b=fVZm4R1LNr6qUybeNeRubvzFT8n4U720JzJuN+Fe20 NNYErt7b5V2t4obdss6aOWjoQZqy+3E3w3Ibh9iYNkuVO0t2UWtaexhS2y833YAg enJvfOxP6OBYcXQ7+mgBW1YOYZBUu1UCtusyZ3RaXtt8a3N9zZoVq76dMSoX3h4h qRlvkdnFBtg2rZzP/fW/trraO+kxF7TaT2giftuknB1128quss9JUWFnRc8oMQ0g yZPRbCKctB37kjS1aZkfwtAAV700+ued/TR15EWH8dvRfnBQby1nVfF+LiCMx9cq mOxibalfAYL5sCmmk+ycak/mHhK6ijyRGA+yLF1+sHTQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=2LhiVj NekSL/Z9/WePaWCSRAl1dC8qdaYRGw5Fts4iI=; b=j9DGrd0M9bEo5IK5gT03i9 SeukhlNm8DX2inQHFV9+e8EQkTqu2V9SUo8qer/iho9uMsYA3wknxpTqJR13pMkw ySabIGoR1VCTFOqiA8sIQyz24hHz/jbEK9nJa/CIPfrs+J7R+8yJVDuanCufGGWd ixfrWMUqH9EGxGDd8hD9dcqKISU9UQ0LuKcgLvJ/t4d2pr+9ozt1EOaH1qP++P1e bP3+JsB53IIKGfWRWZ05oQdhu6mpCx1HW5MDEvVYf17GQPynsGVSEgB+ZsciWQH1 WkYJgtop+dihpd/8cDeoiCGVQeH26NjyQuAEp1ASrzDwbMr1Pm+NimO5fOf1A56Q == X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrjeekgdekgecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpehffgfhvffujgffkfggtgesthdtredttdertdenucfhrhhomhepffhimhgrucfm ohhgrghnuceoughimhgrsehsvggtrhgvthhsrghutggvrdhnvghtqeenucggtffrrghtth gvrhhnpeefveefgfettdefvdetgfekvddtffevhfevleelhfejheelgfduffdtieduueek heenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpeguih hmrgesshgvtghrvghtshgruhgtvgdrnhgvth X-ME-Proxy: Original-Received: by mail.messagingengine.com (Postfix) with ESMTPA; Tue, 10 Aug 2021 00:26:40 -0400 (EDT) In-reply-to: <871r72ssqi.fsf@gmx.de> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:211470 Archived-At: Hi Michael. There's no rush to work on this. I'll reply here for whenever you get around to looking at it. I did some debugging, and it appears that the tramp property-caching mechanism is failing. We exit the (with-tramp-locked-connection ...) form, but when we try to enter the next (with-tramp-locked-connection ...) form, it looks locked because (tramp-get-connection-property proc "locked" nil) is evaluating to t. I instrumented (tramp-get-connection-property), and I can see that this t comes from the property cache. I can "fix" the bug by removing the (when (and (not (eq cached tramp-cache-undefined)) ;; If the key is an auxiliary process object, check ;; whether the process is still alive. (not (and (processp key) (not (process-live-p key))))) (setq value cached cache-used t)) form from (tramp-get-connection-property) Can I get the intent of this form? Are you trying to use this form if the process is alive, or if the process is dead? My process is very much alive, so this form is being used. Is this what we want? If it is what we want, then the cached value of t is the problem. I haven't looked into why that's happening yet. Thanks!