From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4E938EB64D9 for ; Wed, 14 Jun 2023 07:24:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243485AbjFNHYd (ORCPT ); Wed, 14 Jun 2023 03:24:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S243202AbjFNHYJ (ORCPT ); Wed, 14 Jun 2023 03:24:09 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 869A426A0; Wed, 14 Jun 2023 00:22:40 -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=oa4DOgSZwpr5FlHNC7X1lgn1eoGAQfcjs1SrNJbVpTg=; b=hHT9ZAx1PTiGPKAhnhqP1Wfk0i gl/+ELIbVzYi5aXI1kQPrGZT96YPRBF6A6Q7Js1mZev89Tt+WQpCZIcUJDbkCkNKYX/ty67ZCP5/P xoJqBWhbwkQav79q7n1SdUb6wTcMTdmC9e+2NSFUl99xYNPRAsRaQAGirQTj9uQY3mOibluV2sY1L AIlwwJo/zRDSagW0sI9yDcUMOwZBIJ5Y2aOBzK7ZTio+Klw1348reyBx3oaf2Bw1gr8gOVe9sTVbe byFTWDgVz7bOlLr3OX/XR+jBRRciFXAxJf2eEzKxWJCHrWF6gj4LWBgLTh4iMAQc416FZGai/enI5 xyH6aaMg==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1q9Kpy-00AeEL-0l; Wed, 14 Jun 2023 07:22:38 +0000 Date: Wed, 14 Jun 2023 00:22:38 -0700 From: Christoph Hellwig To: Ayush Jain Cc: sfr@canb.auug.org.au, Linux Kernel Mailing List , Linux Next Mailing List , Wyes Karny , hch@infradead.org, Jens Axboe , linux-block@vger.kernel.org Subject: Re: Kernel null pointer dereference on stopping raid device Message-ID: References: <3c4911c4-d3d7-a93e-5f14-e97384ae4f21@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3c4911c4-d3d7-a93e-5f14-e97384ae4f21@amd.com> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-next@vger.kernel.org Hi Ayush, can you try this patch? diff --git a/drivers/md/md.c b/drivers/md/md.c index ca0de7ddd9434d..828c4e6b9c5013 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -2460,7 +2460,7 @@ static void export_rdev(struct md_rdev *rdev, struct mddev *mddev) if (test_bit(AutoDetected, &rdev->flags)) md_autodetect_dev(rdev->bdev->bd_dev); #endif - blkdev_put(rdev->bdev, mddev->major_version == -2 ? &claim_rdev : rdev); + blkdev_put(rdev->bdev, &claim_rdev); rdev->bdev = NULL; kobject_put(&rdev->kobj); } @@ -3644,7 +3644,7 @@ static struct md_rdev *md_import_device(dev_t newdev, int super_format, int supe goto out_clear_rdev; rdev->bdev = blkdev_get_by_dev(newdev, BLK_OPEN_READ | BLK_OPEN_WRITE, - super_format == -2 ? &claim_rdev : rdev, NULL); + &claim_rdev, NULL); if (IS_ERR(rdev->bdev)) { pr_warn("md: could not open device unknown-block(%u,%u).\n", MAJOR(newdev), MINOR(newdev)); @@ -3681,7 +3681,7 @@ static struct md_rdev *md_import_device(dev_t newdev, int super_format, int supe return rdev; out_blkdev_put: - blkdev_put(rdev->bdev, super_format == -2 ? &claim_rdev : rdev); + blkdev_put(rdev->bdev, &claim_rdev); out_clear_rdev: md_rdev_clear(rdev); out_free_rdev: