Update documentation, trivial cleanup, change OpenCore prefs #139

kwvg · 2020-07-29T14:40:47Z

Things done

Have passthrough GPU devices share the same port and bus (fixes HDMI audio detection, courtesy of foxlet)
Add gtk initialization failed fix (common on Ubuntu based hosts) for users who are running the QEMU script from SSH
Mention ScanPolicy fix to allow detection of ISOs made from createinstallmedia
Mention RadeonBoost.kext for AMD GPU users
Remove reference to virtualhere (USB passthrough recommended)
Add link to post instructing the enabling of rc.local with systemd (for Ubuntu users)
Adds tweakable variables that can be changed (ALLOCATED_RAM, CPU_SOCKETS, CPU_CORES, CPU_THREADS) and variables that can be defined so that script can be launched from rc.local (REPO_PATH, OVMF_DIR) without relative references (e.g. ./ and ../)
Add instructions for attaching physical disk drives as a raw block device for performance improvement
Add quick SSH setup instructions for single-GPU passthrough setup
Enable verbose mode and set ScanPolicy to 0 by default in OpenCore configs
~~Add algrey's cpuid_set_cache_info - don't set cpuid_cores_per_package patch from AMD-Vanilla to fix odd topology issues (as suggested by Leoyzen here)~~ This doesn't work, more info in commit ed75a20
Stop using Penryn for OpenCore builds (still keeping them for Clover just in case)
Make passthrough script use vmxnet3, rearrange arguments to make them neater to read

Additional Credits

Load VFIO first courtesy of https://github.com/intel/nemu/wiki/Testing-VFIO-with-GPU
The leaf7 and OpenCore -cpu host patch ideas and the NVME controller note were courtesy of Pavo from the AMD-OSX Discord

kholia · 2020-07-30T08:10:25Z

Thank you! <3

It might take me a while to go through these changes and merge them.

kwvg · 2020-07-30T08:27:34Z

Thank you 😊, this is a WIP so there are still a few optimizations I'm working on...

~~EDIT: This PR as is breaks some stuff, working on it...~~ Fixed! It's all yours @kholia!

Things changed - Add adjustable variables in all scripts, make them "any-location launch" ready - Remove "Penryn", inhert host CPU and it's capabilities instead [still shows up as GenuineIntel] - Rearrange arguments to make it easier to read - Add foxlet's passthrough fix for passthrough launch script - Replace e1000-82545em with vmxnet3 in passthrough launch script

Notes: - Patches added for all three OpenCore folders but disabled for Big Sur - The "skip cpuid_cores_per_package test" patches seem to compliment "don't set cpuid_cores_per_package" and "jmp to calculations and set cpuid_cores_per_package" but the first complement causes a crash and with the second one, it refuses to boot altogether. The patches added alone don't seem to fix the odd cores crash issue but I figured it might help so I've added it anyways - Verbose has been enabled by default just to make debugging easier

I've added the expected value, 18809603 as a comment in the plist file just for convenience sake

