On Tue, 2016-05-17 at 09:19 +0200, Jan Kara wrote:
> On Tue 17-05-16 06:52:47, Verma, Vishal L wrote:
[...]
> >
> >
> > - err = dax_clear_sectors(inode->i_sb->s_bdev,
> > - le32_to_cpu(chain[depth-1].key) <<
> > - (inode->i_blkbits - 9),
> > - 1 << inode->i_blkbits);
> > + err = sb_issue_zeroout(inode->i_sb,
> > + le32_to_cpu(chain[depth-1].key),
> > count, GFP_NOFS);
> >
> > If so, I'll update my series tomorrow to include in both of these
> > changes.
> I'd prefer these two to stay separate commits (they are really
> independent). Since you already depend on other patches from the DAX
> cleanup series, just add this patch to the list of dependencies and
> base
> your change on that... Hmm?
Yes I agree. I'm preparing a branch that adds your fix, and modifying my
sb_issue_zeroout to use the same fix. I've verified that those tests
pass after both changes.
Thanks!
-Vishal
��.n��������+%������w��{.n�����{�����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
