[libvirt] [PATCH 2/3] Create virFileIsOnNetworkShare utility function

Laine Stump laine at laine.org
Fri Jun 25 13:09:42 UTC 2010


>
> On Fri, Jun 25, 2010 at 07:42:14AM -0400, Laine Stump wrote:
>    
>> This code was previously used inline during qemudDomainSaveFlag. This
>> patch moves it into a utility function in preparation for it to be
>> used elsewhere.
>>      
> How about just switching to use virStorageFileIsSharedFS() ?
>    

Ah! I either hadn't noticed that function go in, or had forgotten about it.

One problem I see is that virStorageFileIsSharedFS doesn't traverse back 
up the path elements looking for a partial path it can stat. This means 
it will fail if the file in question (or any of its ancestor 
directories) isn't stat'able by root.

Any objections to me changing virStorageFileIsSharedFS to behave like that?




More information about the libvir-list mailing list