-
Notifications
You must be signed in to change notification settings - Fork 723
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
Test-extended.system win SharedClassesAPI_0 assert The softmx limit for shared cache usage is smaller than the minimum feasible value #4351
Comments
Mesbah said in adoptium/openj9-systemtest#74 The Workload that crashes, WL3 runs the following command:
|
@Mesbah-Alam I was thinking more along the lines of, what is the test doing that causes the assert.
It seems to be related to the cache contents. The assert checks the softmx value is between the amount of data which is in the cache, but not bigger than the total size of the cache. |
There was a coma missing from the sharedClasses options being formulated to be used in the testcase. Added here: adoptium/openj9-systemtest#76 Not sure if it is the root cause of the crash though. Running a 5x Grinder in internal Jenkins on Windows with the above pull request atm. |
This test is now passing: Windows JDK8:
https://ci.eclipse.org/openj9/job/Test-extended.system-JDK8-win_x86-64_cmprssptrs/152/consoleFull Window JDK11: |
Linking the systemtest repo PR here: adoptium/openj9-systemtest#80 |
@hangshao0 do you know what condition caused the VM to assert? A user shouldn't be able to cause the VM to assert via command line arguments, the underlying problem should still be fixed. |
We call the above code to check the cache file size. We are not able to open the cache file, so cache file size _actualCacheSize is 0. When setting the softmx, we assert than the sotfmx < _actualCacheSize (=0). |
I will create a Pull Request to ensure that the cache file path is not longer than the OS limit before starting up the shared cache. |
1. Remove duplicated define of J9SH_MAXPATH in j9sharedhelper.h 2. Make sure the length of non-persistent cache mutex name is less than _MAX_PATH on Windows. 3. Make sure the cache file path is shorter than J9SH_MAXPATH before starting up the shared cache. closes eclipse-openj9#4351 Signed-off-by: hangshao <[email protected]>
Reopened since #4532 was reverted. |
OMR PR (eclipse-omr/omr#3553) to export file_get_unicode_path() which prepend |
On Windows, "\\?\" needs to be prepended to the path if its length is > MAX_PATH. port_convertFromUTF8() won't prepend "\\?\". Call port_file_get_unicode_path() so that "\\?\" is prepended. Closes eclipse-openj9#4351 Signed-off-by: hangshao <[email protected]>
On Windows, "\\?\" needs to be prepended to the path if its length is > MAX_PATH. port_convertFromUTF8() won't prepend "\\?\". Call port_file_get_unicode_path() so that "\\?\" is prepended. Closes eclipse-openj9#4351 Signed-off-by: hangshao <[email protected]>
in stderr output
in javacore
The text was updated successfully, but these errors were encountered: