Received: by 2002:a05:6a10:2726:0:0:0:0 with SMTP id ib38csp1065607pxb; Fri, 1 Apr 2022 03:46:09 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzJmSgaWdK2MDQ7lOaetrdbng6Tsh1pkLb4QeWm/5zjwX9fmeagfrbctzsTPhOyWBmUPMGJ X-Received: by 2002:aa7:d945:0:b0:418:f041:f9bc with SMTP id l5-20020aa7d945000000b00418f041f9bcmr20350411eds.305.1648809968861; Fri, 01 Apr 2022 03:46:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1648809968; cv=none; d=google.com; s=arc-20160816; b=V4kx8V60OqpgjJbEhGohlHAk7GQmZepp5gHAqUFzfmLl+Y4zUgOZ5w3ARuhjPnu7FH hCN/lBiKsMFoZ5tj6iSXzbt9MJJUGmCSzks72qSCrrPJge8EHjJVmtZkt9acIVO0Xn/Z 2x4XgLYtFBXpTLhZPp+sYori73JzyvoqBdMPrnqLfKE+061/yexz5UVBnQfnSvrOSA+c xOhkm+1+lq019uiMUepZudYEiOxdpOxeGZceTQzCE4N4jEnnTRbN67vrhpX7rc9mFc7l lHOCTEea+gsf6l5xWk7ZnAr8CSH2P+CD3JpSE0WmbaSgqXms/KFk+7l3UjBM7LvNRxC9 P1gw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=dKkMeptSr36K5MwCeZt80OcWRzRb8UQj5WuimYmARbo=; b=cgkpleL3/pg3r35DPbjFLHjpd9N7l3Un/rgs7xlOGjB/L4cJzMJRBhqUHTN98YyweM 9dF/msMAwcabSj6xObdWK7uOV4hILsJ4u7I+TodfnIWKMTiWbz145afqHlfh4Vpj8Uyr wsRCLRGzmz6jQ2rXtqpBekCh4q811yFW2V7UXQyOANDLkGZSW7Z0vz83HqXu9mzDSe1x PBfKOWczxyS9qhTAIhMn5g2EBtxnGOPmseaOoZFbiQIOBjzRdaarb2n5VA1MpzkUY3a1 NQNIxIOl53Un3BfcAwm1Y7zHxc4wFdvReENFVSEKRhy3L4bXuPc6pg7KoTLRZ7kCCC/C Jw4Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=c10QXVAA; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id s17-20020a056402521100b0041907911396si1855809edd.1.2022.04.01.03.45.42; Fri, 01 Apr 2022 03:46:08 -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=@gmail.com header.s=20210112 header.b=c10QXVAA; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234545AbiCaKO5 (ORCPT + 99 others); Thu, 31 Mar 2022 06:14:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44112 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234528AbiCaKOy (ORCPT ); Thu, 31 Mar 2022 06:14:54 -0400 Received: from mail-wm1-x329.google.com (mail-wm1-x329.google.com [IPv6:2a00:1450:4864:20::329]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3C6E81EECC; Thu, 31 Mar 2022 03:13:07 -0700 (PDT) Received: by mail-wm1-x329.google.com with SMTP id p12-20020a05600c430c00b0038cbdf52227so1700011wme.2; Thu, 31 Mar 2022 03:13:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=dKkMeptSr36K5MwCeZt80OcWRzRb8UQj5WuimYmARbo=; b=c10QXVAA0Dxut1I4GVKnE6PpwlNm0nURglIWMhhy1wcvOUIEJmH6LQPMS4+/m9NsM+ O+8PZ2Py09DfMoTm4qXKMZ/tjQhrkBlfiF/ErHDRMUPPnlmZ/eSzuTG1V6v8BG/VyEhd z57tNNUuecaGNY88UTnbsv1gh3F/ZntaLP47gacjslesFqdl6wIwkC5usWzpYrWJAPI8 my1EDtJOn5iNc2D3YdzLjXzjGZxKnOCPrzRIDGC92xb/LA8g+Kuz0zIOvN588l7AlEUD V99MVKVsr3zjmDomQt3p2uMAIN+pXCkggSQPjX/1JXBwj/CUVbHRB44ztrJptjto2MxY HroA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=dKkMeptSr36K5MwCeZt80OcWRzRb8UQj5WuimYmARbo=; b=ckX1hQeMnVfQX6qGf3AHpRyLLh+Ef3O/dP+FuclM2W1Ube60CDMJqWcqwwAQHMFIcQ fBWJmyijpQdOwlEYgBaxXpJViwGd7aRoFJPiPy/Hsn8Iau+PFovgdRHXgBl/bRvwoB3E SjlFfTE+B3/wRxBgyn5bL2q1bLEV4bQbpGpabnITf0f40yuRKybfJlotM/jh4j4FhWA+ 1n9VgEIbIf2s+wNBG9oYYuekfQya8r/axS6iysvgUJUK5qcLH9vvP0kpTBV4sVr0lx3X Bd5pb2F4I5Xg6xL1SzVOM+jd5mOpuMpEmRh8MVkRB++qJzog/znJaFLCiboOKI+9fig7 x2qA== X-Gm-Message-State: AOAM530KrNp0/RzIq5QQojWAgfEMHDhbhF7X47iobqs3hb6uozl8foIt KHEWyILcg6ltFGmQR2EVDo7c2xbyOtE= X-Received: by 2002:a05:600c:6001:b0:38c:6c43:4427 with SMTP id az1-20020a05600c600100b0038c6c434427mr4036846wmb.186.1648721585268; Thu, 31 Mar 2022 03:13:05 -0700 (PDT) Received: from leap.localnet (host-95-249-145-232.retail.telecomitalia.it. [95.249.145.232]) by smtp.gmail.com with ESMTPSA id t9-20020a05600c198900b0038cb8b38f9fsm7163875wmq.21.2022.03.31.03.13.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 31 Mar 2022 03:13:03 -0700 (PDT) From: "Fabio M. De Francesco" To: Christoph Hellwig Cc: Dan Carpenter , Christoph Hellwig , Wenchao Hao , axboe@kernel.dk, jejb@linux.ibm.com, linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org, martin.petersen@oracle.com, syzbot+f08c77040fa163a75a46@syzkaller.appspotmail.com, syzkaller-bugs@googlegroups.com, linfeilong@huawei.com Subject: Re: [PATCH] scsi: sd: call device_del() if device_add_disk() fails Date: Thu, 31 Mar 2022 12:13:01 +0200 Message-ID: <1722561.VLH7GnMWUR@leap> In-Reply-To: References: <20220329154948.10350-1-fmdefrancesco@gmail.com> <10056508.nUPlyArG6x@leap> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,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 On gioved? 31 marzo 2022 11:13:27 CEST Christoph Hellwig wrote: > > .. and then the cleanup patch would need the same logic. But thinking > about it I don't think we actually can do that due to the split > unregistration. So I take my suggestion back. > If I understand correctly, after thinking about it some more, you decided to withdraw your own suggestion. Dan had already approved this patch. Therefore, I'll leave the patch as it is now and wait for someone to place a "Reviewed-by" tag and Maintainers to merge (if, in the meantime, nobody else require further changes). Thanks, Fabio