On Tue, Aug 29, 2017 at 17:05:23 +0200, Michal Privoznik wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1484234 > > Turns out, only vhostuser type of interfaces are supported > currently. > > Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; > --- > > I'd like to get this one in since the whole feature is being introduced in this > release. > > docs/formatdomain.html.in | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in > index 19e869f12..8ca7637a4 100644 > --- a/docs/formatdomain.html.in > +++ b/docs/formatdomain.html.in > @@ -5213,7 +5213,10 @@ qemu-kvm -net nic,model=? /dev/null > The default value is hypervisor dependent and may change > across its releases. Moreover, some hypervisors may pose > some restrictions on actual value. For instance, QEMU > - v2.9 requires value to be a power of two from [256, 1024] range. > + v2.9 requires value to be a power of two from [256, 1024] > + range. In addition to that, this may work only for a subset of > + interface types, e.g. aforementioned QEMU enables this option > + only for <code>vhostuser</code> type. > <span class="since">Since 3.7.0 (QEMU and KVM only)</span><br/><br/> > > <b>In general you should leave this option alone, unless you So it's getting even more riddiculous. I should have NACKed the feature.

ACK, safe for freeze.

On Wed, Aug 30, 2017 at 10:18:17 +0200, Michal Privoznik wrote: > On 08/30/2017 10:03 AM, Peter Krempa wrote: >> On Tue, Aug 29, 2017 at 17:05:23 +0200, Michal Privoznik wrote: >>> https://bugzilla.redhat.com/show_bug.cgi?id=1484234 >>> >>> Turns out, only vhostuser type of interfaces are supported >>> currently. >>> >>> Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; >>> --- >>> >>> I'd like to get this one in since the whole feature is being introduced in this >>> release. >>> >>> docs/formatdomain.html.in | 5 ++++- >>> 1 file changed, 4 insertions(+), 1 deletion(-) >>> >>> diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in >>> index 19e869f12..8ca7637a4 100644 >>> --- a/docs/formatdomain.html.in >>> +++ b/docs/formatdomain.html.in >>> @@ -5213,7 +5213,10 @@ qemu-kvm -net nic,model=? /dev/null >>> The default value is hypervisor dependent and may change >>> across its releases. Moreover, some hypervisors may pose >>> some restrictions on actual value. For instance, QEMU >>> - v2.9 requires value to be a power of two from [256, 1024] range. >>> + v2.9 requires value to be a power of two from [256, 1024] >>> + range. In addition to that, this may work only for a subset of >>> + interface types, e.g. aforementioned QEMU enables this option >>> + only for <code>vhostuser</code> type. >>> <span class="since">Since 3.7.0 (QEMU and KVM only)</span><br/><br/> >>> >>> <b>In general you should leave this option alone, unless you >> >> So it's getting even more riddiculous. I should have NACKed the feature. > > Why ridiculous? Some features work only for some devices. By the same > token, some features work only for some hypervisors if you see where I'm > going. Ridiculous is the fact, that the documentation blob only states when it does not work, which also apparently was incomplete, while there is no absolutely no guidance on how to set it up.