From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============5702914822117416007=="
MIME-Version: 1.0
From: kernel test robot <lkp@intel.com>
Subject: mm/damon/dbgfs.c:525 dbgfs_init_regions_write() warn: passing a valid pointer to 'PTR_ERR'
Date: Thu, 31 Mar 2022 21:30:42 +0800
Message-ID: <202203312110.1I6NoFQu-lkp@intel.com>
List-Id: <oe-kbuild.lists.linux.dev>
To: kbuild@lists.01.org

--===============5702914822117416007==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

CC: kbuild-all(a)lists.01.org
BCC: lkp(a)intel.com
CC: linux-kernel(a)vger.kernel.org
TO: SeongJae Park <sj@kernel.org>
CC: Andrew Morton <akpm@linux-foundation.org>
CC: Linux Memory Management List <linux-mm@kvack.org>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git =
master
head:   787af64d05cd528aac9ad16752d11bb1c6061bb9
commit: 90bebce9fcd6488ba6b010af3a16a0a0d7e44cb6 mm/damon/dbgfs: allow user=
s to set initial monitoring target regions
date:   5 months ago
:::::: branch date: 15 hours ago
:::::: commit date: 5 months ago
config: arc-randconfig-m031-20220331 (https://download.01.org/0day-ci/archi=
ve/20220331/202203312110.1I6NoFQu-lkp(a)intel.com/config)
compiler: arceb-elf-gcc (GCC) 11.2.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>

smatch warnings:
mm/damon/dbgfs.c:525 dbgfs_init_regions_write() warn: passing a valid point=
er to 'PTR_ERR'

vim +/PTR_ERR +525 mm/damon/dbgfs.c

90bebce9fcd648 SeongJae Park 2021-11-05  513  =

90bebce9fcd648 SeongJae Park 2021-11-05  514  static ssize_t dbgfs_init_reg=
ions_write(struct file *file,
90bebce9fcd648 SeongJae Park 2021-11-05  515  					  const char __user *buf=
, size_t count,
90bebce9fcd648 SeongJae Park 2021-11-05  516  					  loff_t *ppos)
90bebce9fcd648 SeongJae Park 2021-11-05  517  {
90bebce9fcd648 SeongJae Park 2021-11-05  518  	struct damon_ctx *ctx =3D fi=
le->private_data;
90bebce9fcd648 SeongJae Park 2021-11-05  519  	char *kbuf;
90bebce9fcd648 SeongJae Park 2021-11-05  520  	ssize_t ret =3D count;
90bebce9fcd648 SeongJae Park 2021-11-05  521  	int err;
90bebce9fcd648 SeongJae Park 2021-11-05  522  =

90bebce9fcd648 SeongJae Park 2021-11-05  523  	kbuf =3D user_input_str(buf,=
 count, ppos);
90bebce9fcd648 SeongJae Park 2021-11-05  524  	if (IS_ERR(kbuf))
90bebce9fcd648 SeongJae Park 2021-11-05 @525  		return PTR_ERR(kbuf);
90bebce9fcd648 SeongJae Park 2021-11-05  526  =

90bebce9fcd648 SeongJae Park 2021-11-05  527  	mutex_lock(&ctx->kdamond_loc=
k);
90bebce9fcd648 SeongJae Park 2021-11-05  528  	if (ctx->kdamond) {
90bebce9fcd648 SeongJae Park 2021-11-05  529  		ret =3D -EBUSY;
90bebce9fcd648 SeongJae Park 2021-11-05  530  		goto unlock_out;
90bebce9fcd648 SeongJae Park 2021-11-05  531  	}
90bebce9fcd648 SeongJae Park 2021-11-05  532  =

90bebce9fcd648 SeongJae Park 2021-11-05  533  	err =3D set_init_regions(ctx=
, kbuf, ret);
90bebce9fcd648 SeongJae Park 2021-11-05  534  	if (err)
90bebce9fcd648 SeongJae Park 2021-11-05  535  		ret =3D err;
90bebce9fcd648 SeongJae Park 2021-11-05  536  =

90bebce9fcd648 SeongJae Park 2021-11-05  537  unlock_out:
90bebce9fcd648 SeongJae Park 2021-11-05  538  	mutex_unlock(&ctx->kdamond_l=
ock);
90bebce9fcd648 SeongJae Park 2021-11-05  539  	kfree(kbuf);
90bebce9fcd648 SeongJae Park 2021-11-05  540  	return ret;
90bebce9fcd648 SeongJae Park 2021-11-05  541  }
90bebce9fcd648 SeongJae Park 2021-11-05  542  =


-- =

0-DAY CI Kernel Test Service
https://01.org/lkp
--===============5702914822117416007==--