diff options
| author | mav <mav@FreeBSD.org> | 2014-05-08 12:07:40 +0000 |
|---|---|---|
| committer | mav <mav@FreeBSD.org> | 2014-05-08 12:07:40 +0000 |
| commit | df9f601cbcdbfe062c43e38da3dd1a4351c46fdf (patch) | |
| tree | 40fa3a9c7ec23e561339796d3792157d5630df85 /sys/geom/raid/g_raid.c | |
| parent | 69f94d7a6b01beaadc4c8acbcabcada8e4c9291f (diff) | |
| download | FreeBSD-src-df9f601cbcdbfe062c43e38da3dd1a4351c46fdf.zip FreeBSD-src-df9f601cbcdbfe062c43e38da3dd1a4351c46fdf.tar.gz | |
MFC r265054:
Reduce number of opens by REOM RAID during provider taste.
Instead opening/closing provider by each of metadata classes, do it only
once in core code. Since for SCSI disks open/close means sending some
SCSI commands to the device, this change reduces taste time.
Sponsored by: iXsystems, Inc.
Diffstat (limited to 'sys/geom/raid/g_raid.c')
| -rw-r--r-- | sys/geom/raid/g_raid.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/sys/geom/raid/g_raid.c b/sys/geom/raid/g_raid.c index a161f8a..858cf4c 100644 --- a/sys/geom/raid/g_raid.c +++ b/sys/geom/raid/g_raid.c @@ -2251,6 +2251,8 @@ g_raid_taste(struct g_class *mp, struct g_provider *pp, int flags __unused) return (NULL); G_RAID_DEBUG(2, "Tasting provider %s.", pp->name); + geom = NULL; + status = G_RAID_MD_TASTE_FAIL; gp = g_new_geomf(mp, "raid:taste"); /* * This orphan function should be never called. @@ -2259,8 +2261,9 @@ g_raid_taste(struct g_class *mp, struct g_provider *pp, int flags __unused) cp = g_new_consumer(gp); cp->flags |= G_CF_DIRECT_RECEIVE; g_attach(cp, pp); + if (g_access(cp, 1, 0, 0) != 0) + goto ofail; - geom = NULL; LIST_FOREACH(class, &g_raid_md_classes, mdc_list) { if (!class->mdc_enable) continue; @@ -2276,6 +2279,9 @@ g_raid_taste(struct g_class *mp, struct g_provider *pp, int flags __unused) break; } + if (status == G_RAID_MD_TASTE_FAIL) + (void)g_access(cp, -1, 0, 0); +ofail: g_detach(cp); g_destroy_consumer(cp); g_destroy_geom(gp); |
