]> www.infradead.org Git - users/dwmw2/qemu.git/commit
hw/xen: Clean up event channel 'type_val' handling to use union
authorDavid Woodhouse <dwmw@amazon.co.uk>
Wed, 2 Aug 2023 16:04:49 +0000 (17:04 +0100)
committerDavid Woodhouse <dwmw@amazon.co.uk>
Mon, 16 Oct 2023 08:47:40 +0000 (09:47 +0100)
commit63691c20bfacdae0e92eb9c7d712048e03588c28
tree09593cfaa802308ffb315a06373006b251a13d09
parent970fdc79e6cd813bc64bd93cf442fc34ce656336
hw/xen: Clean up event channel 'type_val' handling to use union

A previous implementation of this stuff used a 64-bit field for all of
the port information (vcpu/type/type_val) and did atomic exchanges on
them. When I implemented that in Qemu I regretted my life choices and
just kept it simple with locking instead.

So there's no need for the XenEvtchnPort to be so simplistic. We can
use a union for the pirq/virq/interdomain information, which lets us
keep a separate bit for the 'remote domain' in interdomain ports. A
single bit is enough since the only possible targets are loopback or
qemu itself.

So now we can ditch PORT_INFO_TYPEVAL_REMOTE_QEMU and the horrid
manual masking, although the in-memory representation is identical
so there's no change in the saved state ABI.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
Reviewed-by: Paul Durrant <paul@xen.org>
hw/i386/kvm/xen_evtchn.c