Christoph, On 12 August 2015 at 08:05, Christoph Hellwig <hch@xxxxxx> wrote: > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > --- > include/asm-generic/dma-mapping-common.h | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/include/asm-generic/dma-mapping-common.h b/include/asm-generic/dma-mapping-common.h > index 940d5ec..afc3eaf 100644 > --- a/include/asm-generic/dma-mapping-common.h > +++ b/include/asm-generic/dma-mapping-common.h > @@ -51,8 +51,10 @@ static inline int dma_map_sg_attrs(struct device *dev, struct scatterlist *sg, > int i, ents; > struct scatterlist *s; > > - for_each_sg(sg, s, nents, i) > - kmemcheck_mark_initialized(sg_virt(s), s->length); > + for_each_sg(sg, s, nents, i) { > + if (sg_has_page(s)) > + kmemcheck_mark_initialized(sg_virt(s), s->length); > + } Just a nitpick for the subject, it should say "kmemcheck" rather than "kmemleak" (different features ;)). -- Catalin