On Fri, Mar 25, 2016 at 12:02 PM, NeilBrown <neilb@xxxxxxxx> wrote: > On Sun, Mar 06 2016, Sage Weil wrote: > >> Hi Linus, >> >> Please pull the following Ceph patch from >> >> git://git.kernel.org/pub/scm/linux/kernel/git/sage/ceph-client.git for-linus >> >> This is a final commit we missed to align the protocol compatibility with >> the feature bits. It decodes a few extra fields in two different messages >> and reports EIO when they are used (not yet supported). >> >> Thanks! >> sage >> >> >> ---------------------------------------------------------------- >> Yan, Zheng (1): >> ceph: initial CEPH_FEATURE_FS_FILE_LAYOUT_V2 support > > Just wondering, but was CEPH_FEATURE_FS_FILE_LAYOUT_V2 supposed to have > exactly the same value as CEPH_FEATURE_NEW_OSDOPREPLY_ENCODING (and > CEPH_FEATURE_CRUSH_TUNABLES5)?? > Because when I backported this patch (and many others) to some ancient > enterprise kernel, it caused mounts to fail. If it really is meant to > be the same value, then I must have some other backported issue to find > and fix. Yes, it's mean to be some value. For the mount failure, please make sure the MDS is compiled from the newest ceph code. Regards Yan, Zheng > > Thanks, > NeilBrown > > >> >> fs/ceph/addr.c | 4 ++++ >> fs/ceph/caps.c | 27 ++++++++++++++++++++++++--- >> fs/ceph/inode.c | 2 ++ >> fs/ceph/mds_client.c | 16 ++++++++++++++++ >> fs/ceph/mds_client.h | 1 + >> fs/ceph/super.h | 1 + >> include/linux/ceph/ceph_features.h | 1 + >> 7 files changed, 49 insertions(+), 3 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
