Yeah, looks fine.
A few nitpicks:
The title seems too long. You can just say "Sienna Cichlid", as
opposed to using the macro name in the title.
Perhaps you can add Sienna Cichlid in the description of the
commit as well.
On
In passthrough mode, if we unloaded driver in BACO mode(RTPM). Kernel would
receive thousands of unhandled interrupts. That's because there was
doorbell moniter interrupt on BIF, so KVM would keep injecting
interrupt to guest VM. So we should clear door bell interrupt status
after BACO exit.
