-
Notifications
You must be signed in to change notification settings - Fork 139
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
AArch64: cxl-region-sysfs.sh: line 49: echo: write error: Numerical result out of range #278
Comments
I think I know why, stay tuned. Will share more in the next few days. |
@marc-hb thanks! |
@marc-hb I couldn't understand well why echo'ing 2GB to the sysfs size entry ends up with an out of range, but smaller values fails with invalid argument. Writing 0 seems to work but no meaning at all. The string to integer conversion shouldn't be an issue should it? |
Sorry I got distracted. I just replied on the list: |
I spent a bit more time on this and realized that |
@marc-hb thanks for letting me know! I'm still looking at the host physical address allocation code, that ends up searching for free area, provided we only need 8 x 256MB region size, something kernel or QEMU is having issues or both? |
This is a follow-up on this Issue, again tested on a CXL capable emulated system laucned from run_qemu.sh. |
|
Jonathan replied in pmem/run_qemu#150, you can find his account name there. A trick to find anyone's account is to look at any commit they authored in any git repo hosted or mirrored on GitHub. |
The kernel logs are very likely more useful. |
Is this better?
Could @jic23 comment on this too? |
Interesting, I get a similar error when x86-specific KASLR is enabled by default. That's because KASLR reduces the maximum range available. You seem to land in a similar place in a different and unrelated way? |
@marc-hb @AlisonSchofield do you happen to know how the CXL setup on the q35 machine avoids this situation? The VM launched from the run_qemu.sh script has CEDT, but not sure if it is a valid ACPI table, nor if the kernel (driver) parses it correctly. I never edited the options set in |
$ meson test cxl-region-sysfs.sh
fails with the messages below from testlog.txt file:The kernel and cxl_test module is built from cxl-next:
https://git.kernel.org/pub/scm/linux/kernel/git/cxl/cxl.git/
ndctl version is HEAD.
The text was updated successfully, but these errors were encountered: