-
Notifications
You must be signed in to change notification settings - Fork 720
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Assert failure " old api and new api did not match, feature 107" #9833
Comments
@mpirvu Is this a blocker for the |
There is only one instance in the codebase. I think I have an idea why the assertion triggered. TR_ASSERT_FATAL(comp->target().cpu.supportsFeature(OMR_FEATURE_X86_RTM) == _targetProcessorInfo.supportsTM(), "supportsTM() failed\n");
if (comp->target().cpu.supportsFeature(OMR_FEATURE_X86_RTM) && !comp->getOption(TR_DisableTM))
{
/**
* Due to many verions of Haswell and a small number of Broadwell have defects for TM and then disabled by Intel,
* we will return false for any versions before Broadwell.
*
* TODO: Need to figure out from which mode of Broadwell start supporting TM
*/
case OMR_FEATURE_X86_RTM:
return TR::CodeGenerator::getX86ProcessorInfo().supportsTM() == ans; Coming up with a fix now. |
@mpirvu is this running with JITServer? and does it happen consistently? |
Okay some explanations for the failure and I can fix this by the end of today: My assertions are testing the newly implemented processor detection api's against the old processor detection api's. And when they are different, the assert gets triggered. I'm guessing the assert failure will only be trigged in JITServer mode. |
Yes, I saw it on the JITServer and it happens every single time. |
If it's convenient for you, I have the fixes here #9837 and eclipse-omr/omr#5298 that you can cherry-pick in and verify. |
This should have be fixed now. I'll verify by building 0.21.0 with JITServer |
I verified that the problem is fixed |
I encountered the following assert while running on an Amazon t3.micro instance:
Processor info as given by
lscpu
:OpenJ9 version info:
Attn: @harryyu1994
The text was updated successfully, but these errors were encountered: