Re: [PATCH v1 09/11] network: Allow class ID to be reused

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

 



On 30.11.2012 21:55, Laine Stump wrote:
> On 11/19/2012 11:51 AM, Michal Privoznik wrote:
>> With current implementation, class ID is just incremented. This can
>> lead to its exhaustion as tc accepts only 16 bits long identifiers.
>> Hence, it's better if we allow class ID to be reused. To keep track
>> which IDs are free and which are taken virBitmap is used. This requires
>> network status file to change a bit: from <class_id next="5"/> to
>> <class_id bitmap="0-4"/>. But since the previous format hasn't been
>> released, it doesn't really matter.
> 
> Heh. Well, there you have it. :-) You've already implemented what I
> suggested in the review of 5/10. But rather than introducing one
> implementation that we need to review, then almost immediately replacing
> it with something else, why not just implement it this way to begin with?

Because I think 5/10 is big enough already :)

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]