On Fri, Mar 22 2013 at 4:50pm -0400, Darrick J. Wong <darrick.wong@xxxxxxxxxx> wrote: > On Fri, Mar 22, 2013 at 08:21:56PM +0000, Ben Hutchings wrote: > > On Fri, 2013-03-22 at 13:11 -0700, Darrick J. Wong wrote: > > > The new writethrough strategy for dm-cache issues a bio to the origin device, > > > remaps the bio to the cache device, and issues the bio to the cache device. > > > However, the block layer modifies bi_sector and bi_size, so we need to preserve > > > these or else nothing gets written to the cache (bi_size == 0). This fixes the > > > problem where someone writes a block through the cache, but a subsequent reread > > > (from the cache) returns old contents. > > > > > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > > --- > > [...] > > > > This is not the correct way to submit a change to stable. See > > Documentation/stable_kernel_rules.txt > > Frankly, I'm not sure why agk sent the 3.9-fixes pull request to -stable. 7 of > the 10 commits are for dm-cache, and dm-cache is a new feature for 3.9. I > probably could have dropped -stable from the cc: list when I started > complaining about bugs. :) Alasdair's pull mail was sent to stable because commit f046f89a99c ("dm thin: fix discard corruption") cc'd stable and was included in the pull request. -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel