anitazha / rpms / ndctl

Forked from rpms/ndctl 2 years ago
Clone

Blame 0009-zero_info_block-skip-seed-devices.patch

Jeff Moyer 2c91dc
From fb13dfb8d84c4f0a749665c8f07179450b199f3e Mon Sep 17 00:00:00 2001
Jeff Moyer 2c91dc
From: Jeff Moyer <jmoyer@redhat.com>
Jeff Moyer 2c91dc
Date: Tue, 9 Feb 2021 16:51:53 -0500
Jeff Moyer 2c91dc
Subject: [PATCH 009/217] zero_info_block: skip seed devices
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
Currently, ndctl destroy-namespace -f all will output errors of the
Jeff Moyer 2c91dc
form:
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
  Error: destroy namespace: namespace0.0 failed to enable for zeroing, continuing
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
for any zero-sized namespace.  That particular namespace looks like this:
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
  {
Jeff Moyer 2c91dc
    "dev":"namespace0.0",
Jeff Moyer 2c91dc
    "mode":"raw",
Jeff Moyer 2c91dc
    "size":0,
Jeff Moyer 2c91dc
    "uuid":"00000000-0000-0000-0000-000000000000",
Jeff Moyer 2c91dc
    "sector_size":512,
Jeff Moyer 2c91dc
    "state":"disabled"
Jeff Moyer 2c91dc
  }
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
This patch skips over namespaces with size=0 when zeroing out info
Jeff Moyer 2c91dc
blocks.
Jeff Moyer 2c91dc
Jeff Moyer 2c91dc
Fixes: 46654c2d60b70 ("ndctl/namespace: Always zero info-blocks")
Jeff Moyer 2c91dc
Reported-by: Zhang Yi <yizhan@redhat.com>
Jeff Moyer 2c91dc
Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
Jeff Moyer 2c91dc
Signed-off-by: Vishal Verma <vishal.l.verma@intel.com>
Jeff Moyer 2c91dc
Link: https://lore.kernel.org/r/x49r1lohpty.fsf@segfault.boston.devel.redhat.com
Jeff Moyer 2c91dc
---
Jeff Moyer 2c91dc
 ndctl/namespace.c | 3 +++
Jeff Moyer 2c91dc
 1 file changed, 3 insertions(+)
Bryan Gurney bc084d
Bryan Gurney bc084d
diff --git a/ndctl/namespace.c b/ndctl/namespace.c
Bryan Gurney bc084d
index 1feb74d..1e8a2cd 100644
Bryan Gurney bc084d
--- a/ndctl/namespace.c
Bryan Gurney bc084d
+++ b/ndctl/namespace.c
Bryan Gurney bc084d
@@ -1052,6 +1052,9 @@ static int zero_info_block(struct ndctl_namespace *ndns)
Bryan Gurney bc084d
 	void *buf = NULL, *read_buf = NULL;
Bryan Gurney bc084d
 	char path[50];
Bryan Gurney bc084d
 
Bryan Gurney bc084d
+	if (ndctl_namespace_get_size(ndns) == 0)
Bryan Gurney bc084d
+		return 1;
Bryan Gurney bc084d
+
Bryan Gurney bc084d
 	ndctl_namespace_set_raw_mode(ndns, 1);
Bryan Gurney bc084d
 	rc = ndctl_namespace_enable(ndns);
Bryan Gurney bc084d
 	if (rc < 0) {
Jeff Moyer 2c91dc
-- 
Jeff Moyer 2c91dc
2.27.0
Jeff Moyer 2c91dc