[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cfd18e0f0805260338o6d95c645oef4cfe10cca971a1@mail.gmail.com>
Date: Mon, 26 May 2008 12:38:43 +0200
From: "Michael Kerrisk" <mtk.manpages@...glemail.com>
To: Nadia.Derbey@...l.net
Cc: akpm@...ux-foundation.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Restore MSGPOOL original value
On Mon, May 26, 2008 at 11:17 AM, <Nadia.Derbey@...l.net> wrote:
> Hi,
>
> When posting:
> [PATCH 1/8] Scaling msgmni to the amount of lowmem
> (see http://article.gmane.org/gmane.linux.kernel/637849/), I have changed the
> MSGPOOL value to make it fit what is said in the man pages (i.e. a size in
> bytes).
>
> But Michael Kerrisk rightly complained that this change could affect the ABI.
> So I'm posting this patch to make MSGPOOL expressed back in Kbytes.
> Michael, on his side, has fixed the man page.
>
> This patch applies to 2.6.26-rc2-mm1.
>
> Signed-off-by: Nadia Derbey <Nadia.Derbey@...l.net>
>
> ---
> include/linux/msg.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> Index: linux-2.6.26-rc2-mm1/include/linux/msg.h
> ===================================================================
> --- linux-2.6.26-rc2-mm1.orig/include/linux/msg.h 2008-05-26 11:06:53.000000000 +0200
> +++ linux-2.6.26-rc2-mm1/include/linux/msg.h 2008-05-26 11:18:34.000000000 +0200
> @@ -64,11 +64,11 @@ struct msginfo {
> #define MSGMNB 16384 /* <= INT_MAX */ /* default max size of a message queue */
>
> /* unused */
> -#define MSGPOOL (MSGMNI * MSGMNB) /* size in bytes of message pool */
> +#define MSGPOOL (MSGMNI * MSGMNB / 1024) /* size in kbytes of message pool */
> #define MSGTQL MSGMNB /* number of system message headers */
> #define MSGMAP MSGMNB /* number of entries in message map */
> #define MSGSSZ 16 /* message segment size */
> -#define __MSGSEG (MSGPOOL / MSGSSZ) /* max no. of segments */
> +#define __MSGSEG ((MSGPOOL * 1024) / MSGSSZ) /* max no. of segments */
> #define MSGSEG (__MSGSEG <= 0xffff ? __MSGSEG : 0xffff)
>
> #ifdef __KERNEL__
Acked-by: Michael Kerrisk <mtk.manpages@...il.com>
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Found a bug? http://www.kernel.org/doc/man-pages/reporting_bugs.html
--
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