On Fri, Jan 29, 2016 at 6:07 PM, Ilya Dryomov <idryomov@xxxxxxxxx> wrote: > On Fri, Jan 29, 2016 at 5:09 PM, Sage Weil <sweil@xxxxxxxxxx> wrote: >> Hi Zheng, >> >> We have one feature already merged to ceph.git, CRUSH_TUNABLES5, that's a >> pretty trivial change to crush/mapper.c that we can add to the kernel. >> >> But there are two more coming: >> >> OSDOPREPLY_ENCODING makes a small change to the OSDOpReply encoding for >> performance reasons. Here is the PR >> >> https://github.com/ceph/ceph/pull/7386 >> >> The other is the file layout changes, which are much more invovled: >> >> https://github.com/ceph/ceph/pull/7098 >> >> Ideally we'd use the same feature bit for all of these, but it means they >> all need to upstream into Linux together. Do you see a problem with that? >> Assuming we get it all ready to go before the next merge window.. >> >> The tricky bit is we really need to know that we'll be able to do it >> together before we cut jewel too and commit to the overlapping feature >> bits. > > In the worst case, we could advertise support for the new bit for rbd > and not for cephfs - I don't think we ever get/put ceph file layout > from/to the wire in the rbd case. > > As for the two easy ones (TUNABLES5 in particular), I think we should > try and get them in ASAP into 4.5 and then finalize the feature bit > arrangement closer to the end of the cycle, if necessary. Otherwise > we'll have jewel and no kernel supporting its default tunables. To that end, I've pulled crush changes into the kernel client, aiming for -rc3. Thanks, Ilya -- 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
