Message ID | 20231214222107.2016042-1-song@kernel.org |
---|---|
Headers |
Return-Path: <linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:3b04:b0:fb:cd0c:d3e with SMTP id c4csp8889036dys; Thu, 14 Dec 2023 14:22:08 -0800 (PST) X-Google-Smtp-Source: AGHT+IHwX85OIJx/dDDjcIvXJMe/JTImj19gEipquKoJDj5fJ4VM2Zj+zB4A1lvbNNWkLB9gjC3z X-Received: by 2002:ac2:592f:0:b0:50b:fce6:19b3 with SMTP id v15-20020ac2592f000000b0050bfce619b3mr4858951lfi.78.1702592528236; Thu, 14 Dec 2023 14:22:08 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1702592528; cv=none; d=google.com; s=arc-20160816; b=dyFUwvA1Ywj6FuBeb3X//rw5ZleyiSRVjawj9gaPU/6M6f0Z6CWwPPBCsL0on8+TaV 0lEXlq/uZ6FSSsHigpiuiXZwZ8k1tyyJ/szJ85Rq4KvY0w4TbtKhj/21iMDh6F9APhAg pFr8Sst2VGrhjckemZLQ93diW26XdZuhA4CfHl/3qbqdyXSJ52XjW8+wkSkImPEouy/p 3VVKjpOEsc1e2lPgQuRW016bTALxSDXRyTdbsV+tWULGAgmyTNhLyC+NZQHmCcUhNlOV mnrhgGKCR94zgs1YrSV+fdpW/HrFRNT3w7cClNJMP8pqtw8OfxD717eReUXBqRYk8kjR fMsA== ARC-Message-Signature: i=1; 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:message-id:date:subject:cc:to :from:dkim-signature; bh=aPiyoQ9HleCeE72FRuHFYir6x6H2NM2KTWgbx1zOoyU=; fh=cTxSEqYxn2VEubmzhlO3RUXfLLRMSI/YgDiNWvwcK18=; b=dNRdmmI94i8Ih0mSchssB5lqa4kYl45ppf/OtuvOdb5C8r382+6mxEe2T3Nrk5PVby jOBCXyYmE/Qp50iJuDJu3n9ANZevK97HFJR6MRpBpx+AzmQMJdXBkMzuEJ6rhvuB9VuG 1scdvkvb53eDn+b9kN0IrfF/cCBsEE+pHNQeMbGS8FIZw8Pe+ZOHVzbZGJmnQoPs0ma+ 7NKNw2k0/s6jf4dRl6wxdVAMMD2a0PV6Fr0CndnSdvy7+x207Ip7ii+qAzxKfhp99Lmd RmJ4RcZeqTJIU4U2HtQA833YLlhrB5MA52XFeka8VMjx7SUg8jssSfSKdTEo8psUzQTi sZyg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="DuCak/0/"; spf=pass (google.com: domain of linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id p8-20020a170906604800b00a1d9014448fsi6845143ejj.218.2023.12.14.14.22.07 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 14 Dec 2023 14:22:08 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b="DuCak/0/"; spf=pass (google.com: domain of linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-182-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=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 am.mirrors.kernel.org (Postfix) with ESMTPS id C663A1F21978 for <ouuuleilei@gmail.com>; Thu, 14 Dec 2023 22:22:07 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 165262C689; Thu, 14 Dec 2023 22:21:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="DuCak/0/" X-Original-To: linux-kernel@vger.kernel.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 8C001671FD; Thu, 14 Dec 2023 22:21:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6B0A0C433C7; Thu, 14 Dec 2023 22:21:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702592506; bh=jO7MRbXXdGDQpTCdLNxdAdtZPGNzYDrxpX/fHJWSCAo=; h=From:To:Cc:Subject:Date:From; b=DuCak/0/K2NvO6yZ5qSoxkf0mhVQR/8CVGvyZeBjYxgH5FdRAKt1WrE4Fy4Z/vuCj gRnczw5S5kcFaZJJa5Irfwc2ooyqNYK8gjXEckLSFC/wkqJKMg6vZHUDd7Q8XblSUL sOAU1O9YM9KrR6fDoCxCyhvXQeG/eqoYQRlaSTOQZ7ShJw6mEhq17LLg6L2usRFtms /H69YD2JQCKpjzq8koI1iyZMMGye2Qv+nYQTic0joGlnkgJqjLZR5Wq4E+lObgvQg8 y5jjhhR1hQsxt4VWAVwd28crbY+a62Cr2FyhTYjIzrnNLK4o3YyNP0xNknUt6PW/On kPSYiJ95AzvHw== From: Song Liu <song@kernel.org> To: linux-raid@vger.kernel.org, linux-kernel@vger.kernel.org, linux-block@vger.kernel.org Cc: Song Liu <song@kernel.org> Subject: [PATCH 0/3] md: Remove deprecated flavors Date: Thu, 14 Dec 2023 14:21:04 -0800 Message-Id: <20231214222107.2016042-1-song@kernel.org> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1785297662938373303 X-GMAIL-MSGID: 1785297662938373303 |
Series |
md: Remove deprecated flavors
|
|
Message
Song Liu
Dec. 14, 2023, 10:21 p.m. UTC
Linear, multipath, and faulty have been marked as deprecated for 2.5 years. Let's remove them. Thanks, Song Song Liu (3): md: Remove deprecated CONFIG_MD_LINEAR md: Remove deprecated CONFIG_MD_MULTIPATH md: Remove deprecated CONFIG_MD_FAULTY drivers/md/Kconfig | 34 --- drivers/md/Makefile | 10 +- drivers/md/md-autodetect.c | 8 +- drivers/md/md-faulty.c | 365 -------------------------- drivers/md/md-linear.c | 318 ---------------------- drivers/md/md-multipath.c | 463 --------------------------------- drivers/md/md.c | 186 ++++++------- include/uapi/linux/raid/md_p.h | 8 +- include/uapi/linux/raid/md_u.h | 11 +- 9 files changed, 91 insertions(+), 1312 deletions(-) delete mode 100644 drivers/md/md-faulty.c delete mode 100644 drivers/md/md-linear.c delete mode 100644 drivers/md/md-multipath.c -- 2.34.1
Comments
On Thu, 14 Dec 2023 14:21:04 -0800 Song Liu <song@kernel.org> wrote: > Linear, multipath, and faulty have been marked as deprecated for 2.5 years. > Let's remove them. > > Thanks, > Song Hi Song, Great idea! Please note that there are mdadm tests for those levels. I can approve it only when mdadm clean-up is merged. Our tests must pass continuously. It is a nice code complexity improvement so let me know if you would like to get my help with mdadm patches. Thanks, Mariusz
Hi Mariusz, On Fri, Dec 15, 2023 at 3:51 AM Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com> wrote: > > On Thu, 14 Dec 2023 14:21:04 -0800 > Song Liu <song@kernel.org> wrote: > > > Linear, multipath, and faulty have been marked as deprecated for 2.5 years. > > Let's remove them. > > > > Thanks, > > Song > > Hi Song, > Great idea! > > Please note that there are mdadm tests for those levels. I can approve it only > when mdadm clean-up is merged. Our tests must pass continuously. Is the continuous test result available publicly? > > It is a nice code complexity improvement so let me know if you would > like to get my help with mdadm patches. On my local tests with mdadm, I need to make changes to the following tests: 00linear... 00names... 00raid0... 00readonly... 02lineargrow... 03r0assem... 04r0update... 04update-metadata... The changes are all straightforward (just remove things related to linear/multipath/faulty). Thanks, Song
On Fri, 15 Dec 2023 07:37:54 -0800 Song Liu <song@kernel.org> wrote: > Hi Mariusz, > > On Fri, Dec 15, 2023 at 3:51 AM Mariusz Tkaczyk > <mariusz.tkaczyk@linux.intel.com> wrote: > > > > On Thu, 14 Dec 2023 14:21:04 -0800 > > Song Liu <song@kernel.org> wrote: > > > > > Linear, multipath, and faulty have been marked as deprecated for 2.5 > > > years. Let's remove them. > > > > > > Thanks, > > > Song > > > > Hi Song, > > Great idea! > > > > Please note that there are mdadm tests for those levels. I can approve it > > only when mdadm clean-up is merged. Our tests must pass continuously. > > Is the continuous test result available publicly? We are working on public CI (Paul owns it). On my side I'm not executing all tests, IMSM only. In this case it is obvious that mdadm tests will stop passing, I don't need results to see that. We should keep both mdadm and md compatible. We are continuously adding new MD regression tests to mdadm (at least Kuai is doing that) so we should also care about removing things. > > > > > It is a nice code complexity improvement so let me know if you would > > like to get my help with mdadm patches. > > On my local tests with mdadm, I need to make changes to the following > tests: > > 00linear... > 00names... > 00raid0... > 00readonly... > 02lineargrow... > 03r0assem... > 04r0update... > 04update-metadata... > > The changes are all straightforward (just remove things related to > linear/multipath/faulty). > Please do not forgot remove dead code from mdadm. For example simple find "multipath" (case insensitive) reefers me to multiple places with special handling for this level. We need to remove it from code and documentation. Can you handle this too? Oh and last one, I can't find update for md man in your changes. Could you please remove those levels from md man? Thanks, Mariusz
Hi Mariusz, On Mon, Dec 18, 2023 at 12:32 AM Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com> wrote: > [...] > > > > > > Please note that there are mdadm tests for those levels. I can approve it > > > only when mdadm clean-up is merged. Our tests must pass continuously. > > > > Is the continuous test result available publicly? > > We are working on public CI (Paul owns it). On my side I'm not executing all > tests, IMSM only. In this case it is obvious that mdadm tests will stop passing, > I don't need results to see that. We should keep both mdadm and md compatible. > We are continuously adding new MD regression tests to mdadm (at least Kuai is > doing that) so we should also care about removing things. > > > > > > > > > It is a nice code complexity improvement so let me know if you would > > > like to get my help with mdadm patches. > > > > On my local tests with mdadm, I need to make changes to the following > > tests: > > > > 00linear... > > 00names... > > 00raid0... > > 00readonly... > > 02lineargrow... > > 03r0assem... > > 04r0update... > > 04update-metadata... > > > > The changes are all straightforward (just remove things related to > > linear/multipath/faulty). > > > > Please do not forgot remove dead code from mdadm. For example simple find > "multipath" (case insensitive) reefers me to multiple places with special > handling for this level. We need to remove it from code and documentation. > Can you handle this too? I think this is a bigger discussion: will mdadm stop supporting these flavors on older kernels? Say, mdadm-5.0+ (or a different number) will not support multipath flavor on older kernels? > > Oh and last one, I can't find update for md man in your changes. Could you > please remove those levels from md man? man side is easier. Once we know which major will have this set (6.8 or later), we can update the man pages with the information. Thanks, Song
On Mon, 18 Dec 2023 08:11:53 -0800 Song Liu <song@kernel.org> wrote: > Hi Mariusz, > > On Mon, Dec 18, 2023 at 12:32 AM Mariusz Tkaczyk > <mariusz.tkaczyk@linux.intel.com> wrote: > > > [...] > > > > > > > > Please note that there are mdadm tests for those levels. I can approve > > > > it only when mdadm clean-up is merged. Our tests must pass > > > > continuously. > > > > > > Is the continuous test result available publicly? > > > > We are working on public CI (Paul owns it). On my side I'm not executing all > > tests, IMSM only. In this case it is obvious that mdadm tests will stop > > passing, I don't need results to see that. We should keep both mdadm and md > > compatible. We are continuously adding new MD regression tests to mdadm (at > > least Kuai is doing that) so we should also care about removing things. > > > > > > > > > > > > > It is a nice code complexity improvement so let me know if you would > > > > like to get my help with mdadm patches. > > > > > > On my local tests with mdadm, I need to make changes to the following > > > tests: > > > > > > 00linear... > > > 00names... > > > 00raid0... > > > 00readonly... > > > 02lineargrow... > > > 03r0assem... > > > 04r0update... > > > 04update-metadata... > > > > > > The changes are all straightforward (just remove things related to > > > linear/multipath/faulty). > > > > > > > Please do not forgot remove dead code from mdadm. For example simple find > > "multipath" (case insensitive) reefers me to multiple places with special > > handling for this level. We need to remove it from code and documentation. > > Can you handle this too? > > I think this is a bigger discussion: will mdadm stop supporting these > flavors on > older kernels? Say, mdadm-5.0+ (or a different number) will not support > multipath flavor on older kernels? > Good point, I forgot that we are keeping backward compatibility with old kernels. Currently 3.10 is the lowest one supported so agree, we need to keep this code. Thanks for clarifying! > > > > Oh and last one, I can't find update for md man in your changes. Could you > > please remove those levels from md man? > > man side is easier. Once we know which major will have this set (6.8 or > later), we can update the man pages with the information. Understood! No further questions, please just remove tests and eventually add warning when creating those levels in mdadm i.e "Linear/Multipath/Faulty are deprecated and will be removed in next releases". Thanks, Mariusz