Changes done: - Added passthrough instructions for AMD - Ensure that VFIO driver loads first (courtesy of https://github.com/intel/nemu/wiki/Testing-VFIO-with-GPU) - Add both initramfs and GRUB bootloader VFIO instructions - Remove VirtualHere and virio-net-osx references - Add 'gtk initialization failed' fix, ScanPolicy fix - Add instructions for attaching a physical block drive - Add basic Ubuntu SSH instructions - Add link to RadeonBoost.kext (improved scores on my system, seems good) Note: Because of changes in previous commits, explicitly specifiying AVX2 is not necessary but specifiying AES-NI for some reason is still necessary, still kept because of inconsistent behaviour

I forgot that they don't have the patches from OpenCore, oops!

kwvg · 2020-08-21T00:51:58Z

Pinging @kholia

kholia · 2020-08-23T06:42:42Z

I have started testing and merging selective changes from this PR into master branch.

kholia · 2020-08-23T06:43:22Z

boot-macOS-HS.sh

@@ -1,32 +1,38 @@
 #!/usr/bin/env bash

-# See https://www.mail-archive.com/qemu-devel@nongnu.org/msg471657.html thread.
+# Special thanks to...


Let's leave the old files unchanged.

Same goes for other scripts meant for older macOS versions.

kholia · 2020-08-23T06:44:36Z

run-diagnostics.sh

@@ -14,41 +14,40 @@ echo "\n== Loaded Modules ==" >&2
 lsmod | grep kvm
 lsmod | grep amd_iommu
 lsmod | grep intel_iommu
-v=`cat /sys/module/kvm/parameters/ignore_msrs`
+v=$(cat /sys/module/kvm/parameters/ignore_msrs)


Incorporated. Thanks.

kholia · 2020-08-23T06:45:42Z

Please rebase on latest master (squash if required). This should help in reviewing this PR.

kholia · 2020-08-23T06:54:09Z

OpenCore-BS/config.plist

@@ -340,16 +340,40 @@
 		<key>Emulate</key>
 		<dict>
 			<key>Cpuid1Data</key>


Were these changes tested with Big Sur?

Note: Every change needs to be tested end-to-end.

They weren't. I couldn't get Big Sur to work due to problems on my end, sorry about that. I figured changes from Catalina should translate over

kholia · 2020-08-23T07:01:11Z

OpenCore-BS/config.plist

@@ -516,7 +582,8 @@
 			<key>HaltLevel</key>
 			<integer>2147483648</integer>
 			<key>ScanPolicy</key>
-			<integer>18809603</integer>
+			<integer>0</integer>
+			<!-- <integer>18809603</integer> -->


This (commented stuff) is not valid for OpenCore 0.6.0.

~~OpenCore doesn't like the comment or doesn't like 18809603?~~
Just checked with the commit, it doesn't like the comment.

Are there any problems with the additional algrey patches? The leaf7 patch was really useful to get AVX working on my AMD system w/o manually specifying it.

kholia · 2020-08-23T07:07:33Z

OpenCore-Boot.sh

-	  -netdev tap,id=net0,ifname=tap0,script=no,downscript=no -device vmxnet3,netdev=net0,id=net0,mac=52:54:00:c9:18:27 \
-	  -monitor stdio \
-	  -vga vmware
+ALLOCATED_RAM="3072" # MiB


Incorporated. Thanks.

kholia · 2020-08-23T08:42:48Z

notes.md

@@ -233,7 +233,7 @@ Add `-vga vmware` to QEMU parameters in the booot script (e.g. boot-macOS.sh), i

 *Warning: The OpenCore distribution that comes with OSX-KVM already has AppleALC, do not mix VoodooHDA with AppleALC. You may want to consider HDA passthrough if it is practical or use HDMI audio instead*

-No support is provided for this. You are on your own. The sound output is known to be choppy and distorted.
+No support is provided for this. You are on your own. The sound output is known to be choppy and distorted. *(Though this can be mitigated using [spheenik's version of qemu](https://github.com/spheenik/qemu/tree/3.1.0-patched) but it has not been thoroughly tested)*


We cannot recommend unmaintained and very likely non-working versions of QEMU.

There is a branch with QEMU 4.0 (https://github.com/spheenik/qemu/tree/4.0-glorious) but it should probably be omitted because we don't know how it'll behave

kwvg · 2020-08-24T13:54:39Z

Replaced by #142, closed

kwvg changed the title ~~Update NOTES.md and standardize QEMU launch scripts~~ Update NOTES.md, standardize QEMU launch scripts, fix topology freeze Jul 29, 2020

kwvg added 5 commits July 30, 2020 19:15

Set ScanPolicy to 0 for detection of createinstallmedia images

fedcbf0

I've added the expected value, 18809603 as a comment in the plist file just for convenience sake

Put Penryn back for scripts that rely on Clover

8f5ddab

I forgot that they don't have the patches from OpenCore, oops!

kwvg changed the title ~~Update NOTES.md, standardize QEMU launch scripts, fix topology freeze~~ Update documentation, trivial cleanup, change OpenCore prefs Jul 30, 2020

kholia self-requested a review August 1, 2020 09:00

kwvg and others added 2 commits August 1, 2020 22:17

Fix markdown and add link to potential audio glitch mitigation

cb8ad39

Fix markdown

94e87ef

kwvg marked this pull request as draft August 20, 2020 13:12

kholia force-pushed the master branch 6 times, most recently from b417e6f to 5e2fff2 Compare August 23, 2020 06:41

kholia reviewed Aug 23, 2020

View reviewed changes

kholia force-pushed the master branch from 5e2fff2 to 9a4dae6 Compare August 23, 2020 07:07

kholia reviewed Aug 23, 2020

View reviewed changes

kholia force-pushed the master branch 2 times, most recently from d079513 to 9bc8ca9 Compare August 23, 2020 07:28

kholia force-pushed the master branch from 9bc8ca9 to ca9e6f3 Compare August 23, 2020 07:30

kholia reviewed Aug 23, 2020

View reviewed changes

kholia force-pushed the master branch from 238f0bc to 7029844 Compare August 23, 2020 14:14

kwvg mentioned this pull request Aug 24, 2020

Add algrey patches, passthrough fix, make inheritance easier #142

Merged

kwvg closed this Aug 24, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update documentation, trivial cleanup, change OpenCore prefs #139

Update documentation, trivial cleanup, change OpenCore prefs #139

kwvg commented Jul 29, 2020 •

edited

Loading

kholia commented Jul 30, 2020

kwvg commented Jul 30, 2020 •

edited

Loading

kwvg commented Aug 21, 2020

kholia commented Aug 23, 2020

kholia Aug 23, 2020 •

edited

Loading

kholia Aug 23, 2020

kholia commented Aug 23, 2020

kholia Aug 23, 2020

kholia Aug 23, 2020

kwvg Aug 24, 2020

kholia Aug 23, 2020

kwvg Aug 24, 2020 •

edited

Loading

kholia Aug 23, 2020

kholia Aug 23, 2020

kwvg Aug 24, 2020

kwvg commented Aug 24, 2020

Update documentation, trivial cleanup, change OpenCore prefs #139

Update documentation, trivial cleanup, change OpenCore prefs #139

Conversation

kwvg commented Jul 29, 2020 • edited Loading

kholia commented Jul 30, 2020

kwvg commented Jul 30, 2020 • edited Loading

kwvg commented Aug 21, 2020

kholia commented Aug 23, 2020

kholia Aug 23, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kholia commented Aug 23, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kwvg Aug 24, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kwvg commented Aug 24, 2020

kwvg commented Jul 29, 2020 •

edited

Loading

kwvg commented Jul 30, 2020 •

edited

Loading

kholia Aug 23, 2020 •

edited

Loading

kwvg Aug 24, 2020 •

edited

Loading