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

Make sata/IntelAHCI the default virtual disk controller #4632

Merged
merged 2 commits into from
Feb 22, 2022

Conversation

computezrmle
Copy link
Contributor

Description of the Change
Vboxwrapper configures the default virtual disk controller as ide/PIIX4.

According to the VirtualBox manual sata/IntelAHCI should be the default for different reasons:
https://www.virtualbox.org/manual/ch05.html#harddiskcontrollers

The same chapter from an older VirtualBox manual (5.0.0) is a bit more verbose:
https://download.virtualbox.org/virtualbox/5.0.0/UserManual.pdf#section.5.1

"VirtualBox’s virtual SATA controller operates faster and also
consumes fewer CPU resources than the virtual IDE controller"

"...starting with version 3.2 and depending on the selected guest operating
system, VirtualBox uses SATA as the default for newly created virtual machines"

"The entire SATA controller ... will not be seen by operating systems that do not
have device support for AHCI. In particular, there is no support for AHCI in Windows
before Windows Vista, so Windows XP (even SP3) will not see such disks unless you
install additional drivers"

It should be save not to focus on the last comment since I'm not aware of a BOINC project running Windows XP as VirtualBox guest.
Even then the disk controller could be be set back to ide/PIIX4 via vbox_job.xml.

Existing Linux VMs from LHC@home and Rosetta@home are running fine with sata/IntelAHCI.

Alternate Designs
N/A

Release Notes
vboxwrapper: change default virtual disk controller from ide/PIIX4 to sata/IntelAHCI

Client and Manager automation moved this from Backlog to Testing Feb 22, 2022
@AenBleidd
Copy link
Member

@computezrmle, please resolve conflicts in your branch, then I'll be able to merge it

@AenBleidd AenBleidd removed this from Testing in Client and Manager Feb 22, 2022
@AenBleidd AenBleidd added this to In progress in Client Release 7.20.0 via automation Feb 22, 2022
@AenBleidd AenBleidd merged commit c347a42 into BOINC:master Feb 22, 2022
Client Release 7.20.0 automation moved this from In progress to Done Feb 22, 2022
@computezrmle computezrmle deleted the default_disk_controller branch February 22, 2022 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants