Comment on confusing firmware attribute

Author: nightkr

roles/create-vm/templates/windows-vm.xml.j2

@@ -10,6 +10,7 @@
   <vcpu placement="static">{{ vm_vcpus }}</vcpu>
   <os
     {% if not vm_firmware_override %}
+      {# firmware=... actually controls *autodetection*, so only set if there is no override #}
       firmware="efi"
     {% endif %}>
     <type arch="x86_64" machine="pc-q35-8.0">hvm</type>