lkml.org 
[lkml]   [2011]   [Nov]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH RFC] virtio-spec: flexible configuration layout
    From
    Date
    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.



    \
     
     \ /
      Last update: 2011-11-09 13:33    [W:4.097 / U:0.156 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site