[lkml]   [2001]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] starfire reads irq before pci_enable_device.
    >>>>> "Manfred" == Manfred Spraul <> writes:

    Manfred> Ion Badulescu wrote:
    >> > > +#if defined(__ia64__) || defined(__alpha__) > > +#define
    >> PKT_SHOULD_COPY(pkt_len) 1 > > +#else > > +#define
    >> PKT_SHOULD_COPY(pkt_len) (pkt_len < rx_copybreak) > > +#endif >
    >> [snip]
    >> It's not *required* per se, as far as I know both the Alpha and
    >> IA64 have handlers for unaligned access traps. *However*, copying
    >> each packet is definitely better than taking an exception for each
    >> packet!

    Manfred> What about changing the default for rx_copybreak instead?
    Manfred> Set it to 1536 on ia64 and Alpha, 0 for the rest. tulip and
    Manfred> eepro100 use that aproach.

    Inefficient, my patch will make the unused code path disappear during
    compilation, what you suggest results in an extra branch and unused

    The eepro100 and tulip drivers really should be changed to use the
    same trick.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:13    [W:0.022 / U:10.892 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site