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

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: Dave Jones <davej@...>
To: Arjan van de Ven <arjan@...>
Cc: Andrew Morton <akpm@...>, Christoph Hellwig <hch@...>, Adrian Bunk <bunk@...>, <perex@...>, <linux-kernel@...>
Subject: Re: [-mm patch] unexport sys_{open,read}
Date: Tuesday, September 25, 2007 - 5:17 pm

On Mon, Sep 10, 2007 at 04:14:56PM +0100, Arjan van de Ven wrote:
 
 > Maybe I'm too pessimistic in my assumption that external open module
 > writers don't actually follow mainline closely; and maybe part of me
 > would love for them to follow it closer, so close that they would even
 > consider submitting their driver for it. But sadly from experience...
 > warnings dont' work. Only when things break hard people notice.

We've both done the same job, so it's unsurprising that we share the
same view on this.  A number of times I've had out of tree module
authors complaining to _me_ because "Fedora fucked up my module, it
works fine in $otherdistro"  (ignoring the fact that $otherdistro
is a point release or more behind Fedora).

There's definitly a culture-gap between some out-of-tree module
authors and people who intend from day 1 to get their code merged
into mainline.  The former don't know what happens in mainline
until several distros start moving onto newer kernels, so no amount
of warnings is going to help these people.

	Dave

-- 
http://www.codemonkey.org.uk
-
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)