Re: [PATCH] make cramfs look less hostile

From: Jörn Engel (joern@wohnheim.fh-wedel.de)
Date: Sun Jun 15 2003 - 12:39:26 EST


On Sun, 15 June 2003 18:26:42 +0100, Christoph Hellwig wrote:
> On Sun, Jun 15, 2003 at 06:05:24PM +0200, Jörn Engel wrote:
> >
> > This thing has been biting me now and again. "cramfs: wrong magic\n"
> > looks like an error condition to most people and thus creates bug
> > reports. But there is no bug per se in having cramfs support in the
> > kernel and booting from a jffs2 rootfs. So instead of teaching the
> > users over and over, how about this little one-liner?
>
> Umm, cramfs_fill_super has a silent parameter that's true for
> probing the root filesystem. I'd suggest disabling the printk
> completly if it's set.

Good idea, but only at first glance. cramfs_fill_super() always gets
called with silent=1. So if "(!silent) printk(...);" is functionally
equivalent to ";".

Jörn

-- 
The competent programmer is fully aware of the strictly limited size of
his own skull; therefore he approaches the programming task in full
humility, and among other things he avoids clever tricks like the plague. 
-- Edsger W. Dijkstra
-
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/



This archive was generated by hypermail 2b29 : Sun Jun 15 2003 - 22:00:42 EST