Messages in this thread | | | Date | Fri, 1 Aug 2014 08:58:52 -0500 (CDT) | From | Christoph Lameter <> | Subject | Re: [scheduler] BUG: unable to handle kernel paging request at 000000000000ce50 |
| |
On Thu, 31 Jul 2014, Lai Jiangshan wrote:
> > this_cpu_ptr instead. > > > - struct cpumask *cpus = __get_cpu_var(load_balance_mask); > + struct cpumask *cpus = this_cpu_ptr(load_balance_mask); > > > I think the conversion is wrong. it should be > *this_cpu_ptr(&load_balance_mask);
Hmmm.... that is going to cause some pretty strange looking code. Would the compiler/sparse not complain if the wrong type of pointer is assigned to?
If we are retrieving a pointer then the right way to do things would be:
cpus = this_cpu_read(load_balance_mask);
| |