On 02.07.2013, at 02:56, Scott Wood wrote: > On 07/01/2013 07:18:21 PM, Alexander Graf wrote: >> On 01.07.2013, at 17:35, Mihai Caraman wrote: >> > On Book3E some SPE/FP/AltiVec interrupts share the same number. Use >> > common defines to indentify these numbers. >> So why didn't this happen from the beginning? > > Ask Kumar. > >> Why the change? > > So we can remove this hack in kvm_asm.h: > > /* > * TODO: Unify 32-bit and 64-bit kernel exception handlers to use same defines > */ > #define BOOKE_INTERRUPT_SPE_UNAVAIL BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL > #define BOOKE_INTERRUPT_SPE_FP_DATA BOOKE_INTERRUPT_SPE_FP_DATA_ALTIVEC_ASSIST > #define BOOKE_INTERRUPT_ALTIVEC_UNAVAIL BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL > #define BOOKE_INTERRUPT_ALTIVEC_ASSIST \ > BOOKE_INTERRUPT_SPE_FP_DATA_ALTIVEC_ASSIST > > It was added as a compilation fix, and it was less intrusive to temporarily fix it this way. > > I am curious why the above code wasn't removed at the end of this patchset. :-) Yeah, and why the intention isn't clearly written down in the patch description ;) Alex -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
