Re: + sysct-mqueue-remove-the-binary-sysctl-numbers.patch added to -mm tree

From: Alexey Dobriyan
Date: Fri Aug 10 2007 - 05:56:23 EST


On Thu, Aug 09, 2007 at 06:41:54PM -0700, akpm@xxxxxxxxxxxxxxxxxxxx wrote:
> Subject: sysctl mqueue: remove the binary sysctl numbers
> From: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
>
> Because of a conflict with FS_INODE_NR none of the binary sysctl numbers use
> by mqueue, were available to user space. So just remove them.

OK with me. FS_INODE_NR was there first. :)

> --- a/ipc/mqueue.c~sysct-mqueue-remove-the-binary-sysctl-numbers
> +++ a/ipc/mqueue.c
> @@ -44,12 +44,6 @@
> #define STATE_PENDING 1
> #define STATE_READY 2
>
> -/* used by sysctl */
> -#define FS_MQUEUE 1
> -#define CTL_QUEUESMAX 2
> -#define CTL_MSGMAX 3
> -#define CTL_MSGSIZEMAX 4
> -
> /* default values */
> #define DFLT_QUEUESMAX 256 /* max number of message queues */
> #define DFLT_MSGMAX 10 /* max number of messages in each queue */
> @@ -1197,7 +1191,6 @@ static int msg_maxsize_limit_max = INT_M
>
> static ctl_table mq_sysctls[] = {
> {
> - .ctl_name = CTL_QUEUESMAX,
> .procname = "queues_max",
> .data = &queues_max,
> .maxlen = sizeof(int),
> @@ -1205,7 +1198,6 @@ static ctl_table mq_sysctls[] = {
> .proc_handler = &proc_dointvec,
> },
> {
> - .ctl_name = CTL_MSGMAX,
> .procname = "msg_max",
> .data = &msg_max,
> .maxlen = sizeof(int),
> @@ -1215,7 +1207,6 @@ static ctl_table mq_sysctls[] = {
> .extra2 = &msg_max_limit_max,
> },
> {
> - .ctl_name = CTL_MSGSIZEMAX,
> .procname = "msgsize_max",
> .data = &msgsize_max,
> .maxlen = sizeof(int),
> @@ -1229,7 +1220,6 @@ static ctl_table mq_sysctls[] = {
>
> static ctl_table mq_sysctl_dir[] = {
> {
> - .ctl_name = FS_MQUEUE,
> .procname = "mqueue",
> .mode = 0555,
> .child = mq_sysctls,

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