On Mon, May 16, 2011 at 4:43 PM, Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> wrote: > On Mon, May 16, 2011 at 10:51:10PM +0100, Ben Dooks wrote: >> On Mon, May 16, 2011 at 03:46:08PM -0600, Grant Likely wrote: >> > + >> > + /* >> > + * DMA buffer, with padding to give it its own cache line so that >> > + * the DMA streaming works on non-coherent architectures. >> > + * Question: Is this the proper pattern, and is this really necessary? >> > + */ >> > + uint8_t pad1[L1_CACHE_BYTES]; >> > + uint8_t buf[6]; >> > + uint8_t pad2[L1_CACHE_BYTES]; >> > +}; >> >> I think there's an attribute to do this, starting with an __ defined >> in the kernel. >> > > Yes, it is called "____cacheline_aligned". Fixed and tested, thanks. g. -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
