The branch main has been updated by markj:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=78c51db3c4927db2437ec616b33ba1faf73f08ee

commit 78c51db3c4927db2437ec616b33ba1faf73f08ee
Author:     Mark Johnston <ma...@freebsd.org>
AuthorDate: 2024-04-23 16:20:50 +0000
Commit:     Mark Johnston <ma...@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;
 }
 

Reply via email to