git: 78c51db3c492 - main - udf: uma_zcreate() does not fail
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 24 Apr 2024 13:17:14 UTC
The branch main has been updated by markj:
URL: https://cgit.FreeBSD.org/src/commit/?id=78c51db3c4927db2437ec616b33ba1faf73f08ee
commit 78c51db3c4927db2437ec616b33ba1faf73f08ee
Author: Mark Johnston <markj@FreeBSD.org>
AuthorDate: 2024-04-23 16:20:50 +0000
Commit: Mark Johnston <markj@FreeBSD.org>
CommitDate: 2024-04-24 12:45:59 +0000
udf: uma_zcreate() does not fail
While here remove an old comment regarding preallocation; it appears to
refer to an optimization that is almost certainly irrelevant at this
point.
No functional change intended.
MFC after: 1 week
---
sys/fs/udf/udf_vfsops.c | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/sys/fs/udf/udf_vfsops.c b/sys/fs/udf/udf_vfsops.c
index e844a4e540f4..866d0172f745 100644
--- a/sys/fs/udf/udf_vfsops.c
+++ b/sys/fs/udf/udf_vfsops.c
@@ -140,11 +140,6 @@ static int
udf_init(struct vfsconf *foo)
{
- /*
- * This code used to pre-allocate a certain number of pages for each
- * pool, reducing the need to grow the zones later on. UMA doesn't
- * advertise any such functionality, unfortunately =-<
- */
udf_zone_trans = uma_zcreate("UDF translation buffer, zone", MAXNAMLEN *
sizeof(unicode_t), NULL, NULL, NULL, NULL, 0, 0);
@@ -154,12 +149,6 @@ udf_init(struct vfsconf *foo)
udf_zone_ds = uma_zcreate("UDF Dirstream zone",
sizeof(struct udf_dirstream), NULL, NULL, NULL, NULL, 0, 0);
- if ((udf_zone_node == NULL) || (udf_zone_trans == NULL) ||
- (udf_zone_ds == NULL)) {
- printf("Cannot create allocation zones.\n");
- return (ENOMEM);
- }
-
return 0;
}