svn commit: r196380 - head/sys/dev/usb
Rafal Jaworowski
raj at semihalf.com
Wed Aug 19 15:55:51 UTC 2009
On 2009-08-19, at 17:30, Sam Leffler wrote:
> Rafal Jaworowski wrote:
>> Author: raj
>> Date: Wed Aug 19 14:39:08 2009
>> New Revision: 196380
>> URL: http://svn.freebsd.org/changeset/base/196380
>> Log:
>> Fix USB cache sync operations for platforms with non-coherent DMA.
>> - usb_pc_cpu_invalidate() is called between [consecutive] reads
>> from a device,
>> so a sequence of BUS_DMASYNC_POSTREAD and _PREREAD should be
>> used. Note we
>> cannot use or'ed shorthand ( _POSTREAD | _PREREAD) for
>> BUS_DMASYNC flags, as
>> the low level bus dma sync operation is implementation dependent
>> and we
>> cannot assume the required order of operations to be guaranteed.
>> - usb_pc_cpu_flush() is called before writing to a device, so
>> BUS_DMASYNC_PREWRITE should be used.
>> Submitted by: Grzegorz Bernacki
>> Reviewed by: HPS, arm@, usb@ ML
>> Tested by: HPS, Mike Tancsa
>> Approved by: re (kib)
>> Obtained from: Semihalf
>
> Is this different from the patch I tested on Gateworks 2358 boards
> which didn't completely resolve problems?
Hm, not sure what patch you have tested with GW. There was an initial
workaround for this problem from late June time frame, and this commit
is a refined fix identical to the patch posted 05 Aug to arm at . There
were other ARM patches in the meantime involving cache sync, but they
were pmap-related.
Rafal
More information about the svn-src-head
mailing list