[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <12c511ca0804291328v2f0b87csd0f2cf3accc6ad00@mail.gmail.com>
Date: Tue, 29 Apr 2008 13:28:14 -0700
From: "Tony Luck" <tony.luck@...el.com>
To: Nadia.Derbey@...l.net
Cc: linux-kernel@...r.kernel.org, y-goto@...fujitsu.com,
akpm@...ux-foundation.org, linux-mm@...ck.org,
containers@...ts.linux-foundation.org, matthltc@...ibm.com,
cmm@...ibm.com
Subject: Re: [PATCH 1/8] Scaling msgmni to the amount of lowmem
On Mon, Feb 11, 2008 at 7:16 AM, <Nadia.Derbey@...l.net> wrote:
> Index: linux-2.6.24-mm1/ipc/msg.c
> ===================================================================
> --- linux-2.6.24-mm1.orig/ipc/msg.c 2008-02-07 15:02:29.000000000 +0100
> +++ linux-2.6.24-mm1/ipc/msg.c 2008-02-07 15:24:19.000000000 +0100
...
> +out_callback:
> +
> + printk(KERN_INFO "msgmni has been set to %d for ipc namespace %p\n",
> + ns->msg_ctlmni, ns);
> +}
This patch has now made its way to mainline. I can see how this printk
was really useful to you while developing this patch. But does it add
much value in a production system? It just looks like another piece of
clutter on the console to my uncontainerized eyes.
-Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists