Re: [PATCH] fs: Restore files_lock and set_fs_root exports

[Andrew Morton]

Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
>
> On Fri, Jan 07, 2005 at 02:00:34PM -0800, Andrew Morton wrote:
> > No, I'd say that unexports are different.  They can clearly break existing
> > code and so should only be undertaken with caution, and with lengthy notice
> > if possible.
> 
> As mentioned before we only unexported symbols were we were pretty clear
> that all users of it are indeep utterly broken.  I got about a dozend
> replies for this patches, and for more than half of it where the reporter
> was either the author or the module was opensource I could help them to
> actually fix their code.  In this case the code is far more broken than
> the others, but we've even been trying to help them fix their code for
> more than a year, but IBM folks have been constanly refusing.

They didn't fix their code because it worked - no reason to do so.

Telling people "this is going away in 12 months" gives them reason to fix
the code.  And a reasonable amount of time to do so and to distribute the
new version.

> > The cost to us of maintaining those two lines of code for a year is
> > basically zero.
> 
> But as long as IBM people don't fix their %$^%! they'll continue annoying
> us and the Distibutors about adding more and more hacks for it,

Maybe, maybe not.  But is it appropriate for us to be trying to control
someone else's internal activities via alterations to the kernel?

> and other
> people will write similarly crappy code using it again and making the
> removal even more difficult. 

I doubt if people would be silly enough to use a deprecated export in new
code after the export has been scheduled for removal.  If they do then yes,
sorry, we have to draw the line somewhere.

You still have not demonstrated any benefit to any party from not delaying
the removal of these two exports.
-
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/