Home » Mailing list archives » linux-kernel » 2007 » September » 9

Re: [-mm patch] unexport sys_{open,read}

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Andrew Morton <akpm@...>
To: Christoph Hellwig <hch@...>
Cc: Adrian Bunk <bunk@...>, <perex@...>, <linux-kernel@...>
Subject: Re: [-mm patch] unexport sys_{open,read}
Date: Sunday, September 9, 2007 - 5:59 pm

On Sun, 9 Sep 2007 21:39:20 +0100 Christoph Hellwig <hch@infradead.org> wrote:


Sure.  But I think it is better to give people some warning when we're
planning on breaking out-of-tree things.  I do occasionally receive reports
of "hey, the X driver which I get from Y doesn't work any more".  Often
it's open-source stuff, too.  I see no point in irritating our users more than
we need to.

If we're changing an API or removing a function then there's nothing we can
do, but in the case where we're simply deleting an export, it's exceedingly
easy for us to EXPORT_UNUSED_SYMBOL (and __deprecated_for_modules?) for a
few months.

Adrian knows this, yet he habitually sends zero-warning export-removal
patches and I habitually ignore them.  I guess we must both enjoy this or
something.
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[-mm patch] unexport sys_{open,read}, Adrian Bunk, (Sun Sep 9, 4:25 pm)
Re: [-mm patch] unexport sys_{open,read}, Christoph Hellwig, (Sun Sep 9, 4:39 pm)
Re: [-mm patch] unexport sys_{open,read}, David Miller, (Mon Sep 10, 8:18 am)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Tue Sep 18, 10:10 am)
Re: [-mm patch] unexport sys_{open,read}, Christoph Hellwig, (Mon Sep 10, 8:21 am)
Re: [-mm patch] unexport sys_{open,read}, Andrew Morton, (Sun Sep 9, 5:59 pm)
Re: [-mm patch] unexport sys_{open,read}, Christoph Hellwig, (Mon Sep 10, 5:08 am)
Re: [-mm patch] unexport sys_{open,read}, Andrew Morton, (Mon Sep 10, 5:23 am)
Re: [-mm patch] unexport sys_{open,read}, Arjan van de Ven, (Mon Sep 10, 11:14 am)
Re: [-mm patch] unexport sys_{open,read}, Dave Jones, (Tue Sep 25, 5:17 pm)
Re: [-mm patch] unexport sys_{open,read}, Al Viro, (Mon Sep 10, 8:43 am)
Re: [-mm patch] unexport sys_{open,read}, Andrew Morton, (Mon Sep 10, 1:25 pm)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Mon Sep 10, 3:58 pm)
Re: [-mm patch] unexport sys_{open,read}, Andrew Morton, (Mon Sep 10, 4:17 pm)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Mon Sep 10, 6:18 pm)
Re: [-mm patch] unexport sys_{open,read}, Rene Herman, (Mon Sep 10, 6:15 pm)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Mon Sep 10, 6:41 pm)
Re: [-mm patch] unexport sys_{open,read}, Rene Herman, (Mon Sep 10, 6:56 pm)
Re: [-mm patch] unexport sys_{open,read}, Alan Cox, (Mon Sep 10, 1:44 pm)
Re: [-mm patch] unexport sys_{open,read}, Andrew Morton, (Mon Sep 10, 1:54 pm)
Re: [-mm patch] unexport sys_{open,read}, Greg KH, (Thu Sep 13, 7:23 pm)
Re: [-mm patch] unexport sys_{open,read}, Alan Cox, (Mon Sep 10, 8:31 am)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Mon Sep 10, 8:03 am)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Sun Sep 9, 6:22 pm)
Re: [-mm patch] unexport sys_{open,read}, Arjan van de Ven, (Sun Sep 9, 6:41 pm)
Re: [-mm patch] unexport sys_{open,read}, Adrian Bunk, (Sun Sep 9, 7:18 pm)