On Thu, May 05, 2016 at 11:43:38AM +0200, David Sterba wrote: > From: Zhao Lei <zhaolei@xxxxxxxxxxxxxx> > > reada_zone->end is end pos of segment: > end = start + cache->key.offset - 1; > > So we need to use "<=" in condition to judge is a pos in the > segment. > > The problem happened rearly, because logical pos rarely pointed > to last 4k of a blockgroup, but we need to fix it to make code > right in logic. > > Signed-off-by: Zhao Lei <zhaolei@xxxxxxxxxxxxxx> > Signed-off-by: David Sterba <dsterba@xxxxxxxx> > --- > fs/btrfs/reada.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Can you put the number of the commit in Linus's tree _in_ the patch itself somewhere, either in the format that I use for when I do commits to the stable branch (first line of the changelog text), or in the signed-off-by: area like git cherry-pick does? This way, as a list in patch 0/X separate from the commits and even the descriptions, is pretty impossible to easily match up. Can you redo this series and resend it with that fix? 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