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; }