Received: by 2002:a05:7412:bbc7:b0:fc:a2b0:25d7 with SMTP id kh7csp2907801rdb; Tue, 6 Feb 2024 01:02:05 -0800 (PST) X-Google-Smtp-Source: AGHT+IEVSU3MM3qZKDeb/WL8/4pxqI6NHV5weADGqYuZOBcNZ9bJQr1G2Uc+lm0yFSXN/SaNbJwp X-Received: by 2002:a05:6a00:80f3:b0:6d9:b5ba:7802 with SMTP id ei51-20020a056a0080f300b006d9b5ba7802mr2023185pfb.26.1707210124885; Tue, 06 Feb 2024 01:02:04 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1707210124; cv=pass; d=google.com; s=arc-20160816; b=xvv9Wm6eY/wionoRtwaCbL1wK+8QaRAoum+zjmdGJ1lYJcOHT1i1BtDUlN4V3Sd7D3 1EwWA3OeGirBJgwjmXcwXR+oTqVCYFHY+chSp/r18Cxo25f4mVUfOHi25AwSQbzdxO1r K8mgb7ACl7cvXF8zm9KcYSFozXydGmIn3FlhAVg+bP89jGTsQW0rMU7X+k8Fee0jU2fk JDCN7RBwoR2DbOHm090wJi4qSvfPbUP2ig5H2BJVkv2jHy3mFPgOMSzo9Z9QFXOeBe3D HNUKlQCFOLWIQ5AGyGEhMZhhZ6yTExnFpEUzbxoebcJQTZ0Ui4B/IOEgS+oTelE86qcp g0/g== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:cc:to:from; bh=LNRKlaOn8e9jKeUiYfnBvG4mGWjnY1TMonZlah2UBfY=; fh=+42UvqdwGgbeAEZCS7EJ1wDEROzlQBBPxwS7pz9jWGg=; b=qOgMqVjOW3RDg6BseVJ+t6JQANIk2gY8+gSnlEjUSUEEEA2PdekDNCtmxJ5n684+Ui tavFVpcwv10Z6mix1UVL7itxjqH5s+woU6syme8URpumVhO4D/4sq2rq7roEpzpgFDvy GoPF3s5tWjiZOxa+JYELbmmvc71hfWiskv85F7aveBVPE5WXwzJxpVm2KXUnPK39gzRK w8YomV5erY0Ulm4uAYbgpj6v06+mXz/KRjmoDOYqflW0Obj0K/TqGSerWlB9ammtzWk5 htSkLKfPeqQ15tO9JWoiXeq7R6tiJrhzIM2yOoC31JmJmZHB+WPdW9+w3pHbFtSMYuas Vz3A==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; arc=pass (i=1 spf=pass spfdomain=huaweicloud.com); spf=pass (google.com: domain of linux-kernel+bounces-54547-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-54547-linux.lists.archive=gmail.com@vger.kernel.org" X-Forwarded-Encrypted: i=1; AJvYcCXk9tQUM1ZUN+Z+YeXKcWzgJSFkgjCJZjuA6CQZAVfWf2tRHp6N1xSYi2d3A3sa13I18E6Faw8S3+P13RnNBnxEy7OYMuORZ5smlMvMMw== Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id m13-20020a056a00080d00b006e05cb48e01si468186pfk.242.2024.02.06.01.02.04 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 06 Feb 2024 01:02:04 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-54547-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; arc=pass (i=1 spf=pass spfdomain=huaweicloud.com); spf=pass (google.com: domain of linux-kernel+bounces-54547-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-54547-linux.lists.archive=gmail.com@vger.kernel.org" Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id E564F28888F for ; Tue, 6 Feb 2024 09:01:16 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 8C4D612E1F5; Tue, 6 Feb 2024 09:00:00 +0000 (UTC) Received: from dggsgout12.his.huawei.com (unknown [45.249.212.56]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 06C7C12D161; Tue, 6 Feb 2024 08:59:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=45.249.212.56 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707209999; cv=none; b=fencTMNi6vaTFb5KFKLmeccDDYpwF4rfBSE52xAOtAWDCcpraeWhp/ouM3XuwIGvz5IRBIcDDLLXlRdiMJ8YJ7gNdrEYQmcNpklo6PCfk1ruAQm52o/tk2tCQuKGtKfOctHD2evWo2pxsQnVyecAGviYLU1d8hdfsKreYtuSsPM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707209999; c=relaxed/simple; bh=0eM9a+BLPyshHZ+cYhbvhP0NmcRRNc1MK5/f4P8xSn8=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=DqjVpUNGb/ev2Fe4kmrG7wiLck1p/5FkTHpKnjUMlqCohy+MIWaCTAiZLNzxWkkgTvK1zgztkaw5x3rYj8nVNqNVyVlKgJuvH1dcoI4nlRS7SSLCxNYmasQSYcxE5Sd9NjpkKQp/9JIyhnY3F7KNNlT/gVTHL90djhq7JwUJAhQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=huaweicloud.com; spf=pass smtp.mailfrom=huaweicloud.com; arc=none smtp.client-ip=45.249.212.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=huaweicloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=huaweicloud.com Received: from mail.maildlp.com (unknown [172.19.163.216]) by dggsgout12.his.huawei.com (SkyGuard) with ESMTP id 4TTcdK3PXJz4f3kj3; Tue, 6 Feb 2024 16:59:49 +0800 (CST) Received: from mail02.huawei.com (unknown [10.116.40.112]) by mail.maildlp.com (Postfix) with ESMTP id DE3A71A0C59; Tue, 6 Feb 2024 16:59:53 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.104.67]) by APP1 (Coremail) with SMTP id cCh0CgCXaBEG9cFldA+QDA--.34552S7; Tue, 06 Feb 2024 16:59:53 +0800 (CST) From: linan666@huaweicloud.com To: song@kernel.org, mariusz.tkaczyk@linux.intel.com, shli@fb.com, neilb@suse.com Cc: linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org, linan666@huaweicloud.com, yukuai3@huawei.com, yi.zhang@huawei.com, houtao1@huawei.com, yangerkun@huawei.com Subject: [PATCH v6 3/9] md: clean up invalid BUG_ON in md_ioctl Date: Tue, 6 Feb 2024 16:55:05 +0800 Message-Id: <20240206085511.2841555-4-linan666@huaweicloud.com> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20240206085511.2841555-1-linan666@huaweicloud.com> References: <20240206085511.2841555-1-linan666@huaweicloud.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CM-TRANSID:cCh0CgCXaBEG9cFldA+QDA--.34552S7 X-Coremail-Antispam: 1UD129KBjvdXoWrKry5WFy5tFykWw43Kr4UJwb_yoW3tFc_ua yrZF9rXryxZFyIkr15tr13ZFy0yFn7Wrn7ZF1a9ry5ZFy7XryrGFZYgws8Xan5WFW7uryq yr4ayrsYyrsrKjkaLaAFLSUrUUUUUb8apTn2vfkv8UJUUUU8Yxn0WfASr-VFAUDa7-sFnT 9fnUUIcSsGvfJTRUUUbBkFF20E14v26rWj6s0DM7CY07I20VC2zVCF04k26cxKx2IYs7xG 6rWj6s0DM7CIcVAFz4kK6r1j6r18M28IrcIa0xkI8VA2jI8067AKxVWUWwA2048vs2IY02 0Ec7CjxVAFwI0_Xr0E3s1l8cAvFVAK0II2c7xJM28CjxkF64kEwVA0rcxSw2x7M28EF7xv wVC0I7IYx2IY67AKxVW7JVWDJwA2z4x0Y4vE2Ix0cI8IcVCY1x0267AKxVW8Jr0_Cr1UM2 8EF7xvwVC2z280aVAFwI0_GcCE3s1l84ACjcxK6I8E87Iv6xkF7I0E14v26rxl6s0DM2vY z4IE04k24VAvwVAKI4IrM2AIxVAIcxkEcVAq07x20xvEncxIr21l5I8CrVACY4xI64kE6c 02F40Ex7xfMcIj6xIIjxv20xvE14v26r1j6r18McIj6I8E87Iv67AKxVWUJVW8JwAm72CE 4IkC6x0Yz7v_Jr0_Gr1lF7xvr2IYc2Ij64vIr41lF7I21c0EjII2zVCS5cI20VAGYxC7M4 IIrI8v6xkF7I0E8cxan2IY04v7M4kE6xkIj40Ew7xC0wCF04k20xvY0x0EwIxGrwCFx2Iq xVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E14v26r1j6r18MI8I3I0E7480Y4vE14v26r 106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIxkGc2Ij64vIr41lIxAIcVC0I7IYx2IY67AK xVWUJVWUCwCI42IY6xIIjxv20xvEc7CjxVAFwI0_Gr0_Cr1lIxAIcVCF04k26cxKx2IYs7 xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r4UMIIF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_ Gr1UYxBIdaVFxhVjvjDU0xZFpf9x0JU9Z23UUUUU= X-CM-SenderInfo: polqt0awwwqx5xdzvxpfor3voofrz/ From: Li Nan 'disk->private_data' is set to mddev in md_alloc() and never set to NULL, and users need to open mddev before submitting ioctl. So mddev must not have been freed during ioctl, and there is no need to check mddev here. Clean up it. Signed-off-by: Li Nan Reviewed-by: Yu Kuai --- drivers/md/md.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/md/md.c b/drivers/md/md.c index ea147348bd24..fb4e80a0aa9a 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -7645,11 +7645,6 @@ static int md_ioctl(struct block_device *bdev, blk_mode_t mode, mddev = bdev->bd_disk->private_data; - if (!mddev) { - BUG(); - goto out; - } - /* Some actions do not requires the mutex */ switch (cmd) { case GET_ARRAY_INFO: -- 2.39.2