On Wed, 2011-11-09 at 14:25 +0200, Pekka Enberg wrote: > On Wed, 9 Nov 2011, Michael S. Tsirkin wrote: > >> KVM tool actually has support for 64bit features, we can probably remove > >> that when Pekka isn't looking :) > > > > It's not yet released so maybe it's not an issue yet. > > If it's too late I can re-add them to legacy too. > > > > Pekka, 64 features aren't yet used and we are discussing > > changing the layout for that field. Mind taking it out > > of kvm tool for now? > > Sasha, why did we add 64-bit features to the KVM tool? Wasn't it part of > the virtio spec? Does QEMU not use them? How badly will older versions of > the KVM tool break if you drop 64-bit features? We added 64-bit features to the tool because it just got into the spec when we rewrote our virtio-pci handling - so we just implemented the updated spec. QEMU doesn't use them since while it did get into the spec, it was intended to future-proof the limited feature bits, so no one really needed them yet. They don't exist in kernel code either, for same reason as above. Nothing will break if we remove it since no one really used it, we were probably the first and only implementation of the spec which considered them :) -- Sasha. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
