lkml.org 
[lkml]   [2001]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: alpha iommu fixes

    Andi Kleen writes:
    > On Mon, May 21, 2001 at 03:34:50AM -0700, David S. Miller wrote:
    > > egrep illegal_highdma net/core/dev.c
    >
    > There is just no portable way for the driver to figure out if it should
    > set this flag or not. e.g. acenic.c gets it wrong: it is unconditionally
    > set even on IA32. Currently it requires an architecture ifdef to set properly.

    Well, certainly, this could perhaps be a bug in the Acenic driver.
    It should check if DAC cycles can be used on the platform, for example.

    But please, let's get back to the original problem though.

    The original claim is that the situation was not handled at all. All
    I'm trying to say is simply that the net stack does check via
    illegal_highdma() the condition you stated was not being checked at
    all. To me it sounded like you were claiming that HIGHMEM pages went
    totally unchecked through device transmit, and that is totally untrue.

    If you were trying to point out the problem with what the Acenic
    driver is doind, just state that next time ok? :-)

    There is no question that what Acenic is doing with ifdefs needs a
    clean portable solution. This will be part of the 64-bit DAC API
    interfaces (whenever those become really necessary, I simply don't
    see the need right now).

    Plainly, I'm going to be highly reluctant to make changes to the PCI
    dma API in 2.4.x It is already hard enough to get all the PCI drivers
    in line and using it. Suggesting this kind of change is similar to
    saying "let's change the arguments to request_irq()". We would do it
    to fix a true "people actually hit this" kind of bug, of course. Yet
    we would avoid it at all possible costs due to the disruption this
    would cause.

    I'm not trying to be a big bad guy about this. What I'm trying to do
    is make sure at least one person (me :-) is thinking about the
    ramifications any such change has on all current drivers which use
    these interfaces already. And also, to port maintainers...

    Later,
    David S. Miller
    davem@redhat.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: 2005-03-22 12:53    [W:2.278 / U:2.184 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site