Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp1870152rwd; Wed, 17 May 2023 02:31:18 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6S6F3XSlRuoS2xoCC3NR3zsk0cAk2C9koLeFJV9a+nTfewABc9njp2piEeCdO2gg26bopm X-Received: by 2002:a17:90a:aa81:b0:24b:52cb:9a31 with SMTP id l1-20020a17090aaa8100b0024b52cb9a31mr1865845pjq.22.1684315877889; Wed, 17 May 2023 02:31:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1684315877; cv=none; d=google.com; s=arc-20160816; b=KvmhuwQOR8RhwRoElaglmUViA5jxcPGmVoXTNUtKchyUE9uZXnK1G2RswWQjAM8PC8 ivLKhPKqt6V/Tzaol1y+1tvmMP3Yg1Lepljhg6hJ6ydwz2zUe7nVGurmb3Bkn6RFlz/s epIKrfBqTlqYBAkMAWpF1i1BlVsTk4JQ3Jm1FgYe1M7UFVIS397Hb73kdYxsCRxb9r32 4HNFPu0+xMrkoH8wGciN5rpcF82vpOxZIvc7DtYVvSzgTqUjMRIIBcunQj3r9znKPd8U afERctzjmBxnB7b4ZL3XpMmj6FaIFRr5Oj5KjiEJNfAh3RsupS0Nme59wNex7R0XlUnW M9dQ== 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=81YOFZSHjnf1JcYzLoU/VUaJMnEqM1w8O3ew3wo1Z8g=; b=wDccYDYOb5cNLrLDWAi4m2zvULMV+FVYU6XwxOnOzbNcOqoohn1wdd1SFxj5sP/rx5 RWJCykABVR5cq9M/pijCRJxK+UkHmAfnFHJ+umEhq0tNa6xlNApq/m7PvCpmJv7GkRQh Js22C0CMDOUGAmLtMArqf3ddeWorpHTHMBN57nEXWq8d8nB0JwSqdQbrUu/vzbLmyISP Vmr+bBCYgTZcGO7SN3nyFokKd6MgNag6cQdVwhqcION1iauuiLr43RYqXkxJQqrMXkI9 0jiQEn5l1KEBI0F4AKaXuC/df1DgAjmjcqkH4iOsglrD9qkQjuvguZZ6G/33BZXs01fg 3iEw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b=oyDtW7h7; 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 Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id k64-20020a638443000000b0053079e594aasi9781701pgd.498.2023.05.17.02.31.03; Wed, 17 May 2023 02:31:17 -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=@infradead.org header.s=bombadil.20210309 header.b=oyDtW7h7; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229791AbjEQJVp (ORCPT + 99 others); Wed, 17 May 2023 05:21:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33016 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229654AbjEQJVn (ORCPT ); Wed, 17 May 2023 05:21:43 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 290ED4495; Wed, 17 May 2023 02:21:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=81YOFZSHjnf1JcYzLoU/VUaJMnEqM1w8O3ew3wo1Z8g=; b=oyDtW7h7lIi/s2Ib4UPKrov9Ef gcOpoQqjHci1O7fI6eEAKR5Zgt0ePsISRbKOJ2RwOuvGlBa9XB7dWn7TRRTtLwlsj/aHcDkSg1WvE a7QMikgY9SgBya+gjoz9OzJHg3ByNgRCZUtq2Ecf86i7oo1+dvQvsNX7VPRZn7q0wleV0VLkrdL5f JHTVEnnHIdVaqYFkeeP6wB4LFkRb36o6BJUqBAvWe7HNPMZFKqsXwGyIwe4Xh4arG9TGmVLMlkqpS LnwLV6+PVdW+dlMDVylYH/uoxM29gYbBAS+WNMIwc/ZHr0m/iE5yNEkOiyez0f7uaEZ0kJSjrp1eZ wkODJbSw==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1pzDLk-0092Ls-2o; Wed, 17 May 2023 09:21:36 +0000 Date: Wed, 17 May 2023 02:21:36 -0700 From: Christoph Hellwig To: Damien Le Moal Cc: Yu Kuai , Pradeep P V K , axboe@kernel.dk, linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, "yukuai (C)" Subject: Re: [PATCH V1] block: Fix null pointer dereference issue on struct io_cq Message-ID: References: <20230517084434.18932-1-quic_pragalla@quicinc.com> <07b8b870-a464-25a9-c0a6-c123fad05ff5@huaweicloud.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,SPF_HELO_NONE, SPF_NONE,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED 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 Wed, May 17, 2023 at 06:20:19PM +0900, Damien Le Moal wrote: > twice for the same icq. The missing rcu lock in ioc_exit_icqs() already was in > itself a bug, and the missing flag check is another. spinlocks imply a rcu critical section, no need to duplicate it.