Anthony Liguori wrote: > Part of the feedback we received from Fabrice about the KVM patches for QEMU > is that we should create a separate device for the in-kernel APIC to avoid > having lots of if (kvm_enabled()) within the APIC code that were difficult to > understand why there were needed. > > This patch separates the in-kernel PIT into a separate device. It also > introduces some configure logic to only compile in support for the in-kernel > PIT if it's available. > > The result of this is that we now only need a single if (kvm_enabled()) to > determine which device to use. Besides making it more upstream friendly, I > think this makes the code much easier to understand. > >
It introduces a new issue, the save/restore format is effectively forked and will have to be maintained in parallel if there are any changes. Perhaps keep in the same file, but as two separate devices that can share the save/restore code? -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel
