-
Notifications
You must be signed in to change notification settings - Fork 67
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
Allow custom disk size #46
Comments
I'm not quite sure what you mean. Could you clarify the advantage of zero padded binaries? |
I needed to play around with binary sizes a bit for bochs to load my kernel (https://github.com/64/solstice/blob/master/scripts/bochs.sh) so it might be better to support this inside bootimage if possible |
Currently bootimage generates a "just enough" disk size. Qemu treats it as a bootable disk and works as expected. |
I never really used bochs, but sounds reasonable. |
@vinaychandra Thanks for clarifying!
We had something like this before the rewrite, but it was never finished so we removed it again. I think this would need to be implemented in the
So QEMU treats the loaded image as a hard disk, which allows a file system driver to write to the image file? |
I don't know which crate should be implemented where but my point would be to not autoload these extra files (like we do for kernel) but leave them on the bin file so that we can load them dynamically as required.
Yes, that is my understanding. See Image types |
Sounds reasonable. I think it could be implemented by putting it into a separate ELF section that is not loaded. We could pass the file offset in the boot info to tell the kernel the disk location of the file.
Interesting! Note that I'm very busy for the next four weeks, so I won't have time to implement it. |
Thank you for the great tool. Currently, it allows us to create the main bootable image of the kernel but doesn't have too much of empty space left for other actions from within Qemu. Please allow zero padded binaries to play with disk drivers from kernel.
The text was updated successfully, but these errors were encountered: