Skip to content

Conversation

Korin777
Copy link
Contributor

change resolution from 800x480 to 400x240, speed up from 15s/frame to 7s/frame

@jserv
Copy link
Contributor

jserv commented Jan 26, 2022

You should make use of git rebase -i to rework the commits. Only single commit is required.

@jserv
Copy link
Contributor

jserv commented Jan 26, 2022

You should run strip (in fact, riscv-none-embed-strip -s) to get smaller ELF executable file.

@Korin777
Copy link
Contributor Author

You should make use of git rebase -i to rework the commits. Only single commit is required.

I am trying to use it

@Korin777
Copy link
Contributor Author

You should run strip (in fact, riscv-none-embed-strip -s) to get smaller ELF executable file.

If I use riscv-none-embed-strip -s or riscv64-unknown-elf-strip -s to quake or doom, the game will not display when runing on rv32emu-next, it seems the program stuck in somwhere。

@Korin777 Korin777 closed this Jan 26, 2022
@Korin777 Korin777 reopened this Jan 26, 2022
@jserv
Copy link
Contributor

jserv commented Jan 26, 2022

If I use riscv-none-embed-strip -s or riscv64-unknown-elf-strip -s to quake or doom, the game will not display when runing on rv32emu-next, it seems the program stuck in somwhere。

It would be great if we can look into this later.

Copy link
Contributor

@jserv jserv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Run resolve the conflicts and do git push --force

@jserv jserv changed the title Change resolution to speed up execution time build/quake: Lower resolution to speed up game execution Jan 26, 2022
change resolution form 800x480 to 400x240,speed up from 15s/frame to 7s/frame
@jserv jserv merged commit bec2d13 into sysprog21:master Jan 26, 2022
jserv added a commit that referenced this pull request Sep 2, 2024
Recently, we have been transitioning from using curl to wget for
downloading files from remote hosts. However, the version of wget
provided by Alpine Linux is actually a Busybox implementation, which
lacks many features found in the full wget package.

Error messages:
  #15 [linux/amd64 base_gcc 5/6] RUN make ENABLE_SDL=0
  #15 0.207 Check the file build/.config for configured items.
  #15 0.360 Fetching prebuilt executables from "rv32emu-prebuilt" ...
  #15 0.361 wget: unrecognized option: show-progress

This commit attempts to install wget package instead.
vestata pushed a commit to vestata/rv32emu that referenced this pull request Jan 24, 2025
The resolution of Quake has been changed form 800x480 to 400x240,
resulting speedup from 15s/frame to 7s/frame.
vestata pushed a commit to vestata/rv32emu that referenced this pull request Jan 24, 2025
Recently, we have been transitioning from using curl to wget for
downloading files from remote hosts. However, the version of wget
provided by Alpine Linux is actually a Busybox implementation, which
lacks many features found in the full wget package.

Error messages:
  sysprog21#15 [linux/amd64 base_gcc 5/6] RUN make ENABLE_SDL=0
  sysprog21#15 0.207 Check the file build/.config for configured items.
  sysprog21#15 0.360 Fetching prebuilt executables from "rv32emu-prebuilt" ...
  sysprog21#15 0.361 wget: unrecognized option: show-progress

This commit attempts to install wget package instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants