Re: [PATCH 2/2] zram: idle memory tracking

[Greg KH]

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