Messages in this thread | | | Date | Sat, 27 Apr 2013 17:32:27 +0800 | From | Mike Qiu <> | Subject | Re: [PATCH v2] PowerPC: kernel: compiling issue, make additional room in exception vector area |
| |
于 2013/4/27 17:28, Chen Gang F T 写道: > On 2013年04月26日 11:54, Mike Qiu wrote: >> 于 2013/4/26 11:42, Chen Gang 写道: >>> On 2013年04月26日 11:25, Chen Gang wrote: >>>> On 2013年04月26日 11:08, Mike Qiu wrote: >>>>> 于 2013/4/26 10:06, Chen Gang 写道: >>>>>> On 2013年04月26日 10:03, Mike Qiu wrote: >>>>>>> �� 2013/4/26 9:36, Chen Gang д��: >>>>>>>>> On 2013��04��26�� 09:18, Chen Gang wrote: >>>>>>>>>>> On 2013��04��26�� 09:06, Chen Gang wrote: >>>>>>>>>>>>>>> CFAR is the Come From Register. It saves the location of the >>>>>>>>>>>>>>> last >>>>>>>>>>>>>>>>> branch and is hence overwritten by any branch. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>> Do we process it just like others done (e.g. 0x300, 0xe00, >>>>>>>>>>>>> 0xe20 ...) ? >>>>>>>>>>>>> . = 0x900 >>>>>>>>>>>>> .globl decrementer_pSeries >>>>>>>>>>>>> decrementer_pSeries: >>>>>>>>>>>>> HMT_MEDIUM_PPR_DISCARD >>>>>>>>>>>>> SET_SCRATCH0(r13) >>>>>>>>>>>>> b decrementer_pSeries_0 >>>>>>>>>>>>> >>>>>>>>>>>>> ... >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>> Oh, it seems EXCEPTION_PROLOG_1 will save the regesters which >>>>>>>>> related >>>>>>>>> with CFAR, so I think need move EXCEPTION_PROLOG_1 to near 0x900. >>>>>>> I will try your diff V2, to see if the machine can boot up >>>>>> OK, thanks. (hope it can work) >>>>> It seems that the machine can be bootup in powernv mode, but I'm not >>>>> sure if my machine call that module. >>>>> >>>>> At lease my machine can boot up >>> Please reference commit number: 1707dd161349e6c54170c88d94fed012e3d224e3 >>> (1707dd1 powerpc: Save CFAR before branching in interrupt entry paths) >>> >>> What our diff v2 has done is just the fix for our patch v2 (just like >>> the commit 1707dd1 has done). >>> >>> Please check, thanks. >>> >>> :-) >> I will check this evening or tomorrow, I have something else to do this >> afteroon. > I think the diff v2 is correct, but is not the best one for this issue. > > I prefer the Paul's patch for this issue which has better performance > > :-) yes, I use your patch and it can work, also Paul's patch can work too. > > Thanks. >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |