util: resource: Discard const keyword of struct resource 48/277448/4
authorChanwoo Choi <cw00.choi@samsung.com>
Mon, 4 Jul 2022 21:51:41 +0000 (06:51 +0900)
committerChanwoo Choi <cw00.choi@samsung.com>
Fri, 8 Jul 2022 09:03:24 +0000 (18:03 +0900)
Discard const keyword when accessing struct resource
by hiding the 'struct resource' from use of 'struct resource'.

Except for 'void *priv' field of struct resource,
the fields of struct resource should be accessed by getter/setter
functions.

Change-Id: I80fe2a7c9df8f6a5c273da9f222668a03ba8e62e
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
include/util/resource.h
src/resource/resource-battery.c
src/resource/resource-bus.c
src/resource/resource-cpu.c
src/resource/resource-display.c
src/resource/resource-gpu.c
src/resource/resource-memory.c
src/resource/resource-process-group.c
src/resource/resource-process.c
src/resource/resource-system.c
src/util/resource.c

index d9b62f7d81434e1959941a3a1a8d3817aa004b09..e96d3ba698e41caf509a80b4e88783db283ce8bc 100644 (file)
@@ -40,17 +40,17 @@ struct resource_attribute_value {
 };
 
 struct resource_attribute_ops {
-       int (*set)(const struct resource *res,
+       int (*set)(struct resource *res,
                   const struct resource_attribute *attr,
                   const void *data, int count);
-       int (*get)(const struct resource *res,
+       int (*get)(struct resource *res,
                   const struct resource_attribute *attr,
                   void *data);
        /*
         * If .is_supported ops is not implemented, use .get ops in order to
         * check whether the resource attribute is supported or not.
         */
-       bool (*is_supported)(const struct resource *res,
+       bool (*is_supported)(struct resource *res,
                   const struct resource_attribute *attr);
 };
 
@@ -64,10 +64,10 @@ struct resource_attribute {
 struct resource_control;
 
 struct resource_control_ops {
-       const int (*set)(const struct resource *res,
+       const int (*set)(struct resource *res,
                         const struct resource_control *ctrl,
                         const void *data);
-       const int (*get)(const struct resource *res,
+       const int (*get)(struct resource *res,
                         const struct resource_control *ctrl,
                         void **data);
 };
@@ -144,7 +144,7 @@ get_resource_attr_value(struct resource *resource, u_int64_t attr_id);
 bool is_resource_attr_supported(struct resource *resource, u_int64_t attr_id);
 
 static inline bool
-resource_attr_supported_always(const struct resource *resource,
+resource_attr_supported_always(struct resource *resource,
                               const struct resource_attribute *attr)
 {
        return true;
@@ -166,7 +166,7 @@ int get_resource_attr_ptr(struct resource *resource, u_int64_t attr_id, void **d
 
 int set_resource_attr_interest(struct resource *resource, u_int64_t interest_mask);
 int unset_resource_attr_interest(struct resource *resource, u_int64_t interest_mask);
-bool is_resource_attr_interested(const struct resource *resource, u_int64_t interest_mask);
+bool is_resource_attr_interested(struct resource *resource, u_int64_t interest_mask);
 
 const char *get_resource_name(struct resource *resource);
 void *get_resource_privdata(struct resource *resource);
index 584fe2f69d15d1b1bd4da32e877c3cd468d4bc21..ed51d395f49b19e98aff369f1f8f17de348b9a90 100644 (file)
@@ -30,7 +30,7 @@
 
 #include <resource-monitor/resource-monitor.h>
 
-static int battery_get_info(const struct resource *res,
+static int battery_get_info(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -71,7 +71,7 @@ static int battery_get_info(const struct resource *res,
        return 0;
 }
 
-static int battery_get_status(const struct resource *res,
+static int battery_get_status(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
index a1f2a9afd8b1abfeb9afe4286f1bfd8982901d7a..7366111c20aaf5fd4839f38df41219668a774581 100644 (file)
@@ -37,7 +37,7 @@ struct bus_context {
        int index;
 };
 
-static int bus_get_value_from_hal_power(const struct resource *res,
+static int bus_get_value_from_hal_power(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -89,7 +89,7 @@ static int bus_get_value_from_hal_power(const struct resource *res,
        return 0;
 }
 
-static int bus_get_curr_governor(const struct resource *res,
+static int bus_get_curr_governor(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -164,7 +164,7 @@ static const struct resource_attribute bus_attrs[] = {
        },
 };
 
-static int bus_setup_device_id(const struct resource *res,
+static int bus_setup_device_id(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index 1651e30674141445dd6bfe39ced028aa7e63df18..4613705ff676a90554cf8b121df7800dc6df6d4d 100644 (file)
@@ -37,7 +37,7 @@ struct cpu_context {
        int index;
 };
 
-static int cpu_get_value_from_hal_power(const struct resource *res,
+static int cpu_get_value_from_hal_power(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -88,7 +88,7 @@ static int cpu_get_value_from_hal_power(const struct resource *res,
        return 0;
 }
 
-static int cpu_get_curr_governor(const struct resource *res,
+static int cpu_get_curr_governor(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -163,7 +163,7 @@ static const struct resource_attribute cpu_attrs[] = {
        },
 };
 
-static int cpu_setup_cluster_id(const struct resource *res,
+static int cpu_setup_cluster_id(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index ea05b3204843ce2b6934c30b54729f3bcc73a59c..7cfebdbc469400e0413cf1328f71c9a56b9b5535 100644 (file)
@@ -46,7 +46,7 @@ struct display_fps_data {
        double fps;
 };
 
-static int display_get_fps(const struct resource *res,
+static int display_get_fps(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -139,7 +139,7 @@ static const struct resource_attribute display_attrs[] = {
        },
 };
 
-static int display_setup_device_id(const struct resource *res,
+static int display_setup_device_id(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index c9cc0b34dbe3b540c972a5c13e440ce90247f672..4fdce6ca05d1cc488cbf229147173b5dc207eb89 100644 (file)
@@ -37,7 +37,7 @@ struct gpu_context {
        int index;
 };
 
-static int gpu_get_value_from_hal_power(const struct resource *res,
+static int gpu_get_value_from_hal_power(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -89,7 +89,7 @@ static int gpu_get_value_from_hal_power(const struct resource *res,
        return 0;
 }
 
-static int gpu_get_curr_governor(const struct resource *res,
+static int gpu_get_curr_governor(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -164,7 +164,7 @@ static const struct resource_attribute gpu_attrs[] = {
        },
 };
 
-static int gpu_setup_device_id(const struct resource *res,
+static int gpu_setup_device_id(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index 48c627dc3cf7f3c35432bc396c8712fb519e17a6..a9218d2c8145c8b44aae667d4512e9c6db41dae2 100644 (file)
@@ -76,7 +76,7 @@ static int memory_get_swap_total(u_int64_t *val)
        return 0;
 }
 
-static int memory_get_memory_info(const struct resource *res,
+static int memory_get_memory_info(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
index 5d056ed97ce548f29c3366f65daa9c0714b0da32..e6a28127e46e6ff1f9cb86874aa1109da88a636c 100644 (file)
@@ -52,7 +52,7 @@ struct process_group_context {
 static u_int64_t total_memory;
 static long jiffy;
 
-static int process_group_get_pid_list(const struct resource *res,
+static int process_group_get_pid_list(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -92,7 +92,7 @@ static int process_group_get_pid_list(const struct resource *res,
        return 0;
 }
 
-static int process_group_get_comm_list(const struct resource *res,
+static int process_group_get_comm_list(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -145,7 +145,7 @@ static int process_group_get_comm_list(const struct resource *res,
        return 0;
 }
 
-static int process_group_get_cpu_util(const struct resource *res,
+static int process_group_get_cpu_util(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -167,7 +167,7 @@ static int process_group_get_cpu_util(const struct resource *res,
        return 0;
 }
 
-static int process_group_get_mem(const struct resource *res,
+static int process_group_get_mem(struct resource *res,
                           const struct resource_attribute *attr,
                           void *data)
 {
@@ -210,7 +210,7 @@ static int process_group_get_mem(const struct resource *res,
        return 0;
 }
 
-static int process_group_get_disk_bps(const struct resource *res,
+static int process_group_get_disk_bps(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -240,7 +240,7 @@ static int process_group_get_disk_bps(const struct resource *res,
        return 0;
 }
 
-static bool process_group_check_gpu_support(const struct resource *resource,
+static bool process_group_check_gpu_support(struct resource *resource,
                                            const struct resource_attribute *attr)
 {
        return kernel_check_gpu_support();
@@ -338,7 +338,7 @@ static const struct resource_attribute process_group_attrs[] = {
        },
 };
 
-static int process_group_setup_root_pid(const struct resource *res,
+static int process_group_setup_root_pid(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index 5d99b3e9905bdf7dd3e33bcd07a5ba90ac53958d..600d3d9444cb6feda17593ea279a7820433af166 100644 (file)
@@ -42,7 +42,7 @@ struct process_context {
 
 static long jiffy;
 
-static int process_get_cpu_util(const struct resource *res,
+static int process_get_cpu_util(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -76,7 +76,7 @@ static int process_get_cpu_util(const struct resource *res,
        return 0;
 }
 
-static int process_get_mem_attrs(const struct resource *res,
+static int process_get_mem_attrs(struct resource *res,
                                 const struct resource_attribute *attr,
                                 void *data)
 {
@@ -134,7 +134,7 @@ static int process_get_mem_attrs(const struct resource *res,
        return 0;
 }
 
-static int process_get_disk_bps(const struct resource *res,
+static int process_get_disk_bps(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -174,7 +174,7 @@ static int process_get_disk_bps(const struct resource *res,
        return 0;
 }
 
-static int process_get_context_data(const struct resource *res,
+static int process_get_context_data(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -221,7 +221,7 @@ static int process_get_context_data(const struct resource *res,
        return 0;
 }
 
-static bool process_check_gpu_support(const struct resource *resource,
+static bool process_check_gpu_support(struct resource *resource,
                                            const struct resource_attribute *attr)
 {
        return kernel_check_gpu_support();
@@ -355,7 +355,7 @@ static u_int64_t get_total_cpu_time(void)
        return total_time;
 }
 
-static int process_setup_tgid(const struct resource *res,
+static int process_setup_tgid(struct resource *res,
                                const struct resource_control *ctrl,
                                const void *data)
 {
index 99a22f68546fe85325cca4d6322c165a79d87129..72d58355daf009867f69df75dbfc06a0a951a4e4 100644 (file)
@@ -77,7 +77,7 @@ static double __calculate_cpu_util(int64_t id, struct cpu_stat *prev,
        return util;
 }
 
-static int system_get_avg_cpu_util(const struct resource *res,
+static int system_get_avg_cpu_util(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -101,7 +101,7 @@ static int system_get_avg_cpu_util(const struct resource *res,
        return 0;
 }
 
-static int system_get_per_cpu_util(const struct resource *res,
+static int system_get_per_cpu_util(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
@@ -143,7 +143,7 @@ static int system_get_per_cpu_util(const struct resource *res,
        return 0;
 }
 
-static int system_get_cpu_num(const struct resource *res,
+static int system_get_cpu_num(struct resource *res,
                                const struct resource_attribute *attr,
                                void *data)
 {
index feea4d193812ff0e9e00c35058052160bb7688c6..4f74cc01e2f0b17dbb5a0dc537d3814b9e260a2f 100644 (file)
@@ -963,7 +963,7 @@ int unset_resource_attr_interest(struct resource *resource, u_int64_t interest_m
        return 0;
 }
 
-bool is_resource_attr_interested(const struct resource *resource, u_int64_t interest_mask)
+bool is_resource_attr_interested(struct resource *resource, u_int64_t interest_mask)
 {
        if (!resource)
                return false;