diff mbox

[23/33] libceph: enable OSDMAP_ENC feature bit

Message ID 1395944299-21970-24-git-send-email-ilya.dryomov@inktank.com (mailing list archive)
State New, archived
Headers show

Commit Message

Ilya Dryomov March 27, 2014, 6:18 p.m. UTC 
Announce our support for "new" osdmap enconding.

Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
---
 include/linux/ceph/ceph_features.h |    1 +
 1 file changed, 1 insertion(+)

Comments

Alex Elder March 27, 2014, 8:32 p.m. UTC | #1 
On 03/27/2014 01:18 PM, Ilya Dryomov wrote:
> Announce our support for "new" osdmap enconding.

Looks OK to me.  Isn't there a version of this OSD
map encoding?  Maybe there'll be a "newer" one someday?

Reviewed-by: Alex Elder <elder@linaro.org>

> Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
> ---
>  include/linux/ceph/ceph_features.h |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/ceph/ceph_features.h b/include/linux/ceph/ceph_features.h
> index 77c097fe9ea9..7a4cab50b2cd 100644
> --- a/include/linux/ceph/ceph_features.h
> +++ b/include/linux/ceph/ceph_features.h
> @@ -90,6 +90,7 @@ static inline u64 ceph_sanitize_features(u64 features)
>  	 CEPH_FEATURE_OSD_CACHEPOOL |		\
>  	 CEPH_FEATURE_CRUSH_V2 |		\
>  	 CEPH_FEATURE_EXPORT_PEER |		\
> +	 CEPH_FEATURE_OSDMAP_ENC |		\
>  	 CEPH_FEATURE_CRUSH_TUNABLES3)
>  
>  #define CEPH_FEATURES_REQUIRED_DEFAULT   \
> 

--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Ilya Dryomov March 28, 2014, 3:01 p.m. UTC | #2 
On Thu, Mar 27, 2014 at 10:32 PM, Alex Elder <elder@ieee.org> wrote:
> On 03/27/2014 01:18 PM, Ilya Dryomov wrote:
>> Announce our support for "new" osdmap enconding.
>
> Looks OK to me.  Isn't there a version of this OSD
> map encoding?  Maybe there'll be a "newer" one someday?

Reworded to "Announce our support for "new" (v7 - split and separately
versioned client and osd sections) osdmap enconding."

Thanks,

                Ilya
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/include/linux/ceph/ceph_features.h b/include/linux/ceph/ceph_features.h
index 77c097fe9ea9..7a4cab50b2cd 100644
--- a/include/linux/ceph/ceph_features.h
+++ b/include/linux/ceph/ceph_features.h
@@ -90,6 +90,7 @@  static inline u64 ceph_sanitize_features(u64 features)
 	 CEPH_FEATURE_OSD_CACHEPOOL |		\
 	 CEPH_FEATURE_CRUSH_V2 |		\
 	 CEPH_FEATURE_EXPORT_PEER |		\
+	 CEPH_FEATURE_OSDMAP_ENC |		\
 	 CEPH_FEATURE_CRUSH_TUNABLES3)
 
 #define CEPH_FEATURES_REQUIRED_DEFAULT   \