Magnus, Thanks for testing and approving the patch.
Mauro, Could you merge this bug fix? Murali Karicheri Software Design Engineer Texas Instruments Inc. Germantown, MD 20874 phone: 301-407-9583 email: [email protected] >-----Original Message----- >From: Magnus Damm [mailto:[email protected]] >Sent: Wednesday, December 09, 2009 8:00 AM >To: Karicheri, Muralidharan >Cc: [email protected] >Subject: Re: [PATCH - v1] V4L-Fix videobuf_dma_contig_user_get() for non- >aligned offsets > >On Wed, Dec 9, 2009 at 6:36 AM, <[email protected]> wrote: >> From: Muralidharan Karicheri <[email protected]> >> >> 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. >> >> Updates from last version:- >> >> Adding offset for size calculation as per comment from Magnus Damm. This >> ensures the last page is also included for checking if memory is >> contiguous. >> >> Signed-off-by: Muralidharan Karicheri <[email protected]> > >Hi Murali, > >I've spent some time testing this patch with the SuperH CEU driver in >USERPTR mode. My test case is based on capture.c with places a bunch >of QVGA frames directly after each other. The size of each QVGA frame >is not an even multiple of 4k page size, so some of the frames will >use a non-aligned start addresses. Currently the CEU driver page >aligns the size of each frame, but I'll fix that in an upcoming patch. >Thank you! > >Acked-by: Magnus Damm <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
