Messages in this thread | | | Date | Thu, 22 May 2008 23:37:34 +0200 (CEST) | From | Thomas Gleixner <> | Subject | Re: [RFC] x86: merging nmi 32/64 to nmi.c v2 |
| |
On Wed, 21 May 2008, Cyrill Gorcunov wrote: > > Here is updated version (thanks Hiroshi) - I've moved these funcs > to traps_64.c and it's compiled just fine (so we are going closer > to merge someday even traps-32/64 ;)
Cyrill,
I really like that patch, but I'm a bit wary about the doit in one go approach. Its pretty hard to review that way and in case of breakage bisecting will just point to one large diff which is hard to decode.
Can you please try to make this a more fine grained patch series ?
Gradually add/modify the two files so they are identical at the end. There are a couple of obvious steps, e.g. the move of function to traps_*.c and adding the inline functions like get_timer_irqs().
Thanks, tglx
| |