[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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.


Michal


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]