RE: git: 2eb2079554f4 - main - bhyve: keep physical and virtual COMMAND reg in sync

From: Corvin Köhne <C.Koehne_at_beckhoff.com>
Date: Thu, 25 Nov 2021 08:58:05 UTC
Hi John,

> While it's true that real systems do that on reset, most firmware set those bits
> while populating BARs.  Bhyve doesn't always run boot firmware to do those tasks
> and needs to provide working devices out of the box.  This is similar to how
> bhyve pre-populates BARs with resources rather than relying on firmware to
> assign resources to BARs.
>
> --
> John Baldwin

Sry, I forgot it. I'm always running bhyve with OVMF. So I won't see such issues.
Enabling bus mastering shouldn't hurt. Whether bhyve runs boot firmware or not.
Should be a good idea to always enable it on start up.
I'm unsure if it could cause trouble when enabling INTx by default. I'm missing some
insights how bhyve's INTx support works. If you say that we should enable it by
default, I'll trust you.

What's your plan? Do you want to update your patch or do you want to wait until
someone is facing some issues due to disabled bus mastering on startup?
If you like, I can update your patch too.


Best regards
Corvin

Beckhoff Automation GmbH & Co. KG | Managing Director: Dipl. Phys. Hans Beckhoff
Registered office: Verl, Germany | Register court: Guetersloh HRA 7075