On Mon, Nov 25, 2013 at 05:51:16PM +0100, David Sterba wrote: > On Fri, Sep 27, 2013 at 09:37:00AM -0400, Josef Bacik wrote: > > A user reported a problem where they were getting csum errors when running a > > balance and running systemd's journal. This is because systemd is awesome and > > fallocate()'s its log space and writes into it. Unfortunately we assume that > > when we read in all the csums for an extent that they are sequential starting at > > the bytenr we care about. This obviously isn't the case for prealloc extents, > > where we could have written to the middle of the prealloc extent only, which > > means the csum would be for the bytenr in the middle of our range and not the > > front of our range. Fix this by offsetting the new bytenr we are logging to > > based on the original bytenr the csum was for. With this patch I no longer see > > the csum errors I was seeing. Thanks, > > > > Cc: stable@xxxxxxxxxxxxxxx > > The patch had the right CC but I don't see it in the mail's CC list (now > added by me). I'm afraid that this never reached stable and explains why > the patch did not end up in 3.12.1. No, it made it to my list, I was waiting for 3.13-rc1 to come out with this patch in it before I could queue it up. Don't worry, it's not lost. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html