Re: [PATCH v2 1/2] virtportallocator: new function "virPortAllocatorSetUsed"

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

 



Michal Privoznik <mprivozn@xxxxxxxxxx> writes:

> On 24.06.2014 13:34, Giuseppe Scrivano wrote:
>> virPortAllocatorSetUsed permits to set a port as already used and
>> prevent the port allocator to use it without any attempt to bind it.
>>
>> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx>
>> ---
>>   src/libvirt_private.syms    |  1 +
>>   src/util/virportallocator.c | 44 +++++++++++++++++++++++++++++++++++++++++++-
>>   src/util/virportallocator.h |  4 ++++
>>   3 files changed, 48 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
>> index 2a2b9c0..95a45aa 100644
>> --- a/src/libvirt_private.syms
>> +++ b/src/libvirt_private.syms
>> @@ -1770,6 +1770,7 @@ virPidFileWritePath;
>>   virPortAllocatorAcquire;
>>   virPortAllocatorNew;
>>   virPortAllocatorRelease;
>> +virPortAllocatorSetUsed;
>>
>>
>>   # util/virprocess.h
>> diff --git a/src/util/virportallocator.c b/src/util/virportallocator.c
>> index b68133a..226f386 100644
>> --- a/src/util/virportallocator.c
>> +++ b/src/util/virportallocator.c
>> @@ -1,7 +1,7 @@
>>   /*
>>    * virportallocator.c: Allocate & track TCP port allocations
>>    *
>> - * Copyright (C) 2013 Red Hat, Inc.
>> + * Copyright (C) 2013, 2014 Red Hat, Inc.
>
> How about 2013-2014?

>> +    else {
>

I've no write access to the repo.  Could these fixes be amended before
pushing the patches or shall I go for v3?

Thanks,
Giuseppe

--
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]