Re: [PATCH] Delete obsolete RAW driver feature.

From: Willy Tarreau
Date: Wed Mar 21 2007 - 19:25:14 EST


On Wed, Mar 21, 2007 at 04:13:41PM -0700, Andrew Morton wrote:
> On Thu, 22 Mar 2007 00:10:50 +0000
> Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
>
> > > > This is a user visible API it is therefore not elligable for removal
> > > > anyway
> > >
> > > mutter.
> > >
> > > We've given people years of notice and _some_ applications have converted
> > > over to open("/dev/sda1", O_DIRECT), as they should.
> >
> > Linus said we don't break old applications by removing APIs. People
> > believe him.
> >
>
> That's just dogma, and not credible dogma at that. We regularly remove
> features and we have processes for this.
>
> There is no reason why we cannot, with sufficient effort, remove this
> driver at some time in the future.

Then a printk() on every open() should be enough. We've all been seeing
"Warning: tcpdump uses obsolete AF_PACKET"... and it finally disappeared.
It's a slow but necessary process IMHO. I agree with Alan that it's not
nice for the users to have sudden application errors without enough earlier
notification (not everybody reads sources for removal notifications).

Regards,
Willy

-
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/