lkml.org 
[lkml]   [2008]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH] remove fullflush and nofullflush in IOMMU generic option
    From
    On Mon, 22 Sep 2008 18:23:14 +0200
    Joerg Roedel <joerg.roedel@amd.com> wrote:

    > On Tue, Sep 23, 2008 at 12:25:24AM +0900, FUJITA Tomonori wrote:
    > >
    > > I'm not sure we can because he has a quite different opinion on this,
    > > he thinks that the IOMMU parameters are unimportant and minor issue.
    > >
    >
    > Stop your personal attacks and come down. I've never said that iommu
    > parameters are unimportant. The _move_ of the iommu=[no]fullflush to
    > generic code is a minor issue in my opinion. For the user it changes
    > nothing. There was an iommu=fullflush parameter before and now there is
    > also an iommu=fullflush parameter. The only difference is that it now
    > affects AMD IOMMU too.

    I have no intention to attack you. I just described what you did.

    You moved two IOMMU parameters to generic code, they interact with all
    other IOMMUs specific parameters.

    If you add parameters to AMD IOMMU, they interact with only AMD IOMMU.

    That's a huge difference. You might think that the two parameters are
    minor but it would turn out that they are not. What we know now is
    that the changes you make affect all the IOMMUs. It's not a minor
    issue.


    > > I'm against adding new generic IOMMU parameters without serious
    > > consideration. Joerg thinks that we can clean up the parameters like
    > > adding some for 2.6.28, then add some for 2.6.X, then rename some, in
    > > an ad-hoc way.
    >
    > Again. Stop your personal attacks.

    You added new generic fullflush and nofullflush parameters and
    suggested to remove nofullflush parameter without discussing any plan
    about further IOMMU parameter changes. We don't even start to discuss
    a plan to how to clean up the IOMMU parameters.

    I just described what you did.


    The patch that I sent doesn't make any changes for you. GART works as
    before and AMD IOMMU takes a new parameter, fullflush, as you want. It
    does the exact same thing that the first version of your patchset for
    2.6.28.

    So please let me apply it then we can stop this discussion.


    \
     
     \ /
      Last update: 2008-09-22 18:53    [W:2.695 / U:0.348 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site