[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080919223951.GC11410@amd.com>
Date: Sat, 20 Sep 2008 00:39:51 +0200
From: Joerg Roedel <joerg.roedel@....com>
To: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
CC: mingo@...e.hu, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] remove fullflush and nofullflush in IOMMU generic option
On Sat, Sep 20, 2008 at 07:18:11AM +0900, FUJITA Tomonori wrote:
> I don't know it's acceptable if removing the exported interface even
> if the maintainer of it wants to remove it and it's totally
> meaningless.
>
>
> > reverting is the wrong way. For AMD IOMMU I want to use the
> > iommu=fullflush way because I want to reuse a parameter thats already
> > there. Thats why I am against your reverting patch.
> > So now I stop repeating my points again and again. EOD.
>
> I understood you want to use iommu=fullflush but you can't touch the
> generic code without any discussion.
>
> And even if everyone is happy about the change, it's much better to
> start from scratch rather than try to fix the things done in the wrong
> way.
Just sleep a night about the discussions for this issue and count then
how often you changed your opinions and points. It all started with a
objection from you against 'nofullflush' and ended in that you want to
explain me the develpment process. And then think again if this
unimportant minor change was worth all that ridiculous flaming.
Good night,
Joerg
--
| AMD Saxony Limited Liability Company & Co. KG
Operating | Wilschdorfer Landstr. 101, 01109 Dresden, Germany
System | Register Court Dresden: HRA 4896
Research | General Partner authorized to represent:
Center | AMD Saxony LLC (Wilmington, Delaware, US)
| General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists