Commit c9710d80 authored by Tejun Heo's avatar Tejun Heo

cpuset: drop "const" qualifiers from struct cpuset instances

cpuset uses "const" qualifiers on struct cpuset in some functions;
however, it doesn't work well when a value derived from returned const
pointer has to be passed to an accessor.  It's C after all.

Drop the "const" qualifiers except for the trivially leaf ones.  This
patch doesn't make any functional changes.
Signed-off-by: default avatarTejun Heo <tj@kernel.org>
Acked-by: default avatarLi Zefan <lizefan@huawei.com>
parent 8af01f56
...@@ -128,7 +128,7 @@ static inline struct cpuset *task_cs(struct task_struct *task) ...@@ -128,7 +128,7 @@ static inline struct cpuset *task_cs(struct task_struct *task)
struct cpuset, css); struct cpuset, css);
} }
static inline struct cpuset *parent_cs(const struct cpuset *cs) static inline struct cpuset *parent_cs(struct cpuset *cs)
{ {
struct cgroup *pcgrp = cs->css.cgroup->parent; struct cgroup *pcgrp = cs->css.cgroup->parent;
...@@ -319,8 +319,7 @@ static struct file_system_type cpuset_fs_type = { ...@@ -319,8 +319,7 @@ static struct file_system_type cpuset_fs_type = {
* *
* Call with callback_mutex held. * Call with callback_mutex held.
*/ */
static void guarantee_online_cpus(const struct cpuset *cs, static void guarantee_online_cpus(struct cpuset *cs, struct cpumask *pmask)
struct cpumask *pmask)
{ {
while (!cpumask_intersects(cs->cpus_allowed, cpu_online_mask)) while (!cpumask_intersects(cs->cpus_allowed, cpu_online_mask))
cs = parent_cs(cs); cs = parent_cs(cs);
...@@ -338,7 +337,7 @@ static void guarantee_online_cpus(const struct cpuset *cs, ...@@ -338,7 +337,7 @@ static void guarantee_online_cpus(const struct cpuset *cs,
* *
* Call with callback_mutex held. * Call with callback_mutex held.
*/ */
static void guarantee_online_mems(const struct cpuset *cs, nodemask_t *pmask) static void guarantee_online_mems(struct cpuset *cs, nodemask_t *pmask)
{ {
while (!nodes_intersects(cs->mems_allowed, node_states[N_MEMORY])) while (!nodes_intersects(cs->mems_allowed, node_states[N_MEMORY]))
cs = parent_cs(cs); cs = parent_cs(cs);
...@@ -383,7 +382,7 @@ static int is_cpuset_subset(const struct cpuset *p, const struct cpuset *q) ...@@ -383,7 +382,7 @@ static int is_cpuset_subset(const struct cpuset *p, const struct cpuset *q)
* alloc_trial_cpuset - allocate a trial cpuset * alloc_trial_cpuset - allocate a trial cpuset
* @cs: the cpuset that the trial cpuset duplicates * @cs: the cpuset that the trial cpuset duplicates
*/ */
static struct cpuset *alloc_trial_cpuset(const struct cpuset *cs) static struct cpuset *alloc_trial_cpuset(struct cpuset *cs)
{ {
struct cpuset *trial; struct cpuset *trial;
...@@ -430,7 +429,7 @@ static void free_trial_cpuset(struct cpuset *trial) ...@@ -430,7 +429,7 @@ static void free_trial_cpuset(struct cpuset *trial)
* Return 0 if valid, -errno if not. * Return 0 if valid, -errno if not.
*/ */
static int validate_change(const struct cpuset *cur, const struct cpuset *trial) static int validate_change(struct cpuset *cur, struct cpuset *trial)
{ {
struct cgroup *cgrp; struct cgroup *cgrp;
struct cpuset *c, *par; struct cpuset *c, *par;
...@@ -2343,7 +2342,7 @@ void cpuset_cpus_allowed(struct task_struct *tsk, struct cpumask *pmask) ...@@ -2343,7 +2342,7 @@ void cpuset_cpus_allowed(struct task_struct *tsk, struct cpumask *pmask)
void cpuset_cpus_allowed_fallback(struct task_struct *tsk) void cpuset_cpus_allowed_fallback(struct task_struct *tsk)
{ {
const struct cpuset *cpus_cs; struct cpuset *cpus_cs;
rcu_read_lock(); rcu_read_lock();
cpus_cs = effective_cpumask_cpuset(task_cs(tsk)); cpus_cs = effective_cpumask_cpuset(task_cs(tsk));
...@@ -2416,7 +2415,7 @@ int cpuset_nodemask_valid_mems_allowed(nodemask_t *nodemask) ...@@ -2416,7 +2415,7 @@ int cpuset_nodemask_valid_mems_allowed(nodemask_t *nodemask)
* callback_mutex. If no ancestor is mem_exclusive or mem_hardwall * callback_mutex. If no ancestor is mem_exclusive or mem_hardwall
* (an unusual configuration), then returns the root cpuset. * (an unusual configuration), then returns the root cpuset.
*/ */
static const struct cpuset *nearest_hardwall_ancestor(const struct cpuset *cs) static struct cpuset *nearest_hardwall_ancestor(struct cpuset *cs)
{ {
while (!(is_mem_exclusive(cs) || is_mem_hardwall(cs)) && parent_cs(cs)) while (!(is_mem_exclusive(cs) || is_mem_hardwall(cs)) && parent_cs(cs))
cs = parent_cs(cs); cs = parent_cs(cs);
...@@ -2486,7 +2485,7 @@ static const struct cpuset *nearest_hardwall_ancestor(const struct cpuset *cs) ...@@ -2486,7 +2485,7 @@ static const struct cpuset *nearest_hardwall_ancestor(const struct cpuset *cs)
*/ */
int __cpuset_node_allowed_softwall(int node, gfp_t gfp_mask) int __cpuset_node_allowed_softwall(int node, gfp_t gfp_mask)
{ {
const struct cpuset *cs; /* current cpuset ancestors */ struct cpuset *cs; /* current cpuset ancestors */
int allowed; /* is allocation in zone z allowed? */ int allowed; /* is allocation in zone z allowed? */
if (in_interrupt() || (gfp_mask & __GFP_THISNODE)) if (in_interrupt() || (gfp_mask & __GFP_THISNODE))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment