Since the vfu_log call for whether MSI-X is enabled or not is using the not yet updated msix->mxc.mxe, it will always print the opposite value. ("enable MSI-X" <-> "disable MSI-X")
Instead of changing it to new_msix.mxc.mxe, I simply moved the assignment of both the mxe and the unaffected fm field before their log calls, so both variants are fine to use.
This is also how other functions like handle_px_pxdc_write do it, assignment before log.
Since the vfu_log call for whether MSI-X is enabled or not is using the not yet updated
msix->mxc.mxe
, it will always print the opposite value. ("enable MSI-X" <-> "disable MSI-X")Instead of changing it to
new_msix.mxc.mxe
, I simply moved the assignment of both themxe
and the unaffectedfm
field before their log calls, so both variants are fine to use.This is also how other functions like
handle_px_pxdc_write
do it, assignment before log.