Skip to content

Conversation

@pankoza2-pl
Copy link
Contributor

@pankoza2-pl pankoza2-pl commented Sep 9, 2023

Summary

(hopefully) Add back the Gateway 2000 Venus i440FX Socket 8 Board (OEM Version of Intel VS440FX)

Checklist

References

Provide links to datasheets or other documentation that helped you implement this pull request.
https://theretroweb.com/motherboards/s/intel-vs440fx-venus

sasamiya-bios and others added 21 commits August 27, 2023 13:40
This commit fixes a build issue that occurred when compiling with
GDBSTUB support (-DGDBSTUB=ON). The issue was introduced in commit
565421a, which commented out the GDB_REG_FS_BASE and GDB_REG_GS_BASE
registers, but did not remove the code snippets that referenced them
in gdbstub_client_write_reg(). This commit comments out those code
snippets as well.

Additionally, this commit removes the fs_base and gs_base registers
from the XML file, to ensure the correct numbering of the registers
for the GDB.
Fix GDB stub builds due to undeclared symbol
…adInfo' (86Box#3666)

* Fix out-of-bounds access on gdbstub_client_respond()

The issue happens when the ENABLE_GDBSTUB_LOG flag is set, more
specifically within the gdbstub_client_respond() function. This is
due to the fact that the string 'GDB Stub: Sending response: ' and
the newline character '\n' total 29 characters. When combined with
the 995 characters of client->response, the total is 1024 bytes.
However, the pclog_ex() buffer size is also 1024 bytes, leaving no
room for the null terminator '\0'. As a result, when attempting to
print the text on the screen, a segmentation fault occurs.

This commit fixes this by decreasing the response text's size by one
byte.

* Remove extra space in 'qfThreadInfo' packet response in gdbstub.c

When GDB sends the 'qfThreadInfo' packet, one of the possible responses
is 'm thread-id', as specified in [1]. However, contrary to what the
documentation implies, there is no space between 'm' and the thread-id.
In the current approach, GDB isn't even able to recognize that there's
any active thread, as the code sends "m 1" instead of "m1".

This commit addresses this by removing the space in the response.

Ref:
[1]: https://sourceware.org/gdb/onlinedocs/gdb/General-Query-Packets.html
…anding level-triggered IRQ's, fixes IDE hard disk and ATAPI CD-ROM problems.
Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
…ions/actions/checkout-4

Bump actions/checkout from 3 to 4
Add new 4.0 dependencies to RPM spec file
@OBattler OBattler changed the base branch from master to version/4.1 September 21, 2023 18:41
@OBattler OBattler merged commit 4475a85 into 86Box:version/4.1 Sep 21, 2023
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.

7 participants