Skip to content
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

Add support for Xbox 360 XDVDFS images #94

Merged
merged 1 commit into from
Sep 15, 2024
Merged

Conversation

astarivi
Copy link
Contributor

@astarivi astarivi commented Sep 13, 2024

Adds XGD2 and XGD3 offsets to support Xbox 360 XDVDFS image unpacking. So far, XGD2 and XGD3 support has been tested with the following Redump verified dumps, from my own collection:

XGD2:

  • Battle Fantasia
  • Halo 3
  • Halo Reach
  • Forza Motorsport 2
  • Forza Motorsport 3
  • エスカトス
  • 電脳戦機バーチャロン フォース

XGD3:

  • Titanfall
  • Battlefield 3, Disc 2

All images from the JAP, or WORLD region; all images produced a valid MD5 repack.

More testing is needed with known "problematic" images, like games with long paths, long filenames, .xex files at the end of the image, etc. Sadly, I don't happen to own any game that fits the description.

@astarivi astarivi marked this pull request as draft September 13, 2024 21:43
@antangelo
Copy link
Owner

If this is all that is needed to support Xbox 360 images then it seems reasonable to me. I do not have any 360 discs or hardware, so I don't really have any ability or interest to test or support this myself. Are you willing to support/field any issues that arise from having this functionality?

I don't know if there are any differences between the filesystem structure for Xbox 360 images as opposed to ones on the OG Xbox. You can add additional test cases to the suite for long paths or certain filenames if they are appropriate. The suite doesn't currently have a way to replicate image binary quirks, it just relies on xdvdfs' own packing mechanism to generate the structure to avoid storing binary blobs in the repo.

@astarivi
Copy link
Contributor Author

I have all that is needed to test this feature, and I fully intend to support this feature.

About the 360's XDVDFS version, I also do not know if there are any differences; documentation on this is lacking, although all images that I tested have created valid, fully playable results.

Other XISO tools seem to treat the FS on 360 images the same as regular OG Xbox images. I say it is safe to assume them as compatible filesystems for now.

I believe current testing cases are more than sufficient, if treating both Xbox and Xbox 360 images as the same filesystem. I tested the cases I previously mentioned, and there were no issues, either.

Nevertheless, I am not against adding more testing cases specialized on Xbox 360 images, but I believe it is preferable to wait for actual problems to emerge with Xbox 360 support to start testing for them.

@astarivi astarivi marked this pull request as ready for review September 14, 2024 23:24
@antangelo antangelo merged commit 9278a07 into antangelo:main Sep 15, 2024
12 checks passed
@antangelo
Copy link
Owner

Sounds good to me. Thanks!

@astarivi astarivi deleted the xbox360 branch September 15, 2024 18:47
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