[libvirt] [PATCH 03/10] util: storagefile: Introduce helper to free storage source perms

Peter Krempa pkrempa at redhat.com
Fri Jun 20 06:53:29 UTC 2014


On 06/19/14 20:57, John Ferlan wrote:
> 
> 
> On 06/19/2014 09:46 AM, Peter Krempa wrote:
>> It will also be reused later.
>> ---
>>  src/util/virstoragefile.c | 16 ++++++++++++----
>>  1 file changed, 12 insertions(+), 4 deletions(-)
>>
> 
> 
> Why is it that patches 1-3 do not update libvirt_private.syms to add new
> src/util/vir* APIs, but some of the future patches do update, such as 4,

The functions added in 1-3 are used only in the utils module, especially
in the storage source deep copy function and are not linked to from
other modules and therefore the symbol export isn't really necessary.

I might add them if you insist, but I don't expect the functions to be
linked to other modules any soon.

> 5, 7?
> 
> John

Peter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140620/2b456670/attachment-0001.sig>


More information about the libvir-list mailing list