vt_vga: use textmode when we're running on hypervisor

[kib]

The idea is correct, but implementation is arguably not. TUNABLE_XXX_FETCH() keeps the variable unchanged if the env var is not present. The correct logic would be

textmode = vm_guest != VM_GUEST_NO;
TUNABLE_INT_FETCH("hw.vga.textmode", &textmode);

This would give the desired behaviour of enabling textmode by default for guests only, but also allowing user to override it as she wants.

[decui_microsoft.com]

In D5739#122559, @kib wrote:

The idea is correct, but implementation is arguably not. TUNABLE_XXX_FETCH() keeps the variable unchanged if the env var is not present. The correct logic would be

textmode = vm_guest != VM_GUEST_NO;
TUNABLE_INT_FETCH("hw.vga.textmode", &textmode);

This would give the desired behaviour of enabling textmode by default for guests only, but also allowing user to override it as she wants.

@kib, Thanks! I'll update the patch as you suggested.

[decui_microsoft.com]

I improved the patch as kib suggested.

[kib]

sys/dev/vt/hw/vga/vt_vga.c

1237 ↗

(On Diff #14602)

You do not need braces there, and style(9) requests to not put redundand braces.

[decui_microsoft.com]

removed the unnecessary braces as kib suggested.

[decui_microsoft.com]

sys/dev/vt/hw/vga/vt_vga.c

1237 ↗

(On Diff #14602)

OK. I updated the patch.