On Wed, Dec 2, 2009 at 12:30 AM, <m-karicheri2@xxxxxx> wrote: > From: Muralidharan Karicheri <m-karicheri2@xxxxxx> > > If a USERPTR address that is not aligned to page boundary is passed to the > videobuf_dma_contig_user_get() function, it saves a page aligned address to > the dma_handle. This is not correct. This issue is observed when using USERPTR > IO machism for buffer exchange. > > Signed-off-by: Muralidharan Karicheri <m-karicheri2@xxxxxx> Thanks for the patch. For non-page aligned user space pointers I agree that a fix is needed. Don't you think the while loop in videobuf_dma_contig_user_get() also needs to be adjusted to include the last page? I think the while loop checks one page too little in the non-aligned case today. Cheers, / magnus -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
