lkml.org 
[lkml]   [2010]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: USB mass storage and ARM cache coherency
    [added another __bzero coherency crash victim, see
    http://lkml.org/lkml/2008/6/9/14 ]

    On Tue, Feb 02, 2010 at 03:52:19PM +0100, Oliver Neukum wrote:
    > Am Dienstag, 2. Februar 2010 15:42:49 schrieb Clemens Ladisch:
    > > > Or... usb-audio? I should have verified that it is using bulk endpoints
    > > > (and thus the patch applies to my case).
    > > > usb-audio probably uses isochronous transfers, thus that would be
    > > > an obvious reason why the patch didn't work for me.
    > >
    > > snd-usb-audio indeed uses isochronous transfers, but those buffers are
    > > never mapped into user space. The intermediate vmalloc()ed buffer is,
    > > however, and there was a bugfix for this recently. Do you have these
    > > patches in your tree?
    >
    > Now that I think about it, several video drivers do map it to user space.

    OK, then the urb loop needs to also handle isochronous pipes,
    and IMHO we should have a generic helper for this instead of open-coding
    it, since it probably needs to be done in a couple affected HCDs
    (and, most importantly, only on affected architectures - which the helper
    could handle transparently).

    Clemens: no, both of these patches haven't been applied (yet!!),
    many thanks for the notification!

    Will apply both patches and the isochronous addition, hopefully that
    improves things (will be painful to check which of these things managed to
    fix it - in case it does! -, though). Nope, will apply step by step,
    both patches, then isochronous as a last resort.

    Andreas Mohr


    \
     
     \ /
      Last update: 2010-02-02 16:13    [W:4.343 / U:0.208 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site