dcavalca / rpms / mdadm

Forked from rpms/mdadm 3 years ago
Clone

Blame SOURCES/lib-devid2kname-should-take-a-dev_t.patch

b7f731
From d3c40faba807e3c7a63c5fe34de52bf753c88b2d Mon Sep 17 00:00:00 2001
b7f731
From: Jes Sorensen <jsorensen@fb.com>
b7f731
Date: Fri, 29 Sep 2017 17:54:12 -0400
b7f731
Subject: [RHEL7.5 PATCH 10/13] lib: devid2kname() should take a dev_t
b7f731
b7f731
Make devid2kname() and devid2devnm() consistent in their APIs
b7f731
b7f731
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
b7f731
---
b7f731
 lib.c   | 5 ++---
b7f731
 mdadm.h | 2 +-
b7f731
 2 files changed, 3 insertions(+), 4 deletions(-)
b7f731
b7f731
diff --git a/lib.c b/lib.c
b7f731
index be093e8..60890b9 100644
b7f731
--- a/lib.c
b7f731
+++ b/lib.c
b7f731
@@ -61,7 +61,7 @@ int get_mdp_major(void)
b7f731
 	return mdp_major;
b7f731
 }
b7f731
 
b7f731
-char *devid2kname(int devid)
b7f731
+char *devid2kname(dev_t devid)
b7f731
 {
b7f731
 	char path[30];
b7f731
 	char link[PATH_MAX];
b7f731
@@ -73,8 +73,7 @@ char *devid2kname(int devid)
b7f731
 	 * /sys/dev/block/%d:%d link which must look like
b7f731
 	 * and take the last component.
b7f731
 	 */
b7f731
-	sprintf(path, "/sys/dev/block/%d:%d", major(devid),
b7f731
-		minor(devid));
b7f731
+	sprintf(path, "/sys/dev/block/%d:%d", major(devid), minor(devid));
b7f731
 	n = readlink(path, link, sizeof(link) - 1);
b7f731
 	if (n > 0) {
b7f731
 		link[n] = 0;
b7f731
diff --git a/mdadm.h b/mdadm.h
b7f731
index 191ae8f..db08188 100644
b7f731
--- a/mdadm.h
b7f731
+++ b/mdadm.h
b7f731
@@ -1537,7 +1537,7 @@ extern void print_r10_layout(int layout);
b7f731
 extern char *find_free_devnm(int use_partitions);
b7f731
 
b7f731
 extern void put_md_name(char *name);
b7f731
-extern char *devid2kname(int devid);
b7f731
+extern char *devid2kname(dev_t devid);
b7f731
 extern char *devid2devnm(dev_t devid);
b7f731
 extern dev_t devnm2devid(char *devnm);
b7f731
 extern char *get_md_name(char *devnm);
b7f731
-- 
b7f731
2.7.4
b7f731