review for 7055355: JSR 292: crash while throwing WrongMethodTypeException
John Rose
john.r.rose at oracle.com
Thu Jun 16 13:56:26 PDT 2011
On Jun 16, 2011, at 10:18 AM, Tom Rodriguez wrote:
> I increased the slop value in my last set of changes and we're currently not failing this test that I can see. I agree that the test is slightly mystifying and wouldn't mind removing it.
I would prefer to keep it for the following reason:
When control returns to a RF, the SP is restored to a saved value. Comparing it to the saved argument base is a reasonable thing, since the RF is known to contain only (a) up to 255 saved arguements, and (b) a a few words of fixed overhead. If the restored SP is bad, and/or if the saved argument base is bad, we can detect this quickly using the comparison before the corruption goes further.
The slop has to be large enough to contain the "save_frame" size used by the RF.
-- John
More information about the hotspot-compiler-dev
mailing list