Received: by 2002:a05:6a10:2726:0:0:0:0 with SMTP id ib38csp1150735pxb; Thu, 24 Mar 2022 13:50:29 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzEXfln/dDD6e2BBLPQZ3Wn7Oqit3Fz3GP1VqxmbxXKc/Tpc3o0Qjamz04H07+6KS+R8cui X-Received: by 2002:a17:90a:3906:b0:1bf:a0a6:d208 with SMTP id y6-20020a17090a390600b001bfa0a6d208mr20566384pjb.21.1648155029441; Thu, 24 Mar 2022 13:50:29 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1648155029; cv=none; d=google.com; s=arc-20160816; b=kZJ+qYTHWGnR1TZbWKVlbsVVtA0NC+o4f2NlVE3dZErXf9rM0y/dMA/OHfOirH1vvH 7HxxAYPOZ3CQJ2PNEQUIDsg9Cvcg+yk/vfrfxi+6qcN3EGWL40R4h4B++QRqGu0zRSCS uOV/cY4A8cShdpnmOMVpaJ+cYyY9uQvrXPlj/+tFFvA081TouVwNJE0dCbxyG/tnz/2s YKWzQjB/pIQdu1Y4nvOuIr597FlBNBoiWVR3B5pdp6yQs/53Sycaz35L6RvR/b6KZaax IkNVo7p6izXD5ZcccjemKhuHolcpuAAtBLpaFAXa21Gs3/jqGq/SfgJTzYatL8RcgO/n b+ow== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=u2slNB9LLomsppL3XA2SkAxa/Nw7H3cTSfU/7PftPIQ=; b=pVKomAdZ0utqEq7+cOeFM/tLA2ZwbwkgrPFwvgvufeb0ebEXYiX3Xz3ONJ1+sXGfNd W2t1/D7LiPVubSj5Qz40+pD+CQHnnNqzVfRKSK/CmILWwYyC+UIlv9ifPxMcjYZMgC/O jZmhdVWAcMftZzd0QxQNP0lvAppGZ+JlH5fX7EM9Mr/PLXsA4xYIjHpcFi9vqiT+OpSc PkK6OIS0iUVqBdVoCWdx1BnRaJNcSidUBkf3Q1aiYVp8qKtk26Y/DU1UaD1QiSUqD2mq yzJAaMekz4myHEQMYI7frN88/OHndM/J0Lbfy23On84eKxEXf8ArMjE2OQAJ+h8N6521 WM8A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=nSkZm+DX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id t12-20020a170902bc4c00b00153b2d165d8si297147plz.480.2022.03.24.13.50.15; Thu, 24 Mar 2022 13:50:29 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=nSkZm+DX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344335AbiCWTmu (ORCPT + 99 others); Wed, 23 Mar 2022 15:42:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41642 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238649AbiCWTms (ORCPT ); Wed, 23 Mar 2022 15:42:48 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6BAB8186E2 for ; Wed, 23 Mar 2022 12:41:18 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 0612861524 for ; Wed, 23 Mar 2022 19:41:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DAF7CC340ED; Wed, 23 Mar 2022 19:41:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1648064477; bh=OyjlJzJfIVp7HiKk4ACH/tuEz4i5DlpA3FsJmafSM2M=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nSkZm+DX9pLULF7Ja/imieggcbTcYtgRSSpaMwEUfIUr6ici/r3BmCzp/CTHRV4Tw 8gkY30+fV5ZSjDEoACoHh3ZluxDtBiDnKriphgNTwHnciH8xf/05G4TpiEj5c0O23x cICmhI1ZhSAy5hRHPaMLvx4oHrqQiTWnb4Ve4Kj/MtQbPrF7ytIyhy4o+Zp6Fgk1+U WrTxdOBCwR07JeLBRF81oB3Ajh5k1+TLZyI8jRO+JduA2IyuDhy92SjdbYBRewODO0 So1KmKQHsB2DgJh/txz763fUOKgMNDhRx1GBqKAIE0DZAC77p+wCVEn1xSGPE929/q iCJ4jKYUAYE3A== Date: Wed, 23 Mar 2022 12:41:10 -0700 From: Nathan Chancellor To: Zhen Lei Cc: Jeff Dike , Richard Weinberger , Anton Ivanov , Andrew Morton , Eduard-Gabriel Munteanu , linux-um , linux-kernel , Nick Desaulniers , regressions@lists.linux.dev Subject: Re: [PATCH 1/1] um: fix error return code in winch_tramp() Message-ID: References: <20210508032239.2177-1-thunder.leizhen@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210508032239.2177-1-thunder.leizhen@huawei.com> X-Spam-Status: No, score=-8.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On Sat, May 08, 2021 at 11:22:39AM +0800, Zhen Lei wrote: > Fix to return a negative error code from the error handling case instead > of 0, as done elsewhere in this function. > > Fixes: 89df6bfc0405 ("uml: DEBUG_SHIRQ fixes") > Reported-by: Hulk Robot > Signed-off-by: Zhen Lei > --- > arch/um/drivers/chan_user.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/um/drivers/chan_user.c b/arch/um/drivers/chan_user.c > index d8845d4aac6a..6040817c036f 100644 > --- a/arch/um/drivers/chan_user.c > +++ b/arch/um/drivers/chan_user.c > @@ -256,7 +256,8 @@ static int winch_tramp(int fd, struct tty_port *port, int *fd_out, > goto out_close; > } > > - if (os_set_fd_block(*fd_out, 0)) { > + err = os_set_fd_block(*fd_out, 0); > + if (err) { > printk(UM_KERN_ERR "winch_tramp: failed to set thread_fd " > "non-blocking.\n"); > goto out_close; > -- > 2.25.1 > > > Sorry for the necro bump but this patch as commit ccf1236ecac4 ("um: fix error return code in winch_tramp()") prevents UML from exiting cleanly when it is called from within in a shell script. It is still reproducible at next-20220323. I did see a patch from Richard that touches this area but that patch does not make a difference: https://lore.kernel.org/r/20220101215810.13260-3-richard@nod.at/ My bisect log: # bad: [7d2a07b769330c34b4deabeed939325c77a7ec2f] Linux 5.14 # good: [62fb9874f5da54fdb243003b386128037319b219] Linux 5.13 git bisect start 'v5.14' 'v5.13' # good: [406254918b232db198ed60f5bf1f8b84d96bca00] Merge tag 'perf-tools-for-v5.14-2021-07-01' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux git bisect good 406254918b232db198ed60f5bf1f8b84d96bca00 # good: [4ea90317956718e0648e1f87e56530db809a5a04] Merge tag 'for-linus-5.14-rc1-tag' of git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip git bisect good 4ea90317956718e0648e1f87e56530db809a5a04 # bad: [65ca89c2b12cca0d473f3dd54267568ad3af55cc] ASoC: intel: atom: Fix breakage for PCM buffer address setup git bisect bad 65ca89c2b12cca0d473f3dd54267568ad3af55cc # bad: [6e207b882159ed3e35a4cd4ff0fc155cce5e3cbc] Merge tag 'arm-soc-5.14' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc git bisect bad 6e207b882159ed3e35a4cd4ff0fc155cce5e3cbc # good: [f55966571d5eb2876a11e48e798b4592fa1ffbb7] Merge tag 'drm-next-2021-07-08-1' of git://anongit.freedesktop.org/drm/drm git bisect good f55966571d5eb2876a11e48e798b4592fa1ffbb7 # bad: [1459718d7d79013a4814275c466e0b32da6a26bc] Merge tag 'powerpc-5.14-2' of git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux git bisect bad 1459718d7d79013a4814275c466e0b32da6a26bc # good: [227c4d507c71acb7bece298a98d83e5b44433f62] Merge tag 'f2fs-for-5.14-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs git bisect good 227c4d507c71acb7bece298a98d83e5b44433f62 # good: [96890bc2eaa1f6bfc1b194e0f0815a10824352a4] Merge tag 'nfs-for-5.14-1' of git://git.linux-nfs.org/projects/trondmy/linux-nfs git bisect good 96890bc2eaa1f6bfc1b194e0f0815a10824352a4 # good: [e49d68ce7cc5a865ce14c1e57938438ab01c3ce3] Merge tag 'ext4_for_linus_stable' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4 git bisect good e49d68ce7cc5a865ce14c1e57938438ab01c3ce3 # bad: [ccf1236ecac476d9d2704866d9a476c86e387971] um: fix error return code in winch_tramp() git bisect bad ccf1236ecac476d9d2704866d9a476c86e387971 # good: [68f5d3f3b6543266b29e047cfaf9842333019b4c] um: add PCI over virtio emulation driver git bisect good 68f5d3f3b6543266b29e047cfaf9842333019b4c # good: [c0ecca6604b80e438b032578634c6e133c7028f6] um: enable the use of optimized xor routines in UML git bisect good c0ecca6604b80e438b032578634c6e133c7028f6 # good: [80f849bf541ef9b633a9c08ac208f9c9afd14eb9] um: implement flush_cache_vmap/flush_cache_vunmap git bisect good 80f849bf541ef9b633a9c08ac208f9c9afd14eb9 # good: [b77e81fbe5f5fb4ad9a61ec80f6d1e30b6da093a] um: fix error return code in slip_open() git bisect good b77e81fbe5f5fb4ad9a61ec80f6d1e30b6da093a # first bad commit: [ccf1236ecac476d9d2704866d9a476c86e387971] um: fix error return code in winch_tramp() $ make -skj"$(nproc)" ARCH=um mrproper defconfig all $ ./linux ubd0=... ... Run /sbin/init as init process EXT4-fs (ubda): re-mounted. Quota mode: none. Starting syslogd: OK Starting klogd: OK Running sysctl: OK Initializing random number generator: OK Saving random seed: OK Starting network: OK Linux version 5.17.0-next-20220323 (nathan@dev-arch.thelio-3990X) (gcc (GCC) 11.2.0, GNU ld (GNU Binutils) 2.38) #1 Wed Mar 23 12:05:22 MST 2022 Stopping network: OK Saving random seed: OK Stopping klogd: OK Stopping syslogd: OK EXT4-fs (ubda): re-mounted. Quota mode: none. The system is going down NOW! Sent SIGTERM to all processes Sent SIGKILL to all processes Requesting system poweroff reboot: System halted $ echo $? 0 $ cat test.sh #!/usr/bin/env bash ./linux ubd0=... $ ./test.sh ... Run /sbin/init as init process EXT4-fs (ubda): re-mounted. Quota mode: none. Starting syslogd: OK Starting klogd: OK Running sysctl: OK Initializing random number generator: OK Saving random seed: OK Starting network: OK Linux version 5.17.0-next-20220323 (nathan@dev-arch.thelio-3990X) (gcc (GCC) 11.2.0, GNU ld (GNU Binutils) 2.38) #1 Wed Mar 23 12:10:31 MST 2022 Stopping network: OK Saving random seed: OK Stopping klogd: OK Stopping syslogd: OK EXT4-fs (ubda): re-mounted. Quota mode: none. The system is going down NOW! Sent SIGTERM to all processes Sent SIGKILL to all processes Requesting system poweroff reboot: System halted ./test.sh: line 5: 970978 Killed ./linux ubd0=... $ echo $? 137 The rootfs is a simple Buildroot image, which just prints the version string then runs "poweroff". It is available at: https://github.com/nathanchance/boot-utils/raw/bd4b962ee12e00f666eef12e3413a79d334a0685/images/x86_64/rootfs.ext4.zst in case it helps. I am happy to provide more information or test patches as necessary. Cheers, Nathan