Commit 2a9400e9 authored by Neil Brown's avatar Neil Brown Committed by Trond Myklebust

[PATCH] MD - Rdev list cleanups.

Rdev list cleanups.

An "rdev" can be on three different lists.
 - the list of all rdevs
 - the list of pending rdevs
 - the list of rdevs for a given mddev

The first list is now only used to list "unused" devices in
/proc/mdstat, and only pending rdevs can be unused, so this list
isn't necessary.
An rdev cannot be both pending and in an mddev, so we know rdev will
only be on one list at at time.

This patch discards  the all_raid_disks list, and changes the
pending list to use "same_set" in the rdev.  It also changes
/proc/mdstat to iterate through pending devices, rather than through
all devices.

So now an rdev is only on one list, either the pending list
or the list of rdevs for a given mddev.  This means that
ITERATE_RDEV_GENERIC doesn't need to be told which field,
to walk down: there is ony one.
parent 70e96bef
...@@ -550,7 +550,6 @@ static int match_mddev_units(mddev_t *mddev1, mddev_t *mddev2) ...@@ -550,7 +550,6 @@ static int match_mddev_units(mddev_t *mddev1, mddev_t *mddev2)
return 0; return 0;
} }
static LIST_HEAD(all_raid_disks);
static LIST_HEAD(pending_raid_disks); static LIST_HEAD(pending_raid_disks);
static void bind_rdev_to_array(mdk_rdev_t * rdev, mddev_t * mddev) static void bind_rdev_to_array(mdk_rdev_t * rdev, mddev_t * mddev)
...@@ -629,14 +628,9 @@ static void export_rdev(mdk_rdev_t * rdev) ...@@ -629,14 +628,9 @@ static void export_rdev(mdk_rdev_t * rdev)
printk(KERN_INFO "md: export_rdev(%s)\n",partition_name(rdev->dev)); printk(KERN_INFO "md: export_rdev(%s)\n",partition_name(rdev->dev));
if (rdev->mddev) if (rdev->mddev)
MD_BUG(); MD_BUG();
unlock_rdev(rdev);
free_disk_sb(rdev); free_disk_sb(rdev);
list_del_init(&rdev->all); list_del_init(&rdev->same_set);
if (!list_empty(&rdev->pending)) { unlock_rdev(rdev);
printk(KERN_INFO "md: (%s was pending)\n",
partition_name(rdev->dev));
list_del_init(&rdev->pending);
}
#ifndef MODULE #ifndef MODULE
md_autodetect_dev(rdev->dev); md_autodetect_dev(rdev->dev);
#endif #endif
...@@ -1044,8 +1038,6 @@ static mdk_rdev_t *md_import_device(kdev_t newdev, int on_disk) ...@@ -1044,8 +1038,6 @@ static mdk_rdev_t *md_import_device(kdev_t newdev, int on_disk)
rdev->desc_nr = -1; rdev->desc_nr = -1;
} }
} }
list_add(&rdev->all, &all_raid_disks);
INIT_LIST_HEAD(&rdev->pending);
INIT_LIST_HEAD(&rdev->same_set); INIT_LIST_HEAD(&rdev->same_set);
if (rdev->faulty && rdev->sb) if (rdev->faulty && rdev->sb)
...@@ -1821,7 +1813,7 @@ static void autorun_array(mddev_t *mddev) ...@@ -1821,7 +1813,7 @@ static void autorun_array(mddev_t *mddev)
/* /*
* lets try to run arrays based on all disks that have arrived * lets try to run arrays based on all disks that have arrived
* until now. (those are in the ->pending list) * until now. (those are in pending_raid_disks)
* *
* the method: pick the first pending disk, collect all disks with * the method: pick the first pending disk, collect all disks with
* the same UUID, remove all from the pending list and put them into * the same UUID, remove all from the pending list and put them into
...@@ -1841,7 +1833,7 @@ static void autorun_devices(void) ...@@ -1841,7 +1833,7 @@ static void autorun_devices(void)
printk(KERN_INFO "md: autorun ...\n"); printk(KERN_INFO "md: autorun ...\n");
while (!list_empty(&pending_raid_disks)) { while (!list_empty(&pending_raid_disks)) {
rdev0 = list_entry(pending_raid_disks.next, rdev0 = list_entry(pending_raid_disks.next,
mdk_rdev_t, pending); mdk_rdev_t, same_set);
printk(KERN_INFO "md: considering %s ...\n", partition_name(rdev0->dev)); printk(KERN_INFO "md: considering %s ...\n", partition_name(rdev0->dev));
INIT_LIST_HEAD(&candidates); INIT_LIST_HEAD(&candidates);
...@@ -1854,8 +1846,7 @@ static void autorun_devices(void) ...@@ -1854,8 +1846,7 @@ static void autorun_devices(void)
continue; continue;
} }
printk(KERN_INFO "md: adding %s ...\n", partition_name(rdev->dev)); printk(KERN_INFO "md: adding %s ...\n", partition_name(rdev->dev));
list_del(&rdev->pending); list_move(&rdev->same_set, &candidates);
list_add(&rdev->pending, &candidates);
} }
} }
/* /*
...@@ -1878,9 +1869,9 @@ static void autorun_devices(void) ...@@ -1878,9 +1869,9 @@ static void autorun_devices(void)
mddev_unlock(mddev); mddev_unlock(mddev);
} else { } else {
printk(KERN_INFO "md: created md%d\n", mdidx(mddev)); printk(KERN_INFO "md: created md%d\n", mdidx(mddev));
ITERATE_RDEV_GENERIC(candidates,pending,rdev,tmp) { ITERATE_RDEV_GENERIC(candidates,rdev,tmp) {
list_del_init(&rdev->same_set);
bind_rdev_to_array(rdev, mddev); bind_rdev_to_array(rdev, mddev);
list_del_init(&rdev->pending);
} }
autorun_array(mddev); autorun_array(mddev);
mddev_unlock(mddev); mddev_unlock(mddev);
...@@ -1888,7 +1879,7 @@ static void autorun_devices(void) ...@@ -1888,7 +1879,7 @@ static void autorun_devices(void)
/* on success, candidates will be empty, on error /* on success, candidates will be empty, on error
* it wont... * it wont...
*/ */
ITERATE_RDEV_GENERIC(candidates,pending,rdev,tmp) ITERATE_RDEV_GENERIC(candidates,rdev,tmp)
export_rdev(rdev); export_rdev(rdev);
mddev_put(mddev); mddev_put(mddev);
} }
...@@ -1944,7 +1935,7 @@ static int autostart_array(kdev_t startdev) ...@@ -1944,7 +1935,7 @@ static int autostart_array(kdev_t startdev)
partition_name(startdev)); partition_name(startdev));
goto abort; goto abort;
} }
list_add(&start_rdev->pending, &pending_raid_disks); list_add(&start_rdev->same_set, &pending_raid_disks);
sb = start_rdev->sb; sb = start_rdev->sb;
...@@ -1973,7 +1964,7 @@ static int autostart_array(kdev_t startdev) ...@@ -1973,7 +1964,7 @@ static int autostart_array(kdev_t startdev)
partition_name(dev)); partition_name(dev));
continue; continue;
} }
list_add(&rdev->pending, &pending_raid_disks); list_add(&rdev->same_set, &pending_raid_disks);
} }
/* /*
...@@ -2897,15 +2888,10 @@ static int status_unused(char * page) ...@@ -2897,15 +2888,10 @@ static int status_unused(char * page)
sz += sprintf(page + sz, "unused devices: "); sz += sprintf(page + sz, "unused devices: ");
ITERATE_RDEV_ALL(rdev,tmp) { ITERATE_RDEV_PENDING(rdev,tmp) {
if (list_empty(&rdev->same_set)) { i++;
/* sz += sprintf(page + sz, "%s ",
* The device is not yet used by any array. partition_name(rdev->dev));
*/
i++;
sz += sprintf(page + sz, "%s ",
partition_name(rdev->dev));
}
} }
if (!i) if (!i)
sz += sprintf(page + sz, "<none>"); sz += sprintf(page + sz, "<none>");
...@@ -3556,7 +3542,7 @@ static void autostart_arrays(void) ...@@ -3556,7 +3542,7 @@ static void autostart_arrays(void)
MD_BUG(); MD_BUG();
continue; continue;
} }
list_add(&rdev->pending, &pending_raid_disks); list_add(&rdev->same_set, &pending_raid_disks);
} }
dev_cnt = 0; dev_cnt = 0;
......
...@@ -143,8 +143,6 @@ static inline void mark_disk_nonsync(mdp_disk_t * d) ...@@ -143,8 +143,6 @@ static inline void mark_disk_nonsync(mdp_disk_t * d)
struct mdk_rdev_s struct mdk_rdev_s
{ {
struct list_head same_set; /* RAID devices within the same set */ struct list_head same_set; /* RAID devices within the same set */
struct list_head all; /* all RAID devices */
struct list_head pending; /* undetected RAID devices */
kdev_t dev; /* Device number */ kdev_t dev; /* Device number */
kdev_t old_dev; /* "" when it was last imported */ kdev_t old_dev; /* "" when it was last imported */
...@@ -239,30 +237,23 @@ extern mdp_disk_t *get_spare(mddev_t *mddev); ...@@ -239,30 +237,23 @@ extern mdp_disk_t *get_spare(mddev_t *mddev);
* iterates through some rdev ringlist. It's safe to remove the * iterates through some rdev ringlist. It's safe to remove the
* current 'rdev'. Dont touch 'tmp' though. * current 'rdev'. Dont touch 'tmp' though.
*/ */
#define ITERATE_RDEV_GENERIC(head,field,rdev,tmp) \ #define ITERATE_RDEV_GENERIC(head,rdev,tmp) \
\ \
for ((tmp) = (head).next; \ for ((tmp) = (head).next; \
(rdev) = (list_entry((tmp), mdk_rdev_t, field)), \ (rdev) = (list_entry((tmp), mdk_rdev_t, same_set)), \
(tmp) = (tmp)->next, (tmp)->prev != &(head) \ (tmp) = (tmp)->next, (tmp)->prev != &(head) \
; ) ; )
/* /*
* iterates through the 'same array disks' ringlist * iterates through the 'same array disks' ringlist
*/ */
#define ITERATE_RDEV(mddev,rdev,tmp) \ #define ITERATE_RDEV(mddev,rdev,tmp) \
ITERATE_RDEV_GENERIC((mddev)->disks,same_set,rdev,tmp) ITERATE_RDEV_GENERIC((mddev)->disks,rdev,tmp)
/*
* Iterates through all 'RAID managed disks'
*/
#define ITERATE_RDEV_ALL(rdev,tmp) \
ITERATE_RDEV_GENERIC(all_raid_disks,all,rdev,tmp)
/* /*
* Iterates through 'pending RAID disks' * Iterates through 'pending RAID disks'
*/ */
#define ITERATE_RDEV_PENDING(rdev,tmp) \ #define ITERATE_RDEV_PENDING(rdev,tmp) \
ITERATE_RDEV_GENERIC(pending_raid_disks,pending,rdev,tmp) ITERATE_RDEV_GENERIC(pending_raid_disks,rdev,tmp)
#define xchg_values(x,y) do { __typeof__(x) __tmp = x; \ #define xchg_values(x,y) do { __typeof__(x) __tmp = x; \
x = y; y = __tmp; } while (0) x = y; y = __tmp; } while (0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment