Re: [PATCH] remove fullflush and nofullflush in IOMMU genericoption

[FUJITA Tomonori]

On Fri, 19 Sep 2008 19:30:04 +0200
Joerg Roedel <joerg.roedel@xxxxxxx> wrote:

> On Sat, Sep 20, 2008 at 02:09:21AM +0900, FUJITA Tomonori wrote:
> > 
> > Please keep it for AMD option for now. Please send a patch to make it
> > generic to other IOMMU people and give them a chance to discuss on
> > it.
> >
> 
> Btw, you already agreed with a generic iommu= parameter for lazy IO/TLB
> flushing"
> 
> 
>  > True. We should merge common parameters across IOMMUs into the
>  > iommu= parameter some time in the future, I think. It would also be the
>  > place for the IOMMU size parameter.
> 
>  Hmm, now is better than the future? I think that now you can add
>  something like 'disable_batching_flush' as a common parameter and
>  change AMD IOMMU to use it.
> 
> in http://lkml.org/lkml/2008/9/17/376
> 
> And since we already have a iommu=fullflush parameter it makes sense of
> make it generic.

I'm not against fullflush but we need to discuss it with other people
before making the change.

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