Messages in this thread | | | Date | Mon, 26 Mar 2018 10:15:11 +0200 | From | Greg KH <> | Subject | Re: [PATCH 2/2] zram: idle memory tracking |
| |
On Mon, Mar 26, 2018 at 03:49:51PM +0900, Minchan Kim wrote: > +static int zram_debugfs_register(struct zram *zram) > +{ > + struct dentry *ret; > + > + if (!zram_debugfs_root) > + return -ENOENT;
No need to care, you should not error out if debugfs is not enabled or not working, your code path should be identical either way. debugfs is not required for any functionality, so don't treat it like it matters :)
> + zram->debugfs_dir = debugfs_create_dir(zram->disk->disk_name, > + zram_debugfs_root); > + if (!zram->debugfs_dir) > + return -ENOMEM;
No need to check the return value of any debugfs call, you can always either use it for future debugfs calls, or ignore it.
> + ret = debugfs_create_file("access_time", 0400, zram->debugfs_dir, > + zram, &proc_zram_access_operations); > + if (!ret) > + return -ENOMEM;
Again, you shouldn't care :)
> + > + return 0;
just return void, no need to test any of this.
thanks,
greg k-h
| |