Re: 2.4.19pre8aa2

Andrea Arcangeli (andrea@suse.de)
Sun, 5 May 2002 02:25:08 +0200


On Sat, May 04, 2002 at 04:02:35PM +0100, Christoph Hellwig wrote:
> On Sat, May 04, 2002 at 04:54:40PM +0200, Andrea Arcangeli wrote:
> > Only in 2.4.19pre8aa2: 00_comx-driver-compile-1
> >
> > Export proc_get_inode for kernel/drivers/net/wan/comx.o so
> > it can link as a module, noticed by Eyal Lebedinsky.
>
> Don't do this - proc_get_inode is static for a reason and doing this
> export in the SuSE tree for ages doesn't make it any better.
>
> The driver needs serious fixing instead.

I see the driver would better not use procfs that way, but as far I can
tell it's a clean-desing problem, not a pratical problem so I documented
it with a quite explicit name of the patch (and I don't recommend to
merge it in any monolithic tree out there were it could be forgotten, at
the very least it'd need a bold comment on it) and I'll backout as soon
as the driver is fixed. It seems Al started working on it on top of
2.4.5-ac16 on 22 Jun 2001 and after one year the proc_get_inode is still
there so I don't buy the argument that by not exporting it the driver
gets fixed faster, it just remains unusable as module it seems. the
important thing is that it doesn't get forgotten and the name of the
patch ensures that from my part.

Andrea
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/