lkml.org 
[lkml]   [2006]   [May]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Subjectmemcpy_toio on i386 using byte writes even when n%2==0
    From
    Date
    I'm working on a driver for a custom PCI card on the i386 architecture.
    The card uses a PLX9030 pci bridge to link an FPGA to the PCI bus using
    a 16 bit bus. I found that something broke when moving from 2.6.10 to
    2.6.17-rc4. In the driver, I use memcpy_toio to write 14 bytes to a
    memory region in the FPGA.

    To copy the 14 bytes, 2.6.10 does three 32 bit writes followed by one 16
    bit write. 2.6.10 does three 32 bit writes followed by two 8 bit write.

    The PLX9030 breaks the 32 bit writes into 16 bit writes for its local
    bus just fine. The problem is that my board doesn't handle byte
    enables. It was assumed that if all memory transfers were a multiple of
    2 bytes, then byte accesses wouldn't be used. This is no longer true in
    2.6.7-rc4.

    I've solved the problem by padding to 16 bytes, but should this be
    considered a bug in the kernel?

    Both kernels use __memcpy to implement memcpy_toio. Here is the
    relevent code from <asm-i386/string.h>

    The 2.6.10 version:

    static inline void * __memcpy(void * to, const void * from, size_t n)
    {
    int d0, d1, d2;
    __asm__ __volatile__(
    "rep ; movsl\n\t"
    "testb $2,%b4\n\t"
    "je 1f\n\t"
    "movsw\n"
    "1:\ttestb $1,%b4\n\t"
    "je 2f\n\t"
    "movsb\n"
    "2:"
    : "=&c" (d0), "=&D" (d1), "=&S" (d2)
    :"0" (n/4), "q" (n),"1" ((long) to),"2" ((long) from)
    : "memory");
    return (to);
    }

    The 2.6.17-rc4 version:

    static __always_inline void * __memcpy(void * to, const void * from,
    size_t n)
    {
    int d0, d1, d2;
    __asm__ __volatile__(
    "rep ; movsl\n\t"
    "movl %4,%%ecx\n\t"
    "andl $3,%%ecx\n\t"
    #if 1 /* want to pay 2 byte penalty for a chance to skip microcoded
    rep? */
    "jz 1f\n\t"
    #endif
    "rep ; movsb\n\t"
    "1:"
    : "=&c" (d0), "=&D" (d1), "=&S" (d2)
    : "0" (n/4), "g" (n), "1" ((long) to), "2" ((long) from)
    : "memory");
    return (to);
    }

    --
    Chris Lesiak
    chris.lesiak@licor.com

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-05-26 17:31    [W:0.023 / U:0.024 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site