Most of these look fine > - return debugfs_create_file(name, mode, parent, (void __force *)value, &fops_io_x64); > + return debugfs_create_file_unsafe(name, mode, parent, > + (void __force *)value, &fops_io_x64); Just wondering what this one is about? Cheers, -Ian