These look good.. but where is the crc bug actually fixed? Not seeing that patch... Reviewed-by: Sage Weil <sage@xxxxxxxxxxxx> On Tue, 28 Feb 2012, Alex Elder wrote: > Rename a few variables so it's clearer which indicate that a > CRC should be computed and which hold an actual CRC value. > Separate CRC calculation from byte swapping to improve > readability. And move some code that runs on only on > the last pass through a couple of loops outside the loop > body. > > -Alex > -- > 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 > > -- 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