Skip to content

Update Go packages & support on loongarch64#140

Closed
DWwanghao wants to merge 1 commit intotillitis:mainfrom
DWwanghao:main
Closed

Update Go packages & support on loongarch64#140
DWwanghao wants to merge 1 commit intotillitis:mainfrom
DWwanghao:main

Conversation

@DWwanghao
Copy link
Copy Markdown

Description

Hi
The LoongArch architecture (LoongArch) is an Instruction Set Architecture (ISA) that has a RISC style.
Documentations:
ISA:
https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-EN.html
ABI:
https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-EN.html

During compilation, since the dependency goselectonly added support in version 0.1.3 , we need to update go.mod

Type of change

  • Feature (non breaking change which adds functionality)

Submission checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my changes
  • I have tested and verified my changes on target
  • My changes are well written and CI is passing
  • QEMU is updated to reflect changes

@mchack-work
Copy link
Copy Markdown
Member

mchack-work commented Oct 13, 2025

Thanks for your contribution. We hadn't noticed that there was a new version of goselect. We also hadn't noticed that the repo seems to be archived, which might be of some concern. We'll look into it.

@mchack-work
Copy link
Copy Markdown
Member

Our use of goselect is indirect through the go.bug.st/serial@v1.6.2 package. It's still unchanged in their master branch:

https://github.com/bugst/go-serial/blob/master/go.mod

Current PRs fixing this:

bugst/go-serial#203

bugst/go-serial#206

If anything, the update to goselect should be done there, at least if they're fine with using an archived package.

If they update we will most likely update the serial package, too, and then use the new goselect.

@mchack-work
Copy link
Copy Markdown
Member

Closing since there is nothing for us to do here. We wait for what go-serial does, if they update or if they move away from the archived goselect package.

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