[libvirt] [Patch] Libvirt - Fix locking for readonly devices

[David Weber]

Hi,

I'm currently working on getting sanlock into Debian/Ubuntu.
While testing, I noticed that I wasn't able to add a readonly 
or shared device: "internal error unsupported 
configuration: Readonly leases are not supported".

After looking into the source, it seems to be the following
situation:
- Libvirt passes every device to the sanlock plugin, even if
it is readonly
- The sanlock plugin rejects to add a lease for the readonly
device, returning an error so the machine starts to fail

The attached patch rejects passing readonly and shared devices 
to the lock-plugin so they shouldn't be a problem anymore.

Would this be an appropriated way to fix this problem? (I'm 
neither experienced in libvirt nor C)

What seems odd to me is that Fedora either doesn't hit this
or nobody has ever reported it.

I'm not member of this list so please CC me.

Thanks!

Cheers,
David
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libvirt_sanlock.patch
Type: application/octet-stream
Size: 548 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120508/40eb1f38/attachment-0001.obj>