mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-05 10:31:46 +09:00
UPSTREAM: mm/damon/sysfs: remove parameters of damon_sysfs_region_alloc()
'damon_sysfs_region_alloc()' is always called with zero-filled 'struct
damon_addr_range', because the start and end addresses should set by
users. Remove unnecessary parameters of the function and simplify the
body by using 'kzalloc()'.
Link: https://lkml.kernel.org/r/20221026225943.100429-7-sj@kernel.org
Signed-off-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
(cherry picked from commit 1f71981408)
Bug: 300502883
Change-Id: I6fa54a7474851598fda50a07f5b6b2465bf4cd1f
Signed-off-by: cui yangpei <cuiyangpei@xiaomi.com>
This commit is contained in:
committed by
Suren Baghdasaryan
parent
19364f11a4
commit
b7fc8d59a5
@@ -1065,17 +1065,9 @@ struct damon_sysfs_region {
|
|||||||
struct damon_addr_range ar;
|
struct damon_addr_range ar;
|
||||||
};
|
};
|
||||||
|
|
||||||
static struct damon_sysfs_region *damon_sysfs_region_alloc(
|
static struct damon_sysfs_region *damon_sysfs_region_alloc(void)
|
||||||
struct damon_addr_range ar)
|
|
||||||
{
|
{
|
||||||
struct damon_sysfs_region *region = kmalloc(sizeof(*region),
|
return kzalloc(sizeof(struct damon_sysfs_region), GFP_KERNEL);
|
||||||
GFP_KERNEL);
|
|
||||||
|
|
||||||
if (!region)
|
|
||||||
return NULL;
|
|
||||||
region->kobj = (struct kobject){};
|
|
||||||
region->ar = ar;
|
|
||||||
return region;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static ssize_t start_show(struct kobject *kobj, struct kobj_attribute *attr,
|
static ssize_t start_show(struct kobject *kobj, struct kobj_attribute *attr,
|
||||||
@@ -1184,7 +1176,7 @@ static int damon_sysfs_regions_add_dirs(struct damon_sysfs_regions *regions,
|
|||||||
regions->regions_arr = regions_arr;
|
regions->regions_arr = regions_arr;
|
||||||
|
|
||||||
for (i = 0; i < nr_regions; i++) {
|
for (i = 0; i < nr_regions; i++) {
|
||||||
region = damon_sysfs_region_alloc((struct damon_addr_range){});
|
region = damon_sysfs_region_alloc();
|
||||||
if (!region) {
|
if (!region) {
|
||||||
damon_sysfs_regions_rm_dirs(regions);
|
damon_sysfs_regions_rm_dirs(regions);
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|||||||
Reference in New Issue
Block a user