mirror of
https://git.FreeBSD.org/src.git
synced 2024-12-16 10:20:30 +00:00
Add some comments about the deficiencies of this module. I had hoped to get
around to addressing them some more, but Real Life (tm) has gotten in the way.
This commit is contained in:
parent
e5952d25fc
commit
ddfa3cce05
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=110513
@ -76,6 +76,10 @@ g_vol_ffs_taste(struct g_class *mp, struct g_provider *pp, int flags)
|
||||
g_trace(G_T_TOPOLOGY, "vol_taste(%s,%s)", mp->name, pp->name);
|
||||
g_topology_assert();
|
||||
|
||||
/*
|
||||
* XXX This is a really weak way to make sure we don't recurse.
|
||||
* Probably ought to use BIO_GETATTR to check for this.
|
||||
*/
|
||||
if (flags == G_TF_NORMAL &&
|
||||
!strcmp(pp->geom->class->name, VOL_FFS_CLASS_NAME))
|
||||
return (NULL);
|
||||
@ -118,6 +122,9 @@ g_vol_ffs_taste(struct g_class *mp, struct g_provider *pp, int flags)
|
||||
g_free(fs);
|
||||
continue;
|
||||
}
|
||||
/* XXX We need to check for namespace conflicts. */
|
||||
/* XXX How do you handle a mirror set? */
|
||||
/* XXX We don't validate the volume name. */
|
||||
g_topology_lock();
|
||||
/* Alright, we have a label and a volume name, reconfig. */
|
||||
g_slice_config(gp, 0, G_SLICE_CONFIG_SET, (off_t) 0,
|
||||
|
Loading…
Reference in New Issue
Block a user