On 04/22/2013 05:24 PM, Josh Durgin wrote:
> On 04/21/2013 02:54 PM, Alex Elder wrote:
>> (This patch and the next one are available in the
>> "review/wip-enable-layering" branch of the ceph-client
>> git respository.)
>>
>>
>> Now that we have most everything in place to support layered rbd
>> images, enable support for them in the kernel client. Issue a
>> warning to the log that the support is considered experimental
>> whenever a format 2 layered image is mapped.
>>
>> Note that we also have to claim to support the STRIPINGV2 feature,
>> due to a mistake in the way the rbd CLI set up those flags. This
>> feature can work if it has the right parameters, and safeguards
>> have been put in place to reject those images that do not have
>> compatible parameters.
>>
>> Signed-off-by: Alex Elder <[email protected]>
>> ---
>> drivers/block/rbd.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c
>> index d23cc8f..1ddbcdc 100644
>> --- a/drivers/block/rbd.c
>> +++ b/drivers/block/rbd.c
>> @@ -80,7 +80,7 @@
>>
>> /* Features supported by this (client software) implementation. */
>>
>> -#define RBD_FEATURES_SUPPORTED (0)
>> +#define RBD_FEATURES_SUPPORTED (RBD_FEATURES_ALL)
>>
>> /*
>> * An RBD device name will be "rbd#", where the "rbd" comes from
>> @@ -4723,6 +4723,8 @@ static int rbd_dev_v2_probe(struct rbd_device
>> *rbd_dev)
>> ret = rbd_dev_v2_parent_info(rbd_dev);
>> if (ret)
>> goto out_err;
>> + rbd_warn(rbd_dev, "WARNING: layered rbd image support "
>> + "is EXPERIMENTAL!");
>
> Maybe add "for the kernel rbd driver" or something, since librbd
> support isn't experimental. If you're removing this warning soon
> it doesn't matter though. Either way is fine with me.
I'll reword it slightly. Thanks a lot for all the
reviews, Josh.
-Alex
> Reviewed-by: Josh Durgin <[email protected]>
>
>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
