On Thu, Oct 25, 2018 at 01:53:06PM +0000, Christopher Lameter wrote: >On Thu, 25 Oct 2018, Wei Yang wrote: > >> In current code, we use two forms to access s->cpu_slab >> >> * raw_cpu_ptr() >> * this_cpu_ptr() > >Ok the only difference is that for CONFIG_DEBUG_PREEMPT we will do the >debug checks twice. > >That tolerable I think but is this really a worthwhile change? Agree. My purpose is to make unify the access, looks easy for me to read the code. You can decide whether to change this or not :-) -- Wei Yang Help you, Help me