Skip to content

Update Dreamcast port#453

Merged
sahlberg merged 1 commit intosahlberg:masterfrom
pcercuei:dreamcast-vfs
Mar 23, 2026
Merged

Update Dreamcast port#453
sahlberg merged 1 commit intosahlberg:masterfrom
pcercuei:dreamcast-vfs

Conversation

@pcercuei
Copy link
Copy Markdown
Contributor

  • The CMAKE_SYSTEM_NAME is defined as lowercase 'dreamcast' by KallistiOS' CMake scripts.
  • Added some glue code to allow mapping a SMB server into KallistiOS' virtual filesystem. Users now just need to call kos_smb_init() with a server URL as the argument, to have it mounted as /smb in the VFS.

- The CMAKE_SYSTEM_NAME is defined as lowercase 'dreamcast' by
  KallistiOS' CMake scripts.
- Added some glue code to allow mapping a SMB server into KallistiOS'
  virtual filesystem. Users now just need to call kos_smb_init() with a
  server URL as the argument, to have it mounted as /smb in the VFS.

Signed-off-by: Paul Cercueil <paul@crapouillou.net>
@sahlberg sahlberg merged commit 882fdb0 into sahlberg:master Mar 23, 2026
14 of 16 checks passed
@sahlberg
Copy link
Copy Markdown
Owner

Merged.
I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

@pcercuei
Copy link
Copy Markdown
Contributor Author

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Up to you, if you prefer we can have it in KallistiOS directly.

@sahlberg
Copy link
Copy Markdown
Owner

It is in a separate dedicated subdirectory so I have no issue with it being in libsmb2.

I was thinking more about discoverability for the KallistoOS developers.
It is easier to discover that this feature exists and is available to devs if it is in the sdk than in a separate third party library.

But then, on the other hand, I imagine the KallistoOS is similar to the PS2 community. Maybe centered around a dedicated forum so all potential developers can already discover this features availability via the forum.
So maybe it is a moot point.

@darcagn
Copy link
Copy Markdown
Contributor

darcagn commented Mar 23, 2026

KallistiOS has a maintained repository of add-on libraries. libsmb2 is included, so it's discoverable to our users.

@Wolf3s
Copy link
Copy Markdown
Contributor

Wolf3s commented Mar 24, 2026

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Up to you, if you prefer we can have it in KallistiOS directly.

We have vfs aswell for ps2, but it's wLaunchelf. i'm implementing for a fork of OPL It should be awesome port this guy to other platforms.

@sahlberg
Copy link
Copy Markdown
Owner

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Up to you, if you prefer we can have it in KallistiOS directly.

We have vfs aswell for ps2, but it's wLaunchelf. i'm implementing for a fork of OPL It should be awesome port this guy to other platforms.

I have no objections.
If someone wants to make this into a general wrapper for embedded systems, I will not object to a PR.

@Wolf3s
Copy link
Copy Markdown
Contributor

Wolf3s commented Mar 24, 2026

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Merged. I checked it in but wonder if vfs.c/h shouldn't rather be in the sdk for kallistoos instead of libsmb2.

Up to you, if you prefer we can have it in KallistiOS directly.

We have vfs aswell for ps2, but it's wLaunchelf. i'm implementing for a fork of OPL It should be awesome port this guy to other platforms.

I have no objections. If someone wants to make this into a general wrapper for embedded systems, I will not object to a PR.

It should be more easier than implementing usmb2 to opl for streaming games, i will take a look at it, when i have some time.

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.

4 participants