From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Tue, 02 Jul 2013 00:56:53 +0000 Subject: Re: [PATCH 1/2] powerpc/booke64: Use common defines for AltiVec interrupts numbers Message-Id: <1372726613.8183.104@snotra> List-Id: References: <8F554F03-E6B0-4140-8830-D4FD6249CEF9@suse.de> In-Reply-To: <8F554F03-E6B0-4140-8830-D4FD6249CEF9@suse.de> (from agraf@suse.de on Mon Jul 1 19:18:21 2013) MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Alexander Graf Cc: Mihai Caraman , linuxppc-dev@lists.ozlabs.org, kvm-ppc@vger.kernel.org, kvm@vger.kernel.org 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. :-) -Scott