lkml.org 
[lkml]   [2018]   [Mar]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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

\
 
 \ /
  Last update: 2018-03-26 10:15    [W:0.282 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site