Re: [PATCH] bio.c: reduce verbosity at boot

[Jeff Garzik]

Matthew Wilcox wrote:
> Linux is really far too verbose at boot time.  I don't think these messages
> add anything to either the end user experience or debug ability.
>
> Index: fs/bio.c
> ===================================================================
> RCS file: /var/cvs/linux-2.6/fs/bio.c,v
> retrieving revision 1.2
> diff -u -p -r1.2 bio.c
> --- fs/bio.c	29 Jul 2003 17:25:49 -0000	1.2
> +++ fs/bio.c	21 Aug 2003 14:58:40 -0000
> @@ -793,10 +793,6 @@ static void __init biovec_init_pools(voi
>  					mempool_free_slab, bp->slab);
>  		if (!bp->pool)
>  			panic("biovec: can't init mempool\n");
> -
> -		printk("biovec pool[%d]: %3d bvecs: %3d entries (%d bytes)\n",
> -						i, bp->nr_vecs, pool_entries,
> -						size);


Although I agree with your "too verbose" sentiment above, I think the 
removing the messages outright might not serve the best interests the 
developer.  Since even KERN_DEBUG still spams dmesg, in these situations 
I usually change these type of messages to be conditionally printed iff 
a debug macro is enabled.

As a tangent, the huge x86 IO-APIC verbosity really bugs me, too, and 
often is the direct cause of useful early printk messages being lost 
before boot is even complete.

	Jeff



-
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/