[libvirt] [PATCH] Revert "networkAllocateActualDevice: Set QoS for bridgeless networks too"

[Daniel P. Berrange]

On Wed, Jan 29, 2014 at 07:03:10PM +0100, Michal Privoznik wrote:
> On 29.01.2014 18:48, Daniel P. Berrange wrote:
> > On Wed, Jan 29, 2014 at 06:44:22PM +0100, Michal Privoznik wrote:
> >> This reverts commit 2996e6be19a13199ded7c2aa21039cca97318e01
> >> and some parts of 2636dc8c4de83cd37bc0680a6fbc3f6d25023bd7.
> >>
> >> Conflicts:
> >> 	tests/virnetdevbandwidthtest.c: New test has been introduced since
> >>      then.
> >>
> > 
> > Be useful if the commit message said why we're reverting it.
> > 
> > Regards,
> > Daniel
> > 
> 
> How about:
> 
>     Revert "networkAllocateActualDevice: Set QoS for bridgeless networks too"
>     
>     This reverts commit 2996e6be19a13199ded7c2aa21039cca97318e01
>     and some parts of 2636dc8c4de83cd37bc0680a6fbc3f6d25023bd7.
>     
>     The former one tried to implement QoS setting on bridgeless networks.
>     However, as discussed upstream [1], the patch is far away from being
>     useful in even a single case. The whole idea of network QoS is to have
>     aggregated limits over several interfaces. This patch is doing
>     completely the opposite when merging two QoS settings (from the network
>     and the domain interface) into one which is then set at the domain
>     interface itself, not the network.
>     
>     The latter one is the test for the previous one. Now none of them makes
>     sense.
>     
>     1: https://www.redhat.com/archives/libvir-list/2014-January/msg01441.html
>     
>     Conflicts:
>         tests/virnetdevbandwidthtest.c: New test has been introduced since
>         then.

ACK, that's useful explanation

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|