Messages in this thread | | | Date | Wed, 09 Jul 2008 12:32:15 -0700 | From | Jeremy Fitzhardinge <> | Subject | Re: [RFC 00/15] x86_64: Optimize percpu accesses |
| |
Christoph Lameter wrote: > Well they help in the sense that the patches get rid of the special X_pda(Y) operations. > x86_X_percpu will then exist under 32 bit and 64 bit. > > What is remaining is the task to rename > > pda.Y -> Z > > in order to make variable references the same under both arches. Presumably the Z is the corresponding 32 bit variable. There are likely a number of cases where the transformation > is trivial if we just identify the corresponding 32 bit equivalent. >
Yes, I understand that, but it's still pointless churn. The intermediate step is no improvement over what was there before, and isn't any closer to the desired final result.
Once you've made the pda a percpu variable, and redefined all the X_pda macros in terms of x86_X_percpu, then there's no need to touch all the usage sites until you're *actually* going to unify something. Touching them all just because you find "X_pda" unsightly doesn't help anyone. Ideally every site you touch will remove a #ifdef CONFIG_X86_64, or make two as-yet unified pieces of code closer to unification.
J
| |