On Mon, 2014-04-28 at 12:05 +0000, Alexey Brodkin wrote:

> And in this situation IMHO the only safe solution could be in proper
> design of data layout. In other words we need to keep independent data
> blocks aligned to cache line.
> 
> And as you may see from "designware.h" buffer descriptor structure is
> aligned:

There's no point in taking all this care if you then go and flush
subfields, as the driver does, since they are not necessarily going to
have the required alignment. That was the entire point of this patch!

I'm going to do the roundup thing you asked for, even though it seems
like a pointless optimisation to me given the context.

> ==============
> struct dmamacdescr {
>       u32 txrx_status;
>       u32 dmamac_cntl;
>       void *dmamac_addr;
>       struct dmamacdescr *dmamac_next;
> } __aligned(ARCH_DMA_MINALIGN);
> ==============
> 
> Regards,
> Alexey
> 
> 
> 


-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to