xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH 33/59] sysctl: s390 move sysctl definitions to

To: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
Subject: [xfs-masters] Re: [PATCH 33/59] sysctl: s390 move sysctl definitions to sysctl.h
From: Kirill Korotaev <dev@xxxxx>
Date: Wed, 17 Jan 2007 20:23:22 +0300
Cc: "<Andrew Morton" <akpm@xxxxxxxx>, James.Bottomley@xxxxxxxxxxxx, linux-mips@xxxxxxxxxxxxxx, linux-parport@xxxxxxxxxxxxxxxxxxx, minyard@xxxxxxx, rtc-linux@xxxxxxxxxxxxxxxx, clemens@xxxxxxxxxx, heiko.carstens@xxxxxxxxxx, xfs@xxxxxxxxxxx, linuxppc-dev@xxxxxxxxxx, paulus@xxxxxxxxx, openipmi-developer@xxxxxxxxxxxxxxxxxxxxx, linux-390@xxxxxxxxxxxxx, schwidefsky@xxxxxxxxxx, tim@xxxxxxxxxxxx, codalist@xxxxxxxxxxxxxxxxxxxxxxxx, a.zummo@xxxxxxxxxxxx, tony.luck@xxxxxxxxx, vojtech@xxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, xfs-masters@xxxxxxxxxxx, linux-ntfs-dev@xxxxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxxxxxx, aia21@xxxxxxxxxx, aharkes@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, ralf@xxxxxxxxxxxxxx, mark.fasheh@xxxxxxxxxx, coda@xxxxxxxxxx, lethal@xxxxxxxxxxxx, kurt.hackel@xxxxxxxxxx, containers@xxxxxxxxxxxxxx, linux390@xxxxxxxxxx, philb@xxxxxxx, andrea@xxxxxxx, linuxsh-shmedia-dev@xxxxxxxxxxxxxxxxxxxxx, ak@xxxxxxx
In-reply-to: <11689656572714-git-send-email-ebiederm@xmission.com>
References: <m1ac0jc4no.fsf@ebiederm.dsl.xmission.com> <11689656572714-git-send-email-ebiederm@xmission.com>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.13) Gecko/20060417
IDs not sorted in enum. see below.

> From: Eric W. Biederman <ebiederm@xxxxxxxxxxxx> - unquoted
> 
> We need to have the the definition of all top level sysctl
> directories registers in sysctl.h so we don't conflict by
> accident and cause abi problems.
> 
> Signed-off-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
> ---
>  arch/s390/appldata/appldata.h |    3 +--
>  arch/s390/kernel/debug.c      |    1 -
>  arch/s390/mm/cmm.c            |    4 ----
>  include/linux/sysctl.h        |    7 +++++++
>  4 files changed, 8 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/s390/appldata/appldata.h b/arch/s390/appldata/appldata.h
> index 0429481..4069b81 100644
> --- a/arch/s390/appldata/appldata.h
> +++ b/arch/s390/appldata/appldata.h
> @@ -21,8 +21,7 @@
>  #define APPLDATA_RECORD_NET_SUM_ID   0x03    /* must be < 256 !     */
>  #define APPLDATA_RECORD_PROC_ID              0x04
>  
> -#define CTL_APPLDATA                 2120    /* sysctl IDs, must be unique */
> -#define CTL_APPLDATA_TIMER   2121
> +#define CTL_APPLDATA_TIMER   2121    /* sysctl IDs, must be unique */
>  #define CTL_APPLDATA_INTERVAL        2122
>  #define CTL_APPLDATA_MEM     2123
>  #define CTL_APPLDATA_OS              2124
> diff --git a/arch/s390/kernel/debug.c b/arch/s390/kernel/debug.c
> index bb57bc0..c81f8e5 100644
> --- a/arch/s390/kernel/debug.c
> +++ b/arch/s390/kernel/debug.c
> @@ -852,7 +852,6 @@ debug_finish_entry(debug_info_t * id, debug_entry_t* 
> active, int level,
>  static int debug_stoppable=1;
>  static int debug_active=1;
>  
> -#define CTL_S390DBF 5677
>  #define CTL_S390DBF_STOPPABLE 5678
>  #define CTL_S390DBF_ACTIVE 5679
>  
> diff --git a/arch/s390/mm/cmm.c b/arch/s390/mm/cmm.c
> index 607f50e..df733d5 100644
> --- a/arch/s390/mm/cmm.c
> +++ b/arch/s390/mm/cmm.c
> @@ -256,10 +256,6 @@ cmm_skip_blanks(char *cp, char **endp)
>  }
>  
>  #ifdef CONFIG_CMM_PROC
> -/* These will someday get removed. */
> -#define VM_CMM_PAGES         1111
> -#define VM_CMM_TIMED_PAGES   1112
> -#define VM_CMM_TIMEOUT               1113
>  
>  static struct ctl_table cmm_table[];
>  
> diff --git a/include/linux/sysctl.h b/include/linux/sysctl.h
> index 71c16b4..56d0161 100644
> --- a/include/linux/sysctl.h
> +++ b/include/linux/sysctl.h
> @@ -73,6 +73,8 @@ enum
>       CTL_SUNRPC=7249,        /* sunrpc debug */
>       CTL_PM=9899,            /* frv power management */
>       CTL_FRV=9898,           /* frv specific sysctls */
> +     CTL_S390DBF=5677,       /* s390 debug */
> +     CTL_APPLDATA=2120,      /* s390 appldata */
<<<< not sorted by ID? imho should be sorted. otherwise can'be unnotied when 
inserted above.

>  };
>  
>  /* CTL_BUS names: */
> @@ -205,6 +207,11 @@ enum
>       VM_PANIC_ON_OOM=33,     /* panic at out-of-memory */
>       VM_VDSO_ENABLED=34,     /* map VDSO into new processes? */
>       VM_MIN_SLAB=35,          /* Percent pages ignored by zone reclaim */
> +
> +     /* s390 vm cmm sysctls */
> +     VM_CMM_PAGES=1111,
> +     VM_CMM_TIMED_PAGES=1112,
> +     VM_CMM_TIMEOUT=1113,
>  };
>  
>  


<Prev in Thread] Current Thread [Next in Thread>