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

Proper DVD Emulation #16

Open
remlei opened this issue Jan 24, 2025 · 2 comments
Open

Proper DVD Emulation #16

remlei opened this issue Jan 24, 2025 · 2 comments
Assignees

Comments

@remlei
Copy link

remlei commented Jan 24, 2025

As you may now, Vanilla WIndows ISOs does (windows 7 or later) does not work with gadget_cdrom, for some reason, once the windows iso boots up and goes to the GUI section, the cdrom suddenly goes poof out of existence, making the windows pe environment to install windows complaining for missing installation files. There's a workaround for it so OK I guess.

Fast forward today, people from pikvm finally able to handle DVD ISO and able to install windows from start to finish for using a Vanilla Windows ISO. Based on this issue they were able to achieve it.

I hope you can port the changes on pikvm to here.

@chrisirwin
Copy link

FWIW, the issue doesn't link to any commits.

The commits to pikvm's pikvm and kvmd projects are mostly just changing terms (CD-ROM -> CD/DVD) and adjusting warning messages:

The real magic seems to be a kernel patch applied to their 'packages' project (which makes sense, as this was a kernel limitation):

This may or may not apply cleanly to the kernel being used by gadget_cdrom. Additionally, I haven't researched the lineage of this patch. It may be written by pikvm themselves, as there doesn't seem to be any other attribution.

@mdevaev
Copy link

mdevaev commented Jan 26, 2025

@chrisirwin Hello, I'm PiKVM dev. I saw the link to my Issue and decided to check that here :)

I wrote this patch myself based on few sources:

I tried to leave as many comments as possible to make it easier for people to follow me. I plan to bring this patch to the upstream, but we need to get rid of 0xff in SCSI parameter validation firstly and add some credits. I've been testing this implementation for a long time and it seems that it works in all important cases, I was even able to install windows on a mac mini using the official ISO.

@tjmnmk tjmnmk self-assigned this Jan 30, 2025
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

No branches or pull requests

4 participants