> + p2pmem_debugfs_root = debugfs_create_dir("p2pmem", NULL); > + if (!p2pmem_debugfs_root) > + pr_info("could not create debugfs entry, continuing\n"); > + Why continue? I think it'd be better to just fail it. Besides, this can be safely squashed into patch 1.