Re: Re-4: [Patch] Libvirt - Fix locking for readonly devices

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

 



On 05/14/2012 03:53 AM, David Weber wrote:
>>
>> Thanks, your patch looks good but needs two further small additions
>> to the libvirt_sanlock.aug and test_libvirt_sanlock.aug files to
>> take account of the new config parameter.
> 
> Oops, Updated patch attached (tested with augparse)
> 
> David
> 
> 
> ----------------------------------------
> 
> commit 33678a8b2d294bebf327106d586d41c9b157174f
> Author: David Weber <wb@xxxxxxxxxxxx>
> Date:   Mon May 14 09:43:27 2012 +0200
> 
>     Add ignore param for readonly and shared disk in sanlock

ACK and pushed, after fixing up whitespace nits ('make syntax-check'
would have caught the trailing whitespace, but not the other one), and
adding you to AUTHORS (let me know if you prefer an alternate spelling).


> +    

Trailing whitespace.

> +    if (((flags & VIR_LOCK_MANAGER_RESOURCE_READONLY) ||
> +        (flags &VIR_LOCK_MANAGER_RESOURCE_SHARED)) &&

Space on both sides of binary operators.  Furthermore, you can simplify
this to: (flags & (VIR_LOCK_MANAGER_RESOURCE_READONLY |
VIR_LOCK_MANAGER_RESOURCE_SHARED)).

> +        (driver->ignoreReadonlyShared)) {

Spurious parenthesis.

> +
> +#
> +# Ignore readonly and shared disks as they aren't supportet yet

s/supportet/supported/

Is it sanlock, or libvirt, or a combination of both that don't yet
support shared resource locks?  I modified this as follows (and hope
that it was good enough, since I pushed):

# Enable this flag to have sanlock ignore readonly and shared disks.
# If disabled, then this rejects attempts to share resources until
# sanlock gains support for shared locks.
#
#ignore_readonly_and_shared_disks = 1

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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