-
Notifications
You must be signed in to change notification settings - Fork 200
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
Fixed FPGA build for ubuntu 24.04 and add btb trashing test #1042
Conversation
…o occupy the same location in the btb. Instruction class prediction mispredicts ever time. This is useful incase someone changes the hazard code or fpu.
The new test should probably be added to Looks like Vivado isn't officially supported on Ubuntu 24.04 yet (only 20.04 and 22.04) which is probably why there are issues. I wonder if symlinking the versions of ncurses that are installed in ubuntu 24.04 (v6 I think) to v5 would work instead. Not sure if there are breaking changes that prevent it from working with the newer versions though. |
I'm running into more problems with ubuntu 24.04. glibc has issues with questa and imperasDV. |
I've only tested regular tool flow and Verilator with 24.04, none of the proprietary tools. Hopefully it'll get better once there is official support for it from Vivado. What were the issues with Questa and ImpressDV? |
This is error running lockstep questa version 2023.4_3 with ImperasDV.
|
…ing with older versions of Ubuntu, but one 24.04 it causes vivado to not find the rom and ram.
Should we just say that Ubuntu 24.04 isn't compatible with current Questa? Does the latest Questa make any difference? |
@rosethompson For Vivado on Ubuntu 24.04 I got the following to work instead of the more complicated install script: ln -s /lib/x86_64-linux-gnu/libncurses.so.6 /lib/x86_64-linux-gnu/libncurses.so.5
ln -s /lib/x86_64-linux-gnu/libtinfo.so.6 /lib/x86_64-linux-gnu/libntinfo.so.5 It seems like it is looking for those specific names but works fine with the newer version. This way they will still get updated and managed by the system. |
I like that fix for vivado. As for Questa I'm not able to run the latest version. The 2024 versions require a new license server and James had some issues getting running. |
I'll take bin/vivado-ubuntu24.04-libs.sh out of the PR. Shall we add the symlinks to wally-tool-chain-install.sh? Jordan you probably know best where to put it. |
…lution by symlinking the new libncurses6 and libtinfo6 to the old libraries.
Done in #1046 |
Maybe we merge this for now and then open an issue to track the Questa problem with Ubuntu 24.04 |
That sounds like a good plan. I'll open an issue. |
Resolves issue #1040. Hopefully this time without the ECA error.
Also adds another test which forces the btb to trash. Jump and divide instructions alias to the same location such as btb is forced to miss on every access. Enough accesses are made so the I$ misses during one of these triggering a rare condition.