2010-12-09 04:07:55

by Namhyung Kim

[permalink] [raw]
Subject: [PATCH] alpha/osf_sys: remove unused MAX_SELECT_SECONDS

Remove the leftover from the commit 14e2acd86865 ("select:
fix alpha OSF wrapper").

Signed-off-by: Namhyung Kim <[email protected]>
Cc: Arjan van de Ven <[email protected]>
---
arch/alpha/kernel/osf_sys.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
index 547e8b84b2f7..fe698b5045e9 100644
--- a/arch/alpha/kernel/osf_sys.c
+++ b/arch/alpha/kernel/osf_sys.c
@@ -951,9 +951,6 @@ SYSCALL_DEFINE2(osf_utimes, const char __user *, filename,
return do_utimes(AT_FDCWD, filename, tvs ? tv : NULL, 0);
}

-#define MAX_SELECT_SECONDS \
- ((unsigned long) (MAX_SCHEDULE_TIMEOUT / HZ)-1)
-
SYSCALL_DEFINE5(osf_select, int, n, fd_set __user *, inp, fd_set __user *, outp,
fd_set __user *, exp, struct timeval32 __user *, tvp)
{
--
1.7.3.3.400.g93cef


2010-12-09 04:15:35

by Matt Turner

[permalink] [raw]
Subject: Re: [PATCH] alpha/osf_sys: remove unused MAX_SELECT_SECONDS

On Thu, Dec 9, 2010 at 4:07 AM, Namhyung Kim <[email protected]> wrote:
> Remove the leftover from the commit 14e2acd86865 ("select:
> fix alpha OSF wrapper").
>
> Signed-off-by: Namhyung Kim <[email protected]>
> Cc: Arjan van de Ven <[email protected]>
> ---
> ?arch/alpha/kernel/osf_sys.c | ? ?3 ---
> ?1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
> index 547e8b84b2f7..fe698b5045e9 100644
> --- a/arch/alpha/kernel/osf_sys.c
> +++ b/arch/alpha/kernel/osf_sys.c
> @@ -951,9 +951,6 @@ SYSCALL_DEFINE2(osf_utimes, const char __user *, filename,
> ? ? ? ?return do_utimes(AT_FDCWD, filename, tvs ? tv : NULL, 0);
> ?}
>
> -#define MAX_SELECT_SECONDS \
> - ? ? ? ((unsigned long) (MAX_SCHEDULE_TIMEOUT / HZ)-1)
> -
> ?SYSCALL_DEFINE5(osf_select, int, n, fd_set __user *, inp, fd_set __user *, outp,
> ? ? ? ? ? ? ? ?fd_set __user *, exp, struct timeval32 __user *, tvp)
> ?{
> --
> 1.7.3.3.400.g93cef

Thanks! I'll vacuum this up for the next merge window.

Matt