[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20061003050748.GJ3278@stusta.de>
Date: Tue, 3 Oct 2006 07:07:48 +0200
From: Adrian Bunk <bunk@...sta.de>
To: Andrew Morton <akpm@...l.org>
Cc: Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Judith Lebzelter <judith@...l.org>, linuxppc-dev@...abs.org,
Linux Kernel list <linux-kernel@...r.kernel.org>
Subject: Re: [2.6 patch] mark virt_to_bus/bus_to_virt as __deprecated on i386
On Mon, Oct 02, 2006 at 09:55:27PM -0700, Andrew Morton wrote:
> On Tue, 03 Oct 2006 14:49:39 +1000
> Benjamin Herrenschmidt <benh@...nel.crashing.org> wrote:
>
> > On Mon, 2006-10-02 at 21:44 -0700, Andrew Morton wrote:
> > > On Tue, 03 Oct 2006 14:37:25 +1000
> > > Benjamin Herrenschmidt <benh@...nel.crashing.org> wrote:
> > >
> > > > > > You might want to convince Andrew accepting my patch to make
> > > > > > virt_to_bus/bus_to_virt give compile warnings on i386 for making
> > > > > > people more aware of this problem...
> > > > > >...
> > > >
> > > > Andrew, is there any reason not to take that patch ?
> > >
> > > It generates lots of warnings from drivers which nobody does any work on.
> > >
> > > Net result: lots of new warnings, no fixed bugs.
> >
> > Are you sure the warnings won't cause somebody like Al to go through
> > them all and fix them ?
>
> The drivers simply don't link on some architectures, due to missing
> virt_to_bus/bus_to_virt. They aren't hard to find.
>...
As my patch description says, there had even been one bus_to_virt()
accidentally added in 2.6.17-rc1.
Shouldn't people become aware of it as early and as often as possible?
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
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