Skip to content

remove redundant w opt with ignore_msr#587

Open
ke5C2Fin wants to merge 1 commit intochurchers:masterfrom
ke5C2Fin:w
Open

remove redundant w opt with ignore_msr#587
ke5C2Fin wants to merge 1 commit intochurchers:masterfrom
ke5C2Fin:w

Conversation

@ke5C2Fin
Copy link

With merged PR #525, we add -w to the default _opts on line 112 which makes checking config::yesno "ignore_msr" and conditionally adding w via && _opts="${_opts}w" on line 115 redundant.

Does something else need to be done for Model Specific Registers?
With the addition of w as default, nothing is then changed for msr, but I don't know what would be.

@bileslav
Copy link

bileslav commented Jan 31, 2025

Hi @ke5C2Fin,

In the vm.8 and config.sample there is no ignore_msr, but there is ignore_bad_msr. My proposal:

  1. Fix the man page. In particular, reflect that ignore_bad_msr default (yes) is not limited to UEFI guests only.
  2. Use this code:
# default bhyve options
_opts="-AHP"

# ignore access to unimplemented Model Specific Registers?
config::yesno "ignore_bad_msr" yes && _opts="${_opts}w"

humbly summoning @mateuszkwiatkowski

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

Successfully merging this pull request may close these issues.

2 participants