Re: [libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-16 Thread John Ferlan
Seems like a sane approach to me. Less if-statements and the code still works just fine. You can apply my patch and fix this afterwards? Or do you want a new patch from me? Wido I'll take care of it as a followup patch which I'll push shortly... John -- libvir-list mailing list

Re: [libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-16 Thread John Ferlan
On 07/14/2015 04:13 PM, Josh Durgin wrote: On 07/14/2015 12:42 PM, John Ferlan wrote: On 07/14/2015 04:15 AM, Wido den Hollander wrote: We used to look at the librbd code version and depending on that we would invoke rbd_create3() or rbd_create(). Since librbd version 0.67.9 we can

Re: [libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-16 Thread Wido den Hollander
On 16-07-15 18:28, John Ferlan wrote: On 07/14/2015 04:13 PM, Josh Durgin wrote: On 07/14/2015 12:42 PM, John Ferlan wrote: On 07/14/2015 04:15 AM, Wido den Hollander wrote: We used to look at the librbd code version and depending on that we would invoke rbd_create3() or

[libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-14 Thread Wido den Hollander
We used to look at the librbd code version and depending on that we would invoke rbd_create3() or rbd_create(). Since librbd version 0.67.9 we can however tell RBD that it should create rbd format 2 images even if we invoke rbd_create(). The less options we pass to librbd, the more we can lean

Re: [libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-14 Thread John Ferlan
On 07/14/2015 04:15 AM, Wido den Hollander wrote: We used to look at the librbd code version and depending on that we would invoke rbd_create3() or rbd_create(). Since librbd version 0.67.9 we can however tell RBD that it should create rbd format 2 images even if we invoke rbd_create().

Re: [libvirt] [PATCH] rbd: Use RBD format 2 by default when creating images.

2015-07-14 Thread Josh Durgin
On 07/14/2015 12:42 PM, John Ferlan wrote: On 07/14/2015 04:15 AM, Wido den Hollander wrote: We used to look at the librbd code version and depending on that we would invoke rbd_create3() or rbd_create(). Since librbd version 0.67.9 we can however tell RBD that it should create rbd format 2